Summary: | Paying for a lost item overwrites nonpublic notes | ||
---|---|---|---|
Product: | Koha | Reporter: | D Ruth Holloway <ruth> |
Component: | Circulation | Assignee: | D Ruth Holloway <ruth> |
Status: | CLOSED FIXED | QA Contact: | Jonathan Druart <jonathan.druart> |
Severity: | minor | ||
Priority: | P5 - low | CC: | chris, gmcharlt, jonathan.druart, julian.maurice, kyle.m.hall, paul.poulain |
Version: | 3.8 | ||
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 8569: Paying for lost item overwrites nonpublic notes. |
Description
D Ruth Holloway
2012-08-03 14:46:52 UTC
Created attachment 11333 [details] [review] Proposed patch Created attachment 11335 [details] [review] Bug 8569: Paying for lost item overwrites nonpublic notes. This patch modifies C4::Circulation::ReturnLostItem to append the Paid-for note to existing data, rather than overwriting it. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Looks like the status should be signed off QA comments: Simple patch. We cannot translate the message put into the paidfor field but anyhow there is no easy way to do that. Marked as Passed QA Patch pushed. The use case is not very common, but I think it's more a bugfix than an ENH. chris_c, your final decision for 3.8 Pushed to 3.8.x will be in 3.8.5 |