[#10350] - add getFieldXml method + parameter path
- Closed
- 26 Mar 2017
- Medium
- Build: staging
- # 10350
- Diff
- rdeutz:additions_to_jform
- Success continuous-integration/travis-ci/pr The Travis CI build passed Details
User tests: Successful: Unsuccessful:
Summary of Changes
- New JForm Class method getFieldXml as public getter for the protected method findField
- New parameter path for the setField method of JForm, allows to add a field at a specific place in the xml tree.
Can be merged on review
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Labels |
Added:
?
?
|
||
| Labels | |||
| Category | ⇒ | Libraries Unit Tests |
zero-24
- | Category | Libraries Unit Tests | ⇒ | Unit Tests |
| Category | Unit Tests | ⇒ | Libraries Unit Tests |
@infograf768 don't understand your comment
Robert what's the use case for making this method public?
To add a field you need a simplexml object, so if you like to change the ordering of fields within a fiedset you have to get the field information, remove fields, create simplexml objects and add them with getting the simpleXml object directly you can skip the "create simplexml opbjects" part
@rdeutz can you look at the comment from @infograf768 please
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10350.
@brianteeman I fixed it but I think I need to write more instruction because this is a quite complicated thing to understand, is on my list
well, will not test this then :(
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/10350.
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-03-26 11:26:21 |
| Closed_By | ⇒ | rdeutz |
Looks like Travis did not find the tab instead of space for each
@param string $path Tree elements a doc separated list where we add the node.after $path