Standardize token value capitalization in filter bar

4 files changed, 26 insertions, 24 deletions

diff --git a/app/assets/javascripts/filtered_search/filtered_search_manager.js b/app/assets/javascripts/filtered_search/filtered_search_manager.js
index 4a2af02b40a..33c82778c79 100644
--- a/app/assets/javascripts/filtered_search/filtered_search_manager.js
+++ b/app/assets/javascripts/filtered_search/filtered_search_manager.js
@@ -593,7 +593,7 @@ export default class FilteredSearchManager {
     tokens.forEach(token => {
       const condition = this.filteredSearchTokenKeys.searchByConditionKeyValue(
         token.key,
-        token.value.toLowerCase(),
+        token.value,
       );
       const tokenConfig = this.filteredSearchTokenKeys.searchByKey(token.key) || {};
       const { param } = tokenConfig;
diff --git a/app/assets/javascripts/filtered_search/filtered_search_token_keys.js b/app/assets/javascripts/filtered_search/filtered_search_token_keys.js
index e01dedbb57c..b70da240833 100644
--- a/app/assets/javascripts/filtered_search/filtered_search_token_keys.js
+++ b/app/assets/javascripts/filtered_search/filtered_search_token_keys.js
@@ -65,8 +65,10 @@ export default class FilteredSearchTokenKeys {
 
   searchByConditionKeyValue(key, value) {
     return (
-      this.conditions.find(condition => condition.tokenKey === key && condition.value === value) ||
-      null
+      this.conditions.find(
+        condition =>
+          condition.tokenKey === key && condition.value.toLowerCase() === value.toLowerCase(),
+      ) || null
     );
   }
 
diff --git a/app/assets/javascripts/filtered_search/issuable_filtered_search_token_keys.js b/app/assets/javascripts/filtered_search/issuable_filtered_search_token_keys.js
index b494b7e2de0..fd61030eb13 100644
--- a/app/assets/javascripts/filtered_search/issuable_filtered_search_token_keys.js
+++ b/app/assets/javascripts/filtered_search/issuable_filtered_search_token_keys.js
@@ -60,52 +60,52 @@ export const conditions = [
   {
     url: 'assignee_id=None',
     tokenKey: 'assignee',
-    value: 'none',
+    value: 'None',
   },
   {
     url: 'assignee_id=Any',
     tokenKey: 'assignee',
-    value: 'any',
+    value: 'Any',
   },
   {
     url: 'milestone_title=None',
     tokenKey: 'milestone',
-    value: 'none',
+    value: 'None',
   },
   {
     url: 'milestone_title=Any',
     tokenKey: 'milestone',
-    value: 'any',
+    value: 'Any',
   },
   {
     url: 'milestone_title=%23upcoming',
     tokenKey: 'milestone',
-    value: 'upcoming',
+    value: 'Upcoming',
   },
   {
     url: 'milestone_title=%23started',
     tokenKey: 'milestone',
-    value: 'started',
+    value: 'Started',
   },
   {
     url: 'label_name[]=None',
     tokenKey: 'label',
-    value: 'none',
+    value: 'None',
   },
   {
     url: 'label_name[]=Any',
-    tokenKey: 'any',
-    value: 'any',
+    tokenKey: 'label',
+    value: 'Any',
   },
   {
     url: 'my_reaction_emoji=None',
     tokenKey: 'my-reaction',
-    value: 'none',
+    value: 'None',
   },
   {
     url: 'my_reaction_emoji=Any',
     tokenKey: 'my-reaction',
-    value: 'any',
+    value: 'Any',
   },
 ];
 
diff --git a/app/views/shared/issuable/_search_bar.html.haml b/app/views/shared/issuable/_search_bar.html.haml
index 20847378495..588659c7e9c 100644
--- a/app/views/shared/issuable/_search_bar.html.haml
+++ b/app/views/shared/issuable/_search_bar.html.haml
@@ -57,10 +57,10 @@
                   avatar: { lazy: true, url: '{{avatar_url}}' }
             #js-dropdown-assignee.filtered-search-input-dropdown-menu.dropdown-menu
               %ul{ data: { dropdown: true } }
-                %li.filter-dropdown-item{ data: { value: 'none' } }
+                %li.filter-dropdown-item{ data: { value: 'None' } }
                   %button.btn.btn-link{ type: 'button' }
                     = _('None')
-                %li.filter-dropdown-item{ data: { value: 'any' } }
+                %li.filter-dropdown-item{ data: { value: 'Any' } }
                   %button.btn.btn-link{ type: 'button' }
                     = _('Any')
                 %li.divider.droplab-item-ignore
@@ -73,16 +73,16 @@
                   avatar: { lazy: true, url: '{{avatar_url}}' }
             #js-dropdown-milestone.filtered-search-input-dropdown-menu.dropdown-menu
               %ul{ data: { dropdown: true } }
-                %li.filter-dropdown-item{ data: { value: 'none' } }
+                %li.filter-dropdown-item{ data: { value: 'None' } }
                   %button.btn.btn-link{ type: 'button' }
                     = _('None')
-                %li.filter-dropdown-item{ data: { value: 'any' } }
+                %li.filter-dropdown-item{ data: { value: 'Any' } }
                   %button.btn.btn-link{ type: 'button' }
                     = _('Any')
-                %li.filter-dropdown-item{ data: { value: 'upcoming' } }
+                %li.filter-dropdown-item{ data: { value: 'Upcoming' } }
                   %button.btn.btn-link{ type: 'button' }
                     = _('Upcoming')
-                %li.filter-dropdown-item{ data: { value: 'started' } }
+                %li.filter-dropdown-item{ data: { value: 'Started' } }
                   %button.btn.btn-link{ type: 'button' }
                     = _('Started')
                 %li.divider.droplab-item-ignore
@@ -92,10 +92,10 @@
                     {{title}}
             #js-dropdown-label.filtered-search-input-dropdown-menu.dropdown-menu
               %ul{ data: { dropdown: true } }
-                %li.filter-dropdown-item{ data: { value: 'none' } }
+                %li.filter-dropdown-item{ data: { value: 'None' } }
                   %button.btn.btn-link{ type: 'button' }
                     = _('None')
-                %li.filter-dropdown-item{ data: { value: 'any' } }
+                %li.filter-dropdown-item{ data: { value: 'Any' } }
                   %button.btn.btn-link{ type: 'button' }
                     = _('Any')
                 %li.divider.droplab-item-ignore
@@ -107,10 +107,10 @@
                       {{title}}
             #js-dropdown-my-reaction.filtered-search-input-dropdown-menu.dropdown-menu
               %ul{ data: { dropdown: true } }
-                %li.filter-dropdown-item{ data: { value: 'none' } }
+                %li.filter-dropdown-item{ data: { value: 'None' } }
                   %button.btn.btn-link{ type: 'button' }
                     = _('None')
-                %li.filter-dropdown-item{ data: { value: 'any' } }
+                %li.filter-dropdown-item{ data: { value: 'Any' } }
                   %button.btn.btn-link{ type: 'button' }
                     = _('Any')
                 %li.divider.droplab-item-ignore