SONAR-4303 fix canceling of comment edit form

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/controllers/issue_controller.rb b/sonar-server/src/main/webapp/WEB-INF/app/controllers/issue_controller.rb
index bc5ae43b0292ed5993601507faa6ff2358598cb5..ea985230a3faaefc7527b74726e2531be8fdcbb8 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/app/controllers/issue_controller.rb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/controllers/issue_controller.rb
@@ -39,7 +39,12 @@ class IssueController < ApplicationController
     if params[:modal]
       render :partial => 'issue/show_modal'
     elsif request.xhr?
-      render :partial => 'issue/show'
+      if params[:only_detail]
+        # used when canceling edition of comment -> see issue.js#refreshIssue()
+        render :partial => 'issue/issue', :locals => {:issue => @issue_results.first}
+      else
+        render :partial => 'issue/show'
+      end
     else
       render :action => 'show'
     end

diff --git a/sonar-server/src/main/webapp/javascripts/issue.js b/sonar-server/src/main/webapp/javascripts/issue.js
index f57267528b3dab999ebb5c2572dc33034b996976..410d37a1ed86ddef6958ce950012215bd189e5ca 100644 (file)
--- a/sonar-server/src/main/webapp/javascripts/issue.js
+++ b/sonar-server/src/main/webapp/javascripts/issue.js
@@ -52,10 +52,6 @@ function submitIssueForm(elt) {
       var issueKey = issueElt.attr('data-issue-key');
       var replaced = $j(htmlResponse);
       issueElt.replaceWith(replaced);
-
-      // re-enable the links opening modal popups
-      replaced.find('.open-modal').modal();
-
       notifyIssueChange(issueKey)
     }
   ).fail(function (jqXHR, textStatus) {
@@ -80,10 +76,6 @@ function doIssueAction(elt, action, parameters) {
   ).success(function (htmlResponse) {
       var replaced = $j(htmlResponse);
       issueElt.replaceWith(replaced);
-
-      // re-enable the links opening modal popups
-      replaced.find('.open-modal').modal();
-
       notifyIssueChange(issueKey);
     }
   ).fail(function (jqXHR, textStatus) {
@@ -140,10 +132,6 @@ function doEditIssueComment(elt) {
     success: function (htmlResponse) {
       var replaced = $j(htmlResponse);
       issueElt.replaceWith(replaced);
-
-      // re-enable the links opening modal popups
-      replaced.find('.open-modal').modal();
-
       notifyIssueChange(issueKey);
     }
   });
@@ -153,12 +141,9 @@ function doEditIssueComment(elt) {
 function refreshIssue(elt) {
   var issueElt = $j(elt).closest('[data-issue-key]');
   var issueKey = issueElt.attr('data-issue-key');
-  $j.get(baseUrl + "/issue/show/" + issueKey, function (html) {
+  $j.get(baseUrl + "/issue/show/" + issueKey + "?only_detail=true", function (html) {
     var replaced = $j(html);
     issueElt.replaceWith(replaced);
-
-    // re-enable the links opening modal popups
-    replaced.find('.open-modal').modal();
   });
   return false;
 }
@@ -191,9 +176,6 @@ function submitCreateIssueForm(elt) {
   ).success(function (html) {
       var replaced = $j(html);
       formElt.replaceWith(replaced);
-
-      // enable the links opening modal popups
-      replaced.find('.open-modal').modal();
     }
   ).error(function (jqXHR, textStatus, errorThrown) {
       var errorsElt = formElt.find('.code-issue-errors');