Add `ElementType.PARAMETER` to `@JsonIgnore` to allow use for Constructor parameters

[cowtowncoder]

(note: off-shoot of FasterXML/jackson-databind#4626)

Looks like @JsonIgnore cannot be used on method/constructor parameters: latter can be problematic with Record types in particular (but also regulard POJOs' Constructors or Factory methods).
So let's add that in 2.18.

[yihtserns]

One would expect Kotlin Data Class to handle annotations the same as Java Records, i.e. propagate annotation to fields and/or getters and/or constructor parameters depending on @Target, but apparently not.

Looking at the decompiled Kotlin Data Class, something like this:

data class BooleanPropertyInBody(@JsonIgnore val placeholder: String = "placeholder") { ... }

• Before this change: @JsonIgnore added on the generated field.
• After this change: @JsonIgnore added only on the constructor parameter. 🤨

UPDATE

Annoying - from https://kotlinlang.org/docs/annotations.html#annotation-use-site-targets:

If you don't specify a use-site target, the target is chosen according to the @Target annotation of the annotation being used. If there are multiple applicable targets, the first applicable target from the following list is used:

• param
• property
• field

[yihtserns]

Ugh, I missed out an important fact in FasterXML/jackson-databind#4626:

1. @JsonIgnore's @Target not including ElementType.PARAMETER is only one-half of the problem.
2. prop.addCtor(..., false) <--- this hardcoded false in POJOPropertiesCollector is the other half of the problem.

Sorry!

[cowtowncoder]

Uggh. As little as I like the idea, maybe I should revert this change. WDYT @yihtserns .

But at least it was possible to pinpoint this change as the root cause for failing tests.

[yihtserns]

I think it's fine to revert at the first sign of problem - I believe you have more important things to focus on than this (+ to reduce any unnecessary risk for the upcoming 2 18 release).

And it's not like this is something urgent or requested, we can always do it later if we really want to.

[cowtowncoder]

PR reverted.