diff --git a/code/extensions/SiteTreeContentReview.php b/code/extensions/SiteTreeContentReview.php index d6c0d3e..e221df8 100644 --- a/code/extensions/SiteTreeContentReview.php +++ b/code/extensions/SiteTreeContentReview.php @@ -425,11 +425,14 @@ class SiteTreeContentReview extends DataExtension implements PermissionProvider $nextDate = false; $options = $this->getOptions(); - if ($options && $options->ReviewPeriodDays) { + if ($options && $options->ReviewPeriodDays > 0) { $nextDate = date('Y-m-d', strtotime('+ ' . $options->ReviewPeriodDays . ' days', SS_Datetime::now()->format('U'))); $this->owner->NextReviewDate = $nextDate; $this->owner->write(); + } else { + $this->owner->NextReviewDate = null; + $this->owner->write(); } return (bool) $nextDate; @@ -454,8 +457,12 @@ class SiteTreeContentReview extends DataExtension implements PermissionProvider $options = $this->getOptions(); - if ($options->OwnerGroups()->count() == 0 && $options->OwnerUsers()->count() == 0) { + if (!$options) { return false; + } else { + if ($options->OwnerGroups()->count() == 0 && $options->OwnerUsers()->count() == 0) { + return false; + } } if (!$member) {