Closes #182.

Author: jmdobry

CHANGELOG.md

@@ -10,6 +10,7 @@
 - #170 - Global callbacks
 - #171 - "not in" query
 - #177 - Allow promises to be returned in lifecycle hooks
+- #182 - option to force trailing slash (DSHttpAdapterProvider.defaults.forceTrailingSlash)
 
 ###### Backwards compatible bug fixes
 - #156 - cached findAll pending query doesn't get removed sometimes

dist/angular-data.js

@@ -1659,6 +1659,8 @@ function DSHttpAdapterProvider() {
       return params;
     },
 
+    forceTrailingSlash: false,
+
     /**
      * @doc property
      * @id DSHttpAdapterProvider.properties:defaults.$httpConfig
@@ -1719,6 +1721,9 @@ function DSHttpAdapterProvider() {
       HTTP: function (config) {
         var start = new Date().getTime();
 
+        if (this.defaults.forceTrailingSlash && config.url[config.url.length] !== '/') {
+          config.url += '/';
+        }
         config = DSUtils.deepMixIn(config, defaults.$httpConfig);
         return $http(config).then(function (data) {
           $log.debug(data.config.method + ' request:' + data.config.url + ' Time taken: ' + (new Date().getTime() - start) + 'ms', arguments);

dist/angular-data.min.js

@@ -43,6 +43,8 @@ function DSHttpAdapterProvider() {
       return params;
     },
 
+    forceTrailingSlash: false,
+
     /**
      * @doc property
      * @id DSHttpAdapterProvider.properties:defaults.$httpConfig
@@ -103,6 +105,9 @@ function DSHttpAdapterProvider() {
       HTTP: function (config) {
         var start = new Date().getTime();
 
+        if (this.defaults.forceTrailingSlash && config.url[config.url.length] !== '/') {
+          config.url += '/';
+        }
         config = DSUtils.deepMixIn(config, defaults.$httpConfig);
         return $http(config).then(function (data) {
           $log.debug(data.config.method + ' request:' + data.config.url + ' Time taken: ' + (new Date().getTime() - start) + 'ms', arguments);

src/adapters/http.js

@@ -43,7 +43,8 @@ describe('DS.create(resourceName, attrs[, options])', function () {
     assert.deepEqual(DS.get('post', 5), p1);
   });
   it('should create an item and save it to the server but not inject the result', function () {
-    $httpBackend.expectPOST('http://test.angular-cache.com/posts').respond(200, p1);
+    DSHttpAdapter.defaults.forceTrailingSlash = true;
+    $httpBackend.expectPOST('http://test.angular-cache.com/posts/').respond(200, p1);
 
     DS.create('post', { author: 'John', age: 30 }, { cacheResponse: false }).then(function (post) {
       assert.deepEqual(post, p1, 'post 5 should have been created');
@@ -53,6 +54,7 @@ describe('DS.create(resourceName, attrs[, options])', function () {
     });
 
     $httpBackend.flush();
+    DSHttpAdapter.defaults.forceTrailingSlash = false;
 
     assert.equal(lifecycle.beforeCreate.callCount, 1, 'beforeCreate should have been called');
     assert.equal(lifecycle.afterCreate.callCount, 1, 'afterCreate should have been called');