fix(react-container): should initialize before other component

azu · azu · commit f12cef36e849 · 2018-02-24T13:04:55.000+09:00
We have used componentDidMount instead of componentWillMount in #321 But, Almin React Container should be initialized before other component. Because, this component subscribe `Context#onChange`. In other words, Almin React Container can not handle `Context#onchange` when some store has been changed in Other Component#componentDidMount.
diff --git a/packages/almin-react-container/src/almin-react-container.tsx b/packages/almin-react-container/src/almin-react-container.tsx
@@ -35,9 +35,14 @@ export class AlminReactContainer {
             state: P;
             unSubscribe: () => void | null;
 
+            onChangeHandler = () => {
+                this.setState(context.getState());
+            };
+
             constructor(props: any) {
                 super(props);
                 this.state = context.getState();
+                this.unSubscribe = context.onChange(this.onChangeHandler);
             }
 
             shouldComponentUpdate(_nextProps: any, nextState: any) {
@@ -46,13 +51,6 @@ export class AlminReactContainer {
                 return !shallowEqual(this.state, nextState);
             }
 
-            componentDidMount() {
-                const onChangeHandler = () => {
-                    this.setState(context.getState());
-                };
-                this.unSubscribe = context.onChange(onChangeHandler);
-            }
-
             componentWillUnmount() {
                 if (typeof this.unSubscribe === "function") {
                     this.unSubscribe();
