explicitly mark nullable typed parameters

Author: sgiehl

Auth/Base.php

@@ -98,7 +98,7 @@ abstract class Base implements Auth
      */
     protected $logger;
 
-    public function __construct(LoggerInterface $logger = null)
+    public function __construct(?LoggerInterface $logger = null)
     {
         $this->logger = $logger ?: StaticContainer::get(LoggerInterface::class);
     }
@@ -290,7 +290,7 @@ protected function getUserForLogin()
         return $this->userForLogin;
     }
 
-    protected function tryFallbackAuth($onlySuperUsers = true, Auth $auth = null)
+    protected function tryFallbackAuth($onlySuperUsers = true, ?Auth $auth = null)
     {
         if (empty($auth)) {
             $auth = new \Piwik\Plugins\Login\Auth();

LdapInterop/UserAccessAttributeParser.php

@@ -110,7 +110,7 @@ class UserAccessAttributeParser
      */
     private $logger;
 
-    public function __construct(LoggerInterface $logger = null)
+    public function __construct(?LoggerInterface $logger = null)
     {
         $this->logger = $logger ?: StaticContainer::get(LoggerInterface::class);
     }

LdapInterop/UserAccessMapper.php

@@ -85,7 +85,7 @@ class UserAccessMapper
     /**
      * Constructor.
      */
-    public function __construct(LoggerInterface $logger = null)
+    public function __construct(?LoggerInterface $logger = null)
     {
         $this->logger = $logger ?: StaticContainer::get(LoggerInterface::class);
     }

LdapInterop/UserMapper.php

@@ -76,7 +76,7 @@ class UserMapper
     /**
      * Constructor.
      */
-    public function __construct(LoggerInterface $logger = null)
+    public function __construct(?LoggerInterface $logger = null)
     {
         $this->logger = $logger ?: StaticContainer::get(LoggerInterface::class);
     }

LdapInterop/UserSynchronizer.php

@@ -108,7 +108,7 @@ class UserSynchronizer
      */
     public static $skipPasswordConfirmation = false;
 
-    public function __construct(LoggerInterface $logger = null)
+    public function __construct(?LoggerInterface $logger = null)
     {
         $this->logger = $logger ?: StaticContainer::get(LoggerInterface::class);
     }

Model/LdapUsers.php

@@ -119,7 +119,7 @@ class LdapUsers
     /**
      * Constructor.
      */
-    public function __construct(LoggerInterface $logger = null)
+    public function __construct(?LoggerInterface $logger = null)
     {
         $this->logger = $logger ?: StaticContainer::get(LoggerInterface::class);
     }