Problem/Motivation

Re: A comment from @Wim Leers after #2005546: Use branding block in place of page template branding variables (site name, slogan, site logo) went in:

+++ b/core/modules/system/system.install
@@ -1462,7 +1463,78 @@ function system_update_8005() {
+    'settings.cache.max_age' => 0,

"This line sadly means that any D8 site that already existed will get a branding block (yay!) that is marked to not ever be cached (boo!).

This line simply does not belong here. So… it looks like we need a new update function that takes the site branding block and removes this setting"

Proposed resolution

Remove the offending line.

Remaining tasks

Generate a patch

Comments

dcrocks created an issue. See original summary.

dcrocks’s picture

Status: Active » Closed (fixed)

Well, somebody already did this in another patch. Sorry for the noise.