Hydriz has submitted this change and it was merged.

Change subject: Sync with GitHub repository.
......................................................................


Sync with GitHub repository.

As per a simple code review by SPQRobin, this change:
1) Removes the dependency on globals and replacing it with
$this->getOutput().
2) Escapes the extension messages to make it safer.

Also with this change is:
1) Assigning protection for the functions.

Change-Id: I3055168b08fbaf9f5fe2bc71c428b92e4fb361cf
---
M SpecialHostStats.php
1 file changed, 6 insertions(+), 7 deletions(-)

Approvals:
  Hydriz: Verified; Looks good to me, approved


--
To view, visit https://gerrit.wikimedia.org/r/33047
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I3055168b08fbaf9f5fe2bc71c428b92e4fb361cf
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/HostStats
Gerrit-Branch: master
Gerrit-Owner: Hydriz <ad...@alphacorp.tk>
Gerrit-Reviewer: Hydriz <ad...@alphacorp.tk>
Gerrit-Reviewer: SPQRobin <robinp.1...@gmail.com>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to