[Data Attributes Generator] Output as raw HTML (#237)

* fix: output data attributes as raw HTML * remove raw filter
dachcom-digital · Jan 6, 2025 · 39fc2f3 · 39fc2f3
1 parent 4f57afb
commit 39fc2f3
Show file tree

Hide file tree

Showing 6 changed files with 12 additions and 11 deletions.
diff --git a/src/Twig/Extension/DataAttributesExtension.php b/src/Twig/Extension/DataAttributesExtension.php
@@ -15,8 +15,9 @@ public function __construct(protected DataAttributeService $dataAttributeService
     public function getFunctions(): array
     {
         return [
-            new TwigFunction('toolbox_data_attributes_generator', [$this->dataAttributeService, 'generateDataAttributes']),
+            new TwigFunction('toolbox_data_attributes_generator', [$this->dataAttributeService, 'generateDataAttributes'], [
+                'is_safe' => ['html'],
+            ]),
         ];
     }
-
 }
diff --git a/templates/toolbox/bootstrap4/video/type_asset.html.twig b/templates/toolbox/bootstrap4/video/type_asset.html.twig
@@ -1,4 +1,4 @@
-<div class="player" {{ toolbox_data_attributes_generator('video_parameter', {'video_parameter' : videoParameter}, true)|raw }} data-poster-path="{{ posterPath }}" data-play-in-lightbox="{{ playInLightbox }}" data-video-uri="{{ videoId }}">
+<div class="player" {{ toolbox_data_attributes_generator('video_parameter', {'video_parameter' : videoParameter}, true) }} data-poster-path="{{ posterPath }}" data-play-in-lightbox="{{ playInLightbox }}" data-video-uri="{{ videoId }}">
 {{ pimcore_vhs('video', {
     'attributes': {
         'class': 'video-js vjs-default-skin vjs-big-play-centered',

diff --git a/templates/toolbox/bootstrap4/video/type_vimeo.html.twig b/templates/toolbox/bootstrap4/video/type_vimeo.html.twig
@@ -1,4 +1,4 @@
-<div class="player" {{ toolbox_data_attributes_generator('video_parameter', {'video_parameter' : videoParameter}, true)|raw }} data-poster-path="{{ posterPath }}" data-play-in-lightbox="{{ playInLightbox }}" data-video-uri="{{ videoId }}"></div>
+<div class="player" {{ toolbox_data_attributes_generator('video_parameter', {'video_parameter' : videoParameter}, true) }} data-poster-path="{{ posterPath }}" data-play-in-lightbox="{{ playInLightbox }}" data-video-uri="{{ videoId }}"></div>
 {% if posterPath is not empty %}
     {% include toolbox_area_path(areaId, areaTemplateDirectory, '/partial/overlay') with {'posterPath' : posterPath, 'playInLightbox' : playInLightbox} %}
-{% endif %}
+{% endif %}
diff --git a/templates/toolbox/bootstrap4/video/type_youtube.html.twig b/templates/toolbox/bootstrap4/video/type_youtube.html.twig
@@ -1,5 +1,5 @@
-<div class="player" {{ toolbox_data_attributes_generator('video_parameter', {'video_parameter' : videoParameter}, true)|raw }} data-poster-path="{{ posterPath }}" data-play-in-lightbox="{{ playInLightbox }}" data-video-uri="{{ videoId }}"></div>
+<div class="player" {{ toolbox_data_attributes_generator('video_parameter', {'video_parameter' : videoParameter}, true) }} data-poster-path="{{ posterPath }}" data-play-in-lightbox="{{ playInLightbox }}" data-video-uri="{{ videoId }}"></div>
 
 {% if posterPath is not empty %}
     {% include toolbox_area_path(areaId, areaTemplateDirectory, '/partial/overlay') with {'posterPath' : posterPath, 'playInLightbox' : playInLightbox} %}
-{% endif %}
+{% endif %}
diff --git a/templates/toolbox/uikit3/video/type_vimeo.html.twig b/templates/toolbox/uikit3/video/type_vimeo.html.twig
@@ -9,6 +9,6 @@
             {% if autoPlay %}playsinline allow="autoplay;"{% endif %}
             data-uk-video="autoplay: true"
             width="1920" height="1080" data-uk-responsive
-            {{ toolbox_data_attributes_generator('video_parameter', {'video_parameter' : videoParameter}, true)|raw }}>
+            {{ toolbox_data_attributes_generator('video_parameter', {'video_parameter' : videoParameter}, true) }}>
     </iframe>
-{% endif %}
+{% endif %}
diff --git a/templates/toolbox/uikit3/video/type_youtube.html.twig b/templates/toolbox/uikit3/video/type_youtube.html.twig
@@ -9,6 +9,6 @@
             {% if autoPlay %}playsinline {% endif %}
             data-uk-video="autoplay: true"
             width="1920" height="1080" data-uk-responsive
-            {{ toolbox_data_attributes_generator('video_parameter', {'video_parameter' : videoParameter}, true)|raw }}>
+            {{ toolbox_data_attributes_generator('video_parameter', {'video_parameter' : videoParameter}, true) }}>
     </iframe>
-{% endif %}
+{% endif %}