プロジェクト

全般

プロフィール

Bug(バグ) #1548

OpenPNEから送信されるメールの文言が必ずしも受信する人が期待する言語ではない

Shinichi Urabe13年以上前に追加. 11年以上前に更新.

ステータス:
New(新規)
優先度:
Normal(通常)
担当者:
-
対象バージョン:
開始日:
2010-09-01
期日:
進捗率:

0%

3.6 で発生するか:
Yes (はい)
3.8 で発生するか:
Unknown (未調査)

説明

現象

メールの言語が sf_user->getCalture() で判別されるため、必ずしも送信先のメールアドレスの受信者が希望する言語になっていない

原因

sf_user->getCalture() をメール文言の言語判別に使っている

修正内容


関連するチケット

関連している OpenPNE 3 - Enhancement(機能追加・改善) #1509: The issue that notification email doesn't send to both mobile and PC. (通知メールがモバイルとPCの両方に送信されていない問題) New(新規) 2010-08-18
関連している OpenPNE 3 - Backport(バックポート) #1585: OpenPNEから送信されるメールの文言が必ずしも受信する人が期待する言語ではない Won't fix(対応せず) 2010-09-01

関係しているリビジョン

リビジョン 75ee6d64 (差分)
Masato Nagasawa13年以上前に追加

fixed culture in the mail(dailynews/birthday). (fixes #1548)

リビジョン 8bf27477 (差分)
Masato Nagasawa13年以上前に追加

Revert "fixed culture in the mail(dailynews/birthday). (fixes #1548)"

This reverts commit 75ee6d647adaf4a77316a9d63b09eb6f83e2da1e.

履歴

#1 Masato Nagasawa13年以上前に更新

  • ステータスNew(新規) から Accepted(着手) に変更
  • 担当者Masato Nagasawa にセット

#2 Masato Nagasawa13年以上前に更新

とりあえず誕生日とデイリーニュースだけ対応させます。
個人的には招待時などのメールは、
現在表示している言語で送られる方向で問題ないかなと思ってます。

また mail_template.yml から読み込まれる場合、
ja_JP以外の定義が現状されていない状態なので言語を「English」にしても日本語のメールが送られてくる状態となっています。

#3 Masato Nagasawa13年以上前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

更新履歴 75ee6d647adaf4a77316a9d63b09eb6f83e2da1e で適用されました。

#4 Shogo Kawahara13年以上前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更

とりあえず誕生日とデイリーニュースだけ対応させます。
個人的には招待時などのメールは、
現在表示している言語で送られる方向で問題ないかなと思ってます。

となるとコミュニティ管理者への参加メッセージなどはどうなるでしょう?

#1509 の為に行った修正 (http://github.com/kawahara/OpenPNE3/commit/12d526bbf66fc8b02f4a543bfe62318e6c020238
) 中に含めたような対処をするのが望ましいように思えますが、"バグ" チケットとしにくい感があります。理由は以下です。

  • 新しい送信方法が増えたという点

提案として

  • Enhancement チケットとして 中途半端でない対応をする。 (但し3.6では対処されないということになる)
  • バグチケットとして、 #1509 のように新しいメソッドを作ってでも対処する。 (もちろんプラグインとの関係があるので互換性を保つように作る)

#5 Masato Nagasawa13年以上前に更新

  • ステータスRejected(差し戻し) から Accepted(着手) に変更

すぐ判断できないので、ひとまず今回の変更を戻しておきます。

#6 Masato Nagasawa13年以上前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更

更新履歴 8bf27477f040daaedbc2d657c4a83ef9b562c173 で適用されました。

#7 Masato Nagasawa13年以上前に更新

  • ステータスPending Review(レビュー待ち) から New(新規) に変更
  • 担当者 を削除 (Masato Nagasawa)

一度手放します。

#8 Shingo Yamada12年以上前に更新

  • 3.6 で発生するかYes から No に変更

#9 Shouta Kashiwagi約12年前に更新

  • 対象バージョンOpenPNE 3.7.0 から 252 に変更

#10 Shouta Kashiwagi約12年前に更新

  • 対象バージョン252 から OpenPNE 3.8.x に変更

#11 Youichi Kimura11年以上前に更新

  • 3.6 で発生するかNo から Unknown (未調査) に変更

このバグについて、修正案が提示されていながら進展のない状態が続いていますが、個人的には note-4 で提示された修正で問題ないと思います。ただしこのコミットは #1509 の為の修正も含んでいるためコミットを分割する必要があります。

また、当チケットを Bug とするか Enhancement にするかについては、ガイドライン的な物がないため個人的な見解ですが、結果的な修正がどうであれ外見上の動作が不具合(メールテンプレートが多言語対応しているのは送信先メンバーの話語に合わせるための機能と考えるのが自然であり、それに反する動作をするならバグ)である以上バグチケットにすべきだと認識しています。

#12 Youichi Kimura11年以上前に更新

  • 3.6 で発生するかUnknown (未調査) から Yes (はい) に変更

他の形式にエクスポート: Atom PDF