https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42388 Bug ID: 42388 Summary: Extract patron charge limit resolution into Koha::Policy::Patrons::ChargeLimits Initiative type: --- Sponsorship --- status: Product: Koha Version: Main Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: koha-bugs@lists.koha-community.org Reporter: tomascohen@gmail.com QA Contact: testopia@bugs.koha-community.org Depends on: 41728 Koha::Patron->is_patron_inside_charge_limits is a policy resolution method that resolves effective charge limits from patron category-level overrides falling back to global system preferences (noissuescharge, NoIssuesChargeGuarantees, NoIssuesChargeGuarantorsWithGuarantees). This is the same pattern as Koha::Policy::Holds->holds_control_library, which resolves the controlling library from ReservesControlBranch, and Koha::Policy::Patrons::Cardnumber->is_valid, which resolves cardnumber constraints from CardnumberLength and BorrowerMandatoryField. This bug proposes: - Add Koha::Policy::Patrons::ChargeLimits with methods to resolve effective charge limits for a patron - Have Koha::Patron->can_checkout and Koha::Patron->can_place_holds consume the new policy class instead of inlining the resolution logic - Add unit tests This is a prerequisite for the checkout availability work. Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41728 [Bug 41728] Add `Koha::Item::Checkin::Availability` to centralize logic -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.