larshp / ABAP-Swagger

Expose ABAP REST services with Swagger/openapi spec
MIT License
97 stars 40 forks source link

camel cased JSON requests and responses #47

Closed xinitrc86 closed 4 years ago

xinitrc86 commented 5 years ago

Added usage of /ui2/cl_json to convert from/to camel cased JSON bodies.

larshp commented 5 years ago

what is the expected behavior regarding backwards compatibility?

xinitrc86 commented 5 years ago

Inbound body is expected to be camel cased JSON, same as outbound. I'm not sure how the transformation worked for the response, but if it adds "{DATA:" to i...that is lost. The serializer offers different options. We could add them to the meta maybe? none,
low_case,
camel_case, extended

larshp commented 5 years ago

yea, the change should not break all existing installations

larshp commented 4 years ago

should not break all existing installations