jackson-databind icon indicating copy to clipboard operation
jackson-databind copied to clipboard

Fix #3053 (Serialize enums to lowercase)

Open vojkny opened this issue 3 years ago • 2 comments

vojkny avatar Feb 12 '21 13:02 vojkny

Apologies for letting this one fall through the cracks: I think it's a good PR.

I am +1 for adding configurability, but not through existing SerializationFeature, only because its capacity is limited and the idea is to keep that for more general settings. Up until now, however, there hasn't been any good alternative. But Jackson 2.14 will finally implement this:

https://github.com/FasterXML/jackson-future-ideas/wiki/JSTEP-7

and I will try to collect ideas for EnumFeature options: this one sounds like a very good candidate. To support this, there's some work to be done, similar to how #3421 added JsonNodeFeature: @knyttl if you are still interested, I could help make this happen. Or if you don't have time I can maybe implement this on my own when I get some more time.

cowtowncoder avatar May 02 '22 21:05 cowtowncoder

I am afraid I am not entirely sure about the way this should be implemented so if you want to implement it differently I guess, you better do it yourself 🙏

vojkny avatar May 03 '22 06:05 vojkny

I recently worked on EnumSerialization a little bit. ✌️ I will try to make a PR if I have time

JooHyukKim avatar Feb 06 '23 21:02 JooHyukKim

I recently worked on EnumSerialization a little bit. ✌️ I will try to make a PR if I have time

Yes, that'd be great! I didn't have time to do this but think it would be quite doable.

cowtowncoder avatar Feb 06 '23 22:02 cowtowncoder

I recently worked on EnumSerialization a little bit. ✌️ I will try to make a PR if I have time

Yes, that'd be great! I didn't have time to do this but think it would be quite doable.

For sure! done in #3776

JooHyukKim avatar Feb 06 '23 23:02 JooHyukKim