fix(getItemApiUrl): fix missing ?f=json

AFFECTS PACKAGES: @esri/hub-common ISSUES CLOSED: #330
Esri · Aug 25, 2020 · 083ee02 · 083ee02
1 parent 1a250d8
commit 083ee02
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 12 deletions.
diff --git a/packages/common/src/urls/get-item-api-url.ts b/packages/common/src/urls/get-item-api-url.ts
@@ -18,15 +18,9 @@ export const getItemApiUrl = (
     typeof portalOrUrl === "string" ? portalOrUrl : getPortalUrl(portalOrUrl);
   // TODO: handle case where Portal API url is passed in?
   // const portalUrl = parsePortalUrl(portalRestUrl);
-  const path = `/sharing/rest/content/items/${id}`;
-  // TODO: append f param based on item.type?
-  let query;
+  let url = `${host}/sharing/rest/content/items/${id}?f=json`;
   if (access !== "public") {
-    query = { token };
+    url = `${url}&token=${token}`;
   }
-  return buildUrl({
-    host,
-    path,
-    query
-  });
+  return url;
 };
diff --git a/packages/common/src/urls/get-item-data-url.ts b/packages/common/src/urls/get-item-data-url.ts
@@ -13,7 +13,9 @@ export const getItemDataUrl = (
   token?: string
 ) => {
   const url = getItemApiUrl(item, portalOrUrl, token);
-  const segment = `/${item.id}`;
-  const regExp = new RegExp(segment);
-  return url && url.replace(regExp, `${segment}/data`);
+  const pattern = `\\/${item.id}\\?f=json`;
+  const regExp = new RegExp(pattern);
+  return (
+    url && url.replace(regExp, `/${item.id}/data`).replace(/\&token/, "?token")
+  );
 };