Project

General

Profile

Bug(バグ) #1548

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

Added by Shinichi Urabe over 9 years ago. Updated over 7 years ago.

Status:
New(新規)
Priority:
Normal(通常)
Assignee:
-
Target version:
Start date:
2010-09-01
Due date:
% Done:

0%

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

Description

現象

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

原因

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

修正内容


Related issues

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

Associated revisions

Revision 75ee6d64 (diff)
Added by Masato Nagasawa about 9 years ago

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

Revision 8bf27477 (diff)
Added by Masato Nagasawa about 9 years ago

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

This reverts commit 75ee6d647adaf4a77316a9d63b09eb6f83e2da1e.

History

#1 Updated by Masato Nagasawa about 9 years ago

  • Status changed from New(新規) to Accepted(着手)
  • Assignee set to Masato Nagasawa

#2 Updated by Masato Nagasawa about 9 years ago

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

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

#3 Updated by Masato Nagasawa about 9 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

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

#4 Updated by Shogo Kawahara about 9 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

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

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

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

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

提案として

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

#5 Updated by Masato Nagasawa about 9 years ago

  • Status changed from Rejected(差し戻し) to Accepted(着手)

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

#6 Updated by Masato Nagasawa about 9 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)

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

#7 Updated by Masato Nagasawa about 9 years ago

  • Status changed from Pending Review(レビュー待ち) to New(新規)
  • Assignee deleted (Masato Nagasawa)

一度手放します。

#8 Updated by Shingo Yamada over 8 years ago

  • 3.6 で発生するか changed from Yes to No

#9 Updated by Shouta Kashiwagi over 7 years ago

  • Target version changed from OpenPNE 3.7.0 to 252

#10 Updated by Shouta Kashiwagi over 7 years ago

  • Target version changed from 252 to OpenPNE 3.8.x

#11 Updated by Youichi Kimura over 7 years ago

  • 3.6 で発生するか changed from No to Unknown (未調査)

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

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

#12 Updated by Youichi Kimura over 7 years ago

  • 3.6 で発生するか changed from Unknown (未調査) to Yes (はい)

Also available in: Atom PDF