プロジェクト

全般

プロフィール

Backport(バックポート) #2042

Notification of Registration URLのテンプレートを編集する際、英語版が日本語版として保存される

Masato Nagasawaほぼ13年前に追加. ほぼ13年前に更新.

ステータス:
Fixed(完了)
優先度:
High(高め)
担当者:
対象バージョン:
開始日:
2010-11-09
期日:
進捗率:

100%


説明

概要

管理画面にて英語版のNotification of Registration URLのテンプレートを編集し、SNSメンバーから招待メールを送信した場合、送信されたメールの内容に編集が反映されない。

追記

原因を見れば明らかですが、この問題は「英語のメールテンプレートが反映されない」という問題と同時に「英語のメールテンプレートが日本語版で使用される」という問題を発生させています。

  1. 管理画面の言語設定を英語にする
  2. 管理画面のメール設定のメール通知テンプレート設定ページにアクセスする
  3. Notification of Registration URLのテンプレートを編集する
  4. pc_frontend側にログインし言語設定を【日本語】にする
  5. SNSメンバーから招待メールを送信する
  6. 送信された招待メールの内容を確認する
  7. 【英語(あるいは上記で編集した)メールが届く】

再現方法

  1. 管理画面の言語設定を英語にする
  2. 管理画面のメール設定のメール通知テンプレート設定ページにアクセスする
  3. Notification of Registration URLのテンプレートを編集する
  4. pc_frontend側にログインし言語設定を英語にする
  5. SNSメンバーから招待メールを送信する
  6. 送信された招待メールの内容を確認する
  7. 編集が反映されていない

再現バージョン

  • OpenPNE 3.6beta7-dev
  • OpenPNE 3.6beta6

その他のバージョンについては未確認

原因

テンプレート保存時に "ja_JP" として保存していることが原因で、
どの言語を選択していても ja_JP のテンプレートとして保存されてしまう。

apps/pc_backend/modules/mail/actions/actions.class.php
$translation = $obj->Translation['ja_JP'];

修正内容

文字列リテラルから、sfDoctrineRecord::getDefaultCulture()に変更。

$translation = $obj->Translation[sfDoctrineRecord::getDefaultCulture()];

関連するチケット

関連している OpenPNE 3 - Bug(バグ) #2041: Notification of Registration URLのテンプレートを編集する際、英語版が日本語版として保存される Fixed(完了) 2010-11-09

関係しているリビジョン

リビジョン 09db901a (差分)
Masato Nagasawaほぼ13年前に追加

fixed the language to reflect(fixes #2042 BP from #1771)

リビジョン 1e0fd6cf (差分)
Masato Nagasawaほぼ13年前に追加

Revert "fixed the language to reflect(fixes #2042 BP from #1771)"

This reverts commit 09db901ab72a967bf46ec8a3ab1a380b7e2a5977.

リビジョン 0bea76c0 (差分)
Masato Nagasawaほぼ13年前に追加

fixed the language to reflect(fixes #2042 BP from #1771)

履歴

#1 Masato Nagasawaほぼ13年前に更新

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

更新履歴 09db901ab72a967bf46ec8a3ab1a380b7e2a5977 で適用されました。

#2 Masato Nagasawaほぼ13年前に更新

  • 対象バージョンOpenPNE 3.4.12 から OpenPNE 3.4.13 に変更

#3 Masato Nagasawaほぼ13年前に更新

更新履歴 1e0fd6cf0a714e1e182b558786107581b1a47de1 で適用されました。

#4 Masato Nagasawaほぼ13年前に更新

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

3.4.12 では取り込みを行わないので、一旦取り消します。

#5 Masato Nagasawaほぼ13年前に更新

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

#6 Masato Nagasawaほぼ13年前に更新

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

更新履歴 0bea76c049f291538a170f0a2526cab5107ec0f8 で適用されました。

#7 Yuma Sakataほぼ13年前に更新

テストOKです。

#8 Minoru Takaiほぼ13年前に更新

  • 題名Notification of Registration URLのテンプレートを編集して招待メールを送信しても編集が反映されない から Notification of Registration URLのテンプレートを編集する際、英語版が日本語版として保存される に変更

問題を捉えきれていなかったため概要を追記しました。

#9 Minoru Takaiほぼ13年前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

コードチェックをしました。なお、メール周りは 3.4 系以前 と 3.5 系以降 でテンプレートの所在などの実装が異なっていますが、この問題についてはその実装の差異に影響されない(修正内容も同様でよい)ものと言えそうです。

修正方針は(元チケットで評価されているとおり)問題なく、コーディング規約等の観点からも修正内容は問題ありません。

#10 Minoru Takaiほぼ13年前に更新

  • ステータスPending Testing(テスト待ち) から Fixed(完了) に変更
  • 進捗率70 から 100 に変更

note-7 でも行なわれていますが、動作チェックも行いました。

修正前の動作確認

  • 管理画面上の操作で、英語版を編集した後に、
    • (1) 【バグ】英語版を編集しようとすると、編集前の英語版テンプレートが表示される
    • (2) 【バグ】日本語版を編集しようとすると、編集した英語版のテンプレートが表示される
    • (3) 【バグ】メンバー画面で、日本語設定のまま招待メールを送ると、英語版のテンプレートで送信される
    • (4) 【バグ】メンバー画面で、英語設定のまま招待メールを送ると、編集前の英語版テンプレートで送信される
    • (5) 【バグ】データベースの notification_mail_translation テーブルを見ると lang: ja_JP として保存されている

当初、このチケットで問題とされたのは上記の (4) のみですが、これらは全て同じ原因によって引き起こされているため修正できているかどうかは、これらのいずれかが修正できているかどうかで評価することが可能です(もちろん上記5つが全て解消されていることの確認が重要であることには変わりありません)。

修正後の動作確認

  • 管理画面上の操作で、英語版を編集した後に、
    • (1) 英語版を編集しようとすると、編集した英語版テンプレートが表示される
    • (2) 日本語版を編集しようとすると、日本語版のテンプレートが表示される
    • (3) メンバー画面で、日本語設定のまま招待メールを送ると、日本語版のテンプレートで送信される
    • (4) メンバー画面で、英語設定のまま招待メールを送ると、編集が反映された英語版テンプレートで送信される
    • (5) データベースの notification_mail_translation テーブルを見ると lang: en として保存されている
  • 管理画面上の操作で、日本語版を編集した後に、
    • (6) データベースの notification_mail_translation テーブルを見ると lang: ja_JP として保存されている
      • (6) より (1)-(4) に相当する動作についても、想定通りの動作となることが予想できる

修正は適切に行なわれていると判断し、チケットをクローズします。

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