User tests: Successful: Unsuccessful:
Currently it converts all nested objects to array silently (see #3737). Some components relying on old behavior (prior to #3737) may expect object type values for the form fields.
This fix does not process the entire nested structure, instead converts only binding levels to array for internal iterations.
Status | New | ⇒ | Pending |
Labels |
Added:
?
|
Can This ne tested somehow? E.g. With a extension that don't work bevor but after that patch?
@zero-24 I am not particularly seeing any such extension, however the referenced PR (#3737) has changed the behavior of JForm bind method in this regard, as a result of which few extensions are likely to break.
That issue is fixed here along with the fix made by #3737. Therefore both the test cases should pass now.
The comment #3737 (comment) by @vietvh mentions some issue regarding previous change.
@izharaazmi : the new change in this pull request worked fine with my code. At least from what I tested.
The common form that I've seen from core Joomla components and many 3rd extensions is using this structure:
From your use case, you are nesting 2 same tags (form > fields > fields > field). This is the first time I see this therefore I am a little bit curious, why did you have to use that instead of the common pattern 1 and 2 above?
@vietvh You are right about the patterns used commonly. But that is not restricted by Joomla, and the nesting like fields
> fields
> fields
> ...
at any level creates the form elements at the same depth.
I'd also like to mention that fieldset
does not affect the nesting of input fields in the output.
Therefore to replicate the nesting issue in minimal code I just used such nesting. Hope this answers your query.
@Kubik-Rubik The PR for the extended issue reported after merge in #3737 is here. Kindly have a look at your convenience.
I have tested this item successfully on 64bc802
I have tested this successfully. The tags were not working before, they are working now.
Status | Pending | ⇒ | Ready to Commit |
Setting to RTC as we have 2 successful tests.
Labels |
Added:
?
|
Status | Ready to Commit | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-11-21 18:57:42 |
Closed_By | ⇒ | rdeutz |
Labels |
Removed:
?
|
Use the following code to test:
@vietvh Please check this out and cross test against #3737 (comment)