diff --git a/packages/next-server/lib/router/router.js b/packages/next-server/lib/router/router.js index d4f4c1d5..cca88738 100644 --- a/packages/next-server/lib/router/router.js +++ b/packages/next-server/lib/router/router.js @@ -188,14 +188,14 @@ export default class Router { return true } - const { pathname: asPathname, query: asQuery } = parse(as, true) const { pathname, query } = parse(url, true) // If asked to change the current URL we should reload the current page // (not location.reload() but reload getInitialProps and other Next.js stuffs) // We also need to set the method = replaceState always // as this should not go into the history (That's how browsers work) - if (!this.urlIsNew(asPathname, asQuery)) { + // We should compare the new asPath to the current asPath, not the url + if (!this.urlIsNew(as)) { method = 'replaceState' } @@ -366,8 +366,10 @@ export default class Router { } } - urlIsNew (pathname, query) { - return this.pathname !== pathname || !shallowEquals(query, this.query) + urlIsNew (asPath) { + const { pathname, query } = parse(asPath, true) + const { pathname: curPathname } = parse(this.asPath, true) + return curPathname !== pathname || !shallowEquals(query, this.query) } isShallowRoutingPossible (route) { diff --git a/test/integration/basic/pages/nav/as-path-pushstate.js b/test/integration/basic/pages/nav/as-path-pushstate.js index a5c1482b..a85e0ac2 100644 --- a/test/integration/basic/pages/nav/as-path-pushstate.js +++ b/test/integration/basic/pages/nav/as-path-pushstate.js @@ -9,7 +9,16 @@ export default withRouter(({router: {asPath, query}}) => { hello - +
+ + else + +
+
+ + normal hello + +
{query.something === 'hello' && same query } diff --git a/test/integration/basic/test/client-navigation.js b/test/integration/basic/test/client-navigation.js index fbf5fdc4..f51411dc 100644 --- a/test/integration/basic/test/client-navigation.js +++ b/test/integration/basic/test/client-navigation.js @@ -648,6 +648,11 @@ export default (context) => { await browser.back().waitForElementByCss('#something-hello') const queryThree = JSON.parse(await browser.elementByCss('#router-query').text()) expect(queryThree.something).toBe('hello') + await browser.elementByCss('#else').click().waitForElementByCss('#something-else') + await browser.elementByCss('#hello2').click().waitForElementByCss('#nav-as-path-pushstate') + await browser.back().waitForElementByCss('#something-else') + const queryFour = JSON.parse(await browser.elementByCss('#router-query').text()) + expect(queryFour.something).toBe(undefined) } finally { if (browser) { browser.close()