struts-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Yannick Haudry (JIRA)" <j...@apache.org>
Subject [jira] Commented: (STR-3160) TilesRequestProcessor processTilesDefinition returns false when a definition has been processed
Date Tue, 05 Aug 2008 12:37:29 GMT

    [ https://issues.apache.org/struts/browse/STR-3160?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=44530#action_44530
] 

Yannick Haudry commented on STR-3160:
-------------------------------------

no problem, everything's fine; I actually felt a little bit embarrassed not having even thought
about it ! At least I found a bug, I agree a very little one ;-)
Anyway that is, for sure, a pretty good hint, I'll take a look at the CoR implementation...


> TilesRequestProcessor processTilesDefinition returns false when a definition has been
processed
> -----------------------------------------------------------------------------------------------
>
>                 Key: STR-3160
>                 URL: https://issues.apache.org/struts/browse/STR-3160
>             Project: Struts 1
>          Issue Type: Bug
>          Components: Tiles 2 Plugin
>    Affects Versions: 1.4.0
>            Reporter: Yannick Haudry
>            Assignee: Antonio Petrelli
>         Attachments: TilesRequestProcessor.patch
>
>
> Here is the code involved in TilesRequestProcessor.processTilesDefinition: 
>         if (container.isValidDefinition(definitionName, new Object[] { request,
>                 response })) {
>             retValue = response.isCommitted();
>             try {
>                 container.render(definitionName, new Object[] { request,
>                         response });
>             } catch (TilesException e) {
>                 throw new ServletException("Cannot render definition '"
>                         + definitionName + "'");
>             }
>         } else {
>             // ignore not found
>             if (log.isDebugEnabled()) {
>                 log.debug("Cannot find definition '" + definitionName + "'");
>             }
>         }
> the response.isCommitted() statement seems to return 'false' but the retValue should
be set to 'true'
> as a definition was found at this point..

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message