https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43051 --- Comment #2 from Katariina Pohto <katariina.pohto@ouka.fi> --- We have the same set of rules for checking out and making holds. The fines excluded from noissuecharge are copying fees etc. and they are not supposed to prevent patrons from using any of our services. If this is working as intended, then I guess we are asking for an enhancement then. We alrady have our local "fix" for this. We don't use Koha OPAC as our web library. When people have maxed out their fines included in noissuecharge they won't even see an option to make holds. This was working as intended for us. After someone tried placing a hold but their combined fines went over the limit we've set in noissuescharge and maxoutstading, it did check the other fines too, and returned the error code debt_limit. If I understand this correctly, our first check probably checks noissuescharge, and we wouldn't actually require any kind of limit for maxoutstanding? But I would need to confirm this. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43051 Lari Strand <lmstrand@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |lmstrand@gmail.com --- Comment #4 from Lari Strand <lmstrand@gmail.com> --- The system has two different debt-checking mechanisms: Checkout logic properly filters by restricts_checkouts Hold logic includes all debits This means a patron could be unable to place a hold but able to checkout an item with the same outstanding balance, depending on what type of charges they have. Isn't this at least an inconsistency? Why when placing holds do we not care about your system's/patron's non-issue type charges? Why even make a reserve for a patron who still cannot check the item out when delivered/ready for pickup? In Patron.pm's sub can_place_holds: # debt check unless ( $overrides->{debt_limit} ) { my $max_outstanding = C4::Context->preference("maxoutstanding"); my $outstanding = $self->account->balance; This previous line we changed to: my $outstanding = $self->account->non_issues_charges; if ( $max_outstanding && $outstanding && ( $outstanding > $max_outstanding ) ) { $result->set_value(0); -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.