cafe-grader-web Commit - r336:4e59f67ded1a

cafe-grader-web

Commit `r336:4e59f67ded1a` public

parent | child

Description:

moved send mail code back to helper

Commit status:

[Not Reviewed]

References:

default

Diff options:

| | | |

Comments:

0 Commit comments 0 Inline Comments

Unresolved TODOs:

There are no unresolved TODOs

Add another comment

r336:4e59f67ded1a - Sat, 06 Oct 2012 07:35:19 - 3 files changed: 22 inserted, 50 deleted

app/controllers/user_admin_controller.rb ¶ +4 -16

               class UserAdminController < ApplicationController
+            +   include MailHelperMethods
+            +
                 before_filter :admin_authorization
                 # GETs should be safe (see http://www.w3.org/2001/tag/doc/whenToUseGet.html)
                 verify :method => :post, :only => [ :destroy,
                                                     :create, :create_from_list,
                                                     :update,
                                                     :manage_contest,
                                                     :bulk_mail
                                                   ],
                        :redirect_to => { :action => :list }
                 def index
                   list
                   render :action => 'list'
                 end
                 def list
                   @user_count = User.count
                   if params[:page] == 'all'
                     @users = User.all
                     @paginated = false
                   else
                     @users = User.paginate :page => params[:page]
                     @paginated = true
                 # mass mailing
                 def mass_mailing
                 end
                 def bulk_mail
                   lines = params[:login_list]
                   if !lines or lines.blank?
                     flash[:notice] = 'You entered an empty list.'
                     redirect_to :action => 'mass_mailing' and return
                   end
                   mail_subject = params[:subject]
                   if !mail_subject or mail_subject.blank?
                     flash[:notice] = 'You entered an empty mail subject.'
                     redirect_to :action => 'mass_mailing' and return
                   end
                   mail_body = params[:email_body]
                   if !mail_body or mail_body.blank?
                     flash[:notice] = 'You entered an empty mail body.'
                     redirect_to :action => 'mass_mailing' and return
                   end
-            -     admin_email = GraderConfiguration['system.admin_email']
+            -
                   note = []
                   users = []
                   lines.split("\n").each do |line|
                     user = User.find_by_login(line.chomp)
                     if user
-            -         Mail.deliver do
-            -           from admin_email
-            -           to user.email
-            -           subject mail_subject
-            -           body mail_body
-            -         end
+            +         send_mail(user.email, mail_subject, mail_body)
                       note << user.login
                     end
                   end
                   flash[:notice] = 'User(s) ' + note.join(', ') +
                     ' were successfully modified.  '
                   redirect_to :action => 'mass_mailing'
                 end
                 protected
                 def random_password(length=5)
                   chars = 'abcdefghijkmnopqrstuvwxyz23456789'
                   newpass = ""
                   length.times { newpass << chars[rand(chars.size-1)] }
                   return newpass
                 end
                 def import_from_file(f)
                   data_hash = YAML.load(f)
                   @import_log = ""
                   country_data = data_hash[:countries]
                   site_data = data_hash[:sites]
                 end
                 def logout_users(users)
                   users.each do |user|
                     contest_stat = user.contest_stat(true)
                     if contest_stat and !contest_stat.forced_logout
                       contest_stat.forced_logout = true
                       contest_stat.save
                     end
                   end
                 end
                 def send_contest_update_notification_email(user, contest)
                   contest_title_name = GraderConfiguration['contest.name']
                   contest_name = contest.name
                   mail_subject = t('contest.notification.email_subject', {
                                      :contest_title_name => contest_title_name,
                                      :contest_name => contest_name })
                   mail_body = t('contest.notification.email_body', {
                                   :full_name => user.full_name,
                                   :contest_title_name => contest_title_name,
                                   :contest_name => contest.name,
                                 })
-            -     admin_email = GraderConfiguration['system.admin_email']
+            -
                   logger.info mail_body
-            -     Mail.deliver do
-            -       from admin_email
-            -       to user.email
-            -       subject mail_subject
-            -       body mail_body
-            -     end
+            +     send_mail(user.email, mail_subject, mail_body)
                 end
                 def find_contest_and_user_from_contest_id(id)
                   if id!='none'
                     @contest = Contest.find(id)
                   else
                     @contest = nil
                   end
                   if @contest
                     @users = @contest.users
                   else
                     @users = User.find_users_with_no_contest
                   end
                   return [@contest, @users]
                 end
               end

app/controllers/users_controller.rb ¶ +4 -14

               require 'net/smtp'
               class UsersController < ApplicationController
