Bug 22626

Summary: 'Filter paid transactions' broken on Transactions tab in staff
Product: Koha Reporter: Katrin Fischer <katrin.fischer>
Component: Fines and feesAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: lucas, martin.renvoize, nick, oleonard
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
19.05.00, 18.11.05
Bug Depends on:    
Bug Blocks: 23143    
Attachments: Bug 22626: 'Filter paid transactions' broken on Transactions tab in staff
Bug 22626: 'Filter paid transactions' broken on Transactions tab in staff
Bug 22626: 'Filter paid transactions' broken on Transactions tab in staff

Description Katrin Fischer 2019-04-03 11:53:56 UTC
The filter on the 'Transactions' tab is no longer working. Clicking on it will change the text, but the table will not be fltered.

To test:
- Go to an account with some paid and unpaid fines and fees
- Go to Accounting tab
- Click filter link on top of Transactions table
- Verify nothing happens

I couldn't spot a Javascript error in the console.
Comment 1 Owen Leonard 2019-04-09 16:10:45 UTC
Created attachment 87618 [details] [review]
Bug 22626: 'Filter paid transactions' broken on Transactions tab in staff

This patch changes a parameter in the function which filters the
DataTable of transactions. The addition of more columns to the table
changed the index of the column which is being filtered.

To test, apply the patch and locate a patron who has multiple fines or
charges, some of them paid. Go to Accounting -> Transactions. Test the
"Filter paid transactions" link. Table rows which show "0.00" in the
"Oustanding" column should be hidden.
Comment 2 Liz Rea 2019-04-09 17:25:40 UTC
Created attachment 87655 [details] [review]
Bug 22626: 'Filter paid transactions' broken on Transactions tab in staff

This patch changes a parameter in the function which filters the
DataTable of transactions. The addition of more columns to the table
changed the index of the column which is being filtered.

To test, apply the patch and locate a patron who has multiple fines or
charges, some of them paid. Go to Accounting -> Transactions. Test the
"Filter paid transactions" link. Table rows which show "0.00" in the
"Oustanding" column should be hidden.

Signed-off-by: Liz Rea <wizzyrea@gmail.com>
Comment 3 Katrin Fischer 2019-04-09 18:40:22 UTC
Created attachment 87665 [details] [review]
Bug 22626: 'Filter paid transactions' broken on Transactions tab in staff

This patch changes a parameter in the function which filters the
DataTable of transactions. The addition of more columns to the table
changed the index of the column which is being filtered.

To test, apply the patch and locate a patron who has multiple fines or
charges, some of them paid. Go to Accounting -> Transactions. Test the
"Filter paid transactions" link. Table rows which show "0.00" in the
"Oustanding" column should be hidden.

Signed-off-by: Liz Rea <wizzyrea@gmail.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 4 Nick Clemens (kidclamp) 2019-04-11 13:58:05 UTC
Awesome work all!

Pushed to master for 19.05
Comment 5 Martin Renvoize 2019-04-15 12:25:17 UTC
Pushed to 18.11.x for 18.11.05
Comment 6 Lucas Gass 2019-04-16 21:49:36 UTC
I don't think 18.05.x need this as that functionality works fine. Adding the patch breaks it. Not backporting.