Bug in Pagination
Bug in Pagination
suse
Posts: 1Questions: 0Answers: 0
Hi,
I just detected your Plugin a few days ago and I already love it. Really great work!
But I found a little, little tiny bug in DataTables v1.5.1. where the span-ids for the pagination_buttons are set (type: full_numbers). The id for the "previous-button" is set two times while the one for the "first-button" is missing:
[code]
if ( oSettings.sTableId !== '' )
{
nPaging.setAttribute( 'id', oSettings.sTableId+'_paginate' );
nPrevious.setAttribute( 'id', oSettings.sTableId+'_previous' );
nPrevious.setAttribute( 'id', oSettings.sTableId+'_previous' );
nNext.setAttribute( 'id', oSettings.sTableId+'_next' );
nLast.setAttribute( 'id', oSettings.sTableId+'_last' );
}
[/code]
should be:
[code]
if ( oSettings.sTableId !== '' )
{
nPaging.setAttribute( 'id', oSettings.sTableId+'_paginate' );
nFIRST.setAttribute( 'id', oSettings.sTableId+'_first' );
nPrevious.setAttribute( 'id', oSettings.sTableId+'_previous' );
nNext.setAttribute( 'id', oSettings.sTableId+'_next' );
nLast.setAttribute( 'id', oSettings.sTableId+'_last' );
}
[/code]
Suse
I just detected your Plugin a few days ago and I already love it. Really great work!
But I found a little, little tiny bug in DataTables v1.5.1. where the span-ids for the pagination_buttons are set (type: full_numbers). The id for the "previous-button" is set two times while the one for the "first-button" is missing:
[code]
if ( oSettings.sTableId !== '' )
{
nPaging.setAttribute( 'id', oSettings.sTableId+'_paginate' );
nPrevious.setAttribute( 'id', oSettings.sTableId+'_previous' );
nPrevious.setAttribute( 'id', oSettings.sTableId+'_previous' );
nNext.setAttribute( 'id', oSettings.sTableId+'_next' );
nLast.setAttribute( 'id', oSettings.sTableId+'_last' );
}
[/code]
should be:
[code]
if ( oSettings.sTableId !== '' )
{
nPaging.setAttribute( 'id', oSettings.sTableId+'_paginate' );
nFIRST.setAttribute( 'id', oSettings.sTableId+'_first' );
nPrevious.setAttribute( 'id', oSettings.sTableId+'_previous' );
nNext.setAttribute( 'id', oSettings.sTableId+'_next' );
nLast.setAttribute( 'id', oSettings.sTableId+'_last' );
}
[/code]
Suse
This discussion has been closed.
Replies
Thanks for picking up on that. I've got a fix for this in my development version of DataTables and it will be released in 1.5.2 in the next few days.
Regards,
Allan