Test plan: 1 - Create two baskets: "Source Basket" and "Dest Basket" 2 - Add one command line to "Source Basket" 3 - Close the basket 4 - Receive shipment 5 - Receive order 6 - Cancel the order receive 7 - Transfer the order to the Dest Basket -> When clicking on chose, you will be asked if you want to send again infos (which were sent to cancel the order receive), and if accepting, you will be printed an error. 8 - Apply patch 9 - Repeat the process with another source Basket and notice the page is simply reloaded without sending form.
Created attachment 184988 [details] [review] Bug 40575: Transferring order will reload the page and its form Test plan: 1 - Create two baskets: "Source Basket" and "Dest Basket" 2 - Add one command line to "Source Basket" 3 - Close the basket 4 - Receive shipment 5 - Receive order 6 - Cancel the order receive 7 - Transfer the order to the Dest Basket -> When clicking on chose, you will be asked if you want to send again infos (which were sent to cancel the order receive), and if accepting, you will be printed an error. 8 - Apply patch 9 - Repeat the process with another source Basket and notice the page is simply reloaded without sending form.
Hello, I tried to test the patch for Bug 40575, but currently all links to vendors "http://master-intranet/cgi-bin/koha/acquisition/vendors/add" return a 404 error. This issue is unrelated to your patch, so it’s not a fault of your work. Once the major problem with the vendor links is resolved, I will be able to properly test the patch. For now, it will require another look at that time. Thank you for your understanding. Best regards :)
I attempted to test. The good news is that the patch still applies! The not so good news, is that I'm not sure exactly what this is fixing or what I am supposed to see, particularly for Step 7 of the test plan. (I have some screenshots, but I don't think they will help.) Can you break down Step 7 some more? Everything is okay up until choosing the destination basket: 1. I get the browser message “Transfer order to this basket?”, with Cancel and OK as options. 2. If I click OK, I end up at the "Receive orders from My Vendor" page, which lists the destination basket. - The order is now shown for this basket. - I don't get any error. 3. If I reset everything, apply the patch, and repeat the steps, I don't see any changes. Not sure what I should be seeing, what the error is, or what the expected behavour is.
Hi, On main there is no breaking error, but I could still experience an error. Assuming you are on ktd. After 6, you should be on On /cgi-bin/koha/acqui/parcel.pl, with one line in Pending orders, and no line in "already received". The line is in "source basket", of basket group "source basket": 7.1 - click "transfer" of the line 7.2 - Click search, choose "My vendor" 7.3 - Click on the "Choose" button in the "dest basket". 7.4 - There is a first alert: "Transfer order to this basket?", click ok 7.5 - There is a second alert, which should not be displayed. Since it depends on browser I don't have the exact wording, but it asks you if you want to send again the information to firefox. If you click "send again", it sends again the cancellation form which was sent to display this page during 6-, so there is an error "Cannot cancel receipt. Possible reasons: The order line you are trying to cancel was created from a partial receipt of another order line which is already received. Try to cancel this one first and retry. The order line you are trying to cancel was created from a partial receipt of another order line which has been deleted. Cancellation is not possible." Note: After 6, you can try and reload and see if you have a notification. If you have some, cancel and proceed to 7.1