X-Git-Url: http://git.ithinksw.org/extjs.git/blobdiff_plain/c930e9176a5a85509c5b0230e2bff5c22a591432..25ef3491bd9ae007ff1fc2b0d7943e6eaaccf775:/src/data/DirectProxy.js diff --git a/src/data/DirectProxy.js b/src/data/DirectProxy.js index 14475874..acf84bd4 100644 --- a/src/data/DirectProxy.js +++ b/src/data/DirectProxy.js @@ -1,5 +1,5 @@ /*! - * Ext JS Library 3.0.0 + * Ext JS Library 3.0.3 * Copyright(c) 2006-2009 Ext JS, LLC * licensing@extjs.com * http://www.extjs.com/license @@ -52,31 +52,35 @@ paramOrder: 'param1|param2|param' switch (action) { case Ext.data.Api.actions.create: - args.push(params[reader.meta.root]); // <-- create(Hash) + args.push(params.jsonData[reader.meta.root]); // <-- create(Hash) break; case Ext.data.Api.actions.read: - if(this.paramOrder){ - for(var i = 0, len = this.paramOrder.length; i < len; i++){ - args.push(params[this.paramOrder[i]]); + // If the method has no parameters, ignore the paramOrder/paramsAsHash. + if(directFn.directCfg.method.len > 0){ + if(this.paramOrder){ + for(var i = 0, len = this.paramOrder.length; i < len; i++){ + args.push(params[this.paramOrder[i]]); + } + }else if(this.paramsAsHash){ + args.push(params); } - }else if(this.paramsAsHash){ - args.push(params); } break; case Ext.data.Api.actions.update: - args.push(params[reader.meta.idProperty]); // <-- save(Integer/Integer[], Hash/Hash[]) - args.push(params[reader.meta.root]); + args.push(params.jsonData[reader.meta.root]); // <-- update(Hash/Hash[]) break; case Ext.data.Api.actions.destroy: - args.push(params[reader.meta.root]); // <-- destroy(Int/Int[]) + args.push(params.jsonData[reader.meta.root]); // <-- destroy(Int/Int[]) break; } var trans = { params : params || {}, - callback : callback, - scope : scope, - arg : options, + request: { + callback : callback, + scope : scope, + arg : options + }, reader: reader }; @@ -93,7 +97,7 @@ paramOrder: 'param1|param2|param' this.fireEvent("loadexception", this, trans, res, null); } this.fireEvent('exception', this, 'remote', action, trans, res, null); - trans.callback.call(trans.scope, null, trans.arg, false); + trans.request.callback.call(trans.request.scope, null, trans.request.arg, false); return; } if (action === Ext.data.Api.actions.read) { @@ -120,11 +124,11 @@ paramOrder: 'param1|param2|param' this.fireEvent("loadexception", this, trans, res, ex); this.fireEvent('exception', this, 'response', action, trans, res, ex); - trans.callback.call(trans.scope, null, trans.arg, false); + trans.request.callback.call(trans.request.scope, null, trans.request.arg, false); return; } - this.fireEvent("load", this, res, trans.arg); - trans.callback.call(trans.scope, records, trans.arg, true); + this.fireEvent("load", this, res, trans.request.arg); + trans.request.callback.call(trans.request.scope, records, trans.request.arg, true); }, /** * Callback for write actions @@ -134,8 +138,9 @@ paramOrder: 'param1|param2|param' * @private */ onWrite : function(action, trans, result, res, rs) { - this.fireEvent("write", this, action, result, res, rs, trans.arg); - trans.callback.call(trans.scope, result, res, true); + var data = trans.reader.extractData(result); + this.fireEvent("write", this, action, data, res, rs, trans.request.arg); + trans.request.callback.call(trans.request.scope, data, res, true); } });