SocialNetworkFeedItem::getSocialNetworkProfile() gibt null zurück trotz non-nullable Return

[maltehuebner]

Beschreibung

getSocialNetworkProfile() hat den Rückgabetyp SocialNetworkProfile (nicht nullable), gibt aber null zurück, wenn die Beziehung in der DB null ist.

Fehlermeldung

TypeError: App\Entity\SocialNetworkFeedItem::getSocialNetworkProfile():
Return value must be of type App\Entity\SocialNetworkProfile, null returned
at SocialNetworkFeedItem.php line 83

Häufigkeit

Tritt regelmäßig auf — es gibt FeedItems in der DB ohne zugehöriges SocialNetworkProfile.

Zusammenhang

Dieses Problem hängt mit Issue #1289 zusammen (null tabName durch null Profile). Die NOT NULL Migration (#1287) wurde zwar erstellt, aber offenbar gibt es noch alte Daten ohne Profile.

Betroffene Dateien

• src/Entity/SocialNetworkFeedItem.php:83

Lösungsvorschlag

1. Rückgabetyp auf ?SocialNetworkProfile ändern
2. Alte Daten bereinigen: FeedItems ohne Profile löschen oder zuordnen
3. Nach der Datenbereinigung den NOT NULL Constraint sicherstellen

🤖 Generated with Claude Code

[maltehuebner]

Dieser Bug wurde bereits auf main behoben — der Return-Type von getSocialNetworkProfile() ist bereits ?SocialNetworkProfile.

[maltehuebner]

Untersuchungsergebnis

Der Bug ist teilweise gefixt auf main:

Bereits gefixt:

• Return-Type von getSocialNetworkProfile() ist jetzt ?SocialNetworkProfile (Commit 8d709b36d, PR Fix TypeError for nullable SocialNetworkProfile #1286) ✅
• Migration Version20260228120000 löscht verwaiste FeedItems und setzt NOT NULL Constraint (PR Fix API: require profile when creating feed items #1287) ✅
• SocialNetworkTwigExtension nutzt nullsafe Operator ?-> ✅

Noch offen:

• SocialNetworkFeedItemItem::setSocialNetworkFeedItem() (Zeile 20) weist null dem string-Property $tabName zu, wenn getSocialNetworkProfile() null zurückgibt
• Dieser verbleibende Bug wird durch PR Fix null TypeError in SocialNetworkFeedItemItem tabName #1320 (Issue TypeError: null assigned to string $tabName in SocialNetworkFeedItemItem #1289) behoben — ?? 'standard' Fallback

Lösung

PR #1320 mergen.