Fixed #2789

Author: cowtowncoder

release-notes/VERSION-2.x

@@ -10,6 +10,8 @@ Project: jackson-databind
   `createGenerator()`
 #2785: Polymorphic subtypes not registering on copied ObjectMapper (2.11.1)
  (reported, fix contributed by Joshua S)
+#2789: Failure to read AnnotatedField value in Jackson 2.11
+ (reported by isaki@github)
 #2796: `TypeFactory.constructType()` does not take `TypeBindings` correctly
  (reported by Daniel H)
 

src/main/java/com/fasterxml/jackson/databind/ser/BeanSerializerBuilder.java

@@ -181,11 +181,23 @@ public boolean hasProperties() {
      */
     public JsonSerializer<?> build()
     {
+        // [databind#2789]: There can be a case wherein `_typeId` is used, but
+        // nothing else. Rare but has happened; so force access.
+        if (_typeId != null) {
+            if (_config.isEnabled(MapperFeature.CAN_OVERRIDE_ACCESS_MODIFIERS)) {
+                _typeId.fixAccess(_config.isEnabled(MapperFeature.OVERRIDE_PUBLIC_ACCESS_MODIFIERS));
+            }
+        }
+        if (_anyGetter != null) {
+            _anyGetter.fixAccess(_config);
+        }
+
         BeanPropertyWriter[] properties;
         // No properties, any getter or object id writer?
         // No real serializer; caller gets to handle
         if (_properties == null || _properties.isEmpty()) {
             if (_anyGetter == null && _objectIdWriter == null) {
+                // NOTE! Caller may still call `createDummy()` later on
                 return null;
             }
             properties = NO_PROPERTIES;
@@ -205,14 +217,6 @@ public JsonSerializer<?> build()
 _properties.size(), _filteredProperties.length));
             }
         }
-        if (_anyGetter != null) {
-            _anyGetter.fixAccess(_config);
-        }
-        if (_typeId != null) {
-            if (_config.isEnabled(MapperFeature.CAN_OVERRIDE_ACCESS_MODIFIERS)) {
-                _typeId.fixAccess(_config.isEnabled(MapperFeature.OVERRIDE_PUBLIC_ACCESS_MODIFIERS));
-            }
-        }
         return new BeanSerializer(_beanDesc.getType(), this,
                 properties, _filteredProperties);
     }

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

@@ -1,8 +1,12 @@
 package com.fasterxml.jackson.databind.introspect;
 
+import java.util.Objects;
+
 import com.fasterxml.jackson.annotation.*;
 import com.fasterxml.jackson.annotation.JsonAutoDetect.Visibility;
+
 import com.fasterxml.jackson.core.*;
+
 import com.fasterxml.jackson.databind.*;
 import com.fasterxml.jackson.databind.introspect.VisibilityChecker;
 
@@ -35,13 +39,84 @@ public void setValue(int x) {
         }
     }
 
+    // [databind#1947]
+    static class Entity1947 {
+        public int shouldBeDetected;
+        public String shouldNotBeDetected;
+
+        @JsonProperty
+        public int getShouldBeDetected() {
+            return shouldBeDetected;
+        }
+
+        public void setShouldBeDetected(int shouldBeDetected) {
+            this.shouldBeDetected = shouldBeDetected;
+        }
+
+        public String getShouldNotBeDetected() {
+            return shouldNotBeDetected;
+        }
+
+        public void setShouldNotBeDetected(String shouldNotBeDetected) {
+            this.shouldNotBeDetected = shouldNotBeDetected;
+        }
+    }
+
+    // For [databind#2789]
+
+    @SuppressWarnings("unused")
+    @JsonAutoDetect(
+        getterVisibility = JsonAutoDetect.Visibility.NONE,
+        creatorVisibility = JsonAutoDetect.Visibility.NONE,
+        isGetterVisibility = JsonAutoDetect.Visibility.NONE,
+        fieldVisibility = JsonAutoDetect.Visibility.NONE,
+        setterVisibility = JsonAutoDetect.Visibility.NONE)
+    @JsonTypeInfo(
+        use = JsonTypeInfo.Id.NAME,
+        include = JsonTypeInfo.As.PROPERTY,
+        property = "type",
+        visible = true)
+    @JsonSubTypes({
+        @JsonSubTypes.Type(name = "CLASS_A", value = DataClassA.class)
+    })
+    private static abstract class DataParent2789 {
+
+        @JsonProperty("type")
+        @JsonTypeId
+        private final DataType2789 type;
+
+        DataParent2789() {
+            super();
+            this.type = null;
+        }
+
+        DataParent2789(final DataType2789 type) {
+            super();
+            this.type = Objects.requireNonNull(type);
+        }
+
+        public DataType2789 getType() {
+            return this.type;
+        }
+    }
+
+    private static final class DataClassA extends DataParent2789 {
+        DataClassA() {
+            super(DataType2789.CLASS_A);
+        }
+    }
+
+    private enum DataType2789 {
+        CLASS_A;
+    }
+
     /*
     /********************************************************
     /* Unit tests
     /********************************************************
      */
 
-    private final ObjectMapper MAPPER = new ObjectMapper();
+    private final ObjectMapper MAPPER = newJsonMapper();
 
     public void testPrivateCtor() throws Exception
     {
@@ -101,4 +176,28 @@ public void testVisibilityConfigOverridesForDeser() throws Exception
         Feature1347DeserBean result = mapper.readValue(JSON, Feature1347DeserBean.class);
         assertEquals(3, result.value);
     }
+
+    // [databind#1947]
+    public void testDisablingAll() throws Exception
+    {
+        ObjectMapper mapper = jsonMapperBuilder()
+                .disable(MapperFeature.AUTO_DETECT_SETTERS)
+                .disable(MapperFeature.AUTO_DETECT_FIELDS)
+                .disable(MapperFeature.AUTO_DETECT_GETTERS)
+                .disable(MapperFeature.AUTO_DETECT_CREATORS)
+                .disable(MapperFeature.AUTO_DETECT_IS_GETTERS)
+                .build();
+        String json = mapper.writeValueAsString(new Entity1947());
+        JsonNode n = mapper.readTree(json);
+        assertEquals(1, n.size());
+        assertTrue(n.has("shouldBeDetected"));
+        assertFalse(n.has("shouldNotBeDetected"));
+    }
+
+    // [databind#2789]
+    public void testAnnotatedFieldIssue2789() throws Exception {
+        final String json = MAPPER.writeValueAsString(new DataClassA());
+        final DataParent2789 copy = MAPPER.readValue(json, DataParent2789.class);
+        assertEquals(DataType2789.CLASS_A, copy.getType());
+    }
 }