プロジェクト

全般

プロフィール

Backport(バックポート) #4113

Bug(バグ) #4105: 管理画面 コミュニティカテゴリ設定 カテゴリ名称変更時のエラーメッセージが翻訳されない

管理画面 コミュニティカテゴリ設定 カテゴリ名称変更時のエラーメッセージが翻訳されない

kaoru n7年以上前に追加. 6年以上前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2017-01-06
期日:
進捗率:

100%


説明

Overview (現象)

管理画面 コミュニティカテゴリ設定 カテゴリ名称変更時のエラーメッセージが翻訳されない。

Causes (原因)

https://github.com/openpne/OpenPNE3/blob/master/apps/pc_backend/modules/community/actions/actions.class.php#L206

$this->getUser()->setFlash('error', $form['name']->getError()->getMessage());

https://github.com/openpne/OpenPNE3/blob/master/apps/pc_backend/templates/layout.php#L44
<p id="flashError" class="flash"><?php echo __($sf_user->getFlash('error')) ?></p>

のようにフラッシュメッセージにてエラーメッセージを表示しようとしている。
フラッシュメッセージにて表示する際に翻訳を試みているが、既に編集された文字列であるため翻訳されない。

Way to fix (修正内容)

renderGlobalErrors() または renderError() を使用して表示するように変更する。

フラッシュメッセージにて翻訳されたエラーメッセージを表示するようにする。

関係しているリビジョン

リビジョン 3b49e49e (差分)
kaoru n7年以上前に追加

(fixed #4113, BP from #4105) コミュニティカテゴリ設定 カテゴリ名称変更時のエラーメッセージが翻訳されるように修正

リビジョン 18145745 (差分)
kaoru n6年以上前に追加

(fixed #4113-4) エラーメッセージ表示時の項目名表示方法を変更

リビジョン 6b464edc
kaoru n6年以上前に追加

Merge pull request #421 from nishizoe/t-4113

(fixed #4113, BP from #4105) コミュニティカテゴリ設定 カテゴリ名称変更時のエラーメッセージが翻訳されるように修正

履歴

#1 kaoru n7年以上前に更新

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

https://github.com/openpne/OpenPNE3/pull/421
にてプルリクエストを行いました。
レビューをお願いします。

#2 Youichi Kimura約7年前に更新

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

#4105-6 と同様の理由で差し戻します

#3 kaoru nほぼ7年前に更新

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

#4 Youichi Kimura6年以上前に更新

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

#4105-10 と同様の箇所について修正をお願いします。

#5 kaoru n6年以上前に更新

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

#6 Youichi Kimura6年以上前に更新

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

レビューOKです

#7 kaoru n6年以上前に更新

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

#8 kaoru n6年以上前に更新

  • ステータスPending Testing(テスト待ち) から Pending Merge(マージ待ち) に変更
  • 進捗率70 から 80 に変更

試験行いました。
問題ありません。

#9 kaoru n6年以上前に更新

  • ステータスPending Merge(マージ待ち) から Fixed(完了) に変更
  • 進捗率80 から 100 に変更

マージしました

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