synapse-react-client 3.3.0 → 3.3.1

package/dist/umd/synapse-react-client.development.cjs

@@ -44887,9 +44887,17 @@ Please use another name.` : formatMuiErrorMessage$1(18));
     ).length > 0;
     return hasFacetFilters || hasAdditionalFilters;
   }
-  function canTableQueryBeAddedToDownloadList(entity2) {
+  function canTableQueryBeAddedToDownloadList(entity2, entityColumnId) {
+    const viewCannotIncludeFiles = (
+      // EntityViews without the file bit mask cannot contain files
+      entity2 && isEntityView(entity2) && !hasFilesInView(entity2) || // DatasetCollections cannot contain files
+      entity2 && isDatasetCollection(entity2)
+    );
+    if (viewCannotIncludeFiles) {
+      return false;
+    }
     return Boolean(
-      entity2 && (isEntityView(entity2) && isFileView(entity2) || isDataset(entity2))
+      entityColumnId || entity2 && (isEntityView(entity2) && isFileView(entity2) || isDataset(entity2))
     );
   }
   function queryRequestsHaveSameTotalResults(request1, request2) {
@@ -55718,6 +55726,7 @@ Please use another name.` : formatMuiErrorMessage$1(18));
     return /* @__PURE__ */ jsxRuntimeExports.jsx(
       "div",
       {
+        role: "progressbar",
         className: "spinner",
         style: {
           height: `${size2}px`,
@@ -116708,8 +116717,10 @@ query = syn.tableQuery("${clientSql}")${"\n"}query.asDataFrame()`
     const [showProgrammaticOptions, setShowProgrammaticOptions] = React$2.useState(false);
     const { onDownloadFiles, darkTheme = true } = props;
     const fileColumnId = getFileColumnModelId(queryResultBundle == null ? void 0 : queryResultBundle.columnModels);
-    const isEntityViewWithoutFiles = entity2 && isEntityView(entity2) && !hasFilesInView(entity2);
-    const showAddQueryToDownloadList = !isEntityViewWithoutFiles && (fileColumnId || canTableQueryBeAddedToDownloadList(entity2));
+    const showAddQueryToDownloadList = canTableQueryBeAddedToDownloadList(
+      entity2,
+      fileColumnId
+    );
     const disableDownload = entity2 && isDataset(entity2) && entity2.isLatestVersion;
     return /* @__PURE__ */ jsxRuntimeExports.jsxs(React$2.Fragment, { children: [
       /* @__PURE__ */ jsxRuntimeExports.jsxs(ReactBootstrap.Dropdown, { as: "span", children: [
@@ -117093,7 +117104,10 @@ query = syn.tableQuery("${clientSql}")${"\n"}query.asDataFrame()`
       }
     };
     const fileColumnId = getFileColumnModelId(data == null ? void 0 : data.columnModels);
-    const showAddToDownloadCart = fileColumnId ?? canTableQueryBeAddedToDownloadList(entity2);
+    const showAddToDownloadCart = canTableQueryBeAddedToDownloadList(
+      entity2,
+      fileColumnId
+    );
     return /* @__PURE__ */ jsxRuntimeExports.jsx(
       RowSelectionUI,
       {
@@ -174009,6 +174023,53 @@ dl_list_file_entities = syn.get_download_list()`;
       }
     );
   }
+  const PRINCIPAL_ALREADY_ADDED_ERROR_MESSAGE = "User or team already has permissions.";
+  function useUpdateAcl(options2) {
+    const { onChange = noop$9, onError = noop$9 } = options2;
+    const [resourceAccessList, setResourceAccessList] = React$2.useState([]);
+    React$2.useEffect(() => {
+      onChange(resourceAccessList);
+    }, [resourceAccessList]);
+    const addResourceAccessItem = (newReviewerId) => {
+      if (newReviewerId) {
+        const alreadyReviewer = resourceAccessList.some(
+          (resourceAccess) => resourceAccess.principalId === Number(newReviewerId)
+        );
+        if (alreadyReviewer) {
+          onError(PRINCIPAL_ALREADY_ADDED_ERROR_MESSAGE);
+        } else {
+          const newResourceAccess = {
+            principalId: Number(newReviewerId),
+            accessType: [ACCESS_TYPE.REVIEW_SUBMISSIONS]
+          };
+          const updatedResourceAccessList = [
+            ...resourceAccessList,
+            newResourceAccess
+          ];
+          setResourceAccessList(updatedResourceAccessList);
+        }
+      }
+    };
+    const updateResourceAccessItem = (principalId, accessType) => {
+      const updatedResourceAccessList = resourceAccessList.map((resourceAccess) => {
+        return resourceAccess.principalId === principalId ? { ...resourceAccess, accessType } : resourceAccess;
+      });
+      setResourceAccessList(updatedResourceAccessList);
+    };
+    const removeResourceAccessItem = (principalId) => {
+      const updatedResourceAccessList = resourceAccessList.filter(
+        (raListItem) => raListItem.principalId !== principalId
+      );
+      setResourceAccessList(updatedResourceAccessList);
+    };
+    return {
+      resourceAccessList,
+      setResourceAccessList,
+      addResourceAccessItem,
+      updateResourceAccessItem,
+      removeResourceAccessItem
+    };
+  }
   const PermissionLevelMenu = (props) => {
     const { currentAccessType, availablePermissionLevels: availablePermissionLevels2, onChange } = props;
     return /* @__PURE__ */ jsxRuntimeExports.jsx(
@@ -174039,13 +174100,8 @@ dl_list_file_entities = syn.get_download_list()`;
     );
   };
   const REMOVE_BUTTON_LABEL = "Remove from AR Permissions";
-  const availablePermissionLevels = [
-    "CAN_REVIEW_SUBMISSIONS",
-    "IS_EXEMPTION_ELIGIBLE",
-    "CAN_REVIEW_SUBMISSIONS_AND_IS_EXEMPTION_ELIGIBLE"
-  ];
   const ResourceAccessItem = (props) => {
-    const { resourceAccess, onChange, onRemove: onRemove2 } = props;
+    const { resourceAccess, availablePermissionLevels: availablePermissionLevels2, onChange, onRemove: onRemove2 } = props;
     return /* @__PURE__ */ jsxRuntimeExports.jsx(jsxRuntimeExports.Fragment, { children: /* @__PURE__ */ jsxRuntimeExports.jsxs(
       Stack$6,
       {
@@ -174062,7 +174118,7 @@ dl_list_file_entities = syn.get_download_list()`;
               PermissionLevelMenu,
               {
                 currentAccessType: resourceAccess.accessType,
-                availablePermissionLevels,
+                availablePermissionLevels: availablePermissionLevels2,
                 onChange
               }
             ),
@@ -174084,18 +174140,91 @@ dl_list_file_entities = syn.get_download_list()`;
       }
     ) });
   };
+  function AclEditor(props) {
+    const {
+      isLoading,
+      resourceAccessList,
+      availablePermissionLevels: availablePermissionLevels2,
+      emptyText,
+      addResourceAccessItem,
+      updateResourceAccessItem,
+      removeResourceAccessItem
+    } = props;
+    if (isLoading) {
+      return /* @__PURE__ */ jsxRuntimeExports.jsx(Box, { display: "flex", justifyContent: "center", height: "150px", children: /* @__PURE__ */ jsxRuntimeExports.jsx(SynapseSpinner, { size: 50 }) });
+    }
+    return /* @__PURE__ */ jsxRuntimeExports.jsxs(Box, { children: [
+      /* @__PURE__ */ jsxRuntimeExports.jsxs(Box, { mb: "30px", children: [
+        /* @__PURE__ */ jsxRuntimeExports.jsx(Typography, { variant: "headline3", mb: "10px", children: "Users and Teams with Permissions" }),
+        resourceAccessList.length === 0 ? /* @__PURE__ */ jsxRuntimeExports.jsx(Typography, { variant: "body1Italic", children: emptyText }) : /* @__PURE__ */ jsxRuntimeExports.jsx(TransitionGroup, { children: resourceAccessList.map((resourceAccess) => {
+          return /* @__PURE__ */ jsxRuntimeExports.jsx(Collapse$1, { children: /* @__PURE__ */ jsxRuntimeExports.jsx(
+            ResourceAccessItem,
+            {
+              resourceAccess,
+              availablePermissionLevels: availablePermissionLevels2,
+              onChange: (accessType) => updateResourceAccessItem(
+                resourceAccess.principalId,
+                accessType
+              ),
+              onRemove: () => removeResourceAccessItem(resourceAccess.principalId)
+            },
+            resourceAccess.principalId
+          ) }, resourceAccess.principalId);
+        }) })
+      ] }),
+      /* @__PURE__ */ jsxRuntimeExports.jsxs(Box, { children: [
+        /* @__PURE__ */ jsxRuntimeExports.jsx(Typography, { variant: "headline3", mb: "10px", children: "Add More" }),
+        /* @__PURE__ */ jsxRuntimeExports.jsx(
+          Typography,
+          {
+            sx: {
+              variant: "body1",
+              lineHeight: "20px",
+              fontStyle: "italic",
+              color: "grey.900"
+            },
+            mb: "20px",
+            children: "Search for a username or team to add. You can search by username, first or last names, or team name"
+          }
+        ),
+        /* @__PURE__ */ jsxRuntimeExports.jsxs(Box, { children: [
+          /* @__PURE__ */ jsxRuntimeExports.jsx(
+            Typography,
+            {
+              component: "label",
+              variant: "smallText2",
+              htmlFor: "reviewer-search",
+              children: "Add a user or team"
+            }
+          ),
+          /* @__PURE__ */ jsxRuntimeExports.jsx(
+            UserSearchBoxV2,
+            {
+              value: null,
+              inputId: "reviewer-search",
+              placeholder: "Username, name (first and last) or team name.",
+              onChange: (id2) => addResourceAccessItem(id2)
+            }
+          )
+        ] })
+      ] })
+    ] });
+  }
   const textSx = {
     variant: "body1",
     lineHeight: "20px",
     color: "grey.800"
   };
   const EMPTY_RESOURCE_ACCESS_LIST_TEXT = "Only ACT has permissions on this AR.";
-  const REVIEWER_ALREADY_ADDED_ERROR_MESSAGE = "User or team already has permissions on this AR.";
+  const availablePermissionLevels = [
+    "CAN_REVIEW_SUBMISSIONS",
+    "IS_EXEMPTION_ELIGIBLE",
+    "CAN_REVIEW_SUBMISSIONS_AND_IS_EXEMPTION_ELIGIBLE"
+  ];
   const AccessRequirementAclEditor = React$2.forwardRef(
     function AccessRequirementAclEditor2(props, ref2) {
       const { accessRequirementId, onSaveComplete } = props;
       const [error2, setError] = React$2.useState(null);
-      const [resourceAccessList, setResourceAccessList] = React$2.useState([]);
       const onMutationSuccess = () => {
         setError(null);
         onSaveComplete(true);
@@ -174108,11 +174237,21 @@ dl_list_file_entities = syn.get_download_list()`;
         // Infinite staleTime ensures this won't get re-fetched unless explicitly invalidated by the mutation
         staleTime: Infinity
       });
+      const {
+        resourceAccessList,
+        setResourceAccessList,
+        addResourceAccessItem,
+        updateResourceAccessItem,
+        removeResourceAccessItem
+      } = useUpdateAcl({
+        onChange: () => setError(null),
+        onError: setError
+      });
       React$2.useEffect(() => {
         if (originalAcl) {
           setResourceAccessList(originalAcl.resourceAccess);
         }
-      }, [originalAcl]);
+      }, [originalAcl, setResourceAccessList]);
       const { mutate: deleteAcl } = useDeleteAccessRequirementACL({
         onSuccess: () => onMutationSuccess(),
         onError: (error22) => onMutationError(error22)
@@ -174159,42 +174298,6 @@ dl_list_file_entities = syn.get_download_list()`;
           updateAcl
         ]
       );
-      const addResourceAccessItem = (newReviewerId) => {
-        if (newReviewerId) {
-          const alreadyReviewer = resourceAccessList.some(
-            (resourceAccess) => resourceAccess.principalId === Number(newReviewerId)
-          );
-          if (alreadyReviewer) {
-            setError(REVIEWER_ALREADY_ADDED_ERROR_MESSAGE);
-          } else {
-            const newResourceAccess = {
-              principalId: Number(newReviewerId),
-              accessType: [ACCESS_TYPE.REVIEW_SUBMISSIONS]
-            };
-            const updatedResourceAccessList = [
-              ...resourceAccessList,
-              newResourceAccess
-            ];
-            setResourceAccessList(updatedResourceAccessList);
-          }
-        } else {
-          setError(null);
-        }
-      };
-      const updateResourceAccessItem = (principalId, accessType) => {
-        const updatedResourceAccessList = resourceAccessList.map(
-          (resourceAccess) => {
-            return resourceAccess.principalId === principalId ? { ...resourceAccess, accessType } : resourceAccess;
-          }
-        );
-        setResourceAccessList(updatedResourceAccessList);
-      };
-      const removeResourceAccessItem = (principalId) => {
-        const updatedResourceAccessList = resourceAccessList.filter(
-          (raListItem) => raListItem.principalId !== principalId
-        );
-        setResourceAccessList(updatedResourceAccessList);
-      };
       return /* @__PURE__ */ jsxRuntimeExports.jsxs(Stack$6, { gap: "20px", direction: "column", children: [
         /* @__PURE__ */ jsxRuntimeExports.jsxs(Box, { children: [
           /* @__PURE__ */ jsxRuntimeExports.jsx(Typography, { variant: "headline3", mb: "10px", children: "Guide to AR permissions" }),
@@ -174208,56 +174311,18 @@ dl_list_file_entities = syn.get_download_list()`;
           ] }),
           /* @__PURE__ */ jsxRuntimeExports.jsx(Typography, { sx: { ...textSx, color: "grey.900" }, children: "ACT members always retain the ability to review or modify Access Requirements." })
         ] }),
-        /* @__PURE__ */ jsxRuntimeExports.jsxs(Box, { children: [
-          /* @__PURE__ */ jsxRuntimeExports.jsxs(Box, { mb: "30px", children: [
-            /* @__PURE__ */ jsxRuntimeExports.jsx(Typography, { variant: "headline3", mb: "10px", children: "Users and Teams with Permissions" }),
-            isLoadingOriginalAcl && /* @__PURE__ */ jsxRuntimeExports.jsx(Skeleton, { variant: "rectangular", width: 250, height: 24 }),
-            !isLoadingOriginalAcl && resourceAccessList.length === 0 ? /* @__PURE__ */ jsxRuntimeExports.jsx(Typography, { variant: "body1Italic", children: EMPTY_RESOURCE_ACCESS_LIST_TEXT }) : /* @__PURE__ */ jsxRuntimeExports.jsx(jsxRuntimeExports.Fragment, { children: resourceAccessList.map((resourceAccess) => {
-              return /* @__PURE__ */ jsxRuntimeExports.jsx(
-                ResourceAccessItem,
-                {
-                  resourceAccess,
-                  onChange: (accessType) => updateResourceAccessItem(
-                    resourceAccess.principalId,
-                    accessType
-                  ),
-                  onRemove: () => removeResourceAccessItem(resourceAccess.principalId)
-                },
-                resourceAccess.principalId
-              );
-            }) })
-          ] }),
-          /* @__PURE__ */ jsxRuntimeExports.jsxs(Box, { children: [
-            /* @__PURE__ */ jsxRuntimeExports.jsx(Typography, { variant: "headline3", mb: "10px", children: "Add More" }),
-            /* @__PURE__ */ jsxRuntimeExports.jsx(
-              Typography,
-              {
-                sx: { ...textSx, fontStyle: "italic", color: "grey.900" },
-                mb: "20px",
-                children: "Search for a username or team to add. You can search by username, first or last names, or team name"
-              }
-            ),
-            /* @__PURE__ */ jsxRuntimeExports.jsxs(Box, { children: [
-              /* @__PURE__ */ jsxRuntimeExports.jsx(
-                Typography,
-                {
-                  component: "label",
-                  variant: "smallText2",
-                  htmlFor: "reviewer-search",
-                  children: "Select a reviewer"
-                }
-              ),
-              /* @__PURE__ */ jsxRuntimeExports.jsx(
-                UserSearchBoxV2,
-                {
-                  inputId: "reviewer-search",
-                  placeholder: "Username, name (first and last) or team name.",
-                  onChange: addResourceAccessItem
-                }
-              )
-            ] })
-          ] })
-        ] }),
+        /* @__PURE__ */ jsxRuntimeExports.jsx(
+          AclEditor,
+          {
+            resourceAccessList,
+            availablePermissionLevels,
+            isLoading: isLoadingOriginalAcl,
+            emptyText: EMPTY_RESOURCE_ACCESS_LIST_TEXT,
+            addResourceAccessItem,
+            updateResourceAccessItem,
+            removeResourceAccessItem
+          }
+        ),
         error2 && /* @__PURE__ */ jsxRuntimeExports.jsx(Alert, { severity: "error", children: error2 })
       ] });
     }