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