From 156a76a8e5d82f164ec5ac0ea53002edb003bf19 Mon Sep 17 00:00:00 2001
From: pjayme <pjayme@silverstripe.com>
Date: Wed, 22 Aug 2018 08:20:36 +1200
Subject: [PATCH] BUGFIX: Error trying to access OwnerGroups() method on
 boolean value

---
 src/Reports/PagesDueForReviewReport.php |  8 +++++---
 src/Tasks/ContentReviewEmails.php       | 12 +++++++-----
 2 files changed, 12 insertions(+), 8 deletions(-)

diff --git a/src/Reports/PagesDueForReviewReport.php b/src/Reports/PagesDueForReviewReport.php
index 7fb3545..fc0bd02 100644
--- a/src/Reports/PagesDueForReviewReport.php
+++ b/src/Reports/PagesDueForReviewReport.php
@@ -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;
+                        }
                     }
                 }
 
diff --git a/src/Tasks/ContentReviewEmails.php b/src/Tasks/ContentReviewEmails.php
index 2a6e9ee..2e52873 100644
--- a/src/Tasks/ContentReviewEmails.php
+++ b/src/Tasks/ContentReviewEmails.php
@@ -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);
+                }
             }
         }