Bug 24940: Serials statistics wizard: order vendor list alphabetically
authorDavid Roberts <david@koha-ptfs.co.uk>
Fri, 3 Apr 2020 15:23:01 +0000 (15:23 +0000)
committerMartin Renvoize <martin.renvoize@ptfs-europe.com>
Wed, 8 Apr 2020 10:46:21 +0000 (11:46 +0100)
This patch changes the dropdown from being sorted by aqbookseller.id to
aqbookseller.name

To test:
1) Add at least 2 vendors:
 - First: ZZZZ
 - Second: AAAA
2) Add subscriptions for each of the vendors
3) Check the pull down in the serials statistics wizard and verify it
lists them as ZZZZ, AAAA
4) Apply the patch
5) Re-check the pull down in the wizard and check that the vendors are
now listed AAAA,ZZZZ

Signed-off-by: Devinim <kohadevinim@devinim.com.tr>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

reports/serials_stats.pl

index 0c5ed28..a27c404 100755 (executable)
@@ -138,7 +138,8 @@ if($do_it){
     my $sth = $dbh->prepare("SELECT aqbooksellerid, aqbooksellers.name 
                                 FROM subscription 
                                   LEFT JOIN aqbooksellers ON (subscription.aqbooksellerid=aqbooksellers.id ) 
-                                GROUP BY aqbooksellerid");
+                                ORDER BY aqbooksellers.name ASC
+                               ");
     $sth->execute();
     
     while(my $row = $sth->fetchrow_hashref){