fix: update datasetID to allow persistent id

Author: ChengShi-1

src/datasets/domain/repositories/IDatasetsRepository.ts

@@ -68,7 +68,7 @@ export interface IDatasetsRepository {
   unlinkDataset(datasetId: number, collectionAlias: string): Promise<void>
   getDatasetLinkedCollections(datasetId: number | string): Promise<DatasetLinkedCollection[]>
   getDatasetCitationInOtherFormats(
-    datasetId: number,
+    datasetId: number | string,
     datasetVersionId: string,
     format: CitationFormat,
     includeDeaccessioned?: boolean

src/datasets/domain/useCases/GetDatasetCitationInOtherFormats.ts

@@ -14,14 +14,14 @@ export class GetDatasetCitationInOtherFormats implements UseCase<FormattedCitati
   /**
    * Returns the dataset citation in the specified format.
    *
-   * @param {number} datasetId - The dataset identifier.
+   * @param {number | string} datasetId - The dataset identifier.
    * @param {string | DatasetNotNumberedVersion} [datasetVersionId=DatasetNotNumberedVersion.LATEST] - The dataset version identifier, which can be a version-specific string (e.g., '1.0') or a DatasetNotNumberedVersion enum value. Defaults to LATEST.
    * @param {CitationFormat} format - The citation format to return. One of: 'EndNote', 'RIS', 'BibTeX', 'CSLJson', 'Internal'.
    * @param {boolean} [includeDeaccessioned=false] - Whether to include deaccessioned versions in the search. Defaults to false.
    * @returns {Promise<FormattedCitation>} The citation content, format, and content type.
    */
   async execute(
-    datasetId: number,
+    datasetId: number | string,
     datasetVersionId: string | DatasetNotNumberedVersion = DatasetNotNumberedVersion.LATEST,
     format: CitationFormat,
     includeDeaccessioned = false

src/datasets/infra/repositories/DatasetsRepository.ts

@@ -78,7 +78,7 @@ export class DatasetsRepository extends ApiRepository implements IDatasetsReposi
   }
 
   public async getDatasetCitation(
-    datasetId: number,
+    datasetId: number | string,
     datasetVersionId: string,
     includeDeaccessioned: boolean
   ): Promise<string> {
@@ -98,7 +98,7 @@ export class DatasetsRepository extends ApiRepository implements IDatasetsReposi
   }
 
   public async getDatasetCitationInOtherFormats(
-    datasetId: number,
+    datasetId: number | string,
     datasetVersionId: string | 'LATEST' = 'LATEST',
     format: CitationFormat,
     includeDeaccessioned = false

test/integration/datasets/DatasetsRepository.test.ts

@@ -515,6 +515,17 @@ describe('DatasetsRepository', () => {
       expect(citation.contentType).toMatch(/text\/plain/)
     })
 
+    test('should return citation in BibTeX format using persistent id', async () => {
+      const citation = await sut.getDatasetCitationInOtherFormats(
+        testDatasetIds.persistentId,
+        DatasetNotNumberedVersion.LATEST,
+        CitationFormat.BibTeX
+      )
+
+      expect(typeof citation.content).toBe('string')
+      expect(citation.contentType).toMatch(/text\/plain/)
+    })
+
     test('should return citation in RIS format', async () => {
       const citation = await sut.getDatasetCitationInOtherFormats(
         testDatasetIds.numericId,