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.

Sign up for GitHub

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

Jump to bottom

@JsonbTypeDeserializer is ignored on Enums in 3.0.3; works in 2.0.4 #617

Open

api-from-the-ion opened this issue Oct 5, 2023 · 0 comments

Labels

bug

api-from-the-ion commented Oct 5, 2023

Describe the bug
I wanted a custom Serializer / Deserializer for my enum. I found the adapter code here and separate it into two different classes. But the @JsonbTypeDeserializer is ignored in 3.0.3, and not in 2.0.4. Both @JsonbTypeSerializer and @JsonbTypeAdapter are registered in 3.0.3 and 2.0.4. So it's only @JsonbTypeDeserializer which is ignored.

To Reproduce
I created a small project with both scenarios, Serializer / Deserializer and adapter. One can comment one or another. One can also easily play with Yasson version there and see the test falls through or not.
https://github.com/api-from-the-ion/yasson-enum-bug-evaluation

Expected behavior
The Deserializer and test should work both in 3.x and 2.x versions,

System information:

OS: Linux
Java Version: 11, 17
Yasson Version:3.0.3

api-from-the-ion added the bug label

api-from-the-ion added a commit to api-from-the-ion/yasson that referenced this issue


          [eclipse-ee4j#617]User defined deserializer always should be used bef…

444d929

…ore standard ones

api-from-the-ion mentioned this issue

[#617]User defined deserializer always should be used before standard… #619

Open

api-from-the-ion added a commit to api-from-the-ion/yasson that referenced this issue


          [eclipse-ee4j#617]Test for enum with registered adapter

170103e

It is working fine right now, but it also could be tested.

Signed-off-by: Anton Pinsky <anton.pinsky@ionos.com>

api-from-the-ion added a commit to api-from-the-ion/yasson that referenced this issue


          [eclipse-ee4j#617]Forgot the copyright

429bf76

Signed-off-by: Anton Pinsky <anton.pinsky@ionos.com>

api-from-the-ion added a commit to api-from-the-ion/yasson that referenced this issue


          [eclipse-ee4j#617]Small refactoring of the creators

97f708f

Signed-off-by: Anton Pinsky <anton.pinsky@ionos.com>

api-from-the-ion added a commit to api-from-the-ion/yasson that referenced this issue


          [eclipse-ee4j#617]Extra copyright changes

f0eff31

Signed-off-by: Anton Pinsky <anton.pinsky@ionos.com>

api-from-the-ion added a commit to api-from-the-ion/yasson that referenced this issue


          [eclipse-ee4j#617]Even more copyright changes

Signed-off-by: Anton Pinsky <anton.pinsky@ionos.com>

api-from-the-ion added a commit to api-from-the-ion/yasson that referenced this issue


          [eclipse-ee4j#617]Checkstyle chnages

c63bdf6

Signed-off-by: Anton Pinsky <anton.pinsky@ionos.com>

api-from-the-ion added a commit to api-from-the-ion/yasson that referenced this issue


          [eclipse-ee4j#617]And once again the copyright

c92319f

Signed-off-by: Anton Pinsky <anton.pinsky@ionos.com>

api-from-the-ion added a commit to api-from-the-ion/yasson that referenced this issue


          Revert "[eclipse-ee4j#617]And once again the copyright"

efc574e

This reverts commit c92319f.

api-from-the-ion added a commit to api-from-the-ion/yasson that referenced this issue


          [eclipse-ee4j#617]And once again the copyright

87fce39

Signed-off-by: Anton Pinsky <anton.pinsky@ionos.com>

api-from-the-ion added a commit to api-from-the-ion/yasson that referenced this issue


          [eclipse-ee4j#617]Put every call of JsonbBuilder.create in the tests …

4e5a0a1

…into try-with-resources.

Signed-off-by: Anton Pinsky <anton.pinsky@ionos.com>

api-from-the-ion added a commit to api-from-the-ion/yasson that referenced this issue


          [eclipse-ee4j#617]Made new tests public

e5346b3

Signed-off-by: Anton Pinsky <anton.pinsky@ionos.com>

api-from-the-ion added a commit to api-from-the-ion/yasson that referenced this issue


          [eclipse-ee4j#617]Warnings on static inner classes and interfaces rem…

d3e84ec

…oved

Signed-off-by: Anton Pinsky <anton.pinsky@ionos.com>

api-from-the-ion added a commit to api-from-the-ion/yasson that referenced this issue


          [eclipse-ee4j#617]Removed som not thrown Exceptions in the tests

d54416c

Signed-off-by: Anton Pinsky <anton.pinsky@ionos.com>

api-from-the-ion added a commit to api-from-the-ion/yasson that referenced this issue


          [eclipse-ee4j#617]Less raw usage warnings

9c1a95f

Signed-off-by: Anton Pinsky <anton.pinsky@ionos.com>

api-from-the-ion added a commit to api-from-the-ion/yasson that referenced this issue


          [eclipse-ee4j#617]Small call and variable optimisation

2e9bfe2

Signed-off-by: Anton Pinsky <anton.pinsky@ionos.com>

api-from-the-ion added a commit to api-from-the-ion/yasson that referenced this issue


          [eclipse-ee4j#617]Pattern Matching for instanceof removed - not befor…

6fb1442

…e Java 16

Signed-off-by: Anton Pinsky <anton.pinsky@ionos.com>

api-from-the-ion added a commit to api-from-the-ion/yasson that referenced this issue


          [eclipse-ee4j#617]Use defaultJsonb where JsonbBuilder.create() withou…

062a161

…t configuration is used

Signed-off-by: Anton Pinsky <anton.pinsky@ionos.com>

api-from-the-ion added a commit to api-from-the-ion/yasson that referenced this issue


          [eclipse-ee4j#617]Pattern Matching for instanceof removed - not befor…

…e Java 16

Signed-off-by: Anton Pinsky <anton.pinsky@ionos.com>

api-from-the-ion added a commit to api-from-the-ion/yasson that referenced this issue


          [eclipse-ee4j#617]StringBuilder.isEmpty() only available since Java 15

451246a

Signed-off-by: Anton Pinsky <anton.pinsky@ionos.com>

ossnms added a commit to ossnms/yasson that referenced this issue


          work around for eclipse-ee4j#617

02dd61f

api-from-the-ion added a commit to api-from-the-ion/yasson that referenced this issue


          [eclipse-ee4j#617] Changed the year to meanwhile 2024; dependecies / …

ef7da0a

…plugins update in POM; assertInstanceOf where this is possible

Signed-off-by: Anton Pinsky <anton.pinsky@ionos.com>

api-from-the-ion added a commit to api-from-the-ion/yasson that referenced this issue


          [eclipse-ee4j#617] Refactored repeated code in SerializationModelCrea…

e6054b2

…tor & DeserializationModelCreator in serializer / deserializer creation from serializer / deserializer / adapter configuration in annotations

Signed-off-by: Anton Pinsky <anton.pinsky@ionos.com>

api-from-the-ion added a commit to api-from-the-ion/yasson that referenced this issue


          [eclipse-ee4j#617] Refactored repeated code in SerializationModelCrea…

fa1f4e1

…tor & DeserializationModelCreator in serializer / deserializer creation from serializer / deserializer / adapter configuration in annotations

Signed-off-by: Anton Pinsky <anton.pinsky@ionos.com>

api-from-the-ion added a commit to api-from-the-ion/yasson that referenced this issue


          [eclipse-ee4j#617] Refactored code with Optional - use more Optional API

0574ee2

Signed-off-by: Anton Pinsky <anton.pinsky@ionos.com>

api-from-the-ion added a commit to api-from-the-ion/yasson that referenced this issue


          [eclipse-ee4j#617] More use of PropertyCustomization.Builder as builder

d167fd7

Signed-off-by: Anton Pinsky <anton.pinsky@ionos.com>

api-from-the-ion added a commit to api-from-the-ion/yasson that referenced this issue


          [eclipse-ee4j#617] Less unused code in AnnotationIntrospector & Refle…

ceabcf0

…ctionUtils

Signed-off-by: Anton Pinsky <anton.pinsky@ionos.com>

api-from-the-ion added a commit to api-from-the-ion/yasson that referenced this issue


          [eclipse-ee4j#617] First corrections after review

5ad157e

Signed-off-by: Anton Pinsky <anton.pinsky@ionos.com>

api-from-the-ion added a commit to api-from-the-ion/yasson that referenced this issue


          [eclipse-ee4j#617] Higher test coverage for DeserializationModelCreat…

d287108

…or and DeserializationModelCreator; fixed bug in ReflectionUtils

Signed-off-by: Anton Pinsky <anton.pinsky@ionos.com>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment