Thanks. I will do more for this in the next release not in 3.4.2

Regards

Tony

P.S. I realized that there is a more elegant solution, but it should be tested.

What's your oppinion on:

nmid = nm.match(/(\\w*)/i);
nmid = nmid.join('');

Cheers

Jean

Thanks for this. Yes this is true, but the same thing should be added in inline  and cell editing modules. The same apply when we have name.someothername - also a good approach is to use common regExp for this purpose. Will see what I can do for this purpose.

Thank you

Regards

Tony

I've jquery r1.3.2 and jqgrid 3.4.1 installed and when I edit the data with the modal window it pops up one time - then if I want to edit another row I get the following exception in firefox 3.0.6 (osx):

[Exception... "'Syntax error, unrecognized expression: ]' when calling method: [nsIDOMEventListener::handleEvent]"  nsresult: "0x8057001e (NS_ERROR_XPC_JS_THREW_STRING)"  location: "<unknown>"  data: no]

I realised it was due to I used a multiselect in jqgrid and the name of the field was name[] - due to a php backend and this was the source of the error. Below is a "nasty" diff to get rid of the square braktes [] for the id on the input in the modal window without altering the name attribute. It's a cimple regex to remove the square brakets.

==========

--- grid.formedit-old.js        2009-02-20 23:04:17.000000000 +0100
+++ grid.formedit.js    2009-02-20 23:05:14.000000000 +0100
@@ -425,6 +425,8 @@
                                var nm, hc,trdata, tdl, tde, cnt=0,tmp, dc,elc, retpos=[];
                                $('#'+rowid+' td',obj.grid.bDiv).each( function(i) {
                                        nm = obj.p.colModel[i].name;
+                                       nmid = nm.match(/(\\w*)/i);
+                                       nmid = nmid[0];
                                        // hidden fields are included in the form
                                        if(obj.p.colModel[i].editrules && obj.p.colModel[i].editrules.edithidden == true) {
                                                hc = false;
@@ -442,7 +444,7 @@
                                                                tmp = $.htmlDecode($(this).html());
                                                        }
                                                }
-                                               var opt = $.extend(obj.p.colModel[i].editoptions || {} ,{id:nm,name:nm});
+                                               var opt = $.extend(obj.p.colModel[i].editoptions || {} ,{id:nmid,name:nm});
                                                if(!obj.p.colModel[i].edittype) obj.p.colModel[i].edittype = "text";
                                                elc = createEl(obj.p.colModel[i].edittype,opt,tmp);
                                                $(elc).addClass("FormElement");
@@ -469,7 +471,8 @@
                        function fillData(rowid,obj){
                                var nm, hc,cnt=0,tmp;
                                $('#'+rowid+' td',obj.grid.bDiv).each( function(i) {
-                                       nm = obj.p.colModel[i].name;
+                                       nm = obj.p.colModel[i].name.match(/(\\w*)/i);
+                                       nm = nm[0];
                                        // hidden fields are included in the form
                                        if(obj.p.colModel[i].editrules && obj.p.colModel[i].editrules.edithidden === true) {
                                                hc = false;

========

Furthermore I had a scrollbar in the modalwindow and I got rid of it by changing grid.css to:

.modalcontent {
    margin-bottom: 9px;
    margin-left: 5px;
    width: 97%;
}

Take care, jean