deac/gitlabhq
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Fix missing call to MergeRequest.find in Notify#changed_merge_request_email

Browse source
This commit is contained in:
Robb Kidd 2012-05-14 20:19:28 -04:00
parent 65382133d5
commit e0b709266e
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
app/mailers/notify.rb
View file

@ -61,7 +61,7 @@ class Notify < ActionMailer::Base
def changed_merge_request_email(user, merge_request) def changed_merge_request_email(user, merge_request)
@user = user @user = user
@merge_request = MergeRequest(merge_request.id) @merge_request = MergeRequest.find(merge_request.id)
@assignee_was ||= User.find(@merge_request.assignee_id_was) @assignee_was ||= User.find(@merge_request.assignee_id_was)
@project = @merge_request.project @project = @merge_request.project
mail(:to => @user['email'], :subject => "gitlab | merge request changed | #{@merge_request.title} ") mail(:to => @user['email'], :subject => "gitlab | merge request changed | #{@merge_request.title} ")