Store compiled patterns in a cache object, not on the route element's props.

Fixes compat with React 0.14 and #148.

Author: STRML

lib/matchRoutes.js

@@ -6,6 +6,8 @@ var React = require('react');
 var cloneWithProps = require('react/lib/cloneWithProps');
 var assign = Object.assign || require('object.assign');
 
+var patternCache = {};
+
 /**
  * Match routes against a path
  *
@@ -33,9 +35,12 @@ function matchRoutes(routes, path) {
     }
 
     if (current.props.path) {
-      current.props.pattern = current.props.pattern || new URLPattern(current.props.path, createCompiler(current.props));
+      // Technically, this cache will incorrectly be used if a user defines two routes
+      // with identical paths but different compilers. FIXME?
+      var pattern = patternCache[current.props.path] ||
+                    new URLPattern(current.props.path, createCompiler(current.props));
       if (!page) {
-        match = current.props.pattern.match(path);
+        match = pattern.match(path);
         if (match) {
           page = current;
         }

tests/matchRoutes.js

@@ -20,10 +20,6 @@ describe('matchRoutes', function() {
   ];
 
   afterEach(function() {
-    // Remove any compiled regex patterns from all routes
-    routes.forEach(function(r) {
-      r.props.pattern = null;
-    });
     // In case we overrode this, reset it.
     Router.createURLPatternCompiler = function () { return new URLPattern.Compiler(); };
   });