fix($resource): null default param results in TypeError

jrschumacher · IgorMinar · commit cefbcd470d4c · 2013-05-16T14:26:08.000-07:00
Fixes issue when setting a default param as `null` error
`TypeError: Cannot read property 'charAt' of null`
diff --git a/src/ngResource/resource.js b/src/ngResource/resource.js
@@ -392,7 +392,7 @@ angular.module('ngResource', ['ng']).
         actionParams = extend({}, paramDefaults, actionParams);
         forEach(actionParams, function(value, key){
           if (isFunction(value)) { value = value(); }
-          ids[key] = value.charAt && value.charAt(0) == '@' ? getter(data, value.substr(1)) : value;
+          ids[key] = value && value.charAt && value.charAt(0) == '@' ? getter(data, value.substr(1)) : value;
         });
         return ids;
       }
diff --git a/test/ngResource/resourceSpec.js b/test/ngResource/resourceSpec.js
@@ -205,6 +205,16 @@ describe("resource", function() {
   });
 
 
+  it('should not throw TypeError on null default params', function() {
+    $httpBackend.expect('GET', '/Path?').respond('{}');
+    var R = $resource('/Path', {param: null}, {get: {method: 'GET'}});
+
+    expect(function() {
+      R.get({});
+    }).not.toThrow();
+  });
+
+
   it('should handle multiple params with same name', function() {
     var R = $resource('/:id/:id');
 

Original file line number	Diff line number	Diff line change
`@@ -392,7 +392,7 @@ angular.module('ngResource', ['ng']).`
`392`	`392`	`actionParams = extend({}, paramDefaults, actionParams);`
`393`	`393`	`forEach(actionParams, function(value, key){`
`394`	`394`	`if (isFunction(value)) { value = value(); }`
`395`		`- ids[key] = value.charAt && value.charAt(0) == '@' ? getter(data, value.substr(1)) : value;`
	`395`	`+ ids[key] = value && value.charAt && value.charAt(0) == '@' ? getter(data, value.substr(1)) : value;`
`396`	`396`	`});`
`397`	`397`	`return ids;`
`398`	`398`	`}`