From aa737ccaa790100adb266f9cde3a642b0361ff66 Mon Sep 17 00:00:00 2001 From: Aaron Schulz Date: Mon, 3 Sep 2007 04:19:53 +0000 Subject: [PATCH] *Don't call ArticleSaveComplete if the edit did not go through and no updates (even cache clearing) were done on the page. --- includes/Article.php | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/includes/Article.php b/includes/Article.php index 5c242cc11c..7ba55c548b 100644 --- a/includes/Article.php +++ b/includes/Article.php @@ -1473,8 +1473,10 @@ class Article { wfDoUpdates(); } - wfRunHooks( 'ArticleSaveComplete', array( &$this, &$wgUser, $text, $summary, - $flags & EDIT_MINOR, null, null, &$flags, $revision ) ); + if ( $good ) { + wfRunHooks( 'ArticleSaveComplete', array( &$this, &$wgUser, $text, $summary, + $flags & EDIT_MINOR, null, null, &$flags, $revision ) ); + } wfProfileOut( __METHOD__ ); return $good; -- 2.20.1