Bug 12987

Summary: Update table footer with the visible rows
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: AcquisitionsAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: agustinmoyano, chris, katrin.fischer, tomascohen
Version: master   
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24454
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 12844, 12986    
Bug Blocks: 12988, 12989, 23767    
Attachments: Bug 12987: Update table footer with the visible rows
Bug 12987: Update table footer with the visible rows
Bug 12987: The new format_price include file should be include on using datatables
Bug 12987: Fix remove all what is not number
Bug 12987: Update table footer with the visible rows
Bug 12987: The new format_price include file should be include on using datatables
[PASSED QA] Bug 12986: Upgrade the DataTables jQuery plugin to the latest version (1.10.2)
[PASSED QA] Bug 12987: Update table footer with the visible rows
[PASSED QA] Bug 12987: Update table footer with the visible rows
[PASSED QA] Bug 12987: The new format_price include file should be include on using datatables

Description Jonathan Druart 2014-09-24 15:09:38 UTC
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.
Comment 1 Jonathan Druart 2014-09-24 15:11:52 UTC Comment hidden (obsolete)
Comment 2 Jonathan Druart 2014-09-24 15:16:39 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2014-10-02 13:52:34 UTC Comment hidden (obsolete)
Comment 4 Jonathan Druart 2014-10-06 18:19:52 UTC Comment hidden (obsolete)
Comment 5 Jonathan Druart 2014-10-07 18:01:44 UTC
Last patch is stupid, it does not take into account decimal prices.
Comment 6 Jonathan Druart 2014-10-08 20:48:31 UTC
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.
Comment 7 Chris Cormack 2014-10-09 15:29:36 UTC
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
Comment 8 Chris Cormack 2014-10-09 15:30:27 UTC Comment hidden (obsolete)
Comment 9 Chris Cormack 2014-10-09 15:30:49 UTC Comment hidden (obsolete)
Comment 10 Katrin Fischer 2014-10-27 22:39:30 UTC
More bread crumbs - This can only be tested with another bug, so apply first bug 12984 and then bug 12989.
Comment 11 Katrin Fischer 2014-10-27 22:43:10 UTC Comment hidden (obsolete)
Comment 12 Katrin Fischer 2014-10-27 22:43:25 UTC Comment hidden (obsolete)
Comment 13 Katrin Fischer 2014-10-27 22:44:16 UTC
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>
Comment 14 Katrin Fischer 2014-10-27 22:44:43 UTC
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>
Comment 15 Tomás Cohen Arazi 2014-11-11 12:58:36 UTC
Patches pushed to master.

Thanks Jonathan!
Comment 16 Tomás Cohen Arazi 2020-01-31 14:42:28 UTC
*** Bug 24454 has been marked as a duplicate of this bug. ***