Summary: | Add "do nothing" option to CircAutoPrintQuickSlip system preference | ||
---|---|---|---|
Product: | Koha | Reporter: | David Gustafsson <glasklas> |
Component: | Circulation | Assignee: | David Gustafsson <glasklas> |
Status: | RESOLVED FIXED | QA Contact: | Kyle M Hall (khall) <kyle> |
Severity: | minor | ||
Priority: | P5 - low | CC: | caroline.cyr-la-rose, fridolin.somers, gmcharlt, kyle.m.hall, kyle, lucas |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Caroline Cyr La Rose | Documentation submission: | https://gitlab.com/koha-community/koha-manual/-/merge_requests/971 |
Text to go in the release notes: | Version(s) released in: |
24.05.00,23.11.06
|
|
Circulation function: | |||
Attachments: |
Bug 35149: Add Option to ignore submit of checkout field when empty
Bug 35149: Change CircAutoPrintQuickSlip syspref explanation Bug 35149: Add Option to ignore submit of checkout field when empty Bug 35149: Change CircAutoPrintQuickSlip syspref explanation Bug 35149: Change CircAutoPrintQuickSlip syspref explanation |
Description
David Gustafsson
2023-10-25 13:39:06 UTC
*submitted Created attachment 157800 [details] [review] Bug 35149: Add Option to ignore submit of checkout field when empty 1) Apply patch 2) Set CircAutoPrintQuickSlip to "do nothing" 3) Go to the checkouts form for a specific patron 4) Submit the form with empty value 5) Verify that no action is taken and you stay on the current page Good idea. The CircAutoPrintQuickSlip sys pref is a 'choice' so I think we need a DB update and change to sysprefs.sql to reflect each choice. Created attachment 162323 [details] [review] Bug 35149: Change CircAutoPrintQuickSlip syspref explanation Thanks for the feedback, this should now be fixed. Created attachment 163639 [details] [review] Bug 35149: Add Option to ignore submit of checkout field when empty 1) Apply patch 2) Set CircAutoPrintQuickSlip to "do nothing" 3) Go to the checkouts form for a specific patron 4) Submit the form with empty value 5) Verify that no action is taken and you stay on the current page Signed-off-by: Laura Escamilla <laura.escamilla@bywatersolutions.com> Created attachment 163640 [details] [review] Bug 35149: Change CircAutoPrintQuickSlip syspref explanation Signed-off-by: Laura Escamilla <laura.escamilla@bywatersolutions.com> Created attachment 166582 [details] [review] Bug 35149: Change CircAutoPrintQuickSlip syspref explanation Signed-off-by: Laura Escamilla <laura.escamilla@bywatersolutions.com> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> I remember this option as well, also: didn't we have "close patron account" as an option as well? (or do I just remember that from testing something else?) (In reply to Katrin Fischer from comment #9) > I remember this option as well, also: didn't we have "close patron account" > as an option as well? (or do I just remember that from testing something > else?) Is that different than "Clear the screen"? (In reply to Kyle M Hall from comment #10) > (In reply to Katrin Fischer from comment #9) > > I remember this option as well, also: didn't we have "close patron account" > > as an option as well? (or do I just remember that from testing something > > else?) > > Is that different than "Clear the screen"? No, you are correct. that makes sense! I was mostly justifying switching this to be a bug fix :) Pushed for 24.05! Well done everyone, thank you! Pushed to 23.11.x for 23.11.06 Isn't installer/data/mysql/mandatory/sysprefs.sql missing choices of syspref CircAutoPrintQuickSlip ? (In reply to Fridolin Somers from comment #14) > Isn't installer/data/mysql/mandatory/sysprefs.sql missing choices of syspref > CircAutoPrintQuickSlip ? I stumbled on this before, but it's been missing them before. Since we switched to YAML they are not actually used... so I didn't force the fix here. Ah OK so legacy crap, no problem ;) This seems like an enhancement and I won't backport it for 23.05.x |