From 6ee6bea6bfee301146dc2f631480b227d237e43a Mon Sep 17 00:00:00 2001 From: Nick Clemens Date: Fri, 25 Aug 2023 15:51:12 +0000 Subject: [PATCH] Bug 34552: Don't exclude payments with no status Bug 29664 removed payments with a void status, however, the query also removed any payments with no status set Test Plan: 1 - Set up your cash registers and enable POS and add a debit type that is able to be sold 2 - Take 2 payments via POS 3 - Void one payment 4 - Go to Reports->Statistics wizards->Cash register 4) Verify neither shows for the "All payments to the library" and "Payment" transaction type filters 5) Apply this patch 6) Restart all the things! 7) Verify the non-voided fee shows for the "All payments to the library" and "Payment" transaction type filters and the voided fee does not Signed-off-by: Christopher Brannon Signed-off-by: Katrin Fischer Signed-off-by: Tomas Cohen Arazi --- reports/cash_register_stats.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/reports/cash_register_stats.pl b/reports/cash_register_stats.pl index e88a5d7ab5..1dde63b114 100755 --- a/reports/cash_register_stats.pl +++ b/reports/cash_register_stats.pl @@ -83,7 +83,7 @@ if ($do_it) { } if ( $transaction_type eq 'PAYMENT' || $transaction_type eq 'ACT' ) { - $whereTType .= q{ AND status != 'VOID' }; + $whereTType .= q{ AND (status != 'VOID' OR status IS NULL) }; } my $whereBranchCode = q{}; -- 2.39.5