Auto-merge: 'already can not be merged' alert

Valery Sizov
1 parent 8269a3a7
Showing 4 changed files with 25 additions and 5 deletions Show diff stats
app/assets/javascripts/merge_requests.js
app/controllers/merge_requests_controller.rb
app/views/merge_requests/automerge.js.haml
app/views/merge_requests/show.html.haml
@@ -7,7 +7,7 @@ var MergeRequest = {
     function(opts) {
       this.opts = opts;
-      if($(".automerge_widget").lenght){
+      if($(".automerge_widget").length){
         $.get(opts.url_to_automerge_check, function(data){
           $(".automerge_widget").hide();
           $(".automerge_widget." + data.state).show();
@@ -48,5 +48,11 @@ var MergeRequest = {
           $('.status').removeClass("loading");
         },
         dataType: "script"});
+    },
+
+  already_cannot_be_merged:
+    function(){
+        $(".automerge_widget").hide();
+        $(".automerge_widget.already_cannot_be_merged").show();
     }
 }
@@ -104,9 +104,13 @@ class MergeRequestsController &lt; ApplicationController
   end
   def automerge
-    render_404 unless @merge_request.open? && @merge_request.can_be_merged?
     return access_denied! unless can?(current_user, :accept_mr, @project)
-    @merge_request.automerge!(current_user)
+    if @merge_request.open? && @merge_request.can_be_merged?
+      @merge_request.automerge!(current_user)
+      @status = true
+    else
+      @status = false
+    end
   end
   def destroy
-:plain
-  location.reload();
+-if @status
+  :plain
+    location.reload();
+-else
+  :plain
+    MergeRequest.already_cannot_be_merged()
+
@@ -71,6 +71,11 @@
     .alert-message
       %p
         %strong Checking for ability to automatically merge… 
+  
+  .automerge_widget.already_cannot_be_merged{:style => "display:none"}
+    .alert-message
+      %p
+        %strong This merge request already can not be merged
1	-:plain
2	- location.reload();	1	+-if @status
		2	+ :plain
		3	+ location.reload();
		4	+-else
		5	+ :plain
		6	+ MergeRequest.already_cannot_be_merged()
		7	+