Fix: kein Datenverlust beim 'Buchung auswaehlen' aus dem Submit-Modal
Bug: Im Submit-Modal navigierte 'Buchung auswaehlen' per wire:navigate direkt
zur Buchungsseite, OHNE vorher zu speichern -> im Erstellen ging die ganze PM
verloren, im Bearbeiten die ungespeicherten Aenderungen.
- Submit-Modal: neuer Prop booking-action; ist er gesetzt, speichert der Button
ueber eine Livewire-Aktion zuerst und navigiert erst danach
('Speichern & Buchung auswaehlen'). Fallback-href nur dort, wo bereits
gespeichert ist (Detailansicht).
- create/edit: Persistenz in persistDraft()/persistEdit() extrahiert;
saveDraftAndChooseBooking() speichert den Entwurf bzw. die Aenderungen und
leitet erst dann zur Buchung (Edit reklassifiziert bei Inhaltsaenderung).
- Beide Submit-Modals mit booking-action verdrahtet.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This commit is contained in:
parent
44b676116e
commit
764d56ebd0
4 changed files with 208 additions and 10 deletions
|
|
@ -1,6 +1,7 @@
|
|||
@props([
|
||||
'name' => 'confirm-submit-review',
|
||||
'action',
|
||||
'bookingAction' => null,
|
||||
'confirmLabel' => null,
|
||||
'quotaTotal' => null,
|
||||
'quotaRemaining' => null,
|
||||
|
|
@ -18,6 +19,11 @@
|
|||
Submit-Gate (Decision-Update §5.1): Ohne aktive Buchung zeigt das Modal
|
||||
statt des Prüf-Flows einen Buchungs-Hinweis — der Button konvertiert,
|
||||
er verschwindet nicht. Serverseitig sichert submitForReview() das Gate ab.
|
||||
|
||||
`booking-action` (optional): Livewire-Methode, die den Entwurf zuerst
|
||||
speichert und dann zur Buchungsseite leitet. ZWINGEND in Erstellen/Bearbeiten
|
||||
setzen — sonst ginge der erfasste Inhalt beim Sprung verloren. Fehlt sie
|
||||
(z. B. Detailansicht, dort ist bereits gespeichert), wird direkt verlinkt.
|
||||
--}}
|
||||
@php($bookingRequired = ! (auth()->user()?->hasActiveBooking() ?? true))
|
||||
|
||||
|
|
@ -31,7 +37,7 @@
|
|||
|
||||
<div class="rounded-[6px] border border-[color:var(--color-bg-rule)] bg-[color:var(--color-bg-elev)] p-4 text-[12.5px] leading-[1.6] text-[color:var(--color-ink-2)]">
|
||||
<p class="m-0 mb-2">
|
||||
{{ __('Zum Einreichen einer Pressemitteilung wird eine aktive Buchung benötigt. Ihre Entwürfe bleiben gespeichert und können jederzeit weiter bearbeitet werden.') }}
|
||||
{{ __('Zum Einreichen einer Pressemitteilung wird eine aktive Buchung benötigt. Ihr Entwurf wird dabei gespeichert und bleibt jederzeit bearbeitbar.') }}
|
||||
</p>
|
||||
<p class="m-0 text-[11.5px] text-[color:var(--color-ink-3)]">
|
||||
{{ __('Nach der Buchung reichen Sie die Pressemitteilung mit einem Klick zur Prüfung ein.') }}
|
||||
|
|
@ -42,9 +48,17 @@
|
|||
<flux:modal.close>
|
||||
<flux:button variant="filled">{{ __('Später') }}</flux:button>
|
||||
</flux:modal.close>
|
||||
<flux:button variant="primary" :href="route('me.bookings.index')" wire:navigate>
|
||||
{{ __('Buchung auswählen') }}
|
||||
</flux:button>
|
||||
@if ($bookingAction)
|
||||
{{-- Entwurf zuerst speichern, dann zur Buchung — kein Datenverlust. --}}
|
||||
<flux:button variant="primary" icon="credit-card"
|
||||
wire:click="{{ $bookingAction }}" wire:loading.attr="disabled">
|
||||
{{ __('Speichern & Buchung auswählen') }}
|
||||
</flux:button>
|
||||
@else
|
||||
<flux:button variant="primary" :href="route('me.bookings.index')" wire:navigate>
|
||||
{{ __('Buchung auswählen') }}
|
||||
</flux:button>
|
||||
@endif
|
||||
</div>
|
||||
</div>
|
||||
@else
|
||||
|
|
|
|||
|
|
@ -400,7 +400,16 @@ new #[Layout('components.layouts.app'), Title('Neue Pressemitteilung')] class ex
|
|||
$this->redirect(route('me.press-releases.edit', $pr->id), navigate: true);
|
||||
}
|
||||
|
||||
public function save(string $submitStatus = 'draft'): void
|
||||
/**
|
||||
* Validiert das Formular und legt die Pressemitteilung als Entwurf an.
|
||||
* Gibt den persistierten Entwurf zurück oder null, wenn Firma/Kontakt nicht
|
||||
* auflösbar sind (Validierungsfehler wirft ValidationException).
|
||||
*
|
||||
* Wichtig: Dies ist der einzige Persistenz-Pfad. Jede Aktion (Speichern,
|
||||
* Einreichen, „Buchung auswählen") MUSS zuerst hierüber speichern, bevor
|
||||
* navigiert wird — sonst geht der erfasste Inhalt verloren.
|
||||
*/
|
||||
private function persistDraft(): ?PressRelease
|
||||
{
|
||||
$this->syncScheduledAt();
|
||||
$this->useEmbargo = false;
|
||||
|
|
@ -421,7 +430,7 @@ new #[Layout('components.layouts.app'), Title('Neue Pressemitteilung')] class ex
|
|||
$this->addError('companyId', __('Die gewählte Firma ist nicht Ihrem Account zugeordnet.'));
|
||||
$this->notifyValidationError();
|
||||
|
||||
return;
|
||||
return null;
|
||||
}
|
||||
|
||||
$contact = null;
|
||||
|
|
@ -433,7 +442,7 @@ new #[Layout('components.layouts.app'), Title('Neue Pressemitteilung')] class ex
|
|||
$this->addError('contactId', __('Der gewählte Pressekontakt gehört nicht zu dieser Firma.'));
|
||||
$this->notifyValidationError();
|
||||
|
||||
return;
|
||||
return null;
|
||||
}
|
||||
}
|
||||
|
||||
|
|
@ -457,6 +466,39 @@ new #[Layout('components.layouts.app'), Title('Neue Pressemitteilung')] class ex
|
|||
$pr->contacts()->sync([$contact->id]);
|
||||
}
|
||||
|
||||
return $pr;
|
||||
}
|
||||
|
||||
/**
|
||||
* Speichert den Entwurf und leitet anschließend zur Buchungsseite — wird
|
||||
* vom Submit-Modal genutzt, wenn keine Buchung vorliegt. So bleibt der
|
||||
* erfasste Inhalt erhalten (kein Sprung ohne Speichern).
|
||||
*/
|
||||
public function saveDraftAndChooseBooking(): void
|
||||
{
|
||||
$pr = $this->persistDraft();
|
||||
|
||||
if (! $pr) {
|
||||
return;
|
||||
}
|
||||
|
||||
Flux::toast(
|
||||
heading: __('Entwurf gespeichert'),
|
||||
text: __('Ihr Entwurf ist gesichert. Wählen Sie eine Buchung, um ihn anschließend einzureichen.'),
|
||||
variant: 'success',
|
||||
);
|
||||
|
||||
$this->redirect(route('me.bookings.index'), navigate: true);
|
||||
}
|
||||
|
||||
public function save(string $submitStatus = 'draft'): void
|
||||
{
|
||||
$pr = $this->persistDraft();
|
||||
|
||||
if (! $pr) {
|
||||
return;
|
||||
}
|
||||
|
||||
if ($submitStatus === 'review') {
|
||||
$this->authorize('submitForReview', $pr);
|
||||
|
||||
|
|
@ -1374,6 +1416,7 @@ new #[Layout('components.layouts.app'), Title('Neue Pressemitteilung')] class ex
|
|||
<x-press-release-submit-modal
|
||||
name="confirm-submit-review"
|
||||
action="save('review')"
|
||||
booking-action="saveDraftAndChooseBooking"
|
||||
:confirm-label="__('Zur Prüfung senden')"
|
||||
:quota-total="$quotaTotal"
|
||||
:quota-remaining="$quotaRemaining" />
|
||||
|
|
|
|||
|
|
@ -368,7 +368,16 @@ new #[Layout('components.layouts.app'), Title('Pressemitteilung bearbeiten')] cl
|
|||
}
|
||||
}
|
||||
|
||||
public function save(bool $submitAfterSave = false): void
|
||||
/**
|
||||
* Validiert und speichert die Bearbeitung. Gibt [PressRelease, contentChanged]
|
||||
* zurück oder null bei Firma-/Kontakt-Fehlern (Validierung wirft).
|
||||
*
|
||||
* Wichtig: einziger Persistenz-Pfad. Jede Aktion (Speichern, Einreichen,
|
||||
* „Buchung auswählen") MUSS zuerst hierüber speichern, bevor navigiert wird.
|
||||
*
|
||||
* @return array{0: \App\Models\PressRelease, 1: bool}|null
|
||||
*/
|
||||
private function persistEdit(): ?array
|
||||
{
|
||||
$this->syncScheduledAt();
|
||||
$this->useEmbargo = false;
|
||||
|
|
@ -391,7 +400,7 @@ new #[Layout('components.layouts.app'), Title('Pressemitteilung bearbeiten')] cl
|
|||
$this->addError('companyId', __('Die gewählte Firma ist nicht Ihrem Account zugeordnet.'));
|
||||
$this->notifyValidationError();
|
||||
|
||||
return;
|
||||
return null;
|
||||
}
|
||||
|
||||
$contact = null;
|
||||
|
|
@ -403,7 +412,7 @@ new #[Layout('components.layouts.app'), Title('Pressemitteilung bearbeiten')] cl
|
|||
$this->addError('contactId', __('Der gewählte Pressekontakt gehört nicht zu dieser Firma.'));
|
||||
$this->notifyValidationError();
|
||||
|
||||
return;
|
||||
return null;
|
||||
}
|
||||
}
|
||||
|
||||
|
|
@ -435,6 +444,50 @@ new #[Layout('components.layouts.app'), Title('Pressemitteilung bearbeiten')] cl
|
|||
|
||||
$pr->contacts()->sync($contact ? [$contact->id] : []);
|
||||
|
||||
return [$pr, $contentChanged];
|
||||
}
|
||||
|
||||
/**
|
||||
* Speichert die Bearbeitung und leitet zur Buchungsseite — wird vom
|
||||
* Submit-Modal genutzt, wenn keine Buchung vorliegt. So gehen die
|
||||
* erfassten Änderungen nicht verloren (kein Sprung ohne Speichern).
|
||||
*/
|
||||
public function saveDraftAndChooseBooking(): void
|
||||
{
|
||||
$result = $this->persistEdit();
|
||||
|
||||
if (! $result) {
|
||||
return;
|
||||
}
|
||||
|
||||
[$pr, $contentChanged] = $result;
|
||||
|
||||
if ($contentChanged) {
|
||||
$service = app(PressReleaseService::class);
|
||||
$fresh = $pr->fresh();
|
||||
$service->reclassifyIfClassified($fresh);
|
||||
$service->rescoreIfScored($fresh);
|
||||
}
|
||||
|
||||
Flux::toast(
|
||||
heading: __('Gespeichert'),
|
||||
text: __('Ihre Änderungen sind gesichert. Wählen Sie eine Buchung, um die PM anschließend einzureichen.'),
|
||||
variant: 'success',
|
||||
);
|
||||
|
||||
$this->redirect(route('me.bookings.index'), navigate: true);
|
||||
}
|
||||
|
||||
public function save(bool $submitAfterSave = false): void
|
||||
{
|
||||
$result = $this->persistEdit();
|
||||
|
||||
if (! $result) {
|
||||
return;
|
||||
}
|
||||
|
||||
[$pr, $contentChanged] = $result;
|
||||
|
||||
if ($submitAfterSave) {
|
||||
$this->authorize('submitForReview', $pr);
|
||||
|
||||
|
|
@ -1332,6 +1385,7 @@ new #[Layout('components.layouts.app'), Title('Pressemitteilung bearbeiten')] cl
|
|||
<x-press-release-submit-modal
|
||||
name="confirm-submit-review"
|
||||
action="saveAndSubmit"
|
||||
booking-action="saveDraftAndChooseBooking"
|
||||
:confirm-label="$currentStatus === 'rejected' ? __('Speichern & erneut einreichen') : __('Speichern & einreichen')"
|
||||
:quota-total="$quotaTotal"
|
||||
:quota-remaining="$quotaRemaining" />
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue