272 move filepreview permissions

package.json

@@ -107,6 +107,7 @@
     "@auto-it/released": "10.46.0",
     "@csstools/postcss-cascade-layers": "3.0.1",
     "@cypress/code-coverage": "3.10.7",
+    "@cypress/grep": "^4.0.1",
     "@storybook/addon-a11y": "7.0.2",
     "@storybook/addon-essentials": "7.0.2",
     "@storybook/addon-interactions": "7.0.2",

src/files/domain/models/FilePreview.ts

@@ -1,4 +1,5 @@
 import FileTypeToFriendlyTypeMap from './FileTypeToFriendlyTypeMap'
+import { FilePermissions } from './File'
 
 export enum FileSizeUnit {
   BYTES = 'B',
@@ -182,6 +183,7 @@ export class FilePreview {
     public readonly isDeleted: boolean,
     public readonly ingest: FileIngest,
     public readonly downloadUrls: FileDownloadUrls,
+    public readonly permissions: FilePermissions,
     public thumbnail?: string,
     readonly directory?: string,
     readonly embargo?: FileEmbargo,

src/files/infrastructure/FileJSDataverseRepository.ts

@@ -6,6 +6,7 @@ import {
   File as JSFile,
   FileDataTable as JSFileTabularData,
   FileDownloadSizeMode,
+  FileUserPermissions as JSFileUserPermissions,
   getDatasetFileCounts,
   getDatasetFiles,
   getDatasetFilesTotalDownloadSize,
@@ -49,15 +50,17 @@ export class FileJSDataverseRepository implements FileRepository {
           jsFiles,
           FileJSDataverseRepository.getAllDownloadCount(jsFiles),
           FileJSDataverseRepository.getAllThumbnails(jsFiles),
+          FileJSDataverseRepository.getAllWithPermissions(jsFiles),
           FileJSDataverseRepository.getAllTabularData(jsFiles)
         ])
       )
-      .then(([jsFiles, downloadCounts, thumbnails, jsTabularData]) =>
+      .then(([jsFiles, downloadCounts, thumbnails, jsFileUserPermissions, jsTabularData]) =>
         jsFiles.map((jsFile, index) =>
           JSFileMapper.toFile(
             jsFile,
             datasetVersion,
             downloadCounts[index],
+            jsFileUserPermissions[index],
             thumbnails[index],
             jsTabularData[index]
           )
@@ -87,7 +90,15 @@ export class FileJSDataverseRepository implements FileRepository {
       )
     )
   }
+  private static getAllWithPermissions(files: JSFile[]): Promise<JSFileUserPermissions[]> {
+    return Promise.all(files.map((jsFile) => this.getFileUserPermissionById(jsFile.id)))
+  }
 
+  private static getFileUserPermissionById(id: number): Promise<JSFileUserPermissions> {
+    return getFileUserPermissions.execute(id).then((jsFileUserPermissions) => {
+      return jsFileUserPermissions
+    })
+  }
   private static getAllThumbnails(jsFiles: JSFile[]): Promise<(string | undefined)[]> {
     return Promise.all(jsFiles.map((jsFile) => this.getThumbnailById(jsFile.id)))
   }

src/files/infrastructure/mappers/JSFileMapper.ts

@@ -21,6 +21,7 @@ import {
   FileEmbargo as JSFileEmbargo,
   FileChecksum as JSFileChecksum,
   FileCounts as JSFilesCountInfo,
+  FileUserPermissions as JSFileUserPermissions,
   FileContentTypeCount as JSFileContentTypeCount,
   FileCategoryNameCount as JSFileCategoryNameCount,
   FileAccessStatusCount as JSFileAccessStatusCount,
@@ -36,12 +37,14 @@ import {
   FileTypeCount
 } from '../../domain/models/FilesCountInfo'
 import { FileAccessOption, FileTag } from '../../domain/models/FileCriteria'
+import { FilePermissions } from '../../domain/models/File'
 
 export class JSFileMapper {
   static toFile(
     jsFile: JSFile,
     datasetVersion: DatasetVersion,
     downloadsCount: number,
+    fileUserPermissions: JSFileUserPermissions,
     thumbnail?: string,
     jsTabularData?: JSFileTabularData[]
   ): FilePreview {
@@ -58,6 +61,7 @@ export class JSFileMapper {
       this.toFileIsDeleted(jsFile.deleted),
       { status: FileIngestStatus.NONE }, // TODO - Implement this when it is added to js-dataverse
       this.toFileOriginalFileDownloadUrl(jsFile.id),
+      this.toFilePermissions(fileUserPermissions),
       this.toFileThumbnail(thumbnail),
       this.toFileDirectory(jsFile.directoryLabel),
       this.toFileEmbargo(jsFile.embargo),
@@ -66,7 +70,14 @@ export class JSFileMapper {
       this.toFileChecksum(jsFile.checksum)
     )
   }
-
+  static toFilePermissions(jsFileUserPermissions: {
+    canDownloadFile: boolean
+    canEditOwnerDataset: boolean
+  }): FilePermissions {
+    return {
+      canDownloadFile: jsFileUserPermissions.canDownloadFile
+    }
+  }
   static toFileUserPermissions(
     jsFileId: number,
     jsFileUserPermissions: {

src/sections/dataset/DatasetFactory.tsx

@@ -8,7 +8,6 @@ import { FileJSDataverseRepository } from '../../files/infrastructure/FileJSData
 import { MetadataBlockInfoProvider } from './metadata-block-info/MetadataBlockProvider'
 import { MetadataBlockInfoJSDataverseRepository } from '../../metadata-block-info/infrastructure/repositories/MetadataBlockInfoJSDataverseRepository'
 import { SettingJSDataverseRepository } from '../../settings/infrastructure/SettingJSDataverseRepository'
-import { FilePermissionsProvider } from '../file/file-permissions/FilePermissionsProvider'
 import { SettingsProvider } from '../settings/SettingsProvider'
 import { DatasetProvider } from './DatasetProvider'
 import { MultipleFileDownloadProvider } from '../file/multiple-file-download/MultipleFileDownloadProvider'
@@ -24,19 +23,17 @@ export class DatasetFactory {
   static create(): ReactElement {
     return (
       <MultipleFileDownloadProvider repository={fileRepository}>
-        <FilePermissionsProvider repository={fileRepository}>
-          <SettingsProvider repository={settingRepository}>
-            <NotImplementedModalProvider>
-              <MetadataBlockInfoProvider repository={metadataBlockInfoRepository}>
-                <AnonymizedProvider>
-                  <AlertProvider>
-                    <DatasetWithSearchParams />
-                  </AlertProvider>
-                </AnonymizedProvider>
-              </MetadataBlockInfoProvider>
-            </NotImplementedModalProvider>
-          </SettingsProvider>
-        </FilePermissionsProvider>
+        <SettingsProvider repository={settingRepository}>
+          <NotImplementedModalProvider>
+            <MetadataBlockInfoProvider repository={metadataBlockInfoRepository}>
+              <AnonymizedProvider>
+                <AlertProvider>
+                  <DatasetWithSearchParams />
+                </AlertProvider>
+              </AnonymizedProvider>
+            </MetadataBlockInfoProvider>
+          </NotImplementedModalProvider>
+        </SettingsProvider>
       </MultipleFileDownloadProvider>
     )
   }

src/sections/dataset/dataset-files/files-table/file-actions/file-actions-cell/file-action-buttons/access-file-menu/AccessStatus.tsx

@@ -3,25 +3,22 @@ import { Globe, LockFill, UnlockFill } from 'react-bootstrap-icons'
 import { useTranslation } from 'react-i18next'
 import styles from './AccessFileMenu.module.scss'
 import { DropdownButtonItem } from '@iqss/dataverse-design-system'
-import { useFileDownloadPermission } from '../../../../../../../file/file-permissions/useFileDownloadPermission'
 
 interface AccessStatusProps {
   file: FilePreview
 }
 
 export function AccessStatus({ file }: AccessStatusProps) {
-  const { sessionUserHasFileDownloadPermission } = useFileDownloadPermission(file)
-
   return (
     <DropdownButtonItem disabled>
       <span>
         <AccessStatusIcon
-          sessionUserHasFileDownloadPermission={sessionUserHasFileDownloadPermission}
+          sessionUserHasFileDownloadPermission={file.permissions.canDownloadFile}
           restricted={file.access.restricted}
         />{' '}
         <AccessStatusText
           file={file}
-          sessionUserHasFileDownloadPermission={sessionUserHasFileDownloadPermission}
+          sessionUserHasFileDownloadPermission={file.permissions.canDownloadFile}
         />
       </span>
     </DropdownButtonItem>

src/sections/dataset/dataset-files/files-table/file-actions/file-actions-cell/file-action-buttons/access-file-menu/FileDownloadOptions.tsx

@@ -4,17 +4,15 @@ import { FilePreview } from '../../../../../../../../files/domain/models/FilePre
 import { FileTabularDownloadOptions } from './FileTabularDownloadOptions'
 import { FileNonTabularDownloadOptions } from './FileNonTabularDownloadOptions'
 import { useTranslation } from 'react-i18next'
-import { useFileDownloadPermission } from '../../../../../../../file/file-permissions/useFileDownloadPermission'
 
 interface FileDownloadOptionsProps {
   file: FilePreview
 }
 
 export function FileDownloadOptions({ file }: FileDownloadOptionsProps) {
   const { t } = useTranslation('files')
-  const { sessionUserHasFileDownloadPermission } = useFileDownloadPermission(file)
 
-  if (!sessionUserHasFileDownloadPermission) {
+  if (!file.permissions.canDownloadFile) {
     return <></>
   }
 

src/sections/dataset/dataset-files/files-table/file-actions/file-actions-cell/file-action-buttons/access-file-menu/RequestAccessOption.tsx

@@ -6,18 +6,15 @@ import {
   FilePublishingStatus
 } from '../../../../../../../../files/domain/models/FilePreview'
 import { useTranslation } from 'react-i18next'
-import { useFileDownloadPermission } from '../../../../../../../file/file-permissions/useFileDownloadPermission'
 
 interface RequestAccessButtonProps {
   file: FilePreview
 }
 export function RequestAccessOption({ file }: RequestAccessButtonProps) {
   const { t } = useTranslation('files')
-  const { sessionUserHasFileDownloadPermission } = useFileDownloadPermission(file)
-
   if (
     file.version.publishingStatus === FilePublishingStatus.DEACCESSIONED ||
-    sessionUserHasFileDownloadPermission
+    file.permissions.canDownloadFile
   ) {
     return <></>
   }

src/sections/dataset/dataset-files/files-table/file-info/file-info-cell/file-info-data/file-thumbnail/FileThumbnail.tsx

@@ -3,31 +3,28 @@ import { FileThumbnailPreviewImage } from './FileThumbnailPreviewImage'
 import { FilePreview } from '../../../../../../../../files/domain/models/FilePreview'
 import { FileAccessRestrictedIcon } from '../../../../../../../file/file-access/FileAccessRestrictedIcon'
 import styles from './FileThumbnail.module.scss'
-import { useFileDownloadPermission } from '../../../../../../../file/file-permissions/useFileDownloadPermission'
 
 interface FileThumbnailProps {
   file: FilePreview
 }
 
 export function FileThumbnail({ file }: FileThumbnailProps) {
-  const { sessionUserHasFileDownloadPermission } = useFileDownloadPermission(file)
-
   return (
     <div
       className={`${
-        file.thumbnail && sessionUserHasFileDownloadPermission
+        file.thumbnail && file.permissions.canDownloadFile
           ? styles['container-preview-image']
           : styles['container-icon']
       }`}>
-      {file.thumbnail && sessionUserHasFileDownloadPermission ? (
+      {file.thumbnail && file.permissions.canDownloadFile ? (
         <FileThumbnailPreviewImage thumbnail={file.thumbnail} name={file.name} />
       ) : (
         <FileThumbnailIcon type={file.type} />
       )}
       <div className={styles['restricted-icon']}>
         <FileAccessRestrictedIcon
           restricted={file.access.restricted}
-          canDownloadFile={sessionUserHasFileDownloadPermission}
+          canDownloadFile={file.permissions.canDownloadFile}
         />
       </div>
     </div>

src/sections/dataset/dataset-files/useFiles.tsx

@@ -5,8 +5,6 @@ import { getFilesByDatasetPersistentId } from '../../../files/domain/useCases/ge
 import { FileCriteria } from '../../../files/domain/models/FileCriteria'
 import { FilesCountInfo } from '../../../files/domain/models/FilesCountInfo'
 import { getFilesCountInfoByDatasetPersistentId } from '../../../files/domain/useCases/getFilesCountInfoByDatasetPersistentId'
-import { useFilePermissions } from '../../file/file-permissions/FilePermissionsContext'
-import { FilePermission } from '../../../files/domain/models/FileUserPermissions'
 import { DatasetVersion } from '../../../dataset/domain/models/Dataset'
 import { getFilesTotalDownloadSize } from '../../../files/domain/useCases/getFilesTotalDownloadSize'
 import { FilePaginationInfo } from '../../../files/domain/models/FilePaginationInfo'
@@ -19,7 +17,6 @@ export function useFiles(
   paginationInfo: FilePaginationInfo,
   criteria?: FileCriteria
 ) {
-  const { fetchFilesPermission } = useFilePermissions()
   const [files, setFiles] = useState<FilePreview[]>([])
   const [isLoading, setIsLoading] = useState<boolean>(true)
   const [filesCountInfo, setFilesCountInfo] = useState<FilesCountInfo>()
@@ -58,11 +55,8 @@ export function useFiles(
       )
         .then((files: FilePreview[]) => {
           setFiles(files)
-          return files
+          setIsLoading(false)
         })
-        .then((files: FilePreview[]) =>
-          fetchFilesPermission(FilePermission.DOWNLOAD_FILE, files).then(() => setIsLoading(false))
-        )
         .catch(() => {
           throw new Error('There was an error getting the files')
         })