diff --git a/htdocs/public/cron/cron_run_jobs.php b/htdocs/public/cron/cron_run_jobs.php
index 57cde107f24..2d093db07b1 100644
--- a/htdocs/public/cron/cron_run_jobs.php
+++ b/htdocs/public/cron/cron_run_jobs.php
@@ -44,12 +44,16 @@ dol_include_once("/cron/class/cronjob.class.php");
global $langs, $conf;
// Language Management
-$langs->loadLangs(array("admin", "cron"));
+$langs->loadLangs(array("admin", "cron", "dict"));
+
/*
* View
*/
+// current date
+$now=dol_now();
+
// Check the key, avoid that a stranger starts cron
$key = GETPOST('securitykey','alpha');
if (empty($key))
@@ -75,15 +79,15 @@ $result=$user->fetch('',$userlogin);
if ($result < 0)
{
echo "User Error:".$user->error;
- dol_syslog("cron_run_jobs.php:: User Error:".$user->error, LOG_WARNING);
+ dol_syslog("cron_run_jobs.php:: User Error:".$user->error, LOG_ERR);
exit;
}
else
{
if (empty($user->id))
{
- echo " User user login:".$userlogin." do not exists";
- dol_syslog(" User user login:".$userlogin." do not exists", LOG_WARNING);
+ echo " User login:".$userlogin." do not exists";
+ dol_syslog(" User login:".$userlogin." do not exists", LOG_ERR);
exit;
}
}
@@ -94,8 +98,7 @@ $id = GETPOST('id','alpha'); // We accept non numeric id. We will filter later.
$object = new Cronjob($db);
$filter=array();
-if (! empty($id))
-{
+if (! empty($id)) {
if (! is_numeric($id))
{
echo "Error: Bad value for parameter job id";
@@ -109,7 +112,7 @@ $result = $object->fetch_all('ASC,ASC,ASC','t.priority,t.entity,t.rowid', 0, 0,
if ($result<0)
{
echo "Error: ".$object->error;
- dol_syslog("cron_run_jobs.php fetch Error".$object->error, LOG_WARNING);
+ dol_syslog("cron_run_jobs.php fetch Error".$object->error, LOG_ERR);
exit;
}
@@ -123,37 +126,76 @@ foreach($object->lines as $val)
// TODO Duplicate code. This sequence of code must be shared with code into cron_run_jobs.php script.
// current date
-$now=dol_now();
$nbofjobs=count($qualifiedjobs);
$nbofjobslaunchedok=0;
$nbofjobslaunchedko=0;
if (is_array($qualifiedjobs) && (count($qualifiedjobs)>0))
{
- // Loop over job
+ $savconf = dol_clone($conf);
+
+ // Loop over job
foreach($qualifiedjobs as $line)
{
- dol_syslog("cron_run_jobs.php cronjobid: ".$line->id, LOG_WARNING);
+ dol_syslog("cron_run_jobs.php cronjobid: ".$line->id." priority=".$line->priority." entity=".$line->entity." label=".$line->label, LOG_DEBUG);
+ echo "cron_run_jobs.php cronjobid: ".$line->id." priority=".$line->priority." entity=".$line->entity." label=".$line->label;
+
+ // Force reload of setup for the current entity
+ if ($line->entity != $conf->entity)
+ {
+ dol_syslog("cron_run_jobs.php we work on another entity so we reload user and conf", LOG_DEBUG);
+ echo " -> we change entity so we reload user and conf";
+
+ $conf->entity = (empty($line->entity)?1:$line->entity);
+ $conf->setValues($db); // This make also the $mc->setValues($conf); that reload $mc->sharings
- //If date_next_jobs is less of current dat, execute the program, and store the execution time of the next execution in database
+ // Force recheck that user is ok for the entity to process and reload permission for entity
+ if ($conf->entity != $user->entity && $user->entity != 0)
+ {
+ $result=$user->fetch('', $userlogin, '', 0, $conf->entity);
+ if ($result < 0)
+ {
+ echo "\nUser Error: ".$user->error."\n";
+ dol_syslog("cron_run_jobs.php:: User Error:".$user->error, LOG_ERR);
+ exit(-1);
+ }
+ else
+ {
+ if ($result == 0)
+ {
+ echo "\nUser login: ".$userlogin." does not exists for entity ".$conf->entity."\n";
+ dol_syslog("User login:".$userlogin." does not exists", LOG_ERR);
+ exit(-1);
+ }
+ }
+ $user->getrights();
+ }
+ }
+
+ //If date_next_jobs is less of current date, execute the program, and store the execution time of the next execution in database
if (($line->datenextrun < $now) && (empty($line->datestart) || $line->datestart <= $now) && (empty($line->dateend) || $line->dateend >= $now))
{
- dol_syslog("cron_run_jobs.php:: torun line->datenextrun:".dol_print_date($line->datenextrun,'dayhourtext')." line->dateend:".dol_print_date($line->dateend,'dayhourtext')." now:".dol_print_date($now,'dayhourtext'));
-
+ echo " - qualified";
+
+ dol_syslog("cron_run_jobs.php line->datenextrun:".dol_print_date($line->datenextrun,'dayhourrfc')." line->datestart:".dol_print_date($line->datestart,'dayhourrfc')." line->dateend:".dol_print_date($line->dateend,'dayhourrfc')." now:".dol_print_date($now,'dayhourrfc'));
+
$cronjob=new Cronjob($db);
$result=$cronjob->fetch($line->id);
if ($result<0)
{
- echo "Error cronjob->fetch: ".$cronjob->error."
\n";
+ echo "Error cronjobid: ".$line->id." cronjob->fetch: ".$cronjob->error."\n";
+ echo "Failed to fetch job ".$line->id."\n";
dol_syslog("cron_run_jobs.php::fetch Error".$cronjob->error, LOG_ERR);
exit;
}
- // Execut job
+ // Execute job
$result=$cronjob->run_jobs($userlogin);
if ($result < 0)
{
- echo "Error cronjob->run_job: ".$cronjob->error."
\n";
- dol_syslog("cron_run_jobs.php::run_jobs Error".$cronjob->error, LOG_ERR);
+ echo "Error cronjobid: ".$line->id." cronjob->run_job: ".$cronjob->error."\n";
+ echo "At least one job failed. Go on menu Home-Setup-Admin tools to see result for each job.\n";
+ echo "You can also enable module Log if not yet enabled, run again and take a look into dolibarr.log file\n";
+ dol_syslog("cron_run_jobs.php::run_jobs Error".$cronjob->error, LOG_ERR);
$nbofjobslaunchedko++;
}
else
@@ -161,16 +203,30 @@ if (is_array($qualifiedjobs) && (count($qualifiedjobs)>0))
$nbofjobslaunchedok++;
}
+ echo " - result of run_jobs = ".$result;
+
// We re-program the next execution and stores the last execution time for this job
$result=$cronjob->reprogram_jobs($userlogin, $now);
if ($result<0)
{
- echo "Error cronjob->reprogram_job: ".$cronjob->error."
\n";
- dol_syslog("cron_run_jobs.php::reprogram_jobs Error".$cronjob->error, LOG_ERR);
+ echo "Error cronjobid: ".$line->id." cronjob->reprogram_job: ".$cronjob->error."\n";
+ echo "Enable module Log if not yet enabled, run again and take a look into dolibarr.log file\n";
+ dol_syslog("cron_run_jobs.php::reprogram_jobs Error".$cronjob->error, LOG_ERR);
exit;
}
+
+ echo " - reprogrammed\n";
+ }
+ else
+ {
+ echo " - not qualified\n";
+
+ dol_syslog("cron_run_jobs.php job not qualified line->datenextrun:".dol_print_date($line->datenextrun,'dayhourrfc')." line->datestart:".dol_print_date($line->datestart,'dayhourrfc')." line->dateend:".dol_print_date($line->dateend,'dayhourrfc')." now:".dol_print_date($now,'dayhourrfc'));
}
}
+
+ $conf = $savconf;
+
echo "Result: ".($nbofjobs)." jobs - ".($nbofjobslaunchedok+$nbofjobslaunchedko)." launched = ".$nbofjobslaunchedok." OK + ".$nbofjobslaunchedko." KO";
}
else