Hotfix `write_dwc` failing sql parsing + support uppercase

PietrH commented 5 months ago

Set extra parameter to ensure passing NULL to glue is passed as "NULL" strings to the database, instead of the whole query being dropped.
Updated a snapshot, because the database has changed since the last unit test check.
Convert animal_project_code to lowercase for SQL injection in write_dwc(): fixes #289 : bff9852f

Since glue v1.7.0 a literal NULL will result in the line (in this case, the whole file) being dropped. Before it was returned as "NULL" by default. This change set's it to it's old default behaviour.

In principle this bug would have been present as soon as someone updated glue as of the 10th of January this year.

PietrH commented 5 months ago

@peterdesmet, I also added a fix for #289 , Sorry, I didn't notice you already reviewed.

If you give the go ahead, I'll merge.

PietrH commented 5 months ago

Can't merge without an accepted review! Are we ok to merge @peterdesmet ?

inbo / etn

Hotfix `write_dwc` failing sql parsing + support uppercase #294