From 26fa079c55310c2429da20b68d084ea3fa66af09 Mon Sep 17 00:00:00 2001
From: Vincent Petry <pvince81@owncloud.com>
Date: Tue, 29 Aug 2017 14:56:43 +0200
Subject: [PATCH] Fix REPORT method when going through Backbone-Webdav adapter

---
 core/js/oc-backbone-webdav.js                 | 12 +++-
 core/js/tests/specs/oc-backbone-webdavSpec.js | 67 +++++++++++++++++++
 2 files changed, 77 insertions(+), 2 deletions(-)

diff --git a/core/js/oc-backbone-webdav.js b/core/js/oc-backbone-webdav.js
index 0503abea8bda..5b8146dc51ac 100644
--- a/core/js/oc-backbone-webdav.js
+++ b/core/js/oc-backbone-webdav.js
@@ -252,12 +252,20 @@
 	}
 
 	function callMethod(client, options, model, headers) {
-		headers['Content-Type'] = 'application/json';
+		var data = options.data;
+		if (_.isObject(data)) {
+			headers['Content-Type'] = 'application/json';
+			data = JSON.stringify(data);
+		} else if (_.isString(data) && data.substr(0, 6) === '<?xml ') {
+			headers['Content-Type'] = 'application/xml';
+		} else {
+			headers['Content-Type'] = 'text/plain';
+		}
 		return client.request(
 			options.type,
 			options.url,
 			headers,
-			JSON.stringify(options.data)
+			data
 		).then(function(result) {
 			if (!isSuccessStatus(result.status)) {
 				if (_.isFunction(options.error)) {
diff --git a/core/js/tests/specs/oc-backbone-webdavSpec.js b/core/js/tests/specs/oc-backbone-webdavSpec.js
index 30b64fbdb5d7..5b547e6b3eea 100644
--- a/core/js/tests/specs/oc-backbone-webdavSpec.js
+++ b/core/js/tests/specs/oc-backbone-webdavSpec.js
@@ -898,5 +898,72 @@ describe('Backbone Webdav extension', function() {
 			expect(model.isNew()).toEqual(false);
 		});
 	});
+
+	describe('custom method', function() {
+		var TestModel;
+
+		beforeEach(function() {
+			TestModel = OC.Backbone.Model.extend({
+				url: 'http://example.com/owncloud/remote.php/test/1',
+				sync: OC.Backbone.davSync,
+				customCall: function(data, options) {
+					options = _.extend({}, options);
+					options.data = data;
+					return this.sync('REPORT', this, options);
+				}
+			});
+		});
+
+		it('serializes JSON if a JS object is passed as data', function() {
+			var model = new TestModel();
+			model.customCall({someData: '123'});
+
+			expect(davClientRequestStub.calledOnce).toEqual(true);
+			expect(davClientRequestStub.getCall(0).args[0])
+				.toEqual('REPORT');
+			expect(davClientRequestStub.getCall(0).args[1])
+				.toEqual('http://example.com/owncloud/remote.php/test/1');
+			expect(davClientRequestStub.getCall(0).args[2]['Content-Type'])
+				.toEqual('application/json');
+			expect(davClientRequestStub.getCall(0).args[2]['X-Requested-With'])
+				.toEqual('XMLHttpRequest');
+			expect(davClientRequestStub.getCall(0).args[3])
+				.toEqual(JSON.stringify({someData: '123'}));
+		});
+		it('does not serializes JSON if a string is passed as data', function() {
+			var model = new TestModel();
+			model.customCall('whatever');
+
+			expect(davClientRequestStub.calledOnce).toEqual(true);
+			expect(davClientRequestStub.getCall(0).args[0])
+				.toEqual('REPORT');
+			expect(davClientRequestStub.getCall(0).args[1])
+				.toEqual('http://example.com/owncloud/remote.php/test/1');
+			expect(davClientRequestStub.getCall(0).args[2]['Content-Type'])
+				.toEqual('text/plain');
+			expect(davClientRequestStub.getCall(0).args[2]['X-Requested-With'])
+				.toEqual('XMLHttpRequest');
+			expect(davClientRequestStub.getCall(0).args[3])
+				.toEqual('whatever');
+		});
+		it('sends XML mime type when passed data string is XML', function() {
+			var model = new TestModel();
+			var body = '<?xml version="1.0" encoding="utf-8" ?>';
+			body += '<root></root>';
+			model.customCall(body);
+
+			expect(davClientRequestStub.calledOnce).toEqual(true);
+			expect(davClientRequestStub.getCall(0).args[0])
+				.toEqual('REPORT');
+			expect(davClientRequestStub.getCall(0).args[1])
+				.toEqual('http://example.com/owncloud/remote.php/test/1');
+			expect(davClientRequestStub.getCall(0).args[2]['Content-Type'])
+				.toEqual('application/xml');
+			expect(davClientRequestStub.getCall(0).args[2]['X-Requested-With'])
+				.toEqual('XMLHttpRequest');
+			expect(davClientRequestStub.getCall(0).args[3])
+				.toEqual(body);
+		});
+	});
 });