Core notify functions moved inside notify class.

e107_handlers/notify_class.php

@@ -19,12 +19,19 @@ if (!defined('e107_INIT')) { exit; }
 
 class notify
 {
-	var $notify_prefs;
+	public $notify_prefs;
 
-	function notify()
+	function __construct()
 	{
-	//	global $e_event;
+		include_lan(e_LANGUAGEDIR.e_LANGUAGE.'/lan_notify.php');
+	}
 
+
+	/**
+	 * Register core and plugin notification events.
+	 */
+	public function registerEvents()
+	{
 		$e_event = e107::getEvent();
 
 		$this->notify_prefs = e107::getConfig('notify')->getPref();
@@ -54,9 +61,9 @@ class notify
 					}
 					else // core
 					{
-						if(function_exists('notify_'.$id)) // as found below. 
+						if(method_exists($this, 'notify_'.$id)) // as found below.
 						{
-							$e_event->register($id, 'notify_'.$id);	
+							$e_event->register($id, array('notify', 'notify_'.$id));
 						}
 						else
 						{
@@ -69,13 +76,10 @@ class notify
 			}
 		}
 
-		include_lan(e_LANGUAGEDIR.e_LANGUAGE.'/lan_notify.php');
 	//	e107::getEvent()->debug();
 	}
 
 
-
-
 	/**
 	 * Generic Notification method when none defined. 
 	 */
@@ -95,7 +99,7 @@ class notify
 	 * @param string $id - identifies event actions
 	 * @param string $subject - subject for email
 	 * @param string $message - email message body
-	 * @return none
+	 * @return void
 	 *
 	 *	@todo handle 'everyone except' clauses (email address filter done)
 	 *	@todo set up pref to not notify originator of event which caused notify (see $blockOriginator)
@@ -195,19 +199,18 @@ class notify
 			$e107->admin_log->e_log_event(10,-1,'NOTIFY',$subject,$message,FALSE,LOG_TO_ROLLING);
 		}
 	}
-}
 
 
 
 
-//DEPRECATED, BC, call the method only when needed, $e107->notify caught by __get()
+
-global $nt;
+	// ---------------------------------------
-$nt = e107::getNotify(); //TODO - find & replace $nt, $e107->notify
 
 
-function notify_usersup($data)
+
-{
+	function notify_usersup($data)
-	global $nt;
+	{
+		$message = "";
 		foreach ($data as $key => $value)
 		{
 			if($key != "password1" && $key != "password2" && $key != "email_confirm" && $key != "register")
@@ -225,91 +228,100 @@ function notify_usersup($data)
 				}
 			}
 		}
-	$nt->send('usersup', NT_LAN_US_1, $message);
-}
 
