Merge pull request #30 from chrisdmacrae/hotfix/#14

Hotfix/#14
chrisdmacrae · Jul 23, 2020 · 5ec1720 · 5ec1720
2 parents 1d6718d + c2a2ff8
commit 5ec1720
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/src/update.js b/src/update.js
@@ -1,10 +1,10 @@
+require("dotenv").config();
+
 var actionAdd = require("./utils/actionAdd")
 var actionUpdate = require("./utils/actionUpdate")
 var actionDelete = require("./utils/actionDelete")
 var algoliaSearch = require("algoliasearch")
 var chalk = require("chalk")
-var dotenv = require("dotenv").config()
-var fs = require("fs")
 var getLocalIndex = require("./utils/getLocalIndex")
 var getRemoteIndex = require("./utils/getRemoteIndex")
 var calculateOperations = require("./utils/calculateOperations")

diff --git a/src/utils/idsFromIndex.js b/src/utils/idsFromIndex.js
@@ -7,6 +7,10 @@ module.exports = function idsFromIndex(index) {
     if (hit.objectID !== null && hit.objectID !== undefined) {
         var id = String(hit.objectID)
         return hits.concat(id)
+    } else {
+      debug(`Missing object id, ignored hit: %o`, hit);
+
+      return hits;
     }
   }, []).sort()
 }