Summary: | Current location not updated correctly on the return of an item on hold | ||
---|---|---|---|
Product: | Koha | Reporter: | Alex Buckley <alexbuckley> |
Component: | Circulation | Assignee: | Hayley Pelham <hayleypelham> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | normal | ||
Priority: | P5 - low | CC: | aleisha, alexbuckley, dcook, gmcharlt, kyle.m.hall, mtompset |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
See Also: |
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15439 https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22097 |
||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: | Bug 20419 - Removed updating of holdingbranch in ModItemTransfer() |
Description
Alex Buckley
2018-03-15 00:39:23 UTC
Created attachment 72899 [details] [review] Bug 20419 - Removed updating of holdingbranch in ModItemTransfer() The call to ModItem to update the holdingbranch has been removed from the ModItemTransfer function in C4/Items.pm this is because the holdingbranch should only be changed when the item is checked in at the destination branch not whilst it is in transit. Test plan: 1. Ensure the system preference AutomaticItemReturn is set to 'Dont' 2. Place a reserve on an item to be picked up at branch b 3. Check out the item to a different patron 4. Check the item back in to branch a 5. Select 'Confirm hold and transfer' button in the modal box that appears 6. Notice that the current location of the item visible on the item tab of the detail.pl page of the record is set as branch b even though the item is in transit 7. Apply patch 8. Restart apache2, memcached and plack (if it is enabled) 9. Repeat steps 1, 2, 3, 4, 5 and notice that the current location is set to branch a 10. Check the item in at branch b and notice that the current location of the item is now branch b Sponsored-By: Catalyst IT Please include unit tests when modifying C4 or Koha libraries. This prevents regression. It would be nice if they fail before the fix, and then pass after the fix. I believe this has been fixed in master. Can someone else confirm that this is no longer a bug? |