Fix #299

Author: cowtowncoder

release-notes/VERSION

@@ -10,6 +10,8 @@ Project: jackson-databind
 #113: Problem deserializing polymorphic types with @JsonCreator
 #165: Add `DeserializationContext.getContextualType()` to let deserializer
   known the expected type.
+#299: Add `DeserializationFeature.FAIL_ON_UNRESOLVED_OBJECT_IDS` to allow missing
+  Object Ids (as global default)
 #408: External type id does not allow use of 'visible=true'
 #421: @JsonCreator not used in case of multiple creators with parameter names
  (reported by Lovro P, lpandzic@github)

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

@@ -172,6 +172,22 @@ public enum DeserializationFeature implements ConfigFeature
      */
     FAIL_ON_IGNORED_PROPERTIES(false),
 
+    /**
+     * Feature that determines what happens if an Object Id reference is encountered
+     * that does not refer to an actual Object with that id ("unresolved Object Id"):
+     * either an exception is thrown (<code>true</code>), or a null object is used
+     * instead (<code>false</code>).
+     * Note that if this is set to <code>false</code>, no further processing is done;
+     * specifically, if reference is defined via setter method, that method will NOT
+     * be called.
+     *<p>
+     * Feature is enabled by default, so that unknown Object Ids will result in an
+     * exception being thrown, at the end of deserialization.
+     * 
+     * @since 2.5
+     */
+    FAIL_ON_UNRESOLVED_OBJECT_IDS(true),
+
     /**
      * Feature that determines whether Jackson code should catch
      * and wrap {@link Exception}s (but never {@link Error}s!)
@@ -188,7 +204,7 @@ public enum DeserializationFeature implements ConfigFeature
      * Feature is enabled by default.
      */
     WRAP_EXCEPTIONS(true),
-    
+
     /*
     /******************************************************
     /* Structural conversion features

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

@@ -138,15 +138,18 @@ public void checkUnresolvedObjectId() throws UnresolvedForwardReference
         if (_objectIds == null) {
             return;
         }
-
+        // 29-Dec-2014, tatu: As per [databind#299], may also just let unresolved refs be...
+        if (!isEnabled(DeserializationFeature.FAIL_ON_UNRESOLVED_OBJECT_IDS)) {
+            return;
+        }
         UnresolvedForwardReference exception = null;
         for (Entry<IdKey,ReadableObjectId> entry : _objectIds.entrySet()) {
             ReadableObjectId roid = entry.getValue();
             if (roid.hasReferringProperties()) {
                 if (exception == null) {
                     exception = new UnresolvedForwardReference("Unresolved forward references for: ");
                 }
-                for (Iterator<Referring> iterator = roid.referringProperties(); iterator.hasNext();) {
+                for (Iterator<Referring> iterator = roid.referringProperties(); iterator.hasNext(); ) {
                     Referring referring = iterator.next();
                     exception.addUnresolvedId(roid.getKey().key, referring.getBeanType(), referring.getLocation());
                 }

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

@@ -84,18 +84,25 @@ public ContainerSerializer<?> withValueTypeSerializer(TypeSerializer vts) {
      * {@link com.fasterxml.jackson.databind.SerializerProvider#findValueSerializer}.
      */
     public abstract JsonSerializer<?> getContentSerializer();
-    
+
     /*
     /**********************************************************
     /* Abstract methods for sub-classes to implement
     /**********************************************************
      */
-    
+
     /* Overridden as abstract, to force re-implementation; necessary for all
      * collection types.
      */
     @Override
-    public abstract boolean isEmpty(T value);
+    @Deprecated
+    public boolean isEmpty(T value) {
+        return isEmpty(null, value);
+    }
+
+    // since 2.5: should be declared abstract in future (2.6)
+//    @Override
+//    public abstract boolean isEmpty(SerializerProvider prov, T value);
 
     /**
      * Method called to determine if the given value (of type handled by

src/main/java/com/fasterxml/jackson/databind/ser/impl/IndexedListSerializer.java

@@ -47,7 +47,7 @@ public IndexedListSerializer withResolved(BeanProperty property,
      */
 
     @Override
-    public boolean isEmpty(List<?> value) {
+    public boolean isEmpty(SerializerProvider prov, List<?> value) {
         return (value == null) || value.isEmpty();
     }
 

src/main/java/com/fasterxml/jackson/databind/ser/impl/IteratorSerializer.java

@@ -28,7 +28,7 @@ public IteratorSerializer(IteratorSerializer src,
     }
 
     @Override
-    public boolean isEmpty(Iterator<?> value) {
+    public boolean isEmpty(SerializerProvider prov, Iterator<?> value) {
         return (value == null) || !value.hasNext();
     }
 

src/main/java/com/fasterxml/jackson/databind/ser/impl/MapEntrySerializer.java

@@ -174,7 +174,7 @@ public boolean hasSingleElement(Map.Entry<?,?> value) {
     }
 
     @Override
-    public boolean isEmpty(Entry<?, ?> value) {
+    public boolean isEmpty(SerializerProvider prov, Entry<?, ?> value) {
         return (value == null);
     }
 

src/main/java/com/fasterxml/jackson/databind/ser/impl/StringArraySerializer.java

@@ -130,7 +130,7 @@ public JsonSerializer<?> getContentSerializer() {
     }
 
     @Override
-    public boolean isEmpty(String[] value) {
+    public boolean isEmpty(SerializerProvider prov, String[] value) {
         return (value == null) || (value.length == 0);
     }
 

src/main/java/com/fasterxml/jackson/databind/ser/std/CollectionSerializer.java

@@ -62,7 +62,7 @@ public CollectionSerializer withResolved(BeanProperty property,
      */
 
     @Override
-    public boolean isEmpty(Collection<?> value) {
+    public boolean isEmpty(SerializerProvider prov, Collection<?> value) {
         return (value == null) || value.isEmpty();
     }
 

src/main/java/com/fasterxml/jackson/databind/ser/std/EnumMapSerializer.java

@@ -162,7 +162,7 @@ public JsonSerializer<?> getContentSerializer() {
     }
 
     @Override
-    public boolean isEmpty(EnumMap<? extends Enum<?>,?> value) {
+    public boolean isEmpty(SerializerProvider prov, EnumMap<? extends Enum<?>,?> value) {
         return (value == null) || value.isEmpty();
     }