Gernott / mask

TYPO3 Extension Mask

Home Page:https://www.facebook.com/typo3mask

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Streamline InlineHelper->addIrreToData introduced 8.0.4 results in array becoming an integer

janus-reith opened this issue · comments

Some change introduced in 8.0.4 causes the repeating field on Page Templates to not be filled for me anymore

This was probably caused by "[BUGIFX] Streamline InlineHelper->addIrreToData":
c6c80e3

I have a page template "Default":
mask

The backend layout for this still renders fine and I can configure these items on my pages.
Rendering in the frontend however fails.

In versions up to v8.0.3, data.tx_mask_slideritems was an array and I could iterate over the items.
Starting with v8.0.4, data.tx_mask_slideritems became an integer, presumably with the length of array items, 4 in my case.

I just tested with different new values, it seems like for page templates, any kind of field within the repeating field are not actually resolved in the template but the item length is returned.

But since that was introduced in 8.0.3 already and seems like a common usecase, I have the impression that there must be something I'm missing.

@janus-reith Hi, thanks for reporting the issue. You were correct by pinpointing to the commit. It wrongly did an early return, if CType was not set. However, it is only needed for tt_content records.

But since that was introduced in 8.0.3 already and seems like a common usecase, I have the impression that there must be something I'm missing.

I'm too quite surprised no one raised an issue in 1.5 years. This feature is very niche and even less people use inline relations there. Still, there are so many users it must have caused problems before.

Awesome, works like a charm now - Thanks a lot for the quick fix!

I'm too quite surprised no one raised an issue in 1.5 years. This feature is very niche and even less people use inline relations there. Still, there are so many users it must have caused problems before.

A bit puzzling indeed