diff options
author | Stan Hu <stanhu@gmail.com> | 2016-04-07 07:10:24 +0300 |
---|---|---|
committer | Stan Hu <stanhu@gmail.com> | 2016-04-07 07:11:10 +0300 |
commit | 924e4b370013ae3a8448a2a68e6a4d353b23b699 (patch) | |
tree | a39ea1e7e6a0b11d3a04cc963e8525e3c0df9762 /app/controllers/projects/application_controller.rb | |
parent | 8f0945311b8f5fe30b0b57739ee9f3262499f375 (diff) |
Return status code 303 after a branch DELETE operation to avoid project deletion
Closes #14994
Diffstat (limited to 'app/controllers/projects/application_controller.rb')
-rw-r--r-- | app/controllers/projects/application_controller.rb | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/app/controllers/projects/application_controller.rb b/app/controllers/projects/application_controller.rb index 657ee94cfd7..74150ad606b 100644 --- a/app/controllers/projects/application_controller.rb +++ b/app/controllers/projects/application_controller.rb @@ -68,7 +68,9 @@ class Projects::ApplicationController < ApplicationController end def require_non_empty_project - redirect_to namespace_project_path(@project.namespace, @project) if @project.empty_repo? + # Be sure to return status code 303 to avoid a double DELETE: + # http://api.rubyonrails.org/classes/ActionController/Redirecting.html + redirect_to namespace_project_path(@project.namespace, @project), status: 303 if @project.empty_repo? end def require_branch_head |