| Summary: | Syntax error in referrer in parcel.tt | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Baptiste Wojtkowski (bwoj) <baptiste.wojtkowski> |
| Component: | Acquisitions | Assignee: | Baptiste Wojtkowski (bwoj) <baptiste.wojtkowski> |
| Status: | Signed Off --- | QA Contact: | Testopia <testopia> |
| Severity: | normal | ||
| Priority: | P5 - low | CC: | david |
| Version: | Main | ||
| Hardware: | All | ||
| OS: | All | ||
| 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 fixes the URL for the "Cancel order and catalog record" link when receiving an order for an invoice - the referrer section of the URL was missing.
|
Version(s) released in: | |
| Circulation function: | |||
| Attachments: |
Bug 41420: Fix referrer in parcel.tt
Bug 41420: Fix referrer in parcel.tt |
||
|
Description
Baptiste Wojtkowski (bwoj)
2025-12-10 09:48:06 UTC
Created attachment 190390 [details] [review] Bug 41420: Fix referrer in parcel.tt Test plan: Steps to reproduce : 1 - Go to http://localhost:8081/cgi-bin/koha/acqui/neworderempty.pl?booksellerid=1&basketno=1 and fill in a new item. 2 - Visit http://localhost:8081/cgi-bin/koha/acqui/parcels.pl?booksellerid=1, type an invoice name and validate. Inspect both "Cancel order" and "Cancel order and catalog record". Notice the link in "Cancel order is broken" 3 - Apply patch 4 - Repeat 3 and notice the referrer is well formed. For a reason I ignore, it does not seem to have an impact on main, but on the 24.11 I tested, if you click on cancel order, after confirming the deletion and clicking on "ok", you will be asked again to confirm, however the deletion has been performed Created attachment 190487 [details] [review] Bug 41420: Fix referrer in parcel.tt Test plan: Steps to reproduce : 1 - Go to http://localhost:8081/cgi-bin/koha/acqui/neworderempty.pl?booksellerid=1&basketno=1 and fill in a new item. 2 - Visit http://localhost:8081/cgi-bin/koha/acqui/parcels.pl?booksellerid=1, type an invoice name and validate. Inspect both "Cancel order" and "Cancel order and catalog record". Notice the link in "Cancel order is broken" 3 - Apply patch 4 - Repeat 3 and notice the referrer is well formed. For a reason I ignore, it does not seem to have an impact on main, but on the 24.11 I tested, if you click on cancel order, after confirming the deletion and clicking on "ok", you will be asked again to confirm, however the deletion has been performed Signed-off-by: David Nind <david@davidnind.com> Testing notes (using KTD): 1. Comparison of URL for the "Cancel order and catalog record" on the receive orders page for an invoice - note that the referrer URL is missing before the patch: - Before patch: cgi-bin/koha/acqui/cancelorder.pl?ordernumber=3&biblionumber=21649&del_biblio=1&referrer= - After patch: /cgi-bin/koha/acqui/cancelorder.pl?ordernumber=3&biblionumber=21649&del_biblio=1&referrer=/cgi-bin/koha/acqui/parcel.pl?invoiceid=2 2. The links still worked before the patch and the order and catalog record was deleted--I'm not sure what the referrer part of the URL actually does. 3. Changed bug from "enhancement" to "normal". |