[ 
https://issues.apache.org/jira/browse/VELOCITY-687?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12669004#action_12669004
 ] 

Byron Foster commented on VELOCITY-687:
---------------------------------------

Ok, I would argue the viability of creating generic cross project macro 
libraries currently in Velocity given several factors, including what you 
outline above.  But certainly a macro param behavior switch doesn't improve the 
situation.

I think a 2.0 branch is a good solution.

> Macro pass by value option
> --------------------------
>
>                 Key: VELOCITY-687
>                 URL: https://issues.apache.org/jira/browse/VELOCITY-687
>             Project: Velocity
>          Issue Type: New Feature
>          Components: Engine
>    Affects Versions: 2.0
>            Reporter: Byron Foster
>         Attachments: value.patch
>
>
> Add a configuration property which would change macro parameter passing from 
> pass by name, to pass by value.  I think in many instances this option will 
> provide behavior that is more intuitive to the user.  There are two important 
> exceptions to this, references created by #define, and references created by 
> BlockMacro's $bodyContent.   This allows the user to still specify pass by 
> name semantics when desired, for example:
> #define($x) #if($foo)$foo.bar#{else}-None-#end  #end
> #go($x)
> In the above example '#if($foo)$foo.bar#{else}-None-#end' will be passed by 
> name.  In all other cases pass by value rules apply for example:
> #go($foo.bar  [1, 2, 3]  #if($bar)Yes#{else}No#end)
> In the above example all parameters will be evaluated to a value first, then 
> passed to #go.  This has potential performance improvements also.

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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to