Opened 11 years ago

Closed 10 years ago

Last modified 10 years ago

#9186 closed enhancement (fixed)

All stores should allow "preventCache" to be disabled

Reported by: hieg Owned by: Jared Jurkiewicz
Priority: high Milestone: 1.4
Component: DojoX Data Version: 1.3.0
Keywords: Cc:
Blocked By: Blocking:

Description

Currently the XmlStore? has preventCache: true by default. This should be modified to be an attribute that can be passed when the store is created. I encountered this problem when trying to integrate Dojo with CodeIgniter? which uses urls like: /controller/function/argument rather than controller.php?function=X&argument=X. The problem I had was that my store was calling "/main/xml/" which is a function in main.php called xml() which outputs an xml string. When the store is created and it calls the url for the xml it calls "/main/xml/?dojo.preventCache=XXX" which results in a 404. As my app handles any cacheing through headers (and caching is not a major concern for me) the "dojo.preventCache=XXX" is not needed and I'd like to be able to turn it off. Currently I have to edit dojox/data/XmlStore.js to do so.

I haven't checked all the other store types, but I assume this is the case with most, if not all of them.

Change History (18)

comment:1 Changed 11 years ago by Rob Retchless

I agree that this needs to be fixed. I'm implementing a page using XmlStore?, and the XML file is being redownloaded with every fetch (pagination, filtering, etc). Is there a reason why this data store would want to prevent caching?

comment:2 Changed 10 years ago by Jared Jurkiewicz

The reason is the store is designed to operate both as remote AND local. It has a serverQuery mode, where you would want to prevent caching since the server is doing all the query processing.

comment:3 Changed 10 years ago by Jared Jurkiewicz

(In [17472]) Minor feature add. Ability to turnoff preventCache. refs #9186

comment:4 Changed 10 years ago by Jared Jurkiewicz

(In [17679]) Minor tweak to allow for urlPreventCache. \!strict refs #9186

comment:5 Changed 10 years ago by Jared Jurkiewicz

(In [17680]) Minor tweak to allow for urlPreventCache. \!strict refs #9186

comment:6 Changed 10 years ago by Jared Jurkiewicz

(In [17681]) Minor tweak to allow for urlPreventCache. \!strict refs #9186

comment:7 Changed 10 years ago by Jared Jurkiewicz

(In [17682]) Minor tweak to allow for urlPreventCache. \!strict refs #9186

comment:8 Changed 10 years ago by Jared Jurkiewicz

(In [17683]) Minor tweak to allow for urlPreventCache. \!strict refs #9186

comment:9 Changed 10 years ago by Jared Jurkiewicz

(In [17684]) Minor tweak to allow for urlPreventCache. \!strict refs #9186

comment:10 Changed 10 years ago by Jared Jurkiewicz

(In [17685]) Minor tweak to allow for urlPreventCache. \!strict refs #9186

comment:11 Changed 10 years ago by Jared Jurkiewicz

(In [17686]) Minor tweak to allow for urlPreventCache. \!strict refs #9186

comment:12 Changed 10 years ago by Jared Jurkiewicz

ItemFile?(Read/Write?)Store: Supported AndOr?(Read/Write?)Store: Supported XmlStore?: Supported AppStore?: Supported AtomReadStore?: Supported CouchDBRestStore: Not Supported (Unknown if it should even allow prevent cache. Not touching.) CssClassStore?: Not applicable. CssRulesStore?: Not applicable. FlickrStore?: Supported. FlickrRestStore?: Supported (Though FlickrStore? base class) Google*Store: Supported. HtmlStore?: Supported HtmlTableStore?: Not supported. Deprecated anyway, use HtmlStore?. jsonPathStore: Not used. JsonQueryRestStore?: Not used. JsonRestStore?: Not used. KeyValueStore?: Supported. OpmlStore?: Supported. PicasaStore?: Supported. CdfStore?: Unknown. QueryReadStore?: Not Supported. RailsStore?: Not used. SnapLogicStore?: Not Supported.

comment:13 Changed 10 years ago by Jared Jurkiewicz

ItemFile?(Read/Write?)Store: Supported

AndOr?(Read/Write?)Store: Supported

XmlStore?: Supported

AppStore?: Supported

AtomReadStore?: Supported

CouchDBRestStore: Not Supported (Unknown if it should even allow prevent cache. Not touching.)

CssClassStore?: Not applicable.

CssRulesStore?: Not applicable.

FlickrStore?: Supported.

FlickrRestStore?: Supported (Though FlickrStore? base class)

Google*Store: Supported.

HtmlStore?: Supported

HtmlTableStore?: Not supported. Deprecated anyway, use HtmlStore?.

jsonPathStore: Not used.

JsonQueryRestStore?: Not used.

JsonRestStore?: Not used.

KeyValueStore?: Supported.

OpmlStore?: Supported.

PicasaStore?: Supported.

CdfStore?: Unknown.

QueryReadStore?: Not Supported.

RailsStore?: Not used.

SnapLogicStore?: Not Supported.

comment:14 Changed 10 years ago by Jared Jurkiewicz

Resolution: fixed
Status: newclosed

comment:15 Changed 10 years ago by Jared Jurkiewicz

(In [17688]) Minor tweak to allow for urlPreventCache. \!strict refs #9186

comment:16 Changed 10 years ago by Jared Jurkiewicz

(In [17689]) Minor tweak to allow for urlPreventCache. \!strict refs #9186

comment:17 Changed 10 years ago by Jared Jurkiewicz

(In [17700]) Minor tweak to allow for urlPreventCache. Need to make sure agrs is defined first! \!strict refs #9186

comment:18 Changed 10 years ago by Adam Peller

Milestone: tbd1.4
Note: See TracTickets for help on using tickets.