? ? Pending

User tests: Successful: Unsuccessful:

avatar Fedik
Fedik
27 Sep 2021

Pull Request for Issue #35677 .

Summary of Changes

After #24711 the fieldname is always empty (for subform fields which added in XML),
that leads to incorrect data-base-name attribute.

Testing Instructions

Please follow #35677
Apply patch,
Add subform somwhere in XML, example in Custom HTML module:

<field type="subform" name="foobar" label="subform" multiple="true">
  <form>
    <field type="text" name="text" label="text" />
  </form>
</field>

And look for data-base-name attribute of each section of the field.

Actual result BEFORE applying this Pull Request

The attribute always like __fieldX

Expected result AFTER applying this Pull Request

The attribute equal to field name like foobar

Documentation Changes Required

none

avatar Fedik Fedik - open - 27 Sep 2021
avatar Fedik Fedik - change - 27 Sep 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 27 Sep 2021
Category Libraries
avatar Fedik
Fedik - comment - 27 Sep 2021

@PhocaCz please test

avatar PhocaCz PhocaCz - test_item - 27 Sep 2021 - Tested successfully
avatar PhocaCz
PhocaCz - comment - 27 Sep 2021

I have tested this item successfully on eb5b726

Tested in User Options -> Email Domain Options -> Add Email Domains

Before patch:
img

After patch:
img


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

avatar JonasAtZwetschke
JonasAtZwetschke - comment - 8 Oct 2021

I have tested this item successfully on eb5b726


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

avatar JonasAtZwetschke JonasAtZwetschke - test_item - 8 Oct 2021 - Tested successfully
avatar alikon alikon - change - 8 Oct 2021
Status Pending Ready to Commit
avatar alikon
alikon - comment - 8 Oct 2021

RTC


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

avatar richard67 richard67 - close - 1 Nov 2021
avatar richard67 richard67 - merge - 1 Nov 2021
avatar richard67 richard67 - change - 1 Nov 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-11-01 13:30:24
Closed_By richard67
Labels Added: ? ?
avatar richard67
richard67 - comment - 1 Nov 2021

Thanks

avatar PhocaCz
PhocaCz - comment - 1 Nov 2021

Thank you.

Add a Comment

Login with GitHub to post a comment