操作
Bug(バグ) #4105
完了管理画面 コミュニティカテゴリ設定 カテゴリ名称変更時のエラーメッセージが翻訳されない
開始日:
2017-01-06
期日:
進捗率:
100%
予定工数:
(合計: 0:00時間)
3.6 で発生するか:
Yes (はい)
3.8 で発生するか:
Yes (はい)
説明
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() を使用して表示するように変更する。
フラッシュメッセージにて翻訳されたエラーメッセージを表示するようにする。
kaoru n さんがほぼ8年前に更新
- ステータス を Accepted(着手) から Pending Review(レビュー待ち) に変更
- 進捗率 を 0 から 50 に変更
https://github.com/openpne/OpenPNE3/pull/415
にてプルリクエストを行いました。
レビューをお願いします。
Youichi Kimura さんが7年以上前に更新
- ステータス を Pending Review(レビュー待ち) から Rejected(差し戻し) に変更
一点フィードバックします:
$i18n->__($form->getWidgetSchema()->getLabel('name'), null, 'form_community')
の部分は$form['name']->renderLabelName()
で出力して下さい
CSRF Token のエラーが画面上に表示されない問題がありますが、修正前のコードから存在する不具合のためこのチケットに対する修正としては十分であると思います。
isao sano さんが7年以上前に更新
- 関連している Backport(バックポート) #4118: 管理画面 コミュニティカテゴリ設定 カテゴリ名称追加時のエラーメッセージが翻訳されない を追加
Youichi Kimura さんが約7年前に更新
- ステータス を Pending Review(レビュー待ち) から Rejected(差し戻し) に変更
renderLabelName メソッド内では __()
メソッドによる翻訳も行われているため、$i18n->__($form['name']->renderLabelName(), null, 'form_community')
の箇所については、 $form['name']->renderLabelName()
のみに変更をお願いします。
Youichi Kimura さんが約7年前に更新
- ステータス を Pending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
- 進捗率 を 50 から 70 に変更
- 3.6 で発生するか を Unknown (未調査) から Yes (はい) に変更
- 3.8 で発生するか を Unknown (未調査) から Yes (はい) に変更
レビューOKです
操作