Summary: | Transferred orders show incorrect basket in transferred from/to | ||
---|---|---|---|
Product: | Koha | Reporter: | Colin Campbell <colin.campbell> |
Component: | Acquisitions | Assignee: | Colin Campbell <colin.campbell> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | minor | ||
Priority: | P5 - low | CC: | david.bourgault, jonathan.druart |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | --- | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: | Circulation function: | ||
Bug Depends on: | 15531 | ||
Bug Blocks: | |||
Attachments: |
Patch to read correct basket data
Bug 19340 Read basket details of transfer partner Bug 19340 Read basket details of transfer partner |
Description
Colin Campbell
2017-09-19 11:52:27 UTC
Created attachment 67201 [details] [review] Patch to read correct basket data Simple patch can be easily tested. Transfer an order line from one basket and supplier to another. When displaying the two baskets, the basketname and supplier appearing in the transferred from/to message in the orderline will incorrectly be those of the current basket. Apply the patch and the supplier and basketname should display the correct values Created attachment 67331 [details] [review] Bug 19340 Read basket details of transfer partner Details of the basket an order is tranferred to or from are displayed in the basket display. Unfortunately these details were not being read so the display incorrectly showed the details of the current owning basket. Signed-off-by: David Bourgault <david.bourgault@inlibro.com> Created attachment 67779 [details] [review] Bug 19340 Read basket details of transfer partner Details of the basket an order is tranferred to or from are displayed in the basket display. Unfortunately these details were not being read so the display incorrectly showed the details of the current owning basket. Signed-off-by: David Bourgault <david.bourgault@inlibro.com> Signed-off-by: Nick Clemens <nick@bywatersolutions.com> Looks like it has been caused by commit 9501ac2feffa8aca965fd24156560870df4065c3 Bug 15531: Add support for standing orders Pushed to master for 17.11, thanks to everybody involved! |