Opened 13 years ago
Closed 9 years ago
#9957 closed enhancement (wontfix)
[patch] [ccla] provide xhrargs for QueryReadStore
Reported by: | percious | Owned by: | wolfram |
---|---|---|---|
Priority: | high | Milestone: | future |
Component: | DojoX Data | Version: | 1.3.2 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
This patch provides a new variable for QRS: xhrargs. The xhargs are mixin'd with the args generated by the regular querying method before the xhr request is sent. This is needed for folks who want to provide preventCache setting to their xhr calls within QRS. This is currently implemented in tw.dojo (a python wrapper for Dojo)
Attachments (4)
Change History (8)
Changed 13 years ago by
Attachment: | qrs_xhrargs.2.diff added |
---|
Changed 13 years ago by
Attachment: | qrs_xhrargs.3.diff added |
---|
Changed 13 years ago by
Attachment: | qrs_xhrargs.diff added |
---|
comment:1 Changed 13 years ago by
Owner: | changed from Jared Jurkiewicz to wolfram |
---|
comment:2 Changed 13 years ago by
Milestone: | tbd → future |
---|---|
Summary: | patch: provide xhrargs for QueryReadStore → [patch] [no cla] provide xhrargs for QueryReadStore |
Changed 13 years ago by
Attachment: | 9957_xhrargs_xhrfunc.diff added |
---|
Updated patch to [20681], includes tests and allows overriding IO function via xhrFunc argument.
comment:3 Changed 13 years ago by
Summary: | [patch] [no cla] provide xhrargs for QueryReadStore → [patch] [ccla] provide xhrargs for QueryReadStore |
---|
9957_xhrargs_xhrfunc.diff is a rework of Chris' patch:
- specifying
xhrArgs
- specifying
xhrFunc
so people can usedojo.io.script
or other fun things to access their stores. Only a single function can be passed (same is used for both post & get). - unit tests for the above.
I'm happy to commit it, but QueryReadStore isn't my area of expertise, so I'd appreciate a quick review.
comment:4 Changed 9 years ago by
Resolution: | → wontfix |
---|---|
Status: | new → closed |
Because dojo.data in general is deprecated in favor of dojo.store (with QueryReadStore sort-of replaced by dojo/store/JsonRest)
Hi Chris, please file a CLA from http://dojofoundation.org/cla/.