Summary: | Wrong billing/delivery addresses when creating a basket | ||
---|---|---|---|
Product: | Koha | Reporter: | Adrien SAURAT <adrien.saurat> |
Component: | Acquisitions | Assignee: | Adrien SAURAT <adrien.saurat> |
Status: | CLOSED FIXED | QA Contact: | |
Severity: | minor | ||
Priority: | P5 - low | CC: | chris, elliott, mathsabypro |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Small patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: |
proposed patch
[SIGNED OFF] Bug 9364: fixes billing/delivery addresses for new baskets Bug 9364: fixes billing/delivery addresses for new baskets |
Description
Adrien SAURAT
2013-01-08 11:15:50 UTC
Created attachment 14462 [details] [review] proposed patch The call to NewBasket (Acquisition.pm) was wrong. To test, create a new basket, selecting different billing and delivery sites. Test done : 1. Before applying patch : - create a basket, select a billingplace, select a delivery place - deliveryplace is not saved (you can only see "Billing place : ..." in basket summary) 2. Before applying patch : - create a basket, select a billingplace, select a delivery place - deliveryplace is saved (you can see the 2 lines "Delivery place : ..." and "Billing place:..." in basket summary) I sign off. M. Saby Rennes 2 university Created attachment 14468 [details] [review] [SIGNED OFF] Bug 9364: fixes billing/delivery addresses for new baskets Created attachment 14717 [details] [review] Bug 9364: fixes billing/delivery addresses for new baskets Corrects the parameters used for the NewBasket call, when creating a new basket. Signed-off by: Mathieu Saby <mathieu.saby@univ-rennes2.fr> Signed-off-by: Elliott Davis <elliott@bywatersolions.com> I am passing this with a warning. You must go back and resave the locations. After just rereshing I got the same results but upon resubmitting the form I got the result described in the test plan. This patch has been pushed to master. Pushed to 3.10.x will be in 3.10.3 |