Fixes mgonto#955

grabbou · grabbou · commit 309891a39fb7 · 2015-02-24T10:45:48.000+01:00
diff --git a/src/restangular.js b/src/restangular.js
@@ -1198,13 +1198,13 @@ module.provider('Restangular', function() {
             deferred.reject(response);
           }
         };
-        // Overring HTTP Method
+        // Overriding HTTP Method
         var callOperation = operation;
         var callHeaders = _.extend({}, request.headers);
         var isOverrideOperation = config.isOverridenMethod(operation);
         if (isOverrideOperation) {
           callOperation = 'post';
-          callHeaders = _.extend(callHeaders, {'X-HTTP-Method-Override': operation === 'remove' ? 'DELETE' : operation});
+          callHeaders = _.extend(callHeaders, {'X-HTTP-Method-Override': operation === 'remove' ? 'DELETE' : operation.toUpperCase()});
         } else if (config.jsonp && callOperation === 'get') {
           callOperation = 'jsonp';
         }
diff --git a/test/restangularSpec.js b/test/restangularSpec.js
@@ -883,14 +883,22 @@ describe("Restangular", function() {
     });
   });
 
-  describe("defaultHeaders", function() {
+  describe("headers", function() {
     it("should return defaultHeaders", function() {
       var defaultHeaders = {testheader:'header value'};
-
       Restangular.setDefaultHeaders(defaultHeaders);
-
       expect(Restangular.defaultHeaders).toEqual(defaultHeaders);
     });
+
+    it("should pass uppercase methods in X-HTTP-Method-Override", function() {
+      Restangular.setMethodOverriders(["put"]);
+      $httpBackend.expectPOST('/overriders/1').respond(function(method, url, data, headers) {
+        expect(headers['X-HTTP-Method-Override']).toBe('PUT');
+        return {};
+      });
+      Restangular.one('overriders', 1).put();
+      $httpBackend.flush();
+    });
   });
 
   describe("defaultRequestParams", function() {
