Summary: | Lost item is not anonymized when checked in | ||
---|---|---|---|
Product: | Koha | Reporter: | Joy Nelson <1joynelson> |
Component: | Circulation | Assignee: | Kyle M Hall <kyle.m.hall> |
Status: | CLOSED FIXED | QA Contact: | Ian Walls <koha.sekjal> |
Severity: | normal | ||
Priority: | P5 - low | CC: | chris, gmcharlt, martin.renvoize, paul.poulain |
Version: | 3.6 | ||
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: | ||
Attachments: |
Bug 8005 - Lost item is not anonymized when checked in
Bug 8005 - Lost item is not anonymized when checked in |
Description
Joy Nelson
2012-04-24 22:00:33 UTC
Created attachment 9293 [details] [review] Bug 8005 - Lost item is not anonymized when checked in This bug is caused by the subroutine C4::Circulation::LostItem not passing the privacy parameter when calling C4::Circulation::MarkIssueReturned. This issue is actually anonymized when the item is marked as lost, not when the item is checked in. Note, even if the issue is anonymized, the fine charged for lost items still contains a description of the lost item, which is required for the ability to forgive fees for lost items that are later found. Created attachment 10013 [details] [review] Bug 8005 - Lost item is not anonymized when checked in QA comment: small patch, using a sub (GetMemberDetails) that is an heavy one, but using it quite unfrequentely, so it's OK. Other than this, only 3 lines added, passed QA Side question: in ReturnLostItem, we use the same sub: MarkIssueReturned( $borrowernumber, $itemnum ); shouldn't we also add ->privacy parameter ? This is in 3.8.x |