https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40076
Katrin Fischer <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Version|unspecified |Main Status|Needs Signoff |Failed QA --- Comment #4 from Katrin Fischer <[email protected]> --- Testing here... 1) QA checks - passing! 2) Code review a) Check for existence of email (question) The script uses an SQL statement that checks, if email, emailpro and B_email are empty to determine that the settings should not be set. I wonder if it would make sense to take the system preferences like EmailFieldPrimary into account, using Koha's method for determining the email address to send a notice to is not empty? b) Missing documentation (blocker) The new option is added, but wasn't added to the help. As the script already runs without any options, the new option is not discoverable apart from reading the code. It might also be worth noting how it works (see a) ). -- 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/
