プロジェクト

全般

プロフィール

Bug(バグ) #4105

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

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

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

100%

3.6 で発生するか:
Yes (はい)
3.8 で発生するか:
Yes (はい)

説明

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() を使用して表示するように変更する。

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


子チケット

Backport(バックポート) #4113: 管理画面 コミュニティカテゴリ設定 カテゴリ名称変更時のエラーメッセージが翻訳されないFixed(完了)kaoru n

Backport(バックポート) #4114: 管理画面 コミュニティカテゴリ設定 カテゴリ名称変更時のエラーメッセージが翻訳されないFixed(完了)kaoru n


関連するチケット

関連している OpenPNE 3 - Bug(バグ) #4104: 管理画面 メール通知テンプレート設定 にてエラーメッセージが翻訳されない Fixed(完了) 2017-01-06
関連している OpenPNE 3 - Backport(バックポート) #4118: 管理画面 コミュニティカテゴリ設定 カテゴリ名称追加時のエラーメッセージが翻訳されない Fixed(完了) 2017-01-06

関係しているリビジョン

リビジョン eb39278b (差分)
kaoru nほぼ7年前に追加

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

リビジョン c3db3ea1 (差分)
kaoru nほぼ7年前に追加

(fixed #4105-6) ->getWidgetSchema()->getLabel('name') を ['name']->renderLabelName() に修正

リビジョン 7b2a5ba0 (差分)
kaoru nほぼ7年前に追加

(fixed #4105-6) ->getWidgetSchema()->getLabel('name') を ['name']->renderLabelName() に修正

リビジョン 994765f3 (差分)
kaoru nほぼ7年前に追加

(fixed #4105-6) ->getWidgetSchema()->getLabel('name') を ['name']->renderLabelName() に修正

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

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

リビジョン 39ba8257
kaoru n6年以上前に追加

Merge pull request #415 from nishizoe/t-4105

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

履歴

#1 kaoru n7年以上前に更新

  • 関連している Bug(バグ) #4104: 管理画面 メール通知テンプレート設定 にてエラーメッセージが翻訳されない を追加

#2 kaoru n7年以上前に更新

  • 対象バージョンOpenPNE 3.9.0-old にセット

#3 kaoru n7年以上前に更新

  • ステータスNew(新規) から Accepted(着手) に変更
  • 担当者kaoru n にセット

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

$this->redirect('community/categoryList');

リダイレクトしているため、renderGlobalErrors() または renderError() を使用して表示できないため、翻訳したメッセージをフラッシュメッセージで表示できるようにする。

#4 kaoru n7年以上前に更新

  • 説明 を更新 (diff)

#5 kaoru n7年以上前に更新

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

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

#6 Youichi Kimura約7年前に更新

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

一点フィードバックします:

  • $i18n->__($form->getWidgetSchema()->getLabel('name'), null, 'form_community') の部分は $form['name']->renderLabelName() で出力して下さい

CSRF Token のエラーが画面上に表示されない問題がありますが、修正前のコードから存在する不具合のためこのチケットに対する修正としては十分であると思います。

#7 isao sano約7年前に更新

  • 関連している Backport(バックポート) #4118: 管理画面 コミュニティカテゴリ設定 カテゴリ名称追加時のエラーメッセージが翻訳されない を追加

#8 kaoru n約7年前に更新

  • 対象バージョンOpenPNE 3.9.0-old から OpenPNE 3.9.0 に変更

対象バージョン変更のため、修正内容の確認を行います。

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

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

#10 Youichi Kimura6年以上前に更新

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

renderLabelName メソッド内では __() メソッドによる翻訳も行われているため、
$i18n->__($form['name']->renderLabelName(), null, 'form_community') の箇所については、 $form['name']->renderLabelName() のみに変更をお願いします。

#11 kaoru n6年以上前に更新

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

#12 Youichi Kimura6年以上前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更
  • 3.6 で発生するかUnknown (未調査) から Yes (はい) に変更
  • 3.8 で発生するかUnknown (未調査) から Yes (はい) に変更

レビューOKです

#14 kaoru n6年以上前に更新

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

マージしました

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