Merge pull request #707 from k163377/fix-#526

Changed to use default argument on `null` if `JsonSetter(nulls = Nulls.SKIP)` is specified.

Author: k163377

release-notes/CREDITS-2.x

@@ -21,6 +21,7 @@ kkurczewski
 * #689: Add KotlinDuration support
 
 WrongWrong (@k163377)
+* #707: Changed to use default argument on null if JsonSetter(nulls = Nulls.SKIP) is specified.
 * #700: Reduce the load on the search process for serializers
 * #687: Optimize and Refactor KotlinValueInstantiator.createFromObjectWith
 * #686: Add KotlinPropertyNameAsImplicitName option

release-notes/VERSION-2.x

@@ -18,6 +18,7 @@ Co-maintainers:
 
 2.16.0 (not yet released)
 
+#707: If JsonSetter(nulls = Nulls.SKIP) is specified, the default argument is now used when null.
 #700: Reduce the load on the search process for serializers.
 #689: Added UseJavaDurationConversion feature.
  By enabling this feature and adding the Java Time module, Kotlin Duration can be handled in the same way as Java Duration.

src/main/kotlin/com/fasterxml/jackson/module/kotlin/KotlinValueInstantiator.kt

@@ -1,5 +1,6 @@
 package com.fasterxml.jackson.module.kotlin
 
+import com.fasterxml.jackson.annotation.Nulls
 import com.fasterxml.jackson.databind.BeanDescription
 import com.fasterxml.jackson.databind.DeserializationConfig
 import com.fasterxml.jackson.databind.DeserializationContext
@@ -32,6 +33,9 @@ internal class KotlinValueInstantiator(
 
     private fun List<KTypeProjection>.markedNonNullAt(index: Int) = getOrNull(index)?.type?.isMarkedNullable == false
 
+    private fun SettableBeanProperty.skipNulls(): Boolean =
+        nullIsSameAsDefault || (metadata.valueNulls == Nulls.SKIP)
+
     override fun createFromObjectWith(
         ctxt: DeserializationContext,
         props: Array<out SettableBeanProperty>,
@@ -70,7 +74,7 @@ internal class KotlinValueInstantiator(
             val paramType = paramDef.type
             var paramVal = if (!isMissing || paramDef.isPrimitive() || jsonProp.hasInjectableValueId()) {
                 val tempParamVal = buffer.getParameter(jsonProp)
-                if (nullIsSameAsDefault && tempParamVal == null && paramDef.isOptional) {
+                if (tempParamVal == null && jsonProp.skipNulls() && paramDef.isOptional) {
                     return@forEachIndexed
                 }
                 tempParamVal

src/test/kotlin/com/fasterxml/jackson/module/kotlin/test/github/Github526.kt

@@ -0,0 +1,20 @@
+package com.fasterxml.jackson.module.kotlin.test.github
+
+import com.fasterxml.jackson.annotation.JsonSetter
+import com.fasterxml.jackson.annotation.Nulls
+import com.fasterxml.jackson.module.kotlin.jacksonObjectMapper
+import com.fasterxml.jackson.module.kotlin.readValue
+import org.junit.Test
+import kotlin.test.assertEquals
+
+class Github526 {
+    data class D(@JsonSetter(nulls = Nulls.SKIP) val v: Int = -1)
+
+    @Test
+    fun test() {
+        val mapper = jacksonObjectMapper()
+        val d = mapper.readValue<D>("""{"v":null}""")
+
+        assertEquals(-1, d.v)
+    }
+}