-function notify_userveri($data)
+		$this->send('usersup', NT_LAN_US_1, $message);
-{
+	}
-	global $nt, $e107;
+
+	/**
+	 * @param $data
+	 */
+	function notify_userveri($data)
+	{
 		$msgtext = NT_LAN_UV_2.$data['user_id']."\n";
 		$msgtext .= NT_LAN_UV_3.$data['user_loginname']."\n";
 		$msgtext .= NT_LAN_UV_4.e107::getIPHandler()->getIP(FALSE);
-	$nt->send('userveri', NT_LAN_UV_1, $msgtext);
+
-}
+		$this->send('userveri', NT_LAN_UV_1, $msgtext);
+	}
 
 
-function notify_login($data)
+	function notify_login($data)
-{
+	{
-	global $nt;
+		$message = "";
-	foreach ($data as $key => $value) {
+		foreach ($data as $key => $value)
+		{
 			$message .= $key.': '.$value.'<br />';
 		}
-	$nt->send('login', NT_LAN_LI_1, $message);
-}
 
-function notify_logout()
+		$this->send('login', NT_LAN_LI_1, $message);
-{
+	}
-	global $nt;
-	$nt->send('logout', NT_LAN_LO_1, USERID.'. '.USERNAME.' '.NT_LAN_LO_2);
-}
 
-function notify_flood($data)
+	function notify_logout()
-{
+	{
-	global $nt;
+		$this->send('logout', NT_LAN_LO_1, USERID.'. '.USERNAME.' '.NT_LAN_LO_2);
-	$nt->send('flood', NT_LAN_FL_1, NT_LAN_FL_2.': '.e107::getIPHandler()->ipDecode($data));
+	}
-}
 
-function notify_subnews($data)
+	function notify_flood($data)
-{
+	{
-	global $nt,$tp;
+		$this->send('flood', NT_LAN_FL_1, NT_LAN_FL_2.': '.e107::getIPHandler()->ipDecode($data));
-	foreach ($data as $key => $value) {
+	}
+
+	function notify_subnews($data)
+	{
+		$message = "";
+		foreach ($data as $key => $value)
+		{
 			$message .= $key.': '.$value.'<br />';
 		}
-	$nt->send('subnews', NT_LAN_SN_1, $message);
-}
 
-function notify_newspost($data)
+		$this->send('subnews', NT_LAN_SN_1, $message);
-{
+	}
+
+	function notify_newspost($data)
+	{
 		$message = '<b>'.$data['news_title'].'</b>';
 		if (vartrue($data['news_summary'])) $message .= '<br /><br />'.$data['news_summary'];
 		if (vartrue($data['news_body'])) $message .= '<br /><br />'.$data['news_body'];
 		if (vartrue($data['news_extended'])) $message.= '<br /><br />'.$data['news_extended'];
-	e107::getNotify()->send('newspost', $data['news_title'], e107::getParser()->text_truncate(e107::getParser()->toDB($message), 400, '...'));
+		$this->send('newspost', $data['news_title'], e107::getParser()->text_truncate(e107::getParser()->toDB($message), 400, '...'));
-}
+	}
 
-function notify_newsupd($data)
+	function notify_newsupd($data)
-{
+	{
 		$message = '<b>'.$data['news_title'].'</b>';
 		if (vartrue($data['news_summary'])) $message .= '<br /><br />'.$data['news_summary'];
 		if (vartrue($data['news_body'])) $message .= '<br /><br />'.$data['news_body'];
 		if (vartrue($data['news_extended'])) $message.= '<br /><br />'.$data['news_extended'];
-	e107::getNotify()->send('newsupd', NT_LAN_NU_1.': '.$data['news_title'], e107::getParser()->text_truncate(e107::getParser()->toDB($message), 400, '...'));
+		$this->send('newsupd', NT_LAN_NU_1.': '.$data['news_title'], e107::getParser()->text_truncate(e107::getParser()->toDB($message), 400, '...'));
-}
+	}
 
-function notify_newsdel($data)
+	function notify_newsdel($data)
-{
+	{
-	global $nt;
+		$this->send('newsdel', NT_LAN_ND_1, NT_LAN_ND_2.': '.$data);
-	$nt->send('newsdel', NT_LAN_ND_1, NT_LAN_ND_2.': '.$data);
+	}
-}
 
 
-function notify_maildone($data)
+	function notify_maildone($data)
-{
+	{
 		$message = '<b>'.$data['mail_subject'].'</b><br /><br />'.$data['mail_body'];
-	e107::getNotify()->send('maildone', NT_LAN_ML_1.': '.$data['mail_subject'], $message);
+		$this->send('maildone', NT_LAN_ML_1.': '.$data['mail_subject'], $message);
-}
+	}
 
 
-function notify_fileupload($data)
+	function notify_fileupload($data)
-{
+	{
-	global $nt;
 		$message = '<b>'.$data['upload_name'].'</b><br /><br />'.$data['upload_description'].'<br /><br />'.$data['upload_size'].'<br /><br />'.$data['upload_user'];
-	$nt->send('fileupload', $data['upload_name'], $message);
+		$this->send('fileupload', $data['upload_name'], $message);
+	}
+
+
+
+
 }
 
-
+	/*
-if (isset($nt->notify_prefs['plugins']) && e_PAGE != 'notify.php')
+	if (isset($nt->notify_prefs['plugins']) && e_PAGE != 'notify.php')
-{
+	{
 	foreach ($nt->notify_prefs['plugins'] as $plugin_id => $plugin_settings)
 	{
 	if(is_readable(e_PLUGIN.$plugin_id.'/e_notify.php'))
@@ -317,6 +329,7 @@ if (isset($nt->notify_prefs['plugins']) && e_PAGE != 'notify.php')
 	require_once(e_PLUGIN.$plugin_id.'/e_notify.php');
 	}
 		}
-}
+		}
+
+	*/
 
-?>