Joomla! Issue Tracker - CMS

Download
Launch
Feeling Lucky

[#39848] - [4] Subform field uses wrong variables for tooltip with groupByFieldset layout

No Code Attached Yet bug
avatar HLeithner
HLeithner
12 Feb 2023

Steps to reproduce the issue

The layout uses $field to get the id and the description which is not possible because $field is not defined.

joomla-cms/layouts/joomla/form/field/subform/repeatable-table.php

Line 55 in 669e2fc

$table_head .= '<span class="icon-info-circle" aria-hidden="true" tabindex="0"></span><div role="tooltip" id="tip-' . $field->id . '">' . Text::_($field->description) . '</div>';

Additional comments

I don't use this layout and don't know how to use it just saw this while debugging another thing and just wanted to mention it if someone once to fix it.

avatar HLeithner HLeithner - open - 12 Feb 2023
avatar joomla-cms-bot joomla-cms-bot - change - 12 Feb 2023
Labels Added: No Code Attached Yet
avatar joomla-cms-bot joomla-cms-bot - labeled - 12 Feb 2023
avatar Fedik
Fedik - comment - 12 Feb 2023

yeap that is wrong,
was added by @brianteeman in #26614

There is no field, but fieldset is used.

avatar Fedik Fedik - change - 12 Feb 2023
Labels Added: bug
avatar Fedik Fedik - labeled - 12 Feb 2023
avatar brianteeman
brianteeman - comment - 12 Feb 2023

It is used in the core mail domain options

image

avatar Fedik
Fedik - comment - 12 Feb 2023

Brian it does not use groupByFieldset option there ?
The bug affects only cases when groupByFieldset="true" is used in subform field xml.

avatar brianteeman
brianteeman - comment - 12 Feb 2023

I mentioned it for reference only

avatar Fedik Fedik - change - 20 Feb 2023
Status New ⇒ Closed
Closed_Date 0000-00-00 00:00:00 ⇒ 2023-02-20 09:56:11
Closed_By ⇒ Fedik
avatar Fedik Fedik - close - 20 Feb 2023
avatar Fedik
Fedik - comment - 20 Feb 2023

Please test #39894

Add a Comment

Login with GitHub to post a comment