id 属性がサニタイズされていたのを修正

id 属性が重複するとよろしくないが、厳密にサニタイズするよりも後方互換性を優先する
nanasess · Nov 23, 2024 · 1cc9481 · 1cc9481
1 parent 016c309
commit 1cc9481
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/data/smarty_extends/modifier.script_escape.php b/data/smarty_extends/modifier.script_escape.php
@@ -54,6 +54,7 @@ function smarty_modifier_script_escape($value)
     // 念のために HTMLPurifier でサニタイズ
     $config = HTMLPurifier_Config::createDefault();
     $config->set('Cache.SerializerPath', __DIR__ . '/../cache');
+    $config->set('Attr.EnableID', true); // id 属性はサニタイズしない
     $purify = new HTMLPurifier($config);
 
     return $purify->purify($value ?? '');

diff --git a/tests/class/modifier/Modifier_ScriptEscapeTest.php b/tests/class/modifier/Modifier_ScriptEscapeTest.php
@@ -44,7 +44,7 @@ public function scriptEscapeProvider()
     public function scriptNoEscapeProvider()
     {
         return [
-            ['<p>test</p>', '<p>test</p>'],
+            ['<p id="test" class="test">test</p>', '<p id="test" class="test">test</p>'],
             ['<input type="button">', ''], // 許可タグではないのでHTMLPurifier によって完全に削除される
             ['<p>onclick</p>', '<p>onclick</p>'],
             ['<div>test</div>', '<div>test</div>'],