From 00ad82e07c19bcc0033b008f9f9c5c01d03552f4 Mon Sep 17 00:00:00 2001
From: Ethan Dalool <git@voussoir.net>
Date: Thu, 2 Apr 2020 16:25:19 -0700
Subject: [PATCH] Dedent this code by doing an early return instead of if.

---
 .../etiquette_flask/templates/search.html     | 48 +++++++++----------
 1 file changed, 24 insertions(+), 24 deletions(-)

diff --git a/frontends/etiquette_flask/templates/search.html b/frontends/etiquette_flask/templates/search.html
index ad8e172..5d10215 100644
--- a/frontends/etiquette_flask/templates/search.html
+++ b/frontends/etiquette_flask/templates/search.html
@@ -396,35 +396,35 @@ function add_searchtag(box, value, inputted_list, li_class)
             remove_searchtag(box, existing_tag, inputted_list);
         }
     }
-    if (!already_have)
-    {
-        inputted_list.push(value);
-        var new_li = document.createElement("li");
-        new_li.className = li_class;
+    if (already_have)
+    {return;}
 
-        var new_span = document.createElement("span");
-        new_span.className = "tag_object";
-        new_span.innerHTML = value;
+    inputted_list.push(value);
+    var new_li = document.createElement("li");
+    new_li.className = li_class;
 
-        var new_delbutton = document.createElement("button")
-        new_delbutton.classList.add("remove_tag_button");
-        new_delbutton.classList.add("red_button");
-        new_delbutton.onclick = function(){remove_searchtag(new_delbutton, value, inputted_list)};
+    var new_span = document.createElement("span");
+    new_span.className = "tag_object";
+    new_span.innerHTML = value;
 
-        new_li.appendChild(new_span);
-        new_li.appendChild(new_delbutton);
+    var new_delbutton = document.createElement("button")
+    new_delbutton.classList.add("remove_tag_button");
+    new_delbutton.classList.add("red_button");
+    new_delbutton.onclick = function(){remove_searchtag(new_delbutton, value, inputted_list)};
 
-        box_li = box.parentElement;
-        ul = box_li.parentElement;
-        ul.insertBefore(new_li, box_li);
+    new_li.appendChild(new_span);
+    new_li.appendChild(new_delbutton);
 
-        // The datalist autocomplete box can sometimes stick around after the
-        // tag has already been submitted and the input box moves down -- now
-        // covered by the autocomplete. So we temporarily unfocus it to make
-        // that thing go away.
-        box.blur();
-        box.focus();
-    }
+    box_li = box.parentElement;
+    ul = box_li.parentElement;
+    ul.insertBefore(new_li, box_li);
+
+    // The datalist autocomplete box can sometimes stick around after the
+    // tag has already been submitted and the input box moves down -- now
+    // covered by the autocomplete. So we temporarily unfocus it to make
+    // that thing go away.
+    box.blur();
+    box.focus();
 }
 function remove_searchtag(li_member, value, inputted_list)
 {