sbcgua / ajson

Yet another json parser serializer for ABAP
MIT License
53 stars 16 forks source link

item_order feature #45

Closed sbcgua closed 3 years ago

sbcgua commented 3 years ago

to #38

use keep_item_order to save struc field order and the serialize will pick it up.

@larshp ready for test, though I will add some docs and maybe refactor some stuff. But the feature is ready and covered with UTs.

P.S. I hope 2 extra table keys are ok for older system. 731 seems to support it ...

sbcgua commented 3 years ago

@larshp abaplint false negative ? Not the most canonical implementation ... but sap compiles it without warnings ... stringify is implemented as zif_ajson~stringify but referred as zif_ajson_writer~stringify. I will probably adjust it. But still it looks like a false negative

larshp commented 3 years ago

yea, its probably a bug, will have a look, might be similar/identical to https://github.com/abaplint/abaplint/issues/1561

larshp commented 3 years ago

https://github.com/abaplint/abaplint/issues/1661 opened

larshp commented 3 years ago

try again, 2.63.16 published

sbcgua commented 3 years ago

:+1: works now!

sbcgua commented 3 years ago

i'm lazy to do refactoring ... just added docs and merging, it works