I have tested this item ✅ successfully on d6219ac

Test success.

This PR fixed the issue. However, I still have question if anyone knows the answer

Do we really need this block of code https://github.com/joomla/joomla-cms/blob/staging/administrator/components/com_config/view/component/tmpl/default.php#L60-L64 ?

As I understand showon is used for fields only, so I don't know why we need that block of code to check showon for fieldsets?

IIRC you can showon a fields set based on a option too

I have tested this item ✅ successfully on d6219ac

Thanks!

That is not correct.

1. I tried to add showon="sef_advanced:1" to https://github.com/joomla/joomla-cms/blob/staging/administrator/components/com_content/config.xml#L1033 but it doesn't work. Permissions tab is not being shown when I change URL Routing between Modern and Legacy

2. There is also notice if we add showon to fieldset

See this screenshot

So either it is not supported or if it is supported, there would still be notice like in my screenshot

OK. Checked it more and see that showon support fieldset, too. However, there is still bug in your code prevents it to work and causes the notice

You need to change $fieldSet->formControl in this line of code https://github.com/andrepereiradasilva/joomla-cms/blob/d6219ac06c7c6fdf94d8c12bcaff38d6645cb88e/administrator/components/com_config/view/component/tmpl/default.php#L63 to $this->form->getFormControl() and everything will work as expected

I have tested this item ? unsuccessfully on d6219ac

I have to update my test result. It is still not working as expected, see my comment above

As I understand showon is used for fields only, so I don't know why we need that block of code to check showon for fieldsets?

showon is available for fieldsets since 3.6.0 (#9668)

Thanks Thomas. I tested and saw it.

@joomdonation i will check this better when i have some time later. thanks for your tests

@joomdonation can you test now?

I have tested this item ✅ successfully on 46fc70d

I have tested this item ✅ successfully on 46fc70d

Works OK now, both for fields and fieldsets

RTC. Thanks

Merged thanks!