Page MenuHomePhabricator

Do not access global $wgParserCacheExpireTime directly
Open, Needs TriagePublic


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.