From d133a108c6e37a80e079988e86a621fb689d18ff Mon Sep 17 00:00:00 2001
From: Manuel Schneider <80686@users.mediawiki.org>
Date: Thu, 22 Mar 2007 08:19:47 +0000
Subject: [PATCH] fixed bug in call of hook ArticleViewHeader

---
 includes/Article.php | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/includes/Article.php b/includes/Article.php
index 3c8d10e094..3dd47a01b0 100644
--- a/includes/Article.php
+++ b/includes/Article.php
@@ -718,9 +718,9 @@ class Article {
 		}
 
 		$outputDone = false;
+		wfRunHooks( 'ArticleViewHeader', array( &$this ) );
 		if ( $pcache ) {
 			if ( $wgOut->tryParserCache( $this, $wgUser ) ) {
-				wfRunHooks( 'ArticleViewHeader', array( &$this ) );
 				$outputDone = true;
 			}
 		}
@@ -767,11 +767,6 @@ class Article {
 			}
 		}
 		if( !$outputDone ) {
-			/**
-			 * @fixme: this hook doesn't work most of the time, as it doesn't
-			 * trigger when the parser cache is used.
-			 */
-			wfRunHooks( 'ArticleViewHeader', array( &$this ) ) ;
 			$wgOut->setRevisionId( $this->getRevIdFetched() );
 			# wrap user css and user js in pre and don't parse
 			# XXX: use $this->mTitle->usCssJsSubpage() when php is fixed/ a workaround is found
-- 
2.20.1