Following up on #605880: Settings and menus broken after install due to faulty caching there is an out-of-date code comment in the installer which says that we try to switch cache implementations midway through the installer, once the database is available.

In fact, we intentionally do not do that anymore - as per the patch that was committed a while ago at #557542: Cache module_implements() .

The attached patch fixes the code comment and also expands it to explain why we do things the way we are doing them.

CommentFileSizeAuthor
drupal-fake-cache-installer-comment.patch1.56 KBDavid_Rothstein

Comments

catch’s picture

catch
he/him
Primary language English
commented
Status: Needs review » Reviewed & tested by the community

Looks good.

webchick’s picture

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

Committed to HEAD. Thanks!

Status: Fixed » Closed (fixed)

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