Make name optional for subscribers

apartmentlist · Dec 20, 2012 · 8916aa0 · 8916aa0
1 parent ee14c56
commit 8916aa0
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/lib/yesmail/subscriber.rb b/lib/yesmail/subscriber.rb
@@ -31,11 +31,14 @@ def make_hash
 
     def data
       @attribute_data ||= {}
-      {
-        email: email,
+      name_data = name.blank? ? {} : {
         firstName: first_name,
         lastName: last_name
-      }.merge(attribute_data)
+      }
+
+      {
+        email: email
+      }.merge(name_data).merge(attribute_data)
     end
 
     # These name methods aren't really safe. They might just blow up if the