Thanks to @Krenair we avoided having no event logging during an A/B test.
We confusingly have a global config variable called wgSchemaPopupsSamplingRate that is passed to a mediawiki config variable called
'wgPopupsSchemaPopupsSamplingRate'
This can lead to confusion like seen today in https://gerrit.wikimedia.org/r/310483
We should probably rename the global config variable to wgPopupsSchemaSamplingRate and rename the mw.config value.
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Hygiene: Rename SchemaPopupsSamplingRate | mediawiki/extensions/Popups | mpga | +9 -9 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | Jdlrobson | T144522 [EPIC] Hovercards technical debt and quality enhancements | |||
| Resolved | phuedx | T149801 [EPIC] Refactor Hovercards (née Popups) | |||
| Resolved | • Jhernandez | T146434 SchemaPopupsSamplingRate config variable is badly named |
For the benefit of someone who would want to make this change, what would the process of renaming a global variable like that be, without breaking existing behavior?
Hi there @McIntireEvan, thanks for your interest in helping improve our codebase! Unfortunately, this task should have been marked "Stalled" a few weeks ago when we started working on a major rewrite of the Hovercards code, so I'm not sure there's much to be done here. There are, however, a few other tasks of similar complexity you could help us with, if you are so inclined!
If these don't interest you, another great place to look is the good first task project. Thanks!
Thanks for the response! I'll take a look at these for myself and to mentor for GCI :)
@McIntireEvan: Tell us if we should make this a Google-Code-In-2016 task! :)
Edit: Sorry. Just saw T146434#2854844.
Change 332151 had a related patch set uploaded (by Phuedx):
Hygiene: Rename SchemaPopupsSamplingRate
Renamed on the mpga branch (see rEPOP98b2dcf631f1: Hygiene: Rename SchemaPopupsSamplingRate), created T155386: mw.org Docs: Update config variable docs when mpga merged to master as follow up to update docs when merging to master.