User tests: Successful: Unsuccessful:
Pull Request for Issue #15183 .
Taking care of the case where we have no formcontrol but a group.
Showon works as expected in all places
Showon doesn't work in frontend template editing
None
Status | New | ⇒ | Pending |
Category | ⇒ | Libraries |
ok for Protostar: "Google Font for Headings"
but in frontend "Site Settings" --> SEO Settings --> Search Engine Friendly URLs
if yes don't show the other Seo settings as in Global configuration (Use URL Rewriting, Adds Suffix to URL, Unicode Aliases)
@AlexRed the fields you are talking about do not exist in that form so that form is working as intended. there is no showon stuff in that form https://github.com/joomla/joomla-cms/blob/staging/components/com_config/model/form/config.xml So if you feel that those fields should exist in that view it is a different issue to showon that this PR fixes
ok, sorry
I have tested this item
Patch ok for me.
@brianteeman is right here in that this is the main reason it doesn't work
However even if you do add the missing fields, it would still not work,
You would have to change those lines
<?php foreach ($this->form->getFieldset('seo') as $field) : ?>
<div class="control-group">
<div class="control-label"><?php echo $field->label; ?></div>
<div class="controls"><?php echo $field->input; ?></div>
</div>
<?php endforeach; ?>
(https://github.com/joomla/joomla-cms/blob/staging/components/com_config/view/config/tmpl/default_seo.php#L14-L23)
to the single line
<?php echo $this->form->renderFieldset('seo'); ?>
Status | Pending | ⇒ | Ready to Commit |
RTC after two successful tests.
Just on a quick code review and no more (so might be talking rubbish) - is it possible to have a form control and group with the same name, in that case would this start to cause issues?
Just on a quick code review and no more (so might be talking rubbish) - is it possible to have a form control and group with the same name, in that case would this start to cause issues?
The formcontrol usually is just "jform". If we have both the group and form with the same name, it should still work but if we have one form without groups and another one without formcontrol and they have the same names, then it may become an issue.
In theory, it may be possible. But I doubt they would appear on the same page in practice. Also there will be likely other issues as well.
Usually we only have one form on a page so this should not become an issue.
Status | Ready to Commit | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-04-10 15:09:44 |
Closed_By | ⇒ | rdeutz | |
Labels |
Added:
?
|
I have tested this item✅ successfully on a12fd37
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/15202.