From 46d59f30bf39638f23e0b35a70219861d51512b4 Mon Sep 17 00:00:00 2001
From: "florian.azizian" <florian.azizian@maarch.org>
Date: Wed, 6 Jun 2018 12:47:34 +0000
Subject: [PATCH] FIX #7712 remove todo
---
.../models/NotificationScheduleModelAbstract.php | 11 +++++------
1 file changed, 5 insertions(+), 6 deletions(-)
diff --git a/src/app/notification/models/NotificationScheduleModelAbstract.php b/src/app/notification/models/NotificationScheduleModelAbstract.php
index dffb8fadbac..2f80de27ca2 100644
--- a/src/app/notification/models/NotificationScheduleModelAbstract.php
+++ b/src/app/notification/models/NotificationScheduleModelAbstract.php
@@ -64,10 +64,9 @@ abstract class NotificationScheduleModelAbstract
$aArgs['setHiddenValue'] = true;
}
- $crontab = shell_exec('crontab -l');
- // TODO check crontab is installed
- $lines = explode("\n", $crontab);
- $data = [];
+ $crontab = shell_exec('crontab -l');
+ $lines = explode("\n", $crontab);
+ $data = [];
$customId = CoreConfigModel::getCustomId();
$corePath = str_replace('custom/'.$customId.'/src/app/notification/models', '', __DIR__);
$corePath = str_replace('src/app/notification/models', '', $corePath);
@@ -82,8 +81,8 @@ abstract class NotificationScheduleModelAbstract
}
$cronLine = preg_replace('![ \t]+!', ' ', $cronLine);
if ($cronLine[0] == '@') {
- // TODO $time not used
- list($time, $cmd) = explode(' ', $cronLine, 2);
+ $explodeCronLine = explode(' ', $cronLine, 2);
+ $cmd = $explodeCronLine[1];
} else {
list($m, $h, $dom, $mon, $dow, $cmd) = explode(' ', $cronLine, 6);
}
--
GitLab