[ANN] Struts Plugin V0.53

Somehow still can't make it work. It throws tons of exceptions on me, like:
No message
java.lang.NullPointerException at de.lineas.lit.strutsplugin.util.WebXmlHelper.getActionServlet(WebXmlHelper.java:102)
at de.lineas.lit.strutsplugin.util.WebXmlHelper.getStrutsConfigFilesForModule(WebXmlHelper.java:67)
at de.lineas.lit.strutsplugin.navigation.BaseNavigationAction.handleStrutsConfigNavigation(BaseNavigationAction.java:259)
at de.lineas.lit.strutsplugin.navigation.BaseNavigationAction.actionPerformed(BaseNavigationAction.java:63)

whenever I try to press Ctrl+B in struts-config.xml, etc.
It doesn't seem to recognize my tiles-defs.xml etc.
Here are the excerpts from my project setup:
dd/struts-config.xml :
........
............ .................. dd/tiles-myproject-defs.xml : .......... ......... .............. Web resource directories mappings: ......... .........\dd ->]]> /WEB-INF/
.........

Any ideas on where to look for the fix?

More exceptions:

Exception happened in local inspection tool: Invalid forward resource
java.lang.NullPointerException
at de.lineas.lit.strutsplugin.util.WebXmlHelper.getActionServlet(WebXmlHelper.java:102)
at de.lineas.lit.strutsplugin.util.WebXmlHelper.getStrutsConfigFilesForModule(WebXmlHelper.java:67)
at de.lineas.lit.strutsplugin.inspections.config.action.InvalidForwardResourceInspection$InvalidForwardResourceVisitor.visitXmlAttribute(InvalidForwardResourceInspection.java:110)

Looks like a problem with your ActionServlet definition/mapping in web.xml

Can you please post the relevant part of your web.xml?
Is your web.xml registered as web deployment descriptor in Web Module Settings of your Module?

Cheers,

Yann

Yes, web.xml is registered and packaging/deployment works fine with either Idea itself or ant script. Actually there are two web.xml files in different locations. One is used by Idea and another is auto-generated by xdoclet ant task.
Don't know if this makes any difference with "auto-config" of the plugin. Anyway, the action servlet config is identical in both, here it is:

action org.apache.struts.action.ActionServlet definitions-config /WEB-INF/tiles-myproject-defs.xml definitions-debug 1 definitions-parser-details 0 definitions-parser-validate true application ApplicationResources config /WEB-INF/struts-config.xml debug 2 detail 6 validating true 1 ]]>

and as I already mentioned the folder ".../dd/" with struts-config.xml and tiles-myproject-defs.xml is mapped to /WEB-INF/ in web resource directories mapping.

A couple of questions, again ;)

a) what version of Struts are you using?
b) how is struts.jar attached as library to your web module (module library, .. ) ?
c) where is the other generated web.xml located? Is it mapped in IDEA's Web Module Settings, too? what is its purpose?
d) do all of your config files have the proper DOCTYPE declaration (see the relevant DTDs)?

Tiles won't be supported in your setup. Only tiles-defs.xml-files registered with TilesPlugin (which can be declared in any struts-config.xml) will be scanned; configuring Tiles using ActionServlet is deprecated btw.

Feel free to contact me via mail directly (see plugin manager) to further discuss your problems.

Regards,

Yann

Replied by email :)

Great plugin - we recently moved all of our jsp's under /WEB-INF and this really helped highlight the input forwards that we missed during the migration.

Would be good if the plugin could scan your web.xml and allow servlet mappings as validated forward destinations. We commonly forward onto a servlet that sends out excel reports and this is currently marked red.

Also, would be good if the plugin UI mentioned the fact that the intentions (especially your mandatory success forward rule) are configured elsewhere - I missed this first-time round.

Keep up the good work.
--Richie

Hello Richard,

Would be good if the plugin could scan your web.xml
and allow servlet mappings as validated forward
destinations. We commonly forward onto a servlet that
sends out excel reports and this is currently marked
red.

I have that on my TODO-list already, but low priority.

Also, would be good if the plugin UI mentioned the
fact that the intentions (especially your mandatory
success forward rule) are configured elsewhere - I
missed this first-time round.

I'm currently adding more productivity guide entries to the plugin, so these hidden features will be more obvious.

Keep up the good work.
--Richie

Thanks,
Yann

Yann,

thanks again... this plugin is a great help in following the fairly complicated struts/tiles config our app suffers from. I'll feel a little more confident trying to refactor it down to something more managable now.

dan