From 166e52093c21815087b09f71d7a18071e14e4549 Mon Sep 17 00:00:00 2001 From: David Roberts Date: Fri, 3 Apr 2020 15:23:01 +0000 Subject: [PATCH] Bug 24940: Serials statistics wizard: order vendor list alphabetically 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 --- reports/serials_stats.pl | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/reports/serials_stats.pl b/reports/serials_stats.pl index 0c5ed28331..1438271f47 100755 --- a/reports/serials_stats.pl +++ b/reports/serials_stats.pl @@ -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){ -- 2.11.0