Currently we find this in settings.php

$conf['system.performance']['css.gzip'] = FALSE;
$conf['system.performance']['js.gzip'] = FALSE;

This is incorrect as everything behind system.performance is an array so we have to access it as an array
Following code DOES work.

$conf['system.performance']['css']['gzip'] = FALSE;
$conf['system.performance']['js']['gzip'] = FALSE;

System.performance is a KEY js.css are two ARRAY ELEMENTS. We can't combine array elements with a '.'

Attached patch fixes this misunderstanding. I tested this with css/js aggregation.

CommentFileSizeAuthor
conf_overrides.patch625 bytesaspilicious
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir commented
Status: Needs review » Reviewed & tested by the community

Makes sense. It is a bit unfortunate that this doesn't map to the way it is accessed but changing that would be a different issue.

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!

Status: Fixed » Closed (fixed)

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