@esri/hub-common 20.16.2 → 20.16.4

package/dist/esm/models/fetchModelFromItem.js

@@ -1,4 +1,5 @@
 import { getItemData } from "@esri/arcgis-rest-portal";
+import { includes } from "../utils";
 /**
  * Given an Item, fetch the data json and return an IModel
  * @param item
@@ -6,10 +7,25 @@ import { getItemData } from "@esri/arcgis-rest-portal";
  * @returns
  */
 export async function fetchModelFromItem(item, requestOptions) {
-    const data = await getItemData(item.id, requestOptions).catch(() => null);
+    const data = shouldFetchItemData(item)
+        ? (await getItemData(item.id, requestOptions).catch(() => null))
+        : null;
     return {
         item,
         data,
     };
 }
+/**
+ * Determine if we should attempt to fetch the item data
+ * @param item
+ * @returns
+ */
+export function shouldFetchItemData(item) {
+    // This function can and should be expanded as we discover
+    // more item types or families that should not attempt to
+    // fetch data for. Example: "Image Collection" will try
+    // to download the entire image collection to the browser
+    const typesToExclude = ["Image Collection"];
+    return !includes(typesToExclude, item.type);
+}
 //# sourceMappingURL=fetchModelFromItem.js.map

package/dist/esm/models/fetchModelFromItem.js.map

