Merge pull request #8167 from ruKurz/bugfix-8166

[FIX] Issue #8166 where empty analytics setting breaks to load Piwik script

packages/rocketchat-analytics/client/loadScript.js

@@ -38,10 +38,18 @@ Template.body.onRendered(() => {
 					window._paq.push(['setDomains', domains]);
 				}
 				(() => {
-					const addTrackers = JSON.parse(piwikAdditionalTracker);
-					for (let i = 0; i < addTrackers.length; i++) {
-						const tracker = addTrackers[i];
-						window._paq.push(['addTracker', `${ tracker['trackerURL'] }piwik.php`, tracker['siteId']]);
+					try {
+						if (/\S/.test(piwikAdditionalTracker)) {
+							// piwikAdditionalTracker is not empty or whitespace only
+							const addTrackers = JSON.parse(piwikAdditionalTracker);
+							for (let i = 0; i < addTrackers.length; i++) {
+								const tracker = addTrackers[i];
+								window._paq.push(['addTracker', `${ tracker['trackerURL'] }piwik.php`, tracker['siteId']]);
+							}
+						}
+					} catch (e) {
+						// parsing JSON faild
+						console.log('Error while parsing JSON value of "piwikAdditionalTracker": ', e);
 					}
 					window._paq.push(['setTrackerUrl', `${ piwikUrl }piwik.php`]);
 					window._paq.push(['setSiteId', Number.parseInt(piwikSiteId)]);