From ef8f3870b41032e1ae388baaeb8f13565a08dfaf 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 Signed-off-by: Devinim Signed-off-by: Jonathan Druart Signed-off-by: Martin Renvoize Signed-off-by: Joy Nelson --- 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..a27c404292 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.39.5