Summary: | Add system preference to disable auto-renew checkbox at checkout | ||
---|---|---|---|
Product: | Koha | Reporter: | Andrew Fuerste-Henry <andrewfh> |
Component: | Circulation | Assignee: | Lucas Gass <lucas> |
Status: | CLOSED FIXED | QA Contact: | Marcel de Rooy <m.de.rooy> |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | gmcharlt, jzairo, kelly, kyle.m.hall, kyle, m.de.rooy |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
See Also: | https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25029 | ||
Change sponsored?: | --- | Patch complexity: | Small patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: |
22.05.00
|
|
Attachments: |
Bug 30226: Add system preference AllowSetAutomaticRenewal
Bug 30226: Add system preference AllowSetAutomaticRenewal Bug 30226: Add system preference AllowSetAutomaticRenewal Bug 30226: Add system preference AllowSetAutomaticRenewal Bug 30226: (QA follow-up) Sync atomicupdate with sysprefs.sql Bug 30226: (follow-up) Remove text from database update |
Description
Andrew Fuerste-Henry
2022-03-04 15:55:37 UTC
Created attachment 131388 [details] [review] Bug 30226: Add system preference AllowSetAutomaticRenewal To test: 1. Apply patch, restart the things, and run updatedatabase 2. Look for the system preference 'AllowSetAutomaticRenewal'. 3. To preserve current behavior the system preference should be set to allow by default. 4. Go to the circulation page and see that under 'Checkout settings' there is the option to set a particular item for auto renewal. 5. Set 'AllowSetAutomaticRenewal' to 'don't allow'. 6. Go back to the circulation page and under checkout settings you should no longer see the option to set an item for auto renewal. Created attachment 131389 [details] [review] Bug 30226: Add system preference AllowSetAutomaticRenewal To test: 1. Apply patch, restart the things, and run updatedatabase 2. Look for the system preference 'AllowSetAutomaticRenewal'. 3. To preserve current behavior the system preference should be set to allow by default. 4. Go to the circulation page and see that under 'Checkout settings' there is the option to set a particular item for auto renewal. 5. Set 'AllowSetAutomaticRenewal' to 'don't allow'. 6. Go back to the circulation page and under checkout settings you should no longer see the option to set an item for auto renewal. Created attachment 131618 [details] [review] Bug 30226: Add system preference AllowSetAutomaticRenewal To test: 1. Apply patch, restart the things, and run updatedatabase 2. Look for the system preference 'AllowSetAutomaticRenewal'. 3. To preserve current behavior the system preference should be set to allow by default. 4. Go to the circulation page and see that under 'Checkout settings' there is the option to set a particular item for auto renewal. 5. Set 'AllowSetAutomaticRenewal' to 'don't allow'. 6. Go back to the circulation page and under checkout settings you should no longer see the option to set an item for auto renewal. Signed-off-by: Kelly mcElligott <kelly@bywatersolutions.com> Bug 30226: Add system preference AllowSetAutomaticRenewal To test: 1. Apply patch, restart the things, and run updatedatabase 2. Look for the system preference 'AllowSetAutomaticRenewal'. 3. To preserve current behavior the system preference should be set to allow by default. 4. Go to the circulation page and see that under 'Checkout settings' there is the option to set a particular item for auto renewal. 5. Set 'AllowSetAutomaticRenewal' to 'don't allow'. 6. Go back to the circulation page and under checkout settings you should no longer see the option to set an item for auto renewal. Signed-off-by: Jessica Zairo <jzairo@bywatersolutions.com> + INSERT IGNORE INTO systempreferences (`variable`, `value`, `options`, `explanation`, `type` ) VALUES('AllowSetAutomaticRenewal', '1', '|yes|no', 'Allow or Prevent staff from flagging items for autorenewal on the checkout page', 'YesNo') | before yes seems unneeded Allow or Prevent should probably be Allow or prevent No blocker + # Finished adding system preference AllowSetAutomaticRenewal + say $out "Added system preference AllowSetAutomaticRenewal"; Not needed. You just repeat yourself here. +('AllowSetAutomaticRenewal','1',NULL,'If ON, allows staff to set items for automatic renewal on the check out page','YesNo'), Other description. Not consistent. (In reply to Marcel de Rooy from comment #5) > | before yes seems unneeded Actually it is a YesNo, so there should be no choices. Created attachment 133353 [details] [review] Bug 30226: Add system preference AllowSetAutomaticRenewal To test: 1. Apply patch, restart the things, and run updatedatabase 2. Look for the system preference 'AllowSetAutomaticRenewal'. 3. To preserve current behavior the system preference should be set to allow by default. 4. Go to the circulation page and see that under 'Checkout settings' there is the option to set a particular item for auto renewal. 5. Set 'AllowSetAutomaticRenewal' to 'don't allow'. 6. Go back to the circulation page and under checkout settings you should no longer see the option to set an item for auto renewal. Signed-off-by: Kelly mcElligott <kelly@bywatersolutions.com> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Created attachment 133354 [details] [review] Bug 30226: (QA follow-up) Sync atomicupdate with sysprefs.sql Description and remove choices. Set items => flag items (as in circulation.pref) Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Created attachment 133507 [details] [review] Bug 30226: (follow-up) Remove text from database update The text displayed by database update is not really helpful. And it will be incorrect if update is run several times. Pushed to master for 22.05, thanks to everybody involved 🦄 |