Skip to content
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

Remove subform in subform limitation for custom fields #38178

Closed
wants to merge 1 commit into from

Conversation

HLeithner
Copy link
Member

Summary of Changes

Remove the limitation to select a subform as a subform field in custom fields.

There is no reason we have this limit, subforms can already be used as field of another subform.

Testing Instructions

  • Create a number of custom fields
  • Create a subform !1
  • Add the fields to the subform
  • Create a subform !2
  • Add subform !1 to subform !2
  • check if everything works

Actual result BEFORE applying this Pull Request

It's not possible to select subform !1 in subform !2

Expected result AFTER applying this Pull Request

You can select subform !1 in subform !2 and you also can edit it

@wilsonge
Copy link
Contributor

I think we need to consider a nesting limit to stop the UI from falling apart.

@brianteeman
Copy link
Contributor

a limit of zero - aka the current situation seems perfect to me

@HLeithner
Copy link
Member Author

I think we need to consider a nesting limit to stop the UI from falling apart.

didn't looked at it yet but someone already came up with the question and I hope this protection is already in the field^^

@HLeithner HLeithner closed this Jun 29, 2022
@HLeithner HLeithner reopened this Jun 29, 2022
@wilsonge
Copy link
Contributor

I think that protection is the code your deleting ;)

@crystalenka
Copy link
Member

I would be very happy to have this in core, personally.

@Hackwar Hackwar added the Small A PR which only has a small change label Feb 26, 2023
@Hackwar Hackwar added the bug label Apr 6, 2023
@HLeithner HLeithner changed the base branch from 4.2-dev to 4.3-dev May 2, 2023 16:30
@HLeithner
Copy link
Member Author

This pull request has been automatically rebased to 4.3-dev.

@HLeithner HLeithner changed the base branch from 4.3-dev to 4.4-dev September 30, 2023 22:45
@HLeithner
Copy link
Member Author

This pull request has been automatically rebased to 4.4-dev.

@Quy
Copy link
Contributor

Quy commented Apr 3, 2024

Alternate #43204 that appears to be more complete.

@Quy
Copy link
Contributor

Quy commented Apr 6, 2024

Closing in favor of #43204. Thanks.

@Quy Quy closed this Apr 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug PR-4.4-dev Small A PR which only has a small change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants