#8698 closed (fixed)
Middleware Documentation still refers to CacheMiddleware
Reported by: | anonymous | Owned by: | nobody |
---|---|---|---|
Component: | Documentation | Version: | dev |
Severity: | Keywords: | middleware, cache | |
Cc: | Triage Stage: | Unreviewed | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
On the Cache documentation page, it says you have to use UpdateCacheMiddleware and FetchFromCacheMiddleware instead of the older CacheMiddleware. However the Middleware documentation only mentions CacheMiddleware and does not discuss the two new middlewares.
Attachments (1)
Change History (6)
comment:1 by , 16 years ago
Resolution: | → worksforme |
---|---|
Status: | new → closed |
comment:2 by , 16 years ago
Resolution: | worksforme |
---|---|
Status: | closed → reopened |
This isn't a good default strategy. You can still use CacheMiddleware
is it's working for you now or you really know what you're doing. It's far safer to use the split version, though, since it means you can safely add other middleware later without having to hurt your brain trying to work out if it's something that will affect the cache key. It's an omission in the middleware docs, in any case, since these two middleware pieces exist.
by , 16 years ago
Attachment: | 8698-middleware-docs-cachemiddleware.diff added |
---|
patch for middleware documentation
comment:3 by , 16 years ago
Has patch: | set |
---|
comment:4 by , 16 years ago
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
Most people can still use
CacheMiddleware
, andCacheMiddleware
still exists; it's just in certain situations -- as described in the caching docs -- that you need to use the separate bits to avoid problems.