Commit 3fba2114260531977b5fecfc17941f5b851005ee
1 parent
890b9f3a
Exists in
master
and in
4 other branches
Fix bug MR notes not being loaded when navigate from diff tab
Showing
1 changed file
with
3 additions
and
3 deletions
Show diff stats
app/controllers/merge_requests_controller.rb
| @@ -20,9 +20,6 @@ class MergeRequestsController < ProjectResourceController | @@ -20,9 +20,6 @@ class MergeRequestsController < ProjectResourceController | ||
| 20 | end | 20 | end |
| 21 | 21 | ||
| 22 | def show | 22 | def show |
| 23 | - @target_type = :merge_request | ||
| 24 | - @target_id = @merge_request.id | ||
| 25 | - | ||
| 26 | respond_to do |format| | 23 | respond_to do |format| |
| 27 | format.html | 24 | format.html |
| 28 | format.js | 25 | format.js |
| @@ -144,6 +141,9 @@ class MergeRequestsController < ProjectResourceController | @@ -144,6 +141,9 @@ class MergeRequestsController < ProjectResourceController | ||
| 144 | 141 | ||
| 145 | @allowed_to_merge = allowed_to_merge? | 142 | @allowed_to_merge = allowed_to_merge? |
| 146 | @show_merge_controls = @merge_request.opened? && @commits.any? && @allowed_to_merge | 143 | @show_merge_controls = @merge_request.opened? && @commits.any? && @allowed_to_merge |
| 144 | + | ||
| 145 | + @target_type = :merge_request | ||
| 146 | + @target_id = @merge_request.id | ||
| 147 | end | 147 | end |
| 148 | 148 | ||
| 149 | def allowed_to_merge? | 149 | def allowed_to_merge? |