http://www.mediawiki.org/wiki/Special:Code/MediaWiki/98741

Revision: 98741
Author:   aaron
Date:     2011-10-03 06:54:39 +0000 (Mon, 03 Oct 2011)
Log Message:
-----------
Simplified cacheDirectory() in FileCacheBase and HTMLFileCache. It turns out 
$wgFileCacheDirectory is always set in setup.php already.

Modified Paths:
--------------
    trunk/phase3/includes/cache/FileCacheBase.php
    trunk/phase3/includes/cache/HTMLFileCache.php

Modified: trunk/phase3/includes/cache/FileCacheBase.php
===================================================================
--- trunk/phase3/includes/cache/FileCacheBase.php       2011-10-03 06:11:27 UTC 
(rev 98740)
+++ trunk/phase3/includes/cache/FileCacheBase.php       2011-10-03 06:54:39 UTC 
(rev 98741)
@@ -28,15 +28,8 @@
         * @return string
         */
        final protected function baseCacheDirectory() {
-               global $wgCacheDirectory, $wgFileCacheDirectory, 
$wgFileCacheDepth;
-               if ( $wgFileCacheDirectory ) {
-                       $dir = $wgFileCacheDirectory;
-               } elseif ( $wgCacheDirectory ) {
-                       $dir = $wgCacheDirectory;
-               } else {
-                       throw new MWException( 'Please set $wgCacheDirectory in 
LocalSettings.php if you wish to use the HTML file cache' );
-               }
-               return $dir;
+               global $wgFileCacheDirectory;
+               return $wgFileCacheDirectory;
        }
 
        /**

Modified: trunk/phase3/includes/cache/HTMLFileCache.php
===================================================================
--- trunk/phase3/includes/cache/HTMLFileCache.php       2011-10-03 06:11:27 UTC 
(rev 98740)
+++ trunk/phase3/includes/cache/HTMLFileCache.php       2011-10-03 06:54:39 UTC 
(rev 98741)
@@ -35,19 +35,10 @@
 
        /**
         * Get the base file cache directory
-        * Note: avoids baseCacheDirectory() for b/c to not skip existing cache
         * @return string
         */
        protected function cacheDirectory() {
-               global $wgCacheDirectory, $wgFileCacheDirectory, 
$wgFileCacheDepth;
-               if ( $wgFileCacheDirectory ) {
-                       $dir = $wgFileCacheDirectory;
-               } elseif ( $wgCacheDirectory ) {
-                       $dir = "$wgCacheDirectory/html";
-               } else {
-                       throw new MWException( 'Please set $wgCacheDirectory in 
LocalSettings.php if you wish to use the HTML file cache' );
-               }
-               return $dir;
+               return $this->baseCacheDirectory(); // no subdir for b/c with 
old cache files
        }
 
        /**


_______________________________________________
MediaWiki-CVS mailing list
MediaWiki-CVS@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-cvs

Reply via email to