+            +   include MailHelperMethods
+            +
                 before_filter :authenticate, :except => [:new,
                                                          :register,
                                                          :confirm,
                                                          :forget,
                                                          :retrieve_password]
                 before_filter :verify_online_registration, :only => [:new,
                                                                      :register,
                                                                      :forget,
                                                                      :retrieve_password]
                 verify :method => :post, :only => [:chg_passwd],
                        :redirect_to => { :action => :index }
                 #in_place_edit_for :user, :alias_for_editing
                 #in_place_edit_for :user, :email_for_editing
                 def index
                   if !GraderConfiguration['system.user_setting_enabled']
                     redirect_to :controller => 'main', :action => 'list'
                   else
                     @user = User.find(session[:user_id])
                   end
                 end
                     if last_updated_time > Time.now.gmtime - 5.minutes
                       flash[:notice] = 'The account has recently created or new password has recently been requested.  Please wait for 5 minutes'
                     else
                       user.password = user.password_confirmation = User.random_password
                       user.save
                       send_new_password_email(user)
                       flash[:notice] = 'New password has been mailed to you.'
                     end
                   else
                     flash[:notice] = I18n.t 'registration.password_retrieval.no_email'
                   end
                   redirect_to :action => 'forget'
                 end
                 protected
                 def verify_online_registration
                   if !GraderConfiguration['system.online_registration']
                     redirect_to :controller => 'main', :action => 'login'
                   end
                 end
                 def send_confirmation_email(user)
                   contest_name = GraderConfiguration['contest.name']
-            -     admin_email = GraderConfiguration['system.admin_email']
                   activation_url = url_for(:action => 'confirm',
                                            :login => user.login,
                                            :activation => user.activation_key)
                   home_url = url_for(:controller => 'main', :action => 'index')
                   mail_subject = "[#{contest_name}] Confirmation"
                   mail_body = t('registration.email_body', {
                                   :full_name => user.full_name,
                                   :contest_name => contest_name,
                                   :login => user.login,
                                   :password => user.password,
                                   :activation_url => activation_url,
                                   :admin_email => admin_email
                                 })
                   logger.info mail_body
-            -     Mail.deliver do
-            -       from admin_email
-            -       to user.email
-            -       subject mail_subject
-            -       body mail_body
-            -     end
+            +     send_mail(user.email, mail_subject, mail_body)
                 end
                 def send_new_password_email(user)
                   contest_name = GraderConfiguration['contest.name']
-            -     admin_email = GraderConfiguration['system.admin_email']
                   mail_subject = "[#{contest_name}] Password recovery"
                   mail_body = t('registration.password_retrieval.email_body', {
                                   :full_name => user.full_name,
                                   :contest_name => contest_name,
                                   :login => user.login,
                                   :password => user.password,
                                   :admin_email => admin_email
                                 })
                   logger.info mail_body
-            -     Mail.deliver do
-            -       from admin_email
-            -       to user.email
-            -       subject mail_subject
-            -       body mail_body
-            -     end
+            +     send_mail(user.email, mail_subject, mail_body)
                 end
               end

lib/mail_helper_methods.rb ¶ +14 -20

               module MailHelperMethods
-            -   def send_mail(to, subject, body)
-            -     mail = TMail::Mail.new
-            -     mail.to = to
-            -     mail.from = Configuration['system.online_registration.from']
-            -     mail.subject = subject
-            -     mail.body = body
+            -
-            -     smtp_server = Configuration['system.online_registration.smtp']
+            +   def send_mail(mail_to, mail_subject, mail_body)
+            +     mail_from = GraderConfiguration['system.online_registration.from']
+            +     smtp_server = GraderConfiguration['system.online_registration.smtp']
                   if ['fake', 'debug'].include? smtp_server
                     puts "-------------------------
-            - To: #{mail.to}
-            - From: #{mail.from}
-            - Subject: #{mail.subject}
-            - #{mail.body}
+            + To: #{mail_to}
+            + From: #{mail_from}
+            + Subject: #{mail_subject}
+            + #{mail_body}
               --------------------------
               "
                     return true
                   end
-            -     begin
-            -       Net::SMTP.start(smtp_server) do |smtp|
-            -         smtp.send_message(mail.to_s, mail.from, mail.to)
-            -       end
-            -       result = true
-            -     rescue
-            -       result = false
+            +     mail = Mail.new do
+            +       from mail_from
+            +       to mail_to
+            +       subject mail_subject
+            +       body mail_body
                   end
-            -     result
+            +     mail.delivery_settings = { :address => smtp_server }
+            +     mail.deliver
                 end
               end

Write
Preview

You need to be logged in to leave comments. Login now

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository permissions settings

Sign in to your account

Author

r336:4e59f67ded1a - Sat, 06 Oct 2012 07:35:19 - 3 files changed: 22 inserted, 50 deleted

Sign in to your account

Commit r336:4e59f67ded1a public

Author

r336:4e59f67ded1a - Sat, 06 Oct 2012 07:35:19 - 3 files changed: 22 inserted, 50 deleted

Commit `r336:4e59f67ded1a` public