jquery.tablesorter: Fix broken getElementSortKey()
authorBrad Jorsch <bjorsch@wikimedia.org>
Fri, 8 Mar 2013 02:08:16 +0000 (21:08 -0500)
committerGerrit Code Review <gerrit@wikimedia.org>
Mon, 18 Mar 2013 03:45:10 +0000 (03:45 +0000)
Change I3e9898f3 introduced changes to
getElementText()/getElementSortKey() that broke certain behavior with
respect to tables having colspanned cells. Restore the old behavior for
these cases.

Change-Id: I18be2e6c1f0ab926ff4c795530600b52bc6488c0

resources/jquery/jquery.tablesorter.js

index e252ba5..8bf1f61 100644 (file)
@@ -98,7 +98,9 @@
                        // like charAt, toLowerCase and split are expected.
                        return String( data );
                } else {
-                       if ( node.tagName.toLowerCase() === 'img' ) {
+                       if ( !node ) {
+                               return $node.text();
+                       } else if ( node.tagName.toLowerCase() === 'img' ) {
                                return $node.attr( 'alt' ) || ''; // handle undefined alt
                        } else {
                                return $.map( $.makeArray( node.childNodes ), function( elem ) {