Add release notes wrt #3655

Author: cowtowncoder

release-notes/CREDITS-2.x

@@ -409,6 +409,9 @@ Jonas Konrad (yawkat@github)
   * Contributed #3417: Allow (de)serializing records using Bean(De)SerializerModifier
     even when reflection is unavailable
    (2.14.0)
+  * Contributed fix for #3655: `Enum` values can not be read from single-element array even
+    with `DeserializationFeature.UNWRAP_SINGLE_VALUE_ARRAYS`
+   (2.14.1)
 
 Jirka Kremser (Jiri-Kremser@github)
   * Suggested #924: SequenceWriter.writeAll() could accept Iterable
@@ -1512,3 +1515,8 @@ Joachim Durchholz (toolforger@github)
   * Requested #3633: Expose `translate()` method of standard `PropertyNamingStrategy`
    implementations
   (2.14.0)
+
+Andrej Mitrovic (AndrejMitrovic@github)
+  * Reported #3655: `Enum` values can not be read from single-element array even with
+    `DeserializationFeature.UNWRAP_SINGLE_VALUE_ARRAYS`
+  (2.14.1)

release-notes/VERSION-2.x

@@ -4,6 +4,13 @@ Project: jackson-databind
 === Releases === 
 ------------------------------------------------------------------------
 
+2.14.1 (not yet released)
+
+#3655: `Enum` values can not be read from single-element array even with
+  `DeserializationFeature.UNWRAP_SINGLE_VALUE_ARRAYS`
+ (reported by Andrej M)
+ (fix contributed by Jonas K)
+
 2.14.0 (05-Nov-2022)
 
 #1980: Add method(s) in `JsonNode` that works like combination of `at()`

src/main/java/com/fasterxml/jackson/databind/deser/BeanDeserializer.java

@@ -644,6 +644,8 @@ protected Object _deserializeFromArray(JsonParser p, DeserializationContext ctxt
                 }
                 return value;
             }
+            // 15-Nov-2022, tatu: ... we probably should pass original `JsonToken.START_ARRAY`
+            //     as unexpected token, since `p` now points to `unwrappedToken` instead...
         }
         return ctxt.handleUnexpectedToken(getValueType(ctxt), p);
     }

src/main/java/com/fasterxml/jackson/databind/deser/std/FactoryBasedEnumDeserializer.java

@@ -150,13 +150,15 @@ public Object deserialize(JsonParser p, DeserializationContext ctxt) throws IOEx
             //    value is expected; otherwise Deserializer should have been used earlier
             // 14-Jan-2022, tatu: as per [databind#3369] need to consider structured
             //    value types (Object, Array) as well.
+            // 15-Nov-2022, tatu: Fix for [databind#3655] requires handling of possible
+            //    unwrapping, do it here
             JsonToken t = p.currentToken();
-            boolean unwrapping = false;
-            if (t == JsonToken.START_ARRAY && ctxt.isEnabled(DeserializationFeature.UNWRAP_SINGLE_VALUE_ARRAYS)) {
+            boolean unwrapping = (t == JsonToken.START_ARRAY)
+                    && ctxt.isEnabled(DeserializationFeature.UNWRAP_SINGLE_VALUE_ARRAYS);
+            if (unwrapping) {
                 t = p.nextToken();
-                unwrapping = true;
             }
-            if ((t != null) && !t.isScalarValue()) {
+            if ((t == null) || !t.isScalarValue()) {
                 // Could argue we should throw an exception but...
                 value = "";
                 p.skipChildren();