Changeset 27238 for trunk

Timestamp:

Feb 6, 2014, 2:02:22 PM (11 years ago)

Author:

mistic100

Message:

feature 3041: Give include path to add_event_handler
$accepted_args is replaced by $include_path (retrocompatible)

File:

• trunk/include/functions_plugins.inc.php (modified) (8 diffs)

trunk/include/functions_plugins.inc.php

@@ -173 +173 @@
  * @param Callable $func the callback function
  * @param int $priority greater priority will be executed at last
+ * @param string $include_path file to include before executing the callback
+ * @return bool false is handler already exists
  */
 function add_event_handler($event, $func,
-    $priority=EVENT_HANDLER_PRIORITY_NEUTRAL, $accepted_args=1)
+    $priority=EVENT_HANDLER_PRIORITY_NEUTRAL, $include_path=null)
 {
   global $pwg_event_handlers;
 
-  if ( isset($pwg_event_handlers[$event][$priority]) )
-  {
-    foreach($pwg_event_handlers[$event][$priority] as $handler)
-    {
-      if ( $handler['function'] == $func )
+  if (isset($pwg_event_handlers[$event][$priority]))
+  {
+    foreach ($pwg_event_handlers[$event][$priority] as $handler)
+    {
+      if ($handler['function'] == $func)
       {
         return false;
@@ -190 +192 @@
   }
 
-  $pwg_event_handlers[$event][$priority][] =
-    array(
-      'function'=>$func,
-      'accepted_args'=>$accepted_args);
-  ksort( $pwg_event_handlers[$event] );
+  $pwg_event_handlers[$event][$priority][] = array(
+    'function' => $func,
+    'include_path' => is_string($include_path) ? $include_path : null,
+    );
+
+  ksort($pwg_event_handlers[$event]);
   return true;
 }
@@ -211 +214 @@
   global $pwg_event_handlers;
 
-  if (!isset( $pwg_event_handlers[$event][$priority] ) )
+  if (!isset($pwg_event_handlers[$event][$priority]))
   {
     return false;
@@ -223 +226 @@
         array_values($pwg_event_handlers[$event][$priority]);
 
-      if ( empty($pwg_event_handlers[$event][$priority]) )
-      {
-        unset( $pwg_event_handlers[$event][$priority] );
-        if (empty( $pwg_event_handlers[$event] ) )
+      if (empty($pwg_event_handlers[$event][$priority]))
+      {
+        unset($pwg_event_handlers[$event][$priority]);
+        if (empty($pwg_event_handlers[$event]))
         {
-          unset( $pwg_event_handlers[$event] );
+          unset($pwg_event_handlers[$event]);
         }
       }
@@ -265 +268 @@
   global $pwg_event_handlers;
 
-  if ( isset($pwg_event_handlers['trigger']) )
-  {// just for debugging
+  if (isset($pwg_event_handlers['trigger']))
+  {// debugging
     trigger_action('trigger',
-        array('type'=>'event', 'event'=>$event, 'data'=>$data) );
-  }
-
-  if ( !isset($pwg_event_handlers[$event]) )
+      array('type'=>'event', 'event'=>$event, 'data'=>$data)
+      );
+  }
+
+  if (!isset($pwg_event_handlers[$event]))
   {
     return $data;
@@ -279 +283 @@
   foreach ($pwg_event_handlers[$event] as $priority => $handlers)
   {
-    foreach($handlers as $handler)
-    {
-      $function_name = $handler['function'];
-      $accepted_args = $handler['accepted_args'];
+    foreach ($handlers as $handler)
+    {
       $args[1] = $data;
-      $data = call_user_func_array($function_name, array_slice($args,1,$accepted_args) );
-    }
-  }
-  trigger_action('trigger',
-       array('type'=>'post_event', 'event'=>$event, 'data'=>$data) );
+
+      if (!empty($handler['include_path']))
+      {
+        include_once($handler['include_path']);
+      }
+
+      $data = call_user_func_array($handler['function'], array_slice($args, 1));
+    }
+  }
+
+  if (isset($pwg_event_handlers['trigger']))
+  {// debugging
+    trigger_action('trigger',
+      array('type'=>'post_event', 'event'=>$event, 'data'=>$data)
+      );
+  }
+
   return $data;
 }
@@ -315 +329 @@
 {
   global $pwg_event_handlers;
-  if ( isset($pwg_event_handlers['trigger']) and $event!='trigger' )
-  {// special case for debugging - avoid recursive calls
+
+  if (isset($pwg_event_handlers['trigger']) and $event!='trigger')
+  {// debugging - avoid recursive calls
     trigger_action('trigger',
-        array('type'=>'action', 'event'=>$event, 'data'=>null) );
-  }
-
-  if ( !isset($pwg_event_handlers[$event]) )
+      array('type'=>'action', 'event'=>$event, 'data'=>null)
+      );
+  }
+
+  if (!isset($pwg_event_handlers[$event]))
   {
     return;
@@ -329 +345 @@
   foreach ($pwg_event_handlers[$event] as $priority => $handlers)
   {
-    foreach($handlers as $handler)
-    {
-      $function_name = $handler['function'];
-      $accepted_args = $handler['accepted_args'];
-
-      call_user_func_array($function_name, array_slice($args,1,$accepted_args) );
+    foreach ($handlers as $handler)
+    {
+      if (!empty($handler['include_path']))
+      {
+        include_once($handler['include_path']);
+      }
+
+      call_user_func_array($handler['function'], array_slice($args,1));
     }
   }