fix: make PostIterator->last_post nullable (timber#2918)

Sometimes, PostIterator->last_post can be `null`. Add nullable type and check if last_post is an instance of Post.

---------

Co-authored-by: Nicolas Lemoine <nico.lemoine@gmail.com>
Co-authored-by: Erik van der Bas <erik@basedonline.nl>

Author: 3 people

src/PostsIterator.php

@@ -12,10 +12,10 @@
 class PostsIterator extends ArrayIterator
 {
     /**
-     * @var Post The last post that was returned by the iterator. Used
+     * @var null|Post The last post that was returned by the iterator. Used
      *                   to skip the logic in `current()`.
      */
-    protected Post $last_post;
+    protected ?Post $last_post;
 
     /**
      * Prepares the state before working on a post.
@@ -46,7 +46,10 @@ public function current()
 
         // Lazily instantiate a Timber\Post instance exactly once.
         $post = $factory->from($wp_post);
-        $post->setup();
+
+        if ($post instanceof Post) {
+            $post->setup();
+        }
 
         $this->last_post = $post;
 
@@ -68,7 +71,10 @@ public function next(): void
          * $post->setup() again.
          */
         $post = $this->last_post;
-        $post->teardown();
+
+        if ($post instanceof Post) {
+            $post->teardown();
+        }
 
         // Fire action when the loop has ended.
         if ($this->key() === $this->count() - 1) {