search

sitemule / noxDB

Not only XML. SQL,JSON and XML made easy for IBM i
MIT License
42 stars 20 forks source link

Empty tag issues #85

Open beeceeke opened 9 months ago

beeceeke commented 9 months ago

1) In the latest version the xml_GetElemValue() crashes when an empty tag element is in the XML: . Even with trying to add a default value it isn't taken into account because the node exists: Value+= xml_GetElemValue(Tag@:' ');

2) We receive a pointer not set for location referenced error when using the data-gen operation code with an empty tag. This issue also occurs in the github example: https://github.com/sitemule/noxDB/blob/master/examples/JSONPARS9J.JSON-data-gen.rpgle

Can these issues be fixed?

NielsLiisberg commented 9 months ago

It was noticed by another client, and we are now working on a solution.

Thank you for posting this issue

tors. 1. feb. 2024 kl. 07.03 skrev beeceeke @.***>:

1.

In the latest version the xml_GetElemValue() crashes when an empty tag element is in the XML: . Even with trying to add a default value it isn't taken into account because the node exists: Value+= xml_GetElemValue(Tag@:' '); 2.

We receive a pointer not set for location referenced error when using the data-gen operation code with an empty tag. This issue also occurs in the github example: https://github.com/sitemule/noxDB/blob/master/examples/JSONPARS9J.JSON-data-gen.rpgle

Can these issues be fixed?

— Reply to this email directly, view it on GitHub https://github.com/sitemule/noxDB/issues/85, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAVIPHXUF7R43NQGLPPBL6DYRMV3FAVCNFSM6AAAAABCUJGTNSVHI2DSMVQWIX3LMV43ASLTON2WKOZSGEYTCNRSGUZDEMA . You are receiving this because you are subscribed to this thread.Message ID: @.***>

NielsLiisberg commented 9 months ago

the XML and JSON has the graph and api in common so my guess is this was fixed by issue #84. can you confirm that @beeceeke

beeceeke commented 9 months ago

Thanks for your answer. I used the release save file to install the latest version (https://github.com/sitemule/noxDB/blob/master/release/release.savf).

But it seems the save file version (Oct 27, 2023) doesn't contain fix issue #84? Will a new release save file be created?

NielsLiisberg commented 9 months ago

Sorry - but it is not in the save file, you have to build it or wait until next release.

tirs. 13. feb. 2024 kl. 20.13 skrev beeceeke @.***>:

Thanks for your answer. I used the release save file to install the latest version ( https://github.com/sitemule/noxDB/blob/master/release/release.savf).

But it seems the save file version (Oct 27, 2023) doesn't contain fix issue #84 https://github.com/sitemule/noxDB/issues/84? Will a new release save file be created?

— Reply to this email directly, view it on GitHub https://github.com/sitemule/noxDB/issues/85#issuecomment-1942214047, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAVIPHRXSKIJNIE2J54ZCWTYTO3MNAVCNFSM6AAAAABCUJGTNSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNBSGIYTIMBUG4 . You are receiving this because you commented.Message ID: @.***>

beeceeke commented 9 months ago

Ah ok, when will that next release be?