Use rails tokens for signup confirmations

[rails.git] / app / mailers / user_mailer.rb

diff --git a/app/mailers/user_mailer.rb b/app/mailers/user_mailer.rb
index a9c6b70f91eeb29d8538aa1c18859e199a38b8ad..92c64b4d6d20b70f9d00c2301289e90fbb869063 100644 (file)
--- a/app/mailers/user_mailer.rb
+++ b/app/mailers/user_mailer.rb
@@ -10,11 +10,12 @@ class UserMailer < ApplicationMailer
   before_action :set_shared_template_vars
   before_action :attach_project_logo
 
   before_action :set_shared_template_vars
   before_action :attach_project_logo
 

-  def signup_confirm(user, token)
+  def signup_confirm(user, token, referer = nil)

     with_recipient_locale user do
       @url = url_for(:controller => "confirmations", :action => "confirm",
                      :display_name => user.display_name,
     with_recipient_locale user do
       @url = url_for(:controller => "confirmations", :action => "confirm",
                      :display_name => user.display_name,

-                     :confirm_string => token.token)
+                     :confirm_string => token,
+                     :referer => referer)

 
       mail :to => user.email,
            :subject => t(".subject")
 
       mail :to => user.email,
            :subject => t(".subject")


@@ -25,7 +26,7 @@ class UserMailer < ApplicationMailer
     with_recipient_locale user do
       @address = user.new_email
       @url = url_for(:controller => "confirmations", :action => "confirm_email",
     with_recipient_locale user do
       @address = user.new_email
       @url = url_for(:controller => "confirmations", :action => "confirm_email",

-                     :confirm_string => token.token)
+                     :confirm_string => token)

 
       mail :to => user.new_email,
            :subject => t(".subject")
 
       mail :to => user.new_email,
            :subject => t(".subject")


@@ -34,8 +35,7 @@ class UserMailer < ApplicationMailer
 
   def lost_password(user, token)
     with_recipient_locale user do
 
   def lost_password(user, token)
     with_recipient_locale user do

-      @url = url_for(:controller => "passwords", :action => "reset_password",
-                     :token => token.token)
+      @url = user_reset_password_url(:token => token)

 
       mail :to => user.email,
            :subject => t(".subject")
 
       mail :to => user.email,
            :subject => t(".subject")


@@ -45,11 +45,13 @@ class UserMailer < ApplicationMailer
   def gpx_success(trace, possible_points)
     with_recipient_locale trace.user do
       @to_user = trace.user.display_name
   def gpx_success(trace, possible_points)
     with_recipient_locale trace.user do
       @to_user = trace.user.display_name

+      @trace_url = show_trace_url(trace.user, trace)

       @trace_name = trace.name
       @trace_points = trace.size
       @trace_description = trace.description
       @trace_tags = trace.tags
       @possible_points = possible_points
       @trace_name = trace.name
       @trace_points = trace.size
       @trace_description = trace.description
       @trace_tags = trace.tags
       @possible_points = possible_points

+      @my_traces_url = url_for(:controller => "traces", :action => "mine")

 
       mail :to => trace.user.email,
            :subject => t(".subject")
 
       mail :to => trace.user.email,
            :subject => t(".subject")


@@ -81,7 +83,7 @@ class UserMailer < ApplicationMailer
 
       attach_user_avatar(message.sender)
 
 
       attach_user_avatar(message.sender)
 

-      mail :from => from_address(message.sender.display_name, "m", message.id, message.digest),
+      mail :from => from_address(message.sender.display_name, "m", message.id, message.notification_token),

            :to => message.recipient.email,
            :subject => t(".subject", :message_title => message.title)
     end
            :to => message.recipient.email,
            :subject => t(".subject", :message_title => message.title)
     end


@@ -96,13 +98,14 @@ class UserMailer < ApplicationMailer
       @readurl = diary_entry_url(comment.diary_entry.user, comment.diary_entry, :anchor => "comment#{comment.id}")
       @commenturl = diary_entry_url(comment.diary_entry.user, comment.diary_entry, :anchor => "newcomment")
       @replyurl = new_message_url(comment.user, :message => { :title => "Re: #{comment.diary_entry.title}" })
       @readurl = diary_entry_url(comment.diary_entry.user, comment.diary_entry, :anchor => "comment#{comment.id}")
       @commenturl = diary_entry_url(comment.diary_entry.user, comment.diary_entry, :anchor => "newcomment")
       @replyurl = new_message_url(comment.user, :message => { :title => "Re: #{comment.diary_entry.title}" })

+      @unsubscribeurl = diary_entry_unsubscribe_url(comment.diary_entry.user, comment.diary_entry)

       @author = @from_user
 
       attach_user_avatar(comment.user)
 
       set_references("diary", comment.diary_entry)
 
       @author = @from_user
 
       attach_user_avatar(comment.user)
 
       set_references("diary", comment.diary_entry)
 

-      mail :from => from_address(comment.user.display_name, "c", comment.id, comment.digest, recipient.id),
+      mail :from => from_address(comment.user.display_name, "c", comment.id, comment.notification_token(recipient.id), recipient.id),

            :to => recipient.email,
            :subject => t(".subject", :user => comment.user.display_name)
     end
            :to => recipient.email,
            :subject => t(".subject", :user => comment.user.display_name)
     end


@@ -225,12 +228,12 @@ class UserMailer < ApplicationMailer
     I18n.with_locale(Locale.available.preferred(recipient.preferred_languages), &block)
   end
 
     I18n.with_locale(Locale.available.preferred(recipient.preferred_languages), &block)
   end
 

-  def from_address(name, type, id, digest, user_id = nil)
+  def from_address(name, type, id, token, user_id = nil)

     if Settings.key?(:messages_domain) && domain = Settings.messages_domain
       if user_id
     if Settings.key?(:messages_domain) && domain = Settings.messages_domain
       if user_id

-        "#{name} <#{type}-#{id}-#{user_id}-#{digest[0, 6]}@#{domain}>"
+        "#{name} <#{type}-#{id}-#{user_id}-#{token}@#{domain}>"

       else
       else

-        "#{name} <#{type}-#{id}-#{digest[0, 6]}@#{domain}>"
+        "#{name} <#{type}-#{id}-#{token}@#{domain}>"

       end
     else
       Settings.email_from
       end
     else
       Settings.email_from