Bug 7077 - Add system pref to control behavior of submitting empty barcode field on circulation page
Summary: Add system pref to control behavior of submitting empty barcode field on circ...
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Circulation (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement
Assignee: Owen Leonard
QA Contact: Marcel de Rooy
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-10-21 18:28 UTC by Owen Leonard
Modified: 2013-12-05 20:01 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:
Circulation function:


Attachments
Proposed fix (5.19 KB, patch)
2011-10-21 19:08 UTC, Owen Leonard
Details | Diff | Splinter Review
Fix for Bug 7077 - Add system pref to control behavior of submitting empty barcode field (5.29 KB, patch)
2011-11-03 21:54 UTC, Chris Cormack
Details | Diff | Splinter Review
Signed patch (5.64 KB, patch)
2011-11-30 13:54 UTC, Marcel de Rooy
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Owen Leonard 2011-10-21 18:28:40 UTC
It used to be that submitting an empty barcode input field when a patron's account was open in Circulation would clear the page (the same behavior as the "clear screen" buton offers now). Since inclusion of Bug 4389 this action has been co-opted to pop up a quick slip print window. A system preference should be added to allow the library to choose which behavior they want.
Comment 1 Owen Leonard 2011-10-21 19:08:45 UTC Comment hidden (obsolete)
Comment 2 Paul Poulain 2011-10-24 11:38:19 UTC
Updating Version : This ENH will be for Koha 3.8
Comment 3 Paul Poulain 2011-10-25 15:05:55 UTC
Bug versionned for master. entries will be made against rel_3_8 once the patch has been applied (see thread about that on koha-devel yesterday)
Comment 4 Chris Cormack 2011-11-03 21:54:02 UTC Comment hidden (obsolete)
Comment 5 Marcel de Rooy 2011-11-30 13:53:36 UTC
QA Comment: Code looks good. Marking it as Passed QA.
Note that this patch contains a dbrev.
Comment 6 Marcel de Rooy 2011-11-30 13:54:59 UTC
Created attachment 6470 [details] [review]
Signed patch

Fixed another merge conflict in updatedatabase.
Comment 7 Paul Poulain 2012-01-06 16:46:32 UTC
Patch pushed, please test