'public static final' is the right thing, some variables renamed to make clear what...

[hub.git] / application / hub / main / visitor / tasks / class_ActiveTaskVisitor.php

diff --git a/application/hub/main/visitor/tasks/class_ActiveTaskVisitor.php b/application/hub/main/visitor/tasks/class_ActiveTaskVisitor.php
index ecfe21bef227622c8c117ab587ba81cf39bc8dcf..c3b6179678999306ef6a2eb1d873b64263d5b545 100644 (file)
--- a/application/hub/main/visitor/tasks/class_ActiveTaskVisitor.php
+++ b/application/hub/main/visitor/tasks/class_ActiveTaskVisitor.php
@@ -4,7 +4,7 @@
  *
  * @author             Roland Haeder <webmaster@ship-simu.org>
  * @version            0.0.0
- * @copyright  Copyright (c) 2007, 2008 Roland Haeder, 2009 Hub Developer Team
+ * @copyright  Copyright (c) 2007, 2008 Roland Haeder, 2009, 2010 Hub Developer Team
  * @license            GNU GPL 3.0 or any newer version
  * @link               http://www.ship-simu.org
  *
@@ -21,7 +21,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
-class ActiveTaskVisitor extends BaseVisitor implements TaskVisitor, QueryConnectorVisitor, PoolVisitor, ListenerVisitor, DecoratorVisitor {
+class ActiveTaskVisitor extends BaseVisitor implements TaskVisitor, QueryConnectorVisitor, QueueConnectorVisitor, PoolVisitor, ListenerVisitor, DecoratorVisitor {
        /**
         * Protected constructor
         *
@@ -38,9 +38,9 @@ class ActiveTaskVisitor extends BaseVisitor implements TaskVisitor, QueryConnect
        /**
         * Creates an instance of this class
         *
-        * @return      $visitorInstance                An instance a Visitorable class
+        * @return      $visitorInstance        An instance a Visitorable class
         */
-       public final static function createActiveTaskVisitor () {
+       public static final function createActiveTaskVisitor () {
                // Get new instance
                $visitorInstance = new ActiveTaskVisitor();
 
@@ -56,7 +56,9 @@ class ActiveTaskVisitor extends BaseVisitor implements TaskVisitor, QueryConnect
         */
        public function visitTask (Taskable $taskInstance) {
                // Execute the task from this visitor
-               $taskInstance->execute();
+               //* DEBUG: */ $this->debugOutput('VISITOR: Visiting task ' . $taskInstance->__toString() . ' - START');
+               $taskInstance->executeTask();
+               //* DEBUG: */ $this->debugOutput('VISITOR: Visiting task ' . $taskInstance->__toString() . ' - FINISHED');
        }
 
        /**
@@ -67,7 +69,18 @@ class ActiveTaskVisitor extends BaseVisitor implements TaskVisitor, QueryConnect
         */
        public function visitQueryConnector (Connectable $connectorInstance) {
                // Process all pending queries
-               $connectorInstance->processAllPendingQueries();
+               $connectorInstance->handlePanding();
+       }
+
+       /**
+        * Connector visitor method for active tasks
+        *
+        * @param       $connectorInstance      A Connectable instance
+        * @return      void
+        */
+       public function visitQueueConnector (Connectable $connectorInstance) {
+               // Process all pending queries
+               $connectorInstance->handlePanding();
        }
 
        /**
@@ -77,9 +90,11 @@ class ActiveTaskVisitor extends BaseVisitor implements TaskVisitor, QueryConnect
         * @return      void
         */
        public function visitPool (Poolable $poolInstance) {
-               // We don't need to visit a pool as an active task because a pool can
-               // never become a task. Instead e.g. by a listener pool we should visit
-               // all listeners one by one
+               /**
+                * We don't need to visit a pool as an active task because a pool can
+                * never become a task. Instead e.g. by a listener pool we should visit
+                * all listeners one by one
+                */
        }
 
        /**
@@ -93,6 +108,17 @@ class ActiveTaskVisitor extends BaseVisitor implements TaskVisitor, QueryConnect
                // Empty for now...
        }
 
+       /**
+        * Queue visitor method for active queries
+        *
+        * @param       $queueInstance  A Queueable instance
+        * @return      void
+        * @todo        Does a queue needs to perform some actions as an active task?
+        */
+       public function visitQueue (Queueable $queueInstance) {
+               // Empty for now...
+       }
+
        /**
         * Visits the given listener instance
         *
@@ -101,7 +127,9 @@ class ActiveTaskVisitor extends BaseVisitor implements TaskVisitor, QueryConnect
         */
        public function visitListener (Listenable $listenerInstance) {
                // Do "listen" here
+               //* DEBUG: */ $this->debugOutput('VISITOR: Visiting ' . $listenerInstance->__toString() . ' - START');
                $listenerInstance->doListen();
+               //* DEBUG: */ $this->debugOutput('VISITOR: Visiting ' . $listenerInstance->__toString() . ' - FINISH');
        }
 
        /**
@@ -111,7 +139,7 @@ class ActiveTaskVisitor extends BaseVisitor implements TaskVisitor, QueryConnect
         * @return      void
         */
        public function visitDecorator (BaseDecorator $decoratorInstance) {
-               // A decorator itself can never bevome an active task so this method
+               // A decorator itself can never become an active task so this method
                // remains empty.
        }
 }