fix(vuejs#2725): remove duplicated decodeURIComponent

Author: mestevezdeister

examples/basic/app.js

@@ -35,6 +35,7 @@ const Home = { template: '<div>home</div>' }
 const Foo = { template: '<div>foo</div>' }
 const Bar = { template: '<div>bar</div>' }
 const Unicode = { template: '<div>unicode</div>' }
+const Query = { template: '<div>query: {{ $route.params.q }}</div>' }
 
 // 3. Create the router
 const router = new VueRouter({
@@ -44,7 +45,8 @@ const router = new VueRouter({
     { path: '/', component: Home },
     { path: '/foo', component: Foo },
     { path: '/bar', component: Bar },
-    { path: '/é', component: Unicode }
+    { path: '/é', component: Unicode },
+    { path: '/query/:q', component: Query }
   ]
 })
 
@@ -73,6 +75,7 @@ const vueInstance = new Vue({
           </li>
         </router-link>
         <li><router-link to="/foo" replace>/foo (replace)</router-link></li>
+        <li><router-link to="/query/A%25">/query/A%</router-link></li>
       </ul>
       <button id="navigate-btn" @click="navigateAndIncrement">On Success</button>
       <pre id="counter">{{ n }}</pre>

examples/hash-mode/app.js

@@ -35,6 +35,7 @@ const Home = { template: '<div>home</div>' }
 const Foo = { template: '<div>foo</div>' }
 const Bar = { template: '<div>bar</div>' }
 const Unicode = { template: '<div>unicode: {{ $route.params.unicode }}</div>' }
+const Query = { template: '<div>query: {{ $route.params.q }}</div>' }
 
 // 3. Create the router
 const router = new VueRouter({
@@ -45,7 +46,8 @@ const router = new VueRouter({
     { path: '/foo', component: Foo },
     { path: '/bar', component: Bar },
     { path: '/é', component: Unicode },
-    { path: '/é/:unicode', component: Unicode }
+    { path: '/é/:unicode', component: Unicode },
+    { path: '/query/:q', component: Query }
   ]
 })
 
@@ -66,6 +68,7 @@ const vueInstance = new Vue({
         <li><router-link to="/é/ñ">/é/ñ</router-link></li>
         <li><router-link to="/é/ñ?t=%25ñ">/é/ñ?t=%ñ</router-link></li>
         <li><router-link to="/é/ñ#é">/é/ñ#é</router-link></li>
+        <li><router-link to="/query/A%25">/query/A%</router-link></li>  
       </ul>
       <pre id="query-t">{{ $route.query.t }}</pre>
       <pre id="hash">{{ $route.hash }}</pre>

src/create-matcher.js

@@ -185,10 +185,9 @@ function matchRoute (
 
   for (let i = 1, len = m.length; i < len; ++i) {
     const key = regex.keys[i - 1]
-    const val = typeof m[i] === 'string' ? decodeURIComponent(m[i]) : m[i]
     if (key) {
       // Fix #1994: using * with props: true generates a param named 0
-      params[key.name || 'pathMatch'] = val
+      params[key.name || 'pathMatch'] = m[i]
     }
   }
 

test/e2e/specs/basic.js

@@ -9,8 +9,8 @@ module.exports = {
     browser
       .url('http://localhost:8080/basic/')
       .waitForElementVisible('#app', 1000)
-      .assert.count('li', 9)
-      .assert.count('li a', 9)
+      .assert.count('li', 10)
+      .assert.count('li a', 10)
       // assert correct href with base
       .assert.attributeContains('li:nth-child(1) a', 'href', '/basic/')
       .assert.attributeContains('li:nth-child(2) a', 'href', '/basic/foo')
@@ -20,6 +20,7 @@ module.exports = {
       .assert.attributeContains('li:nth-child(6) a', 'href', '/basic/%C3%A9?t=%25%C3%B1')
       .assert.attributeContains('li:nth-child(7) a', 'href', '/basic/%C3%A9#%25%C3%B1')
       .assert.attributeContains('li:nth-child(8) a', 'href', '/basic/foo')
+      .assert.attributeContains('li:nth-child(10) a', 'href', '/basic/query/A%25')
       .assert.containsText('.view', 'home')
 
       .click('li:nth-child(2) a')
@@ -70,6 +71,15 @@ module.exports = {
       .assert.cssClassPresent('li:nth-child(8)', 'exact-active')
       .assert.attributeEquals('li:nth-child(8) a', 'class', '')
 
+      // encoded percentage as path param
+      // https://github.com/vuejs/vue-router/issues/2725
+      .url('http://localhost:8080/basic/query/A%25')
+      .waitForElementVisible('#app', 1000)
+      .assert.containsText('.view', 'query: A%')
+      .click('li:nth-child(10) a')
+      .assert.urlEquals('http://localhost:8080/basic/query/A%25')
+      .assert.containsText('.view', 'query: A%')
+
       // Listener cleanup
       .assert.containsText('#popstate-count', '1 popstate listeners')
       .click('#unmount')

test/e2e/specs/hash-mode.js

@@ -9,14 +9,15 @@ module.exports = {
     browser
       .url('http://localhost:8080/hash-mode/')
       .waitForElementVisible('#app', 1000)
-      .assert.count('li', 8)
-      .assert.count('li a', 7)
+      .assert.count('li', 9)
+      .assert.count('li a', 8)
       .assert.attributeContains('li:nth-child(1) a', 'href', '/hash-mode/#/')
       .assert.attributeContains('li:nth-child(2) a', 'href', '/hash-mode/#/foo')
       .assert.attributeContains('li:nth-child(3) a', 'href', '/hash-mode/#/bar')
       .assert.attributeContains('li:nth-child(5) a', 'href', '/hash-mode/#/%C3%A9')
       .assert.attributeContains('li:nth-child(6) a', 'href', '/hash-mode/#/%C3%A9/%C3%B1')
       .assert.attributeContains('li:nth-child(7) a', 'href', '/hash-mode/#/%C3%A9/%C3%B1?t=%25%C3%B1')
+      .assert.attributeContains('li:nth-child(9) a', 'href', '/hash-mode/#/query/A%25')
       .assert.containsText('.view', 'home')
 
       .click('li:nth-child(2) a')
@@ -58,6 +59,15 @@ module.exports = {
       .assert.containsText('.view', 'unicode: ñ')
       .assert.containsText('#query-t', '%')
 
+      // percentage as path param
+      // https://github.com/vuejs/vue-router/issues/2725
+      .url('http://localhost:8080/hash-mode/#/query/A%25')
+      .waitForElementVisible('#app', 1000)
+      .assert.containsText('.view', 'query: A%')
+      .click('li:nth-child(9) a')
+      .assert.urlEquals('http://localhost:8080/hash-mode/#/query/A%25')
+      .assert.containsText('.view', 'query: A%')
+
       // Listener cleanup
       .assert.containsText('#popstate-count', '1 popstate listeners')
       .click('#unmount')