catalyst / moodle-tool_dataflows

A generic workflow and processing engine which can be configured to do a large variety of tasks.
GNU General Public License v3.0
10 stars 6 forks source link

CSV Reader - array combine warning to provide better error messages #842

Closed keevan closed 8 months ago

keevan commented 8 months ago

Array combine warning when number of fields and headers do not match. This can be hard to diagnose without seeing the fields in question, much more so if this was part of an integration dataflow and local files were not involved at all.

The current behaviour causes false to be returned in the generator function, and any field references thereafter will fail. Since this is the case, implementing a hard stop/abort here will keep the same behaviour.