It should only be saved in the ParserCache if it IS cacheable.

Confusion in r67185, when translating the original condition (!= -1)
into the new isCacheable().

diff --git a/includes/OutputPage.php b/includes/OutputPage.php
index f9825ff..9f07201 100644 (file)
--- a/includes/OutputPage.php
+++ b/includes/OutputPage.php
@@ -1061,7 +1061,7 @@ class OutputPage {
                        $popts, true, true, $this->mRevisionId
                );
                $popts->setTidy( false );
-               if ( $cache && $article && !$parserOutput->isCacheable() ) {
+               if ( $cache && $article && $parserOutput->isCacheable() ) {
                        $parserCache = ParserCache::singleton();
                        $parserCache->save( $parserOutput, $article, $popts );
                }