User tests: Successful: Unsuccessful:
Pull Request resolves #47451.
Fixes wrong implementation of element removal in Form->load with overwrite=false.
mod_demo47451_1.0.0.zip (System → Install → Extensions).main-top), assigned to all pages.Red FAIL box:
FAIL — bug #47451 present:
titleappears 2× in the form XML (expected 1×) and
the rendered form shows the label “Label from BASE xml (must NOT win)” — the
merge overrode the existing field in the wrong direction.
The dumped form XML contains a second <fieldset name="demo"> with a duplicate
<field name="title">.
Green PASS box:
PASS — the existing field was kept (
titleappears 1× in the form XML, label:
“Label from VARIANT xml (must win)”).
The rendered fieldset shows title once with the VARIANT label, and the new
extra field from the base XML is still added correctly.
Form from a variant XML that defines the field title$form->load($baseXml, false) that also definestitle (label “Label from BASE xml (must NOT win)”) plus a new field extra.com_content's CategoryModel::preprocessForm() doesPlease select:
Documentation link for guide.joomla.org:
No documentation changes for guide.joomla.org needed
Pull Request link for manual.joomla.org:
No documentation changes for manual.joomla.org needed
| Status | New | ⇒ | Pending |
| Category | ⇒ | Libraries |
| Title |
|
||||||
mod_demo47451_1.0.0.zip
I created a minimal module extension to be used for testing.
I will also update testing instruction above accordingly.
new PR instead of #47452