Show Export Metadata for the latest published dataset version and Draft version

CHANGELOG.md

@@ -13,6 +13,8 @@ This changelog follows the principles of [Keep a Changelog](https://keepachangel
 
 ### Changed
 
+- Hide "Export Metadata" on file pages that are not for the latest released dataset version.
+
 ### Fixed
 
 ### Removed

src/dataset/infrastructure/mappers/JSDatasetVersionMapper.ts

@@ -18,15 +18,16 @@ export class JSDatasetVersionMapper {
     jsDatasetLastUpdateTime: string,
     jsDatasetPublicationDate?: string,
     jsDatasettermsOfAccess?: TermsOfAccess,
-    jsDatasetDeaccessionedNote?: string
+    jsDatasetDeaccessionedNote?: string,
+    isLatest = true
   ): DatasetVersion {
     return new DatasetVersion.Builder(
       jDatasetVersionId,
       jsDatasetTitle,
       this.toVersionNumber(jsDatasetVersionInfo),
       this.toStatus(jsDatasetVersionInfo.state),
       jsDatasetCitation,
-      true, // TODO Connect with dataset version isLatest
+      isLatest,
       false, // TODO Connect with dataset version isInReview
       this.toStatus(jsDatasetVersionInfo.state),
       this.toSomeDatasetVersionHasBeenReleased(jsDatasetVersionInfo, jsDatasetPublicationDate),

src/dataset/infrastructure/repositories/DatasetJSDataverseRepository.ts

@@ -3,6 +3,7 @@ import {
   Dataset,
   DatasetLock,
   DatasetNonNumericVersion,
+  DatasetNonNumericVersionSearchParam,
   TermsOfAccess
 } from '../../domain/models/Dataset'
 import { DatasetVersionDiff } from '../../domain/models/DatasetVersionDiff'
@@ -88,6 +89,43 @@ export class DatasetJSDataverseRepository implements DatasetRepository {
     return requireAppConfig().backendUrl
   }
 
+  static isLatestReleasedVersion(
+    datasetId: number | string,
+    datasetVersionNumber?: string
+  ): Promise<boolean> {
+    if (datasetVersionNumber === undefined) {
+      return Promise.resolve(true)
+    }
+
+    return getDatasetVersionsSummaries
+      .execute(datasetId, 2, 0)
+      .then((datasetVersionSummarySubset) => {
+        const latestReleasedDatasetVersion = datasetVersionSummarySubset.summaries.find(
+          (summary) =>
+            summary.versionNumber !== DatasetNonNumericVersion.DRAFT &&
+            summary.versionNumber !== DatasetNonNumericVersionSearchParam.DRAFT
+        )?.versionNumber
+
+        return (
+          latestReleasedDatasetVersion !== undefined &&
+          latestReleasedDatasetVersion ===
+            DatasetJSDataverseRepository.toVersionSummaryVersion(datasetVersionNumber)
+        )
+      })
+      .catch(() => false)
+  }
+
+  private static toVersionSummaryVersion(datasetVersionNumber: string): string {
+    if (
+      datasetVersionNumber === DatasetNonNumericVersion.DRAFT ||
+      datasetVersionNumber === DatasetNonNumericVersionSearchParam.DRAFT
+    ) {
+      return DatasetNonNumericVersionSearchParam.DRAFT
+    }
+
+    return datasetVersionNumber
+  }
+
   getAllWithCount(
     collectionId: string,
     paginationInfo: DatasetPaginationInfo

src/files/infrastructure/FileJSDataverseRepository.ts

@@ -31,6 +31,7 @@ import { FileCriteria } from '../domain/models/FileCriteria'
 import { DomainFileMapper } from './mappers/DomainFileMapper'
 import { JSFileMapper } from './mappers/JSFileMapper'
 import { DatasetVersion, DatasetVersionNumber } from '../../dataset/domain/models/Dataset'
+import { DatasetJSDataverseRepository } from '@/dataset/infrastructure/repositories/DatasetJSDataverseRepository'
 import { File } from '../domain/models/File'
 import { FilePaginationInfo } from '../domain/models/FilePaginationInfo'
 import { requireAppConfig } from '../../config'
@@ -293,7 +294,11 @@ export class FileJSDataverseRepository implements FileRepository {
               FileJSDataverseRepository.getDownloadCountById(jsFile.id, jsFile.publicationDate),
               Promise.resolve(resolvedPermissions),
               FileJSDataverseRepository.getThumbnailById(jsFile.id),
-              FileJSDataverseRepository.getTabularDataById(jsFile.id, jsFile.tabularData)
+              FileJSDataverseRepository.getTabularDataById(jsFile.id, jsFile.tabularData),
+              DatasetJSDataverseRepository.isLatestReleasedVersion(
+                jsDataset.id,
+                datasetVersionNumber
+              )
             ])
           })
       })
