[MediaWiki-commits] [Gerrit] mediawiki/core[master]: Remove unused variables from closures

2017-09-19 Thread jenkins-bot (Code Review)
jenkins-bot has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/378897 )

Change subject: Remove unused variables from closures
..


Remove unused variables from closures

Change-Id: I5d2e0cd423a4080677f8517bcf3dcee10bf44799
---
M resources/src/mediawiki.special/mediawiki.special.changeslist.visitedstatus.js
M 
resources/src/mediawiki.special/mediawiki.special.preferences.convertmessagebox.js
M resources/src/mediawiki.widgets/MediaSearch/mw.widgets.MediaSearchQueue.js
M resources/src/mediawiki.widgets/mw.widgets.TitleSearchWidget.js
M resources/src/mediawiki/mediawiki.checkboxtoggle.js
M tests/qunit/suites/resources/jquery/jquery.makeCollapsible.test.js
M 
tests/qunit/suites/resources/mediawiki.special/mediawiki.special.recentchanges.test.js
M tests/qunit/suites/resources/mediawiki/mediawiki.errorLogger.test.js
8 files changed, 16 insertions(+), 16 deletions(-)

Approvals:
  Bartosz DziewoƄski: Looks good to me, approved
  jenkins-bot: Verified



diff --git 
a/resources/src/mediawiki.special/mediawiki.special.changeslist.visitedstatus.js
 
b/resources/src/mediawiki.special/mediawiki.special.changeslist.visitedstatus.js
index 954e074..6b25327 100644
--- 
a/resources/src/mediawiki.special/mediawiki.special.changeslist.visitedstatus.js
+++ 
b/resources/src/mediawiki.special/mediawiki.special.changeslist.visitedstatus.js
@@ -1,7 +1,7 @@
 /*!
  * JavaScript for Special:Watchlist
  */
-( function ( mw, $ ) {
+( function ( $ ) {
$( function () {
$( '.mw-changeslist-line-watched .mw-title a' ).on( 'click', 
function () {
$( this )
@@ -9,4 +9,4 @@
.removeClass( 'mw-changeslist-line-watched' );
} );
} );
-}( mediaWiki, jQuery ) );
+}( jQuery ) );
diff --git 
a/resources/src/mediawiki.special/mediawiki.special.preferences.convertmessagebox.js
 
b/resources/src/mediawiki.special/mediawiki.special.preferences.convertmessagebox.js
index cee7382..e6b7432 100644
--- 
a/resources/src/mediawiki.special/mediawiki.special.preferences.convertmessagebox.js
+++ 
b/resources/src/mediawiki.special/mediawiki.special.preferences.convertmessagebox.js
@@ -1,9 +1,9 @@
 /*!
  * JavaScript for Special:Preferences: Check for successbox to replace with 
notifications.
  */
-( function ( mw, $ ) {
+( function ( $ ) {
$( function () {
var convertmessagebox = require( 
'mediawiki.notification.convertmessagebox' );
convertmessagebox();
} );
-}( mediaWiki, jQuery ) );
+}( jQuery ) );
diff --git 
a/resources/src/mediawiki.widgets/MediaSearch/mw.widgets.MediaSearchQueue.js 
b/resources/src/mediawiki.widgets/MediaSearch/mw.widgets.MediaSearchQueue.js
index 7ee98bb..3639525 100644
--- a/resources/src/mediawiki.widgets/MediaSearch/mw.widgets.MediaSearchQueue.js
+++ b/resources/src/mediawiki.widgets/MediaSearch/mw.widgets.MediaSearchQueue.js
@@ -4,7 +4,7 @@
  * @copyright 2011-2016 VisualEditor Team and others; see AUTHORS.txt
  * @license The MIT License (MIT); see LICENSE.txt
  */
