Fixed CI (2.x)

composer.json

@@ -46,7 +46,7 @@
     "test": "php -d pcov.enabled=1 ./vendor/bin/phpunit --colors=always",
     "csfixer": "tools/php-cs-fixer/vendor/bin/php-cs-fixer fix --config=.php-cs-fixer.php --using-cache=no --verbose --ansi",
     "cs-check": "tools/php-cs-fixer/vendor/bin/php-cs-fixer fix --config=.php-cs-fixer.php --using-cache=no --verbose --ansi --dry-run",
-    "phpstan": "tools/phpstan/vendor/bin/phpstan --ansi analyse src",
+    "phpstan": "tools/phpstan/vendor/bin/phpstan --ansi analyse src --memory-limit=1G",
     "rector": "tools/rector/vendor/bin/rector process src",
     "update-tools": "composer update --working-dir=tools/php-cs-fixer ; composer update --working-dir=tools/phpstan ; composer update --working-dir=tools/rector"
   },

src/Event/LifecycleEvent.php

@@ -7,6 +7,4 @@
 /**
  * @see \DH\Auditor\Tests\Event\LifecycleEventTest
  */
-final class LifecycleEvent extends AuditEvent
-{
-}
+final class LifecycleEvent extends AuditEvent {}

src/Exception/AccessDeniedException.php

@@ -6,6 +6,4 @@
 
 use Exception;
 
-class AccessDeniedException extends Exception
-{
-}
+class AccessDeniedException extends Exception {}

src/Exception/InvalidArgumentException.php

@@ -6,6 +6,4 @@
 
 use Exception;
 
-class InvalidArgumentException extends Exception
-{
-}
+class InvalidArgumentException extends Exception {}

src/Exception/MappingException.php

@@ -6,6 +6,4 @@
 
 use Exception;
 
-class MappingException extends Exception
-{
-}
+class MappingException extends Exception {}

src/Exception/ProviderException.php

@@ -6,6 +6,4 @@
 
 use Exception;
 
-class ProviderException extends Exception
-{
-}
+class ProviderException extends Exception {}

src/Model/Entry.php

