In opendmarc-import we should never set values to NULL (or anywhere else for that matter).
$dbi_s = $dbi_h->prepare("UPDATE requests SET repuri = NULL WHERE id = ?");
Should be
$dbi_s = $dbi_h->prepare("UPDATE requests SET repuri = '' WHERE id = ?");
Could not find any other locations (didn't dig very deep), however we should always ensure we NEVER set a value to null as to ensure strict mode compatibility and ensure matching on empty records remains sane.
The attached patch fixes the incompatibility with MySQL's strict mode. It obsoletes also ticket #192.
Looks good to me, don't forget to run aclocal and automake if your system doesn't have automake-1.13 anymore.
Why was this never merged?
afaik '= NULL' should at least be 'IS NULL' with mysql/mariadb in order to work as intended