From 0356a3893473463868a85efd52b47f4539a53e55 Mon Sep 17 00:00:00 2001
From: Brion Vibber <brion@users.mediawiki.org>
Date: Tue, 23 Dec 2008 19:31:08 +0000
Subject: [PATCH] Revert r44798 "short-circuit update call"
 Article::viewUpdates() may do things other than update counters (for
 instance, it updates the user-talk and watchlist for last-viewed status, and
 might do other things in the future). Let it centralize its own checks for
 what things are enabled. Since you won't necessarily know them all over in
 Wiki.png, they may get out of sync and lead to bugs if you try to replicate
 the check.

---
 includes/Wiki.php | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/includes/Wiki.php b/includes/Wiki.php
index 4dcad9a628..6e96fba48b 100644
--- a/includes/Wiki.php
+++ b/includes/Wiki.php
@@ -222,18 +222,16 @@ class MediaWiki {
 			if( $title->getNamespace() != NS_MEDIAWIKI && HTMLFileCache::useFileCache() ) {
 				$cache = new HTMLFileCache( $title );
 				if( $cache->isFileCacheGood( /* Assume up to date */ ) ) {
-					global $wgOut, $wgDisableCounters;
+					global $wgOut;
 					/* Check incoming headers to see if client has this cached */
 					if( !$wgOut->checkLastModified( $cache->fileCacheTime() ) ) {
 						wfDebug( "MediaWiki::initializeSpecialCases(): about to load file cache\n" );
 						$cache->loadFromFileCache();
 						# Tell $wgOut that output is taken care of
 						$wgOut->disable();
-						if( !$wgDisableCounters ) {
-							# Do any stats increment/watchlist stuff
-							$article = self::articleFromTitle( $title );
-							$article->viewUpdates();
-						}
+						# Do any stats increment/watchlist stuff
+						$article = self::articleFromTitle( $title );
+						$article->viewUpdates();
 					}
 					wfProfileOut( __METHOD__ );
 					$this->restInPeace();
-- 
2.20.1