Bug in Pagination

Bug in Pagination

susesuse Posts: 1Questions: 0Answers: 0
edited September 2009 in Bug reports
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

Replies

  • allanallan Posts: 63,498Questions: 1Answers: 10,471 Site admin
    Hi Suse,

    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
This discussion has been closed.