From 2708d77da9a04204c93728ada901bd7238de3825 Mon Sep 17 00:00:00 2001
From: Xuan-thi Nguyen <xthi.ngu@gmail.com>
Date: Tue, 9 Jul 2019 14:48:27 +0200
Subject: [PATCH] fix(Dropdown): put empty array to filtered items when
 searchable is false (#628)

---
 src/Dropdown/index.js | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/src/Dropdown/index.js b/src/Dropdown/index.js
index ea3d63e6..5de55936 100644
--- a/src/Dropdown/index.js
+++ b/src/Dropdown/index.js
@@ -161,9 +161,11 @@ class Dropdown extends PureComponent {
     // Filter items based on search key word
     // TODO: when need to refactor this later, children may have some children
     // we need to filter recursively. - @Thomas -
-    const FiltredItem = Children.toArray(children).filter(({ props: { children } }) =>
-      children.toLowerCase().includes(searchKeyword.toLowerCase()),
-    );
+    const FilteredItems = searchable
+      ? Children.toArray(children).filter(({ props: { children } }) =>
+          children.toLowerCase().includes(searchKeyword.toLowerCase()),
+        )
+      : [];
 
     return (
       <div className={className} data-testid="dropdown">
@@ -196,8 +198,8 @@ class Dropdown extends PureComponent {
                 )}
 
                 {searchable &&
-                  (FiltredItem.length !== 0
-                    ? FiltredItem.map(child => (
+                  (FilteredItems.length !== 0
+                    ? FilteredItems.map(child => (
                         <MenuItem key={child.key} searchable={searchable} role="menuitem">
                           {child}
                         </MenuItem>