Skip to content

Change the hasRequiredMarker implementation to the 3.x specification #941

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
k163377 opened this issue Apr 12, 2025 · 0 comments · Fixed by #952
Closed

Change the hasRequiredMarker implementation to the 3.x specification #941

k163377 opened this issue Apr 12, 2025 · 0 comments · Fixed by #952
Labels
3.x Issue affecting/planned for Jackson 3.x

Comments

@k163377
Copy link
Contributor

k163377 commented Apr 12, 2025

The hasRequiredMarker is currently implemented in the KotlinAnnotationIntrospector so that the old required flag does not override the kotlin-module process.
On the other hand, this will override processing by other AnnotationIntrospectors, and therefore will also override the user's own specifications.

In 3.x, the old required flag will be ignored if it is false, so that the kotlin-module process is not overridden.
Therefore, hasRequiredMarker will be moved to KotlinNamesAnnotationIntrospector to avoid overriding user specifications as much as possible.
During this, considerations for JsonProperty annotations in the current implementation will also be removed as they are no longer needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3.x Issue affecting/planned for Jackson 3.x
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant