Skip to content

Expands InternalExportDataProvider per dataverse-spi 2.1.0 and refactors the DDI exporter to utilize the new functionality - #11799

Open
landreev wants to merge 57 commits into
developfrom
11405-refactor-exports
Open

Expands InternalExportDataProvider per dataverse-spi 2.1.0 and refactors the DDI exporter to utilize the new functionality#11799
landreev wants to merge 57 commits into
developfrom
11405-refactor-exports

No need to mention the new parameter in the changelog. #11405

5802628
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Aug 21, 2026 in 1m 36s

Quality Gate failed

Annotations

Check warning on line 522 in src/test/java/edu/harvard/iq/dataverse/export/CroissantExporterSlimTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Don't override this deprecated method, it has been marked for removal.

See more on https://sonarcloud.io/project/issues?id=IQSS_dataverse&issues=AZ-qFsgeIp65em4O2DTe&open=AZ-qFsgeIp65em4O2DTe&pullRequest=11799

Check warning on line 1669 in src/main/java/edu/harvard/iq/dataverse/export/ddi/DdiExportUtil.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use the built-in formatting to construct this argument.

See more on https://sonarcloud.io/project/issues?id=IQSS_dataverse&issues=AZ-qFsnjIp65em4O2DTs&open=AZ-qFsnjIp65em4O2DTs&pullRequest=11799

Check warning on line 1727 in src/main/java/edu/harvard/iq/dataverse/export/ddi/DdiExportUtil.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this call to a deprecated method, it has been marked for removal.

See more on https://sonarcloud.io/project/issues?id=IQSS_dataverse&issues=AZ-qFsnjIp65em4O2DT2&open=AZ-qFsnjIp65em4O2DT2&pullRequest=11799

Check warning on line 179 in src/main/java/edu/harvard/iq/dataverse/export/InternalExportDataProvider.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused "jab" local variable.

See more on https://sonarcloud.io/project/issues?id=IQSS_dataverse&issues=AaAbkSSGecBgKF9JLlNP&open=AaAbkSSGecBgKF9JLlNP&pullRequest=11799

Check warning on line 187 in src/test/java/edu/harvard/iq/dataverse/export/CroissantExporterSlimTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Don't override this deprecated method, it has been marked for removal.

See more on https://sonarcloud.io/project/issues?id=IQSS_dataverse&issues=AZ-qFsgeIp65em4O2DTU&open=AZ-qFsgeIp65em4O2DTU&pullRequest=11799

Check warning on line 52 in src/main/java/edu/harvard/iq/dataverse/util/json/JsonPrinter.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused import 'java.math.BigDecimal'.

See more on https://sonarcloud.io/project/issues?id=IQSS_dataverse&issues=AZ-qFslJIp65em4O2DTp&open=AZ-qFslJIp65em4O2DTp&pullRequest=11799

Check warning on line 1705 in src/main/java/edu/harvard/iq/dataverse/export/ddi/DdiExportUtil.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use the built-in formatting to construct this argument.

See more on https://sonarcloud.io/project/issues?id=IQSS_dataverse&issues=AZ-qFsnjIp65em4O2DTt&open=AZ-qFsnjIp65em4O2DTt&pullRequest=11799

Check warning on line 430 in src/test/java/edu/harvard/iq/dataverse/export/CroissantExporterTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Don't override this deprecated method, it has been marked for removal.

See more on https://sonarcloud.io/project/issues?id=IQSS_dataverse&issues=AZ-qFsdqIp65em4O2DTK&open=AZ-qFsdqIp65em4O2DTK&pullRequest=11799

Check warning on line 154 in src/test/java/edu/harvard/iq/dataverse/export/CroissantExporterSlimTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Don't override this deprecated method, it has been marked for removal.

See more on https://sonarcloud.io/project/issues?id=IQSS_dataverse&issues=AZ-qFsgeIp65em4O2DTS&open=AZ-qFsgeIp65em4O2DTS&pullRequest=11799

Check warning on line 201 in src/main/java/edu/harvard/iq/dataverse/export/ddi/DdiExportUtil.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this call to a deprecated method, it has been marked for removal.

See more on https://sonarcloud.io/project/issues?id=IQSS_dataverse&issues=AZ-qFsnjIp65em4O2DT0&open=AZ-qFsnjIp65em4O2DT0&pullRequest=11799

Check warning on line 1741 in src/main/java/edu/harvard/iq/dataverse/export/ddi/DdiExportUtil.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Invoke method(s) only conditionally. Use the built-in formatting to construct this argument.

See more on https://sonarcloud.io/project/issues?id=IQSS_dataverse&issues=AZ-qFsnjIp65em4O2DTu&open=AZ-qFsnjIp65em4O2DTu&pullRequest=11799

Check warning on line 51 in src/main/java/edu/harvard/iq/dataverse/export/DDIExporter.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this call to a deprecated method, it has been marked for removal.

See more on https://sonarcloud.io/project/issues?id=IQSS_dataverse&issues=AZ-qFspcIp65em4O2DT-&open=AZ-qFspcIp65em4O2DT-&pullRequest=11799

