After one deletes an order from the order receive page the script tries to redirect you to basket.pl. However, the basketno parameter is empty.
Created attachment 6125 [details] [review] Proposed fix The order modification script doesn't retrieve an order's basket when doing a delete, so the template must explicitly pass the basket number if we want the redirect to work.
Created attachment 6187 [details] [review] Signed off patch (Proposed fix) The patch fix the problem, so I signed it. But I wonder if this is the right way to fix the problem. Why do we have to pass ordernumber, basketno and biblionumber for cancelling an order? I think the ordernumber is sufficient (it has to be sufficient).
(In reply to comment #2) > Why do we have to > pass ordernumber, basketno and biblionumber for cancelling an order? See addorder.pl. DelOrder() requires biblionumber and ordernumber. basketno is required if you want to correctly redirect the user back to the right place.
Template only change, passes variable explicitly as necessary to get the redirect working. Marking passed QA
patch pushed, please test
The links are redirecting to the right basket page now, but I think it would be better if they redirected back to the order receive page?
Included in 3.6 prior to 3.6.5.