Summary: | Cannot override default patron messaging preferences when creating a patron in staff interface | ||
---|---|---|---|
Product: | Koha | Reporter: | Nick Clemens (kidclamp) <nick> |
Component: | Patrons | Assignee: | Nick Clemens (kidclamp) <nick> |
Status: | Pushed to oldstable --- | QA Contact: | Martin Renvoize (ashimema) <martin.renvoize> |
Severity: | normal | ||
Priority: | P5 - low | CC: | andreas.jonsson, andrew, chanel.wheeler, david, fridolin.somers, george, gmcharlt, jonathan.druart, kkrueger, kyle.m.hall, lucas, olivia.sperry |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: |
24.05.00,23.11.06,23.05.12
|
|
Circulation function: | |||
Bug Depends on: | |||
Bug Blocks: | 37339 | ||
Attachments: |
Bug 30318: Don't reset messaging preferences when form is empty
Bug 30318: Don't reset messaging preferences when form is empty Bug 30318: Don't reset messaging preferences when form is empty |
Description
Nick Clemens (kidclamp)
2022-03-18 15:41:14 UTC
Still valid, but you have to remove all settings, if you only remove some, the settings are saved correctly. *** Bug 26520 has been marked as a duplicate of this bug. *** Bug and nuances described in Christopher and George's koha-US The Terrific Every-Other-Thursday Training Video (19 January 2024): https://www.youtube.com/watch?v=vtjWbfQhHCU This is a much needed fix for us. We really need this too. Created attachment 165170 [details] [review] Bug 30318: Don't reset messaging preferences when form is empty In testing, the messaging preferences are set in the form when EnhancedMessagingPreferences is enabled. Switching categories silently updates the preferences unless you have made changes, in which case the user is warned and can confirm if they want to reset or not. The code only applies to patron creatoin, and does not seem to be taken into account during OPAC self-registration. It seems the silent defaulting when saving can be removed in favor of the current updates/warns in the actual form. To test: 1 - Set some messaging preferences for at least two categories 2 - Enter a new patron 3 - Confirm switching categories correctly defaults the messaging preferences 4 - Edit the preferences and switch to another category with defaults 5 - Confirm the warning appears, and preferences are set if confirmed and ignored if not 6 - Set to a category with defaults, uncheck all, and save 7 - Ooof, patron is defaulted 8 - Apply patch 9 - Repeat 1-5 and confirm nothing has changed 10 - Set to a category with defaults, uncheck all, save 11 - Confirm patron is saved with no preferences Created attachment 165217 [details] [review] Bug 30318: Don't reset messaging preferences when form is empty In testing, the messaging preferences are set in the form when EnhancedMessagingPreferences is enabled. Switching categories silently updates the preferences unless you have made changes, in which case the user is warned and can confirm if they want to reset or not. The code only applies to patron creatoin, and does not seem to be taken into account during OPAC self-registration. It seems the silent defaulting when saving can be removed in favor of the current updates/warns in the actual form. To test: 1 - Set some messaging preferences for at least two categories 2 - Enter a new patron 3 - Confirm switching categories correctly defaults the messaging preferences 4 - Edit the preferences and switch to another category with defaults 5 - Confirm the warning appears, and preferences are set if confirmed and ignored if not 6 - Set to a category with defaults, uncheck all, and save 7 - Ooof, patron is defaulted 8 - Apply patch 9 - Repeat 1-5 and confirm nothing has changed 10 - Set to a category with defaults, uncheck all, save 11 - Confirm patron is saved with no preferences Signed-off-by: Andrew Fuerste Henry <andrewfh@dubcolib.org> It looks like Andrew has signed this one off, but the status hasn't been changed. (In reply to David Nind from comment #8) > It looks like Andrew has signed this one off, but the status hasn't been > changed. So I did, thank you. Created attachment 165359 [details] [review] Bug 30318: Don't reset messaging preferences when form is empty In testing, the messaging preferences are set in the form when EnhancedMessagingPreferences is enabled. Switching categories silently updates the preferences unless you have made changes, in which case the user is warned and can confirm if they want to reset or not. The code only applies to patron creatoin, and does not seem to be taken into account during OPAC self-registration. It seems the silent defaulting when saving can be removed in favor of the current updates/warns in the actual form. To test: 1 - Set some messaging preferences for at least two categories 2 - Enter a new patron 3 - Confirm switching categories correctly defaults the messaging preferences 4 - Edit the preferences and switch to another category with defaults 5 - Confirm the warning appears, and preferences are set if confirmed and ignored if not 6 - Set to a category with defaults, uncheck all, and save 7 - Ooof, patron is defaulted 8 - Apply patch 9 - Repeat 1-5 and confirm nothing has changed 10 - Set to a category with defaults, uncheck all, save 11 - Confirm patron is saved with no preferences Signed-off-by: Andrew Fuerste Henry <andrewfh@dubcolib.org> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> The change makes sense, QA scripts are happy. Passing QA Pushed for 24.05! Well done everyone, thank you! Backported to 23.05.x for upcoming 23.05.12 |