Use index as entity name in arrays of objects (#65)

metafacture · Nov 19, 2021 · bd902fd · bd902fd
1 parent 181d36d
commit bd902fd
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 8 deletions.
diff --git a/metafix/src/main/java/org/metafacture/metafix/Metafix.java b/metafix/src/main/java/org/metafacture/metafix/Metafix.java
@@ -156,7 +156,7 @@ private void emit(final String field, final Value value) {
                 final Value arrayValue = array.get(i);
 
                 if (arrayValue.isHash()) {
-                    outputStreamReceiver.startEntity(isMulti ? "" : field);
+                    outputStreamReceiver.startEntity(isMulti ? (i + 1) + "" : field);
                     arrayValue.asHash().forEach(this::emit);
                     outputStreamReceiver.endEntity();
                 }

diff --git a/metafix/src/test/java/org/metafacture/metafix/MetafixBindTest.java b/metafix/src/test/java/org/metafacture/metafix/MetafixBindTest.java
@@ -117,11 +117,11 @@ public void doListWithAppendAndLast() {
             }, (o, f) -> {
                 o.get().startRecord("1");
                 o.get().startEntity("author[]");
-                o.get().startEntity("");
+                o.get().startEntity("1");
                 o.get().literal("name", "A University");
                 // o.get().literal("type", "Default"); // FIXME: bind scope broken
                 o.get().endEntity();
-                o.get().startEntity("");
+                o.get().startEntity("2");
                 o.get().literal("name", "Max");
                 o.get().literal("type", "Default");
                 f.apply(2).endEntity();
@@ -182,11 +182,11 @@ public void doListEntitesToEntities() {
             }, (o, f) -> {
                 o.get().startRecord("1");
                 o.get().startEntity("author[]");
-                o.get().startEntity("");
+                o.get().startEntity("1");
                 o.get().literal("name", "A University");
                 o.get().literal("type", "Organization");
                 o.get().endEntity();
-                o.get().startEntity("");
+                o.get().startEntity("2");
                 o.get().literal("name", "Max");
                 o.get().literal("type", "Person");
                 f.apply(2).endEntity();

diff --git a/metafix/src/test/java/org/metafacture/metafix/MetafixRecordTest.java b/metafix/src/test/java/org/metafacture/metafix/MetafixRecordTest.java
@@ -344,10 +344,10 @@ public void copyIntoArrayOfObjects() {
             }, (o, f) -> {
                 o.get().startRecord("1");
                 o.get().startEntity("author[]");
-                o.get().startEntity("");
+                o.get().startEntity("1");
                 o.get().literal("name", "max");
                 o.get().endEntity();
-                o.get().startEntity("");
+                o.get().startEntity("2");
                 o.get().literal("name", "mo");
                 f.apply(2).endEntity();
                 o.get().endRecord();
@@ -653,7 +653,7 @@ public void mixedArray() {
                 o.get().startRecord("1");
                 o.get().startEntity("@context[]");
                 o.get().literal("1", "https://w3id.org/kim/lrmi-profile/draft/context.jsonld");
-                o.get().startEntity("");
+                o.get().startEntity("2");
                 o.get().literal("@language", "de");
                 f.apply(2).endEntity();
                 o.get().endRecord();