Bug 7331

Summary: Circulation.pl generates errors in the log
Product: Koha Reporter: Owen Leonard <oleonard>
Component: CirculationAssignee: Duncan <duncan>
Status: CLOSED FIXED QA Contact: Ian Walls <koha.sekjal>
Severity: minor    
Priority: PATCH-Sent (DO NOT USE) CC: chris, duncan, gmcharlt, paul.poulain
Version: 3.6   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Patch to stop spamming of error logs
Patch to fix warning logs being printed unnecessarily
Signed-off patch

Description Owen Leonard 2011-12-07 16:06:24 UTC
Both when loading circulation.pl initially and after submitting a patron barcode, the page generates multiple entries in the log:

circulation.pl: Use of uninitialized value $C4::Dates::debug in numeric gt (>) at /kohaclone/C4/Dates.pm line 170.
Comment 1 Owen Leonard 2011-12-07 16:08:01 UTC
The same error is generated by moremember.pl:

moremember.pl: Use of uninitialized value $C4::Dates::debug in numeric gt (>) at /kohaclone/C4/Dates.pm line 170
Comment 2 Duncan 2011-12-07 23:46:06 UTC Comment hidden (obsolete)
Comment 3 Owen Leonard 2011-12-08 00:56:33 UTC
Was this patch supposed to go with a different bug?
Comment 4 Chris Cormack 2011-12-08 01:02:19 UTC
Ahh it does look like it should be, Duncan is our Summer of Tech student and is just learning his way around. 

We'll have another go :)
Comment 5 Duncan 2011-12-08 01:19:49 UTC Comment hidden (obsolete)
Comment 6 Owen Leonard 2011-12-08 13:54:40 UTC
Created attachment 6665 [details] [review]
Signed-off patch

Quick work, thank you!
Comment 7 Paul Poulain 2011-12-15 16:38:37 UTC
QA comment: tiny patch, silent log

passed QA

patch pushed, please test
Comment 8 Jared Camins-Esakov 2012-05-23 12:17:30 UTC
Included in 3.6 prior to 3.6.5.