Simplify if else to make code easier to understand

a26cc6b2 · Clement Ho · bf16e91f · a26cc6b2
Commit a26cc6b2 authored Dec 12, 2016 by Clement Ho
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 4 deletions

app/assets/javascripts/filtered_search/filtered_search_manager.js.es6 ...avascripts/filtered_search/filtered_search_manager.js.es6 +5 -4

No files found.
--- a/app/assets/javascripts/filtered_search/filtered_search_manager.js.es6
+++ b/app/assets/javascripts/filtered_search/filtered_search_manager.js.es6
@@ -137,11 +137,12 @@
        if (token.wildcard && condition) {
          tokenPath = condition.url;
-        } else if (!token.wildcard) {
+        } else if (token.wildcard) {
-          // Remove the wildcard token
+          // wildcard means that the token does not have a symbol
-          tokenPath = `${token.key}_${param}=${encodeURIComponent(token.value.slice(1))}`;
-        } else {
          tokenPath = `${token.key}_${param}=${encodeURIComponent(token.value)}`;
+        } else {
+          // Remove the token symbol
+          tokenPath = `${token.key}_${param}=${encodeURIComponent(token.value.slice(1))}`;
        }
        paths.push(tokenPath);