summaryrefslogtreecommitdiff
path: root/app/controllers/merge_requests_controller.rb
diff options
context:
space:
mode:
authorDmitriy Zaporozhets <dzaporozhets@sphereconsultinginc.com>2012-06-13 09:03:53 +0300
committerDmitriy Zaporozhets <dzaporozhets@sphereconsultinginc.com>2012-06-13 09:03:53 +0300
commita243253b10244e8a3b62c40b686b52ac61a3adc8 (patch)
tree73f84c11c9591e3ad3b434d5c962367802277c69 /app/controllers/merge_requests_controller.rb
parentedd81a79c5c2a61e565664cfa787185c4e19729b (diff)
downloadgitlab-ce-2.6.0pre.tar.gz
Refactored project archive. Improved MR usability formv2.6.0pre
Diffstat (limited to 'app/controllers/merge_requests_controller.rb')
-rw-r--r--app/controllers/merge_requests_controller.rb31
1 files changed, 11 insertions, 20 deletions
diff --git a/app/controllers/merge_requests_controller.rb b/app/controllers/merge_requests_controller.rb
index 54227fc2c80..b0d122b55db 100644
--- a/app/controllers/merge_requests_controller.rb
+++ b/app/controllers/merge_requests_controller.rb
@@ -73,29 +73,21 @@ class MergeRequestsController < ApplicationController
@merge_request = @project.merge_requests.new(params[:merge_request])
@merge_request.author = current_user
- respond_to do |format|
- if @merge_request.save
- @merge_request.reload_code
- format.html { redirect_to [@project, @merge_request], notice: 'Merge request was successfully created.' }
- format.json { render json: @merge_request, status: :created, location: @merge_request }
- else
- format.html { render action: "new" }
- format.json { render json: @merge_request.errors, status: :unprocessable_entity }
- end
+ if @merge_request.save
+ @merge_request.reload_code
+ redirect_to [@project, @merge_request], notice: 'Merge request was successfully created.'
+ else
+ render action: "new"
end
end
def update
- respond_to do |format|
- if @merge_request.update_attributes(params[:merge_request].merge(:author_id_of_changes => current_user.id))
- @merge_request.reload_code
- @merge_request.mark_as_unchecked
- format.html { redirect_to [@project, @merge_request], notice: 'Merge request was successfully updated.' }
- format.json { head :ok }
- else
- format.html { render action: "edit" }
- format.json { render json: @merge_request.errors, status: :unprocessable_entity }
- end
+ if @merge_request.update_attributes(params[:merge_request].merge(:author_id_of_changes => current_user.id))
+ @merge_request.reload_code
+ @merge_request.mark_as_unchecked
+ redirect_to [@project, @merge_request], notice: 'Merge request was successfully updated.'
+ else
+ render action: "edit"
end
end
@@ -122,7 +114,6 @@ class MergeRequestsController < ApplicationController
respond_to do |format|
format.html { redirect_to project_merge_requests_url(@project) }
- format.json { head :ok }
end
end