Bug 10725

Summary: Add filters on the pending orders table
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: gmcharlt, katrin.fischer
Version: Main   
Hardware: All   
OS: All   
URL: acqui/parcel.pl
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 10724    
Bug Blocks: 10124    
Attachments: Bug 10725: Add filters on the pending orders table
Bug 10725: Add filters on the pending orders table
[PASSED QA] Bug 10725: Add filters on the pending orders table

Description Jonathan Druart 2013-08-14 08:56:21 UTC
Datatable allows to easily adds filters on the table headers. It would be great to have them on the pending orders table.
Comment 1 Jonathan Druart 2013-08-14 09:01:20 UTC Comment hidden (obsolete)
Comment 2 Katrin Fischer 2013-09-22 12:24:10 UTC
Hi Jonathan,

could you rebase? I applied it on top of 10723+24 and there was a small conflict. I tried fixing it, but it doesn't look right - the table headers are doubled up now for me. Could you take a look and retest? Thx!
Comment 3 Jonathan Druart 2013-09-23 09:22:47 UTC Comment hidden (obsolete)
Comment 4 Katrin Fischer 2013-09-27 06:32:28 UTC
Created attachment 21507 [details] [review]
[PASSED QA] Bug 10725: Add filters on the pending orders table

Since the parcel.pl script get *all* pending orders, there is no reason
not to display all of them.

Like that, DataTable manages pagination and sorting correctly (on all
data).
This patch adds filters on the table header (using columnFilter).

Test plan:
Try filters on the left of the screen and filters on the table header.

Signed-off-by: Pierre Angot <tredok.pierre@gmail.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Passes all tests and QA script.
Searching and sorting the table works correctly.
Larger result sets are a performance problem on this page,
I have filed bug 10595 for that.
Comment 5 Galen Charlton 2013-10-30 05:10:49 UTC
Pushed to master.  Thanks, Jonathan!