fix($resource): functions in params should not be executed if overridden

src/ngResource/resource.js

@@ -557,10 +557,11 @@ angular.module('ngResource', ['ng']).
               }
             });
 
-            if (hasBody) httpConfig.data = data;
-            route.setUrlParams(httpConfig,
-              extend({}, extractParams(data, action.params || {}), params),
-              action.url);
+          if (hasBody) httpConfig.data = data;
+          params = extend({}, action.params || {}, params);
+          route.setUrlParams(httpConfig,
+                             extractParams(data, params),
+                             action.url);
 
             var promise = $http(httpConfig).then(function (response) {
               var data = response.data,

test/ngResource/resourceSpec.js

@@ -380,6 +380,17 @@ describe("resource", function() {
     inst.$post();
   });
 
+  it('should not call function in action param if overriden', function() {
+    $httpBackend.expect('GET', '/item/123').respond({foo: 'bar'});
+
+    var TypeItem = $resource('/item/:id', {}, {get: {method: 'GET', params: {id: callback}}});
+    var item = TypeItem.get({id: 123}); // overrides params.id
+
+    $httpBackend.flush();
+
+    expect(callback).not.toHaveBeenCalled();
+    expect(item).toEqualData({foo: 'bar'});
+  });
 
   it('should not throw TypeError on null default params', function() {
     $httpBackend.expect('GET', '/Path').respond('{}');