search

sitemule / noxDB

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

Regression since e708612 #28

Closed wimjongman closed 11 months ago

wimjongman commented 4 years ago

Commit e708612 introduced a regression.

When building noxDB with this commit, the Json_GetStr procedure returns blanks. When building NoxDB with the previous commit f0a1e75 the procedure works fine.

I will provide my example in a pull request.

NielsLiisberg commented 4 years ago

We are having some issue with different use of ccsid’s. So my guess is it works if you set your ccsid to 500 befor doing the regression test. I am sorry for the inconvinience but i have to make it work on all (if possible ) ccsids .. so this is kind of work in progress. There is anothe branche on this github site where the object graph and all access to it is 100% utf-8 which is the Way to go, however so many is already using this Master so i have to do some work for alle the EBCDIC version as well. Thank you for the input . And until the use the commit that works for you.

wimjongman commented 4 years ago

Ok, the UTF8 branch sounds great. I will give that a spin as well.