Code style

Charcoal-SE · Sep 1, 2018 · d81d7a4 · d81d7a4
1 parent 38fc050
commit d81d7a4
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 21 deletions.
diff --git a/app/controllers/domain_tags_controller.rb b/app/controllers/domain_tags_controller.rb
@@ -31,9 +31,9 @@ def add_review
       @domain.domain_tags.map(&:id).include?(dt.id) ? nil : dt
     end.reject(&:nil?)
     if @domain.save
-      render json: { status: 200, message: "Successfully updated tags" }, status: 200
+      render json: { status: 200, message: 'Successfully updated tags' }, status: 200
     else
-      render text: "Failed to update tags", status: 500
+      render text: 'Failed to update tags', status: 500
     end
   end
 

diff --git a/app/javascript/review.js b/app/javascript/review.js
@@ -23,13 +23,13 @@ route(/\/review\/[\w-]+\/?\d*$/i, async () => {
     installSelectpickers();
 
     if (queuePath.includes('untagged-domains')) {
-      let a = document.getElementsByName('tag_name');
-      for (let i = 0; i < a.length; i++) {
-        Taggify.taggify_element(a[i]);
-        a[i].parentElement.addEventListener('tag_change', function() {
-          let form = $(this.parentElement);
-          let submit_btn = form.find('input[type=submit]');
-          submit_btn.removeClass('btn-success');
+      const taggify_inputs = document.getElementsByName('tag_name');
+      for (let i = 0; i < taggify_inputs.length; i++) {
+        Taggify.taggify_element(taggify_inputs[i]);
+        taggify_inputs[i].parentElement.addEventListener('tag_change', function () {
+          const form = $(this.parentElement);
+          const submitBtn = form.find('input[type=submit]');
+          submitBtn.removeClass('btn-success');
         });
       }
     }
@@ -51,18 +51,18 @@ route(/\/review\/[\w-]+\/?\d*$/i, async () => {
 });
 
 route(/\/review\/untagged-domains(\/\d*)?/, () => {
-  $(document).on('ajax:success', '.review-add-domain-tag', (e) => {
-    let a = document.getElementById('tag_name');
-    a.removeAttribute('disabled');
-    let form = $(a.parentElement.parentElement);
-    let submit_btn = form.find('input[type=submit]');
-    submit_btn.addClass('btn-success');
+  $(document).on('ajax:success', '.review-add-domain-tag', () => {
+    const taggify_input = document.getElementById('tag_name');
+    taggify_input.removeAttribute('disabled');
+    const form = $(taggify_input.parentElement.parentElement);
+    const submitBtn = form.find('input[type=submit]');
+    submitBtn.addClass('btn-success');
   });
-  $(document).on('ajax:failure', '.review-add-domain-tag', (e) => {
-    let a = document.getElementById('tag_name');
-    a.removeAttribute('disabled');
-    let form = $(a.parentElement.parentElement);
-    let submit_btn = form.find('input[type=submit]');
-    submit_btn.addClass('btn-danger');
+  $(document).on('ajax:failure', '.review-add-domain-tag', () => {
+    const taggify_input = document.getElementById('tag_name');
+    taggify_input.removeAttribute('disabled');
+    const form = $(taggify_input.parentElement.parentElement);
+    const submitBtn = form.find('input[type=submit]');
+    submitBtn.addClass('btn-danger');
   });
 });