https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41728 --- Comment #39 from Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> --- (In reply to Martin Renvoize (ashimema) from comment #37)
Created attachment 197359 [details] [review] Bug 41728: (QA follow-up) Collect all blockers instead of short-circuiting
Remove early returns from Koha::Item::Checkin::Availability::check() so that all blockers, confirmations, and warnings are collected in a single pass. Previously, the first blocker triggered an early return, meaning callers could only discover one problem at a time.
I like this but... As part of the endpoint work my plan was to introduce a 'short_circuit' boolean flag to this method. The idea is that the current implementation in C4::Circulation really short-circuits, and we would like to keep the current behavior until the UI is changed. -- You are receiving this mail because: You are watching all bug changes.