Skip to content

Commit 0e94f6c

Browse files
authored
Merge pull request #3423 from dzhw/moellerth/issue3240-analysis-package-doi-registration
#3240: Fixed analysis package registration
2 parents 70f8748 + 77ab21f commit 0e94f6c

1 file changed

Lines changed: 5 additions & 14 deletions

File tree

src/main/java/eu/dzhw/fdz/metadatamanagement/projectmanagement/service/DataCiteService.java

Lines changed: 5 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@
99
import java.util.Locale;
1010
import java.util.Map;
1111
import java.util.Set;
12-
import java.util.stream.Collectors;
1312

1413
import com.fasterxml.jackson.databind.JsonNode;
1514
import com.fasterxml.jackson.databind.ObjectMapper;
@@ -178,28 +177,20 @@ private HttpStatus sendToDataCite(JsonNode payload) {
178177
public JsonNode getDataCiteMetadataForProject(DataAcquisitionProject project) throws DataCiteMetadataException {
179178
Map<String, Object> attrObj = null;
180179
if (project.getConfiguration().getRequirements().isDataPackagesRequired()) {
181-
final var dataPackages = this.dataPackageRepository.findByDataAcquisitionProjectId(project.getId());
182-
if (dataPackages.isEmpty()) {
180+
final var dataPackage = this.dataPackageRepository.findOneByDataAcquisitionProjectId(project.getId());
181+
if (dataPackage == null) {
183182
throw new DataCiteMetadataException(
184183
String.format("The project with id '%s' has no data package linked to it.", project.getId()));
185-
} else if (dataPackages.size() > 1) {
186-
throw new DataCiteMetadataException(
187-
String.format("The project with id '%s' has more than one data package linked to it.", project.getId()));
188184
}
189-
final DataPackage dataPackage = dataPackages.get(0);
190185
final List<Survey> surveys = this.surveyRepository.findByDataAcquisitionProjectId(project.getId());
191186
attrObj = this.createAttrObjectForDataPackage(project, dataPackage, surveys);
192187
} else if (project.getConfiguration().getRequirements().isAnalysisPackagesRequired()) {
193-
final var analysisPackages = this.analysisPackageRepository
194-
.findByDataAcquisitionProjectIdAndShadowIsTrue(project.getId()).collect(Collectors.toList());
195-
if (analysisPackages.isEmpty()) {
188+
final var analysisPackage = this.analysisPackageRepository
189+
.findOneByDataAcquisitionProjectId(project.getId());
190+
if (analysisPackage == null) {
196191
throw new DataCiteMetadataException(
197192
String.format("The project with id '%s' has no analysis package linked to it.", project.getId()));
198-
} else if (analysisPackages.size() > 1) {
199-
throw new DataCiteMetadataException(
200-
String.format("The project with id '%s' has more than one analysis package linked to it.", project.getId()));
201193
}
202-
final AnalysisPackage analysisPackage = analysisPackages.get(0);
203194
attrObj = this.createAttrObjectForAnalysisPackage(project, analysisPackage);
204195
}
205196

0 commit comments

Comments
 (0)