jQuery Grid Plugin - jqGrid - Topic: exception after successful delete in complete event handler http://www.trirand.com/blog/?page_id=393/bugs/exception-after-successful-delete-in-complete-event-handler Simple:Press Version 5.7.5.3 kobruleht on exception after successful delete in complete event handler http://www.trirand.com/blog/?page_id=393/bugs/exception-after-successful-delete-in-complete-event-handler#p25524 Bugs http://www.trirand.com/blog/?page_id=393/bugs/exception-after-successful-delete-in-complete-event-handler#p25524 I tried  

http://www.ok-soft-gmbh.com/jq.....-delete.js

using remote json data, row id is passed separately for every row like

{"total":18,"page":1,"records":851,"rows":[{"id":"444444","cell":[null,"444444","0,00000","","0,00","","0,00000","","","20","501","","","",""]},{"id":"_2EPUIT","cell":[null,".PUIT","0,00000","Taara","0,00","Puitpakend","0,00000","","","","","","TAARA","",""]},{"id":"ERWEWERT","cell":[null,"ERWEWERT","0,00000","","0,00","","0,00000","","","20","501","","","",""]},{"id":"_2EMKLAASO","cell":[null,".MKLAASO","0,00000","Taara","0,00","Roheline ja muu klaaspakend","0,00000","","","","501","","TAARA","",""]},{"id":"_2EPABER","cell":[null,".PABER","0,00000","Taara","0,00","Paber ja papppakenddddd","0,00000","","","","111","","TAARA","",""]},{"id":"KROONID","cell":[null,"KROONID","0,00000","","0,00","","0,00000","","","20","501","","","",""]},{"id":"002","cell":[null,"002","0,01667","Toodang","0,02","Kiiver roheline","0,25565","kott","Asel AS","20","501","tere","TOODANG","kott","37"]},{"id":"003","cell":[null,"003","0,02000","Kaup","0

jqGrid passes only first character from row id to Delete controller so it looks like it is not usable.

]]> Fri, 23 Dec 2011 15:10:30 +0200 OlegK on exception after successful delete in complete event handler http://www.trirand.com/blog/?page_id=393/bugs/exception-after-successful-delete-in-complete-event-handler#p25483 Bugs http://www.trirand.com/blog/?page_id=393/bugs/exception-after-successful-delete-in-complete-event-handler#p25483 I think that the origin of the described bug is the line 1542:

postdata = postdata.split(",");

which modifys the postdata. The postdata variable will be used later not only the the line 1577 

toarr = postdata.split(",");

which you suggest to fix, but also in the line 1589

setTimeout(function(){rp_ge[$t.p.id].afterComplete(data,postdata);},500);

as the parameter of the afterComplete callback function. In the version 4.1.2 and earlier the postdata was unchanged. First in the version 4.2.0 (see here) the problem appears. It was in the long change.

So I suggest to replace the lines 1542-1548

postdata = postdata.split(",");
for( var pk in postdata) {
    if(postdata.hasOwnProperty(pk)) {
        postdata[pk] = $.jgrid.stripPref($t.p.idPrefix, postdata[pk]);
    }
}
postd[idname] = postdata.join(); 

to the following

var pdArray = postdata.split(","), j;
for (j=0; j<pdArray.length; j++) {
    pdArray[j] = $.jgrid.stripPref($t.p.idPrefix, postdata[j]);
}
postd[idname] = pdArray.join(); 

Best regards
Oleg

P.S. The modified version of jquery.jqGrid.src.js (for testing of the fix) you can get here.

]]> Tue, 20 Dec 2011 15:45:27 +0200 kobruleht on exception after successful delete in complete event handler http://www.trirand.com/blog/?page_id=393/bugs/exception-after-successful-delete-in-complete-event-handler#p25481 Bugs http://www.trirand.com/blog/?page_id=393/bugs/exception-after-successful-delete-in-complete-event-handler#p25481 I added push request for this issue. One line fix but after editing with visual studio formatting also was changed

https://github.com/kobruleht/jqGrid/commit/3bbb6d8f0285aec0cb45fd7b2e98b6cdccddc645

]]> Tue, 20 Dec 2011 13:23:44 +0200 kobruleht on exception after successful delete in complete event handler http://www.trirand.com/blog/?page_id=393/bugs/exception-after-successful-delete-in-complete-event-handler#p25447 Bugs http://www.trirand.com/blog/?page_id=393/bugs/exception-after-successful-delete-in-complete-event-handler#p25447 If row is deleted using call to server, exception

Object doen't support property or method split

occurs in delete complete handler in jquery.jqGrid.src.js file near line 8273
 

         toarr = postdata.split(",");

 postdata is already array parsed already near line 8239

postdata = postdata.split(",");

and thus it does not contain split method.

Replacing this line with

toarr = postdata;

fixes it.

How about fixing this ?

]]> Sat, 17 Dec 2011 20:58:50 +0200