Add width to the cache key for thumburls in InstantCommons. No point in not caching...

diff --git a/RELEASE-NOTES b/RELEASE-NOTES
index 84be785..3e221d0 100644 (file)
--- a/RELEASE-NOTES
+++ b/RELEASE-NOTES
@@ -396,6 +396,7 @@ LocalSettings.php. The specific bugs are listed below in the general notes.
 * (bug 16902) Installer spews warnings when exec() and dl() are not available
 * (bug 19129) Only show MyISAM/InnoDB when supported
 * (bug 17762) Only show other e-mail options when e-mail is globally enabled
+* Cache multiple sizes of InstantCommons thumbnails
 
 === API changes in 1.17 ===
 * (bug 22738) Allow filtering by action type on query=logevent.

diff --git a/includes/filerepo/ForeignAPIRepo.php b/includes/filerepo/ForeignAPIRepo.php
index f6b21ee..631acf2 100644 (file)
--- a/includes/filerepo/ForeignAPIRepo.php
+++ b/includes/filerepo/ForeignAPIRepo.php
@@ -200,7 +200,7 @@ class ForeignAPIRepo extends FileRepo {
                        return $this->getThumbUrl( $name, $width, $height );
                }
 
-               $key = $this->getLocalCacheKey( 'ForeignAPIRepo', 'ThumbUrl', $name );
+               $key = $this->getLocalCacheKey( 'ForeignAPIRepo', 'ThumbUrl', $name, $width );
                $thumbUrl = $wgMemc->get($key);
                if ( $thumbUrl ) {
                        wfDebug("Got thumb from local cache. $thumbUrl \n");