From c52d23029bd849f87ea1ce83c41941047655c5b9 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Marcus=20F=C3=B6rster?= <git@xerc.de>
Date: Fri, 7 Jan 2022 14:52:04 +0100
Subject: [PATCH 1/3] [OPTIMIZE] parser & replace speed

OR use \v+ (ref. https://www.php.net/manual/en/regexp.reference.escape.php)
---
 Classes/ViewHelpers/Format/EscapeViewHelper.php | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/Classes/ViewHelpers/Format/EscapeViewHelper.php b/Classes/ViewHelpers/Format/EscapeViewHelper.php
index 8a0bd80..590e9f8 100755
--- a/Classes/ViewHelpers/Format/EscapeViewHelper.php
+++ b/Classes/ViewHelpers/Format/EscapeViewHelper.php
@@ -20,11 +20,11 @@ public static function renderStatic(array $arguments, \Closure $renderChildrenCl
         }
 
         // Remove Linebreaks from RTE fields
-        $value = preg_replace("/\r|\n/", " ", $value);
+        $value = preg_replace('/\R+/', ' ', $value);
 
         // Escape ', ", \
         $value = addslashes($value);
 
         return $value;
     }
-}
\ No newline at end of file
+}

From cb44e0b5085bd39cbfb74c58af01159d28f305d5 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Marcus=20F=C3=B6rster?= <m.foerster@xerc.de>
Date: Fri, 7 Jan 2022 18:35:21 +0100
Subject: [PATCH 2/3] [REMOVE] jQuery usage

---
 Resources/Private/Templates/Consent/List.html         |  2 +-
 .../Public/JavaScript/Controller/ConsentController.js | 11 -----------
 2 files changed, 1 insertion(+), 12 deletions(-)

diff --git a/Resources/Private/Templates/Consent/List.html b/Resources/Private/Templates/Consent/List.html
index 64cd72e..b3695c6 100755
--- a/Resources/Private/Templates/Consent/List.html
+++ b/Resources/Private/Templates/Consent/List.html
@@ -21,6 +21,6 @@
 				</table>
 			</f:if>
 		</f:for>
-		<a href="#" class="btn btn-primary btn-lg js-showConsentModal"><f:translate key="list.button.openConsent" /></a>
+		<a href="javascript:;" onclick="event.preventDefault();klaro.show()" class="btn btn-primary btn-lg"><f:translate key="list.button.openConsent"/></a>
 	</f:section>
 </html>
\ No newline at end of file
diff --git a/Resources/Public/JavaScript/Controller/ConsentController.js b/Resources/Public/JavaScript/Controller/ConsentController.js
index 4665291..0297a0b 100755
--- a/Resources/Public/JavaScript/Controller/ConsentController.js
+++ b/Resources/Public/JavaScript/Controller/ConsentController.js
@@ -35,15 +35,4 @@ let ConsentApp = new function ConsentController() {
             console.error('The Callback function ' + service.ownCallback + ' is not yet defined. Please create it first.');
         }
     };
-
-    //--- constructor ---
-    (function contruct() {
-        $(document).ready(function () {
-            //Listener for the button on the privacy page, to edit the consent
-            $(document).on('click', '.js-showConsentModal', function (event) {
-                event.preventDefault();
-                klaro.show();
-            });
-        });
-    })();
 };
\ No newline at end of file

From c71949c2366fa564dee6814889e57579c807ea94 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Marcus=20F=C3=B6rster?= <m.foerster@xerc.de>
Date: Fri, 7 Jan 2022 18:36:27 +0100
Subject: [PATCH 3/3] [CLEANUP] JavaScript loading

---
 Resources/Private/Partials/Klaro/Service.html |  2 +-
 .../Private/Templates/Consent/Consent.html    | 16 +++++-----
 .../Controller/ConsentController.js           | 30 +++++++++----------
 3 files changed, 23 insertions(+), 25 deletions(-)

diff --git a/Resources/Private/Partials/Klaro/Service.html b/Resources/Private/Partials/Klaro/Service.html
index 9e14ff0..58574d1 100755
--- a/Resources/Private/Partials/Klaro/Service.html
+++ b/Resources/Private/Partials/Klaro/Service.html
@@ -6,7 +6,7 @@
 <f:if condition="{service.categories}">
     <we:variable name="purposes"><f:for each="{service.categories}" as="category" iteration="iterator">'{category.title -> f:format.case(mode: 'lower')}'<f:if condition="{iterator.isLast} != 1">, </f:if></f:for></we:variable>
 </f:if>
