Refactored sql query to make it clearer and also to reduce the number of columns...

[timetracker.git] / cron.php

diff --git a/cron.php b/cron.php
index cf5ac0dde56d11df60ba36c0d544010cd9f8f927..a0bf9853df95994bed6c7f9c084094bcf8744a2c 100644 (file)
--- a/cron.php
+++ b/cron.php
@@ -47,8 +47,10 @@ import('ttReportHelper');
 $mdb2 = getConnection();
 $now = mktime();
 
-$sql = "select * from tt_cron where $now >= next 
-  and status = 1 and report_id is not null and email is not null";
+ $sql = "select c.id, c.cron_spec, c.report_id, c.email, c.report_condition from tt_cron c
+   left join tt_fav_reports fr on (c.report_id = fr.id)
+   where $now >= c.next and fr.status = 1
+   and c.status = 1 and c.report_id is not null and c.email is not null";
 $res = $mdb2->query($sql);
 if (is_a($res, 'PEAR_Error'))
   exit();