BUGFIX: empty string is not null

This commit is contained in:
Jeroen De Meerleer 2021-12-23 11:11:01 +01:00
parent ca84a53cd0
commit 62e6ef7486
Signed by: JeroenED
GPG Key ID: 28CCCB8F62BFADD6
1 changed files with 3 additions and 3 deletions

View File

@ -503,13 +503,13 @@ class Job extends Repository
}
$values['nextrun'] = DateTime::createFromFormat('d/m/Y H:i:s', $values['nextrun'])->getTimestamp();
$values['data']['retention'] = $values['retention'];
$values['data']['retention'] = !empty($values['retention']) ? (int)$values['retention'] : NULL;
$values['data']['crontype'] = $values['crontype'];
$values['data']['hosttype'] = $values['hosttype'];
$values['data']['containertype'] = $values['containertype'];
$values['data']['fail-pct'] = $values['fail-pct'] ?? 50;
$values['data']['fail-days'] = $values['fail-days'] ?? 7;
$values['data']['fail-pct'] = !empty($values['fail-pct']) ? (int)$values['fail-pct'] : 50;
$values['data']['fail-days'] = !empty($values['fail-days']) ? (int)$values['fail-days'] : 7;
if(empty($values['data']['crontype'])) {
throw new \InvalidArgumentException("Crontype cannot be empty");