From 712ae05fc812522a57328cc68be2dbdf4b4be8a2 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Bj=C3=B8rn-Andre=20Skaar?=
 <31540110+bjornandre@users.noreply.github.com>
Date: Fri, 5 Apr 2024 08:47:53 +0200
Subject: [PATCH] Interceptors should decide whether a value should be mapped
 to null, not the MapTraverser. (#41)

---
 .../java/no/ssb/dlp/pseudo/core/map/MapTraverser.java     | 8 +-------
 1 file changed, 1 insertion(+), 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
index 8476637..e2d47b0 100644
--- 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) {