Koha-community/Koha
13
7
Fork 0
Code Issues Releases Wiki Activity

Bug 20439: Sort SMS providers in alphabetical order by DB field

Browse source 
To test:
1/ Add 'Email' to the 'SMSSendDriver' system preference.
2/ Make sure 'EnhancedMessagingPreferencesOPAC' and 'EnhancedMessagingPreferences' are turned on.
3/ Add some SMS providers (/cgi-bin/koha/admin/sms_providers.pl) with different names.
4/ Notice on memberentry.pl and opac-messaging.pl the SMS providers sort by when they were added, not alphabetically.
5/ Apply patch and restart services.
6/ Look at memberentry.pl and opac-messaging.pl and notice that they SMS providers now sort alphabetically.

Signed-off-by: George Williams <george@nekls.org
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
(cherry picked from commit fd49601079)

Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
This commit is contained in:
Lucas Gass 2022-07-15 16:55:53 +00:00
parent f92f6a315d
commit 35b8967b84
2 changed files with 5 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
members/memberentry.pl
View file

@ -68,7 +68,7 @@ if ( $borrowernumber and not $patron ) {
}
if ( C4::Context->preference('SMSSendDriver') eq 'Email' ) {
my @providers = Koha::SMS::Providers->search->as_list;
my @providers = Koha::SMS::Providers->search( {}, { order_by => 'name' } )->as_list;
$template->param( sms_providers => \@providers );
}

6
opac/opac-messaging.pl
View file

@ -79,8 +79,10 @@ $template->param(
TalkingTechItivaPhone => C4::Context->preference("TalkingTechItivaPhoneNotification") );
if ( C4::Context->preference("SMSSendDriver") eq 'Email' ) {
my $providers = Koha::SMS::Providers->search;
$template->param( sms_providers => $providers, sms_provider_id => $patron->sms_provider_id );
my @providers = Koha::SMS::Providers->search( {}, { order_by => 'name' } )->as_list;
$template->param(
sms_providers => \@providers,
sms_provider_id => $patron->sms_provider_id );
}
my $new_session_id = $query->cookie('CGISESSID');