From 298355f176ab94e2dfcb4807b114a444c944a8ef Mon Sep 17 00:00:00 2001
From: Will Gutierrez <will.gutierrez@gmail.com>
Date: Mon, 18 Jan 2021 15:26:14 -0800
Subject: [PATCH 01/13] write setState method with default parameters #9004

---
 app/assets/javascripts/editor.js | 31 +++++++++++++++----------------
 1 file changed, 15 insertions(+), 16 deletions(-)

diff --git a/app/assets/javascripts/editor.js b/app/assets/javascripts/editor.js
index 116c166f2c..b3782d9b4a 100644
--- a/app/assets/javascripts/editor.js
+++ b/app/assets/javascripts/editor.js
@@ -4,29 +4,22 @@ $(function() {
   // on pages with multiple comments, $D.selected needs to be accurate so that rich-text changes (bold, italic, etc.) go into the right comment form
   // however, the editor is also used on pages with JUST ONE form, and no other comments, eg. /wiki/new & /wiki/edit, so this code needs to be reusable for that context
   $('.rich-text-button').on('click', function(e) {
-    const params = getEditorParams(e.target); // defined in editorHelper.js
+    const { textArea, preview, dSelected } = getEditorParams(e.target); // defined in editorHelper.js
     // assign dSelected
-    if (params.hasOwnProperty('dSelected')) {
-      $D.selected = params['dSelected'];
-    }
-    $E.initialize(params);
+    if (dSelected) { $D.selected = dSelected; }
+    $E.setState(textArea, preview);
     const action = e.currentTarget.dataset.action // 'bold', 'italic', etc.
     $E[action](); // call the appropriate editor function
   });
 });
 
 $E = {
-  initialize: function(args) {
-    args = args || {}
-    // title
-    $E.title = $('#title')
-    // textarea
-    args['textarea'] = args['textarea'] || 'text-input'
-    $E.textarea = $('#'+args['textarea'])
-    $E.textarea.bind('input propertychange',$E.save)
-    // preview
-    args['preview'] = args['preview'] || 'preview-main'
-    $E.preview = $('#'+args['preview'])
+  initialize: function() {
+    // call setState with no parameters, aka. default parameters.
+    // default parameters point toward either:
+    //   1. the comment form at the bottom of multi-comment wikis/questions/research notes
+    //   2. the only editor form on /wiki/new and /wiki/edit
+    $E.setState();
     
     marked.setOptions({
       gfm: true,
@@ -44,6 +37,12 @@ $E = {
       }
     });
   },
+  setState: function(textarea = 'text-input', preview = 'preview-main', title = 'title') {
+    $E.title = $('#' + title + 'title'); // not sure why this exists? seems like $E.title is always #title
+    $E.textarea = $('#' + textarea);
+    $E.textarea.bind('input propertychange', $E.save);
+    $E.preview = $('#' + preview);
+  },
   is_editing: function() {
     return ($E.textarea[0].selectionStart == 0 && $E.textarea[0].selectionEnd == 0)
   },

From dda4d2d63c65c039548a43dd78d5ab6aac34a859 Mon Sep 17 00:00:00 2001
From: Will Gutierrez <will.gutierrez@gmail.com>
Date: Mon, 18 Jan 2021 15:27:44 -0800
Subject: [PATCH 02/13] set preview when calling E.setState #9004

---
 app/assets/javascripts/dragdrop.js | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/app/assets/javascripts/dragdrop.js b/app/assets/javascripts/dragdrop.js
index a7dba36628..64e79d4019 100644
--- a/app/assets/javascripts/dragdrop.js
+++ b/app/assets/javascripts/dragdrop.js
@@ -29,12 +29,10 @@ jQuery(function() {
     });
 
     $(this).on('drop',function(e) {
-      const params = getEditorParams(e.target); // defined in editorHelper.js
+      const { textArea, preview, dSelected } = getEditorParams(e.target); // defined in editorHelper.js
       e.preventDefault();
-      if (params.hasOwnProperty('dSelected')) {
-        $D.selected = params['dSelected'];
-      }
-      $E.initialize(params);
+      if (dSelected) { $D.selected = dSelected; }
+      $E.setState(textArea, preview);
     });
 
     $(this).fileupload({

From a4ae7c3ff2f49ec3856180ba39cb2d4d891870bd Mon Sep 17 00:00:00 2001
From: Will Gutierrez <will.gutierrez@gmail.com>
Date: Mon, 18 Jan 2021 15:28:21 -0800
Subject: [PATCH 03/13] set preview when calling E.setState #9004

---
 app/assets/javascripts/editorHelper.js | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/app/assets/javascripts/editorHelper.js b/app/assets/javascripts/editorHelper.js
index 5204aeaefd..d38559f0bb 100644
--- a/app/assets/javascripts/editorHelper.js
+++ b/app/assets/javascripts/editorHelper.js
@@ -8,10 +8,13 @@ const getEditorParams = (targetDiv) => {
     params['dSelected'] = $(closestCommentFormWrapper);
     // assign the ID of the textarea within the closest comment-form-wrapper
     params['textarea'] = closestCommentFormWrapper.querySelector('textarea').id;
+    params['preview'] = closestCommentFormWrapper.querySelector('.comment-preview').id;
   } else {
     // default to #text-input
     // #text-input ID should be unique, and the only comment form on /wiki/new & /wiki/edit
     params['textarea'] = 'text-input';
+    // #preview-main should be unique as well
+    params['preview'] = 'preview-main';
   }
   return params;
 };
\ No newline at end of file

From aff84c9afe4e7c59df877c59a231993387d138a6 Mon Sep 17 00:00:00 2001
From: Will Gutierrez <will.gutierrez@gmail.com>
Date: Mon, 18 Jan 2021 15:28:58 -0800
Subject: [PATCH 04/13] change E.initialize to E.setState; add comment-preview
 class #9004

---
 app/views/comments/_edit.html.erb | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/app/views/comments/_edit.html.erb b/app/views/comments/_edit.html.erb
index 71196006a0..10e2204b44 100644
--- a/app/views/comments/_edit.html.erb
+++ b/app/views/comments/_edit.html.erb
@@ -40,11 +40,9 @@
     </div>
       <script type="application/javascript">
         function setInit(id) {
-          var args = {
-            'textarea': 'c'+id+'text',
-            'preview': 'c'+id+'preview'
-          }
-          $E.initialize(args);
+          const textArea = 'c'+id+'text';
+          const preview = 'c'+id+'preview';
+          $E.setState(textArea, preview);
         }
 
         $('#c<%= comment.id%>div').on('dragover',function(e) {
@@ -115,7 +113,7 @@
         });
       </script>
 
-      <div class="well col-md-11" id="c<%= comment.id %>preview" style="background:white;display: none">
+      <div class="comment-preview well col-md-11" id="c<%= comment.id %>preview" style="background:white;display: none">
       </div>
 
       <div class="control-group">

From 7a6419004ea2aa5111fb3bac245dc1a66b6dec3e Mon Sep 17 00:00:00 2001
From: Will Gutierrez <will.gutierrez@gmail.com>
Date: Mon, 18 Jan 2021 15:29:29 -0800
Subject: [PATCH 05/13] change E.initialize({}) to E.initialize() #9004

---
 app/views/editor/_editor.html.erb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/views/editor/_editor.html.erb b/app/views/editor/_editor.html.erb
index 0bc42acf7c..870edc2da6 100644
--- a/app/views/editor/_editor.html.erb
+++ b/app/views/editor/_editor.html.erb
@@ -21,6 +21,6 @@
 
 <script>
   jQuery(document).ready(function() {
-    $E.initialize({})
+    $E.initialize();
   })
 </script>

From e0b148bf936ec401b1e8f8acd75be7a6e9765f0a Mon Sep 17 00:00:00 2001
From: Will Gutierrez <will.gutierrez@gmail.com>
Date: Mon, 18 Jan 2021 16:17:44 -0800
Subject: [PATCH 06/13] change regex test to match /features/new in wrap #9004

---
 app/assets/javascripts/editor.js | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

diff --git a/app/assets/javascripts/editor.js b/app/assets/javascripts/editor.js
index b3782d9b4a..25b3852037 100644
--- a/app/assets/javascripts/editor.js
+++ b/app/assets/javascripts/editor.js
@@ -55,10 +55,15 @@ $E = {
   },
   // wraps currently selected text in textarea with strings a and b
   wrap: function(a,b,args) {
-    // this RegEx is: /wiki/ + any char not "/" + /edit
-    const isWikiCommentPage = (/\/wiki\/[^\/]+\/edit/).test(window.location.pathname);
-    // we don't need to refresh $E's values if we're on a page with a single comment form, ie. /wiki/new or /wiki/edit
-    if (window.location.pathname !== "/wiki/new" && !isWikiCommentPage && $D.selected) {
+    // this RegEx matches three different cases where the legacy editor is still used:
+    //   1. /wiki/new
+    //   2. /wiki/{wiki name}/edit
+    //   3. /features/new
+    const isLegacyEditorPath = RegExp(/\/(wiki|features)(\/[^\/]+\/edit|\/new)/);
+    const isLegacyEditorPage = isLegacyEditorPath.test(window.location.pathname);
+    // we don't need to refresh $E's values if we're on a page where the legacy editor is still used.
+    // this is because these pages only have one editor form, unlike pages with multiple comments.
+    if (!isLegacyEditorPage && $D.selected) {
       this.refresh();
     }
     var len = $E.textarea.val().length;

From 2af718b86acc31941356a55a5458c8b8a4b579b7 Mon Sep 17 00:00:00 2001
From: Will Gutierrez <will.gutierrez@gmail.com>
Date: Mon, 18 Jan 2021 16:18:17 -0800
Subject: [PATCH 07/13] rewrite selector for more specificity #9004

---
 test/system/comment_test.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/test/system/comment_test.rb b/test/system/comment_test.rb
index 9ea162bfe2..d23fae8fd9 100644
--- a/test/system/comment_test.rb
+++ b/test/system/comment_test.rb
@@ -365,7 +365,7 @@ def get_path(page_type, path)
       page.find('#c' + edit_id_num + 'edit a', text: 'Preview').click
       page.first('a', text: 'Preview').click
       assert_selector('#c' + edit_id_num + 'preview img', count: 1)
-      assert_selector('.comment-preview img', count: 1)
+      assert_selector('#preview-reply-' + reply_id_num, count: 1)
     end
 
     test "#{page_type_string}: ctrl/cmd + enter comment publishing keyboard shortcut" do

From 07843783c02dfc640f44ac6452216bf0e4a865a6 Mon Sep 17 00:00:00 2001
From: Will Gutierrez <will.gutierrez@gmail.com>
Date: Mon, 18 Jan 2021 20:00:49 -0800
Subject: [PATCH 08/13] rewrite for cognitive complexity #9004

---
 app/assets/javascripts/editor.js | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/app/assets/javascripts/editor.js b/app/assets/javascripts/editor.js
index 25b3852037..acca84f118 100644
--- a/app/assets/javascripts/editor.js
+++ b/app/assets/javascripts/editor.js
@@ -59,11 +59,12 @@ $E = {
     //   1. /wiki/new
     //   2. /wiki/{wiki name}/edit
     //   3. /features/new
-    const isLegacyEditorPath = RegExp(/\/(wiki|features)(\/[^\/]+\/edit|\/new)/);
-    const isLegacyEditorPage = isLegacyEditorPath.test(window.location.pathname);
-    // we don't need to refresh $E's values if we're on a page where the legacy editor is still used.
-    // this is because these pages only have one editor form, unlike pages with multiple comments.
-    if (!isLegacyEditorPage && $D.selected) {
+    const url = window.location.pathname;
+    const legacyEditorPath = RegExp(/\/(wiki|features)(\/[^\/]+\/edit|\/new)/);
+    const isRichTextEditor = !legacyEditorPath.test(url);
+    // we only refresh $E's values if with the rich-text editor (most pages).
+    // the legacy editor pages only have one editor form, unlike pages with multiple comments.
+    if (isRichTextEditor && $D.selected) {
       this.refresh();
     }
     var len = $E.textarea.val().length;

From aacb49436d5e9873f8b280c4a9c41e8d7b17ae55 Mon Sep 17 00:00:00 2001
From: Will Gutierrez <will.gutierrez@gmail.com>
Date: Mon, 18 Jan 2021 20:21:18 -0800
Subject: [PATCH 09/13] desperately trying to pass codeclimate #9004

---
 app/assets/javascripts/editor.js | 15 +++++++--------
 1 file changed, 7 insertions(+), 8 deletions(-)

diff --git a/app/assets/javascripts/editor.js b/app/assets/javascripts/editor.js
index acca84f118..99679dbf2f 100644
--- a/app/assets/javascripts/editor.js
+++ b/app/assets/javascripts/editor.js
@@ -53,20 +53,19 @@ $E = {
     // preview
     $E.preview = ($D.selected).find('.comment-preview').eq(0);
   },
-  // wraps currently selected text in textarea with strings a and b
-  wrap: function(a,b,args) {
+  isRichTextEditor: function(url) {
     // this RegEx matches three different cases where the legacy editor is still used:
     //   1. /wiki/new
     //   2. /wiki/{wiki name}/edit
     //   3. /features/new
-    const url = window.location.pathname;
     const legacyEditorPath = RegExp(/\/(wiki|features)(\/[^\/]+\/edit|\/new)/);
-    const isRichTextEditor = !legacyEditorPath.test(url);
-    // we only refresh $E's values if with the rich-text editor (most pages).
+    return !legacyEditorPath.test(url); // if we're not on one of these pages, we are using the rich-text editor.
+  },
+  // wraps currently selected text in textarea with strings a and b
+  wrap: function(a, b, args) {
+    // we only refresh $E's values if we are on a page using the rich-text editor (most pages).
     // the legacy editor pages only have one editor form, unlike pages with multiple comments.
-    if (isRichTextEditor && $D.selected) {
-      this.refresh();
-    }
+    this.isRichTextEditor(window.location.pathname) && this.refresh();
     var len = $E.textarea.val().length;
     var start = $E.textarea[0].selectionStart;
     var end = $E.textarea[0].selectionEnd;

From 16447e23667a142b96a8a676880d1e94b7dac938 Mon Sep 17 00:00:00 2001
From: Will Gutierrez <will.gutierrez@gmail.com>
Date: Mon, 18 Jan 2021 20:26:59 -0800
Subject: [PATCH 10/13] desperately trying to pass codeclimate #9004

---
 app/assets/javascripts/editor.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/assets/javascripts/editor.js b/app/assets/javascripts/editor.js
index 99679dbf2f..332e554e64 100644
--- a/app/assets/javascripts/editor.js
+++ b/app/assets/javascripts/editor.js
@@ -65,7 +65,7 @@ $E = {
   wrap: function(a, b, args) {
     // we only refresh $E's values if we are on a page using the rich-text editor (most pages).
     // the legacy editor pages only have one editor form, unlike pages with multiple comments.
-    this.isRichTextEditor(window.location.pathname) && this.refresh();
+    if (this.isRichTextEditor(window.location.pathname)) { this.refresh(); }
     var len = $E.textarea.val().length;
     var start = $E.textarea[0].selectionStart;
     var end = $E.textarea[0].selectionEnd;

From ca93f146fb63dfcbf2ec98b3997604a75193d8ec Mon Sep 17 00:00:00 2001
From: Will Gutierrez <will.gutierrez@gmail.com>
Date: Mon, 18 Jan 2021 20:49:26 -0800
Subject: [PATCH 11/13] desperately trying to pass codeclimate #9004

---
 app/assets/javascripts/editor.js | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/app/assets/javascripts/editor.js b/app/assets/javascripts/editor.js
index 332e554e64..f90b4f4b01 100644
--- a/app/assets/javascripts/editor.js
+++ b/app/assets/javascripts/editor.js
@@ -69,10 +69,7 @@ $E = {
     var len = $E.textarea.val().length;
     var start = $E.textarea[0].selectionStart;
     var end = $E.textarea[0].selectionEnd;
-    var sel = $E.textarea.val().substring(start, end);
-    if (args && args['fallback']) { // an alternative if nothing has been selected, but we're simply dealing with an insertion point
-      sel = args['fallback']
-    }
+    var sel = (args && args['fallback']) ? $E.textarea.val().substring(start, end) : args['fallback']; // // fallback if nothing has been selected, and we're simply dealing with an insertion point
     var replace = a + sel + b;
     if (args && args['newline']) {
       if ($E.textarea[0].selectionStart > 0) replace = "\n"+replace

From f44a6bbfc0c29db63c0a896f81e4445206a2b57b Mon Sep 17 00:00:00 2001
From: Will Gutierrez <will.gutierrez@gmail.com>
Date: Mon, 18 Jan 2021 20:57:31 -0800
Subject: [PATCH 12/13] desperately trying to pass codeclimate #9004

---
 app/assets/javascripts/editor.js | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/app/assets/javascripts/editor.js b/app/assets/javascripts/editor.js
index f90b4f4b01..f17b75d61d 100644
--- a/app/assets/javascripts/editor.js
+++ b/app/assets/javascripts/editor.js
@@ -69,11 +69,15 @@ $E = {
     var len = $E.textarea.val().length;
     var start = $E.textarea[0].selectionStart;
     var end = $E.textarea[0].selectionEnd;
-    var sel = (args && args['fallback']) ? $E.textarea.val().substring(start, end) : args['fallback']; // // fallback if nothing has been selected, and we're simply dealing with an insertion point
+    const fallbackParameterExists = args && args['fallback'];
+    const newlineParameterExists = args && args['newline'];
+    var sel = fallbackParameterExists ? $E.textarea.val().substring(start, end) : args['fallback']; // // fallback if nothing has been selected, and we're simply dealing with an insertion point
     var replace = a + sel + b;
-    if (args && args['newline']) {
-      if ($E.textarea[0].selectionStart > 0) replace = "\n"+replace
-      replace = replace+"\n\n"
+    if (newlineParameterExists) {
+      replace = replace + "\n\n";
+    }
+    if (newlineParameterExists && $E.textarea[0].selectionStart > 0) { 
+      replace = "\n" + replace; 
     }
     $E.textarea.val($E.textarea.val().substring(0,start) + replace + $E.textarea.val().substring(end,len));
   },

From 2f04589028191a49a31b7981d5842deb1d4631f5 Mon Sep 17 00:00:00 2001
From: Will Gutierrez <will.gutierrez@gmail.com>
Date: Mon, 18 Jan 2021 21:26:57 -0800
Subject: [PATCH 13/13] oops, switched ternary operator #9004

---
 app/assets/javascripts/editor.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/assets/javascripts/editor.js b/app/assets/javascripts/editor.js
index f17b75d61d..11cbcb5e7e 100644
--- a/app/assets/javascripts/editor.js
+++ b/app/assets/javascripts/editor.js
@@ -71,7 +71,7 @@ $E = {
     var end = $E.textarea[0].selectionEnd;
     const fallbackParameterExists = args && args['fallback'];
     const newlineParameterExists = args && args['newline'];
-    var sel = fallbackParameterExists ? $E.textarea.val().substring(start, end) : args['fallback']; // // fallback if nothing has been selected, and we're simply dealing with an insertion point
+    var sel = fallbackParameterExists ? args['fallback'] : $E.textarea.val().substring(start, end); // // fallback if nothing has been selected, and we're simply dealing with an insertion point
     var replace = a + sel + b;
     if (newlineParameterExists) {
       replace = replace + "\n\n";