Remove redundant formatter 'slots' data, using $item object

Created on 2 January 2024, about 1 year ago

Updated 13 March 2024, 10 months ago

Problem/Motivation

In the attempt to make more use of the OfficeHoursItem object, instead of the arrays in the formatter., we need some restructuring.
This way we can remove the redundant 'slots' element.

Proposed resolution

First, restructure ForamtterTrait.php~getRows() function.
Then, remove the 'slots' element.

API changes

$office_hours['slots'] is to be removed.
Use $office_hours['items'] instead.

📌 Task

Status

Fixed

Version

1.16

Component

Code - formatter

Created by

🇳🇱Netherlands johnv

Live updates comments and jobs are added and updated live.

Comments & Activities

Issue created by @johnv
Comment about 1 year ago →
System Message

johnv → committed bd4e7b8c on 8.x-1.x
Issue #3411924: Remove redundant formatter 'slots' data, using object
First commit to issue fork.
Status changed to Fixed 10 months ago4:16pm 28 February 2024
Comment 10 months ago →
🇳🇱Netherlands johnv
This is now done, as part of #3370722-23: Replace today's hours with Exception hours in Week Formatter →
Comment 10 months ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024