IntenseWebs/mattermost
3
0
Fork 0
mirror of https://github.com/mattermost/mattermost.git synced 2025-02-25 18:55:24 -06:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #1 from mattermost/mm-1223

Browse Source 
fixes mm-1223 gracefully handle missing analytics configs
This commit is contained in:
Corey Hulen
2015-06-15 10:01:11 -08:00
parent b7a821c341 753b952bde
commit c068d40c17
2 changed files with 23 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
web/react/utils/client.jsx
View File

@@ -4,12 +4,12 @@
module.exports.track = function(category, action, label, prop, val) { module.exports.track = function(category, action, label, prop, val) {
global.window.snowplow('trackStructEvent', category, action, label, prop, val); global.window.snowplow('trackStructEvent', category, action, label, prop, val);
if (global.window.analytics != null) global.window.analytics.track(action, {category: category, label: label, property: prop, value: val}); global.window.analytics.track(action, {category: category, label: label, property: prop, value: val});
}; };
module.exports.trackPage = function() { module.exports.trackPage = function() {
global.window.snowplow('trackPageView'); global.window.snowplow('trackPageView');
if (global.window.analytics != null) global.window.analytics.page(); global.window.analytics.page();
}; };
function handleError(method_name, xhr, status, err) { function handleError(method_name, xhr, status, err) {

33
web/templates/head.html
View File

@@ -36,6 +36,7 @@
window._LTracker = _LTracker; window._LTracker = _LTracker;
_LTracker.push({'logglyKey': config.LogglyWriteKey, 'sendConsoleErrors' : config.LogglyConsoleErrors }); _LTracker.push({'logglyKey': config.LogglyWriteKey, 'sendConsoleErrors' : config.LogglyConsoleErrors });
} else { } else {
window._LTracker = [];
console.warn("config.js missing LogglyWriteKey, Loggly analytics is not reporting"); console.warn("config.js missing LogglyWriteKey, Loggly analytics is not reporting");
} }
</script> </script>
@@ -58,26 +59,34 @@
analytics.page(); analytics.page();
}}(); }}();
} else { } else {
analytics = {};
analytics.page = function(){};
analytics.track = function(){};
console.warn("config.js missing SegmentWriteKey, SegmentIO analytics is not tracking"); console.warn("config.js missing SegmentWriteKey, SegmentIO analytics is not tracking");
} }
</script> </script>
<!-- Snowplow starts plowing --> <!-- Snowplow starts plowing -->
<script type="text/javascript"> <script type="text/javascript">
;(function(p,l,o,w,i,n,g){if(!p[i]){p.GlobalSnowplowNamespace=p.GlobalSnowplowNamespace||[]; if ('{{ .Props.AnalyticsUrl }}'.trim() !== '') {
p.GlobalSnowplowNamespace.push(i);p[i]=function(){(p[i].q=p[i].q||[]).push(arguments) ;(function(p,l,o,w,i,n,g){if(!p[i]){p.GlobalSnowplowNamespace=p.GlobalSnowplowNamespace||[];
};p[i].q=p[i].q||[];n=l.createElement(o);g=l.getElementsByTagName(o)[0];n.async=1; p.GlobalSnowplowNamespace.push(i);p[i]=function(){(p[i].q=p[i].q||[]).push(arguments)
n.src=w;g.parentNode.insertBefore(n,g)}}(window,document,"script","//d1fc8wv8zag5ca.cloudfront.net/2.4.2/sp.js","snowplow")); };p[i].q=p[i].q||[];n=l.createElement(o);g=l.getElementsByTagName(o)[0];n.async=1;
n.src=w;g.parentNode.insertBefore(n,g)}}(window,document,"script","//d1fc8wv8zag5ca.cloudfront.net/2.4.2/sp.js","snowplow"));
window.snowplow('newTracker', 'cf', '{{ .Props.AnalyticsUrl }}', { window.snowplow('newTracker', 'cf', '{{ .Props.AnalyticsUrl }}', {
appId: '{{ .SiteName }}' appId: '{{ .SiteName }}'
}); });
var user = window.UserStore.getCurrentUser(true); var user = window.UserStore.getCurrentUser(true);
if (user) { if (user) {
window.snowplow('setUserId', user.id); window.snowplow('setUserId', user.id);
}
window.snowplow('trackPageView');
} else {
window.snowplow = function(){};
console.warn("config.json missing AnalyticsUrl, Snowplow analytics is not tracking");
} }
window.snowplow('trackPageView');
</script> </script>
<!-- Snowplow stops plowing --> <!-- Snowplow stops plowing -->
</head> </head>