jQuery Grid Plugin - jqGrid - Topic: exception in click event handler of the cell click http://www.trirand.com/blog/?page_id=393/bugs/exception-in-click-event-handler-of-the-cell-click Simple:Press Version 5.7.5.3 OlegK on exception in click event handler of the cell click http://www.trirand.com/blog/?page_id=393/bugs/exception-in-click-event-handler-of-the-cell-click#p25747 Bugs http://www.trirand.com/blog/?page_id=393/bugs/exception-in-click-event-handler-of-the-cell-click#p25747 Hello Tony,

I had problems to create new pool request, because the old one already exists. So all new requests are placed (appended) in the same old pool request (see here). I don't know whether you can apply only one part from the request.

I posted last time many suggestions to jqGrid which could be interesting for you (in my opinion). Just now I posted the answer which I recommend you to read. The fixes which I suggested in the answer on the stackoverflow allows with some restriction use inline editing. I think it could be interesting for many jqGrid users.

Best regards
Oleg

P.S. Your comments to my last suggestion are very intresting for me. What you think about this, this and this bug reports and about this, this, this and this feature requests?

]]> Fri, 20 Jan 2012 18:38:53 +0200 tony on exception in click event handler of the cell click http://www.trirand.com/blog/?page_id=393/bugs/exception-in-click-event-handler-of-the-cell-click#p25745 Bugs http://www.trirand.com/blog/?page_id=393/bugs/exception-in-click-event-handler-of-the-cell-click#p25745 Thanks Oleg,

I do not see it as Pull in GitHUb.

Instead I have apply the fix.

Regards

Tony

]]> Fri, 20 Jan 2012 18:27:35 +0200 OlegK on exception in click event handler of the cell click http://www.trirand.com/blog/?page_id=393/bugs/exception-in-click-event-handler-of-the-cell-click#p25731 Bugs http://www.trirand.com/blog/?page_id=393/bugs/exception-in-click-event-handler-of-the-cell-click#p25731 Hello Tony,

I sent one more pool request. It fixs the bug in the event handler of the click on the grid cell. The problem is that if the call has more complex structure and the onSelectRow callback calls editRow the exception can be fired.

One can reproduce the bug in IE on the demo. Exception will be in the line. If one clicks on the checkbox one will see that during calculation of $.inArray(c, c.parentNode.cells)

c = (!c.is('td') && !c.is('th') ? c.closest("td,th") : c)[0];
if ($.browser.msie) { return $.inArray(c, c.parentNode.cells); }

the c is undefined. The problem is that the line will be called after setSelection which calls onSelectRow, which calls editRow in my demo. The editRow method change the content of the clicked cell. After that the e.target.parentNode of the original click event and even window.event.srcElement.parentNode will be set to null.

As a workaround I suggest to change the order of executing onCellSelect and jqGridCellSelect and calling of onSelectRow callback or editCell.

The same demo which use the fixed version you can test here.

Best regards
Oleg

]]> Fri, 20 Jan 2012 00:53:54 +0200