Guriddo » Topic: l.p.colModel[x] is undefined with html data

l.p.colModel[x] is undefined with html data

Guriddo
l.p.colModel[x] is undefined with html data

This topic has 5 replies, 4 voices, and was last updated 14 years, 1 month ago by Shift4SMS.

Creator

Topic
December 24, 2011 at 2:28 am #79445

Shift4SMS
Participant

I using the latest jqGrid (4.2.0) and the latest ckeditor (3.6.2). Via various thread around I have the two working together on modal edit dialogs (hopefully that makes sense).

Everything works great until a data cell which hold html data (to be edited by ckeditor) contains a table tag. The may be an issue with other tags but so far I have only experienced a problem with tables (and I have used this to edit all sorts of documents). Once a cell contains a table, the following throws an exception:
Creator

Topic

Viewing 5 replies - 1 through 5 (of 5 total)

Author

Replies
December 24, 2011 at 10:44 am #99278

OlegK
Participant

You should use really the latest version of jqGrid which are 4.3.1 now. Moreover you should use jquery.jqGrid.src.js instead of jquery.jqGrid.min.js if you post an error. You should also always write which web browser you used in your tests. The best would be if you post a small demo which can be used to reproduce the problem. Could you post additional information?

Best regards
Oleg

April 18, 2012 at 1:17 pm #99731

Umer Khalid Butt
Participant

@Shift4SMS

I was facing the same issue. After debugging the issue I figured out that its a Bug in JQGrid (i am using latest 4.3.1). Whenever you define a table in any column (using column formatter) and try to retrieve the value of that column later (given that you have also defined your unformatter functions properly) you will face this issue.

Its fix is as follows:

Open the source file jquery.jqGrid.src.js and replace the following line

$('td',ind).each( function(i) {

with this line

$('td[role=”gridcell”]',ind).each( function(i) {

and you are Done ! 🙂

jquery.jqGrid.src.js

April 18, 2012 at 4:20 pm #99732

tony
Keymaster

Hello,

This is correct. Thanks – fixed.

Regards

April 18, 2012 at 6:10 pm #99733

Shift4SMS
Participant

umerkhalidbutt said:

$('td[role=”gridcell”]',ind).each( function(i) {

jquery.jqGrid.src.js

Very nice. Thank you. That saves me a lot of time trying to get a sample app that demonstrates the issue and it's much more precise than my try/except work around.

April 18, 2012 at 6:31 pm #99735

Shift4SMS
Participant

BTW, tested and worked like a charm! Thanks again.
Author

Replies