jQuery Grid Plugin - jqGridForum

Topic RSS

Found and corrected a bug in saveRow

29/08/2008
12:53

globalisateur

Member

Members

Forum Posts: 9

Member Since:
29/08/2008

Offline

Hello!

I found a bug in inline editing. When I update successively a row via (urledit), the editable cells become blank and the updated value is only displayed when I refresh the griddata. But when I re-edit the same row, the value then re-disappear again.

I found a way to normalize the problem:

In grid.inlinedit.js (3.2.4 version) line 154 and 155 (inside method saveRow), I simply moved this 2 instructions inside the case "select" and the case "checkbox" of the previous switch.

I coudn't erase these instructions completely cause my checkbox column needed it. I suppose the select column needs it too. I haven't any textarea column but I suppose (until I test it later) it will act as its brother text column.

This bug occurred on my 4 browsers: Opera, Firefox, IE7, Safari, all for Windows with the last versions, with a Java/tomcat 5.5 application on a "-//W3C//DTD XHTML 1.0 Strict//EN" html format.

If you find a better way or if my correction brings others (future) problems, please let me know!

Cordially,

Geoffroy

30/08/2008
02:55

tony

Sofia, Bulgaria

Moderator

Members

Moderators

Forum Posts: 7721

Member Since:
30/10/2007

Offline

Hello,

Thank you for this post. I test this again and again and never see such

behavior. Since this problem is posted for first time (I do not exclude

that there is a bug) please do the following:

Before every break in this switch statement alert the value tmp2.

See what is it.

I think that something is wrong in cofiguration.

Another possible problem should be the min file.

Also try to include development version, clear the cache and test.

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.

30/08/2008
09:23

globalisateur

Member

Members

Forum Posts: 9

Member Since:
29/08/2008

Offline

Hi,

Thanks for your quick answer!

The value tmp2 was undefined.

Anyway I understood my problem (all my fault I'm afraid).
I wanted to use setRowData in the saveRow successful function in order to modify some values manually (format the value and modify one depending the value of a checkbox).
But the problem (unknown to me, I thought it was a bug, it wasn't...) is that jqGrid does some job after the return true of the successful saveRow function: The part which contains the moved 2 instructions.

So I couldn't use setRowData but instead refresh manually the whole grid. I would have needed a onSaveRowComplete event (which doesn't exist) to do what I intended to do. Cause the function loadComplete is fired only after the refreshing or loading of the whole grid and no events fires after all process terminated after a saveRow...

Sorry for my misunderstanding of this marvelous tool, jqGrid!!

Cordially,
Geoffroy

31/08/2008
03:02

tony

Sofia, Bulgaria

Moderator

Members

Moderators

Forum Posts: 7721

Member Since:
30/10/2007

Offline

Hello,

there is such callback - the name is aftersavefunc and is parameter in

saveRow. Also check the docs.

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.

All RSS

Forum Timezone: Europe/Sofia

Most Users Ever Online: 715

Currently Online:
36 Guest(s)

Currently Browsing this Page:
1 Guest(s)