use proper JSON-LD for dataset type #10517

IQSS · Aug 13, 2024 · 19e3e51 · 19e3e51
1 parent 1a834de
commit 19e3e51
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 19 deletions.
diff --git a/doc/sphinx-guides/source/_static/api/dataset-create-software.jsonld b/doc/sphinx-guides/source/_static/api/dataset-create-software.jsonld
@@ -12,5 +12,5 @@
   "https://dataverse.org/schema/citation/dsDescription": {
     "https://dataverse.org/schema/citation/dsDescriptionValue": "Darwin's finches (also known as the Galápagos finches) are a group of about fifteen species of passerine birds."
   },
-  "datasetType": "software"
+  "https://dataverse.org/schema/core#datasetType": "software"
 }
diff --git a/src/main/java/edu/harvard/iq/dataverse/util/json/JSONLDUtil.java b/src/main/java/edu/harvard/iq/dataverse/util/json/JSONLDUtil.java
@@ -98,26 +98,14 @@ public static Dataset updateDatasetMDFromJsonLD(Dataset ds, String jsonLDBody,
         //Store the metadatalanguage if sent - the caller needs to check whether it is allowed (as with any GlobalID)
         ds.setMetadataLanguage(jsonld.getString(JsonLDTerm.schemaOrg("inLanguage").getUrl(),null));
 
-        try (StringReader rdr = new StringReader(jsonLDBody)) {
-            try (JsonReader jsonReader = Json.createReader(rdr)) {
-                JsonObject jsonObject = jsonReader.readObject();
-                String datasetTypeIn = jsonObject.getString("datasetType", null);
-                logger.fine("datasetTypeIn: " + datasetTypeIn);
-                DatasetType defaultDatasetType = datasetTypeSvc.getByName(DatasetType.DEFAULT_DATASET_TYPE);
-                if (datasetTypeIn == null) {
-                    ds.setDatasetType(defaultDatasetType);
-                } else {
-                    DatasetType datasetType = datasetTypeSvc.getByName(datasetTypeIn);
-                    if (datasetType != null) {
-                        ds.setDatasetType(datasetType);
-                    } else {
-                        throw new BadRequestException("Invalid dataset type: " + datasetTypeIn);
-                    }
-                }
-            }
+        String datasetTypeIn = jsonld.getString(JsonLDTerm.datasetType.getUrl(), DatasetType.DEFAULT_DATASET_TYPE);
+        DatasetType datasetType = datasetTypeSvc.getByName(datasetTypeIn);
+        if (datasetType != null) {
+            ds.setDatasetType(datasetType);
+        } else {
+            throw new BadRequestException("Invalid dataset type: " + datasetTypeIn);
         }
 
-
         dsv = updateDatasetVersionMDFromJsonLD(dsv, jsonld, metadataBlockSvc, datasetFieldSvc, append, migrating, licenseSvc);
         dsv.setDataset(ds);
 

diff --git a/src/main/java/edu/harvard/iq/dataverse/util/json/JsonLDTerm.java b/src/main/java/edu/harvard/iq/dataverse/util/json/JsonLDTerm.java
@@ -52,6 +52,8 @@ public class JsonLDTerm {
     public static JsonLDTerm fileCount = JsonLDTerm.DVCore("fileCount");
     public static JsonLDTerm maxFileSize = JsonLDTerm.DVCore("maxFileSize");
 
+    public static JsonLDTerm datasetType = JsonLDTerm.DVCore("datasetType");
+
     public JsonLDTerm(JsonLDNamespace namespace, String term) {
         this.namespace = namespace;
         this.term = term;