Page MenuHomePhabricator

[Extension:UserExport] Remove the usage of global variable $wgUser, $wgRequest, $wgOut
Closed, ResolvedPublic

Description

Problem

Now we are replacing the global variable by getting the context. The Classes which extends SpecialPage can directly access the context by $this.

Solution

Replace

  1. $wgUser with $this->getUser()
  2. $wgRequest with $this->getRequest()
  3. $wgOut with $this->getOutput()

in UserExport.body.php. If there are multiple usages then use a variable to store context instead of calling $this again and again.

Level

Easy

Reserved for Mini-MWT at VVIT (Feb 2019)

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Change 492046 had a related patch set uploaded (by Ananth subray; owner: Ananth subray):
[mediawiki/extensions/UserExport@master] Removed the usage of global variable $wgUser, $wgRequest, $wgOut

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

Change 492046 merged by jenkins-bot:
[mediawiki/extensions/UserExport@master] Removed the usage of global variable $wgUser, $wgRequest, $wgOut

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