Opened 8 years ago

Closed 8 years ago

#4549 closed New Feature (fixed)

Make the querystring "anti-cache" feature configurable

Reported by: Frederico Caldeira Knabben Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.1
Component: General Version:
Keywords: Confirmed Review+ Cc:

Description

The anti-cache feature we have in the editor is generally useful, but may also be an issue on certain cases (as commented in our blog).

To give the proper level of flexibility, we could make this feature configurable, so one could decide to not use it. We'll probably need a global variable for that, like "CKEDITOR_NO_ANTI_CACHE".

Attachments (1)

4549.patch (1.3 KB) - added by Garry Yao 8 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 8 years ago by Frederico Caldeira Knabben

Humm... it looks like it should be enough to set CKEDITOR.timestamp to an empty string right after loading the editor code. The only problem is the config.js file URL is built on the core code execution time, so the timestamp will always be there.

more investigation is needed.

Changed 8 years ago by Garry Yao

Attachment: 4549.patch added

comment:2 Changed 8 years ago by Garry Yao

Keywords: Review? added
Owner: set to Garry Yao
Status: newassigned

The correct way for configure this should be simply remove the CKEDITOR.timestamp value.

You'll need to update editor's test profile for this TC, as well as point 'CKEDITOR_ROOT' environment variable to a released folder.
http://ckeditor.t/tt/4549/1.html.

comment:3 Changed 8 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

comment:4 Changed 8 years ago by Garry Yao

Resolution: fixed
Status: assignedclosed

Fixed with [4585] at 3.1.x branch.

Note: See TracTickets for help on using tickets.
© 2003 – 2017 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy