You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@struts.apache.org by "Musachy Barroso (JIRA)" <ji...@apache.org> on 2008/12/15 16:41:37 UTC

[jira] Resolved: (WW-2824) refactored altSyntax expression code

     [ https://issues.apache.org/struts/browse/WW-2824?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Musachy Barroso resolved WW-2824.
---------------------------------

    Resolution: Fixed

Merged into trunk. Thanks for the patch, this has been on my TODO list for a while :)

> refactored altSyntax expression code
> ------------------------------------
>
>                 Key: WW-2824
>                 URL: https://issues.apache.org/struts/browse/WW-2824
>             Project: Struts 2
>          Issue Type: Task
>          Components: Expression Language
>    Affects Versions: 2.1.3
>         Environment: Tested in S2.1.3
>            Reporter: Gabriel Belingueres
>            Priority: Minor
>             Fix For: 2.1.3
>
>         Attachments: altSyntax.patch
>
>
> I did some basic refactorings to the code that tests things like stripping/adding the "%{" and "}" to the expressions, as suggested in https://issues.apache.org/struts/browse/WW-2811
> Now all the "%{" and "}" handling are confined inside the Component class.
> Note that in class StrutsBodyTagSupport.java I changed the translateVariables method implementation for one in TextParseUtil. I *think* they do the same thing (all S2 Core tests passed but this may need more testing.)

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