sbcgua / ajson

Yet another json parser serializer for ABAP
MIT License
50 stars 15 forks source link

Restrict the apply of mapping conversions #111

Closed jrodriguez-rc closed 2 years ago

jrodriguez-rc commented 2 years ago

Changes to fix #110

I added a new with the issue case

image

Unit tests runs ok

image
albertmink commented 2 years ago

Change has been successfully tested in our scenario.

Thanks @jrodriguez-rc this solves the issue. @sbcgua would you mind to merge, so that it becomes available in abapGit too?

sbcgua commented 2 years ago

Yes, sorry for long response, a bit busy these days. Let's merge it, the UTs passes, code looks fine. Though in the long run I think I might adjust the behavior so that the autoCamelConversion is applied at parsing time, as I suggested in the #110 . But later. And maybe.

@jrodriguez-rc Thanks for the fix ! :+1: