From ffdebe9c5b61573ef2350372e4250a1b7910fe43 Mon Sep 17 00:00:00 2001
From: Matthias Schmidt <gravatronics@live.com>
Date: Fri, 22 May 2015 08:09:33 +0200
Subject: [PATCH] Add permissions and options support for event listeners

---
 CHANGELOG.md                                  |  3 +-
 .../event/listener/EventListener.class.php    |  5 ++
 .../lib/system/event/EventHandler.class.php   | 78 ++++++++++---------
 ...istenerPackageInstallationPlugin.class.php |  4 +-
 wcfsetup/setup/db/install.sql                 |  3 +
 5 files changed, 54 insertions(+), 39 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 3a7bfd2442..f0224e10ac 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -4,6 +4,7 @@
 
 ### 2.2.0 Alpha 1 (XXXX-YY-ZZ)
 
-* Add `wcf\data\TDatabaseObjectOptions` and `wcf\data\TDatabaseObjectPermissions` for database object-bound options and permissions validation.
+* `permissions` and `options` support for event listeners.
+* `wcf\data\TDatabaseObjectOptions` and `wcf\data\TDatabaseObjectPermissions` for database object-bound options and permissions validation.
 * `wcf\system\cache\builder\EventListenerCacheBuilder` returns `wcf\data\event\listener\EventListener` objects instead of data arrays.
 
diff --git a/wcfsetup/install/files/lib/data/event/listener/EventListener.class.php b/wcfsetup/install/files/lib/data/event/listener/EventListener.class.php
index f12c8e9f79..bb64640b4d 100644
--- a/wcfsetup/install/files/lib/data/event/listener/EventListener.class.php
+++ b/wcfsetup/install/files/lib/data/event/listener/EventListener.class.php
@@ -1,6 +1,8 @@
 <?php
 namespace wcf\data\event\listener;
 use wcf\data\DatabaseObject;
