diff options
author | Rémy Coutable <remy@rymai.me> | 2018-12-18 15:05:52 +0000 |
---|---|---|
committer | Rémy Coutable <remy@rymai.me> | 2018-12-18 15:05:52 +0000 |
commit | 4e8450aff6125ba48e7460db59a6341c9d680ad7 (patch) | |
tree | 606362a281619bd820099d4ab18cd484eeffedf1 /app/models/merge_request.rb | |
parent | c5a13265f2126919560b18d981754e6eb7f79411 (diff) | |
parent | 0d1922dd7c08eb098390199af3bc7a3e92812a60 (diff) | |
download | gitlab-ce-4e8450aff6125ba48e7460db59a6341c9d680ad7.tar.gz |
Merge branch 'force-reload-arguments-2' into 'master'
Passing an argument to force an association to reload is now deprecated
See merge request gitlab-org/gitlab-ce!23894
Diffstat (limited to 'app/models/merge_request.rb')
-rw-r--r-- | app/models/merge_request.rb | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/app/models/merge_request.rb b/app/models/merge_request.rb index 8052a54c504..6f5ea6e29ab 100644 --- a/app/models/merge_request.rb +++ b/app/models/merge_request.rb @@ -48,8 +48,8 @@ class MergeRequest < ActiveRecord::Base # is the inverse of MergeRequest#merge_request_diff, which means it may not be # the latest diff, because we could have loaded any diff from this particular # MR. If we haven't already loaded a diff, then it's fine to load the latest. - def merge_request_diff(*args) - fallback = latest_merge_request_diff if args.empty? && !association(:merge_request_diff).loaded? + def merge_request_diff + fallback = latest_merge_request_diff unless association(:merge_request_diff).loaded? fallback || super end @@ -620,10 +620,6 @@ class MergeRequest < ActiveRecord::Base end end - def reload_merge_request_diff - merge_request_diff(true) - end - def viewable_diffs @viewable_diffs ||= merge_request_diffs.viewable.to_a end |