Re: [PATCH] BEEHIVE-307: Struts validator support for the msg and args' bundle attribute

[Rich Feit <ri...@gmail.com>]

Thanks, Carlin.  I'm sure this one was a joy to debug.  I will get this 
in after we go out the door with 1.0 (I agree that it can wait until then).

Rich

Carlin Rogers wrote:

>I've attached a patch to http://issues.apache.org/jira/browse/BEEHIVE-307 to 
>fix
>an issue with NetUI to handle an already formatted error ActionMessage.
>
>Updating to Struts version 1.2.7 solved the issue within the validator. 
>However,
>when an alternate resource bundle is used, the struts validator Resources 
>class will
>format the message using the alternate resource bundle and then create the 
>new
>ActionMessage with the formatted message and resource=false. The 
>ActionMessage
>key is the message, not a key for us to use later when processing the errors 
>in
>the Error tag.
>
>The fix changes the ErrorBaseTag.getErrorMessage() method to check if the 
>error 
>ActionMessage is a resource or not. If not, then we just get the already 
>formatted
>message and return.
>
>The patch also has the updated test files along with a new testRecorder BVT.
>
>Let me know if you see any issues.
>
>Thanks,
>Carlin
>
>  
>