Summary: | Sip renewal all response returns incorrect info | ||
---|---|---|---|
Product: | Koha | Reporter: | Colin Campbell <colin.campbell> |
Component: | SIP2 | Assignee: | Colin Campbell <colin.campbell> |
Status: | CLOSED FIXED | QA Contact: | Ian Walls <koha.sekjal> |
Severity: | major | ||
Priority: | P5 - low | CC: | chris, paul.poulain |
Version: | 3.6 | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: |
Proposed Patch
Bug 7454 Initialize return arrays in ILS/Transaction/RenewAll |
Description
Colin Campbell
2012-01-20 12:36:11 UTC
Created attachment 7256 [details] [review] Proposed Patch talkingtech and Staffordshire University have tested on their systems and the proposed patch corrects the system behaviour Created attachment 7750 [details] [review] Bug 7454 Initialize return arrays in ILS/Transaction/RenewAll Explicitly clear these arrays and update them directly. In practice item numbers were already in them when called so that more items were being returned than were actually renewed (or failed) Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz> Functionally sound patch, and tested in production QA comment: tiny patch, passed QA This patch will be included in the forthcoming 3.6.5 release (no string changes). |