[
https://issues.apache.org/jira/browse/WW-4594?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15123751#comment-15123751
]
ASF GitHub Bot commented on WW-4594:
------------------------------------
Github user lukaszlenart commented on a diff in the pull request:
https://github.com/apache/struts/pull/85#discussion_r51284943
--- Diff:
plugins/tiles/src/main/java/org/apache/struts2/views/tiles/TilesResult.java ---
@@ -99,6 +109,18 @@ public TilesResult(String location) {
* HTTP request.
*/
public void doExecute(String location, ActionInvocation invocation)
throws Exception {
+ StrutsTilesAnnotationProcessor annotationProcessor = new
StrutsTilesAnnotationProcessor();
+ TilesDefinition tilesDefinition = null;
+ Object action = invocation.getAction();
+ String actionName = invocation.getInvocationContext().getName();
+
+ if (StringUtils.isEmpty(location)) {
+ // location not set -> action must have one @TilesDefinition
--- End diff --
Could you turn this comment into a `LOG.trace`?
> Configure TilesDefs by annotating Actions
> -----------------------------------------
>
> Key: WW-4594
> URL: https://issues.apache.org/jira/browse/WW-4594
> Project: Struts 2
> Issue Type: New Feature
> Components: Plugin - Tiles
> Reporter: Christoph Nenning
> Assignee: Christoph Nenning
> Fix For: 2.5
>
>
> Large applications often consist of hundrets of actions. IMO it is a major
> benefit of struts-convention-plugin that it is not necessary to configure all
> those actions in xml files. As xml files would become huge and hard to
> maintain. The same applies to tiles xml files. Currently it is necessary to
> configure tiles-definitions for all those actions in xml files which makes
> them huge and hard to maintain.
> It would be great if configuration of tiles-definitions could be done by
> annotating actions, in a similar way actions can be configured with
> annotations from convention-plugin.
> This somewhat relates to WW-3937 and WW-4161
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
