From cd17a23d13663379b769d9c766a4b533b1b4bc8c Mon Sep 17 00:00:00 2001 From: Duncan Sommerville Date: Thu, 29 Oct 2015 09:23:41 -0400 Subject: [PATCH] Fix download counter reset timer --- airtime_mvc/application/common/TaskManager.php | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/airtime_mvc/application/common/TaskManager.php b/airtime_mvc/application/common/TaskManager.php index 18ebe98f7..180c11932 100644 --- a/airtime_mvc/application/common/TaskManager.php +++ b/airtime_mvc/application/common/TaskManager.php @@ -86,7 +86,7 @@ final class TaskManager { $this->_con->beginTransaction(); try { $lock = $this->_getLock(); - if ($lock && (microtime(true) < $lock['valstr'] + self::TASK_INTERVAL_SECONDS)) { + if ($lock && (microtime(true) < ($lock['valstr'] + self::TASK_INTERVAL_SECONDS))) { // Propel caches the database connection and uses it persistently, so if we don't // use commit() here, we end up blocking other queries made within this request $this->_con->commit(); @@ -256,7 +256,7 @@ class StationPodcastTask implements AirtimeTask { */ public function shouldBeRun() { $lastReset = Application_Model_Preference::getStationPodcastDownloadResetTimer(); - return empty($lastReset) || (microtime(true) > $lastReset + self::STATION_PODCAST_RESET_TIMER_SECONDS); + return empty($lastReset) || (microtime(true) > ($lastReset + self::STATION_PODCAST_RESET_TIMER_SECONDS)); } /** @@ -264,6 +264,7 @@ class StationPodcastTask implements AirtimeTask { */ public function run() { Application_Model_Preference::resetStationPodcastDownloadCounter(); + Application_Model_Preference::setStationPodcastDownloadResetTimer(microtime(true)); } }