Broken off from #1849004: One Service Container To Rule Them All based on feedback from Sun in #12.

Just renaming of a variable to make more sense.

CommentFileSizeAuthor
#1 1849636_1_rename_forum_container.patch3.93 KBLetharion
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Letharion’s picture

Letharion CreditAttribution: Letharion commented
FileSize
1849636_1_rename_forum_container.patch3.93 KB
Letharion’s picture

Letharion CreditAttribution: Letharion commented

Code written by chx, I just broke it out.

aspilicious’s picture

aspilicious CreditAttribution: aspilicious commented
Status: Active » Needs review
sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented
Status: Needs review » Reviewed & tested by the community
Issue tags: +Quick fix

Totally correct.

Make sure to credit @chx for this patch.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

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