Fix #1916

FasterXML · Jan 29, 2018 · be45354 · be45354
1 parent f3ba50c
commit be45354
Show file tree

Hide file tree

Showing 4 changed files with 85 additions and 86 deletions.
diff --git a/release-notes/VERSION b/release-notes/VERSION
@@ -24,4 +24,5 @@ Versions: 3.x (for earlier see VERSION-2.x)
  into `JsonDeserializer`
 #1889: Merge `ContextualSerializer` into `JsonSerializer`, `ContextualDeserializer`
  into `JsonDeserializer`
+#1916: Change `MapperFeature.USE_GETTERS_AS_SETTERS)` default to `false`
 - Remove `MappingJsonFactory`
diff --git a/src/main/java/com/fasterxml/jackson/databind/MapperFeature.java b/src/main/java/com/fasterxml/jackson/databind/MapperFeature.java
@@ -46,9 +46,9 @@ public enum MapperFeature implements ConfigFeature
      * precedence than setters, so they are only used if no
      * setter is found for the Map/Collection property.
      *<p>
-     * Feature is enabled by default.
+     * Feature is disabled by default since 3.0 (with 2.x was enabled)
      */
-    USE_GETTERS_AS_SETTERS(true),
+    USE_GETTERS_AS_SETTERS(false),
 
     /**
      * Feature that determines how <code>transient</code> modifier for fields

diff --git a/src/test/java/com/fasterxml/jackson/databind/cfg/DeserializationConfigTest.java b/src/test/java/com/fasterxml/jackson/databind/cfg/DeserializationConfigTest.java
@@ -18,7 +18,7 @@ public void testFeatureDefaults()
 
         // Expected defaults:
         assertTrue(cfg.isEnabled(MapperFeature.USE_ANNOTATIONS));
-        assertTrue(cfg.isEnabled(MapperFeature.USE_GETTERS_AS_SETTERS));
+        assertFalse(cfg.isEnabled(MapperFeature.USE_GETTERS_AS_SETTERS)); // 3.0
         assertTrue(cfg.isEnabled(MapperFeature.CAN_OVERRIDE_ACCESS_MODIFIERS));
 
         assertFalse(cfg.isEnabled(DeserializationFeature.USE_BIG_DECIMAL_FOR_FLOATS));