From 2ffd2a8e0fddf4a0ec3f5ad461e5a38261edcfe8 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Lo=C3=ABck=20V=C3=A9zien?= <nainpuissant@gmail.com>
Date: Fri, 26 Jan 2018 12:11:52 +0100
Subject: [PATCH] Fix SelectAccounts components, clean stuffs

---
 src/components/IsUnlocked/index.js  |  6 ++++++
 src/components/SelectAccount.js     | 12 ++++++++----
 src/components/base/Select/index.js | 10 +++++++---
 src/constants.js                    |  2 +-
 4 files changed, 22 insertions(+), 8 deletions(-)

diff --git a/src/components/IsUnlocked/index.js b/src/components/IsUnlocked/index.js
index f1f9b429..12ddc0fd 100644
--- a/src/components/IsUnlocked/index.js
+++ b/src/components/IsUnlocked/index.js
@@ -57,6 +57,12 @@ class IsUnlocked extends PureComponent<Props, State> {
     ...defaultState,
   }
 
+  componentWillMount() {
+    if (this.props.isLocked) {
+      stopSyncAccounts()
+    }
+  }
+
   componentWillReceiveProps(nextProps) {
     if (this.props.isLocked && !nextProps.isLocked) {
       startSyncAccounts(nextProps.accounts)
diff --git a/src/components/SelectAccount.js b/src/components/SelectAccount.js
index e187f9aa..40e9a6c9 100644
--- a/src/components/SelectAccount.js
+++ b/src/components/SelectAccount.js
@@ -12,6 +12,10 @@ import Text from 'components/base/Text'
 
 import type { Account } from 'types/common'
 
+function renderItem(accounts) {
+  return item => <span>{(accounts.find(a => a.id === item) || {}).name}</span>
+}
+
 const mapStateToProps: MapStateToProps<*, *, *> = state => ({
   accounts: values(getAccounts(state)),
 })
@@ -24,12 +28,13 @@ type Props = {
 
 const SelectAccount = ({ accounts, value, onChange }: Props) => (
   <Select
+    itemToString={item => item}
     value={value}
     placeholder="Choose an account"
-    items={accounts}
-    keyProp="id"
+    items={accounts.map(a => a.id)}
     onChange={onChange}
-    itemToString={item => (item ? item.name : '')}
+    renderItem={renderItem(accounts)}
+    renderSelected={renderItem(accounts)}
     renderHighlight={(text, key) => (
       <Text key={key} fontWeight="bold">
         {text}
@@ -37,5 +42,4 @@ const SelectAccount = ({ accounts, value, onChange }: Props) => (
     )}
   />
 )
-
 export default connect(mapStateToProps)(SelectAccount)
diff --git a/src/components/base/Select/index.js b/src/components/base/Select/index.js
index a8e4a4ff..85283269 100644
--- a/src/components/base/Select/index.js
+++ b/src/components/base/Select/index.js
@@ -15,7 +15,7 @@ import Search from 'components/base/Search'
 import Triangles from './Triangles'
 
 type Props = {
-  items: Array<Object>,
+  items: Array<any>,
   value?: Object | null,
   itemToString?: Function,
   onChange?: Function,
@@ -24,7 +24,7 @@ type Props = {
   searchable?: boolean,
   placeholder?: string,
   renderHighlight?: string => Element<*>,
-  renderSelected?: Object => Element<*>,
+  renderSelected?: any => Element<*>,
   renderItem?: (*) => Element<*>,
   keyProp?: string,
 }
@@ -98,11 +98,15 @@ class Select extends PureComponent<Props> {
   renderItems = (items: Array<Object>, downshiftProps: Object) => {
     const { renderItem, keyProp } = this.props
     const { getItemProps, highlightedIndex } = downshiftProps
+
     return (
       <Dropdown>
         {items.length ? (
           items.map((item, i) => (
-            <ItemWrapper key={keyProp ? item[keyProp] : item.key} {...getItemProps({ item })}>
+            <ItemWrapper
+              key={keyProp ? item[keyProp] : item.key || item}
+              {...getItemProps({ item })}
+            >
               <Item highlighted={i === highlightedIndex}>
                 {renderItem ? renderItem(item) : <span>{item.name_highlight || item.name}</span>}
               </Item>
diff --git a/src/constants.js b/src/constants.js
index 45fd3ea2..a7baf03a 100644
--- a/src/constants.js
+++ b/src/constants.js
@@ -1,2 +1,2 @@
 export const CHECK_UPDATE_TIMEOUT = 5e3
-export const SYNC_ACCOUNT_TIMEOUT = 2e3
+export const SYNC_ACCOUNT_TIMEOUT = 3e3