Check warning on line 95 in src/test/java/edu/harvard/iq/dataverse/export/CroissantExporterSlimTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Don't override this deprecated method, it has been marked for removal.

See more on https://sonarcloud.io/project/issues?id=IQSS_dataverse&issues=AZ-qFsgeIp65em4O2DTR&open=AZ-qFsgeIp65em4O2DTR&pullRequest=11799

Check warning on line 176 in src/test/java/edu/harvard/iq/dataverse/export/CroissantExporterTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Don't override this deprecated method, it has been marked for removal.

See more on https://sonarcloud.io/project/issues?id=IQSS_dataverse&issues=AZ-qFsdqIp65em4O2DTC&open=AZ-qFsdqIp65em4O2DTC&pullRequest=11799

Check warning on line 338 in src/test/java/edu/harvard/iq/dataverse/export/CroissantExporterTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Don't override this deprecated method, it has been marked for removal.

See more on https://sonarcloud.io/project/issues?id=IQSS_dataverse&issues=AZ-qFsdqIp65em4O2DTH&open=AZ-qFsdqIp65em4O2DTH&pullRequest=11799

Check warning on line 235 in src/test/java/edu/harvard/iq/dataverse/export/CroissantExporterTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Don't override this deprecated method, it has been marked for removal.

See more on https://sonarcloud.io/project/issues?id=IQSS_dataverse&issues=AZ-qFsdqIp65em4O2DTD&open=AZ-qFsdqIp65em4O2DTD&pullRequest=11799

Check warning on line 11 in src/main/java/edu/harvard/iq/dataverse/export/DCTermsExporter.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This block of commented-out lines of code should be removed.

See more on https://sonarcloud.io/project/issues?id=IQSS_dataverse&issues=AZ-qFsqDIp65em4O2DUG&open=AZ-qFsqDIp65em4O2DUG&pullRequest=11799

Check failure on line 220 in src/main/java/edu/harvard/iq/dataverse/DatasetVersionFilesServiceBean.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal "label" 3 times.

See more on https://sonarcloud.io/project/issues?id=IQSS_dataverse&issues=AZ-qFsrfIp65em4O2DUK&open=AZ-qFsrfIp65em4O2DUK&pullRequest=11799

Check failure on line 731 in src/main/java/edu/harvard/iq/dataverse/DatasetServiceBean.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to reduce its Cognitive Complexity from 29 to the 15 allowed.

See more on https://sonarcloud.io/project/issues?id=IQSS_dataverse&issues=AZ_fMBSmC-mCpXWPrLfN&open=AZ_fMBSmC-mCpXWPrLfN&pullRequest=11799

Check warning on line 8 in src/main/java/edu/harvard/iq/dataverse/export/DublinCoreExporter.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This block of commented-out lines of code should be removed.

See more on https://sonarcloud.io/project/issues?id=IQSS_dataverse&issues=AZ-qFspLIp65em4O2DT8&open=AZ-qFspLIp65em4O2DT8&pullRequest=11799

Check warning on line 441 in src/test/java/edu/harvard/iq/dataverse/export/CroissantExporterTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Don't override this deprecated method, it has been marked for removal.

See more on https://sonarcloud.io/project/issues?id=IQSS_dataverse&issues=AZ-6WB9dhvOp7Pn7dCQ-&open=AZ-6WB9dhvOp7Pn7dCQ-&pullRequest=11799

Check warning on line 452 in src/test/java/edu/harvard/iq/dataverse/export/CroissantExporterSlimTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Don't override this deprecated method, it has been marked for removal.

See more on https://sonarcloud.io/project/issues?id=IQSS_dataverse&issues=AZ-6WCCMhvOp7Pn7dCRE&open=AZ-6WCCMhvOp7Pn7dCRE&pullRequest=11799

Check warning on line 360 in src/test/java/edu/harvard/iq/dataverse/export/CroissantExporterSlimTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Don't override this deprecated method, it has been marked for removal.

See more on https://sonarcloud.io/project/issues?id=IQSS_dataverse&issues=AZ-6WCCMhvOp7Pn7dCRD&open=AZ-6WCCMhvOp7Pn7dCRD&pullRequest=11799

Check warning on line 315 in src/test/java/edu/harvard/iq/dataverse/export/TabularDataExportIT.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this block of code, fill it in, or add a comment explaining why it is empty.

See more on https://sonarcloud.io/project/issues?id=IQSS_dataverse&issues=AZ-qFsZLIp65em4O2DS7&open=AZ-qFsZLIp65em4O2DS7&pullRequest=11799

Check warning on line 1742 in src/main/java/edu/harvard/iq/dataverse/export/ddi/DdiExportUtil.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this expression which always evaluates to "true"

See more on https://sonarcloud.io/project/issues?id=IQSS_dataverse&issues=AZ-qFsnjIp65em4O2DTr&open=AZ-qFsnjIp65em4O2DTr&pullRequest=11799