From 42f68e4b1ae14d150aa404d6af84b89702548373 Mon Sep 17 00:00:00 2001 From: Happy-melon Date: Tue, 25 May 2010 17:04:57 +0000 Subject: [PATCH] Follow-up r66878 per CR. --- includes/Article.php | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/includes/Article.php b/includes/Article.php index 629574488d..c1fc9574e7 100644 --- a/includes/Article.php +++ b/includes/Article.php @@ -2056,9 +2056,6 @@ class Article { Article::onArticleEdit( $this->mTitle ); # Update links tables, site stats, etc. $this->editUpdates( $text, $summary, $isminor, $now, $revisionId, $changed ); - - $extraQuery = ''; # Give extensions a chance to modify URL query on update - wfRunHooks( 'ArticleUpdateBeforeRedirect', array( $this, &$sectionanchor, &$extraQuery ) ); } else { # Create new article $status->value['new'] = true; @@ -2138,20 +2135,23 @@ class Article { &$redirect) ); # Watch or unwatch the page - if ( $watchthis == true) { + if ( $watchthis === true ) { if ( !$this->mTitle->userIsWatching() ) { $dbw->begin(); $this->doWatch(); $dbw->commit(); } - } elseif ($watchthis == false) { + } elseif ( $watchthis === false ) { if ( $this->mTitle->userIsWatching() ) { $dbw->begin(); $this->doUnwatch(); $dbw->commit(); } } - + + # Give extensions a chance to modify URL query on update + $extraQuery = ''; + wfRunHooks( 'ArticleUpdateBeforeRedirect', array( $this, &$sectionanchor, &$extraQuery ) ); if ( $redirect ) { if ( $sectionanchor || $extraQuery ) { $this->doRedirect( $this->isRedirect( $text ), $sectionanchor, $extraQuery ); -- 2.20.1