[Bug 6335] New: Branch not set consistently in all SIP transactions
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6335 Bug #: 6335 Summary: Branch not set consistently in all SIP transactions Classification: Unclassified Change sponsored?: --- Product: Koha Version: master Platform: All OS/Version: All Status: NEW Severity: enhancement Priority: P5 Component: SIP2 AssignedTo: colin.campbell@ptfs-europe.com ReportedBy: colin.campbell@ptfs-europe.com QAContact: koha-bugs@lists.koha-community.org 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 -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6335 Colin Campbell <colin.campbell@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6335 --- Comment #1 from Colin Campbell <colin.campbell@ptfs-europe.com> 2011-06-02 11:32:22 UTC --- Created attachment 4339 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=4339 Proposed Patch -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6335 Colin Campbell <colin.campbell@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Priority|P5 |PATCH-Sent Patch Status|--- |Needs Signoff --- Comment #2 from Colin Campbell <colin.campbell@ptfs-europe.com> 2011-06-02 11:35:22 UTC --- 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 -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6335 Liz Rea <wizzyrea@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |wizzyrea@gmail.com --- Comment #3 from Liz Rea <wizzyrea@gmail.com> 2011-06-10 15:26:49 UTC --- Colin, What is the best way to test this please? LR -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6335 --- Comment #4 from Colin Campbell <colin.campbell@ptfs-europe.com> 2011-06-10 17:26:26 UTC --- 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 -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6335 --- Comment #5 from Liz Rea <wizzyrea@gmail.com> 2011-06-11 02:58:38 UTC --- Colin, many thanks for the test plan. I'll check it when I get in on Monday. LR -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6335 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |chris@bigballofwax.co.nz --- Comment #6 from Chris Cormack <chris@bigballofwax.co.nz> 2011-08-28 23:28:37 UTC --- Liz, did you have a chance to test this? -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6335 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Version|master |Rel_3_8 --- Comment #7 from Paul Poulain <paul.poulain@biblibre.com> 2011-10-24 11:38:23 UTC --- Updating Version : This ENH will be for Koha 3.8 -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6335 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Version|Rel_3_8 |master --- Comment #8 from Paul Poulain <paul.poulain@biblibre.com> 2011-10-25 15:06:01 UTC --- 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) -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6335 Mickey Coalwell <mcoalwell@nekls.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |mcoalwell@nekls.org --- Comment #9 from Mickey Coalwell <mcoalwell@nekls.org> 2012-02-17 22:35:43 UTC --- 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? -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6335 --- Comment #10 from Liz Rea <wizzyrea@gmail.com> 2012-02-17 22:54:29 UTC --- (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. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6335 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #4339|0 |1 is obsolete| | --- Comment #11 from Chris Cormack <chris@bigballofwax.co.nz> --- Created attachment 9002 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=9002&action=edit 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> -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6335 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Priority|PATCH-Sent (DO NOT USE) |P3 Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6335 M. de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |m.de.rooy@rijksmuseum.nl QA Contact|koha-bugs@lists.koha-commun |m.de.rooy@rijksmuseum.nl |ity.org | -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6335 M. de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA --- Comment #12 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- 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 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6335 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master CC| |paul.poulain@biblibre.com Version|master |rel_3_10 Severity|enhancement |normal --- Comment #13 from Paul Poulain <paul.poulain@biblibre.com> --- 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 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6335 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to Master |Pushed to Stable --- Comment #14 from Chris Cormack <chris@bigballofwax.co.nz> --- This will be in 3.8.1 -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org