For instance, on acqui/acqui-home.pl and admin/aqbudgets.pl, it would be great to see the totals in the footer updated with the filtered rows. This patch will offer the possibility to do that.
Created attachment 31853 [details] [review] Bug 12987: Update table footer with the visible rows This patch is the preparation step for others. On acqui/acqui-home.pl and admin/aqbudgets.pl, it will be possible to see the totals in the footer updated with the filtered rows. Test plan: This cannot be tested alone, you have to apply it with others (see the "Blocked" bug reports).
Created attachment 31854 [details] [review] Bug 12987: Update table footer with the visible rows This patch is the preparation step for others. On acqui/acqui-home.pl and admin/aqbudgets.pl, it will be possible to see the totals in the footer updated with the filtered rows. Test plan: This cannot be tested alone, you have to apply it with others (see the "Blocked" bug reports).
Created attachment 31976 [details] [review] Bug 12987: The new format_price include file should be include on using datatables
Created attachment 32019 [details] [review] Bug 12987: Fix remove all what is not number The regex was wrong, to calculate a total, we should keep number (0-9) and minus sign (-). All others characters should be removed.
Last patch is stupid, it does not take into account decimal prices.
To test this patch (actually bug 12988 and bug 12989), you need to apply bug 12844 (and bug 12979). I forgot this dependency, that's why it does not work. Bug 12844 removes the currency symbol on the aqbudgets page. The problem arises if the currency symbol is displayed. It could (and will) be fixed later. I cannot fix everything without to be sure these patches will be pushed.
In the future we might need to extend the IF to handle other ones, but for now that is no reason not to sign off
Created attachment 32094 [details] [review] Bug 12987: Update table footer with the visible rows This patch is the preparation step for others. On acqui/acqui-home.pl and admin/aqbudgets.pl, it will be possible to see the totals in the footer updated with the filtered rows. Test plan: This cannot be tested alone, you have to apply it with others (see the "Blocked" bug reports). Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Created attachment 32095 [details] [review] Bug 12987: The new format_price include file should be include on using datatables Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
More bread crumbs - This can only be tested with another bug, so apply first bug 12984 and then bug 12989.
Created attachment 32812 [details] [review] [PASSED QA] Bug 12986: Upgrade the DataTables jQuery plugin to the latest version (1.10.2) DataTables 1.10.2 is now available. The footerCallback function does not seem to work correctly with our current version. Test plan: Go on the maximum of pages where DataTables is used and try to catch a bug/regression :) Signed-off-by: Owen Leonard <oleonard@myacpl.org> I tested many many pages with DataTables, taking special care to find pages with different DataTables configurations. I found no errors in the OPAC or staff client. Tested thoroughly in Firefox (latest) on Windows 7. Tested less thoroughly in Chrome (latest) on Windows 7. Tested briefly in Internet Explorer 7 in Vista and Internet Explorer 8 and found no bugs which were related to the DataTables upgrade. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 32813 [details] [review] [PASSED QA] Bug 12987: Update table footer with the visible rows This patch is the preparation step for others. On acqui/acqui-home.pl and admin/aqbudgets.pl, it will be possible to see the totals in the footer updated with the filtered rows. Test plan: This cannot be tested alone, you have to apply it with others (see the "Blocked" bug reports). Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 32814 [details] [review] [PASSED QA] Bug 12987: Update table footer with the visible rows This patch is the preparation step for others. On acqui/acqui-home.pl and admin/aqbudgets.pl, it will be possible to see the totals in the footer updated with the filtered rows. Test plan: This cannot be tested alone, you have to apply it with others (see the "Blocked" bug reports). Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 32815 [details] [review] [PASSED QA] Bug 12987: The new format_price include file should be include on using datatables Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Patches pushed to master. Thanks Jonathan!
*** Bug 24454 has been marked as a duplicate of this bug. ***