Fix active route matching (#1101)

Fix #1091
Fix the doc too

Author: posva

docs/en/api/router-link.md

@@ -94,7 +94,7 @@
 
   - default: `false`
 
-  The default active class matching behavior is **inclusive match**. For example, `<router-link to="/a">` will get this class applied as long as the current path starts with `/a`.
+  The default active class matching behavior is **inclusive match**. For example, `<router-link to="/a">` will get this class applied as long as the current path starts with `/a/` or is `/a`.
 
   One consequence of this is that `<router-link to="/">` will be active for every route! To force the link into "exact match mode", use the `exact` prop:
 

src/util/route.js

@@ -2,6 +2,8 @@
 
 import { stringifyQuery } from './query'
 
+const trailingSlashRE = /\/?$/
+
 export function createRoute (
   record: ?RouteRecord,
   location: Location,
@@ -41,7 +43,6 @@ function getFullPath ({ path, query = {}, hash = '' }) {
   return (path || '/') + stringifyQuery(query) + hash
 }
 
-const trailingSlashRE = /\/$/
 export function isSameRoute (a: Route, b: ?Route): boolean {
   if (b === START) {
     return a === b
@@ -76,7 +77,9 @@ function isObjectEqual (a = {}, b = {}): boolean {
 
 export function isIncludedRoute (current: Route, target: Route): boolean {
   return (
-    current.path.indexOf(target.path.replace(/\/$/, '')) === 0 &&
+    current.path.replace(trailingSlashRE, '/').indexOf(
+      target.path.replace(trailingSlashRE, '/')
+    ) === 0 &&
     (!target.hash || current.hash === target.hash) &&
     queryIncludes(current.query, target.query)
   )

test/unit/specs/route.spec.js

@@ -74,5 +74,16 @@ describe('Route utils', () => {
       expect(isIncludedRoute(a, d)).toBe(false)
       expect(isIncludedRoute(a, e)).toBe(false)
     })
+
+    it('trailing slash', () => {
+      const a = { path: 'user/1' }
+      const b = { path: 'user/10' }
+      const c = { path: 'user/10/' }
+      const d = { path: 'user/1/' }
+      expect(isIncludedRoute(a, b)).toBe(false)
+      expect(isIncludedRoute(a, c)).toBe(false)
+      expect(isIncludedRoute(b, d)).toBe(false)
+      expect(isIncludedRoute(c, d)).toBe(false)
+    })
   })
 })