+use wcf\data\TDatabaseObjectOptions;
+use wcf\data\TDatabaseObjectPermissions;
 
 /**
  * Represents an event listener.
@@ -13,6 +15,9 @@ use wcf\data\DatabaseObject;
  * @category	Community Framework
  */
 class EventListener extends DatabaseObject {
+	use TDatabaseObjectOptions;
+	use TDatabaseObjectPermissions;
+	
 	/**
 	 * @see	\wcf\data\DatabaseObject::$databaseTableName
 	 */
diff --git a/wcfsetup/install/files/lib/system/event/EventHandler.class.php b/wcfsetup/install/files/lib/system/event/EventHandler.class.php
index 03e6be53d8..c66fcbd53f 100644
--- a/wcfsetup/install/files/lib/system/event/EventHandler.class.php
+++ b/wcfsetup/install/files/lib/system/event/EventHandler.class.php
@@ -98,30 +98,32 @@ class EventHandler extends SingletonFactory {
 					$actions = $this->inheritedActions[$member];
 					if (isset($actions[$eventName]) && !empty($actions[$eventName])) {
 						foreach ($actions[$eventName] as $eventListener) {
-							if (isset($this->inheritedActionsObjects[$name][$eventListener->listenerClassName])) continue;
-							
-							// get class object
-							if (isset($this->listenerObjects[$eventListener->listenerClassName])) {
-								$object = $this->listenerObjects[$eventListener->listenerClassName];
-							}
-							else {
-								$object = null;
-								// instance action object
-								if (!class_exists($eventListener->listenerClassName)) {
-									throw new SystemException("Unable to find class '".$eventListener->listenerClassName."'");
+							if ($eventListener->validateOptions() && $eventListener->validatePermissions()) {
+								if (isset($this->inheritedActionsObjects[$name][$eventListener->listenerClassName])) continue;
+								
+								// get class object
+								if (isset($this->listenerObjects[$eventListener->listenerClassName])) {
+									$object = $this->listenerObjects[$eventListener->listenerClassName];
 								}
-								if (!ClassUtil::isInstanceOf($eventListener->listenerClassName, 'wcf\system\event\listener\IParameterizedEventListener')) {
-									// legacy event listeners
-									if (!ClassUtil::isInstanceOf($eventListener->listenerClassName, 'wcf\system\event\IEventListener')) {
-										throw new SystemException("'".$eventListener->listenerClassName."' does not implement 'wcf\system\event\listener\IParameterizedEventListener'");
+								else {
+									$object = null;
+									// instance action object
+									if (!class_exists($eventListener->listenerClassName)) {
+										throw new SystemException("Unable to find class '".$eventListener->listenerClassName."'");
+									}
+									if (!ClassUtil::isInstanceOf($eventListener->listenerClassName, 'wcf\system\event\listener\IParameterizedEventListener')) {
+										// legacy event listeners
+										if (!ClassUtil::isInstanceOf($eventListener->listenerClassName, 'wcf\system\event\IEventListener')) {
+											throw new SystemException("'".$eventListener->listenerClassName."' does not implement 'wcf\system\event\listener\IParameterizedEventListener'");
+										}
 									}
+									
+									$object = new $eventListener->listenerClassName;
+									$this->listenerObjects[$eventListener->listenerClassName] = $object;
 								}
 								
-								$object = new $eventListener->listenerClassName;
-								$this->listenerObjects[$eventListener->listenerClassName] = $object;
+								if ($object !== null) $this->inheritedActionsObjects[$name][$eventListener->listenerClassName] = $object;
 							}
-							
-							if ($object !== null) $this->inheritedActionsObjects[$name][$eventListener->listenerClassName] = $object;
 						}
 					}
 				}
@@ -182,29 +184,31 @@ class EventHandler extends SingletonFactory {
 			
 			$this->actionsObjects[$name] = array();
 			foreach ($this->actions[$name] as $eventListener) {
-				if (isset($this->actionsObjects[$name][$eventListener->listenerClassName])) continue;
-				
-				// get class object
-				if (isset($this->listenerObjects[$eventListener->listenerClassName])) {
-					$object = $this->listenerObjects[$eventListener->listenerClassName];
-				}
-				else {
-					// instance action object
-					if (!class_exists($eventListener->listenerClassName)) {
-						throw new SystemException("Unable to find class '".$eventListener->listenerClassName."'");
+				if ($eventListener->validateOptions() && $eventListener->validatePermissions()) {
+					if (isset($this->actionsObjects[$name][$eventListener->listenerClassName])) continue;
+					
+					// get class object
+					if (isset($this->listenerObjects[$eventListener->listenerClassName])) {
+						$object = $this->listenerObjects[$eventListener->listenerClassName];
 					}
-					if (!ClassUtil::isInstanceOf($eventListener->listenerClassName, 'wcf\system\event\listener\IParameterizedEventListener')) {
-						// legacy event listeners
-						if (!ClassUtil::isInstanceOf($eventListener->listenerClassName, 'wcf\system\event\IEventListener')) {
-							throw new SystemException("'".$eventListener->listenerClassName."' does not implement 'wcf\system\event\listener\IParameterizedEventListener'");
+					else {
+						// instance action object
+						if (!class_exists($eventListener->listenerClassName)) {
+							throw new SystemException("Unable to find class '".$eventListener->listenerClassName."'");
+						}
+						if (!ClassUtil::isInstanceOf($eventListener->listenerClassName, 'wcf\system\event\listener\IParameterizedEventListener')) {
+							// legacy event listeners
+							if (!ClassUtil::isInstanceOf($eventListener->listenerClassName, 'wcf\system\event\IEventListener')) {
+								throw new SystemException("'".$eventListener->listenerClassName."' does not implement 'wcf\system\event\listener\IParameterizedEventListener'");
+							}
 						}
+						
+						$object = new $eventListener->listenerClassName;
+						$this->listenerObjects[$eventListener->listenerClassName] = $object;
 					}
 					
-					$object = new $eventListener->listenerClassName;
-					$this->listenerObjects[$eventListener->listenerClassName] = $object;
+					$this->actionsObjects[$name][$eventListener->listenerClassName] = $object;
 				}
-				
-				$this->actionsObjects[$name][$eventListener->listenerClassName] = $object;
 			}
 		}
 		
diff --git a/wcfsetup/install/files/lib/system/package/plugin/EventListenerPackageInstallationPlugin.class.php b/wcfsetup/install/files/lib/system/package/plugin/EventListenerPackageInstallationPlugin.class.php
index 226bde24ff..e2fb08e860 100644
--- a/wcfsetup/install/files/lib/system/package/plugin/EventListenerPackageInstallationPlugin.class.php
+++ b/wcfsetup/install/files/lib/system/package/plugin/EventListenerPackageInstallationPlugin.class.php
@@ -62,7 +62,9 @@ class EventListenerPackageInstallationPlugin extends AbstractXMLPackageInstallat
 			'eventName' => $data['elements']['eventname'],
 			'inherit' => (isset($data['elements']['inherit'])) ? intval($data['elements']['inherit']) : 0,
 			'listenerClassName' => $data['elements']['listenerclassname'],
-			'niceValue' => $nice
+			'niceValue' => $nice,
+			'options' => (isset($data['elements']['options']) ? $data['elements']['options'] : ''),
+			'permissions' => (isset($data['elements']['permissions']) ? $data['elements']['permissions'] : '')
 		);
 	}
 	
diff --git a/wcfsetup/setup/db/install.sql b/wcfsetup/setup/db/install.sql
index 848cdd0556..25464ad078 100644
--- a/wcfsetup/setup/db/install.sql
+++ b/wcfsetup/setup/db/install.sql
@@ -377,6 +377,9 @@ CREATE TABLE wcf1_event_listener (
 	listenerClassName VARCHAR(200) NOT NULL DEFAULT '',
 	inherit TINYINT(1) NOT NULL DEFAULT 0,
 	niceValue TINYINT(3) NOT NULL DEFAULT 0,
+	permissions TEXT,
+	options TEXT,
+	
 	UNIQUE KEY packageID (packageID, environment, eventClassName, eventName, listenerClassName)
 );
 
-- 
2.20.1