TheDJ has uploaded a new change for review.
https://gerrit.wikimedia.org/r/152774
Change subject: jquery.tablesorter: Limit the scope of two variables
......................................................................
jquery.tablesorter: Limit the scope of two variables
No functional change, but improves readability of the code
Change-Id: If858d198c9ca0e2feb4930f9f94425262f8afbdc
---
M resources/src/jquery/jquery.tablesorter.js
1 file changed, 3 insertions(+), 2 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core
refs/changes/74/152774/1
diff --git a/resources/src/jquery/jquery.tablesorter.js
b/resources/src/jquery/jquery.tablesorter.js
index 5b1e2a7..b46ab37 100644
--- a/resources/src/jquery/jquery.tablesorter.js
+++ b/resources/src/jquery/jquery.tablesorter.js
@@ -296,8 +296,6 @@
colspan,
headerCount,
longestTR,
- matrixRowIndex,
- matrixColumnIndex,
exploded,
$tableHeaders = $( [] ),
$tableRows = $( 'thead:eq(0) > tr', table );
@@ -309,6 +307,9 @@
// Loop through all the dom cells of the thead
$tableRows.each( function ( rowIndex, row ) {
$.each( row.cells, function ( columnIndex, cell
) {
+ var matrixRowIndex,
+ matrixColumnIndex;
+
rowspan = Number( cell.rowSpan );
colspan = Number( cell.colSpan );
--
To view, visit https://gerrit.wikimedia.org/r/152774
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: If858d198c9ca0e2feb4930f9f94425262f8afbdc
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: TheDJ <hartman.w...@gmail.com>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
