Seldaek
diff --git a/‎src/Symfony/Component/Finder/Iterator/RecursiveDirectoryIterator.php
Lines changed: 5 additions & 0 deletions b/‎src/Symfony/Component/Finder/Iterator/RecursiveDirectoryIterator.php
Lines changed: 5 additions & 0 deletions
diff --git a/‎src/Symfony/Component/Finder/Tests/Iterator/FileTypeFilterIteratorTest.php
Lines changed: 3 additions & 3 deletions b/‎src/Symfony/Component/Finder/Tests/Iterator/FileTypeFilterIteratorTest.php
Lines changed: 3 additions & 3 deletions
diff --git a/‎src/Symfony/Component/Finder/Tests/Iterator/FilenameFilterIteratorTest.php
Lines changed: 1 addition & 1 deletion b/‎src/Symfony/Component/Finder/Tests/Iterator/FilenameFilterIteratorTest.php
Lines changed: 1 addition & 1 deletion
diff --git a/‎src/Symfony/Component/Finder/Tests/Iterator/MockSplFileInfo.php
Lines changed: 4 additions & 8 deletions b/‎src/Symfony/Component/Finder/Tests/Iterator/MockSplFileInfo.php
Lines changed: 4 additions & 8 deletions
diff --git a/‎src/Symfony/Component/Finder/Tests/Iterator/SizeRangeFilterIteratorTest.php
Lines changed: 4 additions & 4 deletions b/‎src/Symfony/Component/Finder/Tests/Iterator/SizeRangeFilterIteratorTest.php
Lines changed: 4 additions & 4 deletions
@@ -58,6 +58,7 @@ public function __construct(string $path, int $flags, bool $ignoreUnreadableDirs
      *
      * @return SplFileInfo File information
      */
+    #[\ReturnTypeWillChange]
     public function current()
     {
         // the logic here avoids redoing the same work in all iterations
@@ -82,6 +83,7 @@ public function current()
      *
      * @throws AccessDeniedException
      */
+    #[\ReturnTypeWillChange]
     public function getChildren()
     {
         try {
@@ -109,7 +111,10 @@ public function getChildren()
 
     /**
      * Do nothing for non rewindable stream.
+     *
+     * @return void
      */
+    #[\ReturnTypeWillChange]
     public function rewind()
     {
         if (false === $this->isRewindable()) {
 
@@ -65,17 +65,17 @@ public function getAcceptData()
 
 class InnerTypeIterator extends \ArrayIterator
 {
-    public function current()
+    public function current(): \SplFileInfo
     {
         return new \SplFileInfo(parent::current());
     }
 
-    public function isFile()
+    public function isFile(): bool
     {
         return $this->current()->isFile();
     }
 
-    public function isDir()
+    public function isDir(): bool
     {
         return $this->current()->isDir();
     }
 
@@ -42,7 +42,7 @@ public function getAcceptData()
 
 class InnerNameIterator extends \ArrayIterator
 {
-    public function current()
+    public function current(): \SplFileInfo
     {
         return new \SplFileInfo(parent::current());
     }
 
@@ -48,7 +48,7 @@ public function __construct($param)
         }
     }
 
-    public function isFile()
+    public function isFile(): bool
     {
         if (null === $this->type) {
             return false !== strpos($this->getFilename(), 'file');
@@ -57,7 +57,7 @@ public function isFile()
         return self::TYPE_FILE === $this->type;
     }
 
-    public function isDir()
+    public function isDir(): bool
     {
         if (null === $this->type) {
             return false !== strpos($this->getFilename(), 
8000
'directory');
@@ -66,13 +66,9 @@ public function isDir()
         return self::TYPE_DIRECTORY === $this->type;
     }
 
-    public function isReadable()
+    public function isReadable(): bool
     {
-        if (null === $this->mode) {
-            return preg_match('/r\+/', $this->getFilename());
-        }
-
-        return preg_match('/r\+/', $this->mode);
+        return (bool) preg_match('/r\+/', $this->mode ?? $this->getFilename());
     }
 
     public function getContents()
 
@@ -48,22 +48,22 @@ public function getAcceptData()
 
 class InnerSizeIterator extends \ArrayIterator
 {
-    public function current()
+    public function current(): \SplFileInfo
     {
         return new \SplFileInfo(parent::current());
     }
 
-    public function getFilename()
+    public function getFilename(): string
     {
         return parent::current();
     }
 
-    public function isFile()
+    public function isFile(): bool
     {
         return $this->current()->isFile();
     }
 
-    public function getSize()
+    public function getSize(): int
     {
         return $this->current()->getSize();
     }
Original file line number	Diff line number	Diff line change
`@@ -58,6 +58,7 @@ public function __construct(string $path, int $flags, bool $ignoreUnreadableDirs`
`58`	`58`	`*`
`59`	`59`	`* @return SplFileInfo File information`
`60`	`60`	`*/`
	`61`	`+ #[\ReturnTypeWillChange]`
`61`	`62`	`public function current()`
`62`	`63`	`{`
`63`	`64`	`// the logic here avoids redoing the same work in all iterations`
`@@ -82,6 +83,7 @@ public function current()`
`82`	`83`	`*`
`83`	`84`	`* @throws AccessDeniedException`
`84`	`85`	`*/`
	`86`	`+ #[\ReturnTypeWillChange]`
`85`	`87`	`public function getChildren()`
`86`	`88`	`{`
`87`	`89`	`try {`
`@@ -109,7 +111,10 @@ public function getChildren()`
`109`	`111`
`110`	`112`	`/**`
`111`	`113`	`* Do nothing for non rewindable stream.`
	`114`	`+ *`
	`115`	`+ * @return void`
`112`	`116`	`*/`
	`117`	`+ #[\ReturnTypeWillChange]`
`113`	`118`	`public function rewind()`
`114`	`119`	`{`
`115`	`120`	`if (false === $this->isRewindable()) {`
Original file line number	Diff line number	Diff line change
`@@ -65,17 +65,17 @@ public function getAcceptData()`
`65`	`65`
`66`	`66`	`class InnerTypeIterator extends \ArrayIterator`
`67`	`67`	`{`
`68`		`- public function current()`
	`68`	`+ public function current(): \SplFileInfo`
`69`	`69`	`{`
`70`	`70`	`return new \SplFileInfo(parent::current());`
`71`	`71`	`}`
`72`	`72`
`73`		`- public function isFile()`
	`73`	`+ public function isFile(): bool`
`74`	`74`	`{`
`75`	`75`	`return $this->current()->isFile();`
`76`	`76`	`}`
`77`	`77`
`78`		`- public function isDir()`
	`78`	`+ public function isDir(): bool`
`79`	`79`	`{`
`80`	`80`	`return $this->current()->isDir();`
`81`	`81`	`}`
Original file line number	Diff line number	Diff line change
`@@ -42,7 +42,7 @@ public function getAcceptData()`
`42`	`42`
`43`	`43`	`class InnerNameIterator extends \ArrayIterator`
`44`	`44`	`{`
`45`		`- public function current()`
	`45`	`+ public function current(): \SplFileInfo`
`46`	`46`	`{`
`47`	`47`	`return new \SplFileInfo(parent::current());`
`48`	`48`	`}`
Original file line number	Diff line number	Diff line change
`@@ -48,7 +48,7 @@ public function __construct($param)`
`48`	`48`	`}`
`49`	`49`	`}`
`50`	`50`
`51`		`- public function isFile()`
	`51`	`+ public function isFile(): bool`
`52`	`52`	`{`
`53`	`53`	`if (null === $this->type) {`
`54`	`54`	`return false !== strpos($this->getFilename(), 'file');`
`@@ -57,7 +57,7 @@ public function isFile()`
`57`	`57`	`return self::TYPE_FILE === $this->type;`
`58`	`58`	`}`
`59`	`59`
`60`		`- public function isDir()`
	`60`	`+ public function isDir(): bool`
`61`	`61`	`{`
`62`	`62`	`if (null === $this->type) {`
`63`	`63`	`return false !== strpos($this->getFilename(), 8000 'directory');`
`@@ -66,13 +66,9 @@ public function isDir()`
`66`	`66`	`return self::TYPE_DIRECTORY === $this->type;`
`67`	`67`	`}`
`68`	`68`
`69`		`- public function isReadable()`
	`69`	`+ public function isReadable(): bool`
`70`	`70`	`{`
`71`		`- if (null === $this->mode) {`
`72`		`- return preg_match('/r\+/', $this->getFilename());`
`73`		`- }`
`74`		`-`
`75`		`- return preg_match('/r\+/', $this->mode);`
	`71`	`+ return (bool) preg_match('/r\+/', $this->mode ?? $this->getFilename());`
`76`	`72`	`}`
`77`	`73`
`78`	`74`	`public function getContents()`
Original file line number	Diff line number	Diff line change
`@@ -48,22 +48,22 @@ public function getAcceptData()`
`48`	`48`
`49`	`49`	`class InnerSizeIterator extends \ArrayIterator`
`50`	`50`	`{`
`51`		`- public function current()`
	`51`	`+ public function current(): \SplFileInfo`
`52`	`52`	`{`
`53`	`53`	`return new \SplFileInfo(parent::current());`
`54`	`54`	`}`
`55`	`55`
`56`		`- public function getFilename()`
	`56`	`+ public function getFilename(): string`
`57`	`57`	`{`
`58`	`58`	`return parent::current();`
`59`	`59`	`}`
`60`	`60`
`61`		`- public function isFile()`
	`61`	`+ public function isFile(): bool`
`62`	`62`	`{`
`63`	`63`	`return $this->current()->isFile();`
`64`	`64`	`}`
`65`	`65`
`66`		`- public function getSize()`
	`66`	`+ public function getSize(): int`
`67`	`67`	`{`
`68`	`68`	`return $this->current()->getSize();`
`69`	`69`	`}`