プロジェクト

全般

プロフィール

Bug(バグ) #2041

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

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

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

100%

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

説明

概要

管理画面にて英語版の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 - Backport(バックポート) #1771: Notification of Registration URLのテンプレートを編集する際、英語版が日本語版として保存される Fixed(完了) 2010-11-09 2011-06-24
関連している OpenPNE 3 - Backport(バックポート) #2042: Notification of Registration URLのテンプレートを編集する際、英語版が日本語版として保存される Fixed(完了) 2010-11-09

関係しているリビジョン

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

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

履歴

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

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

更新履歴 3fdc68ce08f309d2bdef168c0b02e09cd1b60405 で適用されました。

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

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

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

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

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

コードチェックをしました。3.6.x と同等であり問題ありません。

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

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

動作テストを行いました。 #1771 で示していますが、修正方針は適切です。

マージミス等がないかについての確認も含めて、以下のテストを行いました。

修正前の動作確認

  • 管理画面上の操作で、英語版を編集した後に、
    • (5) 【バグ】データベースの notification_mail_translation テーブルを見ると lang: ja_JP として保存されている

修正後の動作確認

  • 管理画面上の操作で、英語版を編集した後に、
    • (5) データベースの notification_mail_translation テーブルを見ると lang: en として保存されている

以上が確認できているため master ブランチについても修正は適切に行なわれているものとし、チケットをクローズします。

なお、この問題は、元チケットが 3.6.x 向けとなっており、 BP として 3.4.x および 3.7.0 が作成されていますが、チケットの扱いについては(変更した方がよい強い理由が見当たらないので)このままにしておきます。

#5 Shingo Yamadaほぼ13年前に更新

  • トラッカーBackport(バックポート) から Bug(バグ) に変更

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