From aebee11884d2176a45cb17efa97bf3fdbc95449a Mon Sep 17 00:00:00 2001
From: Clement Ho <ClemMakesApps@gmail.com>
Date: Mon, 12 Dec 2016 10:55:43 -0600
Subject: [PATCH] Fix bug where labels with spaces weren't being escaped when
 selected

---
 .../javascripts/filtered_search/dropdown_non_user.js.es6      | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/app/assets/javascripts/filtered_search/dropdown_non_user.js.es6 b/app/assets/javascripts/filtered_search/dropdown_non_user.js.es6
index 0969df65836..84abaa920d6 100644
--- a/app/assets/javascripts/filtered_search/dropdown_non_user.js.es6
+++ b/app/assets/javascripts/filtered_search/dropdown_non_user.js.es6
@@ -29,7 +29,6 @@
       let escapedText = text;
       const hasSpace = text.indexOf(' ') !== -1;
       const hasDoubleQuote = text.indexOf('"') !== -1;
-      const hasSingleQuote = text.indexOf('\'') !== -1;
 
       // Encapsulate value with quotes if it has spaces
       // Known side effect: values's with both single and double quotes
@@ -37,7 +36,8 @@
       if (hasSpace) {
         if (hasDoubleQuote) {
           escapedText = `'${text}'`;
-        } else if (hasSingleQuote) {
+        } else {
+          // Encapsulate singleQuotes or if it hasSpace
           escapedText = `"${text}"`;
         }
       }