Merge pull request #290 from react-component/yu-fixfilter

fix: value is wrong when blur, close ant-design/ant-design#10134

examples/single.js

@@ -8,7 +8,7 @@ import ReactDOM from 'react-dom';
 class Test extends React.Component {
   state = {
     destroy: false,
-    value: String(9),
+    value: 9,
   };
 
   onChange = (e) => {
@@ -76,11 +76,10 @@ class Test extends React.Component {
             <Option value="21" disabled text="disabled">disabled</Option>
             <Option value="31" text="yiminghe">yiminghe</Option>
             {[0, 1, 2, 3, 4, 5, 6, 7, 8, 9].map((i) => {
-              return <Option key={i} text={String(i)}>{i}</Option>;
+              return <Option key={i} value={i} text={String(i)}>{i}-text</Option>;
             })}
           </Select>
         </div>
-
         <h2>native select</h2>
         <select
           value={this.state.value}

src/Select.jsx

@@ -353,7 +353,7 @@ export default class Select extends React.Component {
         if (options.length) {
           const firstOption = findFirstMenuItem(options);
           if (firstOption) {
-            value = [firstOption.key];
+            value = [getValuePropValue(firstOption)];
             this.fireChange(value);
           }
         }

tests/Select.spec.js

@@ -436,9 +436,10 @@ describe('Select', () => {
           onChange={handleChange}
           onBlur={handleBlur}
           showSearch
+          optionLabelProp="children"
         >
-          <Option value="1">1</Option>
-          <Option value="2">2</Option>
+          <Option value={1} key={1}>1-text</Option>
+          <Option value={2} key={2}>2-text</Option>
         </Select>
       );
       jest.useFakeTimers();
@@ -447,12 +448,16 @@ describe('Select', () => {
       jest.runAllTimers();
     });
 
+    it('will be auto select', () => {
+      expect(wrapper.update().find('.rc-select-selection-selected-value').text()).toBe('1-text');
+    });
+
     it('set _focused to false', () => {
       expect(wrapper.instance()._focused).toBe(false);
     });
 
     it('fires change event', () => {
-      expect(handleChange).toBeCalledWith('1', expect.anything());
+      expect(handleChange).toBeCalledWith(1, expect.anything());
     });
 
     it('fires blur event', () => {