Bug 17680: [QA Follow-up] dt_from_string failing silently
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Fri, 28 Jul 2017 08:29:51 +0000 (10:29 +0200)
committerJonathan Druart <jonathan.druart@bugs.koha-community.org>
Fri, 28 Jul 2017 14:12:37 +0000 (11:12 -0300)
This is a trivial fix, but finding the cause for a failing checkin in
SIPServer surely was not.
Without this simple change, I could not checkin an item while the SIP server
just closed the connection and syslog contained nothing useful.

The error can be easily reproduced without this patch by:
    use Modern::Perl;
    use C4::SIP::ILS::Item;
    my $item = C4::SIP::ILS::Item->new('some_barcode');
where some_barcode should refer to an item checked out.
You should trigger the error (with your due date):
    2017-07-28 23:59:00 is not an object at line 3.

Why the ILS/Item.pm module still compiles without a warning, is not clear
though.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Colin Campbell <colin.campbell@ptfs-europe.com>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
C4/SIP/ILS/Item.pm

index 7ef1d3a..12bf1aa 100644 (file)
@@ -25,6 +25,7 @@ use C4::Reserves;
 use Koha::Database;
 use Koha::Biblios;
 use Koha::Checkouts;
+use Koha::DateUtils;
 use Koha::Patrons;
 use Koha::Items;