Merge pull request #475 from stof/typed_properties

Migrate to typed properties

Author: stof

src/DependencyInjection/StofDoctrineExtensionsExtension.php

@@ -82,9 +82,9 @@ class StofDoctrineExtensionsExtension extends Extension
     );
 
     /** @var list<string> */
-    private $entityManagers   = array();
+    private array $entityManagers = array();
     /** @var list<string> */
-    private $documentManagers = array();
+    private array $documentManagers = array();
 
     /**
      * @return void

src/EventListener/BlameListener.php

@@ -18,12 +18,9 @@
  */
 class BlameListener implements EventSubscriberInterface
 {
-    /** @var AuthorizationCheckerInterface|null */
-    private $authorizationChecker;
-    /** @var TokenStorageInterface|null */
-    private $tokenStorage;
-    /** @var BlameableListener */
-    private $blameableListener;
+    private ?AuthorizationCheckerInterface $authorizationChecker;
+    private ?TokenStorageInterface $tokenStorage;
+    private BlameableListener $blameableListener;
 
     public function __construct(BlameableListener $blameableListener, TokenStorageInterface $tokenStorage = null, AuthorizationCheckerInterface $authorizationChecker = null)
     {

src/EventListener/IpTraceListener.php

@@ -13,8 +13,7 @@
  */
 final class IpTraceListener implements EventSubscriberInterface
 {
-    /** @var IpTraceableListener */
-    private $ipTraceableListener;
+    private IpTraceableListener $ipTraceableListener;
 
     public function __construct(IpTraceableListener $ipTraceableListener)
     {

src/EventListener/LocaleListener.php

@@ -14,8 +14,7 @@
  */
 class LocaleListener implements EventSubscriberInterface
 {
-    /** @var TranslatableListener */
-    private $translatableListener;
+    private TranslatableListener $translatableListener;
 
     public function __construct(TranslatableListener $translatableListener)
     {

src/EventListener/LoggerListener.php

@@ -20,12 +20,10 @@
  */
 class LoggerListener implements EventSubscriberInterface
 {
-    /** @var AuthorizationCheckerInterface|null */
-    private $authorizationChecker;
-    /** @var TokenStorageInterface|null */
-    private $tokenStorage;
+    private ?AuthorizationCheckerInterface $authorizationChecker;
+    private ?TokenStorageInterface  $tokenStorage;
     /** @var LoggableListener<T> */
-    private $loggableListener;
+    private LoggableListener $loggableListener;
 
     /**
      * @param LoggableListener<T> $loggableListener

src/Uploadable/UploadableManager.php

@@ -8,16 +8,14 @@
 
 class UploadableManager
 {
-    /** @var \Gedmo\Uploadable\UploadableListener */
-    private $listener;
+    private UploadableListener $listener;
     /** @var class-string<FileInfoInterface> */
-    private $fileInfoClass;
+    private string $fileInfoClass;
 
     /**
-     * @param UploadableListener              $listener
      * @param class-string<FileInfoInterface> $fileInfoClass
      */
-    public function __construct(UploadableListener $listener, $fileInfoClass)
+    public function __construct(UploadableListener $listener, string $fileInfoClass)
     {
         $this->listener = $listener;
         $this->fileInfoClass = $fileInfoClass;
@@ -51,5 +49,4 @@ public function getUploadableListener()
     {
         return $this->listener;
     }
-
 }

src/Uploadable/UploadedFileInfo.php

@@ -7,8 +7,7 @@
 
 class UploadedFileInfo implements FileInfoInterface
 {
-    /** @var UploadedFile */
-    private $uploadedFile;
+    private UploadedFile $uploadedFile;
 
     public function __construct(UploadedFile $uploadedFile)
     {

src/Uploadable/ValidatorConfigurator.php

@@ -9,13 +9,9 @@
  */
 class ValidatorConfigurator
 {
-    /** @var bool */
-    private $validateWritableDirectory;
+    private bool $validateWritableDirectory;
 
-    /**
-     * @param bool $validateWritableDirectory
-     */
-    public function __construct($validateWritableDirectory)
+    public function __construct(bool $validateWritableDirectory)
     {
         $this->validateWritableDirectory = $validateWritableDirectory;
     }