https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38771
David Nind <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Version|unspecified |Main CC| |[email protected] Text to go in the| |This fixes the log viewer release notes| |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. Patch complexity|--- |Trivial patch --- Comment #5 from David Nind <[email protected]> --- 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) -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
