[Bug 27005] New: Adding a filter in the datatable of opac-readingrecord page
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27005 Bug ID: 27005 Summary: Adding a filter in the datatable of opac-readingrecord page Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: OPAC Assignee: oleonard@myacpl.org Reporter: alexis.ripetti@inLibro.com QA Contact: testopia@bugs.koha-community.org Adding a filter to the datatable in order to facilitate research in the opac-readingrecord page. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27005 --- Comment #1 from Alexis Ripetti <alexis.ripetti@inLibro.com> --- Created attachment 113570 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=113570&action=edit Bug 27005: Adding a filter in the datatable of opac-readingrecord page Test plan: 1) apply patch 2) go to the opac-readingrecord page / your checkout history 3) See that the filter is here and works fine -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27005 Alexis Ripetti <alexis.ripetti@inLibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27005 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |Academy -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27005 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA --- Comment #2 from Owen Leonard <oleonard@myacpl.org> --- This patch adds a DataTable initialization but the old one is still there. The configuration options from the old dataTable() call should be added to the KohaTable() one. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27005 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Small patch Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27005 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #113570|0 |1 is obsolete| | --- Comment #3 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 115683 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115683&action=edit Bug 27005: Adding a filter in the datatable of opac-readingrecord page This patch adds a search field and export options to the patron's checkout history table in the OPAC. This patch also adds some showing/hiding classes to controls which should only be seen by users with or without JavaScript: The <select> for resorting the table is hidden when JS is enabled. The tabs for filtering checkouts from on-site checkouts are hidden when JS is disabled. To test, apply the patch and rebuild the OPAC CSS (https://wiki.koha-community.org/wiki/Working_with_SCSS_in_the_OPAC_and_staff...). - Log in to the OPAC as a user who has both regular and on-site checkouts. - Go to "Your checkout history." - The table of checkouts should have controls at the top: A search field, "Clear filter," "Copy," "CSV," and "Print." - Confirm that the search filter works: Enter some text which will return results among regular or on-site checkouts. - It should work correctly to filter the results. - If you switch tabs, the filter should persist, and correctly filter the results under "Checkouts" and "On-site checkouts." - The "Clear filter" button should work correctly to clear your search. - Test the export and print button to confirm that they work for each table. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27005 David Nind <david@davidnind.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=27005 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #115683|0 |1 is obsolete| | --- Comment #4 from David Nind <david@davidnind.com> --- Created attachment 115737 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115737&action=edit Bug 27005: Adding a filter in the datatable of opac-readingrecord page This patch adds a search field and export options to the patron's checkout history table in the OPAC. This patch also adds some showing/hiding classes to controls which should only be seen by users with or without JavaScript: The <select> for resorting the table is hidden when JS is enabled. The tabs for filtering checkouts from on-site checkouts are hidden when JS is disabled. To test, apply the patch and rebuild the OPAC CSS (https://wiki.koha-community.org/wiki/Working_with_SCSS_in_the_OPAC_and_staff...). - Log in to the OPAC as a user who has both regular and on-site checkouts. - Go to "Your checkout history." - The table of checkouts should have controls at the top: A search field, "Clear filter," "Copy," "CSV," and "Print." - Confirm that the search filter works: Enter some text which will return results among regular or on-site checkouts. - It should work correctly to filter the results. - If you switch tabs, the filter should persist, and correctly filter the results under "Checkouts" and "On-site checkouts." - The "Clear filter" button should work correctly to clear your search. - Test the export and print button to confirm that they work for each table. Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27005 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |david@davidnind.com --- Comment #5 from David Nind <david@davidnind.com> --- I have one minor suggestion, if it is possible. For the export and print lists, in the type column the value is shown as standard_checkout and onsite_checkout. Ideally these should be shown as "normal" human readable information, such as Standard checkout or On-site checkout, instead of the name used by Koha. Testing notes: - To enable the on-site checkout feature, enable the OnSiteCheckouts system preference. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27005 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |ASSIGNED -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27005 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on| |27543 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27543 [Bug 27543] Tooltip on opac-messaging.pl obscured by table headers -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27005 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- 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=27005 --- Comment #6 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 115877 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115877&action=edit Bug 27005: (follow-up) Enable the exclusion of columns from export This patch copies code from the staff interface which allows us to add a class to table columns which should not be included in print or export operations. The hidden column which facilitates the "Checkouts" and "On-site checkouts" tabs can now be hidden in prints and exports. To test, apply the patch and follow the previous test plan. When testing the export and print buttons, confirm that the column with "standard_checkout" data is not included. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27005 --- Comment #7 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 115878 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115878&action=edit Bug 27005: (follow-up) Exclude links column from sort and export I'm adding this as an additional follow-up in case anyone disagrees that the "Links" column (defined in the OPACMySummaryHTML preference) should be excluded from print and export. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27005 David Nind <david@davidnind.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=27005 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #115737|0 |1 is obsolete| | --- Comment #8 from David Nind <david@davidnind.com> --- Created attachment 115928 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115928&action=edit Bug 27005: Adding a filter in the datatable of opac-readingrecord page This patch adds a search field and export options to the patron's checkout history table in the OPAC. This patch also adds some showing/hiding classes to controls which should only be seen by users with or without JavaScript: The <select> for resorting the table is hidden when JS is enabled. The tabs for filtering checkouts from on-site checkouts are hidden when JS is disabled. To test, apply the patch and rebuild the OPAC CSS (https://wiki.koha-community.org/wiki/Working_with_SCSS_in_the_OPAC_and_staff...). - Log in to the OPAC as a user who has both regular and on-site checkouts. - Go to "Your checkout history." - The table of checkouts should have controls at the top: A search field, "Clear filter," "Copy," "CSV," and "Print." - Confirm that the search filter works: Enter some text which will return results among regular or on-site checkouts. - It should work correctly to filter the results. - If you switch tabs, the filter should persist, and correctly filter the results under "Checkouts" and "On-site checkouts." - The "Clear filter" button should work correctly to clear your search. - Test the export and print button to confirm that they work for each table. Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27005 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #115877|0 |1 is obsolete| | --- Comment #9 from David Nind <david@davidnind.com> --- Created attachment 115929 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115929&action=edit Bug 27005: (follow-up) Enable the exclusion of columns from export This patch copies code from the staff interface which allows us to add a class to table columns which should not be included in print or export operations. The hidden column which facilitates the "Checkouts" and "On-site checkouts" tabs can now be hidden in prints and exports. To test, apply the patch and follow the previous test plan. When testing the export and print buttons, confirm that the column with "standard_checkout" data is not included. Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27005 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #115878|0 |1 is obsolete| | --- Comment #10 from David Nind <david@davidnind.com> --- Created attachment 115930 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115930&action=edit Bug 27005: (follow-up) Exclude links column from sort and export I'm adding this as an additional follow-up in case anyone disagrees that the "Links" column (defined in the OPACMySummaryHTML preference) should be excluded from print and export. Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27005 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Text to go in the| |This patch adds a search release notes| |field and export options to | |the OPAC for a patron's | |checkout history table. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27005 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA --- Comment #11 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Hi Owen, I am a little confused by the following 2 warnings, can you please check? FAIL koha-tmpl/opac-tmpl/bootstrap/en/includes/columns_settings.inc OK filters FAIL forbidden patterns forbidden pattern: Font Awesome icons need the aria-hidden attribute for accessibility purpose (see bug 25166) (line 50) forbidden pattern: Font Awesome icons need the aria-hidden attribute for accessibility purpose (see bug 25166) (line 90) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27005 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27005 --- Comment #12 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 116020 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=116020&action=edit Bug 27005: (follow-up) Add missing aria-hidden attributes -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27005 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #115928|0 |1 is obsolete| | --- Comment #13 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 116199 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=116199&action=edit Bug 27005: Adding a filter in the datatable of opac-readingrecord page This patch adds a search field and export options to the patron's checkout history table in the OPAC. This patch also adds some showing/hiding classes to controls which should only be seen by users with or without JavaScript: The <select> for resorting the table is hidden when JS is enabled. The tabs for filtering checkouts from on-site checkouts are hidden when JS is disabled. To test, apply the patch and rebuild the OPAC CSS (https://wiki.koha-community.org/wiki/Working_with_SCSS_in_the_OPAC_and_staff...). - Log in to the OPAC as a user who has both regular and on-site checkouts. - Go to "Your checkout history." - The table of checkouts should have controls at the top: A search field, "Clear filter," "Copy," "CSV," and "Print." - Confirm that the search filter works: Enter some text which will return results among regular or on-site checkouts. - It should work correctly to filter the results. - If you switch tabs, the filter should persist, and correctly filter the results under "Checkouts" and "On-site checkouts." - The "Clear filter" button should work correctly to clear your search. - Test the export and print button to confirm that they work for each table. Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27005 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #115929|0 |1 is obsolete| | --- Comment #14 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 116200 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=116200&action=edit Bug 27005: (follow-up) Enable the exclusion of columns from export This patch copies code from the staff interface which allows us to add a class to table columns which should not be included in print or export operations. The hidden column which facilitates the "Checkouts" and "On-site checkouts" tabs can now be hidden in prints and exports. To test, apply the patch and follow the previous test plan. When testing the export and print buttons, confirm that the column with "standard_checkout" data is not included. Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27005 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #115930|0 |1 is obsolete| | --- Comment #15 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 116201 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=116201&action=edit Bug 27005: (follow-up) Exclude links column from sort and export I'm adding this as an additional follow-up in case anyone disagrees that the "Links" column (defined in the OPACMySummaryHTML preference) should be excluded from print and export. Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27005 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #116020|0 |1 is obsolete| | --- Comment #16 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 116202 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=116202&action=edit Bug 27005: (follow-up) Add missing aria-hidden attributes Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27005 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA CC| |martin.renvoize@ptfs-europe | |.com --- Comment #17 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Nice enhancement, works well. No regressions found and QA scripts are happy. Passing QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27005 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Version(s)| |21.05.00 released in| | Status|Passed QA |Pushed to master -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27005 --- Comment #18 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Pushed to master for 21.05, thanks to everybody involved! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27005 Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |Manual CC| |caroline.cyr-la-rose@inlibr | |o.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27005 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |fridolin.somers@biblibre.co | |m --- Comment #19 from Fridolin Somers <fridolin.somers@biblibre.com> --- Enhancement not pushed to 20.11.x -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org