diff options
author | Robert Speicher <robert@gitlab.com> | 2018-09-14 20:38:29 +0300 |
---|---|---|
committer | Robert Speicher <robert@gitlab.com> | 2018-09-14 20:38:29 +0300 |
commit | 165e9a99c0363934e7676f3968e974af73c6cf95 (patch) | |
tree | ddeb4bbd45faaf3a8cd3efc8e28dd22c55a6f617 /app | |
parent | 69925767b7ac49b59c0292608b6655edb117e31d (diff) | |
parent | ebe89e8bd259a848d737b33a793298220054a023 (diff) |
Merge branch '50824-fix-prepend-concern' into 'master'
CE: Properly implement prepending for Concern
See merge request gitlab-org/gitlab-ce!21444
Diffstat (limited to 'app')
-rw-r--r-- | app/models/concerns/protected_branch_access.rb | 11 | ||||
-rw-r--r-- | app/models/concerns/protected_tag_access.rb | 3 |
2 files changed, 6 insertions, 8 deletions
diff --git a/app/models/concerns/protected_branch_access.rb b/app/models/concerns/protected_branch_access.rb index 744f7f48dc8..58761fce952 100644 --- a/app/models/concerns/protected_branch_access.rb +++ b/app/models/concerns/protected_branch_access.rb @@ -2,18 +2,17 @@ module ProtectedBranchAccess extend ActiveSupport::Concern + include ProtectedRefAccess included do - include ProtectedRefAccess - belongs_to :protected_branch delegate :project, to: :protected_branch + end - def check_access(user) - return false if access_level == Gitlab::Access::NO_ACCESS + def check_access(user) + return false if access_level == Gitlab::Access::NO_ACCESS - super - end + super end end diff --git a/app/models/concerns/protected_tag_access.rb b/app/models/concerns/protected_tag_access.rb index 04bd54d6b1c..3f5696c0749 100644 --- a/app/models/concerns/protected_tag_access.rb +++ b/app/models/concerns/protected_tag_access.rb @@ -2,10 +2,9 @@ module ProtectedTagAccess extend ActiveSupport::Concern + include ProtectedRefAccess included do - include ProtectedRefAccess - belongs_to :protected_tag delegate :project, to: :protected_tag |