From 38deeeab4f0d02b1f65cfe3deb0ff5d2ec57c8cd Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Fri, 19 Oct 2018 12:18:19 -0300 Subject: [PATCH] Bug 21622: Adjust incorrect GROUP BY clauses in acqui script acqui/ordered.pl: GROUP BY aqorders.ordernumber acqui/spent.pl: GROUP BY aqorders.ordernumbe 'koha_kohadev.aqorders.biblionumber' isn't in GROUP BY Test plan: - Create orders for different basket and using different funds - Receive some of them - Hit the ordered and spent pages (from the acqui home page) => The tables must contain the same data with and without this patch --- acqui/ordered.pl | 8 +++++++- acqui/spent.pl | 14 +++++++++++++- 2 files changed, 20 insertions(+), 2 deletions(-) diff --git a/acqui/ordered.pl b/acqui/ordered.pl index cc3f04918d..a588af6734 100755 --- a/acqui/ordered.pl +++ b/acqui/ordered.pl @@ -74,7 +74,13 @@ WHERE (datecancellationprinted IS NULL OR datecancellationprinted='0000-00-00') AND (quantity > quantityreceived OR quantityreceived IS NULL) - GROUP BY aqorders.ordernumber + GROUP BY aqorders.biblionumber, aqorders.basketno, aqorders.ordernumber, + tleft, + ecost, budgetdate, entrydate, + aqbasket.booksellerid, + aqbooksellers.name, + itype, + title EOQ my $sth = $dbh->prepare($query); diff --git a/acqui/spent.pl b/acqui/spent.pl index 8909eb03e0..9f1138d891 100755 --- a/acqui/spent.pl +++ b/acqui/spent.pl @@ -83,7 +83,19 @@ WHERE (datecancellationprinted IS NULL OR datecancellationprinted='0000-00-00') AND datereceived IS NOT NULL - GROUP BY aqorders.ordernumber + GROUP BY aqorders.biblionumber, aqorders.basketno, aqorders.ordernumber, + tleft, + ecost, budgetdate, entrydate, + aqbasket.booksellerid, + itype, + title, + aqorders.invoiceid, + aqinvoices.invoicenumber, + quantityreceived, + unitprice, + datereceived, + aqbooksellers.name + EOQ my $sth = $dbh->prepare($query); $sth->execute($bookfund); -- 2.11.0