Merge pull request #6348 from zinyando/adapter-fix

Make sure super in buildURL passes all arguments
This commit is contained in:
Kevin Ansfield 2016-01-16 18:40:41 +00:00
commit 9fda1dc5ba
2 changed files with 6 additions and 6 deletions

View File

@ -26,9 +26,9 @@ export default RESTAdapter.extend({
return this.ajax(this.buildURL(type.modelName, id), 'GET', {data: query}); return this.ajax(this.buildURL(type.modelName, id), 'GET', {data: query});
}, },
buildURL(type, id) { buildURL() {
// Ensure trailing slashes // Ensure trailing slashes
let url = this._super(type, id); let url = this._super(...arguments);
if (url.slice(-1) !== '/') { if (url.slice(-1) !== '/') {
url += '/'; url += '/';

View File

@ -43,7 +43,7 @@ export default BaseAdapter.extend({
}, },
createRecord(store, type, snapshot) { createRecord(store, type, snapshot) {
return this.saveRecord(store, type, snapshot, {method: 'POST'}); return this.saveRecord(store, type, snapshot, {method: 'POST'}, 'createRecord');
}, },
updateRecord(store, type, snapshot) { updateRecord(store, type, snapshot) {
@ -52,12 +52,12 @@ export default BaseAdapter.extend({
id: get(snapshot, 'id') id: get(snapshot, 'id')
}; };
return this.saveRecord(store, type, snapshot, options); return this.saveRecord(store, type, snapshot, options, 'updateRecord');
}, },
saveRecord(store, type, snapshot, options) { saveRecord(store, type, snapshot, options, requestType) {
let _options = options || {}; let _options = options || {};
let url = this.buildIncludeURL(store, type.modelName, _options.id, snapshot, 'createRecord'); let url = this.buildIncludeURL(store, type.modelName, _options.id, snapshot, requestType);
let payload = this.preparePayload(store, type, snapshot); let payload = this.preparePayload(store, type, snapshot);
return this.ajax(url, _options.method, payload); return this.ajax(url, _options.method, payload);