Siddiqui101 / jquery-datatables-editable

Automatically exported from code.google.com/p/jquery-datatables-editable
0 stars 0 forks source link

When using Ajax Source, jquery-datatables-editable does not honor DT_RowId or leave existing id in place #52

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. use sAjaxSource in datatables
2. use DT_RowId method to pass table row IDs 
(http://www.datatables.net/release-datatables/examples/server_side/ids.html)
3. Have the first column not be an ID column (actual data) (or blank for 
placeholder)

What is the expected output? What do you see instead?

jquery-datatables-editable removes the table row ID that is already there and 
replaces it with whatever it happens to find in the first column

What version of the product are you using? On what operating system?

1.3 jquery-datatables-editable 1.8.1 datatables

Please provide any additional information below.

Perhaps if DT_RowID is set, or if the TR already has a value, do not overwrite 
it (under any circumstances, not even if the id would make sense (not blank, in 
this case))

I fixed it temporarily by commenting out the last line here (line 503)

                            var position = oTable.fnGetPosition(this);

                            var id = oTable.fnGetData(position)[0];

//                            properties.fnSetRowID($(this), id);

Original issue reported on code.google.com by aaron.we...@gmail.com on 5 Aug 2011 at 5:15

GoogleCodeExporter commented 9 years ago
Could you try with the latest verison 1.3.4?

Thanks,
Jovan

Original comment by joc...@gmail.com on 15 Sep 2011 at 9:34

GoogleCodeExporter commented 9 years ago
See example on the 
http://jquery-datatables-editable.googlecode.com/svn/trunk/ajax_object_source.ht
ml and 
http://jquery-datatables-editable.googlecode.com/svn/trunk/ajax_mDataProp.html

Original comment by joc...@gmail.com on 15 Sep 2011 at 10:19

GoogleCodeExporter commented 9 years ago

Original comment by joc...@gmail.com on 22 Sep 2011 at 8:43