[Bug 8082] New: The: IssuingInProcess configuration setting is working in reverse.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8082 Priority: P5 - low Change sponsored?: --- Bug ID: 8082 CC: gmcharlt@gmail.com Assignee: kyle.m.hall@gmail.com Summary: The: IssuingInProcess configuration setting is working in reverse. Severity: major Classification: Unclassified OS: All Reporter: jransom@library.org.nz Hardware: PC Status: NEW Version: rel_3_6 Component: Circulation Product: Koha In the system preferences it is possible to nominate a $ value limit using a couple of confurable settings: maxoutstanding ang noissuescgarge A further system preference (IssuingInProcess) can be set to prevent / not prevent issues in a single session from being blocked if that threshold is reached. For example: Koha Patrons have $5 limit on their account. Patrons often borrow more than that in one transaction/session. So a transaction/session may be carried out to completion even if the amount owed goes over $5. They will however be blocked from any new borrowing until that debt is reduced below $5. If the IssuingInProcess is set to <Prevent> it, illogically, doesn't prevent - and vice versa. It is working backwards. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8082 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |chris@bigballofwax.co.nz --- Comment #1 from Chris Cormack <chris@bigballofwax.co.nz> --- Yep at some point the description was changed It is still right in the database | IssuingInProcess | 1 | NULL | If ON, disables fines if the patron is issuing item that accumulate debt | YesNo | Yes = On ... but now Prevent = On So if switch prevent on, it does the checks about being in process and lets the issue proceed if it is in process. So it is backwards. Easiest fix is to fix the circulation.pref file, so the description matches reality. Patch forthcoming -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8082 --- Comment #2 from Chris Cormack <chris@bigballofwax.co.nz> --- Created attachment 9547 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=9547&action=edit Bug 8082: Fixing circulation.pref to match reality for IssuingInProcess -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8082 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff --- Comment #3 from Chris Cormack <chris@bigballofwax.co.nz> --- .pref file text now matches reality if ( C4::Context->preference("IssuingInProcess") ) { -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8082 Liz Rea <wizzyrea@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #9547|0 |1 is obsolete| | --- Comment #4 from Liz Rea <wizzyrea@gmail.com> --- Created attachment 9866 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=9866&action=edit Bug 8082: Fixing circulation.pref to match reality for IssuingInProcess Signed-off-by: Liz Rea <wizzyrea@gmail.com> passes tests, changes are as advertised. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8082 Liz Rea <wizzyrea@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off CC| |wizzyrea@gmail.com -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8082 Ian Walls <koha.sekjal@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA CC| |koha.sekjal@gmail.com --- Comment #5 from Ian Walls <koha.sekjal@gmail.com> --- Reverses language on the syspref file. Marking Passed QA. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8082 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master CC| |paul.poulain@biblibre.com Version|rel_3_6 |unspecified Assignee|kyle.m.hall@gmail.com |chris@bigballofwax.co.nz -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8082 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Version|unspecified |rel_3_8 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8082 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to Master |Pushed to Stable --- Comment #6 from Chris Cormack <chris@bigballofwax.co.nz> --- Pushed to 3.8.x, will be in 3.8.2 -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org