| Summary: | Filters on subscription search are broken | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Baptiste Wojtkowski (bwoj) <baptiste.wojtkowski> |
| Component: | Serials | Assignee: | Jonathan Druart <jonathan.druart> |
| Status: | RESOLVED FIXED | QA Contact: | Baptiste Wojtkowski (bwoj) <baptiste.wojtkowski> |
| Severity: | normal | ||
| Priority: | P5 - low | CC: | david, fridolin.somers, jonathan.druart, paul.derscheid |
| Version: | Main | Keywords: | rel_24_11_candidate |
| Hardware: | All | ||
| OS: | All | ||
| See Also: | https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39775 | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Comma delimited list of Sponsors: | |
| Crowdfunding goal: | 0 | 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,24.11.10
|
| 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 Bug 39814: [24.11] Restore filtering to the serials search table |
||
|
Description
Baptiste Wojtkowski (bwoj)
2025-05-02 11:29:38 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. 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> 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> 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. 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> Pushed for 25.05! Well done everyone, thank you! How to apply on 24.11.x since it uses dataTable() not kohaTable() ? @Jonathan Druart I'm trying to backport this for 24.11, I've seen the required is in Bug 38255 -- koha-tmpl/intranet-tmpl/prog/en/modules/serials/claims.tt but it looks like something is still missing after adding this chunk. Maybe it depends on a change in datatables I missed ? Created attachment 187983 [details] [review] Bug 39814: [24.11] Restore filtering to the serials search table Here is a patch for 24.11, but we won't be able to backport all the changes we pushed to 25.05. That's exactly why all the DT work must have been pushed into the same branch... Backport tested and pushed to 24.11.x for 24.11.10 Bug appeared with 24.11 DT update, setting to needs documenting Bug fix/regression, no changes to the manual required. |