Uploaded image for project: 'Velocity'
  1. Velocity
  2. VELOCITY-642

Array arguments in method calls are wrapped with an extra array

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.6-beta1
    • Fix Version/s: 1.6
    • Component/s: Engine
    • Labels:
      None

      Description

      Velocity 1.6 b2 wraps array arguments of method calls with an extra array causing the calls to fail and making this version incompatible with previous versions.

      A simple template, like this, demonstrates the behavior:

      #set ($_array = [ 1, 2, 3 ])
      $test.arrayTest($_array.toArray())

      Depending on the signature of test.arrayTest() , the template prints:

      public String arrayTest(Object[] array)
      {
      return java.util.Arrays.deepToString((Object[]) array);
      }

      [[1, 2, 3]]

      public String arrayTest(Object... array)
      {
      return java.util.Arrays.deepToString(array);
      }

      [[1, 2, 3]]

      public String arrayTest(Object array)
      {
      return java.util.Arrays.deepToString((Object[]) array);
      }

      [1, 2, 3]

      Actually, I noticed later that the argument doesn't have to be an array, this version of Velocity seems to wrap any argument into an array if the method expects an array. So maybe this is a feature and not a bug, but it should detect already correct array arguments and leave them intact to keep existing templates working.

      – Ilkka

        Attachments

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              ipriha Ilkka Priha
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: