108055: isClosed method should use xml configuration

atmire · Nov 8, 2023 · d800d80 · d800d80
1 parent 60d4bca
commit d800d80
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/...ce-server-webapp/src/main/java/org/dspace/app/rest/converter/SubmissionFormConverter.java b/...ce-server-webapp/src/main/java/org/dspace/app/rest/converter/SubmissionFormConverter.java
@@ -125,7 +125,7 @@ private SubmissionFormFieldRest getField(DCInput dcinput, String formName) {
                                                         dcinput.getVocabulary()));
                     selMd.setClosed(
                             isClosed(dcinput.getSchema(), dcinput.getElement(), dcinput.getQualifier(),
-                                    dcinput.getPairsType(), dcinput.getVocabulary()));
+                                    dcinput.getPairsType(), dcinput.getVocabulary(), dcinput.isClosedVocabulary()));
                 } else {
                     inputRest.setType(inputType);
                 }
@@ -145,7 +145,7 @@ private SubmissionFormFieldRest getField(DCInput dcinput, String formName) {
                         selMd.setControlledVocabulary(getAuthorityName(dcinput.getSchema(), dcinput.getElement(),
                                 pairs.get(idx + 1), dcinput.getPairsType(), dcinput.getVocabulary()));
                         selMd.setClosed(isClosed(dcinput.getSchema(), dcinput.getElement(),
-                                dcinput.getQualifier(), null, dcinput.getVocabulary()));
+                                dcinput.getQualifier(), null, dcinput.getVocabulary(), dcinput.isClosedVocabulary()));
                     }
                     selectableMetadata.add(selMd);
                 }
@@ -212,9 +212,11 @@ private String getAuthorityName(String schema, String element, String qualifier,
     }
 
     private boolean isClosed(String schema, String element, String qualifier, String valuePairsName,
-            String vocabularyName) {
-        if (StringUtils.isNotBlank(valuePairsName) || StringUtils.isNotBlank(vocabularyName)) {
+            String vocabularyName, boolean isClosedVocabulary) {
+        if (StringUtils.isNotBlank(valuePairsName)) {
             return true;
+        } else if (StringUtils.isNotBlank(vocabularyName)) {
+            return isClosedVocabulary;
         }
         return authorityUtils.isClosed(schema, element, qualifier);
     }