Fix #17

Author: cowtowncoder

datatypes/src/main/java/com/fasterxml/jackson/datatype/jdk8/OptionalSerializer.java

@@ -321,13 +321,15 @@ private final JsonSerializer<Object> _findCachedSerializer(SerializerProvider pr
     private final JsonSerializer<Object> _findSerializer(SerializerProvider provider,
             Class<?> type, BeanProperty prop) throws JsonMappingException
     {
-        return provider.findTypedValueSerializer(type, true, prop);
+        // 13-Mar-2017, tatu: To fix [modules-java8#17] change `findTypedValueSerializer()` int `findValueSerializer()`
+        return provider.findValueSerializer(type, prop);
     }
 
     private final JsonSerializer<Object> _findSerializer(SerializerProvider provider,
         JavaType type, BeanProperty prop) throws JsonMappingException
     {
-        return provider.findTypedValueSerializer(type, true, prop);
+        // 13-Mar-2017, tatu: To fix [modules-java8#17] change `findTypedValueSerializer()` int `findValueSerializer()`
+        return provider.findValueSerializer(type, prop);
     }
 
     // !!! 22-Mar-2016, tatu: Method added in jackson-databind 2.7.4; may be used

datatypes/src/test/java/com/fasterxml/jackson/failing/ContextualOptional17Test.java renamed to datatypes/src/test/java/com/fasterxml/jackson/datatype/jdk8/ContextualOptional17Test.java

@@ -1,6 +1,5 @@
-package com.fasterxml.jackson.failing;
+package com.fasterxml.jackson.datatype.jdk8;
 
-import java.text.DateFormat;
 import java.text.SimpleDateFormat;
 import java.util.Date;
 import java.util.Optional;
@@ -9,7 +8,6 @@
 import com.fasterxml.jackson.annotation.JsonFormat;
 import com.fasterxml.jackson.annotation.JsonPropertyOrder;
 import com.fasterxml.jackson.databind.ObjectMapper;
-import com.fasterxml.jackson.datatype.jdk8.ModuleTestBase;
 
 public class ContextualOptional17Test extends ModuleTestBase
 {
@@ -45,7 +43,7 @@ public void testContextualOptionals() throws Exception
         final String json = mapper.writeValueAsString(input);
 //System.err.println("JSON:\n"+json);
         assertEquals(aposToQuotes(
-                "{'date':'1970/01/01','date1':'1970+01+01','date2':'1970*01*01'"),
+                "{'date':'1970/01/01','date1':'1970+01+01','date2':'1970*01*01'}"),
                 json);
     }        
 }

release-notes/VERSION

@@ -8,6 +8,11 @@ Modules:
 === Releases ===
 ------------------------------------------------------------------------
 
+2.8.8 (not yet released)
+
+#17: Cached `Optional` serializer does not apply annotations for POJO properties
+ (reported by codicusmaximus@github)
+
 2.8.7 (21-Feb-2017)
 2.8.6 (12-Jan-2017)