http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5765 Summary: patron messaging form template tests for SMS inconsistently Change sponsored?: --- Product: Koha Version: rel_3_4 Platform: All OS/Version: All Status: NEW Severity: trivial Priority: P5 Component: MARC Authority data support AssignedTo: gmcharlt@gmail.com ReportedBy: ian.walls@bywatersolutions.com QAContact: koha-bugs@lists.koha-community.org CC: ian.walls@bywatersolutions.com Estimated Hours: 0.0 For the header row of the table, the test is performed on whether SMSSendDriver is defined, but for each row, the first test is on whether 'transport_sms' is defined for that notice, then whether the driver exists. If an sms transport is defined for a notice (or even if the thinks it is), this will add an extra column of '--' to the table, forcing everything over by one, and the 'do not notify' column off the table. Reversing the order of the <TMPL_IF>'s in the messaging_preference_form.inc should resolve this. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug.