@@ -, +, @@ --- C4/Form/MessagingPreferences.pm | 4 +- .../opac-tmpl/prog/en/modules/opac-messaging.tt | 49 +++++++++++++++---- 2 files changed, 40 insertions(+), 13 deletions(-) --- a/C4/Form/MessagingPreferences.pm +++ a/C4/Form/MessagingPreferences.pm @@ -140,9 +140,9 @@ sub set_form_values { } ( 0..30 ); # FIXME: 30 is a magic number. } foreach my $transport ( @{$pref->{'transports'}} ) { - $option->{'transport_'.$transport} = 'checked="checked"'; + $option->{'transports_'.$transport} = 1; } - $option->{'digest'} = 'checked="checked"' if $pref->{'wants_digest'}; + $option->{'digest'} = 1 if $pref->{'wants_digest'}; } $template->param(messaging_preferences => $messaging_options); } --- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-messaging.tt +++ a/koha-tmpl/opac-tmpl/prog/en/modules/opac-messaging.tt @@ -59,29 +59,56 @@ - [% END %] [% IF ( messaging_preference.transport_sms ) %] - [% IF ( SMSSendDriver ) %][% END %] + [% IF ( SMSSendDriver ) %] + + [% IF ( messaging_preference.transports_sms ) %] + + [% ELSE %] + + [% END %] + + [% END %] [% ELSE %] - - + - [% END %] [% IF ( messaging_preference.transport_email ) %] - + + [% IF ( messaging_preference.transports_email ) %] + + [% ELSE %] + + [% END %] [% ELSE %] - [% END %] [% IF ( messaging_preference.has_digest ) %] - + [% IF ( messaging_preference.digest ) %] + + name="digest" checked="checked" onclick = "document.opacmessaging.none[% messaging_preference.message_attribute_id %].checked=false;" /> + [% ELSE %] + + [% END %] [% ELSE %] - --