Project

General

Profile

Backport(バックポート) #4113

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

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

Added by kaoru n almost 3 years ago. Updated over 2 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Assignee:
Target version:
Start date:
2017-01-06
Due date:
% Done:

100%


Description

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

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

History

#1 Updated by kaoru n almost 3 years ago

  • 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 almost 3 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

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

#3 Updated by kaoru n over 2 years ago

  • Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)

#4 Updated by Youichi Kimura over 2 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

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

#5 Updated by kaoru n over 2 years ago

  • Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)

#6 Updated by Youichi Kimura over 2 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

レビューOKです

#7 Updated by kaoru n over 2 years ago

  • Target version changed from OpenPNE 3.6.x to OpenPNE 3.6.28

#8 Updated by kaoru n over 2 years ago

  • Status changed from Pending Testing(テスト待ち) to Pending Merge(マージ待ち)
  • % Done changed from 70 to 80

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

#9 Updated by kaoru n over 2 years ago

  • Status changed from Pending Merge(マージ待ち) to Fixed(完了)
  • % Done changed from 80 to 100

マージしました

Also available in: Atom PDF