Spin-off from #1626584: Combine configuration system changes to verify they are compatible

UpgradePathTestBase::performUpgrade() does not always load all new modules after upgrade.

CommentFileSizeAuthor
#3 next.upgradepath.3.patch1.3 KBsun
next.upgradepath.0.patch1.2 KBsun

Comments

moshe weitzman’s picture

moshe weitzman commented
Status: Needs review » Reviewed & tested by the community

Completely harmless, though something should be done in a new issue if $this->loadedModules is not reliable.

chx’s picture

chx commented
Status: Reviewed & tested by the community » Needs work

Please remove loadedModules then completely.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Needs work » Needs review
StatusFileSize
new next.upgradepath.3.patch1.3 KB

Removed $loadedModules entirely.

chx’s picture

chx commented
Status: Needs review » Reviewed & tested by the community

Thanks.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
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.

Anonymous’s picture

(not verified) commented
Issue summary: View changes

Updated issue summary.