Bug 21928

Summary: CircAutoPrintQuickSlip 'clear' is not working
Product: Koha Reporter: Nick Clemens <nick>
Component: CirculationAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: major    
Priority: P5 - low CC: bwsdonna, chris, fridolin.somers, gmcharlt, jonathan.druart, kyle.m.hall, lucas, martin.renvoize
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 18789    
Bug Blocks:    
Attachments: Bug 21928: (bug 18789 follow-up) Fix CircAutoPrintQuickSlip=clear
Bug 21928: (bug 18789 follow-up) Fix CircAutoPrintQuickSlip=clear
Bug 21928: (bug 18789 follow-up) Fix CircAutoPrintQuickSlip=clear

Description Nick Clemens 2018-11-30 14:01:50 UTC
caused by commit 105b3e10e8b31b7de6d733e676c3e661a39b1280

bug 18789

To recreate:
1 - Set CircAutoPrintQuickSlip to 'clear'
2 - Bring up a patron in circulation
3 - Click in checkout box and hit enter to submit empty
4 - Patron is partially loaded
Comment 1 Jonathan Druart 2018-12-05 21:18:14 UTC Comment hidden (obsolete)
Comment 2 Owen Leonard 2018-12-17 13:55:51 UTC
Created attachment 83281 [details] [review]
Bug 21928: (bug 18789 follow-up) Fix CircAutoPrintQuickSlip=clear

caused by commit 105b3e10e8b31b7de6d733e676c3e661a39b1280

bug 18789

To recreate:
1 - Set CircAutoPrintQuickSlip to 'clear'
2 - Bring up a patron in circulation
3 - Click in checkout box and hit enter to submit empty
4 - Patron is partially loaded

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 3 Katrin Fischer 2019-01-01 22:10:33 UTC
Created attachment 83566 [details] [review]
Bug 21928: (bug 18789 follow-up) Fix CircAutoPrintQuickSlip=clear

caused by commit 105b3e10e8b31b7de6d733e676c3e661a39b1280

bug 18789

To recreate:
1 - Set CircAutoPrintQuickSlip to 'clear'
2 - Bring up a patron in circulation
3 - Click in checkout box and hit enter to submit empty
4 - Patron is partially loaded

Signed-off-by: Owen Leonard <oleonard@myacpl.org>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 4 Nick Clemens 2019-01-02 21:11:08 UTC
Awesome work all!

Pushed to master for 19.05
Comment 5 Martin Renvoize 2019-01-07 15:10:33 UTC
Pushed to 18.11.x for 18.11.02
Comment 6 Lucas Gass 2019-01-08 23:30:51 UTC
backported to 18.05 for 18.05.08
Comment 7 Fridolin Somers 2019-01-09 06:45:00 UTC
Depends on Bug 18789 not in 17.11.x