https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41405

Nick Clemens (kidclamp) <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           See Also|                            |https://bugs.koha-community
                   |                            |.org/bugzilla3/show_bug.cgi
                   |                            |?id=20338,
                   |                            |https://bugs.koha-community
                   |                            |.org/bugzilla3/show_bug.cgi
                   |                            |?id=38704,
                   |                            |https://bugs.koha-community
                   |                            |.org/bugzilla3/show_bug.cgi
                   |                            |?id=25219
           Assignee|[email protected] |[email protected]
                   |ity.org                     |
             Status|Needs Signoff               |In Discussion

--- Comment #2 from Nick Clemens (kidclamp) <[email protected]> ---
Maybe this is actually invalid?

The code that checks these in Koha::Patron::is_patron_inside_charge_limit uses:

3512     my $no_issues_charge = $patron_category->noissuescharge ||
C4::Context->preference('noissuescharge');
3513     my $no_issues_charge_guarantees =
3514         $patron_category->noissueschargeguarantees ||
C4::Context->preference('NoIssuesChargeGuarantees');
3515     my $no_issues_charge_guarantors_with_guarantees =
$patron_category->noissueschargeguarantorswithguarantees
3516         ||
C4::Context->preference('NoIssuesChargeGuarantorsWithGuarantees');
3517 

So if they are 0 they don't matter?

I don't think it is supposed to work this way though - based on 41404 and the
way NoIssuesCharge seems to work a 'blank' is "There is no limit" and Zero
should be "Any fines should block issues"

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to