-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[5.2] Allow nested Subform inside a Subform Custom Field #43204
Conversation
Co-authored-by: Richard Fath <[email protected]>
@richard67 Thanks, fixed! |
Seems to do the job. |
@regularlabs Could you please mark your test result in the issue tracker? For this go to https://issues.joomla.org/tracker/joomla-cms/43204 and use the blue "Test this" button at the top left corner, then select your test result and finally submit. Thanks in advance. |
I have tested this item ✅ successfully on 041cd38 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43204. |
1 similar comment
I have tested this item ✅ successfully on 041cd38 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43204. |
RTC This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43204. |
Thanks guys! |
Is an endless loop considered here? subform1 -> subform2 -> subform1 I don't think so, so maybe we should check for recursion at some point (rendering or saving) |
It's not considered, but then again, that would be the user that purposely breaks the form by doing something like that. It's not different then embedding a module inside another module which embeds the module itself, etc. I don't think we need to cover the case where the user does something dumb |
Thanks @AndySDH ! |
* Allow nested Subform inside a Subform Custom Field * Allow a Subform to be set a Subform-only field as well * Allow Subform Field to be set as only_use_in_subform * Fix PHPCS failing due to white space in empty line. Co-authored-by: Richard Fath <[email protected]> * Remove unused variable and redundant comment
How do I use this? This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43204. |
Summary of Changes
When creating a custom field of type Subform, you can select the child fields that you want as part of the Subform.
In the current situation, the dropdown of fields selection doesn't allow to select a Subform to be nested inside a Subform. This was probably done to avoid cluttering the interface too much if somebody wanted to do it.
But there is a legimitate use for this and somebody may want to intentionally have a nested Subform inside a Subform. I am currently having a legitimate use case where I want a Subform inside a Subform and I can't do it because of this limitation. There is no reason to have this limitation, let the user do it if they want to.
Testing Instructions
Actual result BEFORE applying this Pull Request
Expected result AFTER applying this Pull Request
Link to documentations