From c1ab797037255a92e2841fede74f9dbf3a379180 Mon Sep 17 00:00:00 2001
From: Ian Morland <ianm@giffgaff.co.uk>
Date: Sun, 21 Jun 2020 10:47:04 +0100
Subject: [PATCH] Fix errors when only Google tracking code is provided

---
 resources/js/google-optimize.html    | 19 +++++++++++++++++++
 resources/js/google-tag-manager.html | 18 ------------------
 src/Listeners/AddTrackingJs.php      | 28 ++++++++++++++++++----------
 3 files changed, 37 insertions(+), 28 deletions(-)
 create mode 100644 resources/js/google-optimize.html

diff --git a/resources/js/google-optimize.html b/resources/js/google-optimize.html
new file mode 100644
index 0000000..6a3fd10
--- /dev/null
+++ b/resources/js/google-optimize.html
@@ -0,0 +1,19 @@
+<!-- Google Optimise -->
+<!-- Page-hiding snippet (recommended) -->
+<style>.async-hide { opacity: 0 !important} </style>
+<script>(function(a,s,y,n,c,h,i,d,e){s.className+=' '+y;h.start=1*new Date;
+    h.end=i=function(){s.className=s.className.replace(RegExp(' ?'+y),'')};
+    (a[n]=a[n]||[]).hide=h;setTimeout(function(){i();h.end=null},c);h.timeout=c;
+})(window,document.documentElement,'async-hide','dataLayer',2000,
+    {'%%OPT_TRACKING_CODE%%':true});</script>
+<!-- Google Analytics  with Optimize plugin -->
+<script>
+    (function(i,s,o,g,r,a,m){i['GoogleAnalyticsObject']=r;i[r]=i[r]||function(){
+        (i[r].q=i[r].q||[]).push(arguments)},i[r].l=1*new Date();a=s.createElement(o),
+        m=s.getElementsByTagName(o)[0];a.async=1;a.src=g;m.parentNode.insertBefore(a,m)
+    })(window,document,'script','https://www.google-analytics.com/analytics.js','ga');
+    ga('create', '%%TRACKING_CODE%%', 'auto');
+    ga('require', '%%OPT_TRACKING_CODE%%');
+</script>
+<!-- End Google Optimise -->
+
diff --git a/resources/js/google-tag-manager.html b/resources/js/google-tag-manager.html
index f6b34c5..e72c112 100644
--- a/resources/js/google-tag-manager.html
+++ b/resources/js/google-tag-manager.html
@@ -1,21 +1,3 @@
-<!-- Google Optimise -->
-<!-- Page-hiding snippet (recommended) -->
-<style>.async-hide { opacity: 0 !important} </style>
-<script>(function(a,s,y,n,c,h,i,d,e){s.className+=' '+y;h.start=1*new Date;
-    h.end=i=function(){s.className=s.className.replace(RegExp(' ?'+y),'')};
-    (a[n]=a[n]||[]).hide=h;setTimeout(function(){i();h.end=null},c);h.timeout=c;
-})(window,document.documentElement,'async-hide','dataLayer',2000,
-    {'%%OPT_TRACKING_CODE%%':true});</script>
-<!-- Modified Analytics tracking code with Optimize plugin -->
-<script>
-    (function(i,s,o,g,r,a,m){i['GoogleAnalyticsObject']=r;i[r]=i[r]||function(){
-        (i[r].q=i[r].q||[]).push(arguments)},i[r].l=1*new Date();a=s.createElement(o),
-        m=s.getElementsByTagName(o)[0];a.async=1;a.src=g;m.parentNode.insertBefore(a,m)
-    })(window,document,'script','https://www.google-analytics.com/analytics.js','ga');
-    ga('create', '%%TRACKING_CODE%%', 'auto');
-    ga('require', '%%OPT_TRACKING_CODE%%');
-</script>
-<!-- End Google Optimise -->
 <!-- Google Tag Manager -->
 <script>
     function gtagpush(virtualpath){dataLayer.push(virtualpath);}
diff --git a/src/Listeners/AddTrackingJs.php b/src/Listeners/AddTrackingJs.php
index baa298e..83d9625 100644
--- a/src/Listeners/AddTrackingJs.php
+++ b/src/Listeners/AddTrackingJs.php
@@ -36,31 +36,39 @@ public function __invoke(Document $document, ServerRequestInterface $request)
 
     private function analytics(Document &$document)
     {
-        if($statusGoogle = $this->settings->get('fof-analytics.statusGoogle')) {
-            $js = file_get_contents(realpath(__DIR__ . '/../../resources/js/google-tag-manager.html'));
-
-            // Add google analytics if tracking UA has been configured.
-            if ($googleTrackingCode = $this->settings->get('fof-analytics.googleTrackingCode')) {
+        $statusGoogle = (bool) $this->settings->get('fof-analytics.statusGoogle');
+        $googleTrackingCode = $this->settings->get('fof-analytics.googleTrackingCode');
+        $googleGTMCode = $this->settings->get('fof-analytics.googleGTMCode');
+        $optTrackingCode = $this->settings->get('fof-analytics.optTrackingCode');
+
+        if($statusGoogle) {
+            // Add google analytics if tracking UA only has been configured.
+            if ($googleTrackingCode && !$googleGTMCode && !$optTrackingCode) {
+                $js = file_get_contents(realpath(__DIR__ . '/../../resources/js/google-analytics.html'));
                 $js = str_replace("%%TRACKING_CODE%%", $googleTrackingCode, $js);
 
                 $document->payload['googleTrackingCode'] = $googleTrackingCode;
+                $document->head[] = $js;
             }
 
             // Add google tag manager if tracking GTM has been configured.
-            if ($googleGTMCode = $this->settings->get('fof-analytics.googleGTMCode')) {
+            if ($googleGTMCode) {
+                $js = file_get_contents(realpath(__DIR__ . '/../../resources/js/google-tag-manager.html'));
                 $js = str_replace("%%GTM_TRACKING_CODE%%", $googleGTMCode, $js);
-                $js = str_replace("%%TRACKING_CODE%%", $googleTrackingCode, $js);
 
                 $document->payload['googleGTMCode'] = $googleGTMCode;
+                $document->head[] = $js;
             }
 
-            if ($optTrackingCode = $this->settings->get('fof-analytics.optTrackingCode')) {
+            // Add google analytics with google optimize plugin if configured
+            if ($googleTrackingCode && $optTrackingCode) {
+                $js = file_get_contents(realpath(__DIR__ . '/../../resources/js/google-optimize.html'));
                 $js = str_replace("%%OPT_TRACKING_CODE%%", $optTrackingCode, $js);
+                $js = str_replace("%%TRACKING_CODE%%", $googleTrackingCode, $js);
 
                 $document->payload['optTrackingCode'] = $optTrackingCode;
+                $document->head[] = $js;
             }
-
-            $document->head[] = $js;
         }
     }