# You should have received a copy of the GNU General Public License
# along with Koha; if not, see <http://www.gnu.org/licenses>.
-use strict;
-use warnings;
+use Modern::Perl;
use CGI qw ( -utf8 );
use C4::Output;
use C4::Members;
use C4::Members::Messaging;
-use C4::Branch;
use C4::Form::MessagingPreferences;
+use Koha::Patrons;
+use Koha::SMS::Providers;
my $query = CGI->new();
+unless ( C4::Context->preference('EnhancedMessagingPreferencesOPAC') and
+ C4::Context->preference('EnhancedMessagingPreferences') ) {
+ print $query->redirect("/cgi-bin/koha/errors/404.pl");
+ exit;
+}
+
my ( $template, $borrowernumber, $cookie ) = get_template_and_user(
{
template_name => 'opac-messaging.tt',
}
);
-my $borrower = C4::Members::GetMember( borrowernumber => $borrowernumber );
+my $patron = Koha::Patrons->find( $borrowernumber ); # FIXME and if borrowernumber is invalid?
+
my $messaging_options = C4::Members::Messaging::GetMessagingOptions();
if ( defined $query->param('modify') && $query->param('modify') eq 'yes' ) {
my $sms = $query->param('SMSnumber');
- if ( defined $sms && ( $borrower->{'smsalertnumber'} // '' ) ne $sms ) {
- ModMember(
- borrowernumber => $borrowernumber,
+ my $sms_provider_id = $query->param('sms_provider_id');
+ if ( defined $sms && ( $patron->smsalertnumber // '' ) ne $sms
+ or ( $patron->sms_provider_id // '' ) ne $sms_provider_id ) {
+ $patron->set({
smsalertnumber => $sms,
- sms_provider_id => $query->param('sms_provider_id'),
- );
- $borrower = C4::Members::GetMember( borrowernumber => $borrowernumber );
+ sms_provider_id => $sms_provider_id,
+ })->store;
}
- C4::Form::MessagingPreferences::handle_form_action($query, { borrowernumber => $borrowernumber }, $template);
+ C4::Form::MessagingPreferences::handle_form_action($query, { borrowernumber => $patron->borrowernumber }, $template);
}
-C4::Form::MessagingPreferences::set_form_values({ borrowernumber => $borrower->{'borrowernumber'} }, $template);
+C4::Form::MessagingPreferences::set_form_values({ borrowernumber => $patron->borrowernumber }, $template);
-$template->param( BORROWER_INFO => $borrower,
+$template->param(
messagingview => 1,
- SMSnumber => $borrower->{'smsalertnumber'},
+ SMSnumber => $patron->smsalertnumber, # FIXME This is already sent 2 lines above
SMSSendDriver => C4::Context->preference("SMSSendDriver"),
TalkingTechItivaPhone => C4::Context->preference("TalkingTechItivaPhoneNotification") );
if ( C4::Context->preference("SMSSendDriver") eq 'Email' ) {
- my @providers = Koha::SMS::Provider->all();
- $template->param( sms_providers => \@providers, sms_provider_id => $borrower->{'sms_provider_id'} );
+ my @providers = Koha::SMS::Providers->search();
+ $template->param( sms_providers => \@providers, sms_provider_id => $patron->sms_provider_id );
}
output_html_with_http_headers $query, $cookie, $template->output, undef, { force_no_caching => 1 };