Velocity
  1. Velocity
  2. VELOCITY-758

Include line number in message "#parse() null argument"

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: 1.6.2
    • Fix Version/s: 1.7, 2.x
    • Component/s: Engine
    • Labels:
      None

      Description

      Assume the following scenarios:

      Scenario A)
      A template has the following line:
      #parse("never-found-template.vm")

      Scenario B)
      A template has the following line:
      #parse($nonsense)

      Assume further, that neither the "never-found-template.vm" is present nor the variable "$nonsense" is set.

      In scenario A) the log reports "#parse(): cannot find template 'never-found-template.vm', called from template ... at (1, 1)"
      In scenario B) the log says "#parse() null argument"

      Problem:
      In scenario B), having multiple nested #parse-Statements, it's hard to find the error finally.

      Sugestion / Feature request:
      Include source information and line numbers when reporting "#parse() null argument" - analogous to scenario A)

      Thanks and kind regards

      Edy Hinzen

      1. velocity-758.patch
        3 kB
        Jarkko Viinamäki

        Activity

        Mark Thomas made changes -
        Workflow Default workflow, editable Closed status [ 12551721 ] jira [ 12552262 ]
        Mark Thomas made changes -
        Workflow jira [ 12501501 ] Default workflow, editable Closed status [ 12551721 ]
        Nathan Bubna made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 1.7 [ 12313453 ]
        Fix Version/s 2.0 [ 12310291 ]
        Resolution Fixed [ 1 ]
        Jarkko Viinamäki made changes -
        Field Original Value New Value
        Attachment velocity-758.patch [ 12439972 ]
        Edy Hinzen created issue -

          People

          • Assignee:
            Unassigned
            Reporter:
            Edy Hinzen
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development