refs issue#238

- crean code.

resource/js/components/HeaderSearchBox/SearchForm.js

@@ -19,16 +19,10 @@ export default class SearchForm extends React.Component {
     this.crowi = window.crowi; // FIXME
 
     this.state = {
-      input: '',
-      keyword: '',
-      pages: [],
-      isLoading: false,
       searchError: null,
     };
 
-    this.onSearchSuccess = this.onSearchSuccess.bind(this);
     this.onSearchError = this.onSearchError.bind(this);
-    this.onInputChange = this.onInputChange.bind(this);
     this.onChange = this.onChange.bind(this);
   }
 
@@ -38,25 +32,12 @@ export default class SearchForm extends React.Component {
   componentWillUnmount() {
   }
 
-  onSearchSuccess(res) {
-    this.setState({
-      isLoading: false,
-      keyword: '',
-      pages: res.data,
-    });
-  }
-
   onSearchError(err) {
     this.setState({
-      isLoading: false,
       searchError: err,
     });
   }
 
-  onInputChange(text) {
-    this.setState({input: text});
-  }
-
   onChange(selected) {
     const page = selected[0];  // should be single page selected
 
@@ -80,9 +61,6 @@ export default class SearchForm extends React.Component {
           <InputGroup>
             <SearchTypeahead
               crowi={this.crowi}
-              onSearchSuccess={this.onSearchSuccess}
-              onSearchError={this.onSearchError}
-              onInputChange={this.onInputChange}
               onChange={this.onChange}
               emptyLabel={emptyLabel}
               placeholder="Search ..."

resource/js/components/NewPageNameInputter.js

@@ -14,14 +14,10 @@ export default class NewPageNameInputter extends React.Component {
     super(props);
 
     this.state = {
-      input: '',
-      keyword: '',
-      isLoading: false,
       searchError: null,
     };
     this.crowi = this.props.crowi;
 
-    this.onSearchSuccess = this.onSearchSuccess.bind(this);
     this.onSearchError = this.onSearchError.bind(this);
     this.getParentPageName = this.getParentPageName.bind(this);
   }
@@ -32,17 +28,8 @@ export default class NewPageNameInputter extends React.Component {
   componentWillUnmount() {
   }
 
-  onSearchSuccess(res) {
-    this.setState({
-      isLoading: false,
-      keyword: '',
-      pages: res.data,
-    });
-  }
-
   onSearchError(err) {
     this.setState({
-      isLoading: false,
       searchError: err,
     });
   }
@@ -71,7 +58,6 @@ export default class NewPageNameInputter extends React.Component {
       >
         <SearchTypeahead
           crowi={this.crowi}
-          onSearchSuccess={this.onSearchSuccess}
           onSearchError={this.onSearchError}
           emptyLabel={emptyLabel}
           placeholder="Input page name"

resource/js/components/SearchTypeahead.js

@@ -103,7 +103,7 @@ export default class SearchTypeahead extends React.Component {
       keyword: '',
       pages: res.data,
     });
-    this.props.onSearchSuccess(res);
+    this.props.onSearchSuccess && this.props.onSearchSuccess(res);
   }
 
   /**
@@ -115,7 +115,7 @@ export default class SearchTypeahead extends React.Component {
       isLoading: false,
       searchError: err,
     });
-    this.props.onSearchError(err);
+    this.props.onSearchError && this.props.onSearchError(err);
   }
 
   onInputChange(text) {
@@ -207,7 +207,6 @@ SearchTypeahead.propTypes = {
   onSearchSuccess: PropTypes.func,
   onSearchError:   PropTypes.func,
   onChange:        PropTypes.func,
-  onInputChange:   PropTypes.func,
   emptyLabel:      PropTypes.string,
   placeholder:     PropTypes.string,
   keywordOnInit:   PropTypes.string,
@@ -220,7 +219,6 @@ SearchTypeahead.defaultProps = {
   onSearchSuccess: noop,
   onSearchError:   noop,
   onChange:        noop,
-  onInputChange:   noop,
   emptyLabel:      null,
   placeholder:     "",
   keywordOnInit:   "",