-( function ( $, mw ) {
+( function ( mw ) {
 
/**
 * MediaWiki media resource queue.
@@ -79,4 +79,4 @@
mw.widgets.MediaSearchQueue.prototype.getSearchQuery = function () {
return this.getParams().gsrsearch;
};
-}( jQuery, mediaWiki ) );
+}( mediaWiki ) );
diff --git a/resources/src/mediawiki.widgets/mw.widgets.TitleSearchWidget.js 
b/resources/src/mediawiki.widgets/mw.widgets.TitleSearchWidget.js
index 5ba9481..cd937f7 100644
--- a/resources/src/mediawiki.widgets/mw.widgets.TitleSearchWidget.js
+++ b/resources/src/mediawiki.widgets/mw.widgets.TitleSearchWidget.js
@@ -4,7 +4,7 @@
  * @copyright 2011-2015 MediaWiki Widgets Team and others; see AUTHORS.txt
  * @license The MIT License (MIT); see LICENSE.txt
  */
-( function ( $, mw ) {
+( function ( mw ) {
 
/**
 * Creates an mw.widgets.TitleSearchWidget object.
@@ -102,4 +102,4 @@
return response.query || {};
};
 
-}( jQuery, mediaWiki ) );
+}( mediaWiki ) );
diff --git a/resources/src/mediawiki/mediawiki.checkboxtoggle.js 
b/resources/src/mediawiki/mediawiki.checkboxtoggle.js
index 36090ea..76bc86c 100644
--- a/resources/src/mediawiki/mediawiki.checkboxtoggle.js
+++ b/resources/src/mediawiki/mediawiki.checkboxtoggle.js
@@ -9,7 +9,7 @@
  * Jeroen De Dauw 
  */
 
-( function ( mw, $ ) {
+( function ( $ ) {
'use strict';
 
$( function () {
@@ -35,4 +35,4 @@
 
} );
 
-}( mediaWiki, jQuery ) );
+}( jQuery ) );
diff --git a/tests/qunit/suites/resources/jquery/jquery.makeCollapsible.test.js 
b/tests/qunit/suites/resources/jquery/jquery.makeCollapsible.test.js
index 0c91e43..d51dc37 100644
--- a/tests/qunit/suites/resources/jquery/jquery.makeCollapsible.test.js
+++ b/tests/qunit/suites/resources/jquery/jquery.makeCollapsible.test.js
@@ -1,4 +1,4 @@
-( function ( mw, $ ) {
+( 

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: Remove unused variables from closures

2017-09-19 Thread Fomafix (Code Review)
Fomafix has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/378897 )

Change subject: Remove unused variables from closures
..

Remove unused variables from closures

Change-Id: I5d2e0cd423a4080677f8517bcf3dcee10bf44799
---
M resources/src/mediawiki.special/mediawiki.special.changeslist.visitedstatus.js
M 
resources/src/mediawiki.special/mediawiki.special.preferences.convertmessagebox.js
M resources/src/mediawiki.widgets/MediaSearch/mw.widgets.MediaSearchQueue.js
M resources/src/mediawiki.widgets/mw.widgets.TitleSearchWidget.js
M resources/src/mediawiki/mediawiki.checkboxtoggle.js
M tests/qunit/suites/resources/mediawiki/mediawiki.errorLogger.test.js
6 files changed, 12 insertions(+), 12 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core 
refs/changes/97/378897/1

diff --git 
a/resources/src/mediawiki.special/mediawiki.special.changeslist.visitedstatus.js
 
b/resources/src/mediawiki.special/mediawiki.special.changeslist.visitedstatus.js
index 954e074..6b25327 100644
--- 
a/resources/src/mediawiki.special/mediawiki.special.changeslist.visitedstatus.js
+++ 
b/resources/src/mediawiki.special/mediawiki.special.changeslist.visitedstatus.js
@@ -1,7 +1,7 @@
 /*!
  * JavaScript for Special:Watchlist
  */
-( function ( mw, $ ) {
+( function ( $ ) {
$( function () {
$( '.mw-changeslist-line-watched .mw-title a' ).on( 'click', 
function () {
$( this )
@@ -9,4 +9,4 @@
.removeClass( 'mw-changeslist-line-watched' );
} );
} );
-}( mediaWiki, jQuery ) );
+}( jQuery ) );
diff --git 
a/resources/src/mediawiki.special/mediawiki.special.preferences.convertmessagebox.js
 
b/resources/src/mediawiki.special/mediawiki.special.preferences.convertmessagebox.js
index cee7382..e6b7432 100644
--- 
a/resources/src/mediawiki.special/mediawiki.special.preferences.convertmessagebox.js
+++ 
b/resources/src/mediawiki.special/mediawiki.special.preferences.convertmessagebox.js
@@ -1,9 +1,9 @@
 /*!
  * JavaScript for Special:Preferences: Check for successbox to replace with 
notifications.
  */
-( function ( mw, $ ) {
+( function ( $ ) {
$( function () {
var convertmessagebox = require( 
'mediawiki.notification.convertmessagebox' );
convertmessagebox();
} );
-}( mediaWiki, jQuery ) );
+}( jQuery ) );
diff --git 
a/resources/src/mediawiki.widgets/MediaSearch/mw.widgets.MediaSearchQueue.js 
b/resources/src/mediawiki.widgets/MediaSearch/mw.widgets.MediaSearchQueue.js
index 7ee98bb..3639525 100644
--- a/resources/src/mediawiki.widgets/MediaSearch/mw.widgets.MediaSearchQueue.js
+++ b/resources/src/mediawiki.widgets/MediaSearch/mw.widgets.MediaSearchQueue.js
@@ -4,7 +4,7 @@
  * @copyright 2011-2016 VisualEditor Team and others; see AUTHORS.txt
  * @license The MIT License (MIT); see LICENSE.txt
  */
-( function ( $, mw ) {
+( function ( mw ) {
 
/**
 * MediaWiki media resource queue.
@@ -79,4 +79,4 @@
mw.widgets.MediaSearchQueue.prototype.getSearchQuery = function () {
return this.getParams().gsrsearch;
};
-}( jQuery, mediaWiki ) );
+}( mediaWiki ) );
diff --git a/resources/src/mediawiki.widgets/mw.widgets.TitleSearchWidget.js 
b/resources/src/mediawiki.widgets/mw.widgets.TitleSearchWidget.js
index 5ba9481..cd937f7 100644
--- a/resources/src/mediawiki.widgets/mw.widgets.TitleSearchWidget.js
+++ b/resources/src/mediawiki.widgets/mw.widgets.TitleSearchWidget.js
@@ -4,7 +4,7 @@
  * @copyright 2011-2015 MediaWiki Widgets Team and others; see AUTHORS.txt
  * @license The MIT License (MIT); see LICENSE.txt
  */
-( function ( $, mw ) {
+( function ( mw ) {
 
/**
 * Creates an mw.widgets.TitleSearchWidget object.
@@ -102,4 +102,4 @@
return response.query || {};
};
 
-}( jQuery, mediaWiki ) );
+}( mediaWiki ) );
diff --git a/resources/src/mediawiki/mediawiki.checkboxtoggle.js 
b/resources/src/mediawiki/mediawiki.checkboxtoggle.js
index 36090ea..76bc86c 100644
--- a/resources/src/mediawiki/mediawiki.checkboxtoggle.js
+++ b/resources/src/mediawiki/mediawiki.checkboxtoggle.js
@@ -9,7 +9,7 @@
  * Jeroen De Dauw 
  */
 
-( function ( mw, $ ) {
+( function ( $ ) {
'use strict';
 
$( function () {
@@ -35,4 +35,4 @@
 
} );
 
-}( mediaWiki, jQuery ) );
+}( jQuery ) );
diff --git 
a/tests/qunit/suites/resources/mediawiki/mediawiki.errorLogger.test.js 
b/tests/qunit/suites/resources/mediawiki/mediawiki.errorLogger.test.js
index 46d7837..2a4d991 100644
--- a/tests/qunit/suites/resources/mediawiki/mediawiki.errorLogger.test.js
+++ b/tests/qunit/suites/resources/mediawiki/mediawiki.errorLogger.test.js
@@ -1,4 +1,4 @@
-( function ( $, mw ) {
+( function ( mw ) {
QUnit.module( 'mediawiki.errorLogger', QUnit.newMwEnvironment() );
 
QUnit.test( 'installGlobalHandler', function ( assert ) {