diff options
author | James Edwards-Jones <jedwardsjones@gitlab.com> | 2017-04-03 23:00:51 +0300 |
---|---|---|
committer | James Edwards-Jones <jedwardsjones@gitlab.com> | 2017-04-03 23:14:33 +0300 |
commit | 3c91841d032f02b0b0d4c532998bbc923247e804 (patch) | |
tree | 874423ab53847ef1bcd290284bbfe179e8493a43 /app/controllers/projects/protected_tags_controller.rb | |
parent | 9f4b8dba805915bd21d315f159035449f9f4bef0 (diff) |
Created ProtectedRefsController to reduce Tags/Branches duplication
Fixes ProtectedBranches#create flash errors bug due to
typo in 'flash[:alert] = @protected_branches.errors'
Diffstat (limited to 'app/controllers/projects/protected_tags_controller.rb')
-rw-r--r-- | app/controllers/projects/protected_tags_controller.rb | 58 |
1 files changed, 18 insertions, 40 deletions
diff --git a/app/controllers/projects/protected_tags_controller.rb b/app/controllers/projects/protected_tags_controller.rb index 5ab5d1d997b..8f407b42ac8 100644 --- a/app/controllers/projects/protected_tags_controller.rb +++ b/app/controllers/projects/protected_tags_controller.rb @@ -1,58 +1,36 @@ -class Projects::ProtectedTagsController < Projects::ApplicationController - include RepositorySettingsRedirect - # Authorize - before_action :require_non_empty_project - before_action :authorize_admin_project! - before_action :load_protected_tag, only: [:show, :update, :destroy] +class Projects::ProtectedTagsController < Projects::ProtectedRefsController - layout "project_settings" + protected - def index - redirect_to_repository_settings(@project) + def protected_ref + @protected_tag end - def create - @protected_tag = ::ProtectedTags::CreateService.new(@project, current_user, protected_tag_params).execute - unless @protected_tag.persisted? - flash[:alert] = @protected_tags.errors.full_messages.join(', ').html_safe - end - redirect_to_repository_settings(@project) + def protected_ref=(val) + @protected_tag = val end - def show - @matching_tags = @protected_tag.matching(@project.repository.tags) + def matching_refs=(val) + @matching_tags = val end - def update - @protected_tag = ::ProtectedTags::UpdateService.new(@project, current_user, protected_tag_params).execute(@protected_tag) - - if @protected_tag.valid? - respond_to do |format| - format.json { render json: @protected_tag, status: :ok } - end - else - respond_to do |format| - format.json { render json: @protected_tag.errors, status: :unprocessable_entity } - end - end + def project_refs + @project.repository.tags end - def destroy - @protected_tag.destroy - - respond_to do |format| - format.html { redirect_to_repository_settings(@project) } - format.js { head :ok } - end + def create_service + ::ProtectedTags::CreateService end - private + def update_service + ::ProtectedTags::UpdateService + end - def load_protected_tag - @protected_tag = @project.protected_tags.find(params[:id]) + def load_protected_ref + self.protected_ref = @project.protected_tags.find(params[:id]) end - def protected_tag_params + def protected_ref_params params.require(:protected_tag).permit(:name, push_access_levels_attributes: [:access_level, :id]) end end |