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

hyphen in identifiers cause parse error

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Not A Bug
    • 2.0
    • None
    • Engine
    • None

    Description

      Upon upgrading to Velocity 2.0 from a very old version (1.4), hyphens in identifiers in templates throw a ParseException.

      To verify what was happening, I did a clean install of velocity 2.0 to test your example code.  After running the unmodified "example2.vm" successfully I added an identifier with a hyphen and the parse error occurred.

      Apologies if I am missing something (e.g. spec change on identifiers).  If this is an issue that will take a while to fix, please let me know so that I can change out the hyphens to underbars in id's in my template sets.

      Attached are:   1) code snippet 2) parse exception msg 3) cl used to invoke velocity 2.0 upon example2.vm

      Best Regards

      --------------------

      1) code snippet

      #set($test-var-w-hyphen = "test")

      test-var-w-hyphen : $test-var-w-hyphen

      ------------------------------

      2) Parse exception

      [main] ERROR org.apache.velocity.parser - example2.vm: Encountered "-" at line 20, column 11.
      Was expecting one of:
      "[" ...
      <WHITESPACE> ...
      <NEWLINE> ...
      "=" ...

      --------------------------

      3) Command Script

      #/bin/bash
      java -cp .:/home/test-a/.ant/lib/commons-lang3-3.8.jar:/home/test-a/.ant/lib/slf4j-api-1.7.25.jar:/home/test-a/.ant/lib/velocity-engine-core-2.0.jar:/home/test-a/.ant/lib/velocity-engine-scripting-2.0.jar:/home/test-a/.ant/lib/slf4j-simple-1.7.25.jar org.apache.velocity.example.Example2

       

       

      Attachments

        1. err
          0.7 kB
          Mark Newton
        2. example2.vm.snip
          0.1 kB
          Mark Newton
        3. run.sh
          0.3 kB
          Mark Newton

        Issue Links

          Activity

            People

              cbrisson Claude Brisson
              mobileboy66 Mark Newton
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: