Matrix/element-web-Github
1
0
Fork 0
mirror of https://github.com/vector-im/element-web.git synced 2024-11-16 21:24:59 +08:00
Code Issues Packages Projects Releases Wiki Activity

Null check before accessing _paq

Browse Source
This commit is contained in:
Luke Barnard 2018-05-03 18:25:00 +01:00
parent 22f2d6f292
commit 3ba51ba695
1 changed files with 8 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
src/Analytics.js
View File

@ -164,13 +164,6 @@ class Analytics {
} }
trackPageChange(generationTimeMs) { trackPageChange(generationTimeMs) {
if (typeof generationTimeMs === 'number') {
this._paq.push(['setGenerationTimeMs', generationTimeMs]);
} else {
console.warn('Analytics.trackPageChange: expected generationTimeMs to be a number');
// But continue anyway because we still want to track the change
}
if (this.disabled) return; if (this.disabled) return;
if (this.firstPage) { if (this.firstPage) {
// De-duplicate first page // De-duplicate first page
@ -178,6 +171,14 @@ class Analytics {
this.firstPage = false; this.firstPage = false;
return; return;
} }
if (typeof generationTimeMs === 'number') {
this._paq.push(['setGenerationTimeMs', generationTimeMs]);
} else {
console.warn('Analytics.trackPageChange: expected generationTimeMs to be a number');
// But continue anyway because we still want to track the change
}
this._paq.push(['setCustomUrl', getRedactedUrl()]); this._paq.push(['setCustomUrl', getRedactedUrl()]);
this._paq.push(['trackPageView']); this._paq.push(['trackPageView']);
} }