diff --git a/app/mailers/notify.rb b/app/mailers/notify.rb index a4ffd97fb7cd8e07db5c21276b4fd7256ec10569..b776878b03ba66b6ef42522fe74dc9fa9ad3e8e7 100644 --- a/app/mailers/notify.rb +++ b/app/mailers/notify.rb @@ -13,12 +13,9 @@ class Notify < ActionMailer::Base mail(:to => @user.email, :subject => "gitlab | Account was created for you") end - def new_issue_email(issue) - @issue = Issue.find(issue['id']) - @user = @issue.assignee - @project = @issue.project - - mail(:to => @user.email, :subject => "gitlab | New Issue was created") + def new_issue_email(issue_id) + @issue = Issue.find(issue_id) + mail(:to => @issue.assignee_email, :subject => "gitlab | New Issue was created") end def note_wall_email(recipient_id, note_id) diff --git a/app/views/notify/new_issue_email.html.haml b/app/views/notify/new_issue_email.html.haml index 64c5aa611eb6d3af90f69cc7e8ff949aaa34a083..dd6f50c0686bafc1c2276ddee120de144f621141 100644 --- a/app/views/notify/new_issue_email.html.haml +++ b/app/views/notify/new_issue_email.html.haml @@ -10,7 +10,7 @@ %td{:style => "font-size: 1px; line-height: 1px;", :width => "21"} %td{:align => "left", :style => "padding: 20px 0 0;"} %h2{:style => "color:#646464 !important; font-weight: bold; margin: 0; padding: 0; line-height: 26px; font-size: 18px; font-family: Helvetica, Arial, sans-serif; "} - = link_to project_issue_url(@project, @issue), :title => @issue.title do + = link_to project_issue_url(@issue.project, @issue), :title => @issue.title do = "Issue ##{@issue.id.to_s}" = truncate(@issue.title, :length => 45) %br diff --git a/spec/mailers/notify_spec.rb b/spec/mailers/notify_spec.rb index 60c8c8ab9a02443597e9e948cc7b5c35772a1506..40a51437908ee3d43a2f8bdca2cd1a5bd1aaff85 100644 --- a/spec/mailers/notify_spec.rb +++ b/spec/mailers/notify_spec.rb @@ -59,7 +59,7 @@ describe Notify do let(:issue) { Factory.create(:issue, :assignee => assignee, :project => project ) } describe 'that are new' do - subject { Notify.new_issue_email(issue) } + subject { Notify.new_issue_email(issue.id) } it_behaves_like 'an assignee email'