Fix #32 as per suggestion

Author: cowtowncoder

jaxb/src/main/java/com/fasterxml/jackson/module/jaxb/JaxbAnnotationIntrospector.java

@@ -433,21 +433,19 @@ public boolean hasAnySetterAnnotation(AnnotatedMethod am)
 
     @Override
     public Boolean hasRequiredMarker(AnnotatedMember m) {
-        XmlElement elem = m.getAnnotation(XmlElement.class);
-        if ((elem != null) && elem.required()) {
-            return Boolean.TRUE;
-        }
+        // 17-Oct-2017, tatu: [modules-base#32]
+        //   Before 2.9.3, was handling `true` correctly,
+        //   but otherwise used confusing logic (probably in attempt to try to avoid
+        //   reporting not-required for default value case
         XmlAttribute attr = m.getAnnotation(XmlAttribute.class);
-        if ((attr != null) && attr.required()) {
-            return Boolean.TRUE;
+        if (attr != null) {
+            return attr.required();
         }
-        // 09-Sep-2015, tatu: Not 100% sure that we should ever return `false`
-        //   here (as it blocks calls to secondary introspector), but since that
-        //   was the existing behavior before 2.6, is retained for now.
-        if ((elem != null) || (attr != null)) {
-            return null;
+        XmlElement elem = m.getAnnotation(XmlElement.class);
+        if (elem != null) {
+            return elem.required();
         }
-        return Boolean.FALSE;
+        return null;
     }
 
     @Override

release-notes/CREDITS

@@ -0,0 +1,11 @@
+Here are people who have contributed to development Jackson JSON processor
+Base modules
+(version numbers in brackets indicate release in which the problem was fixed)
+(NOTE: incomplete -- need to collect info from sub-projects, pre-2.9)
+
+Tatu Saloranta, [email protected]: author
+
+Vojtěch Habarta (vojtechhabarta@gitub)
+
+* Reported [JAXB#32]: Fix introspector chaining in `JaxbAnnotationIntrospector.hasRequiredMarker()`
+ (2.9.3)

release-notes/VERSION

@@ -8,6 +8,11 @@ Modules:
 === Releases ===
 ------------------------------------------------------------------------
 
+2.9.3 (not yet released)
+
+#32: Fix introspector chaining in `JaxbAnnotationIntrospector.hasRequiredMarker()`
+ (reported by Vojtěch H)
+
 2.9.2 (14-Oct-2017)
 
 #30: (afterburner) `IncompatibleClassChangeError` deserializing interface