Fixed #42

FasterXML · Sep 30, 2014 · 25ba4b6 · 25ba4b6
1 parent e5b06cf
commit 25ba4b6
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/src/main/java/com/fasterxml/jackson/module/afterburner/deser/BeanPropertyMutator.java b/src/main/java/com/fasterxml/jackson/module/afterburner/deser/BeanPropertyMutator.java
@@ -13,7 +13,6 @@
  */
 public abstract class BeanPropertyMutator
 {
-
     // Intentionally not volatile for performance, worst case is we throw a few extra exceptions
     private boolean broken = false;
 

diff --git a/src/main/java/com/fasterxml/jackson/module/afterburner/deser/DeserializerModifier.java b/src/main/java/com/fasterxml/jackson/module/afterburner/deser/DeserializerModifier.java
@@ -109,7 +109,7 @@ protected List<OptimizedSettableBeanProperty<?>> findOptimizableProperties(
                 continue;
             }
             // (although, interestingly enough, can seem to access private classes...)
-
+            
             // 30-Jul-2012, tatu: [Issue-6]: Needs to skip custom deserializers, if any.
             if (prop.hasValueDeserializer()) {
                 if (!isDefaultDeserializer(config, prop.getValueDeserializer())) {
@@ -132,6 +132,13 @@ protected List<OptimizedSettableBeanProperty<?>> findOptimizableProperties(
                     }
                 }
             } else if (prop instanceof FieldProperty) { // regular fields
+                // And as to fields, can not overwrite final fields (which may
+                // be overwritable via Reflection)
+                if (Modifier.isFinal(prop.getMember().getMember().getModifiers())) {
+                    System.err.println("FINAL '"+prop.getName()+"' -- > NO GO!");
+                    continue;
+                }
+
                 Class<?> type = member.getRawType();
                 if (type.isPrimitive()) {
                     if (type == Integer.TYPE) {