https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42314 --- Comment #10 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 198008 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=198008&action=edit Bug 42314: (follow-up) Simplify list_closed_dates using closed_dates_in_range Related to Bug 42206 (which introduced the list_closed_dates controller) and Bug 42310 (which added the closed_dates_in_range accessor). The list_closed_dates controller previously reached into Koha::Calendar private state ($calendar->_holidays, $calendar->{weekly_closed_days}, $calendar->{day_month_closed_days}) to compute the closed-dates list. This broke encapsulation and would silently break if Koha::Calendar internals changed. Replace that block with a single call to the public Koha::Calendar->closed_dates_in_range accessor, chained off the Koha::Library->calendar accessor added in this bug. Test plan: 1. prove t/db_dependent/api/v1/libraries.t 2. GET /api/v1/libraries/{id}/closed_dates still returns the same list across weekly, repeating, single and exception scenarios -- You are receiving this mail because: You are watching all bug changes.