stephenswilliams / jquery-datatables-editable

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

Adding data using a form doing include checkbox information #64

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
If you use a form to create a new row in the table, the form may include 
checkboxes, but the current selector doesn't pick this up: 
http://code.google.com/p/jquery-datatables-editable/source/browse/trunk/media/js
/jquery.dataTables.editable.js#368 . It just needs to have the input:checkbox 
type added.

I wonder if it might be an idea to use $.serialize ( 
http://api.jquery.com/serialize/ ) or similar to pick up all of the form 
information, incase there are any other input types which are missing from the 
list?

Allan

Original issue reported on code.google.com by allan.ja...@gmail.com on 20 Sep 2011 at 6:44

GoogleCodeExporter commented 9 years ago
This issue was closed by revision r210.

Original comment by joc...@gmail.com on 21 Sep 2011 at 11:47

GoogleCodeExporter commented 9 years ago
Hi Allan,

You are right checkbox is not included - I have added your fix in the code. You 
can try it on the 
http://jquery-datatables-editable.googlecode.com/svn/trunk/issue64.html.
Note that serialize is already used in the line 
http://code.google.com/p/jquery-datatables-editable/source/browse/trunk/media/js
/jquery.dataTables.editable.js#342.
Plugin works fine when it sends checkbox data to the server-side however in the 
code that copies form data into the row this element was not included.

Thanks,
Jovan

Original comment by joc...@gmail.com on 21 Sep 2011 at 11:54