Bugfix send accept / reject event mail

This commit is contained in:
Chrisbr 2014-05-13 15:24:21 +02:00
parent 05392701a9
commit 0db2f3bb06
2 changed files with 19 additions and 18 deletions

View file

@ -132,14 +132,15 @@ class Event < ActiveRecord::Base
end end
def process_acceptance(options) def process_acceptance(options)
if options[:send_mail] == "true" if self.conference.email_settings.send_on_accepted && options[:send_mail].blank?
Rails.logger.debug "Sending acceptance mail" Rails.logger.debug 'Sending event acceptance mail'
Mailbot.acceptance_mail(self).deliver Mailbot.acceptance_mail(self).deliver
end end
end end
def process_rejection(options) def process_rejection(options)
if options[:send_mail] == "true" if self.conference.email_settings.send_on_rejected && options[:send_mail].blank?
Rails.logger.debug 'Sending rejected mail'
Mailbot.rejection_mail(self).deliver Mailbot.rejection_mail(self).deliver
end end
end end

View file

@ -106,21 +106,21 @@
<b class="caret"></b> <b class="caret"></b>
%ul.dropdown-menu %ul.dropdown-menu
- if event.transition_possible? :accept - if event.transition_possible? :accept
%li= link_to 'Accept event (no email)', %li= link_to 'Accept event',
update_state_admin_conference_event_path(@conference.short_title, event, transition: :accept, send_mail: false), update_state_admin_conference_event_path(@conference.short_title, event, transition: :accept),
method: :patch, hint: 'Accept this event without sending an automated email.' method: :patch
%li= link_to 'Accept event (WITH email)', - if @conference.email_settings.send_on_accepted?
update_state_admin_conference_event_path(@conference.short_title, event, transition: :accept, send_mail: true), %li= link_to 'Accept event (without email)',
method: :patch, hint: 'Accept this event and send an automated email.' update_state_admin_conference_event_path(@conference.short_title, event, transition: :accept, send_mail: false),
method: :patch, hint: 'Accept this event without sending an automated email.'
- if event.transition_possible? :reject - if event.transition_possible? :reject
%li= link_to 'Reject event (no email)', %li= link_to 'Reject event',
update_state_admin_conference_event_path(@conference.short_title, event, transition: :reject, send_mail: false), update_state_admin_conference_event_path(@conference.short_title, event, transition: :reject),
method: :patch, confirm: 'Are you sure?', method: :patch, confirm: 'Are you sure?'
hint: 'Reject this event without sending an automated email.' - if @conference.email_settings.send_on_rejected?
%li= link_to 'Reject event (WITH email)', %li= link_to 'Reject event (without email)',
update_state_admin_conference_event_path(@conference.short_title, event, transition: :reject, send_mail: true), update_state_admin_conference_event_path(@conference.short_title, event, transition: :reject, send_mail: false),
method: :patch, confirm: 'Are you sure?', method: :patch, confirm: 'Are you sure?'
hint: 'Reject this event and send an automated email.'
- if event.transition_possible? :start_review - if event.transition_possible? :start_review
%li= link_to 'Start review', %li= link_to 'Start review',
update_state_admin_conference_event_path(@conference.short_title, event, transition: :start_review), update_state_admin_conference_event_path(@conference.short_title, event, transition: :start_review),