Fix separator for comments

Created on 30 March 2023, about 2 years ago

Problem/Motivation

If multiple days with same times are grouped together, a unnecessary separator is printed at the end of the line.

Steps to reproduce

Enable 'Group consecutive days with same hours into one set' at '/admin/structure/types/manage/BUNDLE/display'.

Proposed resolution

If there are more than one slots present, the 'implode' function inserts a separator, even if there is nothing to display as a comment.
Change the code in order to check for any non-empty comments first before imploding the array.

Remaining tasks

Review and test.

User interface changes

none

API changes

none

Data model changes

none

🐛 Bug report
Status

Fixed

Version

1.8

Component

Code - formatter

Created by

🇨🇭Switzerland piridium

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Production build 0.71.5 2024