REFACTOR: removes various assignments and superfluous params (#9699)

discourse · May 8, 2020 · 00a038d · 00a038d
1 parent 9d172ee
commit 00a038d
Show file tree

Hide file tree

Showing 6 changed files with 5 additions and 7 deletions.
diff --git a/app/assets/javascripts/discourse/app/models/topic-tracking-state.js b/app/assets/javascripts/discourse/app/models/topic-tracking-state.js
@@ -452,14 +452,13 @@ const TopicTrackingState = EmberObject.extend({
     }
 
     let categoryId = category ? get(category, "id") : null;
-    let categoryName = category ? get(category, "name") : null;
 
     if (name === "new") {
       return this.countNew(categoryId);
     } else if (name === "unread") {
       return this.countUnread(categoryId);
     } else {
-      categoryName = name.split("/")[1];
+      const categoryName = name.split("/")[1];
       if (categoryName) {
         return this.countCategory(categoryId);
       }

diff --git a/app/assets/javascripts/locales/i18n.js b/app/assets/javascripts/locales/i18n.js
@@ -197,7 +197,6 @@ I18n.toNumber = function(number, options) {
       .toFixed(options.precision)
       .toString(),
     parts = string.split(this.SEPARATOR),
-    precision,
     buffer = [],
     formattedNumber;
 

diff --git a/app/assets/javascripts/pretty-text/addon/engines/discourse-markdown/bbcode-inline.js b/app/assets/javascripts/pretty-text/addon/engines/discourse-markdown/bbcode-inline.js
@@ -225,7 +225,7 @@ export function setup(helper) {
         token = state.push("text", "", 0);
         token.content = content;
 
-        token = state.push("link_close", "a", -1);
+        state.push("link_close", "a", -1);
         return true;
       }
     });

diff --git a/app/assets/javascripts/pretty-text/addon/engines/discourse-markdown/html-img.js b/app/assets/javascripts/pretty-text/addon/engines/discourse-markdown/html-img.js
@@ -66,7 +66,7 @@ function rule(state, startLine, endLine) {
   token.map = [startLine, state.line];
   token.children = [];
 
-  token = state.push("paragraph_close", "p", -1);
+  state.push("paragraph_close", "p", -1);
   state.parentType = oldParentType;
 
   return true;

diff --git a/app/assets/javascripts/pretty-text/addon/engines/discourse-markdown/paragraph.js b/app/assets/javascripts/pretty-text/addon/engines/discourse-markdown/paragraph.js
@@ -78,7 +78,7 @@ function paragraph(state, startLine /*, endLine*/) {
   // CUSTOM
   token.leading_space = hasLeadingSpace;
 
-  token = state.push("paragraph_close", "p", -1);
+  state.push("paragraph_close", "p", -1);
 
   state.parentType = oldParentType;
   return true;

diff --git a/app/assets/javascripts/pretty-text/addon/oneboxer.js b/app/assets/javascripts/pretty-text/addon/oneboxer.js
@@ -78,7 +78,7 @@ function loadNext(ajax) {
         let $html = $(html);
         setLocalCache(normalize(url), $html);
         $elem.replaceWith($html);
-        applySquareGenericOnebox($html, normalize(url));
+        applySquareGenericOnebox($html);
       },
       result => {
         if (result && result.jqXHR && result.jqXHR.status === 429) {