patrickp / wobj Goto Github PK

Eliminate need for separate code for different platforms

• MX0C vs MX
• $options SECONDS
• SORT

MX0C should be handled with a equate
Options can be done via a include (that could set the equate)
S0RT -??

Currently this library is encoding any characters outside the printable range. This is incorrect. It needs to only encode what is required by the RFC spec. Need to review what to do on platforms that cannot handle UTF-8? Did a quick test on jBase and removing the high encoding did make utf-8 work correctly.

Add error handling for.

No object passed.
Invalid Action
Any null entries passed as options.

When the feature to be able to add a actual json object/array when doing a set.object or set.array was added the old behavior broke.

CALL WOBJ(OBJ,"SET.OBJECT","newobject","","",RERR) ; * This used to add a blank object, it now errors.
CALL WOBJ(OBJ,"SET.OBJECT","newobject","{}","",RERR); * this is what is needed now.

same with set.array
CALL WOBJ(OBJ,"SET.ARRAY","Array","","",RERR); * while this looks to work you cannot add a object to it
CALL WOBJ(OBJ,"SET.ARRAY,"Array","[]","",RERR); * this seems to work.

Propose that for either function if no value is sent to default to {} for object and [] for aray.

It would be nice to be able to pull normal arrays (not array of objects) into a pick multivalue. Same for setting.

Example
ARRAY=1
ARRAY<-1>=2
ARRAY<-1>=3
CALL WOBJ(OBJ,"SET.ARRAY","testarray",ARRAY,"",RERR)

Recommend looking to see if ARRAY starts with a { or a [. If it does not then look for @am and use that to build
[ "1", "2", "3" ] and add that.

Same on extract. Perhaps a new action such as GET.ARRAY.MV and return it as a multivalue.

Verify parsing is working right when last item is a NULL. It will convert to json just fine, but a get generates an error. You will also see an error on the parse complaining about missing last }. It looks to be an issue possibly with determine the end of those items and a } with no space. Example

{ "value": null}
May have something to do with no quote and knowing we are pass the NULL entry.

The latest version is no longer allowing you to parse json when the root element is a array. This looks to be a error check on the fromstring function checking that the first character is a {. this needs to also allow a [. Also verify the root being an array continues to work.

Allow set object and set array to take an actual object vs requiring it to be converted first json. This is an issue in that you cannot just WOBJ again due to the named common. Better solution may be to just interrogate the object/arrays and punch them in.