search

openthc / ccrs

Tracking for the WSLCB CCRS
https://lcb.wa.gov/ccrs
MIT License
4 stars 0 forks source link

CheckSum Errors #39

Closed djbusby closed 2 years ago

djbusby commented 2 years ago

There appears to be a condition on Product uploads where the LCB parser breaks and fails to count rows correctly.

The error message is "CheckSum and number of records don't match"

CheckSum refers to the the "NumberRecords" field provided in the uploaded CSV file.

Observed on Inventory and on Product files

djbusby commented 2 years ago

Related to Bug #38 is also this error is appended with a COMMA -- when all others seem to use a colon?

412717, Main Section, , 412717-018NY6XC00SECT10N000000000, -system-, 12/29/2021, -system-, 12/29/2021, INSERT, Integrator is not authorized to update licensee,  CheckSum and number of records don't match
djbusby commented 2 years ago

Saw this on a Sections (Area) file too. Uploaded a file with mixed INSERT, UPDATE and DELETE -- and the Canary row. Filed as: IR51958

djbusby commented 2 years ago

IR51958 -- hexdump on the files shows mixed line-endings -- there is only CSRF

LCB says I've got mixed line endings.

djbusby commented 2 years ago

So, I've just reviewed my files again on this. Many of them had this mixed line-ending issue -- on the header rows. But only some of them returned the CheckSum error -- which is odd. Shouldn't all of the files that had the mixed line endings have errored out?