Interceptors should decide whether a value should be mapped to null, …

…not the MapTraverser.
statisticsnorway · Apr 4, 2024 · 74598bc · 74598bc
1 parent b619682
commit 74598bc
Showing 1 changed file with 1 addition and 7 deletions.
diff --git a/src/main/java/no/ssb/dlp/pseudo/core/map/MapTraverser.java b/src/main/java/no/ssb/dlp/pseudo/core/map/MapTraverser.java
@@ -54,13 +54,7 @@ else if (node instanceof Collection) {
 
     static Object processValue(Object value, String path, ValueInterceptor interceptor) {
         String newValue = interceptor.apply(new FieldDescriptor(path), (value == null) ? null : String.valueOf(value));
-        if (newValue != null) {
-            return (value == null)
-              ? newValue
-              : FromString.convert(newValue, value.getClass());
-        }
-
-        return value;
+        return (value == null) ? newValue : FromString.convert(newValue, value.getClass());
     }
 
     private static boolean isTraversable(Object o) {