web: Entfernung mit 410 Gone als Alternative zur Löschung
Rechtlich oder redaktionell veranlasste Entfernungen brauchten bisher die Löschung — danach antwortet die URL mit 404, als hätte es die Meldung nie gegeben. Newsroom und Pressemitteilung lassen sich jetzt stattdessen mit gone_at markieren: Der Inhalt verschwindet aus allen öffentlichen Listen, die bestehende URL liefert aber einen Tombstone mit Status 410. Die vollständige Löschung bleibt unverändert daneben stehen und erzeugt weiterhin 404. Umkehrbar heißt hier wirklich umkehrbar. markGone() überschreibt zwei Felder — companies.is_active (damit der Newsroom ohne Sonderbehandlung aus Newsroom-Liste und Sitemap fällt, die beide darauf filtern) und press_releases.no_export. Beide Vorzustände werden gesichert und beim Aufheben zurückgestellt, sonst stünde eine schon vorher deaktivierte Firma danach öffentlich in der Übersicht und eine Mitteilung dauerhaft als "kein Export" da, ohne dass das je jemand gewählt hätte. Der Auslieferungs-Cache kennt keine Tags, und die Keys der Listenseiten sind über Portal, Sprache, Seite und Filter verstreut. Eine Entfernung erhöht deshalb eine Generations-Nummer im Cache-Key und verwirft damit alle Seiten-Caches auf einen Schlag — grob, aber Entfernungen sind selten und dürfen nicht bis zu einer Stunde nachhängen. Der Zähler liegt bewusst außerhalb des Namensraums, den er invalidiert, und wird zwischen zwei Queue-Jobs neu gelesen: Ein Worker lebt über beliebig viele Jobs und arbeitete sonst dauerhaft mit dem Stand seines ersten Zugriffs. Beim Newsroom läuft die Tombstone-Prüfung bewusst VOR dem Seiten-Cache — eine Entfernung muss sofort greifen, auch wenn für den Slug noch ein Stale-Eintrag liegt. Kaskadiert man die Entfernung auf die Meldungen einer Firma, wird der Cache einmal am Ende verworfen statt einmal je Meldung; sonst zwänge eine Bestandsfirma mit dreistellig vielen Meldungen die gesamte öffentliche Seite ebenso oft in den Neuaufbau. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This commit is contained in:
parent
6a34ab48ee
commit
0c8e94a6ff
24 changed files with 1150 additions and 57 deletions
|
|
@ -27,6 +27,7 @@ enum ActivityType: string
|
|||
case SyncLock = 'sync_lock';
|
||||
case Anonymization = 'anonymization';
|
||||
case Correction = 'correction';
|
||||
case ContentRemoval = 'content_removal';
|
||||
|
||||
public function label(): string
|
||||
{
|
||||
|
|
@ -49,6 +50,7 @@ enum ActivityType: string
|
|||
self::SyncLock => 'Re-Sync-Sperre',
|
||||
self::Anonymization => 'Anonymisierung',
|
||||
self::Correction => 'Korrektur',
|
||||
self::ContentRemoval => 'Entfernung (410)',
|
||||
};
|
||||
}
|
||||
|
||||
|
|
@ -71,6 +73,7 @@ enum ActivityType: string
|
|||
self::SyncLock => 'red',
|
||||
self::Anonymization => 'red',
|
||||
self::Correction => 'blue',
|
||||
self::ContentRemoval => 'red',
|
||||
};
|
||||
}
|
||||
}
|
||||
|
|
|
|||
|
|
@ -6,13 +6,22 @@ use App\Http\Controllers\Controller;
|
|||
use App\Services\Web\NewsroomDetailPageData;
|
||||
use App\Support\WebPortalContext;
|
||||
use Illuminate\Contracts\View\View;
|
||||
use Illuminate\Http\Response;
|
||||
|
||||
class NewsroomDetailController extends Controller
|
||||
{
|
||||
// $edition stammt aus dem Gruppen-Prefix {edition} und steht positionsbedingt
|
||||
// vor {slug}; es wird hier nicht gebraucht, muss aber in der Signatur stehen.
|
||||
public function __invoke(string $edition, string $slug, NewsroomDetailPageData $pageData): View
|
||||
public function __invoke(string $edition, string $slug, NewsroomDetailPageData $pageData): View|Response
|
||||
{
|
||||
return view('web.newsroom', $pageData->build(WebPortalContext::fromRequest(), $slug));
|
||||
$data = $pageData->build(WebPortalContext::fromRequest(), $slug);
|
||||
|
||||
// Bewusst entfernter Newsroom → Tombstone mit 410 Gone (ohne Inhalte).
|
||||
// Vollständig gelöschte Firmen bleiben bei 404.
|
||||
if (isset($data['tombstone'])) {
|
||||
return response()->view('web.newsroom-tombstone', [], 410);
|
||||
}
|
||||
|
||||
return view('web.newsroom', $data);
|
||||
}
|
||||
}
|
||||
|
|
|
|||
|
|
@ -16,9 +16,11 @@ class ReleaseDetailController extends Controller
|
|||
{
|
||||
$data = $pageData->build(WebPortalContext::fromRequest(), $slug);
|
||||
|
||||
// T12: depublizierte PM → Tombstone mit 410 Gone (ohne Inhalte).
|
||||
// Bewusst entfernte PM → Tombstone mit 410 Gone (ohne Inhalte).
|
||||
// 'depublished' = Depublikation auf Wunsch des Absenders (T12),
|
||||
// 'removed' = redaktionelle/rechtliche Entfernung.
|
||||
if (isset($data['tombstone'])) {
|
||||
return response()->view('web.release-tombstone', [], 410);
|
||||
return response()->view('web.release-tombstone', ['reason' => $data['tombstone']], 410);
|
||||
}
|
||||
|
||||
return view('web.release-detail', $data);
|
||||
|
|
|
|||
|
|
@ -57,6 +57,8 @@ class Company extends Model
|
|||
'logo_variants',
|
||||
'boilerplate',
|
||||
'is_active',
|
||||
'gone_at',
|
||||
'is_active_before_gone',
|
||||
'disable_footer_code',
|
||||
'legacy_portal',
|
||||
'legacy_id',
|
||||
|
|
@ -69,6 +71,8 @@ class Company extends Model
|
|||
'type' => CompanyType::class,
|
||||
'logo_variants' => 'array',
|
||||
'is_active' => 'boolean',
|
||||
'gone_at' => 'datetime',
|
||||
'is_active_before_gone' => 'boolean',
|
||||
'disable_footer_code' => 'boolean',
|
||||
'deleted_at' => 'datetime',
|
||||
];
|
||||
|
|
@ -117,6 +121,16 @@ class Company extends Model
|
|||
});
|
||||
}
|
||||
|
||||
/**
|
||||
* Mit 410 entfernter Newsroom: die Newsroom-URL bleibt bestehen und liefert
|
||||
* einen Tombstone statt eines 404. Gegenstück zur vollständigen Löschung
|
||||
* (Soft Delete).
|
||||
*/
|
||||
public function isGone(): bool
|
||||
{
|
||||
return $this->gone_at !== null;
|
||||
}
|
||||
|
||||
public function owner(): BelongsTo
|
||||
{
|
||||
return $this->belongsTo(User::class, 'owner_user_id');
|
||||
|
|
|
|||
|
|
@ -91,6 +91,8 @@ class PressRelease extends Model
|
|||
'no_export',
|
||||
'published_at',
|
||||
'depublished_at',
|
||||
'gone_at',
|
||||
'no_export_before_gone',
|
||||
'scheduled_at',
|
||||
'embargo_at',
|
||||
'withdrawal_waiver_consented_at',
|
||||
|
|
@ -117,6 +119,8 @@ class PressRelease extends Model
|
|||
'no_export' => 'boolean',
|
||||
'published_at' => 'datetime',
|
||||
'depublished_at' => 'datetime',
|
||||
'gone_at' => 'datetime',
|
||||
'no_export_before_gone' => 'boolean',
|
||||
'sync_locked_at' => 'datetime',
|
||||
'scheduled_at' => 'datetime',
|
||||
'embargo_at' => 'datetime',
|
||||
|
|
@ -482,7 +486,17 @@ class PressRelease extends Model
|
|||
*/
|
||||
public function scopeVisibleInArchive(Builder $query): Builder
|
||||
{
|
||||
return $query->published()->where('published_at', '<=', now());
|
||||
return $query->published()->where('published_at', '<=', now())->whereNull('gone_at');
|
||||
}
|
||||
|
||||
/**
|
||||
* Mit 410 entfernt: die URL bleibt bestehen und liefert einen Tombstone,
|
||||
* der Inhalt ist aber nicht mehr öffentlich. Gegenstück zur vollständigen
|
||||
* Löschung (Soft Delete), die 404 erzeugt.
|
||||
*/
|
||||
public function isGone(): bool
|
||||
{
|
||||
return $this->gone_at !== null;
|
||||
}
|
||||
|
||||
/**
|
||||
|
|
@ -494,7 +508,8 @@ class PressRelease extends Model
|
|||
{
|
||||
return $this->status === PressReleaseStatus::Published
|
||||
&& $this->published_at !== null
|
||||
&& $this->published_at->lessThanOrEqualTo(now());
|
||||
&& $this->published_at->lessThanOrEqualTo(now())
|
||||
&& $this->gone_at === null;
|
||||
}
|
||||
|
||||
/**
|
||||
|
|
|
|||
|
|
@ -32,6 +32,7 @@ use App\Services\Admin\AdminRequestPerformanceMetrics;
|
|||
use App\Services\Newsletter\NullNewsletterSyncClient;
|
||||
use App\Services\PressRelease\PressReleaseService;
|
||||
use App\Support\OperationalAlert;
|
||||
use App\Support\WebPageCache;
|
||||
use Illuminate\Auth\Events\Login;
|
||||
use Illuminate\Auth\Events\Registered;
|
||||
use Illuminate\Auth\Events\Verified;
|
||||
|
|
@ -86,6 +87,12 @@ class AppServiceProvider extends ServiceProvider
|
|||
{
|
||||
Model::preventLazyLoading(app()->isLocal());
|
||||
|
||||
// Die Generations-Nummer der Seiten-Caches wird prozessintern gemerkt.
|
||||
// Ein Queue-Worker lebt über beliebig viele Jobs hinweg und würde sonst
|
||||
// dauerhaft mit dem Stand seines ersten Zugriffs arbeiten — Keys einer
|
||||
// längst verworfenen Generation lesen und löschen.
|
||||
Queue::looping(static fn () => WebPageCache::resetMemo());
|
||||
|
||||
$scheme = request()->header('X-Forwarded-Proto')
|
||||
?? request()->server('HTTP_X_FORWARDED_PROTO')
|
||||
?? (request()->secure() ? 'https' : 'http');
|
||||
|
|
|
|||
123
app/Services/Company/CompanyRemovalService.php
Normal file
123
app/Services/Company/CompanyRemovalService.php
Normal file
|
|
@ -0,0 +1,123 @@
|
|||
<?php
|
||||
|
||||
namespace App\Services\Company;
|
||||
|
||||
use App\Enums\ActivityType;
|
||||
use App\Models\Company;
|
||||
use App\Models\PressRelease;
|
||||
use App\Scopes\PortalScope;
|
||||
use App\Services\ActivityLog;
|
||||
use App\Services\PressRelease\PressReleaseService;
|
||||
use App\Support\WebCaches;
|
||||
|
||||
/**
|
||||
* Entfernung eines Newsrooms aus der öffentlichen Ansicht.
|
||||
*
|
||||
* Zwei bewusst getrennte Wege:
|
||||
* - Vollständige Löschung (Soft Delete, weiterhin in der Firmen-Verwaltung):
|
||||
* die Newsroom-URL antwortet danach mit 404.
|
||||
* - Entfernung mit 410 Gone (hier): die URL bleibt bestehen und liefert einen
|
||||
* Tombstone. Umkehrbar über restoreFromGone().
|
||||
*/
|
||||
class CompanyRemovalService
|
||||
{
|
||||
public function __construct(
|
||||
private readonly PressReleaseService $pressReleases,
|
||||
private readonly ActivityLog $activityLog,
|
||||
) {}
|
||||
|
||||
/**
|
||||
* Markiert den Newsroom als entfernt (410). Die Firma wird zugleich
|
||||
* deaktiviert — dadurch fällt sie ohne Sonderbehandlung aus
|
||||
* Newsroom-Liste und Sitemap, die beide auf `is_active` filtern.
|
||||
*
|
||||
* Die Pressemitteilungen der Firma bleiben absichtlich unangetastet,
|
||||
* solange $includePressReleases nicht gesetzt ist: eine stille
|
||||
* Massen-Depublikation wäre nicht umkehrbar unterscheidbar von einzeln
|
||||
* entfernten Mitteilungen.
|
||||
*
|
||||
* @return int Anzahl der zusätzlich mit 410 entfernten Pressemitteilungen
|
||||
*/
|
||||
public function markGone(Company $company, bool $includePressReleases = false, ?string $reason = null): int
|
||||
{
|
||||
$reason ??= 'Newsroom aus der öffentlichen Ansicht entfernt (410 Gone)';
|
||||
|
||||
if ($company->gone_at === null) {
|
||||
$company->forceFill([
|
||||
'gone_at' => now(),
|
||||
// Vorzustand sichern, sonst ist die Entfernung nicht wirklich
|
||||
// umkehrbar: Eine Firma, die bereits vorher deaktiviert war —
|
||||
// im Bestand der Normalfall bei Karteileichen —, stünde nach
|
||||
// dem Aufheben öffentlich in Newsroom-Übersicht und Sitemap.
|
||||
'is_active_before_gone' => (bool) $company->is_active,
|
||||
'is_active' => false,
|
||||
])->save();
|
||||
|
||||
$this->activityLog->record(
|
||||
ActivityType::ContentRemoval,
|
||||
sprintf('Newsroom „%s“ mit 410 entfernt: %s', $company->name, $reason),
|
||||
subject: $company,
|
||||
);
|
||||
}
|
||||
|
||||
$affected = 0;
|
||||
|
||||
if ($includePressReleases) {
|
||||
// chunkById statt get(): Eine Bestandsfirma kann dreistellig viele
|
||||
// Mitteilungen haben, die hier sonst allesamt gleichzeitig im
|
||||
// Speicher lägen. Der Filter auf `gone_at` schrumpft mit jedem
|
||||
// Durchgang, deshalb chunkById und nicht chunk — ein Offset würde
|
||||
// Zeilen überspringen.
|
||||
PressRelease::query()
|
||||
->withoutGlobalScope(PortalScope::class)
|
||||
->where('company_id', $company->getKey())
|
||||
->whereNull('gone_at')
|
||||
->chunkById(200, function ($releases) use ($reason, &$affected): void {
|
||||
foreach ($releases as $release) {
|
||||
// Ohne Flush je Mitteilung: der verwirft ALLE
|
||||
// Seiten-Caches auf einmal und würde die gesamte
|
||||
// öffentliche Seite pro Zeile in den Neuaufbau zwingen.
|
||||
// Einmal am Ende genügt.
|
||||
$this->pressReleases->markGone($release, $reason, flushPublicPages: false);
|
||||
$affected++;
|
||||
}
|
||||
});
|
||||
}
|
||||
|
||||
WebCaches::forgetPublicPages();
|
||||
|
||||
return $affected;
|
||||
}
|
||||
|
||||
/**
|
||||
* Hebt die 410-Entfernung des Newsrooms auf. Bereits entfernte
|
||||
* Pressemitteilungen bleiben entfernt — sie werden einzeln freigegeben,
|
||||
* damit eine rechtlich veranlasste Einzel-Entfernung nicht versehentlich
|
||||
* mit zurückkommt.
|
||||
*
|
||||
* `is_active` geht auf den Stand VOR der Entfernung zurück, nicht pauschal
|
||||
* auf „aktiv“. Fehlt der Merker (Entfernung aus der Zeit vor dieser
|
||||
* Spalte), bleibt es beim bisherigen Verhalten — für diese Altfälle ist
|
||||
* „aktiv“ die einzige Annahme, die überhaupt zur Verfügung steht.
|
||||
*/
|
||||
public function restoreFromGone(Company $company): void
|
||||
{
|
||||
if ($company->gone_at === null) {
|
||||
return;
|
||||
}
|
||||
|
||||
$company->forceFill([
|
||||
'gone_at' => null,
|
||||
'is_active' => $company->is_active_before_gone ?? true,
|
||||
'is_active_before_gone' => null,
|
||||
])->save();
|
||||
|
||||
$this->activityLog->record(
|
||||
ActivityType::ContentRemoval,
|
||||
sprintf('410-Entfernung des Newsrooms „%s“ aufgehoben', $company->name),
|
||||
subject: $company,
|
||||
);
|
||||
|
||||
WebCaches::forgetPublicPages();
|
||||
}
|
||||
}
|
||||
|
|
@ -13,6 +13,7 @@ use App\Models\User;
|
|||
use App\Scopes\PortalScope;
|
||||
use App\Services\Billing\CreditPricingService;
|
||||
use App\Services\Billing\CreditWalletService;
|
||||
use App\Support\WebPageCache;
|
||||
use Illuminate\Database\Eloquent\Builder;
|
||||
use Illuminate\Support\Carbon;
|
||||
use Illuminate\Support\Facades\Cache;
|
||||
|
|
@ -229,12 +230,12 @@ class BoostService
|
|||
|
||||
foreach ($portals as $portal) {
|
||||
if ($placement === BoostPlacement::Home) {
|
||||
Cache::forget(sprintf('web.home.%s.%s', $portal, $pressRelease->language));
|
||||
Cache::forget(WebPageCache::key('home', [$portal, $pressRelease->language]));
|
||||
}
|
||||
|
||||
foreach ($translations as $translation) {
|
||||
if (filled($translation->slug)) {
|
||||
Cache::forget(sprintf('web.category.%s.%s.%s', $portal, $translation->locale, $translation->slug));
|
||||
Cache::forget(WebPageCache::key('category', [$portal, $translation->locale, $translation->slug]));
|
||||
}
|
||||
}
|
||||
}
|
||||
|
|
|
|||
|
|
@ -411,7 +411,7 @@ class PressReleaseService
|
|||
'depublished_at' => now(),
|
||||
]);
|
||||
$pressRelease->lockSync($syncLockReason ?? 'Depublikation auf Wunsch des Absenders');
|
||||
WebCaches::forgetSitemaps();
|
||||
WebCaches::forgetPublicPages();
|
||||
$this->logStatusChange($pressRelease, $previous, PressReleaseStatus::Archived, 'Depublikation auf Wunsch des Absenders (kostenpflichtige Serviceleistung)', 'depublication');
|
||||
}
|
||||
|
||||
|
|
@ -464,6 +464,89 @@ class PressReleaseService
|
|||
$pressRelease->delete();
|
||||
}
|
||||
|
||||
/**
|
||||
* Entfernung mit 410 Gone: die Mitteilung verschwindet aus allen
|
||||
* öffentlichen Listen, ihre Detail-URL liefert aber weiterhin einen
|
||||
* Tombstone statt eines 404.
|
||||
*
|
||||
* Bewusste Abgrenzung zu deleteFromAdmin(): dort wird der Inhalt durch den
|
||||
* Ersatztext überschrieben bzw. die PM soft-gelöscht. Hier bleibt der
|
||||
* Datensatz unverändert erhalten, damit die Entfernung über
|
||||
* restoreFromGone() umkehrbar und im Audit nachvollziehbar bleibt.
|
||||
*
|
||||
* Die Re-Sync-Sperre ist Pflicht — ohne sie würde der nächste
|
||||
* Importer-Lauf die Entfernung wieder überschreiben.
|
||||
*
|
||||
* $flushPublicPages steuert den Auslieferungs-Cache. Der Flush verwirft
|
||||
* ALLE Seiten-Caches auf einmal (siehe WebPageCache::flush) und ist damit
|
||||
* genau einmal je Vorgang richtig — wer die Methode in einer Schleife
|
||||
* aufruft, schaltet ihn ab und flusht selbst am Ende. Sonst zwingt jede
|
||||
* einzelne Mitteilung die gesamte öffentliche Seite in den Neuaufbau.
|
||||
*/
|
||||
public function markGone(PressRelease $pressRelease, ?string $reason = null, bool $flushPublicPages = true): void
|
||||
{
|
||||
if ($pressRelease->gone_at !== null) {
|
||||
return;
|
||||
}
|
||||
|
||||
$previous = $pressRelease->status;
|
||||
|
||||
$pressRelease->update([
|
||||
'status' => PressReleaseStatus::Archived->value,
|
||||
'gone_at' => now(),
|
||||
// Vorzustand sichern: `no_export` kann auch redaktionell oder vom
|
||||
// Kunden gesetzt sein und ist in beiden Detailansichten sichtbar.
|
||||
// Ohne diesen Merker stünde die Mitteilung nach dem Aufheben
|
||||
// dauerhaft als „kein Export“ da, ohne dass das je jemand gewählt
|
||||
// hätte.
|
||||
'no_export_before_gone' => (bool) $pressRelease->no_export,
|
||||
'no_export' => true,
|
||||
]);
|
||||
|
||||
$pressRelease->lockSync($reason ?? 'Entfernung mit 410 (Tombstone)');
|
||||
|
||||
if ($flushPublicPages) {
|
||||
WebCaches::forgetPublicPages();
|
||||
}
|
||||
|
||||
$this->logStatusChange(
|
||||
$pressRelease,
|
||||
$previous,
|
||||
PressReleaseStatus::Archived,
|
||||
$reason ?? 'Aus der öffentlichen Ansicht entfernt (410 Gone)',
|
||||
'gone',
|
||||
);
|
||||
}
|
||||
|
||||
/**
|
||||
* Hebt eine 410-Entfernung auf. Der Status bleibt bewusst „archived“ —
|
||||
* ob die Mitteilung wieder veröffentlicht wird, entscheidet die Redaktion
|
||||
* anschließend über den regulären Statuswechsel.
|
||||
*
|
||||
* `no_export` geht dagegen auf den Stand VOR der Entfernung zurück: Die
|
||||
* Sperre gehört zur Entfernung, nicht zur Mitteilung. Fehlt der Merker
|
||||
* (Entfernung aus der Zeit vor dieser Spalte), bleibt der aktuelle Wert
|
||||
* stehen — raten wäre schlechter als nichts zu tun.
|
||||
*
|
||||
* Zu $flushPublicPages siehe markGone().
|
||||
*/
|
||||
public function restoreFromGone(PressRelease $pressRelease, bool $flushPublicPages = true): void
|
||||
{
|
||||
if ($pressRelease->gone_at === null) {
|
||||
return;
|
||||
}
|
||||
|
||||
$pressRelease->update([
|
||||
'gone_at' => null,
|
||||
'no_export' => $pressRelease->no_export_before_gone ?? $pressRelease->no_export,
|
||||
'no_export_before_gone' => null,
|
||||
]);
|
||||
|
||||
if ($flushPublicPages) {
|
||||
WebCaches::forgetPublicPages();
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
* @param PressReleaseStatus[] $allowed
|
||||
*/
|
||||
|
|
|
|||
|
|
@ -31,6 +31,14 @@ class NewsroomDetailPageData
|
|||
// (Cache-Poisoning über z. B. ?page=2.5).
|
||||
$page = Paginator::resolveCurrentPage();
|
||||
|
||||
// Bewusst VOR dem Seiten-Cache: eine Entfernung (410) muss sofort
|
||||
// greifen, auch wenn für den Slug noch ein bis zu eine Stunde alter
|
||||
// Stale-Eintrag im Cache liegt. Kostet einen indizierten Slug-Lookup
|
||||
// pro Aufruf — vertretbar, weil Entfernungen rechtlich motiviert sind.
|
||||
if ($this->isGone($context, $slug)) {
|
||||
return ['tombstone' => 'removed'];
|
||||
}
|
||||
|
||||
$cacheKey = WebPageCache::key('newsroom', [
|
||||
implode('-', $context->portalValues),
|
||||
$context->language,
|
||||
|
|
@ -45,6 +53,20 @@ class NewsroomDetailPageData
|
|||
);
|
||||
}
|
||||
|
||||
/**
|
||||
* Newsroom mit 410 entfernt? Vollständig gelöschte Firmen (Soft Delete)
|
||||
* bleiben bewusst außen vor — deren URL antwortet weiterhin mit 404.
|
||||
*/
|
||||
private function isGone(WebPortalContext $context, string $slug): bool
|
||||
{
|
||||
return Company::query()
|
||||
->withoutGlobalScope(PortalScope::class)
|
||||
->whereIn('portal', $context->portalValues)
|
||||
->where('slug', $slug)
|
||||
->whereNotNull('gone_at')
|
||||
->exists();
|
||||
}
|
||||
|
||||
/**
|
||||
* @return array<string, mixed>
|
||||
*/
|
||||
|
|
|
|||
|
|
@ -45,19 +45,23 @@ class ReleaseDetailPageData
|
|||
->first();
|
||||
|
||||
if (! $release) {
|
||||
// T12-Tombstone: depublizierte PMs (kostenpflichtige Entfernung
|
||||
// auf Wunsch des Absenders) melden 410 statt 404 — die URL gab
|
||||
// es, der Inhalt wurde bewusst entfernt.
|
||||
$depublished = PressRelease::query()
|
||||
// Tombstone statt 404: die URL gab es, der Inhalt wurde bewusst
|
||||
// entfernt. Zwei Anlässe mit eigenem Seitentext —
|
||||
// `depublished_at` = kostenpflichtige Depublikation auf Wunsch des
|
||||
// Absenders (T12), `gone_at` = redaktionelle/rechtliche Entfernung
|
||||
// aus dem Admin. Vollständig gelöschte PMs bleiben bei 404.
|
||||
$removed = PressRelease::query()
|
||||
->withoutGlobalScope(PortalScope::class)
|
||||
->forPortalValues($portalValues)
|
||||
->forLanguage($language)
|
||||
->whereNotNull('depublished_at')
|
||||
->where('slug', $slug)
|
||||
->exists();
|
||||
->where(fn (Builder $query) => $query
|
||||
->whereNotNull('depublished_at')
|
||||
->orWhereNotNull('gone_at'))
|
||||
->first(['depublished_at', 'gone_at']);
|
||||
|
||||
if ($depublished) {
|
||||
return ['tombstone' => true];
|
||||
if ($removed) {
|
||||
return ['tombstone' => $removed->depublished_at !== null ? 'depublished' : 'removed'];
|
||||
}
|
||||
|
||||
abort(404);
|
||||
|
|
|
|||
|
|
@ -10,6 +10,18 @@ use Illuminate\Support\Facades\Cache;
|
|||
*/
|
||||
class WebCaches
|
||||
{
|
||||
/**
|
||||
* Kompletter Auslieferungs-Cache nach einer Entfernung: Sitemaps UND alle
|
||||
* Seiten-Caches (Ticker, Startseite, Kategorie-, Newsroom- und Suchlisten).
|
||||
* Ohne den zweiten Teil bliebe eine entfernte Meldung bis zu einer Stunde
|
||||
* in gecachten Listen sichtbar.
|
||||
*/
|
||||
public static function forgetPublicPages(): void
|
||||
{
|
||||
self::forgetSitemaps();
|
||||
WebPageCache::flush();
|
||||
}
|
||||
|
||||
/**
|
||||
* Sitemap-Caches aller Hosts verwerfen (SitemapController cached 1h je
|
||||
* Host) — nach einer Mutation darf kein Cache den alten Stand halten.
|
||||
|
|
|
|||
|
|
@ -33,9 +33,40 @@ class WebPageCache
|
|||
*/
|
||||
public const StaleSeconds = 3600;
|
||||
|
||||
/**
|
||||
* Cache-Key der Generations-Nummer (siehe generation()).
|
||||
*
|
||||
* Bewusst OHNE `web.`-Präfix: Der Zähler ist der eine Eintrag, der eine
|
||||
* Verwerfung des `web.`-Namensraums überleben muss. Läge er darin, würde
|
||||
* ihn ein pauschales Aufräumen dieses Präfixes mitnehmen — und ein Flush
|
||||
* wäre danach stillschweigend wirkungslos.
|
||||
*/
|
||||
private const GenerationKey = 'page-cache.generation';
|
||||
|
||||
/**
|
||||
* Generations-Nummer innerhalb des Requests, damit key() nicht bei jedem
|
||||
* Aufruf erneut den Cache-Store befragt.
|
||||
*
|
||||
* ACHTUNG Lebensdauer: Der Wert gilt für einen Request. In langlebigen
|
||||
* Prozessen (queue:work, künftig Octane) muss er zwischen zwei Vorgängen
|
||||
* verworfen werden — sonst arbeitet der Worker beliebig lange mit einer
|
||||
* veralteten Generation und würde etwa in BoostService::forgetPlacementCaches()
|
||||
* die Keys einer alten Generation löschen, während der aktuelle Eintrag
|
||||
* stehen bliebe. Dafür sorgt resetMemo(), registriert im AppServiceProvider
|
||||
* auf `Queue::looping()`.
|
||||
*/
|
||||
private static ?int $generation = null;
|
||||
|
||||
/**
|
||||
* Baut einen `web.`-Cache-Key aus Segmenten (leere Segmente entfallen).
|
||||
*
|
||||
* Der Key enthält eine Generations-Nummer: Der Store kennt keine Tags, und
|
||||
* die Keys der Listenseiten sind über Portal, Sprache, Seite und Filter
|
||||
* verstreut. Um eine Entfernung (410/Depublikation) sofort wirksam zu
|
||||
* machen, wird deshalb die Generation erhöht — dadurch verfallen alle
|
||||
* `web.`-Seiten-Caches auf einen Schlag. Das ist bewusst grob, aber
|
||||
* Entfernungen sind selten und dürfen nicht bis zu einer Stunde nachhängen.
|
||||
*
|
||||
* @param array<int, string|int> $segments
|
||||
*/
|
||||
public static function key(string $group, array $segments = []): string
|
||||
|
|
@ -45,7 +76,47 @@ class WebPageCache
|
|||
static fn (string $segment): bool => $segment !== '',
|
||||
);
|
||||
|
||||
return implode('.', array_merge(['web', $group], $parts));
|
||||
return implode('.', array_merge(['web', 'g'.self::generation(), $group], $parts));
|
||||
}
|
||||
|
||||
/**
|
||||
* Aktuelle Generation der Seiten-Caches.
|
||||
*/
|
||||
public static function generation(): int
|
||||
{
|
||||
return self::$generation ??= (int) Cache::get(self::GenerationKey, 0);
|
||||
}
|
||||
|
||||
/**
|
||||
* Verwirft die prozessinterne Merkung der Generation, sodass der nächste
|
||||
* Zugriff wieder den Store fragt. Für langlebige Prozesse (Queue-Worker)
|
||||
* und für Tests, die den Cache-Store zurücksetzen.
|
||||
*/
|
||||
public static function resetMemo(): void
|
||||
{
|
||||
self::$generation = null;
|
||||
}
|
||||
|
||||
/**
|
||||
* Verwirft sämtliche `web.`-Seiten-Caches, indem die Generation erhöht
|
||||
* wird. Die alten Einträge laufen über ihre TTL aus und werden nie wieder
|
||||
* gelesen.
|
||||
*/
|
||||
public static function flush(): void
|
||||
{
|
||||
$current = self::generation();
|
||||
$next = (int) Cache::increment(self::GenerationKey);
|
||||
|
||||
// Der Zähler MUSS echt weiterlaufen. Wurde er im Store verworfen
|
||||
// (Eviction, `cache:clear`), liefert increment() wieder 1 und die alte
|
||||
// Generation bliebe gültig — dann würde ein Flush stillschweigend
|
||||
// wirkungslos und entfernte Inhalte blieben in den Listen stehen.
|
||||
if ($next <= $current) {
|
||||
$next = $current + 1;
|
||||
Cache::forever(self::GenerationKey, $next);
|
||||
}
|
||||
|
||||
self::$generation = $next;
|
||||
}
|
||||
|
||||
/**
|
||||
|
|
|
|||
|
|
@ -0,0 +1,65 @@
|
|||
<?php
|
||||
|
||||
use Illuminate\Database\Migrations\Migration;
|
||||
use Illuminate\Database\Schema\Blueprint;
|
||||
use Illuminate\Support\Facades\Schema;
|
||||
|
||||
/**
|
||||
* Entfernung mit 410 Gone: Marker für Newsrooms (Firmen) und Pressemitteilungen,
|
||||
* die bewusst aus der öffentlichen Ansicht genommen wurden, deren URL aber
|
||||
* weiterhin einen Tombstone (410 statt 404) ausliefern soll.
|
||||
*
|
||||
* Abgrenzung zu bestehenden Markern:
|
||||
* - `deleted_at` (Soft Delete) = vollständige Löschung, URL antwortet 404.
|
||||
* - `press_releases.depublished_at` = kostenpflichtige Depublikation auf Wunsch
|
||||
* des Absenders (T12); ebenfalls 410, aber mit eigenem Seitentext.
|
||||
*
|
||||
* REIHENFOLGE BEACHTEN: Diese Migration läuft absichtlich NACH dem Entfernen
|
||||
* der Volltext-Indizes (`drop_fulltext_indexes_from_press_releases`). Grund:
|
||||
* InnoDB lässt beim Löschen eines FULLTEXT-Index eine versteckte
|
||||
* `FTS_DOC_ID`-Spalte zurück, die `ALGORITHM=INSTANT` weiterhin blockiert.
|
||||
* Erst der nächste vollständige Rebuild räumt sie weg — und genau der ist das
|
||||
* `ADD COLUMN` hier. Läuft diese Migration vorher, wird der teure Rebuild
|
||||
* zweimal fällig statt einmal.
|
||||
*
|
||||
* Bewusst ohne Index: Beide Lookups (Tombstone-Prüfung) laufen über den Slug,
|
||||
* und als Filter (`whereNull('gone_at')`) ist die Spalte zu wenig selektiv, um
|
||||
* einen Index zu rechtfertigen — der auf den großen Bestandstabellen nur die
|
||||
* ALTER-Laufzeit im Deploy verlängern würde.
|
||||
*
|
||||
* Die hasColumn-Prüfungen halten die Migration wiederholbar: Auf den großen
|
||||
* Tabellen (>100k Zeilen) kann ein abgebrochener Deploy die Spalte bereits
|
||||
* angelegt haben, ohne dass der Migrations-Eintrag geschrieben wurde.
|
||||
*/
|
||||
return new class extends Migration
|
||||
{
|
||||
public function up(): void
|
||||
{
|
||||
if (! Schema::hasColumn('companies', 'gone_at')) {
|
||||
Schema::table('companies', function (Blueprint $table) {
|
||||
$table->timestamp('gone_at')->nullable()->after('is_active');
|
||||
});
|
||||
}
|
||||
|
||||
if (! Schema::hasColumn('press_releases', 'gone_at')) {
|
||||
Schema::table('press_releases', function (Blueprint $table) {
|
||||
$table->timestamp('gone_at')->nullable()->after('depublished_at');
|
||||
});
|
||||
}
|
||||
}
|
||||
|
||||
public function down(): void
|
||||
{
|
||||
if (Schema::hasColumn('companies', 'gone_at')) {
|
||||
Schema::table('companies', function (Blueprint $table) {
|
||||
$table->dropColumn('gone_at');
|
||||
});
|
||||
}
|
||||
|
||||
if (Schema::hasColumn('press_releases', 'gone_at')) {
|
||||
Schema::table('press_releases', function (Blueprint $table) {
|
||||
$table->dropColumn('gone_at');
|
||||
});
|
||||
}
|
||||
}
|
||||
};
|
||||
|
|
@ -0,0 +1,60 @@
|
|||
<?php
|
||||
|
||||
use Illuminate\Database\Migrations\Migration;
|
||||
use Illuminate\Database\Schema\Blueprint;
|
||||
use Illuminate\Support\Facades\Schema;
|
||||
|
||||
/**
|
||||
* Merkt sich, welchen Wert die Entfernung mit 410 überschrieben hat.
|
||||
*
|
||||
* Hintergrund: `markGone()` schaltet den Newsroom auf `is_active = false` (damit
|
||||
* er ohne Sonderbehandlung aus Newsroom-Liste und Sitemap fällt, die beide auf
|
||||
* `is_active` filtern) und die Pressemitteilung auf `no_export = true`. Das
|
||||
* Aufheben setzte beide Werte bisher bedingungslos auf den Normalzustand
|
||||
* zurück — eine Firma, die schon VOR der Entfernung deaktiviert war, stand
|
||||
* danach also öffentlich in der Newsroom-Übersicht. Die Oberfläche verspricht
|
||||
* an dieser Stelle ausdrücklich Umkehrbarkeit; das war keine.
|
||||
*
|
||||
* `null` heißt: nicht mit 410 entfernt (oder bereits zurückgenommen).
|
||||
*
|
||||
* Kosten: Auf `press_releases` ist das seit dem Entfernen der Volltext-Indizes
|
||||
* ein `ALGORITHM=INSTANT` und damit eine Sache von Millisekunden. `companies`
|
||||
* trägt seinen FULLTEXT-Index weiterhin und wird deshalb kopiert — bei 70 MB
|
||||
* sind das wenige Sekunden, was die Tabelle bewusst behalten darf (siehe
|
||||
* CLAUDE.md).
|
||||
*
|
||||
* Die hasColumn-Prüfungen halten die Migration wiederholbar, falls ein Deploy
|
||||
* zwischen ALTER und Migrations-Eintrag abbricht.
|
||||
*/
|
||||
return new class extends Migration
|
||||
{
|
||||
public function up(): void
|
||||
{
|
||||
if (! Schema::hasColumn('companies', 'is_active_before_gone')) {
|
||||
Schema::table('companies', function (Blueprint $table) {
|
||||
$table->boolean('is_active_before_gone')->nullable()->after('gone_at');
|
||||
});
|
||||
}
|
||||
|
||||
if (! Schema::hasColumn('press_releases', 'no_export_before_gone')) {
|
||||
Schema::table('press_releases', function (Blueprint $table) {
|
||||
$table->boolean('no_export_before_gone')->nullable()->after('gone_at');
|
||||
});
|
||||
}
|
||||
}
|
||||
|
||||
public function down(): void
|
||||
{
|
||||
if (Schema::hasColumn('companies', 'is_active_before_gone')) {
|
||||
Schema::table('companies', function (Blueprint $table) {
|
||||
$table->dropColumn('is_active_before_gone');
|
||||
});
|
||||
}
|
||||
|
||||
if (Schema::hasColumn('press_releases', 'no_export_before_gone')) {
|
||||
Schema::table('press_releases', function (Blueprint $table) {
|
||||
$table->dropColumn('no_export_before_gone');
|
||||
});
|
||||
}
|
||||
}
|
||||
};
|
||||
|
|
@ -2,10 +2,12 @@
|
|||
|
||||
use App\Enums\Portal;
|
||||
use App\Models\Company;
|
||||
use App\Services\Company\CompanyRemovalService;
|
||||
use App\Services\Image\ImageService;
|
||||
use Illuminate\Support\Str;
|
||||
use Illuminate\Validation\ValidationException;
|
||||
use Livewire\Attributes\Layout;
|
||||
use Livewire\Attributes\Locked;
|
||||
use Livewire\Attributes\Title;
|
||||
use Livewire\Attributes\Validate;
|
||||
use Livewire\Volt\Component;
|
||||
|
|
@ -15,6 +17,14 @@ new #[Layout('components.layouts.app'), Title('Firma bearbeiten')] class extends
|
|||
{
|
||||
use WithFileUploads;
|
||||
|
||||
/**
|
||||
* Die bearbeitete Firma. #[Locked], weil save(), deleteCompany(),
|
||||
* markGone() und restoreGone() alle auf diesem Wert arbeiten — ohne das
|
||||
* Attribut könnte der Client ihn im Livewire-Payload austauschen und die
|
||||
* Aktion träfe eine andere Firma als die im Formular angezeigte.
|
||||
* Gegenstück zum #[Locked] $id der PM-Bearbeitung.
|
||||
*/
|
||||
#[Locked]
|
||||
public int $companyId;
|
||||
|
||||
public string $portal = 'both';
|
||||
|
|
@ -72,6 +82,12 @@ new #[Layout('components.layouts.app'), Title('Firma bearbeiten')] class extends
|
|||
|
||||
public bool $is_active = true;
|
||||
|
||||
/** Zeitpunkt der 410-Entfernung des Newsrooms (formatiert) oder null. */
|
||||
public ?string $goneAt = null;
|
||||
|
||||
/** Beim Entfernen mit 410 zusätzlich alle PMs der Firma entfernen. */
|
||||
public bool $goneIncludesPressReleases = false;
|
||||
|
||||
public function mount(int $id): void
|
||||
{
|
||||
$this->companyId = $id;
|
||||
|
|
@ -98,6 +114,7 @@ new #[Layout('components.layouts.app'), Title('Firma bearbeiten')] class extends
|
|||
$this->country = $company->country_code ?? 'DE';
|
||||
$this->is_verified = false;
|
||||
$this->is_active = (bool) $company->is_active;
|
||||
$this->goneAt = $company->gone_at?->format('d.m.Y H:i');
|
||||
$this->current_logo_url = $company->logoUrl();
|
||||
}
|
||||
|
||||
|
|
@ -186,6 +203,50 @@ new #[Layout('components.layouts.app'), Title('Firma bearbeiten')] class extends
|
|||
];
|
||||
}
|
||||
|
||||
/**
|
||||
* Entfernung mit 410 statt Löschung: die Newsroom-URL liefert danach einen
|
||||
* Tombstone (410 Gone) statt eines 404. Umkehrbar über restoreGone().
|
||||
*/
|
||||
public function markGone(CompanyRemovalService $removal): void
|
||||
{
|
||||
$this->authorize('companies:write');
|
||||
|
||||
$company = Company::query()->find($this->companyId);
|
||||
if (! $company) {
|
||||
session()->flash('error', __('Die angeforderte Firma wurde nicht gefunden.'));
|
||||
$this->redirect(route('admin.companies.index'), navigate: true);
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
$affected = $removal->markGone($company, $this->goneIncludesPressReleases);
|
||||
|
||||
session()->flash('success', $affected > 0
|
||||
? __('Newsroom mit 410 entfernt — inklusive :count Pressemitteilung(en).', ['count' => $affected])
|
||||
: __('Newsroom mit 410 entfernt. Die URL liefert jetzt 410 Gone.'));
|
||||
|
||||
$this->redirect(route('admin.companies.index'), navigate: true);
|
||||
}
|
||||
|
||||
public function restoreGone(CompanyRemovalService $removal): void
|
||||
{
|
||||
$this->authorize('companies:write');
|
||||
|
||||
$company = Company::query()->find($this->companyId);
|
||||
if (! $company) {
|
||||
session()->flash('error', __('Die angeforderte Firma wurde nicht gefunden.'));
|
||||
$this->redirect(route('admin.companies.index'), navigate: true);
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
$removal->restoreFromGone($company);
|
||||
|
||||
session()->flash('success', __('410-Entfernung aufgehoben. Bereits entfernte Pressemitteilungen bleiben entfernt.'));
|
||||
|
||||
$this->redirect(route('admin.companies.edit', $this->companyId), navigate: true);
|
||||
}
|
||||
|
||||
public function deleteCompany(): void
|
||||
{
|
||||
$this->authorize('companies:write');
|
||||
|
|
@ -411,18 +472,46 @@ new #[Layout('components.layouts.app'), Title('Firma bearbeiten')] class extends
|
|||
<div class="panel-head">
|
||||
<span class="section-eyebrow">{{ __('Aktionen') }}</span>
|
||||
</div>
|
||||
@if ($goneAt)
|
||||
<div class="mx-5 mt-5 rounded-[5px] border border-red-300 bg-red-50 p-3 text-[12.5px] text-red-900">
|
||||
{{ __('Dieser Newsroom ist mit 410 entfernt (seit :date). Die öffentliche Newsroom-URL liefert einen Tombstone statt Inhalten.', ['date' => $goneAt]) }}
|
||||
</div>
|
||||
@endif
|
||||
<div class="p-5 flex justify-between items-center gap-3 flex-wrap">
|
||||
<flux:modal.trigger name="confirm-company-deletion">
|
||||
<flux:button
|
||||
variant="danger"
|
||||
icon="trash"
|
||||
type="button"
|
||||
x-data=""
|
||||
x-on:click.prevent="$dispatch('open-modal', 'confirm-company-deletion')"
|
||||
>
|
||||
{{ __('Löschen') }}
|
||||
</flux:button>
|
||||
</flux:modal.trigger>
|
||||
<div class="flex gap-3 flex-wrap">
|
||||
@if ($goneAt)
|
||||
<flux:button
|
||||
variant="filled"
|
||||
icon="arrow-uturn-left"
|
||||
type="button"
|
||||
wire:click.prevent="restoreGone"
|
||||
wire:confirm="{{ __('410-Entfernung aufheben? Der Newsroom wird wieder aktiv geschaltet. Bereits entfernte Pressemitteilungen bleiben entfernt.') }}"
|
||||
>
|
||||
{{ __('410-Entfernung aufheben') }}
|
||||
</flux:button>
|
||||
@else
|
||||
<flux:button
|
||||
variant="danger"
|
||||
icon="no-symbol"
|
||||
type="button"
|
||||
x-data=""
|
||||
x-on:click.prevent="$dispatch('open-modal', 'confirm-company-gone')"
|
||||
>
|
||||
{{ __('Entfernen mit 410') }}
|
||||
</flux:button>
|
||||
@endif
|
||||
<flux:modal.trigger name="confirm-company-deletion">
|
||||
<flux:button
|
||||
variant="danger"
|
||||
icon="trash"
|
||||
type="button"
|
||||
x-data=""
|
||||
x-on:click.prevent="$dispatch('open-modal', 'confirm-company-deletion')"
|
||||
>
|
||||
{{ __('Vollständig löschen') }}
|
||||
</flux:button>
|
||||
</flux:modal.trigger>
|
||||
</div>
|
||||
<div class="flex gap-3">
|
||||
<flux:button variant="filled" href="{{ route('admin.companies.index') }}" wire:navigate>
|
||||
{{ __('Abbrechen') }}
|
||||
|
|
@ -435,12 +524,39 @@ new #[Layout('components.layouts.app'), Title('Firma bearbeiten')] class extends
|
|||
</article>
|
||||
</form>
|
||||
|
||||
<flux:modal name="confirm-company-gone" class="max-w-lg">
|
||||
<div class="space-y-6">
|
||||
<div>
|
||||
<flux:heading size="lg">{{ __('Newsroom mit 410 entfernen?') }}</flux:heading>
|
||||
<flux:subheading>
|
||||
{{ __('Der Newsroom wird deaktiviert und verschwindet aus Newsroom-Übersicht und Sitemap. Seine bestehende URL antwortet danach mit „410 Gone“ und zeigt einen Hinweis auf die Entfernung — statt eines 404. Die Firma bleibt in der Verwaltung erhalten, die Entfernung ist umkehrbar.') }}
|
||||
</flux:subheading>
|
||||
</div>
|
||||
|
||||
<flux:checkbox
|
||||
wire:model="goneIncludesPressReleases"
|
||||
label="{{ __('Auch alle Pressemitteilungen dieses Newsrooms mit 410 entfernen') }}"
|
||||
description="{{ __('Ohne diesen Haken bleiben veröffentlichte Pressemitteilungen der Firma weiterhin einzeln abrufbar. Das Aufheben der Entfernung gibt Pressemitteilungen nicht automatisch wieder frei.') }}"
|
||||
/>
|
||||
|
||||
<div class="flex justify-end space-x-2 rtl:space-x-reverse">
|
||||
<flux:modal.close>
|
||||
<flux:button variant="filled">{{ __('Abbrechen') }}</flux:button>
|
||||
</flux:modal.close>
|
||||
|
||||
<flux:button variant="danger" wire:click="markGone">
|
||||
{{ __('Mit 410 entfernen') }}
|
||||
</flux:button>
|
||||
</div>
|
||||
</div>
|
||||
</flux:modal>
|
||||
|
||||
<flux:modal name="confirm-company-deletion" class="max-w-lg">
|
||||
<div class="space-y-6">
|
||||
<div>
|
||||
<flux:heading size="lg">{{ __('Firma wirklich löschen?') }}</flux:heading>
|
||||
<flux:subheading>
|
||||
{{ __('Diese Aktion kann nicht direkt rückgängig gemacht werden. Die Firma wird archiviert (Soft Delete) und aus den Standardlisten entfernt.') }}
|
||||
{{ __('Diese Aktion kann nicht direkt rückgängig gemacht werden. Die Firma wird archiviert (Soft Delete) und aus den Standardlisten entfernt. Die öffentliche Newsroom-URL antwortet danach mit 404. Soll die Adresse stattdessen einen Entfernungs-Hinweis mit „410 Gone“ ausliefern, brechen Sie hier ab und wählen „Entfernen mit 410“.') }}
|
||||
</flux:subheading>
|
||||
</div>
|
||||
|
||||
|
|
|
|||
|
|
@ -34,6 +34,10 @@ new #[Layout('components.layouts.app'), Title('Pressemitteilung bearbeiten')] cl
|
|||
#[Locked]
|
||||
public int $id;
|
||||
|
||||
/** Zeitpunkt der 410-Entfernung (formatiert) oder null. */
|
||||
#[Locked]
|
||||
public ?string $goneAt = null;
|
||||
|
||||
public string $portal = '';
|
||||
|
||||
public string $language = 'de';
|
||||
|
|
@ -94,6 +98,7 @@ new #[Layout('components.layouts.app'), Title('Pressemitteilung bearbeiten')] cl
|
|||
|
||||
$pr = $this->findAdminPR($id);
|
||||
|
||||
$this->goneAt = $pr->gone_at?->format('d.m.Y H:i');
|
||||
$this->portal = $pr->portal->value;
|
||||
$this->language = $pr->language;
|
||||
$this->companyId = $pr->company_id;
|
||||
|
|
@ -367,6 +372,32 @@ new #[Layout('components.layouts.app'), Title('Pressemitteilung bearbeiten')] cl
|
|||
Flux::modal('confirm-status-change')->close();
|
||||
}
|
||||
|
||||
/**
|
||||
* Entfernung mit 410 statt Löschung: die Detail-URL liefert danach einen
|
||||
* Tombstone (410 Gone) statt eines 404. Umkehrbar über restoreGone().
|
||||
*/
|
||||
public function markGone(): void
|
||||
{
|
||||
$this->authorize('press-releases:publish');
|
||||
|
||||
app(PressReleaseService::class)->markGone($this->findAdminPR($this->id));
|
||||
|
||||
Flux::toast(text: __('Pressemitteilung entfernt — die URL liefert jetzt 410 Gone.'), variant: 'success');
|
||||
|
||||
$this->redirect(route('admin.press-releases.index'), navigate: true);
|
||||
}
|
||||
|
||||
public function restoreGone(): void
|
||||
{
|
||||
$this->authorize('press-releases:publish');
|
||||
|
||||
app(PressReleaseService::class)->restoreFromGone($this->findAdminPR($this->id));
|
||||
|
||||
Flux::toast(text: __('410-Entfernung aufgehoben. Die Pressemitteilung ist archiviert und kann neu veröffentlicht werden.'), variant: 'success');
|
||||
|
||||
$this->redirect(route('admin.press-releases.edit', $this->id), navigate: true);
|
||||
}
|
||||
|
||||
public function deletePressRelease(): void
|
||||
{
|
||||
$this->authorize('press-releases:publish');
|
||||
|
|
@ -1018,9 +1049,25 @@ new #[Layout('components.layouts.app'), Title('Pressemitteilung bearbeiten')] cl
|
|||
<flux:button type="button" variant="primary" icon="bookmark" class="w-full" wire:click="save">
|
||||
{{ __('Änderungen speichern') }}
|
||||
</flux:button>
|
||||
@if ($goneAt)
|
||||
<div class="rounded-[5px] border border-red-300 bg-red-50 p-3 text-[12.5px] text-red-900">
|
||||
{{ __('Diese Pressemitteilung ist mit 410 entfernt (seit :date). Die öffentliche URL liefert einen Tombstone statt Inhalten.', ['date' => $goneAt]) }}
|
||||
</div>
|
||||
<flux:button type="button" variant="filled" icon="arrow-uturn-left" class="w-full"
|
||||
wire:click="restoreGone"
|
||||
wire:confirm="{{ __('410-Entfernung aufheben? Die Mitteilung bleibt archiviert und muss anschließend neu veröffentlicht werden.') }}">
|
||||
{{ __('410-Entfernung aufheben') }}
|
||||
</flux:button>
|
||||
@else
|
||||
<flux:modal.trigger name="confirm-mark-press-release-gone">
|
||||
<flux:button type="button" variant="danger" icon="no-symbol" class="w-full">
|
||||
{{ __('Entfernen mit 410 (Tombstone)') }}
|
||||
</flux:button>
|
||||
</flux:modal.trigger>
|
||||
@endif
|
||||
<flux:modal.trigger name="confirm-delete-press-release">
|
||||
<flux:button type="button" variant="danger" icon="trash" class="w-full">
|
||||
{{ __('Pressemitteilung löschen') }}
|
||||
{{ __('Vollständig löschen') }}
|
||||
</flux:button>
|
||||
</flux:modal.trigger>
|
||||
</div>
|
||||
|
|
@ -1065,16 +1112,34 @@ new #[Layout('components.layouts.app'), Title('Pressemitteilung bearbeiten')] cl
|
|||
</div>
|
||||
</flux:modal>
|
||||
|
||||
<flux:modal name="confirm-mark-press-release-gone" class="max-w-lg">
|
||||
<div class="space-y-6">
|
||||
<div>
|
||||
<flux:heading size="lg">{{ __('Pressemitteilung mit 410 entfernen?') }}</flux:heading>
|
||||
<flux:subheading>
|
||||
{{ __('Die Mitteilung verschwindet aus allen öffentlichen Listen, Feeds und der Sitemap. Ihre bestehende URL antwortet danach mit „410 Gone“ und zeigt einen Hinweis auf die Entfernung — statt eines 404. Der Datensatz bleibt erhalten, die Entfernung ist umkehrbar.') }}
|
||||
</flux:subheading>
|
||||
</div>
|
||||
<div class="flex justify-end space-x-2 rtl:space-x-reverse">
|
||||
<flux:modal.close>
|
||||
<flux:button variant="filled">{{ __('Abbrechen') }}</flux:button>
|
||||
</flux:modal.close>
|
||||
<flux:button variant="danger" wire:click="markGone">{{ __('Mit 410 entfernen') }}</flux:button>
|
||||
</div>
|
||||
</div>
|
||||
</flux:modal>
|
||||
|
||||
<flux:modal name="confirm-delete-press-release" class="max-w-lg">
|
||||
<div class="space-y-6">
|
||||
<div>
|
||||
<flux:heading size="lg">{{ __('Pressemitteilung löschen?') }}</flux:heading>
|
||||
<flux:subheading>
|
||||
@if ($currentStatus === 'published')
|
||||
{{ __('Diese Pressemitteilung ist veröffentlicht. Sie wird nicht entfernt, sondern archiviert und der Inhalt wird durch den voreingestellten Ersatztext ersetzt, damit die URL keinen 404-Fehler erzeugt.') }}
|
||||
{{ __('Diese Pressemitteilung ist veröffentlicht. Sie wird nicht entfernt, sondern archiviert und der Inhalt wird durch den voreingestellten Ersatztext ersetzt.') }}
|
||||
@else
|
||||
{{ __('Diese Pressemitteilung wird per Soft Delete aus den Standardlisten entfernt.') }}
|
||||
@endif
|
||||
{{ __('Die öffentliche URL antwortet danach mit 404. Soll die Adresse stattdessen einen Entfernungs-Hinweis mit „410 Gone“ ausliefern, brechen Sie hier ab und wählen „Entfernen mit 410 (Tombstone)“.') }}
|
||||
</flux:subheading>
|
||||
</div>
|
||||
<div class="flex justify-end space-x-2 rtl:space-x-reverse">
|
||||
|
|
|
|||
37
resources/views/web/newsroom-tombstone.blade.php
Normal file
37
resources/views/web/newsroom-tombstone.blade.php
Normal file
|
|
@ -0,0 +1,37 @@
|
|||
@extends('web.layouts.web-master')
|
||||
|
||||
@section('title', 'Newsroom entfernt')
|
||||
@section('meta_description', 'Dieser Newsroom wurde aus der öffentlichen Ansicht entfernt.')
|
||||
|
||||
{{--
|
||||
Tombstone (410 Gone) für einen Newsroom: Die Unternehmensseite wurde
|
||||
bewusst aus der öffentlichen Ansicht genommen. Bewusst ohne Firmenname
|
||||
und Inhalte — der Zweck der Entfernung ist die Entfernung. Gegenstück zur
|
||||
vollständigen Löschung, die weiterhin 404 liefert.
|
||||
--}}
|
||||
@section('content')
|
||||
<x-web.static-page
|
||||
eyebrow="Hinweis"
|
||||
title="Dieser Newsroom wurde entfernt"
|
||||
breadcrumb="Entfernt"
|
||||
>
|
||||
<div class="pm-body max-w-[680px]">
|
||||
<p>
|
||||
Der Newsroom unter dieser Adresse wurde dauerhaft aus der öffentlichen Ansicht
|
||||
entfernt und steht nicht mehr zur Verfügung.
|
||||
</p>
|
||||
<p>
|
||||
Bitte beachten Sie: Kopien in Suchmaschinen-Caches oder Archiven Dritter können
|
||||
unabhängig davon zeitweise weiter verfügbar sein — darauf haben wir keinen Einfluss.
|
||||
</p>
|
||||
<p>
|
||||
Eine Übersicht der verfügbaren Newsrooms finden Sie in unserer
|
||||
<a href="{{ route('newsrooms') }}">Newsroom-Übersicht</a>.
|
||||
</p>
|
||||
<p>
|
||||
Wenn Sie ein rechtliches Anliegen zu Inhalten auf diesem Portal haben, nutzen Sie
|
||||
bitte unser <a href="{{ route('melden') }}">Meldeverfahren</a> — Meldungen sind kostenlos.
|
||||
</p>
|
||||
</div>
|
||||
</x-web.static-page>
|
||||
@endsection
|
||||
|
|
@ -1,13 +1,19 @@
|
|||
@php
|
||||
/** @var string $reason 'depublished' (T12, Wunsch des Absenders) oder 'removed' (redaktionell/rechtlich) */
|
||||
$reason = $reason ?? 'depublished';
|
||||
@endphp
|
||||
@extends('web.layouts.web-master')
|
||||
|
||||
@section('title', 'Pressemitteilung entfernt')
|
||||
@section('meta_description', 'Diese Pressemitteilung wurde aus der öffentlichen Ansicht entfernt.')
|
||||
|
||||
{{--
|
||||
T12-Tombstone (410 Gone): Die PM wurde auf Wunsch des Absenders
|
||||
kostenpflichtig depubliziert. Bewusst ohne Titel/Inhalte — der Zweck
|
||||
der Depublikation ist die Entfernung; ehrlicher Hinweis auf Grenzen
|
||||
(Suchmaschinen-Caches, Drittarchive) statt stillem 404.
|
||||
Tombstone (410 Gone): Die PM wurde bewusst aus der öffentlichen Ansicht
|
||||
genommen — entweder auf Wunsch des Absenders (kostenpflichtige
|
||||
Depublikation, T12) oder redaktionell/rechtlich über die Verwaltung.
|
||||
Bewusst ohne Titel/Inhalte — der Zweck der Entfernung ist die Entfernung;
|
||||
ehrlicher Hinweis auf Grenzen (Suchmaschinen-Caches, Drittarchive) statt
|
||||
stillem 404.
|
||||
--}}
|
||||
@section('content')
|
||||
<x-web.static-page
|
||||
|
|
@ -16,10 +22,17 @@
|
|||
breadcrumb="Entfernt"
|
||||
>
|
||||
<div class="pm-body max-w-[680px]">
|
||||
<p>
|
||||
Die Pressemitteilung unter dieser Adresse wurde auf Wunsch des Absenders dauerhaft
|
||||
aus der öffentlichen Ansicht entfernt.
|
||||
</p>
|
||||
@if ($reason === 'depublished')
|
||||
<p>
|
||||
Die Pressemitteilung unter dieser Adresse wurde auf Wunsch des Absenders dauerhaft
|
||||
aus der öffentlichen Ansicht entfernt.
|
||||
</p>
|
||||
@else
|
||||
<p>
|
||||
Die Pressemitteilung unter dieser Adresse wurde dauerhaft aus der öffentlichen
|
||||
Ansicht entfernt und steht nicht mehr zur Verfügung.
|
||||
</p>
|
||||
@endif
|
||||
<p>
|
||||
Bitte beachten Sie: Kopien in Suchmaschinen-Caches oder Archiven Dritter können
|
||||
unabhängig davon zeitweise weiter verfügbar sein — darauf haben wir keinen Einfluss.
|
||||
|
|
|
|||
|
|
@ -195,7 +195,7 @@ test('booking a category boost invalidates the category page but keeps the homep
|
|||
|
||||
// Branchenseite sofort aktualisiert, Startseiten-Cache bleibt stehen
|
||||
// (der Kategorie-Boost berührt die Startseite nicht).
|
||||
expect(Cache::has('web.home.presseecho.de'))->toBeTrue();
|
||||
expect(Cache::has(webPageCacheKey('web.home.presseecho.de')))->toBeTrue();
|
||||
$this->get('https://presseecho.test/de/category/energie-klima')
|
||||
->assertSuccessful()
|
||||
->assertSeeText('Top-Platzierung');
|
||||
|
|
|
|||
|
|
@ -32,9 +32,9 @@ test('category navigation caches per portal, language and limit separately', fun
|
|||
|
||||
CategoryNavigation::items(5);
|
||||
|
||||
expect(Cache::has('web.category-nav.presseecho.de.5'))->toBeTrue()
|
||||
->and(Cache::has('web.category-nav.presseecho.de.all'))->toBeFalse();
|
||||
expect(Cache::has(webPageCacheKey('web.category-nav.presseecho.de.5')))->toBeTrue()
|
||||
->and(Cache::has(webPageCacheKey('web.category-nav.presseecho.de.all')))->toBeFalse();
|
||||
|
||||
CategoryNavigation::items(null);
|
||||
expect(Cache::has('web.category-nav.presseecho.de.all'))->toBeTrue();
|
||||
expect(Cache::has(webPageCacheKey('web.category-nav.presseecho.de.all')))->toBeTrue();
|
||||
});
|
||||
|
|
|
|||
359
tests/Feature/Web/GoneTombstoneTest.php
Normal file
359
tests/Feature/Web/GoneTombstoneTest.php
Normal file
|
|
@ -0,0 +1,359 @@
|
|||
<?php
|
||||
|
||||
use App\Enums\Portal;
|
||||
use App\Enums\PressReleaseStatus;
|
||||
use App\Models\Company;
|
||||
use App\Models\PressRelease;
|
||||
use App\Models\User;
|
||||
use App\Services\Company\CompanyRemovalService;
|
||||
use App\Services\PressRelease\PressReleaseService;
|
||||
use App\Support\WebPageCache;
|
||||
use Database\Seeders\RolesAndPermissionsSeeder;
|
||||
use Livewire\Volt\Volt as LivewireVolt;
|
||||
use Tests\TestCase;
|
||||
|
||||
/**
|
||||
* Entfernung mit 410 Gone (Newsroom und Pressemitteilung) als Alternative zur
|
||||
* vollständigen Löschung, die weiterhin 404 liefert.
|
||||
*/
|
||||
beforeEach(function (): void {
|
||||
/** @var TestCase $this */
|
||||
$this->seed(RolesAndPermissionsSeeder::class);
|
||||
});
|
||||
|
||||
function goneTestCompany(): Company
|
||||
{
|
||||
return Company::factory()->create([
|
||||
'portal' => Portal::Businessportal24,
|
||||
'is_active' => true,
|
||||
]);
|
||||
}
|
||||
|
||||
function goneTestRelease(?Company $company = null): PressRelease
|
||||
{
|
||||
return PressRelease::factory()
|
||||
->published()
|
||||
->forPortal(Portal::Businessportal24)
|
||||
->create([
|
||||
'company_id' => $company?->getKey(),
|
||||
'published_at' => now()->subDay(),
|
||||
]);
|
||||
}
|
||||
|
||||
function goneTestAdmin(): User
|
||||
{
|
||||
$admin = User::factory()->create(['is_active' => true]);
|
||||
$admin->assignRole('admin');
|
||||
|
||||
return $admin;
|
||||
}
|
||||
|
||||
test('a press release marked gone answers 410 with a tombstone instead of 404', function () {
|
||||
/** @var TestCase $this */
|
||||
$release = goneTestRelease();
|
||||
$url = 'https://businessportal24.test/de/press-release/'.$release->slug;
|
||||
|
||||
$this->get($url)->assertOk()->assertSeeText($release->title);
|
||||
|
||||
app(PressReleaseService::class)->markGone($release);
|
||||
|
||||
$this->get($url)
|
||||
->assertStatus(410)
|
||||
->assertSeeText('Diese Pressemitteilung wurde entfernt')
|
||||
->assertDontSeeText($release->title)
|
||||
// Eigener Wortlaut: keine Depublikation "auf Wunsch des Absenders".
|
||||
->assertDontSeeText('auf Wunsch des Absenders');
|
||||
});
|
||||
|
||||
test('marking a press release gone archives it, blocks export and locks the re-sync', function () {
|
||||
/** @var TestCase $this */
|
||||
$release = goneTestRelease();
|
||||
|
||||
app(PressReleaseService::class)->markGone($release);
|
||||
$release->refresh();
|
||||
|
||||
expect($release->gone_at)->not->toBeNull()
|
||||
->and($release->status)->toBe(PressReleaseStatus::Archived)
|
||||
->and($release->no_export)->toBeTrue()
|
||||
// Ohne Re-Sync-Sperre würde der nächste Importer-Lauf die Entfernung
|
||||
// wieder überschreiben.
|
||||
->and($release->isSyncLocked())->toBeTrue()
|
||||
->and($release->deleted_at)->toBeNull()
|
||||
->and($release->statusLogs()->where('source', 'gone')->count())->toBe(1);
|
||||
});
|
||||
|
||||
test('a fully deleted press release still answers 404', function () {
|
||||
/** @var TestCase $this */
|
||||
$release = goneTestRelease();
|
||||
$url = 'https://businessportal24.test/de/press-release/'.$release->slug;
|
||||
|
||||
$release->delete();
|
||||
|
||||
$this->get($url)->assertNotFound();
|
||||
});
|
||||
|
||||
test('lifting the gone marker makes the press release publishable again', function () {
|
||||
/** @var TestCase $this */
|
||||
$release = goneTestRelease();
|
||||
$service = app(PressReleaseService::class);
|
||||
|
||||
$service->markGone($release);
|
||||
$service->restoreFromGone($release->fresh());
|
||||
|
||||
$release->refresh();
|
||||
|
||||
expect($release->gone_at)->toBeNull()
|
||||
// Der Status bleibt bewusst archiviert — die Freigabe ist ein
|
||||
// eigener redaktioneller Schritt.
|
||||
->and($release->status)->toBe(PressReleaseStatus::Archived);
|
||||
|
||||
$this->get('https://businessportal24.test/de/press-release/'.$release->slug)->assertNotFound();
|
||||
});
|
||||
|
||||
test('a newsroom marked gone answers 410 with a tombstone instead of 404', function () {
|
||||
/** @var TestCase $this */
|
||||
$company = goneTestCompany();
|
||||
goneTestRelease($company);
|
||||
$url = 'https://businessportal24.test/de/newsroom/'.$company->slug;
|
||||
|
||||
$this->get($url)->assertOk();
|
||||
|
||||
app(CompanyRemovalService::class)->markGone($company);
|
||||
|
||||
$this->get($url)
|
||||
->assertStatus(410)
|
||||
->assertSeeText('Dieser Newsroom wurde entfernt')
|
||||
->assertDontSeeText($company->name);
|
||||
});
|
||||
|
||||
test('the newsroom tombstone beats a warm page cache', function () {
|
||||
/** @var TestCase $this */
|
||||
$company = goneTestCompany();
|
||||
goneTestRelease($company);
|
||||
$url = 'https://businessportal24.test/de/newsroom/'.$company->slug;
|
||||
|
||||
// Erst aufwärmen, damit für den Slug ein Seiten-Cache existiert …
|
||||
$this->get($url)->assertOk();
|
||||
|
||||
// … dann den Marker OHNE Cache-Invalidierung setzen. Damit prüft der Test
|
||||
// die Absicherung in NewsroomDetailPageData (Gone-Prüfung vor dem Cache)
|
||||
// und nicht bloß das Verwerfen der Caches im Service.
|
||||
$company->forceFill(['gone_at' => now(), 'is_active' => false])->saveQuietly();
|
||||
|
||||
$this->get($url)->assertStatus(410);
|
||||
});
|
||||
|
||||
test('removing content drops the cached public listings immediately', function () {
|
||||
/** @var TestCase $this */
|
||||
$release = goneTestRelease();
|
||||
|
||||
// Startseiten-/Ticker-Caches aufwärmen …
|
||||
$this->get('https://businessportal24.test/de')->assertOk();
|
||||
|
||||
app(PressReleaseService::class)->markGone($release);
|
||||
|
||||
// … die entfernte Meldung darf danach in keiner Liste mehr auftauchen,
|
||||
// auch nicht aus dem Stale-Fenster (bis zu einer Stunde).
|
||||
$this->get('https://businessportal24.test/de')
|
||||
->assertOk()
|
||||
->assertDontSeeText($release->title);
|
||||
});
|
||||
|
||||
test('a fully deleted newsroom still answers 404', function () {
|
||||
/** @var TestCase $this */
|
||||
$company = goneTestCompany();
|
||||
goneTestRelease($company);
|
||||
$url = 'https://businessportal24.test/de/newsroom/'.$company->slug;
|
||||
|
||||
$company->delete();
|
||||
|
||||
$this->get($url)->assertNotFound();
|
||||
});
|
||||
|
||||
test('removing a newsroom leaves its press releases alone unless asked to cascade', function () {
|
||||
/** @var TestCase $this */
|
||||
$company = goneTestCompany();
|
||||
$release = goneTestRelease($company);
|
||||
|
||||
app(CompanyRemovalService::class)->markGone($company);
|
||||
|
||||
expect($release->fresh()->gone_at)->toBeNull();
|
||||
$this->get('https://businessportal24.test/de/press-release/'.$release->slug)->assertOk();
|
||||
});
|
||||
|
||||
test('removing a newsroom can cascade the 410 to its press releases', function () {
|
||||
/** @var TestCase $this */
|
||||
$company = goneTestCompany();
|
||||
$release = goneTestRelease($company);
|
||||
|
||||
$affected = app(CompanyRemovalService::class)->markGone($company, includePressReleases: true);
|
||||
|
||||
expect($affected)->toBe(1)
|
||||
->and($release->fresh()->gone_at)->not->toBeNull();
|
||||
|
||||
$this->get('https://businessportal24.test/de/press-release/'.$release->slug)->assertStatus(410);
|
||||
});
|
||||
|
||||
test('cascading a newsroom removal flushes the page cache once, not per release', function () {
|
||||
/** @var TestCase $this */
|
||||
$company = goneTestCompany();
|
||||
goneTestRelease($company);
|
||||
goneTestRelease($company);
|
||||
goneTestRelease($company);
|
||||
|
||||
$before = WebPageCache::generation();
|
||||
|
||||
$affected = app(CompanyRemovalService::class)->markGone($company, includePressReleases: true);
|
||||
|
||||
// Ein Flush verwirft ALLE `web.`-Seiten-Caches auf einmal. Einer pro
|
||||
// Mitteilung wäre ein selbst ausgelöster Cache-Stampede: Eine Bestandsfirma
|
||||
// mit dreistellig vielen Meldungen zwänge die gesamte öffentliche Seite
|
||||
// ebenso oft in den Neuaufbau.
|
||||
expect($affected)->toBe(3)
|
||||
->and(WebPageCache::generation())->toBe($before + 1);
|
||||
});
|
||||
|
||||
test('lifting a newsroom removal keeps individually removed press releases gone', function () {
|
||||
/** @var TestCase $this */
|
||||
$company = goneTestCompany();
|
||||
$release = goneTestRelease($company);
|
||||
$removal = app(CompanyRemovalService::class);
|
||||
|
||||
$removal->markGone($company, includePressReleases: true);
|
||||
$removal->restoreFromGone($company->fresh());
|
||||
|
||||
$company->refresh();
|
||||
|
||||
expect($company->gone_at)->toBeNull()
|
||||
->and($company->is_active)->toBeTrue()
|
||||
// Eine einmal entfernte PM kommt nicht automatisch zurück — sonst
|
||||
// könnte eine rechtlich veranlasste Einzel-Entfernung versehentlich
|
||||
// mit aufgehoben werden.
|
||||
->and($release->fresh()->gone_at)->not->toBeNull();
|
||||
});
|
||||
|
||||
test('lifting a newsroom removal leaves an already inactive newsroom inactive', function () {
|
||||
/** @var TestCase $this */
|
||||
$company = Company::factory()->create([
|
||||
'portal' => Portal::Businessportal24,
|
||||
'is_active' => false,
|
||||
]);
|
||||
$removal = app(CompanyRemovalService::class);
|
||||
|
||||
$removal->markGone($company);
|
||||
$removal->restoreFromGone($company->fresh());
|
||||
|
||||
$company->refresh();
|
||||
|
||||
// Die Oberfläche verspricht Umkehrbarkeit. Eine Firma, die schon vor der
|
||||
// Entfernung deaktiviert war, darf durch das Aufheben nicht öffentlich
|
||||
// werden — sie stünde sonst plötzlich in Newsroom-Übersicht und Sitemap.
|
||||
expect($company->gone_at)->toBeNull()
|
||||
->and($company->is_active)->toBeFalse()
|
||||
->and($company->is_active_before_gone)->toBeNull();
|
||||
});
|
||||
|
||||
test('lifting a press release removal restores the export flag it overwrote', function () {
|
||||
/** @var TestCase $this */
|
||||
$release = goneTestRelease();
|
||||
$release->update(['no_export' => false]);
|
||||
$service = app(PressReleaseService::class);
|
||||
|
||||
$service->markGone($release);
|
||||
expect($release->fresh()->no_export)->toBeTrue();
|
||||
|
||||
$service->restoreFromGone($release->fresh());
|
||||
|
||||
// `no_export` gehört zur Entfernung, nicht zur Mitteilung — sonst stünde
|
||||
// sie in Kunden- und Admin-Ansicht dauerhaft als „kein Export“ da, ohne
|
||||
// dass das je jemand gewählt hätte.
|
||||
expect($release->fresh()->no_export)->toBeFalse()
|
||||
->and($release->fresh()->no_export_before_gone)->toBeNull();
|
||||
});
|
||||
|
||||
test('lifting a press release removal keeps an export block that predates it', function () {
|
||||
/** @var TestCase $this */
|
||||
$release = goneTestRelease();
|
||||
$release->update(['no_export' => true]);
|
||||
$service = app(PressReleaseService::class);
|
||||
|
||||
$service->markGone($release);
|
||||
$service->restoreFromGone($release->fresh());
|
||||
|
||||
// Umgekehrt gilt dasselbe: Eine schon vorher gesetzte Sperre bleibt.
|
||||
expect($release->fresh()->no_export)->toBeTrue();
|
||||
});
|
||||
|
||||
test('a press release marked gone disappears from listings and the sitemap', function () {
|
||||
/** @var TestCase $this */
|
||||
$company = goneTestCompany();
|
||||
$release = goneTestRelease($company);
|
||||
|
||||
$this->get('https://businessportal24.test/sitemap.xml')->assertSee($release->slug, false);
|
||||
|
||||
app(PressReleaseService::class)->markGone($release);
|
||||
|
||||
$this->get('https://businessportal24.test/sitemap.xml')->assertDontSee($release->slug, false);
|
||||
$this->get('https://businessportal24.test/de/newsrooms')->assertDontSeeText($company->name);
|
||||
});
|
||||
|
||||
test('admin can remove a press release with 410 from the edit page', function () {
|
||||
/** @var TestCase $this */
|
||||
$release = goneTestRelease();
|
||||
$this->actingAs(goneTestAdmin());
|
||||
|
||||
LivewireVolt::test('admin.press-releases.edit', ['id' => $release->id])
|
||||
->call('markGone')
|
||||
->assertRedirect(route('admin.press-releases.index'));
|
||||
|
||||
expect($release->fresh()->gone_at)->not->toBeNull();
|
||||
});
|
||||
|
||||
test('admin can remove a newsroom with 410 from the edit page', function () {
|
||||
/** @var TestCase $this */
|
||||
$company = goneTestCompany();
|
||||
$release = goneTestRelease($company);
|
||||
$this->actingAs(goneTestAdmin());
|
||||
|
||||
LivewireVolt::test('admin.companies.edit', ['id' => $company->id])
|
||||
->set('goneIncludesPressReleases', true)
|
||||
->call('markGone')
|
||||
->assertRedirect(route('admin.companies.index'));
|
||||
|
||||
$company->refresh();
|
||||
|
||||
expect($company->gone_at)->not->toBeNull()
|
||||
->and($company->is_active)->toBeFalse()
|
||||
->and($release->fresh()->gone_at)->not->toBeNull();
|
||||
});
|
||||
|
||||
test('admin can lift a press release removal from the edit page', function () {
|
||||
/** @var TestCase $this */
|
||||
$release = goneTestRelease();
|
||||
app(PressReleaseService::class)->markGone($release);
|
||||
$this->actingAs(goneTestAdmin());
|
||||
|
||||
LivewireVolt::test('admin.press-releases.edit', ['id' => $release->id])
|
||||
->assertSee('410-Entfernung aufheben')
|
||||
->call('restoreGone')
|
||||
->assertRedirect(route('admin.press-releases.edit', $release->id));
|
||||
|
||||
expect($release->fresh()->gone_at)->toBeNull();
|
||||
});
|
||||
|
||||
test('admin can lift a newsroom removal from the edit page', function () {
|
||||
/** @var TestCase $this */
|
||||
$company = goneTestCompany();
|
||||
app(CompanyRemovalService::class)->markGone($company);
|
||||
$this->actingAs(goneTestAdmin());
|
||||
|
||||
LivewireVolt::test('admin.companies.edit', ['id' => $company->id])
|
||||
->assertSee('410-Entfernung aufheben')
|
||||
->call('restoreGone')
|
||||
->assertRedirect(route('admin.companies.edit', $company->id));
|
||||
|
||||
$company->refresh();
|
||||
|
||||
expect($company->gone_at)->toBeNull()
|
||||
->and($company->is_active)->toBeTrue();
|
||||
});
|
||||
|
|
@ -41,7 +41,7 @@ test('homepage data is cached per portal and language', function () {
|
|||
->assertSuccessful()
|
||||
->assertSeeText('Erste Meldung im Cache');
|
||||
|
||||
expect(Cache::has('web.home.presseecho.de'))->toBeTrue();
|
||||
expect(Cache::has(webPageCacheKey('web.home.presseecho.de')))->toBeTrue();
|
||||
|
||||
PressRelease::factory()
|
||||
->published()
|
||||
|
|
@ -119,23 +119,23 @@ test('the press release feed caches its front pages but not deep pages or junk t
|
|||
->assertSuccessful()
|
||||
->assertSeeText('Feed-Meldung eins');
|
||||
|
||||
expect(Cache::has('web.feed.presseecho.de.all.page.1'))->toBeTrue();
|
||||
expect(Cache::has(webPageCacheKey('web.feed.presseecho.de.all.page.1')))->toBeTrue();
|
||||
|
||||
// Tiefe Seiten bleiben ungecacht (Schutz vor unbegrenztem Key-Raum).
|
||||
$this->get('https://presseecho.test/de?page=25')->assertSuccessful();
|
||||
|
||||
expect(Cache::has('web.feed.presseecho.de.all.page.25'))->toBeFalse();
|
||||
expect(Cache::has(webPageCacheKey('web.feed.presseecho.de.all.page.25')))->toBeFalse();
|
||||
|
||||
// Unbekannte timeframe-Werte aus der URL fallen auf "all" zurück und
|
||||
// erzeugen keinen eigenen (poisonbaren) Cache-Key.
|
||||
$this->get('https://presseecho.test/de?timeframe=boesewert')->assertSuccessful();
|
||||
|
||||
expect(Cache::has('web.feed.presseecho.de.boesewert.page.1'))->toBeFalse();
|
||||
expect(Cache::has(webPageCacheKey('web.feed.presseecho.de.boesewert.page.1')))->toBeFalse();
|
||||
|
||||
// Der Cache hält nur items + total — der Paginator (inkl. Host-URLs der
|
||||
// Pagination-Links) wird pro Request neu gebaut, damit der gecachte Feed
|
||||
// nicht den Host des Cache-Erzeugers an andere Besucher ausliefert.
|
||||
expect(Cache::get('web.feed.presseecho.de.all.page.1'))
|
||||
expect(Cache::get(webPageCacheKey('web.feed.presseecho.de.all.page.1')))
|
||||
->toBeArray()
|
||||
->toHaveKeys(['items', 'total']);
|
||||
});
|
||||
|
|
@ -156,18 +156,18 @@ test('category page data is cached per portal, language and slug but 404s are no
|
|||
|
||||
$this->get('https://businessportal24.test/de/category/mobilitaet')->assertSuccessful();
|
||||
|
||||
expect(Cache::has('web.category.businessportal24.de.mobilitaet'))->toBeTrue();
|
||||
expect(Cache::has(webPageCacheKey('web.category.businessportal24.de.mobilitaet')))->toBeTrue();
|
||||
|
||||
$this->get('https://businessportal24.test/de/category/gibt-es-nicht')->assertNotFound();
|
||||
|
||||
expect(Cache::has('web.category.businessportal24.de.gibt-es-nicht'))->toBeFalse();
|
||||
expect(Cache::has(webPageCacheKey('web.category.businessportal24.de.gibt-es-nicht')))->toBeFalse();
|
||||
});
|
||||
|
||||
test('search sidebar data is cached per portal and language', function () {
|
||||
/** @var TestCase $this */
|
||||
$this->get('https://businessportal24.test/de/suche')->assertSuccessful();
|
||||
|
||||
expect(Cache::has('web.search.most-read.businessportal24-both.de'))->toBeTrue();
|
||||
expect(Cache::has(webPageCacheKey('web.search.most-read.businessportal24-both.de')))->toBeTrue();
|
||||
});
|
||||
|
||||
test('newsroom detail data is cached per portal language slug and page', function () {
|
||||
|
|
@ -188,7 +188,7 @@ test('newsroom detail data is cached per portal language slug and page', functio
|
|||
$this->get('https://businessportal24.test/de/newsroom/cache-newsroom?page=2')
|
||||
->assertSuccessful();
|
||||
|
||||
expect(Cache::has('web.newsroom.businessportal24-both.de.cache-newsroom.page.2'))->toBeTrue();
|
||||
expect(Cache::has(webPageCacheKey('web.newsroom.businessportal24-both.de.cache-newsroom.page.2')))->toBeTrue();
|
||||
});
|
||||
|
||||
test('newsroom detail cache key aligns with the paginator so an invalid page cannot poison another page', function () {
|
||||
|
|
@ -211,8 +211,8 @@ test('newsroom detail cache key aligns with the paginator so an invalid page can
|
|||
$this->get('https://businessportal24.test/de/newsroom/poison-newsroom?page=2.5')
|
||||
->assertSuccessful();
|
||||
|
||||
expect(Cache::has('web.newsroom.businessportal24-both.de.poison-newsroom.page.1'))->toBeTrue();
|
||||
expect(Cache::has('web.newsroom.businessportal24-both.de.poison-newsroom.page.2'))->toBeFalse();
|
||||
expect(Cache::has(webPageCacheKey('web.newsroom.businessportal24-both.de.poison-newsroom.page.1')))->toBeTrue();
|
||||
expect(Cache::has(webPageCacheKey('web.newsroom.businessportal24-both.de.poison-newsroom.page.2')))->toBeFalse();
|
||||
});
|
||||
|
||||
test('newsrooms unfiltered index is cached but filtered searches are not', function () {
|
||||
|
|
@ -232,7 +232,7 @@ test('newsrooms unfiltered index is cached but filtered searches are not', funct
|
|||
->assertSuccessful()
|
||||
->assertSeeText('SolarTec Bestand');
|
||||
|
||||
expect(Cache::has('web.newsrooms.businessportal24,both.de.page.1'))->toBeTrue();
|
||||
expect(Cache::has(webPageCacheKey('web.newsrooms.businessportal24,both.de.page.1')))->toBeTrue();
|
||||
|
||||
// Gefilterte Suche wird NICHT gecacht (Schutz vor unbegrenztem Key-Raum):
|
||||
// eine nach dem ersten Aufruf angelegte Firma erscheint sofort.
|
||||
|
|
@ -259,7 +259,7 @@ test('deep newsrooms pages are not cached', function () {
|
|||
/** @var TestCase $this */
|
||||
$this->get('https://businessportal24.test/de/newsrooms?page=25')->assertSuccessful();
|
||||
|
||||
expect(Cache::has('web.newsrooms.businessportal24,both.de.page.25'))->toBeFalse();
|
||||
expect(Cache::has(webPageCacheKey('web.newsrooms.businessportal24,both.de.page.25')))->toBeFalse();
|
||||
});
|
||||
|
||||
test('newsrooms cache does not retain unrelated query parameters in pagination links', function () {
|
||||
|
|
|
|||
|
|
@ -3,6 +3,7 @@
|
|||
use App\Models\Plan;
|
||||
use App\Models\User;
|
||||
use App\Services\CurrentPortalContext;
|
||||
use App\Support\WebPageCache;
|
||||
use Illuminate\Foundation\Testing\RefreshDatabase;
|
||||
use Tests\TestCase;
|
||||
|
||||
|
|
@ -74,3 +75,14 @@ function subscribeUserToPlan(User $user, Plan $plan, string $interval = 'monthly
|
|||
'quantity' => 1,
|
||||
]);
|
||||
}
|
||||
|
||||
/**
|
||||
* Seiten-Cache-Keys tragen seit der Entfernung mit 410 eine Generations-Nummer
|
||||
* (siehe WebPageCache::key): Ein Flush erhöht sie und verwirft damit alle
|
||||
* `web.`-Seiten-Caches auf einmal. Die Tests prüfen weiterhin die fachliche
|
||||
* Key-Struktur, nur eben generationsunabhängig.
|
||||
*/
|
||||
function webPageCacheKey(string $key): string
|
||||
{
|
||||
return 'web.g'.WebPageCache::generation().'.'.substr($key, strlen('web.'));
|
||||
}
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue