tonyair
/
silverstripe-contentreview
mirror of https://github.com/silverstripe/silverstripe-contentreview
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

advanceReviewDate() should be cleared after a review, don't use unless there are any

This commit is contained in:
Julian 2016-01-19 05:47:18 +13:00
parent facac031e4
commit e310f18dae
1 changed files with 9 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
code/extensions/SiteTreeContentReview.php
View File

@ -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) {