modmore / importX

Addon for MODX Revolution which will take your CSV Formatted files (or raw paste) and transforms it into resources. Funded by Working Party (digital agency based in Sydney, @workingparty)
http://www.markhamstra.com/
23 stars 17 forks source link

change reported line number for errors #44

Open christianhanvey opened 11 years ago

christianhanvey commented 11 years ago

When there's an error with column count, the reported line the error is on is actually one less than the actual error. I guess this is because the rows are counted from the actual import data, and don't include the header row - but as files/raw data always require a header row, why not report the actual line?

Sample output below, where the actual error is on line 9, it is reported as on line 8:

Console running...
Attempted to set execution time to infinite. Max execution time currently: infinite.
Running pre-import tests on submitted data...
Data file upload overriding any manual input. Filename: products.txt
No errors in pre-import found. Preparing import values...
Error: Element count do not match. Please check for correct syntax on line 8
--- Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/35449585-change-reported-line-number-for-errors?utm_campaign=plugin&utm_content=tracker%2F1511225&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F1511225&utm_medium=issues&utm_source=github).