Fix return type for SeekableIterator::* implementations
authorTim Düsterhus <duesterhus@woltlab.com>
Mon, 20 Sep 2021 14:23:41 +0000 (16:23 +0200)
committerTim Düsterhus <duesterhus@woltlab.com>
Mon, 20 Sep 2021 14:33:58 +0000 (16:33 +0200)
The `mixed` type is not available with our current minimum PHP version, thus
using ReturnTypeWillChange for these.

wcfsetup/install/files/lib/data/DatabaseObjectList.class.php
wcfsetup/install/files/lib/data/label/group/ViewableLabelGroup.class.php
wcfsetup/install/files/lib/data/smiley/category/SmileyCategory.class.php

index 2741d806962fb7433d7801e5ab34203cad1156f9..596cc4c98ab48c19d229d9506e97a111e1b24753 100644 (file)
@@ -317,6 +317,7 @@ abstract class DatabaseObjectList implements \Countable, ITraversableObject
     /**
      * @inheritDoc
      */
+    #[\ReturnTypeWillChange]
     public function current()
     {
         $objectID = $this->indexToObject[$this->index];
@@ -330,6 +331,7 @@ abstract class DatabaseObjectList implements \Countable, ITraversableObject
      *
      * @see \Iterator::key()
      */
+    #[\ReturnTypeWillChange]
     public function key()
     {
         return $this->indexToObject[$this->index];
@@ -338,7 +340,7 @@ abstract class DatabaseObjectList implements \Countable, ITraversableObject
     /**
      * @inheritDoc
      */
-    public function next()
+    public function next(): void
     {
         $this->index++;
     }
@@ -346,7 +348,7 @@ abstract class DatabaseObjectList implements \Countable, ITraversableObject
     /**
      * @inheritDoc
      */
-    public function rewind()
+    public function rewind(): void
     {
         $this->index = 0;
     }
@@ -354,7 +356,7 @@ abstract class DatabaseObjectList implements \Countable, ITraversableObject
     /**
      * @inheritDoc
      */
-    public function valid()
+    public function valid(): bool
     {
         return isset($this->indexToObject[$this->index]);
     }
@@ -362,7 +364,7 @@ abstract class DatabaseObjectList implements \Countable, ITraversableObject
     /**
      * @inheritDoc
      */
-    public function seek($offset)
+    public function seek($offset): void
     {
         $this->index = $offset;
 
index f9923c7b0e969709cb8c274941ed79bce71dfaa6..d80f9e844a66d09379cd3847c53e023614a96adb 100644 (file)
@@ -179,6 +179,7 @@ class ViewableLabelGroup extends DatabaseObjectDecorator implements \Countable,
     /**
      * @inheritDoc
      */
+    #[\ReturnTypeWillChange]
     public function current()
     {
         $objectID = $this->indexToObject[$this->index];
@@ -192,6 +193,7 @@ class ViewableLabelGroup extends DatabaseObjectDecorator implements \Countable,
      *
      * @inheritDoc
      */
+    #[\ReturnTypeWillChange]
     public function key()
     {
         return $this->indexToObject[$this->index];
@@ -200,7 +202,7 @@ class ViewableLabelGroup extends DatabaseObjectDecorator implements \Countable,
     /**
      * @inheritDoc
      */
-    public function next()
+    public function next(): void
     {
         $this->index++;
     }
@@ -208,7 +210,7 @@ class ViewableLabelGroup extends DatabaseObjectDecorator implements \Countable,
     /**
      * @inheritDoc
      */
-    public function rewind()
+    public function rewind(): void
     {
         $this->index = 0;
     }
@@ -216,7 +218,7 @@ class ViewableLabelGroup extends DatabaseObjectDecorator implements \Countable,
     /**
      * @inheritDoc
      */
-    public function valid()
+    public function valid(): bool
     {
         return isset($this->indexToObject[$this->index]);
     }
@@ -224,7 +226,7 @@ class ViewableLabelGroup extends DatabaseObjectDecorator implements \Countable,
     /**
      * @inheritDoc
      */
-    public function seek($offset)
+    public function seek($offset): void
     {
         $this->index = $offset;
 
index ac43aed81c6b4015414e976556388b974a2caaf8..0ad5dc3c8a58dcf37d74c09d92bee3d1593b5c65 100644 (file)
@@ -66,6 +66,7 @@ class SmileyCategory extends AbstractDecoratedCategory implements \Countable, IT
      * @inheritDoc
      * @return  Smiley
      */
+    #[\ReturnTypeWillChange]
     public function current()
     {
         $objectID = $this->indexToObject[$this->index];
@@ -79,6 +80,7 @@ class SmileyCategory extends AbstractDecoratedCategory implements \Countable, IT
      *
      * @see \Iterator::key()
      */
+    #[\ReturnTypeWillChange]
     public function key()
     {
         return $this->indexToObject[$this->index];
@@ -87,7 +89,7 @@ class SmileyCategory extends AbstractDecoratedCategory implements \Countable, IT
     /**
      * @inheritDoc
      */
-    public function next()
+    public function next(): void
     {
         $this->index++;
     }
@@ -95,7 +97,7 @@ class SmileyCategory extends AbstractDecoratedCategory implements \Countable, IT
     /**
      * @inheritDoc
      */
-    public function rewind()
+    public function rewind(): void
     {
         $this->index = 0;
     }
@@ -103,7 +105,7 @@ class SmileyCategory extends AbstractDecoratedCategory implements \Countable, IT
     /**
      * @inheritDoc
      */
-    public function valid()
+    public function valid(): bool
     {
         return isset($this->indexToObject[$this->index]);
     }
@@ -111,7 +113,7 @@ class SmileyCategory extends AbstractDecoratedCategory implements \Countable, IT
     /**
      * @inheritDoc
      */
-    public function seek($offset)
+    public function seek($offset): void
     {
         $this->index = $offset;