Page MenuHomePhabricator

MediaWiki UI shows unintentionally on Categories and Contributions due to incorrect use of getConfig return value
Closed, ResolvedPublic

Description

This affects wmf/1.25wmf2 (currently on all Wikipedias) but not 1.25wmf3 or master. It happened to get fixed by a155ac55ea0854b0989efe23158d33b5a548f14e .

Basically, Special:Categories and Special:Contributions were calling:

if ( $this->getConfig( 'UseMediaWikiUIEverywhere' ) ) {

(passing an unused parameter to a no-args method, then using the config object as a boolean)

instead of the correct:

if ( $this->getConfig()->get( 'UseMediaWikiUIEverywhere' ) ) {

I think we should backport the fix on Monday (it can be done by making the above change rather than the whole a155ac55) since it's very inconsistent (only the button is MW UI, not the text boxes or checkboxes) and Special:Contributions is a prominent page.


Version: 1.25-git
Severity: normal

Details

Reference
bz71945

Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 3:47 AM
bzimport set Reference to bz71945.
bzimport added a subscriber: Unknown Object (MLST).

See my email to design mailing list UI form standardisation groans

I say we just wait till it corrects itself Thursday. It's just a blue button.

Matt/Jon: If you want to SWAT this out on Tuesday morning that's totally fine.

I don't. it's just a fully functional button that just happens to be blue that will go away 2 days later. Seems like a bunch of effort for the sake of 2 days.