review feedback

Author: nreese

x-pack/plugins/maps/public/actions/map_actions.js

@@ -587,8 +587,8 @@ export function fitToDataBounds() {
       const b = bounds[i];
 
       //filter out undefined bounds (uses Infinity due to turf responses)
-
       if (
+        b === null ||
         b.minLon === Infinity ||
         b.maxLon === Infinity ||
         b.minLat === -Infinity ||

x-pack/plugins/maps/public/classes/sources/es_source/es_source.js

@@ -161,16 +161,15 @@ export class AbstractESSource extends AbstractVectorSource {
     let esBounds;
     try {
       const esResp = await searchSource.fetch();
-      esBounds = _.get(esResp, 'aggregations.fitToBounds.bounds');
+      if (!esResp.aggregations.fitToBounds.bounds) {
+        // aggregations.fitToBounds is empty object when there are no matching documents
+        return null;
+      }
+      esBounds = esResp.aggregations.fitToBounds.bounds;
     } catch (error) {
       return null;
     }
 
-    // aggregations.fitToBounds is empty object when there are no matching documents
-    if (!esBounds) {
-      return null;
-    }
-
     const minLon = esBounds.top_left.lon;
     const maxLon = esBounds.bottom_right.lon;
     return {