Bug 39814

Summary: Filters on subscription search are broken
Product: Koha Reporter: Baptiste Wojtkowski (bwoj) <baptiste.wojtkowski>
Component: SerialsAssignee: Jonathan Druart <jonathan.druart>
Status: Pushed to main --- QA Contact: Baptiste Wojtkowski (bwoj) <baptiste.wojtkowski>
Severity: normal    
Priority: P5 - low CC: david, jonathan.druart
Version: Main   
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39775
GIT URL: Change sponsored?: ---
Patch complexity: --- Documentation contact:
Documentation submission: Text to go in the release notes:
This restores the column filters to the top of the table for serials search - they are now located at the top of the table, and actually work. This was a regression.
Version(s) released in:
25.05.00
Circulation function:
Bug Depends on: 36640    
Bug Blocks:    
Attachments: Bug 39814: Restore filtering to the serials search table
Bug 39814: Restore filtering to the serials search table
Bug 39814: Restore filtering to the serials search table
Bug 39814: Restore filtering to the serials search table

Description Baptiste Wojtkowski (bwoj) 2025-05-02 11:29:38 UTC
On following pages:
1/ serials/claims.pl
2/ serials/serials-search.pl
There are filters at the bottom of each array, but they are not related to any action, hence are totally useless.

This was introduced by https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36640 which removed "sTable.fnAddFilters("filter", "200");"


Is the solution to add it to columnsettings ? Is there a policy to add tables to columnsettings ?
Comment 1 Jonathan Druart 2025-05-06 13:51:42 UTC
Created attachment 181973 [details] [review]
Bug 39814: Restore filtering to the serials search table

Similar to bug 39775, but for the serials search.

Test plan:
Create some serials then go to http://localhost:8081/cgi-bin/koha/serials/serials-search.pl
and confirm that the filters now appear at the top of the table (which
is consistent with the other tables) and that they work correctly.
Comment 2 Jonathan Druart 2025-05-06 13:52:29 UTC
serials/claims.pl is fixed on bug 39775.
Comment 3 Roman Dolny 2025-05-06 15:41:25 UTC
Created attachment 181984 [details] [review]
Bug 39814: Restore filtering to the serials search table

Similar to bug 39775, but for the serials search.

Test plan:
Create some serials then go to http://localhost:8081/cgi-bin/koha/serials/serials-search.pl
and confirm that the filters now appear at the top of the table (which
is consistent with the other tables) and that they work correctly.

Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl>
Comment 4 David Nind 2025-05-06 16:10:30 UTC
Created attachment 181989 [details] [review]
Bug 39814: Restore filtering to the serials search table

Similar to bug 39775, but for the serials search.

Test plan:
Create some serials then go to http://localhost:8081/cgi-bin/koha/serials/serials-search.pl
and confirm that the filters now appear at the top of the table (which
is consistent with the other tables) and that they work correctly.

Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl>
Signed-off-by: David Nind <david@davidnind.com>
Comment 5 David Nind 2025-05-06 16:14:11 UTC
Note that the library and location column filters only work with what is displayed, and that the codes for the values (such as CPL) don't work.
Comment 6 Baptiste Wojtkowski (bwoj) 2025-05-07 12:07:57 UTC
Created attachment 182019 [details] [review]
Bug 39814: Restore filtering to the serials search table

Similar to bug 39775, but for the serials search.

Test plan:
Create some serials then go to http://localhost:8081/cgi-bin/koha/serials/serials-search.pl
and confirm that the filters now appear at the top of the table (which
is consistent with the other tables) and that they work correctly.

Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl>
Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Baptiste Wojtkowski <baptiste.wojtkowski@biblibre.com>
Comment 7 Katrin Fischer 2025-05-08 06:01:25 UTC
Pushed for 25.05!

Well done everyone, thank you!