Adds null checks (#23)

brave-intl · Feb 28, 2018 · 13686d8 · 13686d8
1 parent d512f6c
commit 13686d8
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/index.js b/index.js
@@ -325,12 +325,14 @@ Synopsis.prototype._topN = function (n, scorekeeper, allP) {
 
   results = []
   underscore.keys(this.publishers).forEach(function (publisher) {
-    if (!this.publishers[publisher].scores[scorekeeper]) return
+    if (
+      !this.publishers[publisher].scores ||
+      !this.publishers[publisher].scores[scorekeeper]) return
 
-    if (!this.publishers[publisher].options.stickyP) {
+    if (!this.publishers[publisher].options || !this.publishers[publisher].options.stickyP) {
       if ((!allP) &&
             ((this.options.minPublisherDuration > this.publishers[publisher].duration) ||
-             (this.options.minPublisherVisits > this.publishers[publisher].vists))) return
+             (this.options.minPublisherVisits > this.publishers[publisher].visits))) return
     }
 
     results.push(underscore.extend({ publisher: publisher }, underscore.omit(this.publishers[publisher], 'window')))

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "bat-publisher",
-  "version": "2.0.12",
+  "version": "2.0.13",
   "description": "Routines to identify publishers for the BAT.",
   "main": "index.js",
   "scripts": {