Report/SendEventErrors.php

[Pman.Admin] / Report / SendEventErrors.php

diff --git a/Report/SendEventErrors.php b/Report/SendEventErrors.php
index cb331ed..2c3b4a0 100644 (file)
--- a/Report/SendEventErrors.php
+++ b/Report/SendEventErrors.php
@@ -145,11 +145,6 @@ class Pman_Admin_Report_SendEventErrors extends Pman_Roo
         if(empty($summary)){
             $this->jerror('ERROR-REPORT-' . $this->opts['uid'], 'Nothing to be sent');
         }
-        $this->addEvent('ERROR-REPORT-' . $this->opts['uid'], 'false', ;
-        $e = DB_DataObject::factory('Events');
-        $e->init('ERROR-REPORT',false,"Sending");
-        $e->event_when = date('Y-m-d H:i:s');
-        $eid = $e->insert();
         
         $subject = array();
         
@@ -163,6 +158,7 @@ class Pman_Admin_Report_SendEventErrors extends Pman_Roo
             $subject = "{$this->opts['subject']} $subject";
         }
         
+        
         $events = DB_DataObject::factory('Events');
         $events->autoJoin();
         
@@ -175,7 +171,7 @@ class Pman_Admin_Report_SendEventErrors extends Pman_Roo
             
         ");
         $events->selectAddPersonEmail();
-        
+        $events->whereAdd('id > '. $min);
         $events->whereAdd("Events.event_when > NOW() - INTERVAL 1 DAY");
         
         $exclude = array_unique(array_filter(array_map('trim', explode(',', $this->opts['exclude']))));
@@ -183,11 +179,18 @@ class Pman_Admin_Report_SendEventErrors extends Pman_Roo
         if(!empty($exclude)){
             $events->whereAddIn('!Events.action', $exclude, 'string');
         }
+        if(!empty($only)){
+            $events->whereAddIn('Events.action', $only, 'string');
+        }
         
-        if(!$events->count()){
+        if(!$events->count()){  // this is the second count we are doing...
             $this->jerr('Nothing to be sent');
         }
         
+        
+        $this->addEvent('ERROR-REPORT-' . $this->opts['uid'], false, $subject);
+
+        
         $errors = $events->fetchAll();
         
         if(!empty($this->opts['host'])){