The CacheTime class currently accesses $wgParserCacheExpireTime directly to check for expiry. However, CacheTime, which acts as a base class for entries in ParserCache, should be a value object - it not use global state and should not know about configuration variables. Checking the expiry time against configuration and wall clock time should be left to the cache object, it should not be done by the entry itself.
Description
Description
Details
Details
Project | Branch | Lines +/- | Subject | |
---|---|---|---|---|
mediawiki/core | master | +58 -43 | ParserCache: don't rely on $wgParserCacheExpireTime |
Status | Subtype | Assigned | Task | ||
---|---|---|---|---|---|
Open | daniel | T270710 Allow values other than ParserOutput to be stored in a ParserCache instance | |||
Resolved | Pchelolo | T250500 ParserCache / RESTBase / Parsoid integration | |||
Open | daniel | T271112 Do not access global $wgParserCacheExpireTime directly |
Event Timeline
Comment Actions
Change 651205 had a related patch set uploaded (by Daniel Kinzler; owner: Daniel Kinzler):
[mediawiki/core@master] ParserCache: don't rely on $wgParserCacheExpireTime
Comment Actions
Adding Platform Engineering as Platform Team Workboards (Green) was archived and as open tasks should have an active project tag