Bartosz Dziewoński has uploaded a new change for review.

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


Change subject: Rename SpecialRecentChanges::addRecentChangesJS() to 
addModules()
......................................................................

Rename SpecialRecentChanges::addRecentChangesJS() to addModules()

…and make it protected. It makes no sense to include class name in the
function name, especially since subclasses might want to customize it.

Change-Id: Ic0d23f4400b88596fdb11f6c0d8aefc9aea589ba
---
M RELEASE-NOTES-1.22
M includes/specials/SpecialRecentchanges.php
2 files changed, 5 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core 
refs/changes/75/89475/1

diff --git a/RELEASE-NOTES-1.22 b/RELEASE-NOTES-1.22
index a365da6..32c1854 100644
--- a/RELEASE-NOTES-1.22
+++ b/RELEASE-NOTES-1.22
@@ -489,6 +489,8 @@
   of media handler overriding MediaHandler::parseParamString.
 * (bug 46512) The collapsibleNav feature from the Vector extension has been 
moved
   to the Vector skin in core.
+* SpecialRecentChanges::addRecentChangesJS() function has been renamed
+  to addModules() and made protected.
 
 == Compatibility ==
 
diff --git a/includes/specials/SpecialRecentchanges.php 
b/includes/specials/SpecialRecentchanges.php
index 60cdb0e..a699478 100644
--- a/includes/specials/SpecialRecentchanges.php
+++ b/includes/specials/SpecialRecentchanges.php
@@ -154,7 +154,7 @@
                $opts = $this->getOptions();
                $this->setHeaders();
                $this->outputHeader();
-               $this->addRecentChangesJS();
+               $this->addModules();
 
                // Fetch results, prepare a batch link existence check query
                $conds = $this->buildMainQueryConds( $opts );
@@ -911,9 +911,9 @@
        }
 
        /**
-        * Add JavaScript to the page
+        * Add page-specific modules.
         */
-       function addRecentChangesJS() {
+       protected function addModules() {
                $this->getOutput()->addModules( array(
                        'mediawiki.special.recentchanges',
                ) );

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic0d23f4400b88596fdb11f6c0d8aefc9aea589ba
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Bartosz Dziewoński <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to