Backport（バックポート） #4113
Bug（バグ） #4105: 管理画面 コミュニティカテゴリ設定 カテゴリ名称変更時のエラーメッセージが翻訳されない
管理画面 コミュニティカテゴリ設定 カテゴリ名称変更時のエラーメッセージが翻訳されない
Start date:
2017-01-06
Due date:
% Done:
100%
Description
Overview (現象)¶
管理画面 コミュニティカテゴリ設定 カテゴリ名称変更時のエラーメッセージが翻訳されない。
Causes (原因)¶
$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() を使用して表示するように変更する。
フラッシュメッセージにて翻訳されたエラーメッセージを表示するようにする。
History
#1
Updated by kaoru n almost 4 years ago
Updated by - Status changed from New（新規） to Pending Review（レビュー待ち）
- % Done changed from 0 to 50
https://github.com/openpne/OpenPNE3/pull/421
にてプルリクエストを行いました。
レビューをお願いします。
#2
Updated by Youichi Kimura over 3 years ago
Updated by - Status changed from Pending Review（レビュー待ち） to Rejected（差し戻し）
#4105-6 と同様の理由で差し戻します
#3
Updated by kaoru n over 3 years ago
- Status changed from Rejected（差し戻し） to Pending Review（レビュー待ち）
#4
Updated by Youichi Kimura about 3 years ago
- Status changed from Pending Review（レビュー待ち） to Rejected（差し戻し）
#4105-10 と同様の箇所について修正をお願いします。
#5
Updated by kaoru n about 3 years ago
- Status changed from Rejected（差し戻し） to Pending Review（レビュー待ち）
#6
Updated by Youichi Kimura about 3 years ago
- Status changed from Pending Review（レビュー待ち） to Pending Testing（テスト待ち）
- % Done changed from 50 to 70
レビューOKです
#7
Updated by kaoru n about 3 years ago
- Target version changed from OpenPNE 3.6.x to OpenPNE 3.6.28
#8
Updated by kaoru n about 3 years ago
- Status changed from Pending Testing（テスト待ち） to Pending Merge（マージ待ち）
- % Done changed from 70 to 80
試験行いました。
問題ありません。
#9
Updated by kaoru n about 3 years ago
- Status changed from Pending Merge（マージ待ち） to Fixed（完了）
- % Done changed from 80 to 100
マージしました