@@ -106,8 +106,6 @@ public function getUserFirewall(): ?string
 
     /**
      * Get the value of ip.
-     *
-     * @return string
      */
     public function getIp(): ?string
     {
@@ -124,8 +122,6 @@ public function getCreatedAt(): string
 
     /**
      * Get diff values.
-     *
-     * @return array
      */
     public function getDiffs(bool $includeMedadata = false): ?array
     {

src/Provider/ConfigurationInterface.php

@@ -4,6 +4,4 @@
 
 namespace DH\Auditor\Provider;
 
-interface ConfigurationInterface
-{
-}
+interface ConfigurationInterface {}

src/Provider/Doctrine/Auditing/Annotation/Auditable.php

@@ -10,9 +10,7 @@
 
 /**
  * @Annotation
- *
  * @NamedArgumentConstructor
- *
  * @Target("CLASS")
  *
  * @Attributes({

src/Provider/Doctrine/Auditing/Annotation/Ignore.php

@@ -9,10 +9,7 @@
 
 /**
  * @Annotation
- *
  * @Target("PROPERTY")
  */
 #[Attribute(Attribute::TARGET_PROPERTY)]
-final class Ignore
-{
-}
+final class Ignore {}

src/Provider/Doctrine/Auditing/Annotation/Security.php

@@ -11,9 +11,7 @@
 
 /**
  * @Annotation
- *
  * @NamedArgumentConstructor
- *
  * @Target("CLASS")
  *
  * @Attributes({

src/Provider/Doctrine/Auditing/Logger/Logger.php

@@ -30,7 +30,5 @@ public function startQuery($sql, ?array $params = null, ?array $types = null): v
         }
     }
 
-    public function stopQuery(): void
-    {
-    }
+    public function stopQuery(): void {}
 }

src/Provider/Doctrine/Persistence/Exception/UpdateException.php

@@ -6,6 +6,4 @@
 
 use Exception;
 
-class UpdateException extends Exception
-{
-}
+class UpdateException extends Exception {}

src/Provider/Doctrine/Service/AuditingService.php

@@ -6,6 +6,4 @@
 
 use DH\Auditor\Provider\Service\AuditingServiceInterface;
 
-class AuditingService extends DoctrineService implements AuditingServiceInterface
-{
-}
+class AuditingService extends DoctrineService implements AuditingServiceInterface {}

src/Provider/Doctrine/Service/StorageService.php

@@ -6,6 +6,4 @@
 
 use DH\Auditor\Provider\Service\StorageServiceInterface;
 
-class StorageService extends DoctrineService implements StorageServiceInterface
-{
-}
+class StorageService extends DoctrineService implements StorageServiceInterface {}

src/Provider/Service/AuditingServiceInterface.php

@@ -4,6 +4,4 @@
 
 namespace DH\Auditor\Provider\Service;
 
-interface AuditingServiceInterface extends ServiceInterface
-{
-}
+interface AuditingServiceInterface extends ServiceInterface {}

src/Provider/Service/StorageServiceInterface.php

@@ -4,6 +4,4 @@
 
 namespace DH\Auditor\Provider\Service;
 
-interface StorageServiceInterface extends ServiceInterface
-{
-}
+interface StorageServiceInterface extends ServiceInterface {}

tests/Fixtures/Provider/AuditNoStorageProvider.php

@@ -10,13 +10,9 @@
 
 class AuditNoStorageProvider extends AbstractProvider
 {
-    public function getConfiguration(): ConfigurationInterface
-    {
-    }
+    public function getConfiguration(): ConfigurationInterface {}
 
-    public function persist(LifecycleEvent $event): void
-    {
-    }
+    public function persist(LifecycleEvent $event): void {}
 
     public function supportsStorage(): bool
     {
@@ -28,11 +24,7 @@ public function supportsAuditing(): bool
         return true;
     }
 
-    public function getStorageServices(): array
-    {
-    }
+    public function getStorageServices(): array {}
 
-    public function getAuditingServices(): array
-    {
-    }
+    public function getAuditingServices(): array {}
 }

tests/Fixtures/Provider/NoStorageNoAuditProvider.php

@@ -10,13 +10,9 @@
 
 class NoStorageNoAuditProvider extends AbstractProvider
 {
-    public function getConfiguration(): ConfigurationInterface
-    {
-    }
+    public function getConfiguration(): ConfigurationInterface {}
 
-    public function persist(LifecycleEvent $event): void
-    {
-    }
+    public function persist(LifecycleEvent $event): void {}
 
     public function supportsStorage(): bool
     {
@@ -28,11 +24,7 @@ public function supportsAuditing(): bool
         return false;
     }
 
-    public function getStorageServices(): array
-    {
-    }
+    public function getStorageServices(): array {}
 
-    public function getAuditingServices(): array
-    {
-    }
+    public function getAuditingServices(): array {}
 }

tests/Fixtures/Provider/StorageAndAuditProvider.php

@@ -10,13 +10,9 @@
 
 class StorageAndAuditProvider extends AbstractProvider
 {
-    public function getConfiguration(): ConfigurationInterface
-    {
-    }
+    public function getConfiguration(): ConfigurationInterface {}
 
-    public function persist(LifecycleEvent $event): void
-    {
-    }
+    public function persist(LifecycleEvent $event): void {}
 
     public function supportsStorage(): bool
     {
@@ -28,11 +24,7 @@ public function supportsAuditing(): bool
         return true;
     }
 
-    public function getStorageServices(): array
-    {
-    }
+    public function getStorageServices(): array {}
 
-    public function getAuditingServices(): array
-    {
-    }
+    public function getAuditingServices(): array {}
 }

tests/Fixtures/Provider/StorageNoAuditProvider.php

@@ -10,13 +10,9 @@
 
 class StorageNoAuditProvider extends AbstractProvider
 {
-    public function getConfiguration(): ConfigurationInterface
-    {
-    }
+    public function getConfiguration(): ConfigurationInterface {}
 
-    public function persist(LifecycleEvent $event): void
-    {
-    }
+    public function persist(LifecycleEvent $event): void {}
 
     public function supportsStorage(): bool
     {
@@ -28,11 +24,7 @@ public function supportsAuditing(): bool
         return false;
     }
 
-    public function getStorageServices(): array
-    {
-    }
+    public function getStorageServices(): array {}
 
-    public function getAuditingServices(): array
-    {
-    }
+    public function getAuditingServices(): array {}
 }

tests/Model/TransactionTest.php

@@ -14,13 +14,9 @@
  */
 final class TransactionTest extends TestCase
 {
-    protected function setUp(): void
-    {
-    }
+    protected function setUp(): void {}
 
-    protected function tearDown(): void
-    {
-    }
+    protected function tearDown(): void {}
 
     public function testGetTransactionHash(): void
     {

tests/Provider/Doctrine/Fixtures/Entity/Inheritance/Joined/Cat.php

@@ -12,6 +12,4 @@
  * @ORM\Table(name="cat", schema="auditor")
  */
 #[ORM\Entity, ORM\Table(name: 'cat', schema: 'auditor')]
-class Cat extends Animal
-{
-}
+class Cat extends Animal {}

tests/Provider/Doctrine/Fixtures/Entity/Inheritance/Joined/Dog.php

@@ -12,6 +12,4 @@
  * @ORM\Table(name="dog", schema="auditor")
  */
 #[ORM\Entity, ORM\Table(name: 'dog', schema: 'auditor')]
-class Dog extends Animal
-{
-}
+class Dog extends Animal {}

tests/Provider/Doctrine/Fixtures/Entity/Inheritance/SingleTable/Bike.php

@@ -10,6 +10,4 @@
  * @ORM\Entity
  */
 #[ORM\Entity]
-class Bike extends Vehicle
-{
-}
+class Bike extends Vehicle {}

tests/Provider/Doctrine/Fixtures/Entity/Inheritance/SingleTable/Car.php

@@ -10,6 +10,4 @@
  * @ORM\Entity
  */
 #[ORM\Entity]
-class Car extends Vehicle
-{
-}
+class Car extends Vehicle {}

tests/Provider/Doctrine/Fixtures/Entity/Standard/Blog/Author.php

@@ -78,8 +78,6 @@ public function setId(int $id): self
 
     /**
      * Get the value of id.
-     *
-     * @return int
      */
     public function getId(): ?int
     {
@@ -98,8 +96,6 @@ public function setFullname(string $fullname): self
 
     /**
      * Get the value of fullname.
-     *
-     * @return string
      */
     public function getFullname(): ?string
     {
@@ -118,8 +114,6 @@ public function setEmail(string $email): self
 
     /**
      * Get the value of email.
-     *
-     * @return string
      */
     public function getEmail(): ?string
     {

tests/Provider/Doctrine/Fixtures/Entity/Standard/Blog/Comment.php

@@ -67,9 +67,7 @@ class Comment
     #[ORM\JoinColumn(name: 'post_id', referencedColumnName: 'id', nullable: true)]
     protected $post;
 
-    public function __construct()
-    {
-    }
+    public function __construct() {}
 
     public function __sleep()
     {
@@ -88,8 +86,6 @@ public function setId(int $id): self
 
     /**
      * Get the value of id.
-     *
-     * @return int
      */
     public function getId(): ?int
     {
@@ -108,8 +104,6 @@ public function setBody(string $body): self
 
     /**
      * Get the value of body.
-     *
-     * @return string
      */
     public function getBody(): ?string
     {
@@ -128,8 +122,6 @@ public function setAuthor(string $author): self
 
     /**
      * Get the value of author.
-     *
-     * @return string
      */
     public function getAuthor(): ?string
     {
@@ -166,8 +158,6 @@ public function setPostId(int $post_id): self
 
     /**
      * Get the value of post_id.
-     *
-     * @return int
      */
     public function getPostId(): ?int
     {