Bug 15423 - Batch checkout dateformat error
Summary: Batch checkout dateformat error
Status: RESOLVED DUPLICATE of bug 15429
Alias: None
Product: Koha
Classification: Unclassified
Component: Circulation (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal (vote)
Assignee: Bugs List
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-12-26 20:42 UTC by Claudio Costales
Modified: 2015-12-28 13:31 UTC (History)
4 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Batch checkout dateformat error (105.54 KB, application/pdf)
2015-12-26 20:42 UTC, Claudio Costales
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Claudio Costales 2015-12-26 20:42:02 UTC
When checking out in batch mode the system crashes:

The given date (26/08/2021 00:00) does not match the date format (sql) at /usr/share/koha/lib/Koha/DateUtils.pm line 153.

Syspref dateformat is set to dd/mm/yyyy but the error occurs with other dateformat configuration.

See image attached please.
Comment 1 Claudio Costales 2015-12-26 20:42:56 UTC
Created attachment 45988 [details]
Batch checkout dateformat error
Comment 2 Claudio Costales 2015-12-26 20:46:22 UTC
Tested in 3.22 and 3.22.1
Comment 3 Marc Véron 2015-12-27 06:44:35 UTC
Confirmed for the following situation:

Enter 2 valid barcodes
Then enter a line with one space character
Then enter one more empty line or a barcode

e.g.
1
2
 
3
('empty' line contains one space character)


Result:
The given date (13/05/2022 00:00) does not match the date format (iso) at /usr/share/kohaclone/Koha/DateUtils.pm line 153.
Comment 4 Marc Véron 2015-12-27 23:56:20 UTC
Strange, in circ/circulation.pl the patron's expiration date format suddenly changes:

Around line 413, before the line my $issue = AddIssue(...
$borrower->{'dateexpiry'} is: 2022-05-13
after Addissue(...
$borrower->{'dateexpiry'} is: 05/13/2022 00:00

Digging deeper, the value seems to change in C4/Letters.pm
Line 803 (sub _parseletter)
$values->{'dateexpiry'} = format_sqldatetime( $values->{'dateexpiry'} );
Comment 5 Marc Véron 2015-12-28 09:37:54 UTC
With Bug 15429 applied I can no longe reproduce the issue.
Comment 6 Claudio Costales 2015-12-28 12:27:55 UTC
Issue resolved applying patch described in bug #15429.

Tested in 3.22.1

Thanks Marc!

(Should I change the status of this bug to RESOLVED?)
Comment 7 Marc Véron 2015-12-28 13:31:54 UTC

*** This bug has been marked as a duplicate of bug 15429 ***