Skip to content
Snippets Groups Projects

Issue #3512254: Removed the #type fieldgroup declarations from SiteConfigureForm.php.

Closed Issue #3512254: Removed the #type fieldgroup declarations from SiteConfigureForm.php.
1 unresolved thread
1 unresolved thread

Closes #3512254

Merge request reports

Approval is optional
Code Quality is loading
Test summary results are being parsed

Closed by Théodore BiadalaThéodore Biadala 1 month ago (Apr 4, 2025 1:15pm UTC)

Merge details

  • The changes were not merged into 11.x.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
113 113 // work during installation.
114 114 $form['#attached']['drupalSettings']['copyFieldValue']['edit-site-mail'] = ['edit-account-mail'];
115 115
116 // @deprecated The #type 'fieldgroup' is deprecated in drupal:11.2.0 and is
117 // removed from drupal:12.0.0.
118 // @todo Use #type 'fieldset' and add fieldgroup as class attribute instead.
  • The original change here was correct - to actually change the type in the form.

    The deprecation that needs to be added is to Drupal\Core\Render\Element\Fieldgroup - probably in a constructor.

    Additionally, the places that used to use #type 'fieldgroup' will now need to add the 'core/drupal.fieldgroup/library' - e.g. just this form.

  • sayan_k_dutta changed this line in version 9 of the diff

    changed this line in version 9 of the diff

  • Please register or sign in to reply
  • sayan_k_dutta added 1 commit

    added 1 commit

    • c249218b - Issue #3512254: Added the deprication in Fieldgroup class and made the changes...

    Compare with previous version

  • Please register or sign in to reply
    Loading