Fix null object values not being skipped to NBT, Closes #22

Author: rubensworks

src/main/java/org/cyclops/integratedscripting/evaluate/translation/translator/ValueTranslatorNbt.java

@@ -122,8 +122,11 @@ public ValueTypeNbt.ValueNbt translateFromGraal(Context context, Value value, IE
         // In all other cases, assume we have a compound tag
         CompoundTag tag = new CompoundTag();
         for (String memberKey : value.getMemberKeys()) {
-            IValue subValue = ValueTranslators.REGISTRY.translateFromGraal(context, value.getMember(memberKey), exceptionFactory);
-            tag.put(memberKey, ValueTranslators.REGISTRY.translateToNbt(context, subValue, exceptionFactory));
+            Value memberValue = value.getMember(memberKey);
+            if (!memberValue.isNull()) {
+                IValue subValue = ValueTranslators.REGISTRY.translateFromGraal(context, memberValue, exceptionFactory);
+                tag.put(memberKey, ValueTranslators.REGISTRY.translateToNbt(context, subValue, exceptionFactory));
+            }
         }
         return ValueTypeNbt.ValueNbt.of(tag);
     }

src/test/java/org/cyclops/integratedscripting/evaluate/translation/ValueTranslatorsJavaScriptTests.java

@@ -352,6 +352,13 @@ public void testNbtCompound() throws EvaluationException {
         assertThat(listValue.getArrayElement(2), equalTo(CTX.asValue(3)));
     }
 
+    @Test
+    public void testNbtCompoundWithNullAndUndefined() throws EvaluationException {
+        CompoundTag compoundTag = new CompoundTag();
+        compoundTag.put("a", StringTag.valueOf("bla"));
+        assertThat(ValueTranslators.REGISTRY.translateFromGraal(CTX, getJsValue("exports = { 'a': 'bla', 'b': null, 'c': undefined }"), EF), equalTo(ValueTypeNbt.ValueNbt.of(compoundTag)));
+    }
+
     @Test
     public void testNbtCompoundBidirectional() throws EvaluationException {
         CompoundTag compoundTag = new CompoundTag();