The SIP config file allows an institution id to be set which is used in some transactions to give the branch id. However this is not always the case because it is not (unlike in an iteractive staff session) passed to the session, therefore some activities (e.g. checkout ) occur. They default to branch of the sip user. This can skew statistics and the location of returned items ( and transits). To behave consistently the session branch should be set in the session from the value in config. Have a patch running in production, will submit after testing in a couple of other config environments
Created attachment 4339 [details] [review] Proposed Patch
Patch sets branch in sip servers session. Otherwise the branch being applied in e.g. checkin was not the same as was being reported to the unit in the status return. In practice this led to incorrect stats and incorrect transit and hold capture behaviour
Colin, What is the best way to test this please? LR
For testing: if the checkin does not send a location (this is the case with 3M units) the rules applied will be those of the sip user not of the machine location in its configuration. If the syspreference AutomaticItemReturn is set to 'Do' the behaviour is driven by a comparison of the item's home location with that of the checkin transaction. Without the fix items from branch A were not being transferred back from a unit in branch B. (Sip user home location being A) Implementing the fix meant the specified behaviour did occur
Colin, many thanks for the test plan. I'll check it when I get in on Monday. LR
Liz, did you have a chance to test this?
Updating Version : This ENH will be for Koha 3.8
Bug versionned for master. entries will be made against rel_3_8 once the patch has been applied (see thread about that on koha-devel yesterday)
Ok thanks.I would have thought we had hundreds of gig available for our test server. Please excuse my ignorance on this matter. How much do you allocate to our test instance?
(In reply to comment #9) > Ok thanks.I would have thought we had hundreds of gig available for our test > server. Please excuse my ignorance on this matter. How much do you allocate to > our test instance? I think you meant to put this in the ticketing system for your support provider.
Created attachment 9002 [details] [review] Bug 6335 Set branch in sip session The server returns its branch as the institution in the config file However in transactions it will default to the homebranch of the sip user which may not be consistent with that. Needs to set branch in the session if there is a value in config otherwise statistics and behaviour can be wrong (esp in discharge) Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
QA Comment: Did not test this patch myself, but base my comments on tests of Chris who signed off, Liz and Colin, as comments above show. Main thing this patch changes, is passing branchcode to Auth.pm via CGI object. Looks good to me. Passed QA
QA comment : this patch also (silently) remove indentation and commented code in api_auth, but it to be more coding guideline-compliant, so I won't complain for that. I also don't think it's an ENH, but a bugfix (please change the severity again if i'm wrong) patch pushed
This will be in 3.8.1