bug Small PR-4.4-dev Pending

User tests: Successful: Unsuccessful:

avatar HLeithner
HLeithner
29 Jun 2022

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

avatar HLeithner HLeithner - open - 29 Jun 2022
avatar HLeithner HLeithner - change - 29 Jun 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 29 Jun 2022
Category Administration com_fields
avatar wilsonge
wilsonge - comment - 29 Jun 2022

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

avatar brianteeman
brianteeman - comment - 29 Jun 2022

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

avatar HLeithner
HLeithner - comment - 29 Jun 2022

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^^

avatar HLeithner HLeithner - close - 29 Jun 2022
avatar HLeithner HLeithner - change - 29 Jun 2022
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2022-06-29 14:58:54
Closed_By HLeithner
Labels Added: ?
avatar HLeithner HLeithner - change - 29 Jun 2022
Status Closed New
Closed_Date 2022-06-29 14:58:54
Closed_By HLeithner
avatar HLeithner HLeithner - change - 29 Jun 2022
Status New Pending
avatar HLeithner HLeithner - reopen - 29 Jun 2022
avatar wilsonge
wilsonge - comment - 29 Jun 2022

I think that protection is the code your deleting ;)

avatar crystalenka
crystalenka - comment - 28 Oct 2022

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

avatar HLeithner
HLeithner - comment - 2 May 2023

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

avatar HLeithner
HLeithner - comment - 30 Sep 2023

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

avatar Quy
Quy - comment - 3 Apr 2024

Alternate #43204 that appears to be more complete.

avatar Quy Quy - change - 6 Apr 2024
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2024-04-06 15:41:14
Closed_By Quy
Labels Added: bug Small PR-4.4-dev
Removed: ?
avatar Quy
Quy - comment - 6 Apr 2024

Closing in favor of #43204. Thanks.

avatar Quy Quy - close - 6 Apr 2024

Add a Comment

Login with GitHub to post a comment