Also check timestamp types for update() when trying to update a timestamp field with a blank value

[demeritcowboy]

Overview

In #319 insert() was patched to also look for timestamp db fields when inserting a blank date in a field that allows nulls. But update() was not also patched so it has the same problem with timestamp types.

Before

See unit test at civicrm/civicrm-core#20891 - it fails with:
DB Error: unknown error
Incorrect datetime value: '0' for column whatever at row 1

After

stores null in the db

Technical Details

It only fails if sql_mode has NO_ZERO_DATE.

Comments

civicrm/civicrm-core#20476

[civibot]

(Standard links)

• If this is your first pull-request for CiviCRM, please browse CONTRIBUTING.md for information about the development and testing processes.
• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[seamuslee001]

Looks fine to me