If you check-in an item through the "Check Out" tab of the Patron Record, the return/check-in ignores the "InProcessingToShelvingCart" and "ReturnToShelvingCart" circulation system preferences. Or rather, there just isn't any logic to handle them in the renewscript.pl file. I'm pretty much just bringing the code straight across from returns.pl, although some minor changes need to be made due to context. Both system preferences are important to include, since you can check-out an in processing item to a patron. While a regular check-in will clear the in processing status, a check-in through the Patron module will cause the item to have its status returned to "In Processing" rather than going to "Shelving Cart", if the preference is set, of course. To test the scenario, turn on one of (or both of) these settings and try checking books in through the Circulation module and the Patron module. Before the patch, you will see that only the Circulation module will change the shelving location. After the patch, you should see identical behaviour. Yay consistency!
Created attachment 10883 [details] [review] Bug 8455 - Check ins processed through "Check Out" tab of the Patron Record ignore Circulation System Preferences
Created attachment 10977 [details] [review] Bug 8455 - Check ins processed through "Check Out" tab of the Patron Record ignore Circulation System Preferences If you check-in an item through the "Check Out" tab of the Patron Record, the return/check-in ignores the "InProcessingToShelvingCart" and "ReturnToShelvingCart" circulation system preferences. Or rather, there just isn't any logic to handle them in the renewscript.pl file. I'm pretty much just bringing the code straight across from returns.pl, although some minor changes need to be made due to context. Both system preferences are important to include, since you can check-out an in processing item to a patron. While a regular check-in will clear the in processing status, a check-in through the Patron module will cause the item to have its status returned to "In Processing" rather than going to "Shelving Cart", if the preference is set, of course. To test the scenario, turn on one of (or both of) these settings and try checking books in through the Circulation module and the Patron module. Before the patch, you will see that only the Circulation module will change the shelving location. After the patch, you should see identical behaviour. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> http://bugs.koha-community.org/show_bug.cgi?id=7189
Created attachment 10978 [details] [review] Bug 8455 - Followup - perltidy renewscript.pl
Signed off on primary patch. I've added a followup to perltidy renewscript.pl, as this seems like a good time to do it. I doubt there are any other patches in process that touch this file. David, if you can sign off on the followup, you could then mark the status of this bug as "Signed Off".
Created attachment 10989 [details] [review] [SIGNED-OFF] Bug 8455 - Followup - perltidy renewscript.pl Signed-off-by: David Cook <dcook@prosentient.com.au>
Signed off on follow-up patch. Perltidy looks good and everything seems to work.
QA comment: * fixes a real problem. The best fix would have been to refactor circulation to avoid code duplication. If we had a single entry point for circulation, the behaviour would have been correct. But I won't reject your patch because you haven't changed 100000 lines :-) No other comment, passed QA
Another comment: on the members pages members/moremember.pl?borrowernumber=NNN, you can also check-in. This page calls reserve/renewscript.pl as well, so it fixes the behaviour here as well. ( Note that I haven't tested it works, but at least it does not break anything if you don't have InProcessingToShelvingCart or ReturnToShelvingCart set)
Bug fix patch is in 3.8.x, will be in 3.8.4