SmartNull: getIterator() delegates to the empty collection, not SmartString

Both classes define getIterator(), and the HTML-mode SmartString-first delegation routed it to SmartString, so an explicit ->getIterator() on a missing value threw "Can't foreach over SmartString NULL" - a type the caller never touched. A missing value now iterates like an empty collection (zero times), matching what foreach already did via SmartNull's own Iterator implementation.

Author: daveedis

src/SmartNull.php

@@ -285,8 +285,11 @@ public function __call($name, array $arguments): mixed
         // SmartString methods only delegate in HTML mode: raw values are plain scalars
         // with no methods, so a miss answers SmartString calls the same way - with the
         // standard undefined-method Error. The isPublic() check keeps private helpers
-        // out: method_exists() reports them, but they aren't part of the API
+        // out: method_exists() reports them, but they aren't part of the API.
+        // getIterator is defined by both classes and skips SmartString: a missing value
+        // iterates like an empty collection, it doesn't throw SmartString's can't-foreach error
         $isSmartStringMethod = $this->useSmartStrings
+            && $name !== 'getIterator'
             && method_exists(SmartString::class, $name)
             && (new ReflectionMethod(SmartString::class, $name))->isPublic();
 

tests/Unit/SmartNullTest.php

@@ -183,6 +183,16 @@ public function testForeachIteratesZeroTimes(string $class): void
         $this->assertSame([], $seen);
     }
 
+    #[DataProvider('modeProvider')]
+    public function testGetIteratorReturnsEmptyCollectionIterator(string $class): void
+    {
+        // getIterator exists on SmartString too, but a missing value iterates like
+        // an empty collection instead of throwing SmartString's can't-foreach error
+        $smartNull = $this->smartNullFrom($class);
+
+        $this->assertSame([], iterator_to_array($smartNull->getIterator()));
+    }
+
     //endregion
     //region Method delegation: SmartArray methods and mode inheritance