Merge pull request #609 from Esri/bug/item-access

fix(portal): setting item access to public shares correctly
Esri · Aug 8, 2019 · 025406e · 025406e
2 parents 1635eb2 + 6a2b115
commit 025406e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/packages/arcgis-rest-portal/src/sharing/access.ts b/packages/arcgis-rest-portal/src/sharing/access.ts
@@ -77,7 +77,9 @@ function updateItemAccess(
   }
   // if sharing with everyone, share with the entire organization as well.
   if (requestOptions.access === "public") {
-    requestOptions.params.org = true;
+    // this is how the ArcGIS Online Home app sets public access
+    // setting org = true instead of account = true will cancel out all sharing
+    requestOptions.params.account = true;
     requestOptions.params.everyone = true;
   }
   return request(url, requestOptions);

diff --git a/packages/arcgis-rest-portal/test/sharing/access.test.ts b/packages/arcgis-rest-portal/test/sharing/access.test.ts
@@ -47,7 +47,7 @@ describe("setItemAccess()", () => {
         expect(response).toEqual(SharingResponse);
         expect(options.body).toContain("f=json");
         expect(options.body).toContain("everyone=true");
-        expect(options.body).toContain("org=true");
+        expect(options.body).toContain("account=true");
         done();
       })
       .catch(e => {