[Koha-bugs] [Bug 7376] Transfer limits should be checked at check-in

[bugzilla-daemon at bugs.koha-community.org]

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

Koha Team Lyon 3 <koha at univ-lyon3.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #76876|0                           |1
        is obsolete|                            |

--- Comment #58 from Koha Team Lyon 3 <koha at univ-lyon3.fr> ---
Created attachment 92740
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=92740&action=edit
Bug 7376 Transfer limits should be checked at check-in

to answer to Victor (comment 56), the response is no, seems that bug 22330
don't encompass this one.
When you return an unauthorized item (according to the branch transfer limits
table), you get only a message "Please return this item to such branch", but
the return is done as well.

So the need to add a checking on branch transfert status is still on the
agenda.
The patch proposed is essentially the initial one from Martin Renvoize with a
little variant to tie the checking with the "anywhere" position of
ReturnToBranch syspref (because a priori, the need is much likely to appear in
this case).

Test Plan :
* ReturnToBranch must be set to "anywhere"
* UseBranchTransferLimits must be set
1) define your branch transfer limit. Refuse transfers from libraryA to
libraryOwner (= refuse return at libraryA of a book located at libraryOwner).
2) checkout a book owned by libraryOwner, at libraryOwner, with a librarian
located at libraryOwner
3) move the librarian to libraryA ("Set Library" link top/right)
4) check-in the book => "you get the message Please return this item to
libraryOwner" 
Note that you get the same message whether the transfert from libraryA to
libraryOwner is authorized or not.
5) Install the patch
6) Redo steps 2 to 4
The message you get now at step 4 is "Cannot check in ... This item must be
checked in at following library: libraryOwner"

Olivier Crouzet

-- 
You are receiving this mail because:
You are watching all bug changes.