Merge branch '2.19'

Author: cowtowncoder

release-notes/VERSION-2.x

@@ -71,6 +71,8 @@ Project: jackson-databind
  (reported by @dbachdev)
 #4922: Failing `@JsonMerge` with a custom Map
  (reported by @nlisker)
+#4932: Conversion of `MissingNode` throws `JsonProcessingException`
+ (reported by @ludgerb)
 
 2.18.2 (27-Nov-2024)
 

src/main/java/tools/jackson/databind/ObjectMapper.java

@@ -2382,6 +2382,9 @@ protected Object _convert(Object fromValue, JavaType toValueType)
                 result = _findRootDeserializer(readCtxt, toValueType).getNullValue(readCtxt);
             } else if (t == JsonToken.END_ARRAY || t == JsonToken.END_OBJECT) {
                 result = null;
+            } else if (t == JsonToken.NOT_AVAILABLE) {
+                // 28-Jan-2025, tatu: [databind#4932] Need to handle this case too
+                result = null;
             } else { // pointing to event other than null
                 ValueDeserializer<Object> deser = _findRootDeserializer(readCtxt, toValueType);
                 // note: no handling of unwrapping
@@ -2539,6 +2542,9 @@ protected Object _readValue(DeserializationContextExt ctxt, JsonParser p,
             result = _findRootDeserializer(ctxt, valueType).getNullValue(ctxt);
         } else if (t == JsonToken.END_ARRAY || t == JsonToken.END_OBJECT) {
             result = null;
+        } else if (t == JsonToken.NOT_AVAILABLE) {
+            // 28-Jan-2025, tatu: [databind#4932] Need to handle this case too
+            result = null;
         } else { // pointing to event other than null
             result = ctxt.readRootValue(p, valueType, _findRootDeserializer(ctxt, valueType), null);
         }
@@ -2563,6 +2569,9 @@ protected Object _readMapAndClose(DeserializationContextExt ctxt,
                 result = _findRootDeserializer(ctxt, valueType).getNullValue(ctxt);
             } else if (t == JsonToken.END_ARRAY || t == JsonToken.END_OBJECT) {
                 result = null;
+            } else if (t == JsonToken.NOT_AVAILABLE) {
+                // 28-Jan-2025, tatu: [databind#4932] Need to handle this case too
+                result = null;
             } else {
                 result = ctxt.readRootValue(p, valueType,
                         _findRootDeserializer(ctxt, valueType), null);

src/main/java/tools/jackson/databind/ObjectReader.java

@@ -1618,6 +1618,9 @@ protected Object _bind(DeserializationContextExt ctxt,
             }
         } else if (t == JsonToken.END_ARRAY || t == JsonToken.END_OBJECT) {
             result = valueToUpdate;
+        } else if (t == JsonToken.NOT_AVAILABLE) {
+            // 28-Jan-2025, tatu: [databind#4932] Need to handle this case too
+            result = valueToUpdate;
         } else { // pointing to event other than null
             result = ctxt.readRootValue(p, _valueType, _findRootDeserializer(ctxt), _valueToUpdate);
         }
@@ -1643,6 +1646,9 @@ protected Object _bindAndClose(DeserializationContextExt ctxt,
                 }
             } else if (t == JsonToken.END_ARRAY || t == JsonToken.END_OBJECT) {
                 result = _valueToUpdate;
+            } else if (t == JsonToken.NOT_AVAILABLE) {
+                // 28-Jan-2025, tatu: [databind#4932] Need to handle this case too
+                result = _valueToUpdate;
             } else {
                 result = ctxt.readRootValue(p, _valueType, _findRootDeserializer(ctxt), _valueToUpdate);
             }

src/test/java/tools/jackson/databind/node/JsonNodeConversionsTest.java

@@ -437,4 +437,15 @@ public void testValueToTree() throws Exception
         assertEquals(wrapRootMapper.readValue(expected, Map.class), wrapRootMapper.readValue(wrapRootMapper.writeValueAsString(value), Map.class));
         assertEquals(wrapRootMapper.readValue(expected, Map.class), wrapRootMapper.readValue(wrapRootMapper.valueToTree(value).toString(), Map.class));
     }
+
+    // [databind#4932]: handling of `MissingNode` wrt conversions
+    @Test
+    public void treeToValueWithMissingNode4932() throws Exception {
+        assertNull(MAPPER.treeToValue(MAPPER.nullNode(), Object.class));
+        assertNull(MAPPER.treeToValue(MAPPER.missingNode(), Object.class));
+
+        ObjectReader r = MAPPER.readerFor(Object.class);
+        assertNull(r.treeToValue(MAPPER.nullNode(), Object.class));
+        assertNull(r.treeToValue(MAPPER.missingNode(), Object.class));
+    }
 }