Tweak intialisation

src/models/messagecontentmodel.cpp

@@ -63,7 +63,6 @@ void MessageContentModel::initializeModel()
     Quotient::connectUntil(m_room.get(), &NeoChatRoom::extraEventLoaded, this, [this](const QString &eventId) {
         if (m_room != nullptr) {
             if (eventId == m_eventId) {
-                intiializeEvent(eventId);
                 updateReplyModel();
                 resetContent();
                 return true;
@@ -169,7 +168,9 @@ void MessageContentModel::intiializeEvent(const Quotient::RoomEvent *event)
     if (senderId.isEmpty()) {
         senderId = m_room->localMember().id();
     }
-    m_eventSenderObject = std::unique_ptr<NeochatRoomMember>(new NeochatRoomMember(m_room, senderId));
+    if (m_eventSenderObject == nullptr) {
+        m_eventSenderObject = std::unique_ptr<NeochatRoomMember>(new NeochatRoomMember(m_room, senderId));
+    }
     Q_EMIT eventUpdated();
 }
 

src/models/messagecontentmodel.h

@@ -117,7 +117,7 @@ private:
     QPointer<NeoChatRoom> m_room;
     QString m_eventId;
     QString m_eventSenderId;
-    std::unique_ptr<NeochatRoomMember> m_eventSenderObject;
+    std::unique_ptr<NeochatRoomMember> m_eventSenderObject = nullptr;
     Quotient::RoomEventPtr m_event;
 
     bool m_isPending;

src/neochatroommember.cpp

@@ -9,6 +9,8 @@ NeochatRoomMember::NeochatRoomMember(NeoChatRoom *room, const QString &memberId)
     : m_room(room)
     , m_memberId(memberId)
 {
+    Q_ASSERT(!m_memberId.isEmpty());
+
     if (m_room != nullptr) {
         connect(m_room, &NeoChatRoom::memberNameUpdated, this, [this](Quotient::RoomMember member) {
             if (member.id() == m_memberId) {

src/neochatroommember.h

@@ -80,5 +80,5 @@ Q_SIGNALS:
 
 private:
     QPointer<NeoChatRoom> m_room;
-    const QString m_memberId;
+    const QString m_memberId = QString();
 };