Cron job spams

Check if some SMS have to be sent before echoing text, so the cron jobs don't send useless mail every minute.
Vérifie que le nombre de SMS à envoyer soit positif avant d'afficher le texte de log, ainsi le cron job n'envoie pas de mail intempestif toute les minutes.
This commit is contained in:
Pierre BLANCHE 2019-02-10 17:53:04 +01:00 committed by GitHub
parent e585647583
commit 60965ffde2
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -81,8 +81,6 @@
$now = new DateTime(); $now = new DateTime();
$now = $now->format('Y-m-d H:i:s'); $now = $now->format('Y-m-d H:i:s');
echo "Début de l'envoi des SMS programmés\n";
$scheduleds = $db->getScheduledsNotInProgressBefore($now); $scheduleds = $db->getScheduledsNotInProgressBefore($now);
$ids_scheduleds = array(); $ids_scheduleds = array();
@ -93,7 +91,12 @@
$ids_scheduleds[] = $scheduled['id']; $ids_scheduleds[] = $scheduled['id'];
} }
echo count($ids_scheduleds) . " SMS à envoyer ont été trouvés et ajoutés à la liste des SMS en cours d'envoi.\n"; $nb_ids_scheduleds = count($ids_scheduleds);
if ($nb_ids_scheduleds > 0)
{
echo "Début de l'envoi des SMS programmés\n";
echo $nb_ids_scheduleds . " SMS à envoyer ont été trouvés et ajoutés à la liste des SMS en cours d'envoi.\n";
}
$db->updateProgressScheduledsIn($ids_scheduleds, true); $db->updateProgressScheduledsIn($ids_scheduleds, true);
@ -182,7 +185,7 @@
} }
} }
echo "Tous les SMS sont en cours d'envoi.\n"; if ($nb_ids_scheduleds > 0) echo "Tous les SMS sont en cours d'envoi.\n";
//Tous les SMS ont été envoyés. //Tous les SMS ont été envoyés.
$db->deleteScheduledsIn($ids_scheduleds); $db->deleteScheduledsIn($ids_scheduleds);