When I run a Drupal cache clear, the css_js_query_string variable gets reset, which avoids having to hard-clear my browser's cached files in order to get updated JS. However, CKEditor uses its own cache-busting string, and the scripts loaded by CKEditor can get out of date when doing a deployment. It would be good to force CKEditor to re-use the same string.

This was a particular problem with an install profile I was using that provided it's own 'customConfig' file.

For reference:
http://ckeditor.com/forums/CKEditor/Problem-with-config.js-not-updating
http://ckeditor.com/forums/CKEditor/howto-disable-anti-cache-for-toolbar...
http://stackoverflow.com/questions/14940452/force-ckeditor-to-refresh-co...

CommentFileSizeAuthor
#4 2679106-ckeditor-reuse-cache-busting-query-898d02-do-not-test.patch1.4 KBDave Reid
#2 2679106-ckeditor-reuse-cache-busting-query.patch1.39 KBDave Reid
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dave Reid created an issue. See original summary.

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid as a volunteer and at Lullabot commented
Status: Active » Needs review
FileSize
2679106-ckeditor-reuse-cache-busting-query.patch1.39 KB
Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid as a volunteer and at Lullabot commented
Issue summary: View changes
Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid as a volunteer and at Lullabot commented
FileSize
2679106-ckeditor-reuse-cache-busting-query-898d02-do-not-test.patch1.4 KB

Patch against earlier hash 898d02 which we were currently using on our project.

Wim Leers’s picture

Wim Leers
Location Ghent 🇧🇪🇪🇺
CreditAttribution: Wim Leers at Acquia commented

Already landed in Drupal 8 core, with test coverage: #2679903: CKEditor uses separate cache-busting query string from Drupal's.

naxoc’s picture

naxoc CreditAttribution: naxoc commented
Status: Needs review » Reviewed & tested by the community

Patch in #2679106-2: CKEditor uses separate cache-busting query string from Drupal's works for me.

Liam Morland’s picture

Liam Morland
Primary language English
Location Ontario, CA 🇨🇦
CreditAttribution: Liam Morland at University of Waterloo commented

University of Waterloo has been running this patch for several months.

  • TwoD committed c69f19b on 6.x-2.x authored by Dave Reid 
    - #2679106 by Dave Reid, Wim Leers, naxoc, Liam Morland: CKEditor uses...
  • TwoD committed 803fdf4 on 7.x-2.x authored by Dave Reid
    - #2679106 by Dave Reid, Wim Leers, naxoc, Liam Morland: CKEditor uses...
TwoD’s picture

TwoD
Primary language Swedish
Location Sweden
CreditAttribution: TwoD as a volunteer commented
Status: Reviewed & tested by the community » Fixed

Simple (and overdue) enough, thanks all!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.