Extending bug 10993, 11005 to the staff client.
Created attachment 21924 [details] [review] Bug 11020 [ENH] - Transfer limits should be checked when placing a hold/reservation in Staff client
Created attachment 21925 [details] manual test cases
Sponsored by the Joensuu Regional Library
Created attachment 21997 [details] Manual test cases
Created attachment 21998 [details] [review] Bug 11020 - Transfer limits should be checked when placing a hold/reservation in Staff client Extended bug 11005. Extended C4/Reserves.pm to support checking BranchTransferLimits when requested. Extension does in no way disturb existing functionality. Modified the templates to show branch transfer blocks to the user. Modified the business layer to re-check branch transfer limits in case user fools the POST-request. Created manual test cases.
QA comments: none, except the 4 spaces/tabs one. Just in case, you could/should read : http://wiki.koha-community.org/wiki/Coding_Guidelines
Created attachment 22321 [details] [review] Bug 11020 - Transfer limits should be checked when placing a hold/reservation in Staff client Extended bug 11005. Extended C4/Reserves.pm to support checking BranchTransferLimits when requested. Extension does in no way disturb existing functionality. Modified the templates to show branch transfer blocks to the user. Modified the business layer to re-check branch transfer limits in case user fools the POST-request. Created manual test plans.
Fixed tabs to spaces in indentation. Didn't realize my IDE uses tabs instead of spaces until bug 11024/7376.
Created attachment 23165 [details] [review] Bug 11020 - Transfer limits should be checked when placing a hold/reservation in Staff client Extended bug 11005. Extended C4/Reserves.pm to support checking BranchTransferLimits when requested. Extension does in no way disturb existing functionality. Modified the templates to show branch transfer blocks to the user. Modified the business layer to re-check branch transfer limits in case user fools the POST-request. Created manual test plans. Formatted using perltidy and Perl::Critic
Is this still a problem?
Moving to new bug that encompasses multiple old bugs. *** This bug has been marked as a duplicate of bug 22330 ***