View | Details | Raw Unified | Return to bug 12100
Collapse All | Expand All

(-)a/C4/Form/MessagingPreferences.pm (-4 / +6 lines)
Lines 25-33 use C4::Context; Link Here
25
use C4::Members::Messaging;
25
use C4::Members::Messaging;
26
use C4::Debug;
26
use C4::Debug;
27
27
28
use constant MAX_DAYS_IN_ADVANCE => 30;
29
28
=head1 NAME
30
=head1 NAME
29
31
30
C4::Form::MessagingPreferences - manage messaging prefernces form
32
C4::Form::MessagingPreferences - manage messaging preferences form
31
33
32
=head1 SYNOPSIS
34
=head1 SYNOPSIS
33
35
Lines 134-143 sub set_form_values { Link Here
134
        if ( $option->{'takes_days'} ) {
136
        if ( $option->{'takes_days'} ) {
135
            my $days_in_advance = $pref->{'days_in_advance'} ? $pref->{'days_in_advance'} : 0;
137
            my $days_in_advance = $pref->{'days_in_advance'} ? $pref->{'days_in_advance'} : 0;
136
            $option->{days_in_advance} = $days_in_advance;
138
            $option->{days_in_advance} = $days_in_advance;
137
            @{$option->{'select_days'}} = map {; {
139
            @{$option->{'select_days'}} = map { {
138
                day        => $_,
140
                day        => $_,
139
                selected   => $_ == $days_in_advance ? 'selected="selected"' :'' } 
141
                selected   => $_ == $days_in_advance  }
140
            } ( 0..30 ); # FIXME: 30 is a magic number.
142
            } ( 0..MAX_DAYS_IN_ADVANCE );
141
        }
143
        }
142
        foreach my $transport ( keys %{$pref->{'transports'}} ) {
144
        foreach my $transport ( keys %{$pref->{'transports'}} ) {
143
            $option->{'transports_'.$transport} = 1;
145
            $option->{'transports_'.$transport} = 1;
(-)a/C4/Members/Messaging.pm (-2 / +1 lines)
Lines 74-80 LEFT JOIN borrower_message_transport_preferences Link Here
74
ON     borrower_message_transport_preferences.borrower_message_preference_id = borrower_message_preferences.borrower_message_preference_id
74
ON     borrower_message_transport_preferences.borrower_message_preference_id = borrower_message_preferences.borrower_message_preference_id
75
LEFT JOIN message_attributes
75
LEFT JOIN message_attributes
76
ON     message_attributes.message_attribute_id = borrower_message_preferences.message_attribute_id
76
ON     message_attributes.message_attribute_id = borrower_message_preferences.message_attribute_id
77
JOIN message_transports
77
LEFT JOIN message_transports
78
ON     message_transports.message_attribute_id = message_attributes.message_attribute_id
78
ON     message_transports.message_attribute_id = message_attributes.message_attribute_id
79
AND    message_transports.message_transport_type = borrower_message_transport_preferences.message_transport_type
79
AND    message_transports.message_transport_type = borrower_message_transport_preferences.message_transport_type
80
WHERE  message_attributes.message_name = ?
80
WHERE  message_attributes.message_name = ?
81
- 

Return to bug 12100