summaryrefslogtreecommitdiff
path: root/app/services/protected_branches/api_update_service.rb
diff options
context:
space:
mode:
Diffstat (limited to 'app/services/protected_branches/api_update_service.rb')
-rw-r--r--app/services/protected_branches/api_update_service.rb10
1 files changed, 5 insertions, 5 deletions
diff --git a/app/services/protected_branches/api_update_service.rb b/app/services/protected_branches/api_update_service.rb
index 1c27d32aad8..c28bffee2f4 100644
--- a/app/services/protected_branches/api_update_service.rb
+++ b/app/services/protected_branches/api_update_service.rb
@@ -4,10 +4,10 @@
# lives in this service.
module ProtectedBranches
class ApiUpdateService < BaseService
- def initialize(project, user, params, developers_can_push:, developers_can_merge:)
+ def initialize(project, user, params)
+ @developers_can_merge = params.delete(:developers_can_merge)
+ @developers_can_push = params.delete(:developers_can_push)
super(project, user, params)
- @developers_can_merge = developers_can_merge
- @developers_can_push = developers_can_push
end
def execute(protected_branch)
@@ -38,11 +38,11 @@ module ProtectedBranches
private
def delete_redundant_access_levels
- if @developers_can_merge || @developers_can_merge == false
+ unless @developers_can_merge.nil?
@protected_branch.merge_access_levels.destroy_all
end
- if @developers_can_push || @developers_can_push == false
+ unless @developers_can_push.nil?
@protected_branch.push_access_levels.destroy_all
end
end