From 9789a43025d139d4d0b872fa0b51c0dc1df0d7dc 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 --- 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