[fixed] <Link href> when using HashHistory

Author: mjackson

modules/HashHistory.js

@@ -164,8 +164,8 @@ class HashHistory extends DOMHistory {
     this._notifyChange();
   }
 
-  makeHref(pathname, query) {
-    return '#' + super.makeHref(pathname, query);
+  makeHref(path) {
+    return '#' + path;
   }
 
 }

modules/RouterContextMixin.js

@@ -71,7 +71,13 @@ var RouterContextMixin = {
    * pathname and query.
    */
   makeHref(pathname, query) {
-    return this.makePath(pathname, query);
+    var path = this.makePath(pathname, query);
+    var { history } = this.props;
+
+    if (history && history.makeHref)
+      return history.makeHref(path);
+
+    return path;
   },
 
   /**

modules/__tests__/Link-test.js

@@ -64,12 +64,12 @@ describe('A <Link>', function () {
     });
 
     render((
-      <Router history={new HashHistory()}>
+      <Router history={new HashHistory}>
         <Route path="/" component={LinkWrapper}/>
       </Router>
     ), div, function () {
       var a = div.querySelector('a');
-      expect(a.getAttribute('href')).toEqual('/#/hello/michael?the=query');
+      expect(a.getAttribute('href')).toEqual('#/hello/michael?the=query');
     });
   });