Don't manually quote stuff
[lhc/web/wiklou.git] / includes / parser / ParserCache.php
index a4b79b2..6a4ef0c 100644 (file)
@@ -2,7 +2,23 @@
 /**
  * Cache for outputs of the PHP parser
  *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License along
+ * with this program; if not, write to the Free Software Foundation, Inc.,
+ * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
+ * http://www.gnu.org/copyleft/gpl.html
+ *
  * @file
+ * @ingroup Cache Parser
  */
 
 /**
  */
 class ParserCache {
        private $mMemc;
+       const try116cache = false; /* Only useful $wgParserCacheExpireTime after updating to 1.17 */
 
        /**
         * Get an instance of this object
+        *
+        * @return ParserCache
         */
        public static function singleton() {
                static $instance;
@@ -30,13 +49,18 @@ class ParserCache {
         *
         * @param $memCached Object
         */
-       function __construct( $memCached ) {
+       protected function __construct( $memCached ) {
                if ( !$memCached ) {
-                       throw MWException( "Tried to create a ParserCache with an invalid memcached" );
+                       throw new MWException( "Tried to create a ParserCache with an invalid memcached" );
                }
                $this->mMemc = $memCached;
        }
 
+       /**
+        * @param $article Article
+        * @param $hash string
+        * @return mixed|string
+        */
        protected function getParserOutputKey( $article, $hash ) {
                global $wgRequest;
 
@@ -48,6 +72,10 @@ class ParserCache {
                return $key;
        }
 
+       /**
+        * @param $article Article
+        * @return mixed|string
+        */
        protected function getOptionsKey( $article ) {
                $pageid = $article->getID();
                return wfMemcKey( 'pcache', 'idoptions', "{$pageid}" );
@@ -62,15 +90,22 @@ class ParserCache {
         * $article. For example give a Chinese interface to a user with
         * English preferences. That's why we take into account *all* user
         * options. (r70809 CR)
+        *
+        * @param $article Article
+        * @param $popts ParserOptions
+        * @return string
         */
        function getETag( $article, $popts ) {
                return 'W/"' . $this->getParserOutputKey( $article,
-                       $popts->optionsHash( ParserOptions::legacyOptions() ) ) .
-                               "--" . $article->mTouched . '"';
+                       $popts->optionsHash( ParserOptions::legacyOptions(), $article->getTitle() ) ) .
+                               "--" . $article->getTouched() . '"';
        }
 
        /**
         * Retrieve the ParserOutput from ParserCache, even if it's outdated.
+        * @param $article Article
+        * @param $popts ParserOptions
+        * @return ParserOutput|bool False on failure
         */
        public function getDirty( $article, $popts ) {
                $value = $this->get( $article, $popts, true );
@@ -82,7 +117,12 @@ class ParserCache {
         * It would be preferable to have this code in get()
         * instead of having Article looking in our internals.
         *
-        * Precondition: $article->checkTouched() has been called.
+        * @todo Document parameter $useOutdated
+        *
+        * @param $article     Article
+        * @param $popts       ParserOptions
+        * @param $useOutdated Boolean (default true)
+        * @return bool|mixed|string
         */
        public function getKey( $article, $popts, $useOutdated = true ) {
                global $wgCacheEpoch;
@@ -95,27 +135,34 @@ class ParserCache {
                // Determine the options which affect this article
                $optionsKey = $this->mMemc->get( $this->getOptionsKey( $article ) );
                if ( $optionsKey != false ) {
-                       if ( !$useOutdated && $optionsKey->expired( $article->mTouched ) ) {
+                       if ( !$useOutdated && $optionsKey->expired( $article->getTouched() ) ) {
                                wfIncrStats( "pcache_miss_expired" );
                                $cacheTime = $optionsKey->getCacheTime();
-                               wfDebug( "Parser options key expired, touched {$article->mTouched}, epoch $wgCacheEpoch, cached $cacheTime\n" );
+                               wfDebug( "Parser options key expired, touched " . $article->getTouched() . ", epoch $wgCacheEpoch, cached $cacheTime\n" );
                                return false;
                        }
 
                        $usedOptions = $optionsKey->mUsedOptions;
                        wfDebug( "Parser cache options found.\n" );
                } else {
-                       # TODO: Fail here $wgParserCacheExpireTime after deployment unless $useOutdated
-
+                       if ( !$useOutdated && !self::try116cache ) {
+                               return false;
+                       }
                        $usedOptions = ParserOptions::legacyOptions();
                }
 
-               return $this->getParserOutputKey( $article, $popts->optionsHash( $usedOptions ) );
+               return $this->getParserOutputKey( $article, $popts->optionsHash( $usedOptions, $article->getTitle() ) );
        }
 
        /**
         * Retrieve the ParserOutput from ParserCache.
         * false if not found or outdated.
+        *
+        * @param $article     Article
+        * @param $popts       ParserOptions
+        * @param $useOutdated Boolean (default false)
+        *
+        * @return ParserOutput|bool False on failure
         */
        public function get( $article, $popts, $useOutdated = false ) {
                global $wgCacheEpoch;
@@ -128,24 +175,35 @@ class ParserCache {
                        return false;
                }
 
-               // Having called checkTouched() ensures this will be loaded
-               $touched = $article->mTouched;
+               $touched = $article->getTouched();
 
                $parserOutputKey = $this->getKey( $article, $popts, $useOutdated );
                if ( $parserOutputKey === false ) {
+                       wfIncrStats( 'pcache_miss_absent' );
                        wfProfileOut( __METHOD__ );
                        return false;
                }
 
                $value = $this->mMemc->get( $parserOutputKey );
+               if ( self::try116cache && !$value && strpos( $value, '*' ) !== -1 ) {
+                       wfDebug( "New format parser cache miss.\n" );
+                       $parserOutputKey = $this->getParserOutputKey( $article,
+                               $popts->optionsHash( ParserOptions::legacyOptions(), $article->getTitle() ) );
+                       $value = $this->mMemc->get( $parserOutputKey );
+               }
                if ( !$value ) {
-                       wfDebug( "Parser cache miss.\n" );
+                       wfDebug( "ParserOutput cache miss.\n" );
                        wfIncrStats( "pcache_miss_absent" );
                        wfProfileOut( __METHOD__ );
                        return false;
                }
 
-               wfDebug( "Found.\n" );
+               wfDebug( "ParserOutput cache found.\n" );
+
+               // The edit section preference may not be the appropiate one in 
+               // the ParserOutput, as we are not storing it in the parsercache 
+               // key. Force it here. See bug 31445.
+               $value->setEditSectionTokens( $popts->getEditSection() );
 
                if ( !$useOutdated && $value->expired( $touched ) ) {
                        wfIncrStats( "pcache_miss_expired" );
@@ -153,9 +211,6 @@ class ParserCache {
                        wfDebug( "ParserOutput key expired, touched $touched, epoch $wgCacheEpoch, cached $cacheTime\n" );
                        $value = false;
                } else {
-                       if ( isset( $value->mTimestamp ) ) {
-                               $article->mTimestamp = $value->mTimestamp;
-                       }
                        wfIncrStats( "pcache_hit" );
                }
 
@@ -163,7 +218,11 @@ class ParserCache {
                return $value;
        }
 
-
+       /**
+        * @param $parserOutput ParserOutput
+        * @param $article Article
+        * @param $popts ParserOptions
+        */
        public function save( $parserOutput, $article, $popts ) {
                $expire = $parserOutput->getCacheExpiry();
 
@@ -171,7 +230,7 @@ class ParserCache {
                        $now = wfTimestampNow();
 
                        $optionsKey = new CacheTime;
-                       $optionsKey->mUsedOptions = $popts->usedOptions();
+                       $optionsKey->mUsedOptions = $parserOutput->getUsedOptions();
                        $optionsKey->updateCacheExpiry( $expire );
 
                        $optionsKey->setCacheTime( $now );
@@ -179,10 +238,11 @@ class ParserCache {
 
                        $optionsKey->setContainsOldMagic( $parserOutput->containsOldMagic() );
 
-                       $parserOutputKey = $this->getParserOutputKey( $article, $popts->optionsHash( $optionsKey->mUsedOptions ) );
+                       $parserOutputKey = $this->getParserOutputKey( $article,
+                               $popts->optionsHash( $optionsKey->mUsedOptions, $article->getTitle() ) );
 
                        // Save the timestamp so that we don't have to load the revision row on view
-                       $parserOutput->mTimestamp = $article->getTimestamp();
+                       $parserOutput->setTimestamp( $article->getTimestamp() );
 
                        $parserOutput->mText .= "\n<!-- Saved in parser cache with key $parserOutputKey and timestamp $now -->\n";
                        wfDebug( "Saved in parser cache with key $parserOutputKey and timestamp $now\n" );