-<script type="text/javascript">
+<script type="module">
     klaroConfig.services.push({
         name: '<f:format.raw>{service.provider}-{service.uid}</f:format.raw>',
         title: '<f:format.raw><we:format.escape>{service.title}</we:format.escape></f:format.raw>',
diff --git a/Resources/Private/Templates/Consent/Consent.html b/Resources/Private/Templates/Consent/Consent.html
index 6c2e4fc..b0998ec 100755
--- a/Resources/Private/Templates/Consent/Consent.html
+++ b/Resources/Private/Templates/Consent/Consent.html
@@ -18,20 +18,20 @@
 
 	<f:section name="FooterAssets">
 		<f:spaceless>
-			<script type="text/javascript" src="{f:uri.resource(path: 'JavaScript/Controller/ConsentController.js')}"></script>
-			<script type="text/javascript">var klaroConfig = {f:format.htmlentitiesDecode(value: '{klaroConfig -> we:format.json()}')}</script>
-			<f:for each="{services}" as="service">
-				<f:render partial="Service/{service.provider}" section="Footer" arguments="{service:service}" optional="1"/>
-				<f:render partial="Klaro/Service" arguments="{service:service}"/>
-			</f:for>
 			<f:if condition="{settings.klaro.stylePrefix}">
 				<f:then>
-					<script type="text/javascript" src="{f:uri.resource(path: 'Library/klaro/klaro.js')}"></script>
+					<script defer src="{f:uri.resource(path: 'Library/klaro/klaro.js')}"></script>
 				</f:then>
 				<f:else>
-					<script type="text/javascript" src="{f:uri.resource(path: 'Library/klaro/klaro-no-css.js')}"></script>
+					<script defer src="{f:uri.resource(path: 'Library/klaro/klaro-no-css.js')}"></script>
 				</f:else>
 			</f:if>
+			<script defer src="{f:uri.resource(path: 'JavaScript/Controller/ConsentController.js')}"></script>
+			<script>var klaroConfig = {f:format.htmlentitiesDecode(value: '{klaroConfig -> we:format.json()}')}</script>
+			<f:for each="{services}" as="service">
+				<f:render partial="Service/{service.provider}" section="Footer" arguments="{service:service}" optional="1"/>
+				<f:render partial="Klaro/Service" arguments="{service:service}"/>
+			</f:for>
 		</f:spaceless>
 	</f:section>
 </html>
\ No newline at end of file
diff --git a/Resources/Public/JavaScript/Controller/ConsentController.js b/Resources/Public/JavaScript/Controller/ConsentController.js
index 0297a0b..23c3397 100755
--- a/Resources/Public/JavaScript/Controller/ConsentController.js
+++ b/Resources/Public/JavaScript/Controller/ConsentController.js
@@ -10,26 +10,24 @@ let ConsentApp = new function ConsentController() {
      * @param object service
      */
     this.consentChanged = function (state, service) {
-        if (state === true) {
-            if (service.name.indexOf('google-tagmanager-service') !== -1) {
-                let tempObj = {
-                    event: service.gtm.trigger
-                };
-                tempObj[service.gtm.variable] = true;
-                window.dataLayer.push(tempObj);
+        if (state === true && service.name.indexOf('google-tagmanager-service') !== -1) {
+            let tempObj = {
+                event: service.gtm.trigger
+            };
+            tempObj[service.gtm.variable] = true;
+            window.dataLayer.push(tempObj);
 
-                /*
-                //ES6 - https://stackoverflow.com/questions/11508463/javascript-set-object-key-by-variable
-                window.dataLayer.push({
-                    event: service.name,
-                    [service.name]: true
-                });
-                */
-            }
+            /*
+            //ES6 - https://stackoverflow.com/questions/11508463/javascript-set-object-key-by-variable
+            window.dataLayer.push({
+                event: service.name,
+                [service.name]: true
+            });
+            */
         }
 
         //Check if the own callback function is allready defined
-        if (typeof window[service.ownCallback] === "function") {
+        if (typeof window[service.ownCallback] === 'function') {
             window[service.ownCallback](state, service);
         } else if (service.ownCallback !== '') {
             console.error('The Callback function ' + service.ownCallback + ' is not yet defined. Please create it first.');