only render MR description toggle javascript if description is available

1 files changed, 19 insertions, 20 deletions

diff --git a/app/views/shared/_commit_message_container.html.haml b/app/views/shared/_commit_message_container.html.haml
index 706eef5a331..a151731ba0a 100644
--- a/app/views/shared/_commit_message_container.html.haml
+++ b/app/views/shared/_commit_message_container.html.haml
@@ -14,7 +14,7 @@
       %p.hint
         Try to keep the first line under 52 characters
         and the others under 72.
-    - if local_assigns[:description]
+    - if local_assigns[:description] 
       %p.hint.use-description-hint
         = link_to "#", class: "use-description-link" do
           Use Merge Request description as merge commit message
@@ -22,29 +22,28 @@
         = link_to "#", class: "use-default-message-link" do
           Use default Gitlab merge commit message
 
+      :javascript
+        $('.use-description-link').on('click', function(e) {
+          e.preventDefault();
 
-  :javascript
-    $('.use-description-link').on('click', function(e) {
-      e.preventDefault();
+          var message = "Merge branch '#{j @merge_request.source_branch}' into '#{j @merge_request.target_branch}'\n\n"
+          message = message + "#{j @merge_request.title}\n\n"
+          message = message + "#{j local_assigns[:description]}\n\n";
+          message = message + "See merge request #{j @merge_request.to_reference}"
 
-      var message = "Merge branch '#{j @merge_request.source_branch}' into '#{j @merge_request.target_branch}'\n\n"
-      message = message + "#{j @merge_request.title}\n\n"
-      message = message + "#{j local_assigns[:description]}\n\n";
-      message = message + "See merge request #{j @merge_request.to_reference}"
 
+          $('.use-description-hint').hide();
+          $('.use-default-message-hint').show();
+          $('.js-commit-message').val(message)
+        });
 
-      $('.use-description-hint').hide();
-      $('.use-default-message-hint').show();
-      $('.js-commit-message').val(message)
-    });
+        $('.use-default-message-link').on('click', function(e) {
+          e.preventDefault();
 
-    $('.use-default-message-link').on('click', function(e) {
-      e.preventDefault();
+          var defaultMessage = "#{j (params[:commit_message] || local_assigns[:text] || local_assigns[:placeholder])}";
 
-      var defaultMessage = "#{j (params[:commit_message] || local_assigns[:text] || local_assigns[:placeholder])}";
-
-      $('.use-description-hint').show();
-      $('.use-default-message-hint').hide();
-      $('.js-commit-message').val(defaultMessage);
-    });
+          $('.use-description-hint').show();
+          $('.use-default-message-hint').hide();
+          $('.js-commit-message').val(defaultMessage);
+        });