Merge pull request #8398 from readthedocs/fix-analytics-gtag

Only call analytics tracking of flyout when analytics are enabled

Author: ericholscher

readthedocs/core/static-src/core/js/doc-embed/sphinx.js

@@ -14,22 +14,25 @@ function init() {
     $(document).on('click', "[data-toggle='rst-current-version']", function () {
         var flyout_state = $("[data-toggle='rst-versions']").hasClass('shift-up') ? 'was_open' : 'was_closed';
 
-        // This needs to handle old style legacy analytics for previously built docs
-        // as well as the newer universal analytics and Google Site Tag
-        if (typeof gtag !== 'undefined') {
-            // https://developers.google.com/analytics/devguides/collection/gtagjs/events
-            gtag('event', 'Click', {
-                'event_category': 'Flyout',
-                'event_label': flyout_state,
-                'send_to': 'rtfd'
-            });
-        } else if (typeof ga !== 'undefined') {
-            ga('rtfd.send', 'event', 'Flyout', 'Click', flyout_state);
-        } else if (typeof _gaq !== 'undefined') {
-            _gaq.push(
-                ['rtfd._setAccount', 'UA-17997319-1'],
-                ['rtfd._trackEvent', 'Flyout', 'Click', flyout_state]
-            );
+        // Only report back if analytics is enabled
+        if (typeof READTHEDOCS_DATA !== 'undefined' && READTHEDOCS_DATA.global_analytics_code) {
+            // This needs to handle old style legacy analytics for previously built docs
+            // as well as the newer universal analytics and Google Site Tag
+            if (typeof gtag !== 'undefined') {
+                // https://developers.google.com/analytics/devguides/collection/gtagjs/events
+                gtag('event', 'Click', {
+                    'event_category': 'Flyout',
+                    'event_label': flyout_state,
+                    'send_to': 'rtfd'
+                });
+            } else if (typeof ga !== 'undefined') {
+                ga('rtfd.send', 'event', 'Flyout', 'Click', flyout_state);
+            } else if (typeof _gaq !== 'undefined') {
+                _gaq.push(
+                    ['rtfd._setAccount', 'UA-17997319-1'],
+                    ['rtfd._trackEvent', 'Flyout', 'Click', flyout_state]
+                );
+            }
         }
     });