Fixed bug that would instanciate new components when not necessary

CHANGELOG.md

@@ -1,5 +1,9 @@
 # CHANGELOG
 
+## 0.2.2 (October 8th, 2016)
+
+- Fixed bug that would instantiate new components when not necessary
+
 ## 0.2.1 (October 6th, 2016)
 
 - Fixed issue with webpack bundle that would import the wrong module

package.json

@@ -1,7 +1,7 @@
 {
   "name": "react-router-server",
   "author": "Gabriel Bull",
-  "version": "0.2.1",
+  "version": "0.2.2",
   "description": "Server Side Rendering library for React Router v4",
   "main": "./build/index.js",
   "keywords": [

src/match.js

@@ -17,17 +17,26 @@ class Match extends Component {
     serverRouter: PropTypes.object
   }
 
-  render() {
-    const { match, render, ...props } = this.props;
-    const nextRender = matchProps => wrapperComponent(
+  componentWillMount() {
+    const { render } = this.props;
+    this.wrapperComponent = wrapperComponent(
       render,
       { ...this.props },
       this.context.serverRouter && this.context.serverRouter.asyncRenderer ?
         this.context.serverRouter.asyncRenderer : null
-    )(matchProps);
+    );
+    const WrapperComponent = this.wrapperComponent;
+    this.nextRender = matchProps => {
+      return <WrapperComponent matchProps={matchProps}/>;
+    }
+  }
+
+  render() {
+    const { match, ...props } = this.props;
+    delete props.render;
     delete props.willEnter;
     const Match = match;
-    return <Match render={nextRender} {...props}/>;
+    return <Match render={this.nextRender} {...props}/>;
   }
 }
 

src/wrapperComponent.js

@@ -1,8 +1,12 @@
 import React, { Component, PropTypes, isValidElement } from 'react'
 import { setAsyncRenderer } from './importModule';
 
-export default (render, options, asyncRenderer) => (matchProps) => {
-  class WrapperComponent extends Component {
+export default (render, options, asyncRenderer) => {
+  return class extends Component {
+    static propTypes = {
+      matchProps: PropTypes.object
+    };
+
     isComponentMounted = false;
 
     constructor() {
@@ -28,7 +32,7 @@ export default (render, options, asyncRenderer) => (matchProps) => {
 
       if (!WrappedComponent) {
         setAsyncRenderer(asyncRenderer);
-        WrappedComponent = render(matchProps, { ...this.state.nextProps });
+        WrappedComponent = render(this.props.matchProps, { ...this.state.nextProps });
       }
 
       if (WrappedComponent.then) {
@@ -64,6 +68,4 @@ export default (render, options, asyncRenderer) => (matchProps) => {
       return null;
     }
   }
-
-  return <WrapperComponent/>;
 };