diff --git a/apps/datahub-e2e/src/e2e/dataset.cy.ts b/apps/datahub-e2e/src/e2e/dataset.cy.ts index c97aa55..f25c515 100644 --- a/apps/datahub-e2e/src/e2e/dataset.cy.ts +++ b/apps/datahub-e2e/src/e2e/dataset.cy.ts @@ -245,7 +245,7 @@ describe('datasets', () => { beforeEach(() => { cy.visit('/dataset/04bcec79-5b25-4b16-b635-73115f7456e4') }) - it('should download button should contain the correct link and open in new tab', () => { + it('should contain the correct link in download button', () => { cy.get('[data-cy="download-button"]') .first() .invoke('attr', 'href') @@ -258,6 +258,25 @@ describe('datasets', () => { .first() .should('have.attr', 'target', '_blank') }) + it('should contain empty download attribute for other files than json and geojson', () => { + cy.get('[data-cy="download-button"]') + .first() + .should('have.attr', 'download', '') + }) + it('should contain download attribute with filename for json files', () => { + cy.get('[data-cy="download-button"]') + .eq(2) + .should( + 'have.attr', + 'download', + 'insee:rectangles_200m_menage_erbm.json' + ) + }) + it('should open link in new tab as fallback (if download attribute is ignored, for not same-origin)', () => { + cy.get('[data-cy="download-button"]') + .first() + .should('have.attr', 'target', '_blank') + }) it('should copy the link resource to clipboard when clicking on copy button', () => { cy.get('body').focus() cy.get('[data-cy="copy-button"]').first().click() diff --git a/apps/datahub/src/app/dataset/dataset-downloads/download-item/download-item.component.html b/apps/datahub/src/app/dataset/dataset-downloads/download-item/download-item.component.html index 136a1c0..abb9a52 100644 --- a/apps/datahub/src/app/dataset/dataset-downloads/download-item/download-item.component.html +++ b/apps/datahub/src/app/dataset/dataset-downloads/download-item/download-item.component.html @@ -28,7 +28,7 @@ rel="noopener" [class]="'mel-primary-button font-medium text-nowrap'" data-cy="download-button" - download + [download]="downloadFileName" >mel.dataset.download download diff --git a/apps/datahub/src/app/dataset/dataset-downloads/download-item/download-item.component.ts b/apps/datahub/src/app/dataset/dataset-downloads/download-item/download-item.component.ts index 75ccaae..5721643 100644 --- a/apps/datahub/src/app/dataset/dataset-downloads/download-item/download-item.component.ts +++ b/apps/datahub/src/app/dataset/dataset-downloads/download-item/download-item.component.ts @@ -12,4 +12,16 @@ export class MelDownloadItemComponent extends DownloadItemComponent { navigator.clipboard.writeText(this.link.url.href) ;(event.target as HTMLElement).blur() } + + // note that the download attribute calling this getter only takes effect on same-origin resources + get downloadFileName() { + let completeFileName = '' + const fileName = this.link.name ?? 'data' + if (this.format === 'geojson') { + completeFileName = `${fileName}.geojson` + } else if (this.format === 'json') { + completeFileName = `${fileName}.json` + } + return completeFileName + } }