From bb3faf9d9b5039cfa30dd03250e0cd9be88dc008 Mon Sep 17 00:00:00 2001 From: Brion Vibber Date: Thu, 10 Jul 2008 17:02:39 +0000 Subject: [PATCH] Revert r37490, 37493 for now -- 'NotifyOnPageChangeComplete hook added' The hook seems a bit odd; it triggers on some of the composes, but not all. There's another loop right after this one. Also, there's no need to pass objects as references here. --- docs/hooks.txt | 4 ---- includes/UserMailer.php | 1 - 2 files changed, 5 deletions(-) diff --git a/docs/hooks.txt b/docs/hooks.txt index b7a3dd0dd0..bad937b2f4 100644 --- a/docs/hooks.txt +++ b/docs/hooks.txt @@ -869,10 +869,6 @@ $baseID: the revision ID this was based off, if any &$langCode: the language code to get the message for (string) - or - whether to use the content language (true) or site language (false) (bool) -'NotifyOnPageChangeComplete': called after user email notification. -$EmailNotification: EmailNotification object being used -$watchingUser: user (object) to whom email was sent - 'OpenSearchUrls': Called when constructing the OpenSearch description XML. Hooks can alter or append to the array of URLs for search & suggestion formats. &$urls: array of associative arrays with Url element attributes diff --git a/includes/UserMailer.php b/includes/UserMailer.php index 75a37ad18c..89753f2a54 100644 --- a/includes/UserMailer.php +++ b/includes/UserMailer.php @@ -383,7 +383,6 @@ class EmailNotification { $watchingUser->isEmailConfirmed() ) { $this->compose( $watchingUser ); - wfRunHooks('NotifyOnPageChangeComplete', array( &$this, &$watchingUser )); } } } -- 2.20.1