Summary: | deleting an order when receiving redirects to closed basket | ||
---|---|---|---|
Product: | Koha | Reporter: | Nicole C. Engard <nengard> |
Component: | Acquisitions | Assignee: | Jonathan Druart <jonathan.druart> |
Status: | CLOSED FIXED | QA Contact: | Katrin Fischer <katrin.fischer> |
Severity: | major | ||
Priority: | P5 - low | CC: | chris, jonathan.druart, mathsabypro |
Version: | 3.8 | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Trivial patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: |
Bug 9236: FIX delete order when receiving
Bug 9236 : Redirect to the parcel page after deleting an item/order Bug 9236 : Redirect to the parcel page after deleting an item/order [PATCH] Bug 9236: FIX delete order when receiving [PATCH] Bug 9236 : Redirect to the parcel page after deleting an item/order [SIGNED-OFF] Bug 9236: FIX delete order when receiving [SIGNED-OFF] Bug 9236 : Redirect to the parcel page after deleting an item/order |
Description
Nicole C. Engard
2012-12-06 19:54:01 UTC
Created attachment 13995 [details] [review] Bug 9236: FIX delete order when receiving Before this patch the "delete order" links were broken on the receiving page (parcel.pl) I tested the patch : it works, "Delete order" suppress the command line, and "Delete order and catalog record" suppress the command line and the record. But one point can be improved : After deletion, we are redirected to basket page. I think we should stay in receipt page. It would be much more convenient for staff receiving parcels. M. Saby Rennes 2 University Hi Mathieu, Thanks for testing. It is not the goal of this patch. The only ambition of this patch is to fix a bug and not to change the previous behaviour. Hum... You are right, it is the subject of the ticket :) I was about to sign it off... But I realized the problem was described in slighlty different way in this BZ and in Biblibre Mantis. M. Saby Created attachment 14039 [details] [review] Bug 9236 : Redirect to the parcel page after deleting an item/order I applied the 2 patches. It does not work : I am still redirected to the basket and not to the Receipt Summary. M. Saby (In reply to comment #7) > I applied the 2 patches. It does not work : I am still redirected to the > basket and not to the Receipt Summary. > > M. Saby Could you confirm me that you are on the acqui/parcel.pl?invoiceid=XXX, you click on the link "Delete order" or "Delete order and catalog record", you confirm the message and you are redirected on acqui/basket.pl?basketno=YYY ? If yes, could you add a console.log with the string ("addorder.pl?ordernumber="+ordernumber+"&basketno="+basketno+"&quantity=0&biblionumber="+biblionumber+"&invoiceid=[% invoiceid %]") just before the window.location (parcel.tt l.83). Do you test on a sandbox ? I am on my own VM, rebased to master, but maybe did something wrong... I am on the page acqui/parcel.pl?invoiceid=8 "Receipt summary for fournisseur1 [ qq ] on 12/12/2012 "... I have 1 command line (same thing with 2 command lines...): basket 7 order line 10 summary q qty 1 unit cost 18 order cost 18 fund bu-1-2 I click on "delete order" for this command line. ("addorder.pl?ordernumber="+ordernumber+"&basketno="+basketno+"&quantity=0&biblionumber="+biblionumber+"&invoiceid=[% invoiceid %]") = addorder.pl?ordernumber=10&basketno=7&quantity=0&biblionumber=6014&invoiceid=8 And after validation, I arrive on this page : acqui/basket.pl?basketno=7 M. Saby Created attachment 14045 [details] [review] Bug 9236 : Redirect to the parcel page after deleting an item/order Ooops! I forgot to add a file to the patch, sorry! Created attachment 14047 [details] [review] [PATCH] Bug 9236: FIX delete order when receiving Before applying the patch, when recieving a parcel, if you delete an order line, Koha tries to open the page of the basket linked to that order, but can not find it. Koha displays "Vendor not found". With the patch, Koha finds the right basket. I Sign off this patch, to be applied with the other patch ("Redirect to the parcel page after deleting an item/order") M. Saby Rennes 2 University Created attachment 14048 [details] [review] [PATCH] Bug 9236 : Redirect to the parcel page after deleting an item/order Before this patch, when receiving a parcel, after deleting an orderline, Koha tries to open the basket linked to this order. With this patch applied, Koha redirect to the reception page. Works with "Delete order" and "Deleter order and record" I sign off. M. Saby Thanks for this, but the bug I reported was in version 3.8.5 ... and the version has been changed on this. Is this patch just for 3.10? Changing the version back to 3.8. Nicole I am not sure for the version. We have 2 problems in 3.10, resolved by the 2 patches. But maybe in 3.8 we have only the pbm resolved by this one : [PATCH] Bug 9236 : Redirect to the parcel page after deleting an item/order M. Saby Created attachment 14729 [details] [review] [SIGNED-OFF] Bug 9236: FIX delete order when receiving Before this patch the "delete order" links were broken on the receiving page (parcel.pl) Signed-off-by: mathieu saby <mathieu.saby@univ-rennes2.fr> Before applying the patch, when recieving a parcel, if you delete an order line, Koha tries to open the page of the basket linked to that order, but can not find it. Koha displays "Vendor not found". With the patch, Koha finds the right basket. I Sign off this patch, to be applied with the other patch ("Redirect to the parcel page after deleting an item/order") Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Created attachment 14730 [details] [review] [SIGNED-OFF] Bug 9236 : Redirect to the parcel page after deleting an item/order Signed-off-by: mathieu saby <mathieu.saby@univ-rennes2.fr> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> 1) Receive shipment for a previously created basket with multiple order lines 2) Verify 'Delete order' links only have 2 parameters and when you delete an order, you are not redirected to the basket. 3) Verify the same is true for 'Delete order and catalog record'. 4) Apply both patches from Bug 9236. 5) Redo tests and verify page redirects correctly now. Links now also show the basket number as third parameter. Also: make sure orders/items and records are deleted correctly. Passes all tests and QA script. This patch has been pushed to master. Pushed to 3.10.x will be in 3.10.3 |