6807: Fix online clan friends not being removable via UI r=edg-l a=Robyt3



## Checklist

- [X] Tested the change ingame
- [ ] Provided screenshots if it is a visual change
- [ ] Tested in combination with possibly related configuration options
- [ ] Written a unit test (especially base/) or added coverage to integration test
- [ ] Considered possible null pointers and out of bounds array indexing
- [ ] Changed no physics that affect existing maps
- [ ] Tested the change with [ASan+UBSan or valgrind's memcheck](https://github.com/ddnet/ddnet/#using-addresssanitizer--undefinedbehavioursanitizer-or-valgrinds-memcheck) (optional)


Co-authored-by: Robert Müller <robytemueller@gmail.com>
This commit is contained in:
bors[bot] 2023-07-04 19:20:39 +00:00 committed by GitHub
commit c555d049d4
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -1645,7 +1645,7 @@ void CMenus::RenderServerbrowserFriends(CUIRect View)
void CMenus::PopupConfirmRemoveFriend() void CMenus::PopupConfirmRemoveFriend()
{ {
m_pClient->Friends()->RemoveFriend(m_pRemoveFriend->Name(), m_pRemoveFriend->Clan()); m_pClient->Friends()->RemoveFriend(m_pRemoveFriend->FriendState() == IFriends::FRIEND_PLAYER ? m_pRemoveFriend->Name() : "", m_pRemoveFriend->Clan());
FriendlistOnUpdate(); FriendlistOnUpdate();
Client()->ServerBrowserUpdate(); Client()->ServerBrowserUpdate();
m_pRemoveFriend = nullptr; m_pRemoveFriend = nullptr;