Skip empty nodes at serialization

jrodriguez-rc commented 2 years ago

This new option will skip all the nodes that have no content:

Simple node values empty/null
Structures/Arrays with no subnodes

sbcgua commented 2 years ago

Do you mean without filter ? https://github.com/sbcgua/ajson#predefined-filters

Also:

structures/arrays should be ok
but values are not. Is false empty ? Or null ? Or 0. I think not always.

mbtools commented 4 months ago

I would go with the most famous isEmpty implementation: https://github.com/lodash/lodash/blob/main/src/isEmpty.ts

It's working differently now:

https://github.com/sbcgua/ajson/blob/2504b311e7e4695f1aea94f11c3cc883c4df8391/src/libs/zcl_ajson_filter_lib.clas.locals_imp.abap#L11-L18

I changed my custom empty filter to the following which also removes null and initial numbers (zero).

    rv_keep = boolc(
      ( iv_visit = zif_ajson_filter=>visit_type-value AND
        ( is_node-type = zif_ajson_types=>node_type-string AND is_node-value IS NOT INITIAL OR
          is_node-type = zif_ajson_types=>node_type-boolean OR
          is_node-type = zif_ajson_types=>node_type-number AND is_node-value <> 0 ) ) OR
      ( iv_visit <> zif_ajson_filter=>visit_type-value AND is_node-children > 0 ) ).

sbcgua / ajson

Skip empty nodes at serialization #121