fnDeleteRow is broken when using Infinite scrolling and object based data #75

Closed
erikcw opened this Issue Aug 9, 2012 · 2 comments

Projects

None yet

2 participants

@erikcw
erikcw commented Aug 9, 2012

In 19.0-19.2 (haven't tested on 1.9.3 yet since it isn't on the CDN), fnDeleteRow doesn't behave.

It will either ignore the request all together, add duplicates of the row to be removed to the table, or remove the row and add multiple entries of the oLanguage.sEmptyTable label row to the table.


$('#selected_segments').dataTable( {
                    "sDom": "<'row toolbar'<'span6'l><'span6'f>r>t<'row'<'span6'i><'span6'p>>",
                    "sPaginationType": "bootstrap",
                    "bRetrieve": true,
                    #"bScrollInfinite": true,
                    #"bScrollCollapse": true,
                    #"sScrollY": "400px",
                    "oLanguage": {
                        "sLengthMenu": "_MENU_ records per page",
                        "sEmptyTable": "No data segments have been selected.",
                    },
                    "aoColumns": [
                        { "sTitle": "Segment", "mDataProp": (seg) -> seg.segment },
                        { "sTitle": "N" , "mDataProp": (seg) -> seg.number},
                        { "sTitle": "Cost" , "mDataProp": (seg) -> seg.cost},
                        ],
                } )
@DataTables
Owner

Very likely this isn't fixed in 1.9.3 either. Infinite scrolling has effectively been superseded by Scroller ( http://datatables.net/extras/scroller/ ) which is superior in every way but one - you must know the size of the data set. It is likely that infinite scrolling will be removed at some point in future, although a plug-in could be created to match its feature set.

As such, unless the fix for this is trivial, it is likely to be a "known issue" bug I'm afraid.

@DataTables
Owner

Infinite scrolling ( bScrollInfinite ) has been removed in DataTables 1.10 (which has just been merged to master) due to issues like this. Resolving them would add a large amount of code, so infinite scrolling has now been replaced by Scroller.

@DataTables DataTables closed this Dec 12, 2013
@DataTables DataTables added a commit that referenced this issue Dec 2, 2016
@DataTables Fix: Bootstrap 4 flexbox grid
Merge pull request #75 from stroebjo/master
28aa805
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment