Ticket #3282 (closed Bug: fixed)

Opened 12 months ago

Last modified 11 months ago

Skin files are not being loaded with timestamp

Reported by: fredck Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.0
Component: General Version:
Keywords: Confirmed Review+ Cc:

Description

Looking at file requests in nightly build (skins) sample, it's possible to note that the skin.js file and the main CSS files don't have the timestamp ("t") parameter in the URL. We must ensure this parameter is properly sent to the server.

Attachments

3282.patch Download (1.1 KB) - added by garry.yao 12 months ago.
3282_2.patch Download (0.9 KB) - added by garry.yao 12 months ago.

Change History

Changed 12 months ago by garry.yao

  • owner set to garry.yao
  • status changed from new to assigned

Changed 12 months ago by garry.yao

Changed 12 months ago by garry.yao

  • keywords Review? added

Changed 12 months ago by fredck

  • keywords Review- added; Review? removed

The patch is broken. I'm not able to apply it.

Even though, the changes look by reading the patch file directly. But, I would prefer applying it for a final check.

Changed 12 months ago by garry.yao

Changed 12 months ago by garry.yao

  • keywords Review? added; Review- removed

Not sure why the original patch is broken.

Changed 11 months ago by martinkou

  • keywords Review+ added; Review? removed

Changed 11 months ago by garry.yao

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

Fixed with [3380].

Note: See TracTickets for help on using tickets.