[Bug 29933] New: Fix stray usage of jquery.cookie.js plugin
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29933 Bug ID: 29933 Summary: Fix stray usage of jquery.cookie.js plugin Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: ASSIGNED Severity: minor Priority: P5 - low Component: Templates Assignee: oleonard@myacpl.org Reporter: oleonard@myacpl.org QA Contact: testopia@bugs.koha-community.org CC: testopia@bugs.koha-community.org Depends on: 23944 Blocks: 27750 There are a few instances of "$.removeCookie" usage which should be been replaced when Bug 23944 was submitted. Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23944 [Bug 23944] Phase out use of jquery.cookie.js in favor of js.cookie.js https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27750 [Bug 27750] Remove jquery.cookie.js plugin -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29933 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=29933 --- Comment #1 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 129736 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=129736&action=edit Bug 29933: Fix stray usage of jquery.cookie.js plugin This patch removes a few instances where the old cookie plugin was still being used for deleting cookies even though the new plugin is being used to set the cookies. To test, apply the patch and test the following processes in the staff interface: Search to hold: - View a patron record and click the "Search to hold" button. - Perform a catalog search which will return results. - On the search results page there should be a "Place hold for <patron>" link under each holdable title. - Click the "Place hold" button's dropdown arrow and choose "Forget <patron>." The "Place hold for..." links should disappear. Search to hold for a patron club: - If necessary, create a patron club. - Add one or more patrons to the club. - In Tools -> Patron clubs, find the club you added patrons to. - Click the "Actions" button and then "Search to hold." - Perform a catalog search which will return results. - On the search results page there should be a "Place hold for <club>" link under each holdable title. - Click the "Place hold" button's dropdown arrow and choose "Forget <club>." The "Place hold for..." links should disappear. Batch item modification show/hide columns: - Go to Tools -> Batch item modification. - Submit a list of items for modification. - Uncheck some checkboxes to hide columns on the page showing the items you submitted. - In the browser's storage inspector (e.g. https://developer.mozilla.org/en-US/docs/Tools/Storage_Inspector), should see a "showColumns" cookie with a string of numbers and slashes. - Click the "Show all columns" checkbox. - The "showColumns" cookie should disappear. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29933 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=29933 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #129736|0 |1 is obsolete| | --- Comment #2 from David Nind <david@davidnind.com> --- Created attachment 129811 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=129811&action=edit Bug 29933: Fix stray usage of jquery.cookie.js plugin This patch removes a few instances where the old cookie plugin was still being used for deleting cookies even though the new plugin is being used to set the cookies. To test, apply the patch and test the following processes in the staff interface: Search to hold: - View a patron record and click the "Search to hold" button. - Perform a catalog search which will return results. - On the search results page there should be a "Place hold for <patron>" link under each holdable title. - Click the "Place hold" button's dropdown arrow and choose "Forget <patron>." The "Place hold for..." links should disappear. Search to hold for a patron club: - If necessary, create a patron club. - Add one or more patrons to the club. - In Tools -> Patron clubs, find the club you added patrons to. - Click the "Actions" button and then "Search to hold." - Perform a catalog search which will return results. - On the search results page there should be a "Place hold for <club>" link under each holdable title. - Click the "Place hold" button's dropdown arrow and choose "Forget <club>." The "Place hold for..." links should disappear. Batch item modification show/hide columns: - Go to Tools -> Batch item modification. - Submit a list of items for modification. - Uncheck some checkboxes to hide columns on the page showing the items you submitted. - In the browser's storage inspector (e.g. https://developer.mozilla.org/en-US/docs/Tools/Storage_Inspector), should see a "showColumns" cookie with a string of numbers and slashes. - Click the "Show all columns" checkbox. - The "showColumns" cookie should disappear. 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=29933 Jonathan Druart <jonathan.druart+koha@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #129811|0 |1 is obsolete| | --- Comment #3 from Jonathan Druart <jonathan.druart+koha@gmail.com> --- Created attachment 129859 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=129859&action=edit Bug 29933: Fix stray usage of jquery.cookie.js plugin This patch removes a few instances where the old cookie plugin was still being used for deleting cookies even though the new plugin is being used to set the cookies. To test, apply the patch and test the following processes in the staff interface: Search to hold: - View a patron record and click the "Search to hold" button. - Perform a catalog search which will return results. - On the search results page there should be a "Place hold for <patron>" link under each holdable title. - Click the "Place hold" button's dropdown arrow and choose "Forget <patron>." The "Place hold for..." links should disappear. Search to hold for a patron club: - If necessary, create a patron club. - Add one or more patrons to the club. - In Tools -> Patron clubs, find the club you added patrons to. - Click the "Actions" button and then "Search to hold." - Perform a catalog search which will return results. - On the search results page there should be a "Place hold for <club>" link under each holdable title. - Click the "Place hold" button's dropdown arrow and choose "Forget <club>." The "Place hold for..." links should disappear. Batch item modification show/hide columns: - Go to Tools -> Batch item modification. - Submit a list of items for modification. - Uncheck some checkboxes to hide columns on the page showing the items you submitted. - In the browser's storage inspector (e.g. https://developer.mozilla.org/en-US/docs/Tools/Storage_Inspector), should see a "showColumns" cookie with a string of numbers and slashes. - Click the "Show all columns" checkbox. - The "showColumns" cookie should disappear. Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29933 Jonathan Druart <jonathan.druart+koha@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA CC| |jonathan.druart+koha@gmail. | |com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29933 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Version(s)| |22.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=29933 --- Comment #4 from Fridolin Somers <fridolin.somers@biblibre.com> --- Pushed to master for 22.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=29933 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |kyle@bywatersolutions.com Version(s)|22.05.00 |22.05.00,21.11.03 released in| | Status|Pushed to master |Pushed to stable --- Comment #5 from Kyle M Hall <kyle@bywatersolutions.com> --- Pushed to 21.11.x for 21.11.03 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29933 Andrew Fuerste-Henry <andrew@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |andrew@bywatersolutions.com Version(s)|22.05.00,21.11.03 |22.05.00,21.11.03,21.05.11 released in| | Status|Pushed to stable |Pushed to oldstable --- Comment #6 from Andrew Fuerste-Henry <andrew@bywatersolutions.com> --- Pushed to 21.05.x for 21.05.11 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29933 Victor Grousset/tuxayo <victor@tuxayo.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to oldstable |RESOLVED Resolution|--- |FIXED CC| |victor@tuxayo.net --- Comment #7 from Victor Grousset/tuxayo <victor@tuxayo.net> --- Not backported to oldoldstable (20.11.x). Feel free to ask if it's needed. -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org