Summary: | Cannot change my notice language when EnhancedMessagingPreferencesOPAC is off | ||
---|---|---|---|
Product: | Koha | Reporter: | Marcel de Rooy <m.de.rooy> |
Component: | Notices | Assignee: | Marcel de Rooy <m.de.rooy> |
Status: | CLOSED FIXED | QA Contact: | Katrin Fischer <katrin.fischer> |
Severity: | normal | ||
Priority: | P5 - low | CC: | marie-luce.laflamme |
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: |
22.11.00
|
|
Circulation function: | |||
Attachments: |
Bug 31743: Change condition for messaging tab
Bug 31743: Change condition for messaging tab Bug 31743: Change condition for messaging tab |
Description
Marcel de Rooy
2022-10-12 14:56:18 UTC
Created attachment 141768 [details] [review] Bug 31743: Change condition for messaging tab Note for QA: The code for SMSnumber is still made dependent on EnhancedMessagingPreferencesOPAC in order to not disturb current behavior. This fixes Enhanced on, EnhancedOPAC off; the case of Enhanced off is not considered here. Test plan: Enable EnhancedMessaging, disable EnhancedMessagingOPAC. Enable TranslateNotices. Check if you can only change notice language on OPAC. Enable EnhancedMessagingOPAC. Check if you change messaging prefs too. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Suggestion for the test plan 1. Apply the patch 3. Enable EnhancedMessaging, disable EnhancedMessagingOPAC. 4. Enable TranslateNotices. 5. Login on the OPAC and go in "your messaging" 6. Check if you can only change notice language 7. Enable EnhancedMessagingOPAC. 8. Check if you can change messaging prefs too. Created attachment 141849 [details] [review] Bug 31743: Change condition for messaging tab Note for QA: The code for SMSnumber is still made dependent on EnhancedMessagingPreferencesOPAC in order to not disturb current behavior. This fixes Enhanced on, EnhancedOPAC off; the case of Enhanced off is not considered here. Test plan: Enable EnhancedMessaging, disable EnhancedMessagingOPAC. Enable TranslateNotices. Check if you can only change notice language on OPAC. Enable EnhancedMessagingOPAC. Check if you change messaging prefs too. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: marie-luce <marie-luce.laflamme@inlibro.com> (In reply to Marie-Luce Laflamme from comment #2) > Suggestion for the test plan > > 1. Apply the patch > 3. Enable EnhancedMessaging, disable EnhancedMessagingOPAC. > 4. Enable TranslateNotices. > 5. Login on the OPAC and go in "your messaging" > 6. Check if you can only change notice language > 7. Enable EnhancedMessagingOPAC. > 8. Check if you can change messaging prefs too. Thx Marie-Luce for signing off ! Created attachment 143024 [details] [review] Bug 31743: Change condition for messaging tab Note for QA: The code for SMSnumber is still made dependent on EnhancedMessagingPreferencesOPAC in order to not disturb current behavior. This fixes Enhanced on, EnhancedOPAC off; the case of Enhanced off is not considered here. Test plan: Enable EnhancedMessaging, disable EnhancedMessagingOPAC. Enable TranslateNotices. Check if you can only change notice language on OPAC. Enable EnhancedMessagingOPAC. Check if you change messaging prefs too. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: marie-luce <marie-luce.laflamme@inlibro.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Pushed to master for 22.11. Nice work everyone, thanks! |