FIX Set empty date to null

silverstripe · Oct 15, 2024 · 3ed6eb0 · 3ed6eb0
1 parent cd1b9ed
commit 3ed6eb0
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/src/Extensions/SiteTreeContentReview.php b/src/Extensions/SiteTreeContentReview.php
@@ -194,7 +194,7 @@ protected function updateCMSActions(FieldList $actions)
      *
      * @param SiteTree $page
      *
-     * @return bool|DBDate
+     * @return null|DBDate
      */
     public function getReviewDate(SiteTree $page = null)
     {
@@ -209,11 +209,11 @@ public function getReviewDate(SiteTree $page = null)
         $options = $this->owner->getOptions();
 
         if (!$options) {
-            return false;
+            return null;
         }
 
         if (!$options->ReviewPeriodDays) {
-            return false;
+            return null;
         }
 
         // Failover to check on ReviewPeriodDays + LastEdited
@@ -509,6 +509,9 @@ public function advanceReviewDate()
                 DBDatetime::now()->getTimestamp()
             );
 
+            $x=DBDate::create()->setValue($nextDateTimestamp)->Format(DBDate::ISO_DATE);
+            ;
+
             $this->owner->NextReviewDate = DBDate::create()->setValue($nextDateTimestamp)->Format(DBDate::ISO_DATE);
             $this->owner->write();
         }
@@ -662,6 +665,8 @@ protected function setDefaultReviewDateForInherited()
 
         if (is_object($nextDate)) {
             $this->owner->NextReviewDate = $nextDate->getValue();
+        } elseif ($nextDate === false) {
+            $this->owner->NextReviewDate = null;
         } else {
             $this->owner->NextReviewDate = $nextDate;
         }