Bug 29952: Replace obsolete DataTable column filter function
authorOwen Leonard <oleonard@myacpl.org>
Thu, 27 Jan 2022 11:50:28 +0000 (11:50 +0000)
committerFridolin Somers <fridolin.somers@biblibre.com>
Wed, 16 Feb 2022 21:51:11 +0000 (11:51 -1000)
The "Filter paid transactions" link on the Accounting -> Transactions
page is broken because it uses an obsolete DataTables function for
filtering. This patch updates it to use the current syntax, available in
DataTables since version 1.10.

To test, apply the patch and locate a patron in the staff interface who
has multiple fines, some paid.

- View the patron's "Accounting" page and click the "Transactions" tab.
- Click the "Filter paid transactions" link. The table should be
  filtered so that only transactions with an outstanding amount > zero
  are shown.
- The filter link should change to read "Show all transactions."
- Clicking "Show all transactions" should clear the filter.
- Test with one or more columns hidden using the "Columns" control.
  Filtering should still work correctly with columns hidden.

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
koha-tmpl/intranet-tmpl/prog/en/modules/members/boraccount.tt

index 4b0452b..2fd9240 100644 (file)
                     var filteredValue = '^((?!0.00).*)$'; //Filter not matching 0.00 http://stackoverflow.com/a/406408
                     $(this).html('<i class="fa fa-filter"></i> '+txtInactivefilter);
                 }
-                table_account_fines.fnFilter(filteredValue, 10, true, false);
+                table_account_fines.DataTable().columns( 13 ).search( filteredValue, true, false ).draw();
                 $(this).toggleClass('filtered');
             });