diff options
-rw-r--r-- | lib/user.cpp | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/lib/user.cpp b/lib/user.cpp index 7143620f..f0831733 100644 --- a/lib/user.cpp +++ b/lib/user.cpp @@ -171,13 +171,11 @@ void User::rename(const QString& newName, const Room* r) rename(newName); return; } - Q_ASSERT_X(r->memberJoinState(this) == JoinState::Join, __FUNCTION__, + // #481: take the current state and update it with the new name + auto evtC = r->getCurrentState<RoomMemberEvent>(id())->content(); + Q_ASSERT_X(evtC.membership == MembershipType::Join, __FUNCTION__, "Attempt to rename a user that's not a room member"); - const auto actualNewName = sanitized(newName); - MemberEventContent evtC; - evtC.displayName = actualNewName; - // #481: fill in the current avatar URL in order to not clear it out - evtC.avatarUrl = r->getCurrentState<RoomMemberEvent>(id())->avatarUrl(); + evtC.displayName = sanitized(newName); r->setState<RoomMemberEvent>(id(), move(evtC)); // The state will be updated locally after it arrives with sync } |