Bug 23463: Fix selenium tests (highlight a bug in date management)
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Mon, 23 Mar 2020 12:09:07 +0000 (13:09 +0100)
committerMartin Renvoize <martin.renvoize@ptfs-europe.com>
Mon, 23 Mar 2020 12:23:16 +0000 (12:23 +0000)
commit757ca57bfdbd4346b90293429e658d239b8d0102
tree19496d9a7410a12fe0d1dde92f914aa6183d0337
parent243a5395de9be3e3971ae8171a8b2b3aa79e74bb
Bug 23463: Fix selenium tests (highlight a bug in date management)

There is something wrong, and a regression has been caught by those
tests:
If an invalid date is passed from the add item form, the app now
crashes.
Before:
 * if the date was completely invalid, the field was blanked
silently
 * DateTime::Format::MySQL was used to convert dates, and it's not
 strict at all. For instance, what happened in the selenium tests for
 dateaccessionned: %Y-%m-%d was prefilled by the framework plugin, then
 the biblionumber was added, we ended with something like (eg for today)
 2020-03-234242 (with biblionumber=4242). DateTime::Format::MySQL
 converts that to 2020-03-23

We must deal with invalid dates, but I do not think it is good to add it
back to Koha::Item->store, we will prefer to raise the error to the end
user, saying that something went wrong (and more specifically the
dates).

The (ugly) trick was in C4::Items::_mod_item_dates

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
t/db_dependent/selenium/basic_workflow.t