Summary: | Closing a basket without creating a basketgroup should not redirect to baskets list | ||
---|---|---|---|
Product: | Koha | Reporter: | Mathieu Saby <mathsabypro> |
Component: | Acquisitions | Assignee: | Aleisha Amohia <aleisha> |
Status: | ASSIGNED --- | QA Contact: | |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | aleisha, jonathan.druart, jzairo |
Version: | unspecified | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | Sponsored | Patch complexity: | Trivial patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: |
Bug 9989: Stay on basket page after closing it
Bug 9989: Stay on basket page after closing it Bug 9989: Stay on basket page after closing it |
Description
Mathieu Saby
2013-04-04 11:35:40 UTC
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 |