| Summary: | Behavior change when an empty barcode field is submitted in circulation | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Fridolin Somers <fridolin.somers> |
| Component: | Circulation | Assignee: | Fridolin Somers <fridolin.somers> |
| Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
| Severity: | normal | ||
| Priority: | P5 - low | CC: | gmcharlt, kyle.m.hall, kyle, victor |
| 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: |
21.11.00,21.05.05,20.11.11
|
|
| Circulation function: | |||
| Bug Depends on: | 17845 | ||
| Bug Blocks: | |||
| Attachments: |
Bug 29026: Restore behavior when an empty barcode field is submitted in circulation
Bug 29026: Restore behavior when an empty barcode field is submitted in circulation Bug 29026: Restore behavior when an empty barcode field is submitted in circulation |
||
|
Description
Fridolin Somers
2021-09-15 07:26:07 UTC
Created attachment 124877 [details] [review] Bug 29026: Restore behavior when an empty barcode field is submitted in circulation When preference CircAutoPrintQuickSlip is 'clear the screen', when an empty barcode field is submitted in circulation : Before Bug 17845, behavior was to display empty circulation page (no patron selected). Now behavior is displaying the circulation page of current patron. I think previous behavior was used when scanner is used to clear form and wait for another patron card to be scanned. Test plan : 1) Set system preference CircAutoPrintQuickSlip to 'clear the screen' 2) Go to a patron circulation page 3) Post form without a barcode => Check you go to circulation page with no patron selected 4) Set system preference CircAutoPrintQuickSlip to 'open a print quick slip window' 5) Go to a patron circulation page 6) Post form without a barcode => Check you go to print quick slip window with correct datas Created attachment 124884 [details] [review] Bug 29026: Restore behavior when an empty barcode field is submitted in circulation When preference CircAutoPrintQuickSlip is 'clear the screen', when an empty barcode field is submitted in circulation : Before Bug 17845, behavior was to display empty circulation page (no patron selected). Now behavior is displaying the circulation page of current patron. I think previous behavior was used when scanner is used to clear form and wait for another patron card to be scanned. Test plan : 1) Set system preference CircAutoPrintQuickSlip to 'clear the screen' 2) Go to a patron circulation page 3) Post form without a barcode => Check you go to circulation page with no patron selected 4) Set system preference CircAutoPrintQuickSlip to 'open a print quick slip window' 5) Go to a patron circulation page 6) Post form without a barcode => Check you go to print quick slip window with correct datas Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Created attachment 125338 [details] [review] Bug 29026: Restore behavior when an empty barcode field is submitted in circulation When preference CircAutoPrintQuickSlip is 'clear the screen', when an empty barcode field is submitted in circulation : Before Bug 17845, behavior was to display empty circulation page (no patron selected). Now behavior is displaying the circulation page of current patron. I think previous behavior was used when scanner is used to clear form and wait for another patron card to be scanned. Test plan : 1) Set system preference CircAutoPrintQuickSlip to 'clear the screen' 2) Go to a patron circulation page 3) Post form without a barcode => Check you go to circulation page with no patron selected 4) Set system preference CircAutoPrintQuickSlip to 'open a print quick slip window' 5) Go to a patron circulation page 6) Post form without a barcode => Check you go to print quick slip window with correct datas Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Good catch! Pushed to master for 21.11, thanks to everybody involved! Pushed to 21.05.x for 21.05.05 Pushed to 20.11.x for 20.11.11 Not backported to oldoldstable (20.05.x). Feel free to ask if it's needed. |