- 🇺🇸United States pfrilling Minster, OH
@RandalV, See my comment in #20: https://www.drupal.org/project/drupal/issues/2269823#comment-13249588 ✨ Enable grouping for checkboxes and radio buttons Needs work . It looks like you're missing the target container of the sub-key.
Did you try the following?
$form['element']['category_container']['option_key']['#disabled'] = TRUE;
In order to target the groups, you need to lowercase the group name and add '_container' as a suffix to the key
- Status changed to Needs review
almost 2 years ago 4:41am 21 February 2023 - 🇮🇳India Abhisheksingh27
Adding Reroll for D10 as the above #40 patch failed to apply in D10.
The last submitted patch, 46: 2269823-46.patch, failed testing. View results →
- Status changed to Needs work
over 1 year ago 2:08pm 29 August 2023 Patch tested on D10.1.2
$element[$key] += [
logic should be inside the else statement otherwise there are errors on node edit page related to empty $default_value.- last update
over 1 year ago 29,470 pass - Status changed to Needs review
12 months ago 2:13pm 25 January 2024 - last update
12 months ago Custom Commands Failed - Status changed to Needs work
12 months ago 9:33pm 28 January 2024 - 🇺🇸United States smustgrave
Didn't give a review but #50 appears to have failures.
Recommend using MRs
Issue summary could use some attention also, using standard issue template
- last update
12 months ago Custom Commands Failed - last update
12 months ago Custom Commands Failed - last update
12 months ago Custom Commands Failed - last update
12 months ago Custom Commands Failed - Assigned to mahaveer003
- Status changed to Needs review
8 months ago 12:52pm 1 June 2024 - 🇮🇳India mahaveer003
I have re-written from Patch #50 for 11.x and not checked with PHPCBF. I will check it once it get some more time.. Till that i would like to assign this to myself and let the test run.
I believe No interdiff required. - Status changed to Needs work
8 months ago 8:37pm 1 June 2024 The Needs Review Queue Bot → tested this issue.
While you are making the above changes, we recommend that you convert this patch to a merge request → . Merge requests are preferred over patches. Be sure to hide the old patch files as well. (Converting an issue to a merge request without other contributions to the issue will not receive credit.)
- First commit to issue fork.
- Merge request !105472269823 | Grouping of checkboxes/radio buttons → (Open) created by Unnamed author
- 🇧🇪Belgium borisson_ Mechelen, 🇧🇪
The merge request has merge conflicts that need to resolved. I also think there needs to be better test coverage, it is currently only tested through
core/modules/field/tests/src/Functional/EntityReference/EntityReferenceXSSTest.php
.