mbk-lab
/
nextcloud
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Fix redundant headers in files dav client 

The defaultHeaders are already injected with the xhrProvider, so no need
to pass them again a second time.
This commit is contained in:
Vincent Petry 2016-02-16 17:28:35 +01:00
parent 273fb75824
commit 5575443be9
1 changed files with 7 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
core/js/files/client.js
View File

@ -394,7 +394,6 @@
properties = options.properties;
}
// TODO: headers
this._client.propFind(
this._buildUrl(path),
properties,
@ -441,7 +440,6 @@
throw 'Missing filter argument';
}
var headers = _.extend({}, this._defaultHeaders);
// root element with namespaces
var body = '<oc:filter-files ';
var namespace;
@ -472,7 +470,7 @@
this._client.request(
'REPORT',
this._buildUrl(),
headers,
{},
body
).then(function(result) {
if (self._isSuccessStatus(result.status)) {
@ -542,8 +540,7 @@
this._client.request(
'GET',
this._buildUrl(path),
this._defaultHeaders
this._buildUrl(path)
).then(
function(result) {
if (self._isSuccessStatus(result.status)) {
@ -575,7 +572,7 @@
var deferred = $.Deferred();
var promise = deferred.promise();
options = options || {};
var headers = _.extend({}, this._defaultHeaders);
var headers = {};
var contentType = 'text/plain;charset=utf-8';
if (options.contentType) {
contentType = options.contentType;
@ -616,8 +613,7 @@
this._client.request(
method,
this._buildUrl(path),
this._defaultHeaders
this._buildUrl(path)
).then(
function(result) {
if (self._isSuccessStatus(result.status)) {
@ -673,10 +669,9 @@
var self = this;
var deferred = $.Deferred();
var promise = deferred.promise();
var headers =
_.extend({
'Destination' : this._buildUrl(destinationPath)
}, this._defaultHeaders);
var headers = {
'Destination' : this._buildUrl(destinationPath)
};
if (!allowOverwrite) {
headers['Overwrite'] = 'F';