short circuit searchEvents() if there are no sites to search

Esri · Feb 22, 2019 · 371e642 · 371e642
1 parent 7785f76
commit 371e642
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/packages/events/src/search.ts b/packages/events/src/search.ts
@@ -97,6 +97,7 @@ function formatEventResponse(
 ) {
   const siteIds: string[] = [];
   const data: IEventResourceObject[] = [];
+  const included: IEventResourceObject[] = [];
   const cacheBust = new Date().getTime();
   let siteSearchQuery = "";
 
@@ -128,14 +129,15 @@ function formatEventResponse(
       siteSearchQuery += attributes.siteId;
     }
   });
-
+  if (siteIds.length === 0) {
+    return { included, data };
+  }
+  // search for site items and include those in the response
   const searchRequestOptions = requestOptions as ISearchRequestOptions;
   searchRequestOptions.searchForm = {
     q: siteSearchQuery
   };
   return searchItems(searchRequestOptions).then(function(siteInfo) {
-    const included: IEventResourceObject[] = [];
-
     siteInfo.results.forEach(siteItem => {
       included.push({
         id: siteItem.id,