@@ -1 +1 @@
-{"version":3,"file":"fetchModelFromItem.js","sourceRoot":"","sources":["../../../src/models/fetchModelFromItem.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,WAAW,EAAS,MAAM,0BAA0B,CAAC;AAI9D;;;;;GAKG;AACH,MAAM,CAAC,KAAK,UAAU,kBAAkB,CACtC,IAAW,EACX,cAA+B;IAE/B,MAAM,IAAI,GAAG,MAAM,WAAW,CAAC,IAAI,CAAC,EAAE,EAAE,cAAc,CAAC,CAAC,KAAK,CAAC,GAAG,EAAE,CAAC,IAAI,CAAC,CAAC;IAC1E,OAAO;QACL,IAAI;QACJ,IAAI;KACK,CAAC;AACd,CAAC"}
+{"version":3,"file":"fetchModelFromItem.js","sourceRoot":"","sources":["../../../src/models/fetchModelFromItem.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,WAAW,EAAS,MAAM,0BAA0B,CAAC;AAG9D,OAAO,EAAE,QAAQ,EAAE,MAAM,UAAU,CAAC;AAEpC;;;;;GAKG;AACH,MAAM,CAAC,KAAK,UAAU,kBAAkB,CACtC,IAAW,EACX,cAA+B;IAE/B,MAAM,IAAI,GAAG,mBAAmB,CAAC,IAAI,CAAC;QACpC,CAAC,CAAE,CAAC,MAAM,WAAW,CAAC,IAAI,CAAC,EAAE,EAAE,cAAc,CAAC,CAAC,KAAK,CAAC,GAAG,EAAE,CAAC,IAAI,CAAC,CAG5D;QACJ,CAAC,CAAC,IAAI,CAAC;IACT,OAAO;QACL,IAAI;QACJ,IAAI;KACK,CAAC;AACd,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,mBAAmB,CAAC,IAAW;IAC7C,0DAA0D;IAC1D,yDAAyD;IACzD,uDAAuD;IACvD,yDAAyD;IACzD,MAAM,cAAc,GAAG,CAAC,kBAAkB,CAAC,CAAC;IAC5C,OAAO,CAAC,QAAQ,CAAC,cAAc,EAAE,IAAI,CAAC,IAAI,CAAC,CAAC;AAC9C,CAAC"}

package/dist/node/models/fetchModelFromItem.js

@@ -1,7 +1,8 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.fetchModelFromItem = void 0;
+exports.shouldFetchItemData = exports.fetchModelFromItem = void 0;
 const arcgis_rest_portal_1 = require("@esri/arcgis-rest-portal");
+const utils_1 = require("../utils");
 /**
  * Given an Item, fetch the data json and return an IModel
  * @param item
@@ -9,11 +10,27 @@ const arcgis_rest_portal_1 = require("@esri/arcgis-rest-portal");
  * @returns
  */
 async function fetchModelFromItem(item, requestOptions) {
-    const data = await arcgis_rest_portal_1.getItemData(item.id, requestOptions).catch(() => null);
+    const data = shouldFetchItemData(item)
+        ? (await arcgis_rest_portal_1.getItemData(item.id, requestOptions).catch(() => null))
+        : null;
     return {
         item,
         data,
     };
 }
 exports.fetchModelFromItem = fetchModelFromItem;
+/**
+ * Determine if we should attempt to fetch the item data
+ * @param item
+ * @returns
+ */
+function shouldFetchItemData(item) {
+    // This function can and should be expanded as we discover
+    // more item types or families that should not attempt to
+    // fetch data for. Example: "Image Collection" will try
+    // to download the entire image collection to the browser
+    const typesToExclude = ["Image Collection"];
+    return !utils_1.includes(typesToExclude, item.type);
+}
+exports.shouldFetchItemData = shouldFetchItemData;
 //# sourceMappingURL=fetchModelFromItem.js.map

package/dist/node/models/fetchModelFromItem.js.map

@@ -1 +1 @@
-{"version":3,"file":"fetchModelFromItem.js","sourceRoot":"","sources":["../../../src/models/fetchModelFromItem.ts"],"names":[],"mappings":";;;AAAA,iEAA8D;AAI9D;;;;;GAKG;AACI,KAAK,UAAU,kBAAkB,CACtC,IAAW,EACX,cAA+B;IAE/B,MAAM,IAAI,GAAG,MAAM,gCAAW,CAAC,IAAI,CAAC,EAAE,EAAE,cAAc,CAAC,CAAC,KAAK,CAAC,GAAG,EAAE,CAAC,IAAI,CAAC,CAAC;IAC1E,OAAO;QACL,IAAI;QACJ,IAAI;KACK,CAAC;AACd,CAAC;AATD,gDASC"}
+{"version":3,"file":"fetchModelFromItem.js","sourceRoot":"","sources":["../../../src/models/fetchModelFromItem.ts"],"names":[],"mappings":";;;AAAA,iEAA8D;AAG9D,oCAAoC;AAEpC;;;;;GAKG;AACI,KAAK,UAAU,kBAAkB,CACtC,IAAW,EACX,cAA+B;IAE/B,MAAM,IAAI,GAAG,mBAAmB,CAAC,IAAI,CAAC;QACpC,CAAC,CAAE,CAAC,MAAM,gCAAW,CAAC,IAAI,CAAC,EAAE,EAAE,cAAc,CAAC,CAAC,KAAK,CAAC,GAAG,EAAE,CAAC,IAAI,CAAC,CAG5D;QACJ,CAAC,CAAC,IAAI,CAAC;IACT,OAAO;QACL,IAAI;QACJ,IAAI;KACK,CAAC;AACd,CAAC;AAdD,gDAcC;AAED;;;;GAIG;AACH,SAAgB,mBAAmB,CAAC,IAAW;IAC7C,0DAA0D;IAC1D,yDAAyD;IACzD,uDAAuD;IACvD,yDAAyD;IACzD,MAAM,cAAc,GAAG,CAAC,kBAAkB,CAAC,CAAC;IAC5C,OAAO,CAAC,gBAAQ,CAAC,cAAc,EAAE,IAAI,CAAC,IAAI,CAAC,CAAC;AAC9C,CAAC;AAPD,kDAOC"}

package/dist/types/core/_internal/createOrUpdateEntitySettings.d.ts

@@ -1,4 +1,4 @@
-import { Polygon } from "geojson";
+import type { Polygon } from "geojson";
 import { IHubRequestOptions } from "../../hub-types";
 import { IEntitySetting } from "../../discussions/api/types";
 import { IWithDiscussions } from "../traits/IWithDiscussions";

package/dist/types/discussions/api/types.d.ts

@@ -1,5 +1,5 @@
 import { IPagingParams, IPagedResponse as IRestPagedResponse, IUser } from "@esri/arcgis-rest-portal";
-import { Geometry, Polygon } from "geojson";
+import type { Geometry, Polygon } from "geojson";
 import { IHubRequestOptions } from "../../hub-types";
 /**
  * sort orders

package/dist/types/extent.d.ts

@@ -1,7 +1,7 @@
 import type { IExtent } from "@esri/arcgis-rest-feature-service";
 import type { IPoint, IPolygon } from "@esri/arcgis-rest-request";
 import { IHubRequestOptions, BBox } from "./hub-types";
-import { Polygon } from "geojson";
+import type { Polygon } from "geojson";
 /**
  * Turns an bounding box coordinate array into an extent object
  * @param bBox bounding box coordinate array

package/dist/types/models/fetchModelFromItem.d.ts

@@ -8,3 +8,9 @@ import { IModel } from "../hub-types";
  * @returns
  */
 export declare function fetchModelFromItem(item: IItem, requestOptions: IRequestOptions): Promise<IModel>;
+/**
+ * Determine if we should attempt to fetch the item data
+ * @param item
+ * @returns
+ */
+export declare function shouldFetchItemData(item: IItem): boolean;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@esri/hub-common",
-  "version": "20.16.2",
+  "version": "20.16.4",
   "description": "Common TypeScript types and utility functions for @esri/hub.js.",
   "main": "dist/node/index.js",
   "module": "dist/esm/index.js",
@@ -17,6 +17,7 @@
     "adlib": "^3.0.8",
     "ajv": "^6.12.6",
     "fast-xml-parser": "^3.21.0",
+    "geojson": "^0.5.0",
     "json-schema-typed": "^7.0.3",
     "jsonapi-typescript": "^0.1.3",
     "tslib": "^1.13.0"