jQuery Grid Plugin - jqGridForum

Topic RSS

Bug in editGridRow

29/09/2008
03:34

rebus

Palermo (Italy)

New Member

Members

Forum Posts: 2

Member Since:
26/09/2008

Offline

Hi,

First of all thank, jqGrid is wonderful 🙂

I found a possible bug.

The follow code in grid.formedit.js:

291) if(!ret[2]) ret[2] = parseInt($($t).getGridParam('records'))+1;

292) postdata.id = ret[2];

it can cause dulicate row id.

Thanks in advance

Thanks.

29/09/2008
09:17

tony

Sofia, Bulgaria

Moderator

Members

Moderators

Forum Posts: 7721

Member Since:
30/10/2007

Offline

Hello,

Yes this is true, but we actually do not know what to set if this ret[2] is

not set. If you have better idea your are welcome.

Thank you

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.

01/10/2008
07:56

rebus

Palermo (Italy)

New Member

Members

Forum Posts: 2

Member Since:
26/09/2008

Offline

Hi Tony,

For solve this problem I have add the follow method in grid.base.js:

$.fn.sortId = function (id1,id2) {
return id1 - id2;
};

$.fn.calculateRowId = function() {
    var rowid = 1;
    var count = parseInt(this.getGridParam('records'));
    if (count > 0) {
        var ids = this.getDataIDs();
        ids = ids.sort(this.sortId);
        for (i = 0; i < ids.length; i++) {
            if (rowid < ids[i])
                break;
            else
                rowid++;
        }
    }
    return rowid;
};

In grid.formedit.js I have replace the follow row:

291) if(!ret[2]) ret[2] = parseInt($($t).getGridParam('records'))+1;

with

291) if(!ret[2]) ret[2] = parseInt($($t).calculateRowId());

Best Regards

Alessandro

All RSS

Forum Timezone: Europe/Sofia

Most Users Ever Online: 715

Currently Online:
53 Guest(s)

Currently Browsing this Page:
1 Guest(s)