Fixing replaceSignatureTokens(): Argument #1 ($signature) must be of …

…type string, null given
galvani · Jan 29, 2024 · 3639945 · 3639945
1 parent 829c975
commit 3639945
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/app/bundles/EmailBundle/Helper/FromEmailHelper.php b/app/bundles/EmailBundle/Helper/FromEmailHelper.php
@@ -125,17 +125,15 @@ public function getSignature(): string
             return '';
         }
 
-        $owner = $this->lastOwner;
-
-        return $this->replaceSignatureTokens($owner['signature'], $owner);
+        return $this->replaceSignatureTokens($this->lastOwner);
     }
 
     /**
      * @param mixed[] $owner
      */
-    private function replaceSignatureTokens(string $signature, array $owner): string
+    private function replaceSignatureTokens(array $owner): string
     {
-        $signature = nl2br($signature);
+        $signature = nl2br($owner['signature'] ?? '');
         $signature = str_replace('|FROM_NAME|', $owner['first_name'].' '.$owner['last_name'], $signature);
 
         foreach ($owner as $key => $value) {