2

I have just download the latest and greatest sources from the Free-jqGrid GIT.

Just replacing the old version I had yield an error, that after a debug found out to be an issue with the getCell function.

The following:

celValue = $('#'+gridName).jqGrid ('getCell', rowid, 'id');

ending up with celValue ="", instead of the actual value. I was clicking on the second line, the rowid came back as 'mainSession2', the gridName came back as 'mainSession' which is all good, yet the output that is expected is "2", and came back empty.

In a test I did on last Tuesday (If i'm not mistaken), with the sources I download back then, there was no issue.

Thanks,

Tal
  • 391
  • 2
  • 11
  • I made many changes last days. Could you verify that the problem still exist now? If it's exist, please post small demo which can be used to reproduce the problem. If I try just use `getCell` after the grid is created I can't see any problem. – Oleg Jun 05 '15 at 13:25
  • Thanks, I was using a download from 60 minutes ago, so yep, it's the latest. I'm using idPrefix in my grids, so since it takes part in this function, this might be the reason you couldn't re-create? I'll try to upload. – Tal Jun 05 '15 at 13:28
  • http://jsfiddle.net/mrfvsyc2/11/ – Tal Jun 05 '15 at 13:37
  • Above is the uploaded example. look for the line alert('rowid is: "' + rowid + '", Value is "' + celValue + '"'); the celValue is always undefined. – Tal Jun 05 '15 at 13:38

1 Answers1

2

Thank you for the bug report!

I made today the changes to fix problems in IE8, reported in the issue. I used the post as the basis for the changes, but I tested my changes only in IE10 and not in Chrome. The problem is that typeof HTMLElement is "function" in Chrome and "object" in IE.

It's very good that there are people who uses the current sources from GitHub. It helps to fix the problem very quickly. I posted the fix which should solve the problem. The demo which you posted works now. Please refresh the sources which you used.

Community
  • 1
  • 1
Oleg
  • 220,925
  • 34
  • 403
  • 798