Bug 8056

Summary: CircAutoPrintQuickSlip set to clear doesn't work
Product: Koha Reporter: Nicole C. Engard <nengard>
Component: CirculationAssignee: Srdjan Jankovic <srdjan>
Status: CLOSED FIXED QA Contact:
Severity: critical    
Priority: P5 - low CC: chris, gmcharlt, julian.maurice, koha.sekjal, paul.poulain
Version: 3.8   
Hardware: All   
OS: All   
See Also: http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7001
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: patch
[SIGNED-OFF] bug_8056: sorted out slip print calls
patch
patch
[SIGNED-OFF] bug_8056: sorted out slip print calls

Description Nicole C. Engard 2012-05-07 15:08:43 UTC
I have CircAutoPrintQuickSlip set to clear the screen but when I enter a blank barcode I get the following error:


Software error:

Undefined subroutine &main::PrintIssueSlip called at /home/nengard/kohaclone/circ/circulation.pl line 175.
Comment 1 Srdjan Jankovic 2012-05-09 02:52:15 UTC Comment hidden (obsolete)
Comment 2 Nicole C. Engard 2012-05-09 14:08:30 UTC Comment hidden (obsolete)
Comment 3 Ian Walls 2012-05-10 16:11:01 UTC
Breaks Plack scoping fix for our $todaysdate, throwing it back to 'my'.

Marking Failed QA.
Comment 4 Srdjan Jankovic 2012-05-11 02:25:55 UTC Comment hidden (obsolete)
Comment 5 Julian Maurice 2012-05-11 11:52:03 UTC
I don't know if this file is used, but t/db_dependent/lib/KohaTest/Print.pm contains:

    my @methods = qw( printslip );

and the patch remove this sub, so I think this file should be modified too.
Comment 6 Srdjan Jankovic 2012-05-14 03:19:54 UTC Comment hidden (obsolete)
Comment 7 Nicole C. Engard 2012-05-14 11:54:42 UTC
Created attachment 9557 [details] [review]
[SIGNED-OFF] bug_8056: sorted out slip print calls

Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
preference works as expected.
Comment 8 Paul Poulain 2012-05-28 11:56:39 UTC
QA comment:
* remove more code than it adds. Good point ;-)
* clean code, passes perlcritic (remove 1 perlcritic problem. 2 still here, but were here before the patch)

passed QA
Comment 9 Chris Cormack 2012-05-28 20:58:21 UTC
No database changes, fixes a regression, pushed to 3.8.x will be in 3.8.x