diff options
author | Ciro Santilli <ciro.santilli@gmail.com> | 2014-09-21 12:29:52 +0400 |
---|---|---|
committer | Ciro Santilli <ciro.santilli@gmail.com> | 2014-09-21 13:43:05 +0400 |
commit | ad47993ac46cef672600f2384ee5fa2e661ec8be (patch) | |
tree | 07769b4efeb4c18762b3ed0e705179a5423926c1 /app/services/delete_branch_service.rb | |
parent | fda61a047ffb9b04bc4dd38e897088fde17fb3c1 (diff) |
Factor error and success methods from services.
Diffstat (limited to 'app/services/delete_branch_service.rb')
-rw-r--r-- | app/services/delete_branch_service.rb | 21 |
1 files changed, 10 insertions, 11 deletions
diff --git a/app/services/delete_branch_service.rb b/app/services/delete_branch_service.rb index a94dabcdfc0..cae6327fe72 100644 --- a/app/services/delete_branch_service.rb +++ b/app/services/delete_branch_service.rb @@ -1,5 +1,7 @@ -class DeleteBranchService - def execute(project, branch_name, current_user) +require_relative 'base_service' + +class DeleteBranchService < BaseService + def execute(branch_name) repository = project.repository branch = repository.find_branch(branch_name) @@ -31,17 +33,14 @@ class DeleteBranchService end def error(message, return_code = 400) - { - message: message, - return_code: return_code, - state: :error - } + out = super(message) + out[:return_code] = return_code + out end def success(message) - { - message: message, - state: :success - } + out = super() + out[:message] = message + out end end |