Forum

November 2nd, 2014
A A A
Avatar

Lost password?
Advanced Search

— Forum Scope —




— Match —





— Forum Options —





Minimum search word length is 3 characters - maximum search word length is 84 characters

The forums are currently locked and only available for read only access
sp_Feed Topic RSS sp_TopicIcon
Possible bug in grid.base.js line 337
25/12/2009
05:03
Avatar
StevenBlack
Member
Members
Forum Posts: 9
Member Since:
06/02/2009
sp_UserOfflineSmall Offline

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

26/12/2009
13:17
Avatar
tony
Sofia, Bulgaria
Moderator
Members

Moderators
Forum Posts: 7721
Member Since:
30/10/2007
sp_UserOfflineSmall Offline

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

For professional UI suites for Java Script and PHP visit us at our commercial products site - guriddo.net - by the very same guys that created jqGrid.

Forum Timezone: Europe/Sofia

Most Users Ever Online: 715

Currently Online:
54 Guest(s)

Currently Browsing this Page:
1 Guest(s)

Top Posters:

OlegK: 1255

markw65: 179

kobruleht: 144

phicarre: 132

YamilBracho: 124

Renso: 118

Member Stats:

Guest Posters: 447

Members: 11373

Moderators: 2

Admins: 1

Forum Stats:

Groups: 1

Forums: 8

Topics: 10592

Posts: 31289

Newest Members:

, razia, Prankie, psky, praveen neelam, greg.valainis@pa-tech.com

Moderators: tony: 7721, Rumen[Trirand]: 81

Administrators: admin: 66

Comments are closed.
Privacy Policy   Terms and Conditions   Contact Information