There is a case that KotlinModule can overwrite hasRequiredMarker incorrectly.

[k163377]

Search before asking

• I searched in the issues and found nothing similar.
• I have confirmed that the same problem is not reproduced if I exclude the KotlinModule.
• I searched in the issues of databind and other modules used and found nothing similar.
• I have confirmed that the problem does not reproduce in Java and only occurs when using Kotlin and KotlinModule.

Describe the bug

KotlinAnnotationIntrospector.hasRequiredMarker has following problems

1. If nullToEmptyCollection or nullToEmptyMap is specified, the specification by annotation is overridden.
2. If nullToEmptyCollection or nullToEmptyMap is specified, the KotlinModule specification is applied to non Kotlin members.
3. The nullToEmptyCollection and nullToEmptyMap only work with parameters for now, but as for hasRequiredMarker, it applies to fields, setters and getters.
   • I created a test and confirmed that the setters were not affected.

jackson-module-kotlin/src/main/kotlin/com/fasterxml/jackson/module/kotlin/KotlinAnnotationIntrospector.kt

Lines 41 to 55 in bab70c6

	try {
	when {
	nullToEmptyCollection && m.type.isCollectionLikeType -> false
	nullToEmptyMap && m.type.isMapLikeType -> false
	m.member.declaringClass.isKotlinClass() -> when (m) {
	is AnnotatedField -> m.hasRequiredMarker()
	is AnnotatedMethod -> m.hasRequiredMarker()
	is AnnotatedParameter -> m.hasRequiredMarker()
	else -> null
	}
	else -> null
	}
	} catch (ex: UnsupportedOperationException) {
	null
	}

To Reproduce

It can be read from the code.

Expected behavior

The behavior should satisfy the following three points

1. Annotation specifications are not overridden as much as possible.
2. Only apply to members defined in Kotlin.
3. The nullToEmptyCollection and nullToEmptyMap are applied only to parameters.

Versions

This has not been corrected in 2.19.

Additional context

In the modification process, changes need to be made regarding the new isRequired property.
FasterXML/jackson-databind#5020 (comment)