jQuery Grid Plugin - jqGrid

Hi - I'm having the same problem in celledit mode - would greately appreciate a solution.

Thanks,

Jo

Not yet ready to test any of this, sorry, but I've looked over the changes.

Not much to say, except that I think its doing more or less what /I/ will want (once I get there!).

A couple of observations:

In grid.celledit.js, lines 119 and following:

				nm = $t.p.colModel[iCol].name.replace('.',"\\\\.");
				switch ($t.p.colModel[iCol].edittype) {
					case "select":
						if(!$t.p.colModel[iCol].editoptions.multiple) {
							v = $("#"+iRow+"_"+nm.replace('.',"\\\\.")+">option:selected",$t.rows[iRow]).val();
							v2 = $("#"+iRow+"_"+nm.replace('.',"\\\\.")+">option:selected",$t.rows[iRow]).text();
						} else {
							var sel = $("#"+iRow+"_"+nm.replace('.',"\\\\."),$t.rows[iRow]), selectedText = [];
							v = $(sel).val();

Im not sure - but those three nm.replace('.',"\\\\.") look wrong, given the definition of nm...

Also, this looks wrong:
if($t.p.colModel[iCol].formatter && $t.p.colModel[iCol] == 'select') v2 = v;

Note that $t.p.colModel[iCol] is an object, so its not likely to compare == 'select'.

Not sure if you meant to check edittype (which would be redundant, you've already switched on it) or formatter (which I think would be too restrictive - what about custom formatters?).

I think the correct check is just that it *has* a formatter (ie drop the &&...).

Mark

Hi,

Forgive my ignorance but I've never used GitHub and I see that you've made the changes in the individual

modules and I'm not sure how to concatenate them into my one grid file. 

(with 3.5 I understand all modules are included in the one file when downloaded from the new download manager)

Thanks,

Jo

Can you please give an estimate as to when that would be? Just want to make sure it will fit in with my timelines.

Regards,

Jo