https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12021

Benjamin Rokseth <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|In Discussion               |Signed Off

--- Comment #10 from Benjamin Rokseth <[email protected]> ---
reverting the 'in discussion' status as the discussion died.

The patch is still valid as the initial SIP2 checkin code is flawed. Alert type
CV04 (transfer with no holds) needs to be accompanied with CT destination
branch. This patch fixes this in the cases where permanent location is not the
right return branch (AddReturn returns messages WrongBranch, WrongTransfer or
NeedsTransfer)

If AutomaticItemReturn is disabled, return branch is given as value of
NeedsTransfer, so test plan is still valid

before patch
1) Switch off AutomaticItemReturn
2) Checkin item at a different branch than home or holding branch
3) observe CV04 but no CT field in SIP response

after patch
repeat 1-2
3) observe CT branch according to circulation policies in preferences
4) checkin item at wrong branch, observe again CV04 and CT with correct branch

even if it was originally signed off it might need an extra pair of eyes

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to