Ticket #4924 (closed Bug: fixed)
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
Change History
comment:2 Changed 3 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)
Note: See
TracTickets for help on using
tickets.
