jQuery Grid Plugin - jqGrid - Topic: Possible bug in grid.base.js line 337 http://www.trirand.com/blog/?page_id=393/discussion/possible-bug-in-gridbasejs-line-337 Simple:Press Version 5.7.5.3 tony on Possible bug in grid.base.js line 337 http://www.trirand.com/blog/?page_id=393/discussion/possible-bug-in-gridbasejs-line-337#p13343 Discussion http://www.trirand.com/blog/?page_id=393/discussion/possible-bug-in-gridbasejs-line-337#p13343 Steve,

Merry Christmas!

No there is no error here.

The full code say:

The original gridComplete should obtain the saved one before some operations.

The

if(p.gridComplete = complete) {

complete();

}

in the assigment the value can be either null or function if the value is null - nothing will happen else we restore the original and exectute the event.

Not sure if I explain it good.

Something similar can be see in grid.common.js

function findPos(obj) {
    var curleft = curtop = 0;
    if (obj.offsetParent) {
        do {
            curleft += obj.offsetLeft;
            curtop += obj.offsetTop;
        } while (obj = obj.offsetParent);
        //do not change obj == obj.offsetParent
    }
    return [curleft,curtop];
}

Best Regards

Tony

]]> Sat, 26 Dec 2009 13:17:28 +0200 StevenBlack on Possible bug in grid.base.js line 337 http://www.trirand.com/blog/?page_id=393/discussion/possible-bug-in-gridbasejs-line-337#p13301 Discussion http://www.trirand.com/blog/?page_id=393/discussion/possible-bug-in-gridbasejs-line-337#p13301 Hi Tony!  Happy holidays!

In the latest grid.base.js in GitHub, line 337 -- is it your intention to make an assignment in an if() condition?

337           if (p.gridComplete = complete) {

I'm supposing you mean

337           if (p.gridComplete == complete) {

or

337           if (p.gridComplete === complete) {

**--**  Steve

]]> Fri, 25 Dec 2009 05:03:05 +0200