Instead of preserving slashes, just disable escaping entirely

Andrew Nelson · Andrew Nelson · commit d1b2c42cc131 · 2016-10-13T09:07:48.000-04:00
diff --git a/lib/types/operation.js b/lib/types/operation.js
@@ -483,13 +483,7 @@ Operation.prototype.urlify = function (args) {
           value = this.encodePathCollection(param.collectionFormat, param.name, value);
         } else {
 
-          if(param['x-escape'] === false ) {
-            value = value.split('/')
-              .map(function encodePathPart(part) {
-                return encodeURIComponent(part);
-              })
-              .join('/');
-          } else {
+          if((typeof(param['x-escape']) === 'undefined') || (param['x-escape'] === true)) {
             value = this.encodePathParam(value);
           }
         }

Original file line number	Diff line number	Diff line change
`@@ -483,13 +483,7 @@ Operation.prototype.urlify = function (args) {`
`483`	`483`	`value = this.encodePathCollection(param.collectionFormat, param.name, value);`
`484`	`484`	`} else {`
`485`	`485`
`486`		`- if(param['x-escape'] === false ) {`
`487`		`- value = value.split('/')`
`488`		`- .map(function encodePathPart(part) {`
`489`		`- return encodeURIComponent(part);`
`490`		`- })`
`491`		`- .join('/');`
`492`		`- } else {`
	`486`	`+ if((typeof(param['x-escape']) === 'undefined') \|\| (param['x-escape'] === true)) {`
`493`	`487`	`value = this.encodePathParam(value);`
`494`	`488`	`}`
`495`	`489`	`}`