Merge pull request spree-contrib#16 from iloveitaly/better-subject

Model subject line after Amazon's.
2870 · Aug 30, 2012 · 8610c6f · 8610c6f
2 parents 327975b + 04b71f0
commit 8610c6f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/app/models/spree/mail_to_friend.rb b/app/models/spree/mail_to_friend.rb
@@ -13,9 +13,9 @@ class Spree::MailToFriend
   validates :invalid_recipients, :length => {:maximum => 0, :message => "must be removed"}
 
   def initialize(opts = {})
-    @subject      = opts[:subject]      || I18n.t('email_to_friend.you_would_like_this')
     @sender_email = opts[:sender_email] || ' '
     @sender_name  = opts[:sender_name]  || @sender_email.split('@', 2)[0].titleize
+    @subject      = opts[:subject]      || I18n.t('email_to_friend.you_would_like_this', :sender_name => @sender_name, :site => Spree::Config[:site_url])
 
     @recipients = []
     @invalid_recipients = []

diff --git a/config/locales/en.yml b/config/locales/en.yml
@@ -12,7 +12,7 @@ en:
     send_message: "Tell your friend!"
     mail_sent_to: "Mail sent to %{email} "
     send_to_other: "Send to other friend"
-    you_would_like_this: "I think you would like this item"
+    you_would_like_this: "%{sender_name} wants you to see this item at %{site}"
 
   captcha:
     captcha_settings: "Captcha Settings"