Opened 7 years ago

Closed 7 years ago

#4549 closed New Feature (fixed)

Make the querystring "anti-cache" feature configurable

Reported by: fredck 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 7 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 7 years ago by fredck

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 7 years ago by garry.yao

comment:2 Changed 7 years ago by garry.yao

  • Keywords Review? added
  • Owner set to garry.yao
  • Status changed from new to assigned

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 7 years ago by fredck

  • Keywords Review+ added; Review? removed

comment:4 Changed 7 years ago by garry.yao

  • Resolution set to fixed
  • Status changed from assigned to closed

Fixed with [4585] at 3.1.x branch.

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