ansaurus

Question

Answer 1

A:

Your code seems to me mostly OK. The only potential problem which you can have with IDs is the possible conflict with IDs from /Content/xml/user.xml. You should be careful and not include integer IDs grater as 3 in the /Content/xml/user.xml.

Some more remarks to you code. I find personally not good to use variable with the name lastsel2 in the context which has nothing with selection. The usage of variable with the name i in the scope of jQuery(document).ready (so the variable will be almost global because it will be seen everywhere) not very nice. A later modification of your code could occasionally reset the value of i.

Moreover I recommend you always verify you HTML code in a HTML Validator like http://validator.w3.org and verify your JavaScript code in JSLint.

JSLint will help you to find some problems in your code like

newDom.loadXML('<?xml version="1.0"?><rows><page>1</page><total>1</total><records>'+
               + entries.length + '</records></rows>');

('+' will be used twice: on the end of one line and at the beginning of the first one) or unused variables like groupingView inside of gridComplete functions. JSLint could help you to make your code more robust.

HTML Validator show you other errors in your XHTML code which could follow that in other browser version as you used for testing your Web application could work incorrect:

1) runat in unknown attribute for the <head> element in "-//W3C//DTD XHTML 1.0 Transitional//EN". So you should replace <head runat="server"> to <head>

2) To have no XHTML validation errors you should place the JavaScript code inside of //<![CDATA[ //]]> block:

<script type="text/javascript">
//<![CDATA[
  /* you can place your code here */
//]]>
</script>

3) Replace <input type="BUTTON" ...> to <input type="button" ...> because XHTML is case sensitive.

4) Add mandatory attribute action="" to your <form> to follow "-//W3C//DTD XHTML 1.0 Transitional//EN". You can also replace <form> to <fieldset> if you not really want to use a form submitting.

5) Include <meta http-equiv="Content-Type" content="text/html; charset=utf-8" /> inside of the <head> element.

6) Include <tr><td /></tr> inside of <table> to follow "-//W3C//DTD XHTML 1.0 Transitional//EN"

Oleg 2010-09-24 10:46:20

Thanks Oleg for your valuable comments..Thank you very much...

Shivi 2010-09-24 11:04:29

Answer 2

A:

I have solved the problem using..

i count all the row in grid and then add 1 to the count..

onClickButton: function() {
    var last = jQuery('#test').jqGrid('getGridParam','records'); 
    AddNewRow (last);
}

var AddNewRow = function(id){

    var datarow = {Consultant:"",Role:"",Task:"",SDate:"",EDate:"",
                   Deliverables:"",Complete:""};

    var lastsel2 = parseInt(id) + 1;

    var su=jQuery("#test").addRowData(lastsel2, datarow, "last") ;

};

Shivi 2010-09-24 11:07:12

It can work or not work depends on the values of `id` attributes inside of `/Content/xml/user.xml` and not depend of the count of rows in the table. Moreover if you allow user to remove a row then this algorithm could follow to double IDs. If you use integers as the IDs you could get all current IDs with respect of `getDataIDs`, find the maximum value and then use the max value + 1 for the new ID.

Oleg 2010-09-24 11:42:31

How can i use that getDataIDs property ?? Please Help.

Shivi 2010-09-24 11:49:44

Ok..thanks i got it...thanks Oleg..

Shivi 2010-09-24 11:51:50

ansaurus

tags:

views:

answers:

assign Unique ID Jgrid Row

related questions