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 the ability to specify default parameters to macros, for example:

      #macro(foo $x $y="legit")$x$y#end

      calling this with #foo(2) would give:

      2legit

      Any number of default parameters can be specified, but no non-default parameters can not follow default parameters. Assignment of calling values begins from left to right and all left over default arguments are assigned their default values.

        Activity

        Hide
        Christopher Schultz added a comment -

        Any chance for a back-port to Velocity 1.x? I could really use this and it looks like Velocity 2.0 isn't exactly right around the corner...

        Show
        Christopher Schultz added a comment - Any chance for a back-port to Velocity 1.x? I could really use this and it looks like Velocity 2.0 isn't exactly right around the corner...
        Hide
        Byron Foster added a comment -

        Sounds good, I was leaving it open for any comments/criticism, otherwise the implementation is complete.

        Show
        Byron Foster added a comment - Sounds good, I was leaving it open for any comments/criticism, otherwise the implementation is complete.
        Hide
        Nathan Bubna added a comment -

        Byron, i'm marking this resolved, because the feature looks pretty complete in 2.0 to me. If there's something that remains to be done for this, feel free to re-open.

        Show
        Nathan Bubna added a comment - Byron, i'm marking this resolved, because the feature looks pretty complete in 2.0 to me. If there's something that remains to be done for this, feel free to re-open.

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development