feat: Do not display share action

And share banner in a subfolder
cozy · Nov 29, 2024 · e6882d8 · e6882d8
1 parent a4882a1
commit e6882d8
Show file tree

Hide file tree

Showing 3 changed files with 25 additions and 16 deletions.
diff --git a/src/modules/public/PublicToolbar.jsx b/src/modules/public/PublicToolbar.jsx
@@ -12,8 +12,7 @@ const PublicToolbar = ({
   sharingInfos,
   className
 }) => {
-  const { loading, discoveryLink, sharing, isSharingShortcutCreated } =
-    sharingInfos
+  const { loading, discoveryLink } = sharingInfos
 
   if (loading) return null
   return (
@@ -28,12 +27,7 @@ const PublicToolbar = ({
           refreshFolderContent={refreshFolderContent}
         />
       ) : (
-        <PublicToolbarCozyToCozy
-          discoveryLink={discoveryLink}
-          files={files}
-          isSharingShortcutCreated={isSharingShortcutCreated}
-          sharing={sharing}
-        />
+        <PublicToolbarCozyToCozy files={files} sharingInfos={sharingInfos} />
       )}
     </div>
   )

diff --git a/src/modules/public/PublicToolbarCozyToCozy.jsx b/src/modules/public/PublicToolbarCozyToCozy.jsx
@@ -1,3 +1,4 @@
+import PropTypes from 'prop-types'
 import React from 'react'
 
 import { useClient } from 'cozy-client'
@@ -8,30 +9,34 @@ import useBreakpoints from 'cozy-ui/transpiled/react/providers/Breakpoints'
 import { useI18n } from 'cozy-ui/transpiled/react/providers/I18n'
 
 import { BarRightOnMobile } from 'components/Bar'
+import useCurrentFolderId from 'hooks/useCurrentFolderId'
 import { download, hr, openExternalLink, select } from 'modules/actions'
 import { DownloadFilesButton } from 'modules/public/DownloadFilesButton'
 import { OpenExternalLinkButton } from 'modules/public/OpenExternalLinkButton'
 import PublicToolbarMoreMenu from 'modules/public/PublicToolbarMoreMenu'
 import { useSelectionContext } from 'modules/selection/SelectionProvider'
 
-const PublicToolbarCozyToCozy = ({
-  isSharingShortcutCreated,
-  discoveryLink,
-  files
-}) => {
+const PublicToolbarCozyToCozy = ({ sharingInfos, files }) => {
+  const { loading, discoveryLink, sharing, isSharingShortcutCreated } =
+    sharingInfos
   const { isMobile } = useBreakpoints()
   const { t } = useI18n()
   const { showAlert } = useAlert()
   const client = useClient()
   const { showSelectionBar } = useSelectionContext()
   const vaultClient = useVaultClient()
+  const currentFolderId = useCurrentFolderId()
+
+  const isOnSharedFolder =
+    !loading &&
+    sharing?.rules?.some(rule => rule.values.includes(currentFolderId))
 
   const actions = makeActions(
     [
       isMobile && download,
       files.length > 1 && select,
       ((isMobile && files.length > 0) || files.length > 1) && hr,
-      openExternalLink
+      isOnSharedFolder && openExternalLink
     ],
     {
       t,
@@ -47,7 +52,7 @@ const PublicToolbarCozyToCozy = ({
   return (
     <BarRightOnMobile>
       {!isMobile && files.length > 0 && <DownloadFilesButton files={files} />}
-      {!isMobile && !isSharingShortcutCreated && (
+      {!isMobile && !isSharingShortcutCreated && isOnSharedFolder && (
         <OpenExternalLinkButton
           className="u-ml-half"
           link={discoveryLink}
@@ -63,4 +68,9 @@ const PublicToolbarCozyToCozy = ({
   )
 }
 
+PublicToolbarCozyToCozy.propTypes = {
+  files: PropTypes.array.isRequired,
+  sharingInfos: PropTypes.object.isRequired
+}
+
 export default PublicToolbarCozyToCozy
diff --git a/src/modules/views/Public/PublicFolderView.jsx b/src/modules/views/Public/PublicFolderView.jsx
@@ -87,6 +87,11 @@ const PublicFolderView = () => {
   const sharingInfos = useSharingInfos()
   const { showAlert } = useAlert()
   const styles = useFabStyles()
+  const isOnSharedFolder =
+    !sharingInfos.loading &&
+    sharingInfos.sharing?.rules?.some(rule =>
+      rule.values.includes(currentFolderId)
+    )
 
   const filesResult = usePublicFilesQuery(currentFolderId)
   const files = filesResult.data
@@ -172,7 +177,7 @@ const PublicFolderView = () => {
     <Main isPublic={true}>
       <ModalStack />
       <ModalManager />
-      {!isShareAlreadyAdded && <SharingBannerPlugin />}
+      {!isShareAlreadyAdded && isOnSharedFolder && <SharingBannerPlugin />}
       <span className={cx({ 'u-pt-2': !isMobile })} />
       <FolderViewHeader>
         {currentFolderId && (