History | Log In     View a printable version of the current page.  
Issue Details (XML | Word | Printable)

Key: SM-35
Type: Improvement Improvement
Status: Open Open
Priority: Major Major
Assignee: Matt Raible
Reporter: Matt Raible
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Struts Menu

multiple menu's do not work with multiple struts-config module

Created: 09/May/06 02:09 AM   Updated: 17/Oct/06 01:27 AM
Component/s: Menu Core
Affects Version/s: 2.3
Fix Version/s: 2.5


 Description  « Hide
Moved from SourceForge http://sourceforge.net/tracker/index.php?func=detail&aid=1182078&group_id=48726&atid=453974:

The struts-menu tags do not work in an application
where multiple menus and struts-config files are being
used. Each time the MenuPlugin is defined in the
struts-config file, it creates a new MenuRepository
which is stored in the ServletContext. Since a generic
key is used, previously added repositories are
overwritten. This can be resolved by making the
stuts-config module name part of the key, see the
attached files.

Download patch from http://sourceforge.net/tracker/download.php?group_id=48726&atid=453974&file_id=129677&aid=1182078.

 All   Comments   Change History   FishEye      Sort Order:
Matt Raible - 17/Oct/06 01:26 AM
This patch looks like it may have worked when it was first created. However, Struts Menu no longer depends on Struts, so I'm willing to bet this code will no longer work (ClassNotFoundException, etc). Also, the destroy() method in MenuPlugin doesn't seem to remove the property key + moduleName from the servletContext.