Hi,
based on https://documentation.progress.com/output/pdo/#page/pdo%2Ffill(-)-method.html%23 the JSDO should support filtering by id (the doc says, that the Kendo UI does not support this, but this is not relevant for me right now).
jsdo.fill ({ id: 7 })
However the id parameter is not sent to the server. I debugged the JFP mapping code and the id parameter is not included in the filter object send to the backend.
https://github.com/CloudDataObject/JSDO/blob/master/lib/progress.jsdo.js - line 7730
filter = JSON.stringify({ ablFilter: ablFilter, sqlQuery: sqlQuery, orderBy: sortFields, skip: params.skip, top: params.top }); params = {filter: filter};
I have added the id parameter (id: parameter.id) to the filter object. And then it works.
Is the documentation wrong? Or the code?
Hello Mike,
This issue is now fixed in the develop brach and will be available in a future release.
Thanks,
Edsel
Hello Mike,
It looks like my post from Tuesday did not go through...
This issue was an omission in the code. As you might have seen, it is listed in the reqCapabilities variable as well.
Please noticed that this id parameter to the JFP filter is not the same as the internal id used by findById() or getId().
For this you would need to tell the JSDO to use the remote id as the internal id. I have added a story so this could be addressed in a future release.
Thank you and regards,
Edsel
So it seems that this has been added to the JSDO now :-)
Hello Mike,
It looks like my post from Tuesday did not go through...
This issue was an omission in the code. As you might have seen, it is listed in the reqCapabilities variable as well.
Please noticed that this id parameter to the JFP filter is not the same as the internal id used by findById() or getId().
For this you would need to tell the JSDO to use the remote id as the internal id. I have added a story so this could be addressed in a future release.
Thank you and regards,
Edsel
Hello Mike,
This issue is now fixed in the develop brach and will be available in a future release.
Thanks,
Edsel