From ecda6f730537691054852b16a9d6ecc55f8f2b24 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Fr=C3=A9d=C3=A9rick=20Capovilla?= Date: Tue, 17 May 2011 11:39:11 -0400 Subject: [PATCH] Bug 6355: Correct GetLateOrders so it ignores cancelled orders Signed-off-by: Katrin Fischer Testing plan copied from bug: - Create a new basket for a bookseller - Create two new order in this basket. The orders must have a price or else they won't be shown in the "Late orders" report. - Delete one of the two orders. - Close the basket - Check for late orders and use "0 days ago" as a filter date. - The deleted item appears in the report. Signed-off-by: Chris Cormack --- C4/Acquisition.pm | 1 + 1 file changed, 1 insertion(+) diff --git a/C4/Acquisition.pm b/C4/Acquisition.pm index 45c1948974..cd1e6706f7 100644 --- a/C4/Acquisition.pm +++ b/C4/Acquisition.pm @@ -1435,6 +1435,7 @@ sub GetLateOrders { OR datereceived IS NULL OR aqorders.quantityreceived < aqorders.quantity ) + AND (aqorders.datecancellationprinted IS NULL OR aqorders.datecancellationprinted='0000-00-00') "; my $having = ""; if ($dbdriver eq "mysql") { -- 2.39.5