When we close a basket without creating a basketgroup, we are redirected to baskets list for the vendor. I think it would be more logical to stay on the basket page, for example if we want to put it manually into an existing basketgroup. But maybe some libraries would prefer the current behavior? M. Saby Rennes 2 university
Created attachment 98890 [details] [review] Bug 9989: Stay on basket page after closing it Instead of redirecting to the list of vendor's baskets. To test: 1) Go to Acquisitions. Ensure you have a budget, fund and vendor defined. 2) Create a basket. Add an item to it. 3) Close the basket. Do not attach to a basketgroup. 4) Notice you are redirected to the list of vendor's baskets. 5) Apply the patch and go to the basket's page. Reopen the basket. 6) Close the basket. Do not attach to a basketgroup. 7) Confirm you remain on the closed basket's page. Sponsored-by: Catalyst IT
Created attachment 98970 [details] [review] Bug 9989: Stay on basket page after closing it Instead of redirecting to the list of vendor's baskets. To test: 1) Go to Acquisitions. Ensure you have a budget, fund and vendor defined. 2) Create a basket. Add an item to it. 3) Close the basket. Do not attach to a basketgroup. 4) Notice you are redirected to the list of vendor's baskets. 5) Apply the patch and go to the basket's page. Reopen the basket. 6) Close the basket. Do not attach to a basketgroup. 7) Confirm you remain on the closed basket's page. Sponsored-by: Catalyst IT
Created attachment 98980 [details] [review] Bug 9989: Stay on basket page after closing it Instead of redirecting to the list of vendor's baskets. To test: 1) Go to Acquisitions. Ensure you have a budget, fund and vendor defined. 2) Create a basket. Add an item to it. 3) Close the basket. Do not attach to a basketgroup. 4) Notice you are redirected to the list of vendor's baskets. 5) Apply the patch and go to the basket's page. Reopen the basket. 6) Close the basket. Do not attach to a basketgroup. 7) Confirm you remain on the closed basket's page. Sponsored-by: Catalyst IT Signed-off-by: Jessica Zairo <jzairo@bywatersolutions.com>
There is 2 places now where one can close the basket: - From the basket summary page. I totally agree with the new redirect there. - From the vendor search results list - This one I am not sure about. I think the old redirect works better there as it would allow you to quickly close multiple baskets. Jessica, as you signed off, what do you think?
(In reply to Katrin Fischer from comment #4) > There is 2 places now where one can close the basket: > - From the basket summary page. I totally agree with the new redirect there. > - From the vendor search results list - This one I am not sure about. I > think the old redirect works better there as it would allow you to quickly > close multiple baskets. > > Jessica, as you signed off, what do you think? I agree with that. Failing QA.
I wrote a patch addressing the solution in the description of this bug. If we don't want that solution anymore, surely just close the bug report?
Hi Aleiha, when there was only one page where you could close the basket (the basket summary page) this made a lot of sense - and it still does. The problem is that a not so long while ago another page was added where you can close the basket - the 'search result' page when you search for the vendor. So the thing that was strange here, that it would not return to the page where you started from, applies now there. Would it be possible to just return to the page where you came from?
Ah yup understood, thanks Katrin