Expected behaviour: When you edit sites/default/settings.php and uncomment the line $conf['site_name'] = 'My Drupal site'; and save the file, the site name should change to "My Drupal site."

Actual behaviour: It doesn't. drush cc all doesn't help.

There's a chance this might be 3am syndrome, but I think I double-checked everything.

#5 conf_sitename-1716920-5.patch549 bytesAlbert Volkman
PASSED: [[SimpleTest]]: [MySQL] 39,875 pass(es).
[ View ]


aspilicious’s picture

try this:

$conf['system.site']['name'] = 'My Drupal site'

aspilicious’s picture

Status:Active» Closed (works as designed)

Tried it myself, it works and we have tests for this ;)

webchick’s picture

Title:$conf overrides are not working» $conf override example documentation needs updating
Priority:Critical» Normal
Status:Closed (works as designed)» Active
Issue tags:-regression+Novice

Oh, well that makes sense. However, there's still a bug here. But now it's normal and novice. :)

webchick’s picture

You can find this documentation in sites/default/default.settings.php.

Albert Volkman’s picture

Status:Active» Needs review
new549 bytes
PASSED: [[SimpleTest]]: [MySQL] 39,875 pass(es).
[ View ]

Here ya go.

aspilicious’s picture

I would like to change all the examples once the conversion is done. Most of these will change in the near future.

COBadger’s picture

I applied the patch, removed the leading hash tag in front of:

$conf['system.site']['name'] = 'My Drupal site';

Cleared caches and reloaded the page; site name did not change and I don't see a row in 'variable' table in db for site name.

Didn't work for me.

jfhovinne’s picture

Status:Needs review» Reviewed & tested by the community

@jordanlgraham: I think you need to apply the patch before installing Drupal.

Applied the patch to current 8.x-dev; installed Drupal; uncommented $conf['system.site']['name'] = 'My Drupal site'; in settings.php; the site name is correctly updated to "Welcome to My Drupal site", so the example is ok.

webchick’s picture

Status:Reviewed & tested by the community» Postponed


Committed and pushed to 8.x as a stop-gap. Agreed that we're going to need to update this as further conversions take place, however. Probably makes sense to postpone this for now on #1775842: [meta] Convert all variables to state and/or config systems.

mgifford’s picture

Issue summary:View changes
Status:Postponed» Active
Related issues:+#2167109: Remove Variable subsystem

Ok, so #2167109: Remove Variable subsystem replaced #1775842: [meta] Convert all variables to state and/or config systems so can this be closed or are there further conversions that need to take place?

jhodgdon’s picture

Component:config.module» configuration system

If this is still valid, it seems like it is base config system, not Configuration Manager module?

mgifford’s picture

Status:Active» Closed (fixed)

Currently, without the patch it's # $config['system.site']['name'] = 'My Drupal site'; so I'm marking this closed.