jaydata icon indicating copy to clipboard operation
jaydata copied to clipboard

$data.Decimal is converted incorrect when converting to database

Open Malyngo opened this issue 8 years ago • 1 comments

Tested in Jaydata 1.5.10. When have an entity with a decimal value and want to store that to the database (Post or Patch, doesn't matter), there is an error about a wrong format.

I looked into this and it's just that Jaydata is still using the old odata v3 encoding for that, where decimals were sent as a string. In v4 they should be sent as a number.

My fix for this currently is to add this line in my code to "hotpatch" this:

$data.oDataConverter.toDb['$data.Decimal'] = function (v: string) { return +v; };

It's this line in the oDataConverter that would need to be changed: https://github.com/jaystack/jaydata/blob/development/src/Types/StorageProviders/oData/oDataConverter.js#L86

Malyngo avatar Feb 23 '17 08:02 Malyngo

https://github.com/jaystack/jaydata/issues/259

ysmoradi avatar Feb 25 '17 10:02 ysmoradi