Email bounce error: Velocity transform failed | Community
Skip to main content
Brittany_La
Brittany_La
Level 2
July 8, 2019
Solved

Email bounce error: Velocity transform failed

  • July 8, 2019
  • 1 reply
  • 3225 views

Hi there,

I've had an email campaign set up with a few Velocity-scripted tokens for a while, and have recently been getting this error on email bounces:

Velocity transform failed: ;The nested exception is:<cntrl char>org.apache.velocity.exception.MethodInvocationException: Invocation of method 'compareTo' in  class java.lang.String threw exception java.lang.NullPointerException at 290-ZGA-843:19890:1:static[line 22, column 75]

I've no idea where to even begin troubleshooting this. Any thoughts on where to begin?

This post is no longer active and is closed to new replies. Need help? Start a new post to ask your question.
Best answer by SanfordWhiteman

The NPE is (as the error says) because you can't pass null to compareTo.  And $opportunity.Pipeline_Notification_Date__c can be null.

While you may not have been expecting null there, why are you using .compareTo(other) == 0 instead of .equals(other)?  The former will throw on null, while the latter is a valid method call (it will return false if the argument is null).

And for your purposes, if you want to check equality between 2 native Strings, there's no reason not to use .equals(). 

You only need to use .compareTo() if your objects don't have specialized .equals() logic coded in -- or you don't like either side's .equals() method for some reason -- but the objects do have specialized sorting logic you want to reuse. (In Java-speak, an object that has different rules for sorting and for equality is said to have "ordering inconsistent with equals" and you want to avoid this whenever possible anyway.)

P.S. Looks like you're also mixing up the timezone-d dates and the built-in $date in various places, which means your comparisons can be off (remember that dates that are parsed in different timezones will be different dates for between 1 and 23 hours).

1 reply

SanfordWhiteman
SanfordWhiteman
Level 10
July 8, 2019

Please post the Velocity code itself, highlighting it as Java using the Advanced Editor's syntax highlighter.

And  provide notes about the fields in use if not self-explanatory.

Brittany_La
Brittany_LaAuthor
Level 2
July 8, 2019

Hi Sanford,

Thanks for your reply. See below for the codes. I am using them to create various tokens to insert into an email template.

Token 1

#set( $defaultTimeZone = $date.getTimeZone().getTimeZone("America/New_York") )
#set( $defaultLocale = $date.getLocale() )
#set( $calNow = $date.getCalendar() )
#set( $ret = $calNow.setTimeZone($defaultTimeZone) )
#set( $calConst = $field.in($calNow) )
#set( $ISO8601 = "yyyy-MM-dd'T'HH:mm:ss" )
#set( $ISO8601DateOnly = "yyyy-MM-dd" )
#set($today = $date.format('yyyy-MMM-dd',$date))
$calNow.add($calConst.DATE,45)
#set($today = "MMM dd, yyyy")
${date.format(
 $today,
 $calNow
)}‍‍‍‍‍‍‍‍‍‍‍‍‍‍

Token 2

#set($today = $date.format('yyyy-MM-dd',$date))
#foreach($opportunity in $OpportunityList)
#if($opportunity.Pipeline_Notification_Status__c.contains("45") && $today.compareTo($opportunity.Pipeline_Notification_Date__c)==0)
#set ($matchedOppt = $opportunity)
#break
#end
#end
#if($matchedOppt)
$number.currency($matchedOppt.Amount)
#else
your amount
#end‍‍‍‍‍‍‍‍‍‍‍‍

Token 3

#set($today = $date.format('yyyy-MM-dd',$date))
#foreach($opportunity in $OpportunityList)
#if($opportunity.Pipeline_Notification_Status__c.contains("45") && $today.compareTo($opportunity.Pipeline_Notification_Date__c)==0)
#set ($matchedOppt = $opportunity)
#break
#end
#end
#if($matchedOppt)
$matchedOppt.LH_Loan_Id__c
#else
null
#end‍‍‍‍‍‍‍‍‍‍‍‍

Token 4

#set($today = $date.format('yyyy-MM-dd',$date))
#foreach($opportunity in $OpportunityList)
#if($opportunity.Pipeline_Notification_Status__c.contains("45") && $today.compareTo($opportunity.Pipeline_Notification_Date__c)==0)
#set ($matchedOppt = $opportunity)
#break
#end
#end
#if($matchedOppt)
$matchedOppt.LH_Loan_Id__c
#else
null
#end‍‍‍‍‍‍‍‍‍‍‍‍

Token 5

#set($today = $date.format('yyyy-MM-dd',$date))
#foreach($opportunity in $OpportunityList)
#if($opportunity.Pipeline_Notification_Status__c.contains("45") && $today.compareTo($opportunity.Pipeline_Notification_Date__c)==0)
#set ($matchedOppt = $opportunity)
#break
#end
#end
#if($matchedOppt)
$matchedOppt.Property_Address__c
#else
your address
#end‍‍‍‍‍‍‍‍‍‍‍‍
SanfordWhiteman
SanfordWhitemanAccepted solution
Level 10
July 8, 2019

The NPE is (as the error says) because you can't pass null to compareTo.  And $opportunity.Pipeline_Notification_Date__c can be null.

While you may not have been expecting null there, why are you using .compareTo(other) == 0 instead of .equals(other)?  The former will throw on null, while the latter is a valid method call (it will return false if the argument is null).

And for your purposes, if you want to check equality between 2 native Strings, there's no reason not to use .equals(). 

You only need to use .compareTo() if your objects don't have specialized .equals() logic coded in -- or you don't like either side's .equals() method for some reason -- but the objects do have specialized sorting logic you want to reuse. (In Java-speak, an object that has different rules for sorting and for equality is said to have "ordering inconsistent with equals" and you want to avoid this whenever possible anyway.)

P.S. Looks like you're also mixing up the timezone-d dates and the built-in $date in various places, which means your comparisons can be off (remember that dates that are parsed in different timezones will be different dates for between 1 and 23 hours).

    Terms & ConditionsAccessibility statement

    Loading footer...