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

Lukasz Lenart updated WW-4848:
------------------------------
    Description: 
The {{s:if}} tag has the side effect of assigning to an existing object
if the comparison is badly formed using {{=}} instead of {{==}}

{code:html}
<s:if test="user.id = 0">  
  //at this point the user.id is now 0 not its original value
</s:if>
{code}

even though this is clearly badly formed, the test should not assign anything 
to the object

  was:
The <s:if tag has the side effect of assigning to an existing object
if the comparison is badly formed using = instead of ==

<s:if test="user.id = 0">  
  //at this point the user.id is now 0 not its original value
</s:if>

even though this is clearly badly formed, the test should not assign anything 
to the object


> The if test can accidently incorrectly assign a new value to an object
> ----------------------------------------------------------------------
>
>                 Key: WW-4848
>                 URL: https://issues.apache.org/jira/browse/WW-4848
>             Project: Struts 2
>          Issue Type: Bug
>            Reporter: Alan Comeau
>            Priority: Minor
>             Fix For: 2.5.x
>
>
> The {{s:if}} tag has the side effect of assigning to an existing object
> if the comparison is badly formed using {{=}} instead of {{==}}
> {code:html}
> <s:if test="user.id = 0">  
>   //at this point the user.id is now 0 not its original value
> </s:if>
> {code}
> even though this is clearly badly formed, the test should not assign anything 
> to the object



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to