Skip to content

Use built in JForm method to render fields#5488

Merged
infograf768 merged 1 commit intojoomla:stagingfrom
wilsonge:config_form
Jan 8, 2015
Merged

Use built in JForm method to render fields#5488
infograf768 merged 1 commit intojoomla:stagingfrom
wilsonge:config_form

Conversation

@wilsonge
Copy link
Copy Markdown
Contributor

Summary

Use the method built into JForm since 3.2.3 to render all fields in a fieldset

B/C Implications

None

How to test:

Ensure all fields in the com_config template editing frontend view continue to work as expected

@zero-24
Copy link
Copy Markdown
Contributor

zero-24 commented Dec 22, 2014

@test successfull. Can't see any issues with the patch applyed


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5488.

@pvullioud
Copy link
Copy Markdown

@test successful no problem after applying the patch

@zero-24
Copy link
Copy Markdown
Contributor

zero-24 commented Jan 5, 2015

RTC Thanks @pvullioud and @wilsonge


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5488.

@wilsonge wilsonge added the RTC This Pull Request is Ready To Commit label Jan 7, 2015
infograf768 added a commit that referenced this pull request Jan 8, 2015
Use built in JForm method to render fields
@infograf768 infograf768 merged commit e391923 into joomla:staging Jan 8, 2015
@infograf768
Copy link
Copy Markdown
Member

Merged. Thanks.

@wilsonge wilsonge deleted the config_form branch January 8, 2015 14:19
@rdeutz rdeutz added this to the Joomla! 3.4.0 milestone Feb 3, 2015
@zero-24 zero-24 removed the RTC This Pull Request is Ready To Commit label Oct 14, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants