[ http://issues.apache.org/struts/browse/SB-44?page=all ]
David H. DeWolf updated SB-44:
------------------------------
Attachment: tiles-context-factory.465236.patch
GAK! Try this one. It applies against a clean revision for me. If you have
any problems, it'll be with the TilesContextFactory as it is now an interface
and the changes are pretty significant. As an interface, it's fairly simple to
manually patch.
> TilesContextFactory should be a regular class
> ---------------------------------------------
>
> Key: SB-44
> URL: http://issues.apache.org/struts/browse/SB-44
> Project: Sandbox
> Issue Type: Improvement
> Components: Tiles
> Affects Versions: 2.0
> Environment: None
> Reporter: Antonio Petrelli
> Assigned To: David H. DeWolf
> Attachments: tiles-context-factory.465236.patch
>
>
> The org.apache.tiles.context.TilesContextFactory class should be a regular
> class, not a class with only static methods.
> This is because this object could eventually be replaced with another one
> specified by the developer that uses Tiles, i.e. enhancing configuration.
> Moreover, a class with only static methods should be only an "utility class",
> thing that TilesContextFactory is not.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/struts/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