@@ -306,7 +311,8 @@ export class FileJSDataverseRepository implements FileRepository {
           downloadsCount,
           permissions,
           thumbnail,
-          tabularData
+          tabularData,
+          isLatestDatasetVersion
         ]) =>
           JSFileMapper.toFile(
             jsFile,
@@ -316,7 +322,8 @@ export class FileJSDataverseRepository implements FileRepository {
             downloadsCount,
             permissions,
             thumbnail,
-            tabularData
+            tabularData,
+            isLatestDatasetVersion
           )
       )
       .catch((error: ReadError) => {

src/files/infrastructure/mappers/JSFileMapper.ts

@@ -48,7 +48,8 @@ export class JSFileMapper {
     downloadsCount: number,
     permissions: FilePermissions,
     thumbnail?: string,
-    tabularData?: FileTabularData
+    tabularData?: FileTabularData,
+    isLatestDatasetVersion = true
   ): File {
     const datasetVersion = JSDatasetVersionMapper.toVersion(
       jsDataset.versionId,
@@ -57,7 +58,9 @@ export class JSFileMapper {
       datasetCitation,
       jsDataset.versionInfo.lastUpdateTime,
       jsDataset.publicationDate,
-      jsDataset.termsOfUse?.termsOfAccess
+      jsDataset.termsOfUse?.termsOfAccess,
+      undefined,
+      isLatestDatasetVersion
     )
     return {
       id: this.toFileId(jsFile.id),

src/sections/file/file-metadata/FileMetadata.tsx

@@ -41,18 +41,20 @@ export function FileMetadata({
 
   return (
     <>
-      <div className="d-flex justify-content-end mb-3">
-        <ExportMetadataDropdown
-          datasetPersistentId={datasetPersistentId}
-          anonymizedView={false}
-          datasetIsReleased={datasetVersion.someDatasetVersionHasBeenReleased}
-          datasetIsDeaccessioned={
-            datasetVersion.publishingStatus === DatasetPublishingStatus.DEACCESSIONED
-          }
-          canUpdateDataset={permissions.canEditOwnerDataset}
-          dataverseInfoRepository={dataverseInfoRepository}
-        />
-      </div>
+      {datasetVersion.isLatest && (
+        <div className="d-flex justify-content-end mb-3">
+          <ExportMetadataDropdown
+            datasetPersistentId={datasetPersistentId}
+            anonymizedView={false}
+            datasetIsReleased={datasetVersion.someDatasetVersionHasBeenReleased}
+            datasetIsDeaccessioned={
+              datasetVersion.publishingStatus === DatasetPublishingStatus.DEACCESSIONED
+            }
+            canUpdateDataset={permissions.canEditOwnerDataset}
+            dataverseInfoRepository={dataverseInfoRepository}
+          />
+        </div>
+      )}
       <Accordion defaultActiveKey="0">
         <Accordion.Item eventKey="0">
           <Accordion.Header>{t('metadata.title')}</Accordion.Header>

tests/component/sections/file/file-metadata/FileMetadata.spec.tsx

@@ -11,6 +11,8 @@ import {
 import { FilePermissionsMother } from '../../../files/domain/models/FilePermissionsMother'
 import { DataverseInfoMockRepository } from '@/stories/shared-mock-repositories/info/DataverseInfoMockRepository'
 import { requireAppConfig } from '@/config'
+import { DatasetVersionMother } from '@tests/component/dataset/domain/models/DatasetMother'
+import { DatasetMetadataExportFormatsMother } from '@tests/component/info/domain/models/DatasetMetadataExportFormatsMother'
 
 const appConfig = requireAppConfig()
 
@@ -31,6 +33,47 @@ describe('FileMetadata', () => {
     cy.findByRole('button', { name: 'File Metadata' }).should('exist')
   })
 
+  it('renders export metadata when the file is on the latest dataset version', () => {
+    const dataverseInfoRepository = new DataverseInfoMockRepository()
+    cy.stub(dataverseInfoRepository, 'getAvailableDatasetMetadataExportFormats').resolves(
+      DatasetMetadataExportFormatsMother.create()
+    )
+
+    cy.customMount(
+      <FileMetadata
+        name={file.name}
+        metadata={file.metadata}
+        permissions={file.permissions}
+        datasetPersistentId={file.datasetPersistentId}
+        datasetVersion={DatasetVersionMother.createRealistic()}
+        dataverseInfoRepository={dataverseInfoRepository}
+      />
+    )
+
+    cy.findByRole('button', { name: 'Export Metadata' }).should('exist')
+  })
+
+  it('does not render export metadata when the file is not on the latest dataset version', () => {
+    const dataverseInfoRepository = new DataverseInfoMockRepository()
+    const getAvailableDatasetMetadataExportFormats = cy
+      .stub(dataverseInfoRepository, 'getAvailableDatasetMetadataExportFormats')
+      .resolves(DatasetMetadataExportFormatsMother.create())
+
+    cy.customMount(
+      <FileMetadata
+        name={file.name}
+        metadata={file.metadata}
+        permissions={file.permissions}
+        datasetPersistentId={file.datasetPersistentId}
+        datasetVersion={DatasetVersionMother.createReleased({ isLatest: false })}
+        dataverseInfoRepository={dataverseInfoRepository}
+      />
+    )
+
+    cy.findByRole('button', { name: 'Export Metadata' }).should('not.exist')
+    cy.wrap(getAvailableDatasetMetadataExportFormats).should('not.have.been.called')
+  })
+
   it('renders the file preview', () => {
     cy.customMount(
       <FileMetadata