Summary: | Typo 'AuthScuccessLog' system preference | ||
---|---|---|---|
Product: | Koha | Reporter: | Roman Dolny <roman.dolny> |
Component: | Tools | Assignee: | Roman Dolny <roman.dolny> |
Status: | Pushed to stable --- | QA Contact: | Katrin Fischer <katrin.fischer> |
Severity: | trivial | ||
Priority: | P5 - low | CC: | david, wainuiwitikapark |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | Trivial patch | Documentation contact: | |
Documentation submission: | Text to go in the release notes: |
This fixes the log viewer authentication module "Log not enabled" warning message for the log viewer. If either AuthFailureLog or AuthSuccessLog system preferences are set to "Don't log", the "Log not enabled" warning icon is now shown. Previously, if one of the system preferences was set to "Log", no warning icon was shown.
|
|
Version(s) released in: |
25.05.00,24.11.03
|
Circulation function: | |
Attachments: |
Bug 38771: Typo 'AuthScuccessLog' system preference
Bug 38771: Typo 'AuthScuccessLog' system preference Bug 38771: Typo 'AuthScuccessLog' system preference Bug 38771: (follow-up) Fix the authentication log not enabled message Bug 38771: Add "Auth" prefix to system preference search query |
Description
Roman Dolny
2024-12-22 17:48:31 UTC
Created attachment 175899 [details] [review] Bug 38771: Typo 'AuthScuccessLog' system preference Typo in system preference 'AuthScuccessLog' -> 'AuthSuccessLog'. Created attachment 175900 [details] [review] Bug 38771: Typo 'AuthScuccessLog' system preference Typo in system preference 'AuthScuccessLog' -> 'AuthSuccessLog'. Signed-off-by: David Nind <david@davidnind.com> Created attachment 175901 [details] [review] Bug 38771: Typo 'AuthScuccessLog' system preference Typo in system preference 'AuthScuccessLog' -> 'AuthSuccessLog'. Signed-off-by: David Nind <david@davidnind.com> Created attachment 175902 [details] [review] Bug 38771: (follow-up) Fix the authentication log not enabled message This corrects the display of the "Log not enabled" message for the authentication module in the log viewer. If either AuthFailureLog or AuthSuccessLog system preferences are set to "Don't log": - the "Log not enabled" warning icon is now shown - clicking the triangle icon now links to the two system preferences Signed-off-by: David Nind <david@davidnind.com> I've signed off, as this does fix a typo in the code. However, I have a question for QA, as I think the existing logic is wrong. I've added a follow-up to correct the logic. Please remove or amend the follow-up patch if you think this is incorrect. Question for QA =============== 1. I think the warning icon with "Log not enabled" should show if either AuthFailureLog or AuthSuccessLog are not enabled. 2. After the patch, the warning icon with "Log not enabled" is only shown if both the authentication log system preferences are not enabled. [1] 3. It is the only module that I can see that has the case with two system preferences, and it uses an ||, but I think it should be an &&. That is, if either of the system preference are not enabled, it should show the warning. [1] Settings and the warning triangle in the log viewer, after the patch ==================================================================== 1. If both AuthFailureLog and AuthSuccessLog enabled: => No warning triangle in the log viewer tool with "Log not enabled" (as expected) 2. If both AuthFailureLog and AuthSuccessLog not enabled: => Warning triangle in the log viewer tool with "Log not enabled" (as expected) 3. If AuthFailureLog enabled and AuthSuccessLog not enabled: => No warning triangle in the log viewer tool with "Log not enabled" (not as expected) 4. If AuthFailureLog not enabled and AuthSuccessLog enabled: => No warning triangle in the log viewer tool with "Log not enabled" (not as expected) I have updated this bit to match the system preferences exactly: data-log="FailureLog|SuccessLog"> To data-log="AuthFailureLog|AuthSuccessLog"> Verified the link still works as expected. Created attachment 175957 [details] [review] Bug 38771: Add "Auth" prefix to system preference search query Pushed for 25.05! Well done everyone, thank you! (In reply to Katrin Fischer from comment #6) > I have updated this bit to match the system preferences exactly: > > data-log="FailureLog|SuccessLog"> > > To > > data-log="AuthFailureLog|AuthSuccessLog"> > > Verified the link still works as expected. Thanks Katrin! Nice work everyone! Pushed to 24.11.x for 24.11.03 Not backporting to 24.05.x unless requested |