Closed (fixed)
Project:
Drupal core
Version:
6.x-dev
Component:
update.module
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
15 Jan 2008 at 07:04 UTC
Updated:
30 Jan 2008 at 12:52 UTC
Jump to comment: Most recent file
While working on http://drupal.org/node/209242, I noticed that we don't flush the available update cache during update.php. That'd be another smart time to invalidate the cache, since whenever we're running update.php, it almost certainly means we installed a newer version of something, in which case, we want to check for available update data again.
Splitting this off into a separate issue/patch, since regardless of what you think about all the mtime business over at http://drupal.org/node/209242, this is still worth doing.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | update_flush_caches_comment.patch | 1.04 KB | dww |
| update_flush_caches.patch | 645 bytes | dww |
Comments
Comment #1
dries commentedI've added some code comments and committed this to CVS HEAD. Thanks, Derek.
Comment #2
dwwCool, thanks.
Minor followup patch to clean up the comment to meet our current conventions:
- First sentence of the phpdoc should be a summary and on its own line.
- No 2 spaces after periods.
- Wrap everything to 80 cols.
Comment #3
gábor hojtsyThanks, committed.
Comment #4
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.