Merge pull request #3136 from withspectrum/2.3.1

2.3.1

Author: brianlovin

api/queries/search/searchCommunities.js

@@ -12,14 +12,16 @@ export default (args: Args, { loaders, user }: GraphQLContext) => {
   return communitySearchIndex
     .search({ query: queryString })
     .then(content => {
-      trackQueue.add({
-        userId: user.id,
-        event: events.SEARCHED_COMMUNITIES,
-        properties: {
-          queryString,
-          hitsCount: content.hits ? content.hits.length : 0,
-        },
-      });
+      if (user && user.id) {
+        trackQueue.add({
+          userId: user.id,
+          event: events.SEARCHED_COMMUNITIES,
+          properties: {
+            queryString,
+            hitsCount: content.hits ? content.hits.length : 0,
+          },
+        });
+      }
 
       if (!content.hits || content.hits.length === 0) return [];
       const communityIds = content.hits.map(o => o.objectID);

api/queries/search/searchCommunityMembers.js

@@ -16,14 +16,16 @@ export default (args: Args, { loaders, user }: GraphQLContext) => {
   return usersSearchIndex
     .search({ query: queryString, hitsPerPage })
     .then(content => {
-      trackQueue.add({
-        userId: user.id,
-        event: events.SEARCHED_COMMUNITY_MEMBERS,
-        properties: {
-          queryString,
-          hitsCount: content.hits ? content.hits.length : 0,
-        },
-      });
+      if (user && user.id) {
+        trackQueue.add({
+          userId: user.id,
+          event: events.SEARCHED_COMMUNITY_MEMBERS,
+          properties: {
+            queryString,
+            hitsCount: content.hits ? content.hits.length : 0,
+          },
+        });
+      }
 
       if (!content.hits || content.hits.length === 0) return [];
       // if no search filter was passed, there's no way to be searching for

api/queries/search/searchThreads.js

@@ -38,15 +38,17 @@ export default async (args: Args, { loaders, user }: GraphQLContext) => {
     threadsSearchIndex
       .search({ query: queryString, filters })
       .then(content => {
-        trackQueue.add({
-          userId: user.id,
-          event: events.SEARCHED_CONVERSATIONS,
-          properties: {
-            queryString,
-            filters,
-            hitsCount: content.hits ? content.hits.length : 0,
-          },
-        });
+        if (user && user.id) {
+          trackQueue.add({
+            userId: user.id,
+            event: events.SEARCHED_CONVERSATIONS,
+            properties: {
+              queryString,
+              filters,
+              hitsCount: content.hits ? content.hits.length : 0,
+            },
+          });
+        }
 
         if (!content.hits || content.hits.length === 0) return null;
         return content.hits.map(o => ({

api/queries/search/searchUsers.js

@@ -21,14 +21,16 @@ export default (args: Args, { loaders, user }: GraphQLContext) => {
           ? events.SEARCHED_COMMUNITY_MEMBERS
           : events.SEARCHED_USERS;
 
-      trackQueue.add({
-        userId: user.id,
-        event,
-        properties: {
-          queryString,
-          hitsCount: content.hits ? content.hits.length : 0,
-        },
-      });
+      if (user && user.id) {
+        trackQueue.add({
+          userId: user.id,
+          event,
+          properties: {
+            queryString,
+            hitsCount: content.hits ? content.hits.length : 0,
+          },
+        });
+      }
 
       if (!content.hits || content.hits.length === 0) return [];
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "Spectrum",
-  "version": "2.3.0",
+  "version": "2.3.1",
   "license": "BSD-3-Clause",
   "devDependencies": {
     "babel-cli": "^6.24.1",