struts-community-plugins / struts2-jquery

Struts2 jQuery Plugin
Apache License 2.0
82 stars 50 forks source link

4.0.2 jqGrid not rendering tree grids from struts 3* #111

Open gitchuckles opened 7 years ago

gitchuckles commented 7 years ago

none of our tree grids that use the jqGrid that is bundled in struts render correctly with the 4.0.2 version that work with 2.3.30. we either get a normal grid looking grid with no javscript errors or no grid with this stack trace:

VM243:53 Uncaught TypeError: Cannot create property '0' on boolean 'false' at HTMLTableElement.eval (eval at globalEval (jquery-2.2.4.js:343), :53:31) at Function.each (jquery-2.2.4.js:365) at jQuery.fn.init.each (jquery-2.2.4.js:137) at jQuery.fn.init.groupingSetup (eval at globalEval (jquery-2.2.4.js:343), :30:15) at jQuery.fn.init.$.fn.jqGrid (eval at globalEval (jquery-2.2.4.js:343), :1088:13) at populate (eval at globalEval (jquery-2.2.4.js:343), :2344:12) at HTMLTableElement.eval (eval at globalEval (jquery-2.2.4.js:343), :3332:3) at Function.each (jquery-2.2.4.js:365) at jQuery.fn.init.each (jquery-2.2.4.js:137) at jQuery.fn.init.$.fn.jqGrid (eval at globalEval (jquery-2.2.4.js:343), :1090:14)

sdutry commented 7 years ago

@gitchuckles

As mentioned in the README.md file:

Latest version which is compatible to Struts 2.3.x is version [3.7.1].

the version 4.0.2 only works with Struts 2.5.x (and defaults to 2.5.8)

Either downgrade your struts2-jquery version or upgrade your struts version.

sdutry commented 7 years ago

Can you provide a simple example demonstrating this issue so we can check this with the current version and fix if needed?

When does it fail exactly?

What is the exact code used to render it?

What is the structure (and types) of the data being rendered?

gitchuckles commented 7 years ago

Our production stack is struts 2.3.30 and struts-jquery is 3.7.1, my development branch is struts 2.5.8 and struts-jquery 4.0.2

From: Stefaan Dutry notifications@github.com Reply-To: struts-community-plugins/struts2-jquery reply@reply.github.com Date: Friday, January 20, 2017 at 10:59 PM To: struts-community-plugins/struts2-jquery struts2-jquery@noreply.github.com Cc: "McGregor, Chuckles" mcgregor1@llnl.gov, Mention mention@noreply.github.com Subject: Re: [struts-community-plugins/struts2-jquery] 4.0.2 jqGrid not rendering tree grids from struts 3* (#111)

@gitchuckleshttps://github.com/gitchuckles

As mentioned in the README.md file:

Latest version which is compatible to Struts 2.3.x is version [3.7.1].

the version 4.0.2 only works with Struts 2.5.x (and defaults to 2.5.8)

Either downgrade your struts2-jquery version or upgrade your struts version.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/struts-community-plugins/struts2-jquery/issues/111#issuecomment-274242633, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFYYUsE1_LFt068yFn5p2yqIM1wc-5fCks5rUay6gaJpZM4Lp43-.

sdutry commented 7 years ago

@gitchuckles So it's the current version with the current struts version.

Can you provide us with an example of what's not working?

That way we can figure out

Thanks in advance for the extra info.

sdutry commented 7 years ago

@gitchuckles Did you have time yet to gather some more info so we can reproduce this issue?

Can you provide us with an example of what's not working?

  • data structure that gives the exception
  • exact tags used to render the grid

That way we can figure out

  • what type of data allows to reproduce the exception
  • what attributes need to be set to reproduce the exception

Thanks in advance for the extra info.

gitchuckles commented 7 years ago

Not yet, I’ve been diverted to a high priority aspect of the project, hope to get back to it soon

From: Stefaan Dutry notifications@github.com Reply-To: struts-community-plugins/struts2-jquery reply@reply.github.com Date: Monday, January 30, 2017 at 2:24 PM To: struts-community-plugins/struts2-jquery struts2-jquery@noreply.github.com Cc: "McGregor, Chuckles" mcgregor1@llnl.gov, Mention mention@noreply.github.com Subject: Re: [struts-community-plugins/struts2-jquery] 4.0.2 jqGrid not rendering tree grids from struts 3* (#111)

@gitchuckleshttps://github.com/gitchuckles Did you have time yet to gather some more info so we can reproduce this issue?

Can you provide us with an example of what's not working?

That way we can figure out

Thanks in advance for the extra info.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/struts-community-plugins/struts2-jquery/issues/111#issuecomment-276211498, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFYYUqPBa_2PhXSlR36S4LsQyxWHUJc6ks5rXmMpgaJpZM4Lp43-.

sdutry commented 7 years ago

@gitchuckles Did you have time yet to gather some additional info concerning this issue?

gitchuckles commented 7 years ago

Not yet, we just finished a release and supposed work out what goes into future releases, I would like to get some time to work on this so we can work the latest struts/jquery into production in the future

From: Stefaan Dutry notifications@github.com Reply-To: struts-community-plugins/struts2-jquery reply@reply.github.com Date: Tuesday, February 21, 2017 at 3:48 AM To: struts-community-plugins/struts2-jquery struts2-jquery@noreply.github.com Cc: "McGregor, Chuckles" mcgregor1@llnl.gov, Mention mention@noreply.github.com Subject: Re: [struts-community-plugins/struts2-jquery] 4.0.2 jqGrid not rendering tree grids from struts 3* (#111)

@gitchuckleshttps://github.com/gitchuckles Did you have time yet to gather some additional info concerning this issue?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/struts-community-plugins/struts2-jquery/issues/111#issuecomment-281321259, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFYYUoln3wDMJ194cfgVLU58MeoXXODdks5res8hgaJpZM4Lp43-.

unciax commented 5 years ago

Is there any way to solve it? Have same problem with 4.0.3 + struts 2.5.18.

unciax commented 5 years ago

I think I found a solution to fix it. Replace jquery.jqGrid.js with https://github.com/free-jqgrid/jqGrid/blob/v4.7.0/js/jquery.jqGrid.js and add require(ui.multiselect.js) to jquery.grid.struts2.js