From 1f8a0897a1a82ac7173e2d36788bdbdb7db5be73 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fr=C3=A9d=C3=A9rick=20Capovilla?= Date: Tue, 17 May 2011 11:39:11 -0400 Subject: [PATCH] [SIGNED-OFF] Bug 6355: Correct GetLateOrders so it ignores cancelled orders Content-Type: text/plain; charset="utf-8" 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. --- C4/Acquisition.pm | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/C4/Acquisition.pm b/C4/Acquisition.pm index e431bd2..59493cb 100644 --- a/C4/Acquisition.pm +++ b/C4/Acquisition.pm @@ -1434,6 +1434,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") { -- 1.7.4.1