Ensure setterMethod is always a string

Author: fwolfsjaeger

src/Mapping/Annotation/Blameable.php

@@ -34,7 +34,7 @@ final class Blameable implements GedmoAnnotation
     public $field;
     /** @var mixed */
     public $value;
-    /** @var string|null */
+    /** @var string */
     public $setterMethod;
 
     /**
@@ -47,7 +47,7 @@ public function __construct(
         string $on = 'update',
         $field = null,
         $value = null,
-        ?string $setterMethod = null
+        string $setterMethod = ''
     ) {
         if ([] !== $data) {
             Deprecation::trigger(

src/Mapping/Annotation/IpTraceable.php

@@ -35,7 +35,7 @@ final class IpTraceable implements GedmoAnnotation
     public $field;
     /** @var mixed */
     public $value;
-    /** @var string|null */
+    /** @var string */
     public $setterMethod;
 
     /**
@@ -48,7 +48,7 @@ public function __construct(
         string $on = 'update',
         $field = null,
         $value = null,
-        ?string $setterMethod = null
+        string $setterMethod = ''
     ) {
         if ([] !== $data) {
             Deprecation::trigger(

src/Mapping/Annotation/SoftDeleteable.php

@@ -35,14 +35,19 @@ final class SoftDeleteable implements GedmoAnnotation
 
     public bool $hardDelete = true;
 
-    /** @var string|null */
+    /** @var string */
     public $setterMethod;
 
     /**
      * @param array<string, mixed> $data
      */
-    public function __construct(array $data = [], string $fieldName = 'deletedAt', bool $timeAware = false, bool $hardDelete = true, ?string $setterMethod = null)
-    {
+    public function __construct(
+        array $data = [],
+        string $fieldName = 'deletedAt',
+        bool $timeAware = false,
+        bool $hardDelete = true,
+        string $setterMethod = ''
+    ) {
         if ([] !== $data) {
             Deprecation::trigger(
                 'gedmo/doctrine-extensions',

src/Mapping/Annotation/Timestampable.php

@@ -34,7 +34,7 @@ final class Timestampable implements GedmoAnnotation
     public $field;
     /** @var mixed */
     public $value;
-    /** @var string|null */
+    /** @var string */
     public $setterMethod;
 
     /**
@@ -47,7 +47,7 @@ public function __construct(
         string $on = 'update',
         $field = null,
         $value = null,
-        ?string $setterMethod = null
+        string $setterMethod = ''
     ) {
         if ([] !== $data) {
             Deprecation::trigger(

src/Mapping/Driver/AbstractAnnotationDriver.php

@@ -139,9 +139,9 @@ protected function getRelatedClassName($metadata, $name)
     /**
      * Set the setter method for the given field.
      */
-    protected function setSetterMethod(string $field, ?string $method, array &$config): void
+    protected function setSetterMethod(string $field, string $method, array &$config): void
     {
-        if (empty($method)) {
+        if ('' === $method) {
             return;
         }