added a couple of todo:s as reminders #3648

IQSS · Mar 23, 2023 · 1517b59 · 1517b59
1 parent d5f1e84
commit 1517b59
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/src/main/java/edu/harvard/iq/dataverse/export/ddi/DdiExportUtil.java b/src/main/java/edu/harvard/iq/dataverse/export/ddi/DdiExportUtil.java
@@ -513,6 +513,7 @@ private static void writeSummaryDescriptionElement(XMLStreamWriter xmlw, Dataset
 
                 for (HashSet<FieldDTO> foo : geographicCoverageDTO.getMultipleCompound()) {
                     HashMap<String, String> geoMap = new HashMap<>();
+                    // TODO: the order may still be wrong -L.A. (because of multiples)
                     for (Iterator<FieldDTO> iterator = foo.iterator(); iterator.hasNext();) {
                         FieldDTO next = iterator.next();
                         if (DatasetFieldConstant.country.equals(next.getTypeName())) {
@@ -545,8 +546,10 @@ private static void writeSummaryDescriptionElement(XMLStreamWriter xmlw, Dataset
                 }
             }
 
+            /* TODO: if exists? - L.A. */
             writeFullElementList(xmlw, "geogUnit", dto2PrimitiveList(datasetVersionDTO, DatasetFieldConstant.geographicUnit));
 
+            /* TODO: it really looks like only 1 geoBndBox is allowed in the DDI - ? */
             if (geographicBoundingBoxDTO != null) {
                 for (HashSet<FieldDTO> foo : geographicBoundingBoxDTO.getMultipleCompound()) {
                     xmlw.writeStartElement("geoBndBox");