From 764e975c15752003ee886f3c7166a55a3a76dfe9 Mon Sep 17 00:00:00 2001
From: Fridolin Somers <fridolin.somers@biblibre.com>
Date: Mon, 26 Apr 2021 09:40:08 +0200
Subject: [PATCH] Bug 28216: Fix vendor list group by in serials statistics
 wizard

Bug 24940 added sort vendor list by aqbookseller.name but removed accidentally the group by.
Now vendors are repeated in the list.

Test plan :
1) Create several subscriptions for same vendor
2) Go to Reports > Serials
3) Check list of vendors
=> Without patch you see the same vendor several times
=> With patch you seen it once
4) Check sort is still on vendor name

Signed-off-by: Samu Heiskanen <samu.heiskanen@hypernova.fi>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
(cherry picked from commit f69cedea8ad8fbcc99e7b7068bc7d1b2628fadce)

Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com>
---
 reports/serials_stats.pl | 1 +
 1 file changed, 1 insertion(+)

diff --git a/reports/serials_stats.pl b/reports/serials_stats.pl
index 9a0a071ff7..4533b13ece 100755
--- a/reports/serials_stats.pl
+++ b/reports/serials_stats.pl
@@ -137,6 +137,7 @@ if($do_it){
     my $sth = $dbh->prepare("SELECT aqbooksellerid, aqbooksellers.name 
                                 FROM subscription 
                                   LEFT JOIN aqbooksellers ON (subscription.aqbooksellerid=aqbooksellers.id ) 
+                                GROUP BY aqbooksellerid, aqbooksellers.name
                                 ORDER BY aqbooksellers.name ASC
 				");
     $sth->execute();
-- 
2.39.5