Project

General

Profile

Bug(バグ) #4105

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

Added by kaoru n over 2 years ago. Updated over 1 year ago.

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

100%

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

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

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


Subtasks

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

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


Related issues

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

History

#1 Updated by kaoru n over 2 years ago

  • Related to Bug(バグ) #4104: 管理画面 メール通知テンプレート設定 にてエラーメッセージが翻訳されない added

#2 Updated by kaoru n over 2 years ago

  • Target version set to OpenPNE 3.9.0-old

#3 Updated by kaoru n over 2 years ago

  • Status changed from New(新規) to Accepted(着手)
  • Assignee set to 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 Updated by kaoru n over 2 years ago

  • Description updated (diff)

#5 Updated by kaoru n over 2 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

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

#6 Updated by Youichi Kimura over 2 years ago

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

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

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

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

#7 Updated by isao sano over 2 years ago

#8 Updated by kaoru n over 2 years ago

  • Target version changed from OpenPNE 3.9.0-old to OpenPNE 3.9.0

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

#9 Updated by kaoru n about 2 years ago

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

#10 Updated by Youichi Kimura almost 2 years ago

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

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

#11 Updated by kaoru n almost 2 years ago

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

#12 Updated by Youichi Kimura almost 2 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70
  • 3.6 で発生するか changed from Unknown (未調査) to Yes (はい)
  • 3.8 で発生するか changed from Unknown (未調査) to Yes (はい)

レビューOKです

#14 Updated by kaoru n over 1 year ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

マージしました

Also available in: Atom PDF