Commit e84411ef authored by Dmitriy Zaporozhets's avatar Dmitriy Zaporozhets

Merge pull request #820 from robbkidd/add_missing_call_to_find

Fix bug: missing call to MergeRequest.find in Notify#changed_merge_request_email
parents 65382133 e0b70926
...@@ -61,7 +61,7 @@ class Notify < ActionMailer::Base ...@@ -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} ")
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment