Project

General

Profile

Bug(バグ) #4104

管理画面 メール通知テンプレート設定 にてエラーメッセージが翻訳されない

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 で発生するか:
Unknown (未調査)
3.8 で発生するか:
Yes (はい)

Description

Overview (現象)

#2909 にて、文字数チェックを追加したがエラーメッセージが日本語に翻訳されない。

Causes (原因)

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

$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') ?>


Subtasks

Backport(バックポート) #4110: 管理画面 メール通知テンプレート設定 にてエラーメッセージが翻訳されないFixed(完了)kaoru n

Backport(バックポート) #4111: 管理画面 メール通知テンプレート設定 にてエラーメッセージが翻訳されないFixed(完了)kaoru n


Related issues

Related to OpenPNE 3 - Bug(バグ) #2392: 管理画面:メール通知送信設定 で CSRFtoken の値が異常だった場合にエラーメッセージが表示されない Fixed(完了) 2011-09-05 2011-09-21
Related to OpenPNE 3 - Enhancement(機能追加・改善) #91: Add ability for email notifications (お知らせメールのための機能を追加する) Fixed(完了) 2009-10-13
Related to OpenPNE 3 - Bug(バグ) #2909: 管理画面:メール通知テンプレート設定にて、タイトルが長過ぎる場合にエラーにならず途中で切れて保存されてしまう Fixed(完了) 2017-01-06
Related to OpenPNE 3 - Bug(バグ) #4105: 管理画面 コミュニティカテゴリ設定 カテゴリ名称変更時のエラーメッセージが翻訳されない Fixed(完了) 2017-01-06
Related to OpenPNE 3 - Bug(バグ) #4106: 管理画面 メール通知送信設定 にてエラーメッセージが翻訳されない Won't fix(対応せず) 2017-01-06

History

#1 Updated by kaoru n over 2 years ago

  • Description updated (diff)

#2 Updated by kaoru n over 2 years ago

  • Related to Bug(バグ) #2392: 管理画面:メール通知送信設定 で CSRFtoken の値が異常だった場合にエラーメッセージが表示されない added

#3 Updated by kaoru n over 2 years ago

#4 Updated by kaoru n over 2 years ago

  • Related to Bug(バグ) #2909: 管理画面:メール通知テンプレート設定にて、タイトルが長過ぎる場合にエラーにならず途中で切れて保存されてしまう added

#5 Updated by kaoru n over 2 years ago

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

#6 Updated by kaoru n over 2 years ago

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

#7 Updated by kaoru n over 2 years ago

  • Status changed from New(新規) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

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

#8 Updated by kaoru n over 2 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)
  • Target version changed from OpenPNE 3.9.0-old to OpenPNE 3.9.0

対象バージョン変更により修正内容の確認が必要であるため差し戻します。

#10 Updated by kaoru n about 2 years ago

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

https://github.com/openpne/OpenPNE3/pull/413
master向けに更新しました

#12 Updated by Rimpei Ogawa about 2 years ago

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

レビュー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