Merge pull request #544 from cedric-anne/fix/php8.1

Fix deprecated usages and return types on PHP 8.1

Author: phil-davis

lib/Component.php

@@ -339,6 +339,7 @@ function ($a, $b) use ($sortScore, $tmp) {
      *
      * @return array
      */
+    #[\ReturnTypeWillChange]
     public function jsonSerialize()
     {
         $components = [];

lib/Component/VCard.php

@@ -445,6 +445,7 @@ protected function getDefaults()
      *
      * @return array
      */
+    #[\ReturnTypeWillChange]
     public function jsonSerialize()
     {
         // A vcard does not have sub-components, so we're overriding this

lib/ElementList.php

@@ -25,6 +25,7 @@ class ElementList extends ArrayIterator
      * @param int   $offset
      * @param mixed $value
      */
+    #[\ReturnTypeWillChange]
     public function offsetSet($offset, $value)
     {
         throw new LogicException('You can not add new objects to an ElementList');
@@ -37,6 +38,7 @@ public function offsetSet($offset, $value)
      *
      * @param int $offset
      */
+    #[\ReturnTypeWillChange]
     public function offsetUnset($offset)
     {
         throw new LogicException('You can not remove objects from an ElementList');

lib/Node.php

@@ -73,6 +73,7 @@ abstract public function serialize();
      *
      * @return array
      */
+    #[\ReturnTypeWillChange]
     abstract public function jsonSerialize();
 
     /**
@@ -102,6 +103,7 @@ public function destroy()
      *
      * @return ElementList
      */
+    #[\ReturnTypeWillChange]
     public function getIterator()
     {
         if (!is_null($this->iterator)) {
@@ -157,6 +159,7 @@ public function validate($options = 0)
      *
      * @return int
      */
+    #[\ReturnTypeWillChange]
     public function count()
     {
         $it = $this->getIterator();
@@ -177,6 +180,7 @@ public function count()
      *
      * @return bool
      */
+    #[\ReturnTypeWillChange]
     public function offsetExists($offset)
     {
         $iterator = $this->getIterator();
@@ -193,6 +197,7 @@ public function offsetExists($offset)
      *
      * @return mixed
      */
+    #[\ReturnTypeWillChange]
     public function offsetGet($offset)
     {
         $iterator = $this->getIterator();
@@ -208,6 +213,7 @@ public function offsetGet($offset)
      * @param int   $offset
      * @param mixed $value
      */
+    #[\ReturnTypeWillChange]
     public function offsetSet($offset, $value)
     {
         $iterator = $this->getIterator();
@@ -228,6 +234,7 @@ public function offsetSet($offset, $value)
      *
      * @param int $offset
      */
+    #[\ReturnTypeWillChange]
     public function offsetUnset($offset)
     {
         $iterator = $this->getIterator();

lib/Parameter.php

@@ -321,6 +321,7 @@ function ($out, $item) {
      *
      * @return array
      */
+    #[\ReturnTypeWillChange]
     public function jsonSerialize()
     {
         return $this->value;
@@ -354,6 +355,7 @@ public function __toString()
      *
      * @return ElementList
      */
+    #[\ReturnTypeWillChange]
     public function getIterator()
     {
         if (!is_null($this->iterator)) {

lib/Parser/MimeDir.php

@@ -439,7 +439,7 @@ protected function readProperty($line)
             $propObj->add(null, $namelessParameter);
         }
 
-        if ('QUOTED-PRINTABLE' === strtoupper($propObj['ENCODING'])) {
+        if (isset($propObj['ENCODING']) && 'QUOTED-PRINTABLE' === strtoupper($propObj['ENCODING'])) {
             $propObj->setQuotedPrintableValue($this->extractQuotedPrintableValue());
         } else {
             $charset = $this->charset;

lib/Property.php

@@ -276,6 +276,7 @@ public function setJsonValue(array $value)
      *
      * @return array
      */
+    #[\ReturnTypeWillChange]
     public function jsonSerialize()
     {
         $parameters = [];
@@ -387,6 +388,7 @@ public function __toString()
      *
      * @return bool
      */
+    #[\ReturnTypeWillChange]
     public function offsetExists($name)
     {
         if (is_int($name)) {
@@ -413,6 +415,7 @@ public function offsetExists($name)
      *
      * @return Node
      */
+    #[\ReturnTypeWillChange]
     public function offsetGet($name)
     {
         if (is_int($name)) {
@@ -433,6 +436,7 @@ public function offsetGet($name)
      * @param string $name
      * @param mixed  $value
      */
+    #[\ReturnTypeWillChange]
     public function offsetSet($name, $value)
     {
         if (is_int($name)) {
@@ -453,6 +457,7 @@ public function offsetSet($name, $value)
      *
      * @param string $name
      */
+    #[\ReturnTypeWillChange]
     public function offsetUnset($name)
     {
         if (is_int($name)) {

lib/Property/ICalendar/DateTime.php

@@ -300,6 +300,7 @@ function ($item) {
      * @param string $name
      * @param mixed  $value
      */
+    #[\ReturnTypeWillChange]
     public function offsetSet($name, $value)
     {
         parent::offsetSet($name, $value);