fix($resource): copy provider configuration conventions from $httpBackend

First, this now uses a flat object configuration, similar to
`$httpBackend`.  This should make configuring this provider much more
familiar.

This adds a fourth optional argument to the `$resource()` constructor,
supporting overriding global `$resourceProvider` configuration.

Now, both of these ways of configuring this is supported:

    app.config(function($resourceProvider) {
      $resourceProvider.defaults.stripTrailingSlashes = false;
    });

or per instance:

    var CreditCard = $resource('/some/:url/', ..., ..., {
        stripTrailingSlashes: false
    });

Author: nvie

src/ngResource/resource.js

@@ -82,7 +82,7 @@ function shallowClearAndCopy(src, dst) {
  * <pre>
       app.config(['$resourceProvider', function ($resourceProvider) {
         // Don't strip trailing slashes from calculated URLs
-        $resourceProvider.setStripTrailingSlashes(false);
+        $resourceProvider.defaults.stripTrailingSlashes = false;
       }]);
    </pre>
  *
@@ -153,6 +153,14 @@ function shallowClearAndCopy(src, dst) {
  *   - **`interceptor`** - `{Object=}` - The interceptor object has two optional methods -
  *     `response` and `responseError`. Both `response` and `responseError` interceptors get called
  *     with `http response` object. See {@link ng.$http $http interceptors}.
+ *
+ * @param {Object} options Hash with custom settings that should extend the
+ *   default `$resourceProvider` behavior.  The only supported option is
+ *
+ *   Where:
+ *
+ *   - **`stripTrailingSlashes`** – {boolean} – If true then the trailing
+*   slashes from any calculated URL will be stripped. (Defaults to true.)
  *
  * @returns {Object} A resource "class" object with methods for the default set of resource actions
  *   optionally extended with custom `actions`. The default set contains these actions:
@@ -316,17 +324,13 @@ function shallowClearAndCopy(src, dst) {
  */
 angular.module('ngResource', ['ng']).
   provider('$resource', function () {
-    var _self = this;
+    var provider = this;
 
-    this.config = {
+    this.defaults = {
       // Strip slashes by default
       stripTrailingSlashes: true,
     };
 
-    this.setStripTrailingSlashes = function (value) {
-      _self.config.stripTrailingSlashes = value;
-    };
-
     this.$get = ['$http', '$q', function($http, $q) {
 
       var DEFAULT_ACTIONS = {
@@ -383,7 +387,7 @@ angular.module('ngResource', ['ng']).
 
       function Route(template, defaults) {
         this.template = template;
-        this.defaults = defaults || {};
+        this.defaults = extend({}, provider.defaults, defaults);
         this.urlParams = {};
       }
 
@@ -425,7 +429,7 @@ angular.module('ngResource', ['ng']).
           });
 
           // strip trailing slashes and set the url (unless this behaviour is specifically disabled)
-          if (_self.config.stripTrailingSlashes) {
+          if (self.defaults.stripTrailingSlashes) {
             url = url.replace(/\/+$/, '');
           }
 
@@ -447,8 +451,8 @@ angular.module('ngResource', ['ng']).
       };
 
 
-      function resourceFactory(url, paramDefaults, actions) {
-        var route = new Route(url);
+      function resourceFactory(url, paramDefaults, actions, options) {
+        var route = new Route(url, options);
 
         actions = extend({}, DEFAULT_ACTIONS, actions);
 

test/ngResource/resourceSpec.js

@@ -182,14 +182,21 @@ describe("resource", function() {
   });
 
 
-  it('should not strip trailing slashes if configured not to', function() {
-    // Set the new behaviour
-    resourceProvider.setStripTrailingSlashes(false);
+  it('should not strip the trailing slashes from URLs if configured', function() {
+    // Set the new behaviour for all new resources created by overriding the
+    // provider configuration
+    resourceProvider.defaults.stripTrailingSlashes = false;
 
     var R = $resource('http://localhost:8080/Path/:a/');
 
     $httpBackend.expect('GET', 'http://localhost:8080/Path/foo/').respond();
     R.get({a: 'foo'});
+
+    // Specific instances of $resource can still override the provider's default
+    R = $resource('http://localhost:8080/Path/:a/', {}, {}, {stripTrailingSlashes: true});
+
+    $httpBackend.expect('GET', 'http://localhost:8080/Path/foo').respond();
+    R.get({a: 'foo'});
   });