[Bug 34625] New: Search engine configuration tables header problem
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34625 Bug ID: 34625 Summary: Search engine configuration tables header problem Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Templates Assignee: oleonard@myacpl.org Reporter: fridolin.somers@biblibre.com QA Contact: testopia@bugs.koha-community.org Created attachment 154849 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=154849&action=edit Screenshot of problem Search engine configuration, when selecting second tab you seen no header. When going down you see the floating header of first tab o_O -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34625 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on| |33180 --- Comment #1 from Fridolin Somers <fridolin.somers@biblibre.com> --- I bet it is since Bug 33180 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33180 [Bug 33180] Use template wrapper for tabs: Budgets and Search engine configuration -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34625 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34625 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Trivial patch Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34625 --- Comment #2 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 154880 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=154880&action=edit Bug 34625: Fix search engine configuration tables header problem This patch corrects the JavaScript which allows DataTables to be created and destroyed based on which tab is clicked on the search engine configuration page. When you click from one tab to the other, the original tab's DataTable should be destroyed and the new tab's DataTable initialized. To test, apply the patch and go to Administration -> Search engine configuration (Elasticsearch) - The page should load with the "Search fields" tab enabled and the DataTable initialized: Sorting and filtering should work. If you scroll down the page the floating table header should be correct. - Switch to the "Bibliographic records" tab. This table should have filtering enabled, and the floating table header should work when you scroll down the page. - The same should be true for the "Authorities" tab. - Return to the "Search fields" tab to confirm that the DataTable is still working as expected. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34625 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34625 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #154880|0 |1 is obsolete| | --- Comment #3 from Fridolin Somers <fridolin.somers@biblibre.com> --- Created attachment 154894 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=154894&action=edit Bug 34625: Fix search engine configuration tables header problem This patch corrects the JavaScript which allows DataTables to be created and destroyed based on which tab is clicked on the search engine configuration page. When you click from one tab to the other, the original tab's DataTable should be destroyed and the new tab's DataTable initialized. To test, apply the patch and go to Administration -> Search engine configuration (Elasticsearch) - The page should load with the "Search fields" tab enabled and the DataTable initialized: Sorting and filtering should work. If you scroll down the page the floating table header should be correct. - Switch to the "Bibliographic records" tab. This table should have filtering enabled, and the floating table header should work when you scroll down the page. - The same should be true for the "Authorities" tab. - Return to the "Search fields" tab to confirm that the DataTable is still working as expected. Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34625 Victor Grousset/tuxayo <victor@tuxayo.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #154894|0 |1 is obsolete| | --- Comment #4 from Victor Grousset/tuxayo <victor@tuxayo.net> --- Created attachment 155206 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=155206&action=edit Bug 34625: Fix search engine configuration tables header problem This patch corrects the JavaScript which allows DataTables to be created and destroyed based on which tab is clicked on the search engine configuration page. When you click from one tab to the other, the original tab's DataTable should be destroyed and the new tab's DataTable initialized. To test, apply the patch and go to Administration -> Search engine configuration (Elasticsearch) - The page should load with the "Search fields" tab enabled and the DataTable initialized: Sorting and filtering should work. If you scroll down the page the floating table header should be correct. - Switch to the "Bibliographic records" tab. This table should have filtering enabled, and the floating table header should work when you scroll down the page. - The same should be true for the "Authorities" tab. - Return to the "Search fields" tab to confirm that the DataTable is still working as expected. Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com> Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34625 Victor Grousset/tuxayo <victor@tuxayo.net> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |victor@tuxayo.net Status|Signed Off |Passed QA --- Comment #5 from Victor Grousset/tuxayo <victor@tuxayo.net> --- Works, makes sense, QA script happy, code looks good, passing QA :) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34625 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to master Version(s)| |23.11.00 released in| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34625 --- Comment #6 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Pushed to master for 23.11. Nice work everyone, thanks! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34625 Galen Charlton <gmcharlt@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |gmcharlt@gmail.com --- Comment #7 from Galen Charlton <gmcharlt@gmail.com> --- Noting that this patch also fixes the problem with saving some search mappings changes that was reported in bug 34504 for 23.05. As such, a backport would be greatly appreciated! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34625 Galen Charlton <gmcharlt@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |34504 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34504 [Bug 34504] Elasticsearch error when updating mappings -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34625 Galen Charlton <gmcharlt@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugs.koha-community | |.org/bugzilla3/show_bug.cgi | |?id=34695 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34625 Nick Clemens <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |nick@bywatersolutions.com --- Comment #8 from Nick Clemens <nick@bywatersolutions.com> --- *** Bug 34695 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34625 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Version(s)|23.11.00 |23.11.00,23.05.04 released in| | Status|Pushed to master |Pushed to stable --- Comment #9 from Fridolin Somers <fridolin.somers@biblibre.com> --- Pushed to 23.05.x for 23.05.04 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34625 Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to stable |Pushed to oldstable Version(s)|23.11.00,23.05.04 |23.11.00,23.05.04,22.11.10 released in| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34625 --- Comment #10 from Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com> --- Nice work everyone! Pushed to oldstable for 22.11.x -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org