bug: index off by one

dremonkey · dremonkey · commit 138c7282698e · 2015-02-21T22:11:26.000-08:00
Very minor bug but forceTrailingSlash always adds a / if set to true because the index was off by one.
diff --git a/src/index.js b/src/index.js
@@ -450,7 +450,7 @@
           var _this = this;
           var start = new Date();
           config = deepMixIn(config, _this.defaults.httpConfig);
-          if (_this.defaults.forceTrailingSlash && config.url[config.url.length] !== '/') {
+          if (_this.defaults.forceTrailingSlash && config.url[config.url.length-1] !== '/') {
             config.url += '/';
           }
           config.method = config.method.toUpperCase();

Original file line number	Diff line number	Diff line change
`@@ -450,7 +450,7 @@`
`450`	`450`	`var _this = this;`
`451`	`451`	`var start = new Date();`
`452`	`452`	`config = deepMixIn(config, _this.defaults.httpConfig);`
`453`		`- if (_this.defaults.forceTrailingSlash && config.url[config.url.length] !== '/') {`
	`453`	`+ if (_this.defaults.forceTrailingSlash && config.url[config.url.length-1] !== '/') {`
`454`	`454`	`config.url += '/';`
`455`	`455`	`}`
`456`	`456`	`config.method = config.method.toUpperCase();`