Backport(バックポート) #4110
Bug(バグ) #4104: 管理画面 メール通知テンプレート設定 にてエラーメッセージが翻訳されない
管理画面 メール通知テンプレート設定 にてエラーメッセージが翻訳されない
Start date:
2017-01-06
Due date:
% Done:
100%
Description
Overview (現象)¶
#2909 にて、文字数チェックを追加したがエラーメッセージが日本語に翻訳されない。
Causes (原因)¶
$this->getUser()->setFlash('error', (string)$this->form->getErrorSchema(), false);
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() を使用して表示するように変更する。
--- a/apps/pc_backend/modules/mail/actions/actions.class.php +++ b/apps/pc_backend/modules/mail/actions/actions.class.php @@ -66,7 +66,6 @@ class mailActions extends sfActions $this->getUser()->setFlash('notice', 'Saved.'); $this->redirect('@mail_template_specified?name='.$this->name); } - $this->getUser()->setFlash('error', (string)$this->form->getErrorSchema(), false); } } --- a/apps/pc_backend/modules/mail/templates/templateSuccess.php +++ b/apps/pc_backend/modules/mail/templates/templateSuccess.php @@ -70,10 +70,14 @@ <?php echo __($rawConfig[$_currentTarget][$_currentKey]['caption']) ?> <?php echo $form->renderFormTag(url_for('@mail_template_specified?name='.$name), array('method' => 'post')); ?> +<?php echo $form->renderGlobalErrors() ?> <?php echo $form->renderHiddenFields(); ?> <?php if (isset($form['title'])): ?> -<?php echo $form['title']->renderLabel() ?><?php echo $form['title']->render(array('class' => 'title')) ?> +<?php echo $form['title']->renderLabel() ?> +<?php echo $form['title']->renderError() ?> +<?php echo $form['title']->render(array('class' => 'title')) ?> <?php endif; ?> +<?php echo $form['template']->renderError() ?> <?php echo $form['template']->render(array('rows' => 30, 'cols' => 72)) ?> <?php echo __('Save') ?>
History
#1
Updated by kaoru n about 6 years ago
- Status changed from New(新規) to Pending Review(レビュー待ち)
- % Done changed from 0 to 50
https://github.com/openpne/OpenPNE3/pull/419
にてプルリクエストを行いました。
レビューをお願いします。
#3
Updated by Rimpei Ogawa over 5 years ago
- Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
- % Done changed from 50 to 70
レビューOKです。
#4
Updated by kaoru n over 5 years ago
- Target version changed from OpenPNE 3.6.x to OpenPNE 3.6.28
#5
Updated by kaoru n over 5 years ago
- Status changed from Pending Testing(テスト待ち) to Pending Merge(マージ待ち)
- % Done changed from 70 to 80
試験行いました。
問題ありません。
#6
Updated by kaoru n over 5 years ago
- Status changed from Pending Merge(マージ待ち) to Fixed(完了)
- % Done changed from 80 to 100
マージしました