Closed
Bug 450442
Opened 16 years ago
Closed 16 years ago
Make the front page object invalidation capable
Categories
(addons.mozilla.org Graveyard :: Public Pages, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
4.0.1
People
(Reporter: baz, Assigned: wenzel)
References
Details
Attachments
(1 file)
30.33 KB,
patch
|
clouserw
:
review+
|
Details | Diff | Splinter Review |
Need to modify the AMO front page to use the object invalidation framework.
Reporter | ||
Updated•16 years ago
|
Target Milestone: --- → 4.0.1
Comment 1•16 years ago
|
||
This will cover utilizing the improved cache policy from bug 425315.
Assignee: morgamic → fwenzel
Assignee | ||
Comment 2•16 years ago
|
||
This is the "full version" of attachment 337237 [details] [diff] [review]. The whole front page uses per-addon caching along with association-based add-on fetching, so we only haul around the data that's needed and can expire the cache on a per-addon basis. Note that the add-on model's afterSave() callback is currently the only place where the cache expiration is triggered. That's probably not the only place where this needs to happen though -- in the versions model as well, maybe?
Attachment #338263 -
Flags: review?(clouserw)
Comment 3•16 years ago
|
||
Comment on attachment 338263 [details] [diff] [review] Object-caching capable front page It doesn't have syntax errors. :) I didn't test all the aspects of it so we could land sooner and get more feedback time from everyone.
Attachment #338263 -
Flags: review?(clouserw) → review+
Assignee | ||
Comment 4•16 years ago
|
||
Checked in to r18252. I'll file another bug so we actually invalidate the add-on objects wherever necessary.
Assignee | ||
Comment 5•16 years ago
|
||
(In reply to comment #4) > I'll file another bug so we actually invalidate the > add-on objects wherever necessary. Filed: bug 455385.
Updated•16 years ago
|
Keywords: push-needed
Updated•8 years ago
|
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•