Implement third round of review comments from @DouweM.
Extract/mutate `params` in the `execute` method of the API services, rather than in `initialize`.
This commit is contained in:
parent
1051087ac4
commit
db0182e261
|
@ -4,22 +4,16 @@
|
|||
# lives in this service.
|
||||
module ProtectedBranches
|
||||
class ApiCreateService < BaseService
|
||||
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)
|
||||
end
|
||||
|
||||
def execute
|
||||
push_access_level =
|
||||
if @developers_can_push
|
||||
if params.delete(:developers_can_push)
|
||||
Gitlab::Access::DEVELOPER
|
||||
else
|
||||
Gitlab::Access::MASTER
|
||||
end
|
||||
|
||||
merge_access_level =
|
||||
if @developers_can_merge
|
||||
if params.delete(:developers_can_merge)
|
||||
Gitlab::Access::DEVELOPER
|
||||
else
|
||||
Gitlab::Access::MASTER
|
||||
|
|
|
@ -4,13 +4,10 @@
|
|||
# lives in this service.
|
||||
module ProtectedBranches
|
||||
class ApiUpdateService < BaseService
|
||||
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)
|
||||
end
|
||||
|
||||
def execute(protected_branch)
|
||||
@developers_can_push = params.delete(:developers_can_push)
|
||||
@developers_can_merge = params.delete(:developers_can_merge)
|
||||
|
||||
@protected_branch = protected_branch
|
||||
|
||||
protected_branch.transaction do
|
||||
|
|
Loading…
Reference in New Issue