(bug 30192) Old thumbnails not properly purged. Unlike the bug suggests, we don't...

A lot of this could probably be protected...not much calls these outside of FileRepo code other than File::purgeCache()

diff --git a/includes/filerepo/LocalFile.php b/includes/filerepo/LocalFile.php
index e940fd4..d879f07 100644 (file)
--- a/includes/filerepo/LocalFile.php
+++ b/includes/filerepo/LocalFile.php
@@ -655,6 +655,11 @@ class LocalFile extends File {
                $hashedName = md5( $this->getName() );
                $oldKey = $this->repo->getSharedCacheKey( 'oldfile', $hashedName );
 
+               // Must purge thumbnails for old versions too! bug 30192
+               foreach( $this->getHistory() as $oldFile ) {
+                       $oldFile->purgeThumbnails();
+               }
+
                if ( $oldKey ) {
                        $wgMemc->delete( $oldKey );
                }