minPlain and maxPlain values are inclusive (#1253)

mkurz · web-flow · commit 02f89b0e3e55 · 2025-11-06T11:51:53.000+01:00
diff --git a/play-json/jvm/src/main/scala/play/api/libs/json/jackson/JacksonJson.scala b/play-json/jvm/src/main/scala/play/api/libs/json/jackson/JacksonJson.scala
@@ -86,7 +86,7 @@ private[jackson] class JsValueSerializer(jsonConfig: JsonConfig) extends JsonSer
         // configuration is ignored when called from ObjectMapper.valueToTree
         val shouldWritePlain = {
           val va = v.abs
-          va < jsonConfig.bigDecimalSerializerConfig.maxPlain && va > jsonConfig.bigDecimalSerializerConfig.minPlain
+          va <= jsonConfig.bigDecimalSerializerConfig.maxPlain && va >= jsonConfig.bigDecimalSerializerConfig.minPlain
         }
         val stripped = stripTrailingZeros(v.bigDecimal)
         val raw      = if (shouldWritePlain) stripped.toPlainString else stripped.toString

Original file line number	Diff line number	Diff line change
`@@ -86,7 +86,7 @@ private[jackson] class JsValueSerializer(jsonConfig: JsonConfig) extends JsonSer`
`86`	`86`	`// configuration is ignored when called from ObjectMapper.valueToTree`
`87`	`87`	`val shouldWritePlain = {`
`88`	`88`	`val va = v.abs`
`89`		`- va < jsonConfig.bigDecimalSerializerConfig.maxPlain && va > jsonConfig.bigDecimalSerializerConfig.minPlain`
	`89`	`+ va <= jsonConfig.bigDecimalSerializerConfig.maxPlain && va >= jsonConfig.bigDecimalSerializerConfig.minPlain`
`90`	`90`	`}`
`91`	`91`	`val stripped = stripTrailingZeros(v.bigDecimal)`
`92`	`92`	`val raw = if (shouldWritePlain) stripped.toPlainString else stripped.toString`