Backport（バックポート） #2042
完了
Notification of Registration URLのテンプレートを編集する際、英語版が日本語版として保存される
100%
説明
概要¶
管理画面にて英語版のNotification of Registration URLのテンプレートを編集し、SNSメンバーから招待メールを送信した場合、送信されたメールの内容に編集が反映されない。
追記¶
原因を見れば明らかですが、この問題は「英語のメールテンプレートが反映されない」という問題と同時に「英語のメールテンプレートが日本語版で使用される」という問題を発生させています。
- 管理画面の言語設定を英語にする
- 管理画面のメール設定のメール通知テンプレート設定ページにアクセスする
- Notification of Registration URLのテンプレートを編集する
- pc_frontend側にログインし言語設定を【日本語】にする
- SNSメンバーから招待メールを送信する
- 送信された招待メールの内容を確認する
- 【英語（あるいは上記で編集した）メールが届く】
再現方法¶
- 管理画面の言語設定を英語にする
- 管理画面のメール設定のメール通知テンプレート設定ページにアクセスする
- Notification of Registration URLのテンプレートを編集する
- pc_frontend側にログインし言語設定を英語にする
- SNSメンバーから招待メールを送信する
- 送信された招待メールの内容を確認する
- 編集が反映されていない
再現バージョン¶
- 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()];
Masato Nagasawa さんが13年以上前に更新
- ステータス を Accepted（着手） から Pending Review（レビュー待ち） に変更
- 進捗率 を 0 から 50 に変更
更新履歴 09db901ab72a967bf46ec8a3ab1a380b7e2a5977 で適用されました。
Masato Nagasawa さんが13年以上前に更新
- ステータス を Pending Review（レビュー待ち） から New（新規） に変更
- 担当者 を削除 (
Masato Nagasawa)
3.4.12 では取り込みを行わないので、一旦取り消します。
Masato Nagasawa さんが13年以上前に更新
- ステータス を New（新規） から Accepted（着手） に変更
- 担当者 を Masato Nagasawa にセット
Masato Nagasawa さんが13年以上前に更新
- ステータス を Accepted（着手） から Pending Review（レビュー待ち） に変更
更新履歴 0bea76c049f291538a170f0a2526cab5107ec0f8 で適用されました。
Minoru Takai さんが13年以上前に更新
- 題名 を Notification of Registration URLのテンプレートを編集して招待メールを送信しても編集が反映されない から Notification of Registration URLのテンプレートを編集する際、英語版が日本語版として保存される に変更
問題を捉えきれていなかったため概要を追記しました。
Minoru Takai さんが13年以上前に更新
- ステータス を Pending Review（レビュー待ち） から Pending Testing（テスト待ち） に変更
- 進捗率 を 50 から 70 に変更
コードチェックをしました。なお、メール周りは 3.4 系以前 と 3.5 系以降 でテンプレートの所在などの実装が異なっていますが、この問題についてはその実装の差異に影響されない（修正内容も同様でよい）ものと言えそうです。
修正方針は（元チケットで評価されているとおり）問題なく、コーディング規約等の観点からも修正内容は問題ありません。
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) に相当する動作についても、想定通りの動作となることが予想できる
- (6) データベースの notification_mail_translation テーブルを見ると lang: ja_JP として保存されている
修正は適切に行なわれていると判断し、チケットをクローズします。