See #902860-4: Drush does not update the html after clearing the cache for js and css only
After a "drush cc css+js", cached html can still link to old css and js aggregates.

I only encountered this problem with drush cc (on D6), but admin_menu uses the same logic to flush the css+js cache ("page requisites"), so it is likely to have the same problem.

(it is exactly the same story on dqx_adminmenu, where I fixed it just now on the 6.x-3.x-dev)

Patch going to follow.

CommentFileSizeAuthor
#1 admin_menu.6-x-3-x.requisites.patch585 bytesdonquixote

Comments

donquixote’s picture

donquixote commented
Status: Active » Needs review
StatusFileSize
new admin_menu.6-x-3-x.requisites.patch585 bytes

This patch is a shot in the dark, I actually don't have the module enabled on any site atm.
Anyway, it should be exactly the same as in drush and dqx_adminmenu, so no reason why this should break.

Btw, I think it would be useful to rename this thing from "Page requisites" to "CSS+JS" or "CSS+JS+Page". Maybe even add a few more cache types, like those available in drush.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Title: "Page requisites" should also flush the page cache. » "Page requisites" should also flush the page cache
Version: 6.x-3.x-dev » 7.x-3.x-dev
Status: Needs review » Fixed

Good one!

Thanks for reporting, reviewing, and testing! Committed to all branches.

A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.

donquixote’s picture

donquixote commented

Thanks!
Btw, the drush guys have this as "won't fix" :/

Status: Fixed » Closed (fixed)

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