https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43492 --- Comment #8 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Further expanding this bug's scope after testing surfaced two problems with splitting the migration and CalcDateDue switch into a separate bug (43493): 1. Between this bug landing and 43493 landing, existing library_hours data would vanish from the admin UI (which only reads the new table) while CalcDateDue kept silently reading the old, now-unedited data - a worse intermediate state than either bug alone intended. 2. Editing hours via the new UI would not affect circulation due date calculations until 43493 landed, and the two data sets could drift. Matching how Bug 42310 actually handled this (normalize the schema and migrate its consumer together, in the same bug), this bug now also: - Migrates library_hours data into library_opening_hours (complete open/close pairs only - a row with only one of the two set never had a usable meaning under the old model) and drops library_hours. - Switches C4::Circulation::CalcDateDue onto the new facade, removing Koha::Library::Hour(s) entirely. - Adds a type dropdown to the admin UI (populated from the OPENING_HOURS_TYPE authorised value category), so a slot's purpose can be set without the API. A safety net (Koha::Library::OpeningHours->unlinked_slots) means this simple per-row editor never touches a multi-day closes_opening_id-linked span it has no controls for - those remain API-only. - Adds a minimal typed library_opening_hours embed on /libraries (pulled forward from Bug 43494), since the libraries admin list page's "Library hours" column depends on it and would otherwise be left broken. Bug 43493 is now scoped down to just the Calendar-linkage piece (Koha::Library->opening_hours composing with ->calendar, resolving Bug 36079) on top of an already-complete opening hours system. -- You are receiving this mail because: You are watching all bug changes.