Wikimedia Bugzilla is closed!

When performing cache checks for localization files, both
self::isLocalisationOutOfDate( $cache ) and a foreach loop comparing filemtime
of the dependencies are done.  This appears to be redundant, as
isLocalisationOutOfDate appears to do the same process, and the $expired
variable does not appear to be used.

If they are not actually redundant, recommend adding a comment as to the
function of the code block. I do not have a working memcached extension so I
cannot say.

The discrepancy manifested in trunk on r15823 when Tim Starling merged in the
localisation-work branch. The switch to the isLocalisationOutOfDate appeared to
have happened r15804, and replaced the $expired variable, strongly suggesting
that yes, this was an oversight on Tim Starling's part.

r1=15803&r2=15804">http://svn.wikimedia.org/viewvc/mediawiki/branches/localisation-work/phase3/languages/Language.php?view=diff&r1=15803&r2=15804

It is unlikely that this has caused significant performance degradation.

Patch forthcoming.