Problem/Motivation

#2021959: Refactor module handling responsibilities out of DrupalKernel was closed and marked as a duplicate of #2941757: Extension System, Part IV: Properly register all installed extensions/namespaces during container generation but it is still referenced in a @todo comment in FunctionalTestSetupTrait.

Steps to reproduce

See the docblock of the rebuildContainer method on FunctionalTestSetupTrait.

Proposed resolution

Update the comment to point to the new issue.

Remaining tasks

Determine if any specific language in the comments also needs to be updated.

User interface changes

None.

API changes

None.

Data model changes

None.

Issue fork drupal-3202787

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

richardbporter created an issue. See original summary.

richardbporter opened merge request !411

richardbporter’s picture

richardbporter CreditAttribution: richardbporter as a volunteer commented
Status: Active » Needs review
longwave’s picture

longwave
he/him
Primary language English
Location UK
CreditAttribution: longwave at Full Fat Things commented
Status: Needs review » Reviewed & tested by the community

Confirmed that the new reference is correct and still valid.

  • catch committed 11154c2 on 9.2.x 
    Issue #3202787 by richardbporter, longwave: Update @todo comment in...
catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch at Third and Grove commented
Status: Reviewed & tested by the community » Fixed

Other option would have been to remove this entirely and rely on the issue(s), but this is weird enough it seems worth keeping.

Committed 11154c2 and pushed to 9.2.x. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.