Start and EndActionExecute added/fixed/documented

diff --git a/EVENTS.txt b/EVENTS.txt
index 088ce8c024b05079d724118f983220719277c56d..caa99f9638a82866d051e589f3255a03b83a10c7 100644 (file)
--- a/EVENTS.txt
+++ b/EVENTS.txt
@@ -2,6 +2,13 @@ InitializePlugin: a chance to initialize a plugin in a complete environment
 
 CleanupPlugin: a chance to cleanup a plugin at the end of a program
 
+StartActionExecute: Right before the "prepare" call of the current Action
+- $action:  the current Action object
+- &$args:   array of arguments, referenced so you can modify the array
+
+EndActionExecute:   Right after the "handle" call of the current Action
+- $action:      the current Action object
+
 StartPrimaryNav: Showing the primary nav menu
 - $action: the current action
 

diff --git a/lib/action.php b/lib/action.php
index 7c57cbc432a9e8ba7f911bca38652ddbf3b39e6b..3b3d21680e1975fadfc91221ba786656522be264 100644 (file)
--- a/lib/action.php
+++ b/lib/action.php
@@ -118,16 +118,18 @@ class Action extends HTMLOutputter // lawsuit
             common_config_set('db', 'database', $mirror);
         }
 
-        $status = $this->prepare($args);
-        if ($status) {
-            $this->handle($args);
-        } else {
-            common_debug('Prepare failed for Action.');
+        if (Event::handle('StartActionExecute', array($this, &$args))) {
+            $prepared = $this->prepare($args);
+            if ($prepared) {
+                $this->handle($args);
+            } else {
+                common_debug('Prepare failed for Action.');
+            }
         }
 
         $this->flush();
 
-        Event::handle('EndActionExecute', array($status, $this));
+        Event::handle('EndActionExecute', array($this));
     }
 
     /**

diff --git a/plugins/Cronish/CronishPlugin.php b/plugins/Cronish/CronishPlugin.php
index fb12000c4665e76514bdbed4eb75eff29f74e411..9ae62dc778c1c00a390487e03c883cd864d0aeed 100644 (file)
--- a/plugins/Cronish/CronishPlugin.php
+++ b/plugins/Cronish/CronishPlugin.php
@@ -37,7 +37,7 @@ class CronishPlugin extends Plugin {
      * When the page has finished rendering, let's do some cron jobs
      * if we have the time.
      */
-    public function onEndActionExecute($status, Action $action)
+    public function onEndActionExecute(Action $action)
     {
         $cron = new Cronish(); 
         $cron->callTimedEvents();

diff --git a/plugins/OpportunisticQM/OpportunisticQMPlugin.php b/plugins/OpportunisticQM/OpportunisticQMPlugin.php
index 89bd15a1a0210f087349322c0ae4e7685b021ed7..2e57cae9d1e22db42028e9488fa359a8ff0fa205 100644 (file)
--- a/plugins/OpportunisticQM/OpportunisticQMPlugin.php
+++ b/plugins/OpportunisticQM/OpportunisticQMPlugin.php
@@ -14,7 +14,7 @@ class OpportunisticQMPlugin extends Plugin {
      * When the page has finished rendering, let's do some cron jobs
      * if we have the time.
      */
-    public function onEndActionExecute($status, Action $action)
+    public function onEndActionExecute(Action $action)
     {
         if ($action instanceof RunqueueAction) {
             return true;