Tiles resource chain-config.xml missing RemovedCachedMessages and SetOriginalURI

• Key: STR-2935
• Type: Bug
• Status: Closed
• Resolution: Fixed
• Priority: Minor
• Assignee: Niall Pemberton
• Reporter: Scott Van Wart
• Component/s: Tiles 1 Plugin
• Affects Version/s: 1.3.5
• Fix Version/s: 1.3.6

Description

diff -Naur struts-1.3.5.orig/src/tiles/src/main/resources/org/apache/struts/tiles/chain-config.xml struts-1.3.5/src/tiles/src/main/resources/org/apache/struts/tiles/chain-config.xml
--- struts-1.3.5.orig/src/tiles/src/main/resources/org/apache/struts/tiles/chain-config.xml 2006-08-05 16:21:34.000000000 -0300
+++ struts-1.3.5/src/tiles/src/main/resources/org/apache/struts/tiles/chain-config.xml 2006-08-25 13:35:57.000000000 -0300
@@ -87,6 +87,8 @@
                                    point in the chain without modifying the
                                    standard definition.
 
+ processCachedMessages RemoveCachedMessages
+
            processMapping SelectAction (which also does processPath)
 
            processRoles AuthorizeAction
@@ -128,6 +130,9 @@
       <command
           className="org.apache.struts.chain.commands.servlet.SetContentType"/>
 
+ <!-- Remove messages cached in the Session -->
+ <command
+ className="org.apache.struts.chain.commands.RemoveCachedMessages"/>
 
       <!-- Identify the ActionConfig for this request -->
       <command

Comments

[Niall Pemberton]

Thanks for reporting this.

Looks like the SetOriginalUriCommand is also missing from the tiles version of the chanin-config.xml

[Niall Pemberton]

Fixed, thanks for pointing this out

[Wendy Smoak]

Do you think this should prevent 1.3.5 from going GA?

[Niall Pemberton]

Not IMO.