You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@struts.apache.org by "Paul Benedict (JIRA)" <ji...@apache.org> on 2007/11/15 07:13:33 UTC
[jira] Closed: (STR-2239) [struts] ModuleUtils/TagUtils:
setInstance() missing
[ https://issues.apache.org/struts/browse/STR-2239?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Paul Benedict closed STR-2239.
------------------------------
Resolution: Won't Fix
Assignee: (was: Struts Developers)
It is generally a bad idea to create instance methods for static factory classes. What you really want is an enhancement in the Controller, not in utility classes.
> [struts] ModuleUtils/TagUtils: setInstance() missing
> ----------------------------------------------------
>
> Key: STR-2239
> URL: https://issues.apache.org/struts/browse/STR-2239
> Project: Struts 1
> Issue Type: Improvement
> Components: Core
> Affects Versions: 1.2.4
> Environment: Operating System: other
> Platform: Other
> Reporter: Lars Beuster
> Priority: Minor
>
> Why is there no setInstance() method on both classes and why is the
> instance-member final. If I cannot change the behavior of both classes then
> there's not need to use the singleton and you could have left the static-methods.
> I have a different mapping in my web.xml (/servletname/* -> ActionServlet). But
> this mapping isn't supported by Struts if I have more than one Module, because
> ModuleUtils.getModuleName()-method cannot handle this.
> Thanks Lars
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.