ContentLoader: Allow items_lifetime=0

We documented it as supported in 41215a0 but it would in fact skip adding anything with “item […] older than 0 days”. Let’s make it actually work.
fossar · Mar 29, 2023 · 82fb822 · 82fb822
1 parent b6a9d69
commit 82fb822
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/src/helpers/ContentLoader.php b/src/helpers/ContentLoader.php
@@ -135,9 +135,12 @@ public function fetch($source): void {
             );
 
             // current date
-            $minDate = new \DateTime();
-            $minDate->sub(new \DateInterval('P' . $this->configuration->itemsLifetime . 'D'));
-            $this->logger->debug('minimum date: ' . $minDate->format('Y-m-d H:i:s'));
+            $minDate = null;
+            if ($this->configuration->itemsLifetime !== 0) {
+                $minDate = new \DateTime();
+                $minDate->sub(new \DateInterval('P' . $this->configuration->itemsLifetime . 'D'));
+                $this->logger->debug('minimum date: ' . $minDate->format('Y-m-d H:i:s'));
+            }
 
             // insert new items in database
             $this->logger->debug('start item fetching');
@@ -180,7 +183,7 @@ public function fetch($source): void {
                 if ($itemDate === null) {
                     $itemDate = new \DateTimeImmutable();
                 }
-                if ($itemDate < $minDate) {
+                if ($minDate !== null && $itemDate < $minDate) {
                     $this->logger->debug('item "' . $titlePlainText . '" (' . $itemDate->format(\DateTime::ATOM) . ') older than ' . $this->configuration->itemsLifetime . ' days');
                     continue;
                 }