diff options
author | Peter Leitzen <pl@neopoly.de> | 2018-07-30 14:25:51 +0300 |
---|---|---|
committer | Peter Leitzen <pl@neopoly.de> | 2018-08-10 17:45:11 +0300 |
commit | fbd0f162524b6dd875097f76e1175a55dba5f3f6 (patch) | |
tree | f515ae38c688fea6a1da4b95beac9c26a8bea521 /app/services/commits | |
parent | 9c6fc59c6c95f8439e47d62eb4fd4b11f8d0acdc (diff) |
Let Commits::TagService return a result hash
Diffstat (limited to 'app/services/commits')
-rw-r--r-- | app/services/commits/tag_service.rb | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/app/services/commits/tag_service.rb b/app/services/commits/tag_service.rb index fb841655fee..7961ba4d3c4 100644 --- a/app/services/commits/tag_service.rb +++ b/app/services/commits/tag_service.rb @@ -3,7 +3,9 @@ module Commits class TagService < BaseService def execute(commit) - return unless params[:tag_name] + unless params[:tag_name] + return error('Missing parameter tag_name') + end tag_name = params[:tag_name] message = params[:tag_message] @@ -13,10 +15,12 @@ module Commits .new(commit.project, current_user) .execute(tag_name, commit.sha, message, release_description) - if result[:status] == :success && (tag = result[:tag]) + if result[:status] == :success + tag = result[:tag] SystemNoteService.tag_commit(commit, commit.project, current_user, tag.name) - commit end + + result end end end |