diff options
author | Robert Schilling <rschilling@student.tugraz.at> | 2019-01-06 20:31:37 +0100 |
---|---|---|
committer | Robert Schilling <rschilling@student.tugraz.at> | 2019-01-31 13:49:52 +0100 |
commit | 0ce33f6b4f30ddab14025adb16138f1589d32b6f (patch) | |
tree | 35edfe95e7690aecbcb1e01a6586a43c9b603d7c /lib/api/group_labels.rb | |
parent | a9fdc3118a7f9fb55b6f6b243f7bed2abe1ce48f (diff) | |
download | gitlab-ce-0ce33f6b4f30ddab14025adb16138f1589d32b6f.tar.gz |
Factor out common label API
Diffstat (limited to 'lib/api/group_labels.rb')
-rw-r--r-- | lib/api/group_labels.rb | 55 |
1 files changed, 15 insertions, 40 deletions
diff --git a/lib/api/group_labels.rb b/lib/api/group_labels.rb index 559e6a25f35..3b74dbebb3e 100644 --- a/lib/api/group_labels.rb +++ b/lib/api/group_labels.rb @@ -2,6 +2,7 @@ module API class GroupLabels < Grape::API + include ::API::Helpers::LabelHelpers include PaginationParams before { authenticate! } @@ -18,9 +19,7 @@ module API use :pagination end get ':id/labels' do - group_labels = available_labels_for(user_group) - - present paginate(group_labels), with: Entities::GroupLabel, current_user: current_user, parent: user_group + get_labels(user_group, Entities::GroupLabel) end desc 'Create a new label' do @@ -28,38 +27,10 @@ module API success Entities::GroupLabel end params do - requires :name, type: String, desc: 'The name of the label to be created' - requires :color, type: String, desc: "The color of the label given in 6-digit hex notation with leading '#' sign (e.g. #FFAABB) or one of the allowed CSS color names" - optional :description, type: String, desc: 'The description of label to be created' + use :label_create_params end post ':id/labels' do - authorize! :admin_label, user_group - - label = available_labels_for(user_group).find_by_title(params[:name]) - conflict!('Label already exists') if label - - label = ::Labels::CreateService.new(declared_params(include_missing: false)).execute(group: user_group) - - if label.persisted? - present label, with: Entities::GroupLabel, current_user: current_user, parent: user_group - else - render_validation_error!(label) - end - end - - desc 'Delete an existing label' do - detail 'This feature was added in GitLab 11.7' - success Entities::GroupLabel - end - params do - requires :name, type: String, desc: 'The name of the label to be deleted' - end - delete ':id/labels' do - authorize! :admin_label, user_group - - label = find_label(user_group, params[:name], include_ancestor_groups: false) - - destroy_conditionally!(label) + create_label(user_group, Entities::GroupLabel) end desc 'Update an existing label. At least one optional parameter is required.' do @@ -74,14 +45,18 @@ module API at_least_one_of :new_name, :color, :description end put ':id/labels' do - authorize! :admin_label, user_group - - label = find_label(user_group, params[:name], include_ancestor_groups: false) - - label = ::Labels::UpdateService.new(declared_params(include_missing: false)).execute(label) - render_validation_error!(label) unless label.valid? + update_label(user_group, Entities::GroupLabel) + end - present label, with: Entities::GroupLabel, current_user: current_user, parent: user_group + desc 'Delete an existing label' do + detail 'This feature was added in GitLab 11.7' + success Entities::GroupLabel + end + params do + requires :name, type: String, desc: 'The name of the label to be deleted' + end + delete ':id/labels' do + delete_label(user_group) end end end |