Kevin Adametz
e746deda4f
phpstan lief mit 390 Meldungen außerhalb der Baseline, der größte Teil davon Falschmeldungen. Ursache: Larastan löste keinen einzigen Cast auf, weil alle Models ihre Casts über die casts()-METHODE deklarieren (Laravel-11-Stil). Laravel führt die erst zur Laufzeit mit $casts zusammen, Larastan instanziiert Models aber ohne Konstruktor — $pressRelease->status galt deshalb als string und jeder Enum-Vergleich als "immer falsch". Dafür gibt es den Schalter parseModelCastsMethod, dessen Larastan-Default false ist. Zweite Sammelursache waren fehlende Generics an den Relations-Methoden: Ohne sie landete phpstan auf Illuminate\Database\Eloquent\Model und meldete jede Modell-Eigenschaft und jeden eigenen Scope als unbekannt. Alle 102 Relationen tragen jetzt ihr @return <Related, $this>; das verbessert nebenbei die Vervollständigung in der IDE. Zusammen fällt die Zahl von 390 auf 169. Die neu erzeugte Baseline umfasst 166 Einträge, überwiegend Closure-Parameter, die als blankes Builder typisiert sind, sowie zwei nachgeprüfte Falschmeldungs-Gruppen: die Concerns unter app/Livewire werden ausschließlich aus Volt-Komponenten eingebunden, die phpstan nicht analysiert, und Cashier typisiert findBillable() gegen den TRAIT Billable, was phpstan nicht mit User vereinbaren kann — die Laufzeitprüfung dort ist richtig und bleibt. Was die Analyse an echten toten Zweigen zutage gefördert hat, ist mit entfernt: eine doppelte Null-Prüfung im Nachweis-PDF, Prüfungen gegen NOT-NULL-Spalten in der Rechnungsstellung und im Vorschau-Controller, eine immer wahre Bedingung im Logo-Pfad und in RetireAgencyCompanyType, ein überflüssiges ?? [] auf einer Symfony-Methode, die immer ein Array liefert, sowie der unerreichbare null-Arm in publicPortalLinks(). MigrateLegacyMedia deklariert seine Medientypen jetzt als Literal-Union, womit das match ohne default nachweislich vollständig ist statt nur faktisch. Damit das nicht wieder abdriftet — die Baseline stand seit Mai unverändert — läuft die Analyse ab jetzt als eigener Schritt in der Test-Pipeline. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
96 lines
2.4 KiB
PHP
96 lines
2.4 KiB
PHP
<?php
|
|
|
|
namespace App\Models;
|
|
|
|
use App\Enums\Portal;
|
|
use App\Scopes\PortalScope;
|
|
use Database\Factories\ContactFactory;
|
|
use Illuminate\Database\Eloquent\Builder;
|
|
use Illuminate\Database\Eloquent\Factories\HasFactory;
|
|
use Illuminate\Database\Eloquent\Model;
|
|
use Illuminate\Database\Eloquent\Relations\BelongsTo;
|
|
use Illuminate\Database\Eloquent\Relations\BelongsToMany;
|
|
use Illuminate\Database\Eloquent\SoftDeletes;
|
|
|
|
class Contact extends Model
|
|
{
|
|
/** @use HasFactory<ContactFactory> */
|
|
use HasFactory, SoftDeletes;
|
|
|
|
protected static function booted(): void
|
|
{
|
|
static::addGlobalScope(new PortalScope);
|
|
}
|
|
|
|
/**
|
|
* Kontakte einer Firma, portal-unabhängig (ohne GlobalScope) und in der
|
|
* fürs Kunden-Panel üblichen Sortierung. Single Source of Truth für die
|
|
* Kontaktauswahl in Erstellen/Bearbeiten und für Kontakt-Zählungen.
|
|
*/
|
|
public function scopeForCompany(Builder $query, int $companyId): Builder
|
|
{
|
|
return $query->withoutGlobalScopes()
|
|
->where('company_id', $companyId)
|
|
->orderBy('last_name')
|
|
->orderBy('first_name');
|
|
}
|
|
|
|
protected $fillable = [
|
|
'company_id',
|
|
'portal',
|
|
'salutation_key',
|
|
'title',
|
|
'first_name',
|
|
'last_name',
|
|
'responsibility',
|
|
'phone',
|
|
'fax',
|
|
'email',
|
|
'legacy_portal',
|
|
'legacy_id',
|
|
];
|
|
|
|
protected function casts(): array
|
|
{
|
|
return [
|
|
'portal' => Portal::class,
|
|
'deleted_at' => 'datetime',
|
|
];
|
|
}
|
|
|
|
/**
|
|
* @return BelongsTo<Company, $this>
|
|
*/
|
|
public function company(): BelongsTo
|
|
{
|
|
return $this->belongsTo(Company::class);
|
|
}
|
|
|
|
/**
|
|
* Anzeigename eines Kontakts (Vor- + Nachname), mit Fallback, wenn beides
|
|
* leer ist. Single Source of Truth für die Listen-/Detail-Ansichten.
|
|
*/
|
|
public function displayName(): string
|
|
{
|
|
$name = trim(($this->first_name ?? '').' '.($this->last_name ?? ''));
|
|
|
|
return $name !== '' ? $name : __('Kontakt ohne Name');
|
|
}
|
|
|
|
/**
|
|
* @return BelongsToMany<PressRelease, $this>
|
|
*/
|
|
public function pressReleases(): BelongsToMany
|
|
{
|
|
return $this->belongsToMany(PressRelease::class, 'press_release_contact');
|
|
}
|
|
|
|
/**
|
|
* @return BelongsToMany<User, $this>
|
|
*/
|
|
public function users(): BelongsToMany
|
|
{
|
|
return $this->belongsToMany(User::class)
|
|
->withTimestamps();
|
|
}
|
|
}
|