Wrap ref evaluate exception so vm location is revealed
------------------------------------------------------

                 Key: VELOCITY-656
                 URL: https://issues.apache.org/jira/browse/VELOCITY-656
             Project: Velocity
          Issue Type: Improvement
    Affects Versions: 1.6.1
            Reporter: Byron Foster


The following VTL calls the toString() method on the object in $foo:

#if($foo)#end  

If the toString() method throws an exception, the exception is thrown all the 
way out of velocity without any indication of where the vtl reference occurred. 
 This patch wraps the exception with a VelocityException with a message that 
indicates the location of the offending reference.

However, I wonder why toString() is called at all...  the code tests for 
toString() == null and returns false if so, and true otherwise in the above 
example vtl.  Why is this test necessary, and why isn't just a non-null object 
enough for testing?




-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to