From e32ff75d3a244391e6f57f8688f538f3395f29d4 Mon Sep 17 00:00:00 2001
From: Paul Hayes <fofr@users.noreply.github.com>
Date: Mon, 2 Mar 2015 15:13:04 +0000
Subject: [PATCH] Don't minify analytics initialisers in template
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

In IE8 and below the minified inline javascript was inadvertently
wiping out the GOVUK object, causing JS errors.

We aren’t sure exactly why it was wiping out the object, but by
removing `var` and referring to window the error is eliminated.
---
 app/views/root/_google_analytics.html.erb | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/app/views/root/_google_analytics.html.erb b/app/views/root/_google_analytics.html.erb
index 95b5df949..7434c4fa9 100644
--- a/app/views/root/_google_analytics.html.erb
+++ b/app/views/root/_google_analytics.html.erb
@@ -1,5 +1,7 @@
 <script id="ga-params">
-  var GOVUK=GOVUK||{},_gaq=_gaq||[];GOVUK.Analytics=GOVUK.Analytics||{};
+  window.GOVUK = window.GOVUK || {};
+  window._gaq = window._gaq || [];
+  GOVUK.Analytics = GOVUK.Analytics || {};
   <%#
     Slimmer inserts custom variables here
     https://github.com/alphagov/slimmer/blob/master/lib/slimmer/processors/google_analytics_configurator.rb