mail: Anrede an den tatsächlichen Empfänger binden

Bei Einreichungen über die API geht die Veröffentlichungs- oder
Ablehnungsmail nicht an das API-Konto, sondern an den Pressekontakt der
Meldung oder ersatzweise an die Firmen-Adresse. Die Anrede stammte aber
weiterhin vom Konto — der Empfänger wurde also mit dem Namen des
Einreichers begrüßt, bei connektar mit dem der Agentur.

Adresse und Name werden jetzt gemeinsam bestimmt. Ohne hinterlegten Namen
bleibt er leer und das Template grüßt neutral, statt einen falschen zu raten.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

app/Mail/PressReleasePublished.php

@@ -26,10 +26,18 @@ class PressReleasePublished extends Mailable
      */
     public array $portalLinks;
 
+    /**
+     * Name für die Anrede — bei API-Einlieferungen der Pressekontakt bzw.
+     * die Firma, nie der Kontoname des API-Einlieferers.
+     */
+    public string $recipientName;
+
     public function __construct(
         public readonly User $user,
         public readonly PressRelease $pressRelease,
+        ?string $recipientName = null,
     ) {
+        $this->recipientName = $recipientName ?? $user->name;
         $this->showUrl = $user->canAccessAdmin()
             ? route('admin.press-releases.show', $pressRelease->id)
             : route('me.press-releases.show', $pressRelease->id);

app/Mail/PressReleaseRejected.php

@@ -18,11 +18,19 @@ class PressReleaseRejected extends Mailable
 
     public string $editUrl;
 
+    /**
+     * Name für die Anrede — bei API-Einlieferungen der Pressekontakt bzw.
+     * die Firma, nie der Kontoname des API-Einlieferers.
+     */
+    public string $recipientName;
+
     public function __construct(
         public readonly User $user,
         public readonly PressRelease $pressRelease,
         public readonly ?string $reason = null,
+        ?string $recipientName = null,
     ) {
+        $this->recipientName = $recipientName ?? $user->name;
         $this->editUrl = $user->canAccessAdmin()
             ? route('admin.press-releases.edit', $pressRelease->id)
             : route('me.press-releases.edit', $pressRelease->id);

app/Services/PressRelease/PressReleaseService.php

@@ -669,6 +669,7 @@ class PressReleaseService
         // ist die Quelle des letzten Review-Uebergangs im Status-Log — der
         // Kontotyp ist nicht verlaesslich (connektar ist z. B. 'agency').
         $recipient = $user->email;
+        $recipientName = $user->name;
         $isApiSubmitter = $pressRelease->statusLogs()
             ->where('to_status', PressReleaseStatus::Review->value)
             ->orderByDesc('id')
@@ -676,10 +677,28 @@ class PressReleaseService
 
         if ($isApiSubmitter) {
             // Scope-frei auflösen: der Mail-Versand läuft auch aus Jobs/
-            // Kontexten ohne passenden Portal-Host (PortalScope).
+            // Kontexten ohne passenden Portal-Host (PortalScope). Anrede und
-            $recipient = $pressRelease->contacts()->withoutGlobalScopes()->whereNotNull('email')->value('email')
+            // Adresse gehören zusammen — sobald die Mail an Pressekontakt
-                ?? Company::withoutGlobalScopes()->find($pressRelease->company_id)?->email
+            // oder Firma geht, darf nie der Name des API-Kontos gegrüßt
-                ?: $user->email;
+            // werden. Ohne hinterlegten Namen bleibt $recipientName leer,
+            // das Template grüßt dann neutral.
+            $contact = $pressRelease->contacts()
+                ->withoutGlobalScopes()
+                ->whereNotNull('contacts.email')
+                ->where('contacts.email', '!=', '')
+                ->orderBy('contacts.id')
+                ->first();
+
+            $company = Company::withoutGlobalScopes()->find($pressRelease->company_id);
+            $companyName = trim((string) $company?->name);
+
+            if ($contact) {
+                $recipient = $contact->email;
+                $recipientName = trim($contact->first_name.' '.$contact->last_name) ?: $companyName;
+            } elseif ($company && filled($company->email)) {
+                $recipient = $company->email;
+                $recipientName = $companyName;
+            }
         }
 
         // Opt-out-Gate: Benachrichtigungs-Mails sind je Typ abbestellbar
@@ -701,8 +720,8 @@ class PressReleaseService
         }
 
         $mailable = match ($event) {
-            'published' => new PressReleasePublished($user, $pressRelease),
+            'published' => new PressReleasePublished($user, $pressRelease, $recipientName),
-            'rejected' => new PressReleaseRejected($user, $pressRelease, $reason),
+            'rejected' => new PressReleaseRejected($user, $pressRelease, $reason, $recipientName),
             default => null,
         };
 

resources/views/emails/press-release/published.blade.php

@@ -1,5 +1,5 @@
 <x-emails.layout title="Ihre Pressemitteilung wurde veröffentlicht" preheader="Ihre Pressemitteilung wurde geprüft und veröffentlicht.">
-    <p>Hallo {{ $user->name }},</p>
+    <p>{{ filled($recipientName) ? 'Hallo '.$recipientName.',' : 'Guten Tag,' }}</p>
 
     <p>Ihre Pressemitteilung wurde geprüft und erfolgreich veröffentlicht.</p>
 

resources/views/emails/press-release/rejected.blade.php

@@ -1,5 +1,5 @@
 <x-emails.layout title="Ihre Pressemitteilung wurde abgelehnt" preheader="Ihre eingereichte Pressemitteilung konnte nicht veröffentlicht werden.">
-    <p>Hallo {{ $user->name }},</p>
+    <p>{{ filled($recipientName) ? 'Hallo '.$recipientName.',' : 'Guten Tag,' }}</p>
 
     <p>Ihre eingereichte Pressemitteilung konnte leider nicht veröffentlicht werden.</p>
 
@@ -8,7 +8,7 @@
     @if($reason)
         <div class="meta-danger">
             <strong>Ablehnungsgrund:</strong><br>
-            {{ $reason }}
+            {!! nl2br(e($reason)) !!}
         </div>
     @else
         <p>Bitte überarbeiten Sie den Inhalt und reichen Sie die Pressemitteilung erneut ein.</p>

tests/Feature/PressReleaseWorkflowTest.php

@@ -156,6 +156,23 @@ test('rejected mail uses customer edit url for customer recipients', function ()
     });
 });
 
+test('rejected mail renders multi-line reason with line breaks', function () {
+    /** @var TestCase $this */
+    $author = User::factory()->create(['is_active' => true]);
+    $author->assignRole('customer');
+
+    $pr = PressRelease::factory()->create([
+        'user_id' => $author->id,
+        'status' => PressReleaseStatus::Rejected->value,
+    ]);
+
+    $html = (new PressReleaseRejected($author, $pr, "Erster Punkt fehlt.\nZweiter Punkt fehlt.\n\nBitte überarbeiten."))->render();
+
+    expect($html)
+        ->toContain('Erster Punkt fehlt.<br')
+        ->toContain('Zweiter Punkt fehlt.<br');
+});
+
 test('admin can resubmit cycle creates two audit log entries', function () {
     /** @var TestCase $this */
     $author = User::factory()->create(['is_active' => true]);