From 496bb8738d059cbdd3fc697f772f92f3862cb9c3 Mon Sep 17 00:00:00 2001
From: Matt Farnsworth <matt@perfectline.co>
Date: Wed, 16 Dec 2015 18:15:14 +0200
Subject: [PATCH] Story #109367018 - warn if email missing, and format pending
 email

---
 app/models/domain_mail_model.rb | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/app/models/domain_mail_model.rb b/app/models/domain_mail_model.rb
index 01d01a6dc..45c550f2d 100644
--- a/app/models/domain_mail_model.rb
+++ b/app/models/domain_mail_model.rb
@@ -97,7 +97,7 @@ class DomainMailModel
   end
 
   def registrant_pending
-    @params[:recipient] = @domain.pending_json['new_registrant_email']
+    @params[:recipient] = format @domain.pending_json['new_registrant_email']
     @params[:new_registrant_name] = @domain.pending_json['new_registrant_name']
   end
   
@@ -109,6 +109,7 @@ class DomainMailModel
 
   # puny internet domain name, TODO: username<email>
   def format(email)
+    return warn_no_email if email.nil?
     user, host = email.split('@')
     host = SimpleIDN.to_ascii(host)
     "#{user}@#{host}"
@@ -154,9 +155,8 @@ class DomainMailModel
     @params
   end
 
-  def warn_no_email(item)
-    #reason = "#{item.to_s} for #{@domain.name}"
-    warn_not_delivered item #reason
+  def warn_no_email(item = 'email')
+    warn_missing item
     nil
   end