[Bug 37797] New: Choosing not to delete a budget does not need to be a form submission
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37797 Bug ID: 37797 Summary: Choosing not to delete a budget does not need to be a form submission Change sponsored?: --- Product: Koha Version: Main Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: koha-bugs@lists.koha-community.org Reporter: emily.lamancusa@montgomerycountymd.gov QA Contact: testopia@bugs.koha-community.org Blocks: 37728 On the confirmation page for deleting a budget, if the user selects "cancel", they are choosing not to perform an update. Currently the button is triggering a form POST with no op, but it really just needs to be a link back to the budget administration page. Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37728 [Bug 37728] More "op" are missing in POSTed forms -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37797 --- Comment #1 from Emily Lamancusa <emily.lamancusa@montgomerycountymd.gov> --- Created attachment 170928 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=170928&action=edit Bug 37797: Change cancel button to a link styled as a button On the confirmation page for deleting a budget, change "Do not delete" from a form submission to a link (styled to look like a button) in order to avoid errors from POSTing with no op. To test: 1. Go to Acquisitions > Budgets 2. Create a test budget 3. Attempt to delete the test budget 4. On the confirmation page, click "No, don't delete" --> An error appears saying that the form was POSTed with no op! 5. Apply patch 6. Repeat steps 3-4 --> You are linked back to the Budgets page. The budget was not deleted. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37797 Emily Lamancusa <emily.lamancusa@montgomerycountymd.gov> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff Patch complexity|--- |Trivial patch -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37797 Phil Ringnalda <phil@chetcolibrary.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |phil@chetcolibrary.org --- Comment #2 from Phil Ringnalda <phil@chetcolibrary.org> --- Interesting bustage, that the before state, which is supposed to be a POST without an op, was actually a POST with the wrong op. We have templates that set $template->param(script_name=> to a string containing their actual path, but this one doesn't, so the template var script_name was apparently getting the path from the page's URL through some magic. Bet that's a whole new class of bug 36192 bustage we haven't yet noticed, where a script that was only getting POSTs now also gets GETs, and doesn't set script_name, and has a template that uses script_name. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37797 --- Comment #3 from Phil Ringnalda <phil@chetcolibrary.org> --- Oh, no magic, script_name isn't set so it's action="", which is the current page with its query string. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37797 Chloe Zermatten <chloe.zermatten@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #170928|0 |1 is obsolete| | --- Comment #4 from Chloe Zermatten <chloe.zermatten@ptfs-europe.com> --- Created attachment 170977 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=170977&action=edit Bug 37797: Change cancel button to a link styled as a button On the confirmation page for deleting a budget, change "Do not delete" from a form submission to a link (styled to look like a button) in order to avoid errors from POSTing with no op. To test: 1. Go to Acquisitions > Budgets 2. Create a test budget 3. Attempt to delete the test budget 4. On the confirmation page, click "No, don't delete" --> An error appears saying that the form was POSTed with no op! 5. Apply patch 6. Repeat steps 3-4 --> You are linked back to the Budgets page. The budget was not deleted. Signed-off-by: Chloe Zermatten <chloe.zermatten@ptfs-europe.com> -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37797 Chloe Zermatten <chloe.zermatten@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off CC| |chloe.zermatten@ptfs-europe | |.com -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37797 --- Comment #5 from Phil Ringnalda <phil@chetcolibrary.org> --- Filed bug 37826 for the unset script_name problem. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37797 Phil Ringnalda <phil@chetcolibrary.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |emily.lamancusa@montgomeryc |ity.org |ountymd.gov -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37797 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|testopia@bugs.koha-communit |martin.renvoize@ptfs-europe |y.org |.com CC| |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=37797 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #170977|0 |1 is obsolete| | --- Comment #6 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 171452 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=171452&action=edit Bug 37797: Change cancel button to a link styled as a button On the confirmation page for deleting a budget, change "Do not delete" from a form submission to a link (styled to look like a button) in order to avoid errors from POSTing with no op. To test: 1. Go to Acquisitions > Budgets 2. Create a test budget 3. Attempt to delete the test budget 4. On the confirmation page, click "No, don't delete" --> An error appears saying that the form was POSTed with no op! 5. Apply patch 6. Repeat steps 3-4 --> You are linked back to the Budgets page. The budget was not deleted. Signed-off-by: Chloe Zermatten <chloe.zermatten@ptfs-europe.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=37797 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37797 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Version(s)| |24.11.00 released in| | Status|Passed QA |Pushed to main -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37797 --- Comment #7 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Pushed for 24.11! Well done everyone, thank you! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37797 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED CC| |lucas@bywatersolutions.com Status|Pushed to main |RESOLVED --- Comment #8 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Doesnt apply to 24.05.x, no backport. -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org