Bug(バグ) #3367
デイリーニュースで使用されるガジェットのパラメータがエスケープされている
Start date:
2013-06-25
Due date:
% Done:
0%
3.6 で発生するか:
Unknown (未調査)
3.8 で発生するか:
Unknown (未調査)
Description
Overview (現象)¶
管理画面のガジェット設定 (/design/gadget?type=dailyNews) からデイリーニュースで使用するガジェットを設定できるが、このガジェットのパラメーターに対してエスケープが施された状態でメール送信されてしまう。
例えば、デイリーニュースのガジェットに「フリーエリア」を追加して「内容」に「>>>> aaaaa <<<<」を設定すると、送信されるデイリーニュースのメール本文には「>>>> aaaaa <<<<」が出力される。
Causes (原因)¶
メールテンプレートにおけるエスケープ処理の問題は #1344 によって修正されており、その修正内容は下記のようなものでした。
--- a/lib/util/opMailSend.class.php
+++ b/lib/util/opMailSend.class.php
@@ -81,6 +82,7 @@ class opMailSend
$view = new sfTemplatingComponentPartialView($context, 'superGlobal', 'notify_mail:'.$target.'_'.$template, '');
$context->set('view_instance', $view);
+ $view->getAttributeHolder()->setEscaping(false);
$view->setPartialVars($params);
$view->setAttribute('renderer_config', array('twig' => 'opTemplateRendererTwig'));
$view->setAttribute('rule_config', array('notify_mail' => array(
一方、デイリーニュースの送信には、メールテンプレートこそ使用しているものの本文中で include_component を使用しており、呼び出されるコンポーネントのために新たに sfViewParameterHolder が生成されます。この sfViewParameterHolder は呼び出し元ビューのエスケープ設定を引き継がず、sfConfig から sf_escaping_method や sf_escaping_strategy の値を取得して設定するため、コンポーネント内では #1344 の修正による効果が及ばないことになります。これが原因で、デイリーニュースのガジェット内ではエスケープが有効な状態でテンプレートが出力されます。
Way to fix (修正内容)¶
修正内容を記入
History
#1 Updated by Youichi Kimura over 11 years ago
- Description updated (diff)
#2 Updated by Youichi Kimura over 11 years ago
- File 3367_mail-gadget-escaping.patch View added
応急措置案パッチ。sfConfig の変更はこの箇所だけで十分かどうか要調査。
#3 Updated by kaoru n over 7 years ago
- Target version changed from OpenPNE 3.9.0-old to OpenPNE 3.9.0
#4 Updated by kaoru n almost 5 years ago
- Target version changed from OpenPNE 3.9.0 to OpenPNE 3.10.x