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

BUGFIX: Error trying to access OwnerGroups() method on boolean value

This commit is contained in:
pjayme 2018-08-22 08:20:36 +12:00
parent cc4b6960e6
commit 156a76a8e5
2 changed files with 12 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/Reports/PagesDueForReviewReport.php
View File

@ -217,9 +217,11 @@ class PagesDueForReviewReport extends Report
$records = $records->filterByCallback(function ($page) use ($currentUser) {
$options = $page->getOptions();
foreach ($options->ContentReviewOwners() as $owner) {
if ($currentUser->ID == $owner->ID) {
return true;
if ($options) {
foreach ($options->ContentReviewOwners() as $owner) {
if ($currentUser->ID == $owner->ID) {
return true;
}
}
}

12
src/Tasks/ContentReviewEmails.php
View File

@ -70,12 +70,14 @@ class ContentReviewEmails extends BuildTask
$option = $page->getOptions();
foreach ($option->ContentReviewOwners() as $owner) {
if (!isset($overduePages[$owner->ID])) {
$overduePages[$owner->ID] = ArrayList::create();
}
if ($options) {
foreach ($option->ContentReviewOwners() as $owner) {
if (!isset($overduePages[$owner->ID])) {
$overduePages[$owner->ID] = ArrayList::create();
}
$overduePages[$owner->ID]->push($page);
$overduePages[$owner->ID]->push($page);
}
}
}