Skip to content

Commit

Permalink
Remove delete_dialog from patchinfo controller
Browse files Browse the repository at this point in the history
Its specs and route were also removed. This action isn't needed anymore
in the Bootstrap UI
  • Loading branch information
dmarcoux committed Sep 5, 2019
1 parent 54d6c1c commit 3d3c077
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 14 deletions.
4 changes: 0 additions & 4 deletions src/api/app/controllers/webui/patchinfo_controller.rb
Original file line number Diff line number Diff line change
Expand Up @@ -82,10 +82,6 @@ def destroy
end
end

def delete_dialog
render_dialog
end

def update_issues
authorize @project, :update?

Expand Down
1 change: 0 additions & 1 deletion src/api/config/routes.rb
Original file line number Diff line number Diff line change
Expand Up @@ -170,7 +170,6 @@ def self.public_or_about_path?(request)
post 'patchinfo/update_issues/:project/:package' => :update_issues, as: 'update_issues_patchinfo'
get 'patchinfo/show/:project/:package' => :show, as: 'patchinfo_show', constraints: cons, defaults: { format: 'html' }
get 'patchinfo/new_tracker' => :new_tracker
get 'patchinfo/delete_dialog' => :delete_dialog
end

controller 'webui/repositories' do
Expand Down
9 changes: 0 additions & 9 deletions src/api/spec/controllers/webui/patchinfo_controller_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -317,13 +317,4 @@ def do_proper_post_save
it { expect(response).to have_http_status(:success) }
end
end

describe 'GET #delete_dialog' do
before do
login user
get :delete_dialog, xhr: true, params: { project: user.home_project_name, package: patchinfo_package.name }
end

it { expect(response).to have_http_status(:success) }
end
end

0 comments on commit 3d3c077

Please sign in to comment.