Bug 19551: Fix wrong order of operations in cash register report
authorNick Clemens <nick@bywatersolutions.com>
Tue, 31 Oct 2017 13:17:26 +0000 (13:17 +0000)
committerJonathan Druart <jonathan.druart@bugs.koha-community.org>
Thu, 7 Dec 2017 13:09:51 +0000 (10:09 -0300)
To test:
1 - Find or create a fine for a patron
2 - Write it off
3 - Home -> Reports -> Cash register
4 - Fill in current date and transaction type 'Write off'
5 - Click Submit
-- see your write off
6 - Fill in previous dates and transaction type 'Write off'
7 - Click Submit
-- see your write off (bad)
8 - Apply patch
9 - Rerun steps 3-7
-- write only visible for current date.

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>

reports/cash_register_stats.pl

index 158625d..0a1ff54 100755 (executable)
@@ -75,7 +75,7 @@ if ($do_it) {
         $whereTType = q{ AND accounttype IN ('Pay','C') };
     } else { #Single transac type
         if ($transaction_type eq 'FORW') {
-            $whereTType = q{ AND accounttype = 'FOR' OR accounttype = 'W' };
+            $whereTType = q{ AND accounttype IN ('FOR','W') };
         } else {
             $whereTType = q{ AND accounttype = ? };
             push @extra_params, $transaction_type;