Make sure that some event subscribers are not executed multiple times

1 files changed, 8 insertions, 7 deletions

diff --git a/app/Subscriber/ProjectDailySummarySubscriber.php b/app/Subscriber/ProjectDailySummarySubscriber.php
index d9239264..b70eaf71 100644
--- a/app/Subscriber/ProjectDailySummarySubscriber.php
+++ b/app/Subscriber/ProjectDailySummarySubscriber.php
@@ -6,22 +6,23 @@ use Kanboard\Event\TaskEvent;
 use Kanboard\Model\Task;
 use Symfony\Component\EventDispatcher\EventSubscriberInterface;
 
-class ProjectDailySummarySubscriber extends \Kanboard\Core\Base implements EventSubscriberInterface
+class ProjectDailySummarySubscriber extends BaseSubscriber implements EventSubscriberInterface
 {
     public static function getSubscribedEvents()
     {
         return array(
-            Task::EVENT_CREATE_UPDATE => array('execute', 0),
-            Task::EVENT_CLOSE => array('execute', 0),
-            Task::EVENT_OPEN => array('execute', 0),
-            Task::EVENT_MOVE_COLUMN => array('execute', 0),
-            Task::EVENT_MOVE_SWIMLANE => array('execute', 0),
+            Task::EVENT_CREATE_UPDATE => 'execute',
+            Task::EVENT_CLOSE => 'execute',
+            Task::EVENT_OPEN => 'execute',
+            Task::EVENT_MOVE_COLUMN => 'execute',
+            Task::EVENT_MOVE_SWIMLANE => 'execute',
         );
     }
 
     public function execute(TaskEvent $event)
     {
-        if (isset($event['project_id'])) {
+        if (isset($event['project_id']) && !$this->isExecuted()) {
+            $this->logger->debug('Subscriber executed: '.__CLASS__.'::'.__METHOD__);
             $this->projectDailyColumnStats->updateTotals($event['project_id'], date('Y-m-d'));
             $this->projectDailyStats->updateTotals($event['project_id'], date('Y-m-d'));
         }