SData-2.0 icon indicating copy to clipboard operation
SData-2.0 copied to clipboard

Accounts 50 - Deleted Flag Not Usable in SDATA

Open tazmon95 opened this issue 8 years ago • 1 comments

Hello, I've been working on creating an analytics platform using the SData api and need a way to filter out transactions which have been deleted. I see the flag present, however the values are always set to "false" when viewing the tradingActivities data. However when looking at the same data in the Sage 50 Accounts application, 1500 records are deleted.

I've tried adding in a filter for the deleted part of the record but the filter doesn't seem to work. Filters on the other columns do work.

~/sdata/accounts50/GCRM/-/tradingActivities?where=transactionType eq 'SC' and Deleted eq 'false'

Here is the error when the Deleted column is included in the query.

    <sdata:severity>error</sdata:severity>
    <sdata:sdataCode>ApplicationDiagnosis</sdata:sdataCode>
    <sdata:applicationCode />
    <sdata:message>GET operation failed - Object reference not set to an instance of an object.

   at Sage.Integration.Accounts50.SDOAdapter.SDORequestHandler.OnGetRequest(IRequest request, IFeed feed)
   at Sage.Integration.Accounts50.SDOAdapter.RequestHandler`2.OnGetRequest(IRequest request)
   at Sage.Integration.Accounts50.SDOAdapter.RequestHandler`2.GetRequest(IRequest request)
   at Sage.Integration.Accounts50.SDOAdapter.SDORequestManager.OnGetResource(IRequest request)
   at Sage.Integration.Accounts50.SDOAdapter.SDORequestManager.GetRequest(IRequest request) (-1)</sdata:message>
    <sdata:stackTrace />
    <sdata:payloadPath />
</diagnosis>```

Thanks,

tazmon95 avatar Apr 10 '17 21:04 tazmon95

It looks like the deleted flag in the Sdata feeds for tradingActivities is hard coded to true so does not provide any in indication of if the record has been deleted in the main application. The fact that it is hard coded is also the reason you get an error when trying to filter on that field.

I've logged a bug with the Accounts team

darroncockram avatar Apr 11 '17 06:04 darroncockram