Fix this in getComponent and getComponents (#3306)

Author: taion

CHANGES.md

@@ -1,3 +1,12 @@
+## [HEAD]
+> Unreleased
+
+- **Bugfix:** Fix `this` in `getComponent` and `getComponents` ([#3306])
+
+[HEAD]: https://github.com/reactjs/react-router/compare/latest...HEAD
+[#3306]: https://github.com/reactjs/react-router/pull/3306
+
+
 ## [v2.2.0]
 > April 13, 2016
 

modules/__tests__/Router-test.js

@@ -331,8 +331,11 @@ describe('Router', function () {
 
     it('should support getComponent', function (done) {
       const Component = () => <div />
-      const getComponent = (nextState, callback) => {
+
+      function getComponent(nextState, callback) {
+        expect(this.getComponent).toBe(getComponent)
         expect(nextState.location.pathname).toBe('/')
+
         setTimeout(() => callback(null, Component))
       }
 
@@ -352,8 +355,10 @@ describe('Router', function () {
       const foo = () => <div />
       const bar = () => <div />
 
-      const getComponents = (nextState, callback) => {
+      function getComponents(nextState, callback) {
+        expect(this.getComponents).toBe(getComponents)
         expect(nextState.location.pathname).toBe('/')
+
         setTimeout(() => callback(null, { foo, bar }))
       }
 

modules/getComponents.js

@@ -33,7 +33,7 @@ function getComponentsForRoute(nextState, route, callback) {
       Object.assign(nextStateWithLocation, location)
     }
 
-    getComponent(nextStateWithLocation, callback)
+    getComponent.call(route, nextStateWithLocation, callback)
     return
   }