Details

    • Type: New Feature New Feature
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.x
    • Fix Version/s: 2.x
    • Component/s: Engine
    • Labels:
      None

      Description

      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.

      1. value.patch
        3 kB
        Byron Foster

        Activity

        Mark Thomas made changes -
        Workflow Default workflow, editable Closed status [ 12551353 ] jira [ 12552197 ]
        Mark Thomas made changes -
        Workflow jira [ 12451111 ] Default workflow, editable Closed status [ 12551353 ]
        Nathan Bubna made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 2.0 [ 12310291 ]
        Resolution Fixed [ 1 ]
        Byron Foster made changes -
        Affects Version/s 2.0 [ 12310291 ]
        Affects Version/s 1.7 [ 12313453 ]
        Byron Foster made changes -
        Field Original Value New Value
        Attachment value.patch [ 12398830 ]
        Byron Foster created issue -

          People

          • Assignee:
            Unassigned
            Reporter:
            Byron Foster
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development