? ? Pending

User tests: Successful: Unsuccessful:

avatar okonomiyaki3000
okonomiyaki3000
16 May 2019

Pull Request for Issue # .

Summary of Changes

Subform repeatable has a function fixScripts which calls a few js initialization functions on a few of the field types that require it. There is a now a better way to handle that kind of thing by having any field that may need initialization listen for an event fired by subform repeatable. In this PR, I am transferring responsibility of one such initialization from subform repeatable to the correct field type itself.

Note: this PR doesn't really fix anything. Nest subforms already work with repeatable subforms. This just does it in a somewhat better way.

Testing Instructions

Set up a subform containing a subform. There's a subform in the redirect plugin that's simple to use for this purpose.

Open up the form and create new rows. The subform in each new row should function properly.

Expected result

Nested subforms function normally.

Actual result

Nested subforms function normally.

Documentation Changes Required

None

avatar okonomiyaki3000 okonomiyaki3000 - open - 16 May 2019
avatar okonomiyaki3000 okonomiyaki3000 - change - 16 May 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 16 May 2019
Category JavaScript
avatar Quy
Quy - comment - 18 May 2019

I have tested this item successfully on 6e6290a


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

avatar Quy Quy - test_item - 18 May 2019 - Tested successfully
avatar viocassel
viocassel - comment - 18 May 2019

I have tested this item successfully on 6e6290a


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

avatar viocassel viocassel - test_item - 18 May 2019 - Tested successfully
avatar Quy Quy - change - 18 May 2019
Status Pending Ready to Commit
avatar Quy
Quy - comment - 18 May 2019

RTC


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

avatar okonomiyaki3000 okonomiyaki3000 - change - 20 May 2019
Labels Added: ?
avatar HLeithner HLeithner - change - 20 May 2019
Labels Added: ?
avatar HLeithner HLeithner - close - 20 May 2019
avatar HLeithner HLeithner - merge - 20 May 2019
avatar HLeithner HLeithner - change - 20 May 2019
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-05-20 11:46:04
Closed_By HLeithner
avatar HLeithner
HLeithner - comment - 20 May 2019

thx

Add a Comment

Login with GitHub to post a comment