Merge pull request #115 from FasterXML/tatu/2.17/112-path-custom-seri…

…alizer Fix #112: make custom serialization work for `java.nio.file.Path`
FasterXML · Feb 17, 2024 · 4486d21 · 4486d21
2 parents eff22a2 + c77e7e7
commit 4486d21
Show file tree

Hide file tree

Showing 4 changed files with 18 additions and 9 deletions.
diff --git a/jr-objects/src/main/java/com/fasterxml/jackson/jr/ob/impl/ValueLocatorBase.java b/jr-objects/src/main/java/com/fasterxml/jackson/jr/ob/impl/ValueLocatorBase.java
@@ -236,21 +236,21 @@ protected int _findSimpleType(Class<?> raw, boolean forSer)
         if (UUID.class.isAssignableFrom(raw)) {
             return SER_UUID;
         }
-        /* May or may not help with deser, but recognized nonetheless;
-         * on assumption that Beans should rarely implement `CharSequence`
-         */
+        // May or may not help with deser, but recognized nonetheless;
+        // on assumption that Beans should rarely implement `CharSequence`
         if (CharSequence.class.isAssignableFrom(raw)) {
             return SER_CHARACTER_SEQUENCE;
         }
-        /* `Iterable` can be added on all kinds of things, and it won't
-         * help at all with deserialization; hence only use for serialization.
-         */
+        // `Iterable` can be added on all kinds of things, and it won't
+        // help at all with deserialization; hence only use for serialization.
         if (forSer && Iterable.class.isAssignableFrom(raw)) {
-            return SER_ITERABLE;
+            // 16-Feb-2024, tatu: [jackson-jr#112] java.nio.file.Path is not really Iterable
+            if (!java.nio.file.Path.class.isAssignableFrom(raw)) {
+                return SER_ITERABLE;
+            }
         }
 
         // Ok. I give up, no idea!
         return SER_UNKNOWN;
     }
-
 }
diff --git a/...r/failing/ValueWriterModifier112Test.java → ...r/ob/impl/ValueWriterModifier112Test.java b/...r/failing/ValueWriterModifier112Test.java → ...r/ob/impl/ValueWriterModifier112Test.java
@@ -1,4 +1,4 @@
-package com.fasterxml.jackson.jr.failing;
+package com.fasterxml.jackson.jr.ob.impl;
 
 import java.io.File;
 import java.io.IOException;

diff --git a/release-notes/CREDITS-2.x b/release-notes/CREDITS-2.x
@@ -33,3 +33,9 @@ Reed Passaretti (@reed53)
 
 * Reported #107: Cannot deserialize `byte[]` from JSON `null` value
  (2.15.3)
+
+Julian Honnen (@jhonnen)
+
+* Reported #112: `overrideStandardValueWriter` only applied to first `java.nio.file.Path`
+  valued field of bean
+ (2.17.0)
diff --git a/release-notes/VERSION-2.x b/release-notes/VERSION-2.x
@@ -15,6 +15,9 @@ Modules:
 
 #78: Deserializes "null" to "0.0" for `java.lang.Double` (wrapper)
  (reported by @bill-phast)
+#112: `overrideStandardValueWriter` only applied to first `java.nio.file.Path`
+  valued field of bean
+ (reported by Julian H)
 
 2.16.1 (24-Dec-2023)