Fix #1672

Author: cowtowncoder

src/main/java/com/fasterxml/jackson/databind/AnnotationIntrospector.java

@@ -1221,7 +1221,7 @@ public PropertyName findNameForDeserialization(Annotated a) {
      * @since 2.9
      */
     public Boolean hasAnySetter(Annotated a) {
-        return false;
+        return null;
     }
 
     /**

src/test/java/com/fasterxml/jackson/databind/introspect/IgnoredCreatorProperty1572Test.java

@@ -4,7 +4,6 @@
 
 import com.fasterxml.jackson.databind.BaseMapTest;
 import com.fasterxml.jackson.databind.ObjectMapper;
-import com.fasterxml.jackson.databind.introspect.*;
 
 public class IgnoredCreatorProperty1572Test extends BaseMapTest
 {

src/test/java/com/fasterxml/jackson/databind/introspect/IntrospectorPairTest.java

@@ -197,6 +197,17 @@ public String findTypeName(AnnotatedClass ac) {
             return (String) values.get("findTypeName");
         }
 
+        /*
+        /******************************************************
+        /* Deserialization introspection
+        /******************************************************
+         */
+
+        @Override
+        public Boolean hasAnySetter(Annotated a) {
+            return (Boolean) values.get("hasAnySetter");
+        }
+
         /*
         /******************************************************
         /* Helper methods
@@ -215,7 +226,7 @@ private boolean _boolean(String key) {
     /**********************************************************
      */
 
-    private final IntrospectorWithMap NO_ANNOTATIONS = new IntrospectorWithMap();
+    private final AnnotationIntrospector NO_ANNOTATIONS = AnnotationIntrospector.nopInstance();
 
     public void testVersion() throws Exception
     {
@@ -225,8 +236,9 @@ public void testVersion() throws Exception
                 .version(v);
         assertEquals(v,
                 new AnnotationIntrospectorPair(withVersion, NO_ANNOTATIONS).version());
+        IntrospectorWithMap noVersion = new IntrospectorWithMap();
         assertEquals(Version.unknownVersion(),
-                new AnnotationIntrospectorPair(NO_ANNOTATIONS, withVersion).version());
+                new AnnotationIntrospectorPair(noVersion, withVersion).version());
     }
 
     public void testAccess() throws Exception
@@ -395,7 +407,7 @@ public void testFindDeserializer() throws Exception
     /******************************************************
     /* Property auto-detection
     /******************************************************
-    */
+     */
 
     public void testFindAutoDetectVisibility() throws Exception
     {
@@ -414,7 +426,7 @@ public void testFindAutoDetectVisibility() throws Exception
     /******************************************************
     /* Type handling
     /******************************************************
-    */
+     */
 
     public void testFindTypeResolver() throws Exception
     {
@@ -445,6 +457,29 @@ public void testFindTypeName() {
                 new AnnotationIntrospectorPair(intr2, intr1).findTypeName(null));
     }
 
+    /*
+    /******************************************************
+    /* Deserialization introspection
+    /******************************************************
+     */
+
+    // for [databind#1672]
+    public void testHasAnySetter() {
+        IntrospectorWithMap intr1 = new IntrospectorWithMap()
+                .add("hasAnySetter", Boolean.TRUE);
+        IntrospectorWithMap intr2 = new IntrospectorWithMap()
+                .add("hasAnySetter", Boolean.FALSE);
+        assertNull(new AnnotationIntrospectorPair(NO_ANNOTATIONS, NO_ANNOTATIONS).hasAnySetter(null));
+        assertEquals(Boolean.TRUE,
+                new AnnotationIntrospectorPair(intr1, intr2).hasAnySetter(null));
+        assertEquals(Boolean.TRUE,
+                new AnnotationIntrospectorPair(NO_ANNOTATIONS, intr1).hasAnySetter(null));
+        assertEquals(Boolean.FALSE,
+                new AnnotationIntrospectorPair(intr2, intr1).hasAnySetter(null));
+        assertEquals(Boolean.FALSE,
+                new AnnotationIntrospectorPair(NO_ANNOTATIONS, intr2).hasAnySetter(null));
+    }
+
     /*
     /**********************************************************
     /* Test methods, others