Merge pull request #1556 from cedric-anne/feature/php-8.4

Add PHP 8.4 support

Author: phil-davis

.github/workflows/ci.yml

@@ -53,7 +53,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        php-versions: [ '7.4', '8.0', '8.1', '8.2', '8.3']
+        php-versions: [ '7.4', '8.0', '8.1', '8.2', '8.3', '8.4' ]
         coverage: [ 'xdebug' ]
         streaming: [ false ]
         include:

lib/CardDAV/Plugin.php

@@ -800,7 +800,7 @@ protected function negotiateVCard($input, &$mimeType = null)
      *
      * @return string
      */
-    protected function convertVCard($data, $target, array $propertiesFilter = null)
+    protected function convertVCard($data, $target, ?array $propertiesFilter = null)
     {
         if (is_resource($data)) {
             $data = stream_get_contents($data);

lib/CardDAV/Xml/Property/SupportedAddressData.php

@@ -34,7 +34,7 @@ class SupportedAddressData implements XmlSerializable
     /**
      * Creates the property.
      */
-    public function __construct(array $supportedData = null)
+    public function __construct(?array $supportedData = null)
     {
         if (is_null($supportedData)) {
             $supportedData = [

lib/DAV/Auth/Plugin.php

@@ -58,7 +58,7 @@ class Plugin extends ServerPlugin
      *
      * @param Backend\BackendInterface $authBackend
      */
-    public function __construct(Backend\BackendInterface $authBackend = null)
+    public function __construct(?Backend\BackendInterface $authBackend = null)
     {
         if (!is_null($authBackend)) {
             $this->addBackend($authBackend);

lib/DAV/Exception/Locked.php

@@ -32,7 +32,7 @@ class Locked extends DAV\Exception
      *
      * @param DAV\Locks\LockInfo $lock
      */
-    public function __construct(DAV\Locks\LockInfo $lock = null)
+    public function __construct(?DAV\Locks\LockInfo $lock = null)
     {
         parent::__construct();
 

lib/DAV/Server.php

@@ -211,7 +211,7 @@ class Server implements LoggerAwareInterface, EmitterInterface
      *
      * @throws Exception
      */
-    public function __construct($treeOrNode = null, HTTP\Sapi $sapi = null)
+    public function __construct($treeOrNode = null, ?HTTP\Sapi $sapi = null)
     {
         if ($treeOrNode instanceof Tree) {
             $this->tree = $treeOrNode;
@@ -882,7 +882,7 @@ public function getHTTPHeaders($path)
      *
      * @return \Traversable
      */
-    private function generatePathNodes(PropFind $propFind, array $yieldFirst = null)
+    private function generatePathNodes(PropFind $propFind, ?array $yieldFirst = null)
     {
         if (null !== $yieldFirst) {
             yield $yieldFirst;

tests/Sabre/CalDAV/SharedCalendarTest.php

@@ -11,7 +11,7 @@ class SharedCalendarTest extends \PHPUnit\Framework\TestCase
 {
     protected $backend;
 
-    public function getInstance(array $props = null)
+    public function getInstance(?array $props = null)
     {
         if (is_null($props)) {
             $props = [

tests/Sabre/DAV/Mock/Collection.php

@@ -34,7 +34,7 @@ class Collection extends DAV\Collection
      * @param string     $name
      * @param Collection $parent
      */
-    public function __construct($name, array $children = [], Collection $parent = null)
+    public function __construct($name, array $children = [], ?Collection $parent = null)
     {
         $this->name = $name;
         foreach ($children as $key => $value) {

tests/Sabre/DAV/Mock/File.php

@@ -30,7 +30,7 @@ class File extends DAV\File
      * @param Collection $parent
      * @param int        $lastModified
      */
-    public function __construct($name, $contents, Collection $parent = null, $lastModified = -1)
+    public function __construct($name, $contents, ?Collection $parent = null, $lastModified = -1)
     {
         $this->name = $name;
         $this->put($contents);

tests/Sabre/DAVACL/PrincipalBackend/Mock.php

@@ -9,7 +9,7 @@ class Mock extends AbstractBackend
     public $groupMembers = [];
     public $principals;
 
-    public function __construct(array $principals = null)
+    public function __construct(?array $principals = null)
     {
         $this->principals = $principals;