Page MenuHomePhabricator

GlobalVarConfig throws exceptions for globals with null value
Closed, ResolvedPublic

Description

For example, the default value for $wgRightsPage is null. And https://www.mediawiki.org/w/api.php?action=query&meta=siteinfo&siprop=rightsinfo is throwing an exception when it isn't overridden.

Patch to follow momentarily.


Version: unspecified
Severity: normal

Details

Reference
bz66986

Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 3:32 AM
bzimport set Reference to bz66986.

Change 141468 had a related patch set uploaded by Anomie:
GlobalVarConfig shouldn't throw exceptions for null-valued config settings

https://gerrit.wikimedia.org/r/141468

Change 141468 merged by jenkins-bot:
GlobalVarConfig shouldn't throw exceptions for null-valued config settings

https://gerrit.wikimedia.org/r/141468

Should backport this, no reason to let stuff be broken for a week until wmf11 rolls out.

I've scheduled it for the Tuesday morning SWAT, but if someone wants to do it this evening feel free.

Change 141680 had a related patch set uploaded by Anomie:
GlobalVarConfig shouldn't throw exceptions for null-valued config settings

https://gerrit.wikimedia.org/r/141680

Change 141680 merged by jenkins-bot:
GlobalVarConfig shouldn't throw exceptions for null-valued config settings

https://gerrit.wikimedia.org/r/141680

Change 160688 had a related patch set uploaded by Legoktm:
GlobalVarConfig shouldn't throw exceptions for null-valued config settings

https://gerrit.wikimedia.org/r/160688

Change 160688 merged by Legoktm:
GlobalVarConfig shouldn't throw exceptions for null-valued config settings

https://gerrit.wikimedia.org/r/160688

Will also be included in the next 1.23.x release.