[ http://issues.apache.org/struts/browse/SB-44?page=all ]
David H. DeWolf updated SB-44:
------------------------------
Attachment: tiles-context-factory.465124.patch
OK, try this one. . .it's against the head revision (as of right now).
Since I got my karma today, I could also commit it to save you some time.
Either way I'll give you a chance to review first since it's a good chunk of
changes. Let me know. . .
> 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
> Attachments: tiles-context-factory.465124.patch,
> tiles-context-factory.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
