composer/iTop
6
0
Fork 0
mirror of https://github.com/Combodo/iTop.git synced 2026-04-25 11:38:44 +02:00
Code Issues Packages Projects Releases Wiki Activity

N°4931 Fix background tasks max duration being set to 3 times its periodicity

Browse Source
This commit is contained in:
Stephen Abello
2022-03-14 09:29:53 +01:00
parent b50ba0ad49
commit 93c6cfffda
2 changed files with 17 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
webservices/cron.php
View File

@@ -106,18 +106,18 @@ function RunTask(BackgroundTask $oTask, $iTimeLimit)
$oTask->Set('system_user', utils::GetCurrentUserName());
$oTask->Set('running', 1);
$oTask->DBUpdate();
// Time in seconds allowed to the task
$iCurrTimeLimit = $iTimeLimit;
// Compute allowed time
if ($oRefClass->implementsInterface('iScheduledProcess'))
if ($oRefClass->implementsInterface('iScheduledProcess') === false)
{
$iCurrTimeLimit = $iTimeLimit;
}
else
{
// Periodic task, allow only 3x the period
$iCurrTimeLimit = time() + $oProcess->GetPeriodicity() * 3;
if ($iCurrTimeLimit > $iTimeLimit)
// Periodic task, allow only X times ($iMaxTaskExecutionTime) its periodicity (GetPeriodicity())
$iMaxTaskExecutionTime = MetaModel::GetConfig()->Get('cron_task_max_execution_time');
$iTaskLimit = time() + $oProcess->GetPeriodicity() * $iMaxTaskExecutionTime;
// If our proposed time limit is less than cron limit, and cron_task_max_execution_time is > 0
if ($iTaskLimit < $iTimeLimit && $iMaxTaskExecutionTime > 0)
{
$iCurrTimeLimit = $iTimeLimit;
$iCurrTimeLimit = $iTaskLimit;
}
}
$sMessage = $oProcess->Process($iCurrTimeLimit);