Ticket #4924 (closed Bug: fixed)

Opened 5 years ago

Last modified 5 years ago

PHP integration is not using the timestamp in the ckeditor.js script

Reported by: fredck Owned by: wwalc
Priority: Normal Milestone: CKEditor 3.1
Component: Server : PHP Version: SVN (CKEditor) - OLD
Keywords: Confirmed Review+ Cc:

Description

According to the documentation, the server side scripts must append the timestamp to the ckeditor.js file, so it updates the cache on new releases. The PHP integration is not currently doing so.

Attachments

4924.patch (1.9 KB) - added by wwalc 5 years ago.

Change History

comment:1 Changed 5 years ago by wwalc

  • Owner set to wwalc

Changed 5 years ago by wwalc

comment:2 Changed 5 years ago by wwalc

  • Keywords Review? added

The attached patch does two things:

  • "%TIMESTAMP%" is changed into "%"."TIMESTAMP%" to prevent against replacing this value by CKReleaser - we don't want to add "%TIMESTAMP%" to the query string in the SVN version
  • !empty($this->timestamp) was removed because CKEDITOR.timestamp = ''; should appear on the page even if timestamp is empty (this way config.js and other files are loaded without appended timestamp)

comment:3 Changed 5 years ago by fredck

  • Keywords Review+ added; Review? removed

comment:4 Changed 5 years ago by wwalc

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

Fixed with [4863].

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