Ticket #4549 (closed New Feature: fixed)

Opened 5 months ago

Last modified 4 months ago

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

4549.patch Download (1.3 KB) - added by garry.yao 4 months ago.

Change History

Changed 5 months 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 4 months ago by garry.yao

Changed 4 months 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.

Changed 4 months ago by fredck

  • keywords Review+ added; Review? removed

Changed 4 months ago by garry.yao

  • status changed from assigned to closed
  • resolution set to fixed

Fixed with [4585] at 3.1.x branch.

Note: See TracTickets for help on using tickets.