rooty/DataTables
1
0
Fork 0
mirror of https://github.com/DataTables/DataTables.git synced 2025-02-07 05:54:15 +01:00
Code Issues Releases Activity

Merge pull request #89 from timtucker/patch-6

Browse Source 
Use hasChildNodes() rather than childNodes.length
This commit is contained in:
Allan Jardine 2012-09-02 01:40:10 -07:00
commit 9c82abe79d
1 changed files with 16 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
media/src/ext/ext.paging.js
View File

@ -1,4 +1,3 @@
/* /*
* Variable: oPagination * Variable: oPagination
* Purpose: * Purpose:
@ -73,17 +72,22 @@ $.extend( DataTable.ext.oPagination, {
var oClasses = oSettings.oClasses; var oClasses = oSettings.oClasses;
var an = oSettings.aanFeatures.p; var an = oSettings.aanFeatures.p;
var nNode;
/* Loop over each instance of the pager */ /* Loop over each instance of the pager */
for ( var i=0, iLen=an.length ; i<iLen ; i++ ) for ( var i=0, iLen=an.length ; i<iLen ; i++ )
{ {
if ( an[i].childNodes.length !== 0 ) nNode = an[i].firstChild;
if ( nNode )
{ {
an[i].childNodes[0].className = ( oSettings._iDisplayStart === 0 ) ? /* Previous page */
oClasses.sPagePrevDisabled : oClasses.sPagePrevEnabled; nNode.className = ( oSettings._iDisplayStart === 0 ) ?
oClasses.sPagePrevDisabled : oClasses.sPagePrevEnabled;
an[i].childNodes[1].className = ( oSettings.fnDisplayEnd() == oSettings.fnRecordsDisplay() ) ?
oClasses.sPageNextDisabled : oClasses.sPageNextEnabled; /* Next page */
nNode = nNode.nextSibling;
nNode.className = ( oSettings.fnDisplayEnd() == oSettings.fnRecordsDisplay() ) ?
oClasses.sPageNextDisabled : oClasses.sPageNextEnabled;
} }
} }
} }
@ -172,7 +176,7 @@ $.extend( DataTable.ext.oPagination, {
var sList = ""; var sList = "";
var iStartButton, iEndButton, i, iLen; var iStartButton, iEndButton, i, iLen;
var oClasses = oSettings.oClasses; var oClasses = oSettings.oClasses;
var anButtons, anStatic, nPaginateList; var anButtons, anStatic, nPaginateList, nNode;
var an = oSettings.aanFeatures.p; var an = oSettings.aanFeatures.p;
var fnBind = function (j) { var fnBind = function (j) {
oSettings.oApi._fnBindAction( this, {"page": j+iStartButton-1}, function(e) { oSettings.oApi._fnBindAction( this, {"page": j+iStartButton-1}, function(e) {
@ -223,18 +227,19 @@ $.extend( DataTable.ext.oPagination, {
/* Loop over each instance of the pager */ /* Loop over each instance of the pager */
for ( i=0, iLen=an.length ; i<iLen ; i++ ) for ( i=0, iLen=an.length ; i<iLen ; i++ )
{ {
if ( an[i].childNodes.length === 0 ) nNode = an[i];
if ( !nNode.hasChildNodes() )
{ {
continue; continue;
} }
/* Build up the dynamic list first - html and listeners */ /* Build up the dynamic list first - html and listeners */
$('span:eq(0)', an[i]) $('span:eq(0)', nNode)
.html( sList ) .html( sList )
.children('a').each( fnBind ); .children('a').each( fnBind );
/* Update the permanent button's classes */ /* Update the permanent button's classes */
anButtons = an[i].getElementsByTagName('a'); anButtons = nNode.getElementsByTagName('a');
anStatic = [ anStatic = [
anButtons[0], anButtons[1], anButtons[0], anButtons[1],
anButtons[anButtons.length-2], anButtons[anButtons.length-1] anButtons[anButtons.length-2], anButtons[anButtons.length-1]