feat: Do not update names of deleted users (WPB-14492)

src/i18n/en-US.json

@@ -650,6 +650,7 @@
   "dataSharingModalDecline": "Decline",
   "dataSharingModalDescription": "Help to improve Wire by sharing your usage data via a pseudonymous ID. The data is neither linked to your personal information nor shared with third parties besides Wire Group. It includes, for example, when you use a feature, your app version, device type, or your operating system. This data will be deleted at the latest after 365 days. <br /> Find further details in our [link]Privacy Policy[/link]. You can revoke your consent at any time.",
   "dataSharingModalTitle": "Consent to share user data",
+  "deletedUser": "Deleted User",
   "downloadLatestMLS": "Download the latest MLS Wire version",
   "enumerationAnd": ", and ",
   "ephemeralRemaining": "remaining",

src/script/user/UserRepository.ts

@@ -216,26 +216,36 @@ export class UserRepository extends TypedEventEmitter<Events> {
     // the entries we get back will be used to feed the availabilities of those users
     const nonQualifiedUsers = await this.userService.clearNonQualifiedUsers();
 
-    const dbUsers = await this.userService.loadUserFromDb();
+    const dbUsers = await this.userService.loadUsersFromDb();
 
     // The self user doesn't need to be re-fetched
     const usersToFetch = users.filter(user => !matchQualifiedIds(selfUser.qualifiedId, user));
 
     const {found, failed} = await this.fetchRawUsers(usersToFetch, selfUser.domain);
 
-    const userWithAvailability = found.map(user => {
-      const availability = [...dbUsers, ...nonQualifiedUsers].find(userRecord => userRecord.id === user.id);
+    const usersWithAvailability = found.map(user => {
+      const localUser = dbUsers.find(
+        dbUser => dbUser.id === user.id || matchQualifiedIds(dbUser.qualified_id, user.qualified_id),
+      );
+
+      const userWithAvailability = [...dbUsers, ...nonQualifiedUsers].find(userRecord => userRecord.id === user.id);
+
+      const userWithEscapedDefaultName = this.replaceDeletedUserNameWithNameInDb(user, localUser);
 
-      if (availability) {
-        return {availability: availability.availability, ...user};
+      if (userWithAvailability) {
+        return {
+          availability: userWithAvailability.availability,
+          ...userWithEscapedDefaultName,
+        };
       }
-      return user;
+
+      return userWithEscapedDefaultName;
     });
 
     // Save all new users to the database
-    await Promise.all(userWithAvailability.map(user => this.saveUserInDb(user)));
+    await Promise.all(usersWithAvailability.map(user => this.saveUserInDb(user)));
 
-    const mappedUsers = this.mapUserResponse(userWithAvailability, failed, dbUsers);
+    const mappedUsers = this.mapUserResponse(usersWithAvailability, failed, dbUsers);
 
     // Assign connections to users
     mappedUsers.forEach(user => {
@@ -323,6 +333,14 @@ export class UserRepository extends TypedEventEmitter<Events> {
    * Will update the user both in database and in memory.
    */
   private async updateUser(userId: QualifiedId, user: Partial<UserRecord>, isWebSocket = false): Promise<User> {
+    if (user.deleted && user.name) {
+      const dbUser = await this.userService.loadUserFromDb(userId);
+
+      if (dbUser && dbUser.name) {
+        user.name = dbUser?.name;
+      }
+    }
+
     const selfUser = this.userState.self();
     const isSelfUser = matchQualifiedIds(userId, selfUser.qualifiedId);
     const userEntity = isSelfUser ? selfUser : await this.getUserById(userId);
@@ -575,23 +593,19 @@ export class UserRepository extends TypedEventEmitter<Events> {
   }
 
   // Replaces a deleted user name ("default") with the name from the local database.
-  private restoreDeletedUserNames(apiUsers: APIClientUser[], dbUsers: UserRecord[]): UserRecord[] {
-    return apiUsers.map(user => {
-      if (!user.deleted) {
-        return user;
-      }
-
-      const localUser = dbUsers.find(dbUser => dbUser.id === user.id);
+  private replaceDeletedUserNameWithNameInDb(user: APIClientUser, localUser?: UserRecord): UserRecord {
+    if (!user.deleted) {
+      return user;
+    }
 
-      if (localUser && localUser.name) {
-        return {
-          ...user,
-          name: localUser.name,
-        };
-      }
+    if (localUser && localUser.name) {
+      return {
+        ...user,
+        name: localUser.name,
+      };
+    }
 
-      return user;
-    });
+    return {...user, name: t('deletedUser')};
   }
 
   private mapUserResponse(found: APIClientUser[], failed: QualifiedId[], dbUsers: UserRecord[]): User[] {
@@ -615,9 +629,7 @@ export class UserRepository extends TypedEventEmitter<Events> {
       return new User(userId.id, userId.domain);
     });
 
-    const users = this.restoreDeletedUserNames(found, dbUsers);
-
-    const mappedUsers = this.userMapper.mapUsersFromJson(users, selfDomain).concat(failedToLoad);
+    const mappedUsers = this.userMapper.mapUsersFromJson(found, selfDomain).concat(failedToLoad);
 
     if (this.teamState.isTeam()) {
       this.mapGuestStatus(mappedUsers);
@@ -633,7 +645,7 @@ export class UserRepository extends TypedEventEmitter<Events> {
    */
   private async fetchUsers(userIds: QualifiedId[]): Promise<User[]> {
     const {found, failed} = await this.fetchRawUsers(userIds, this.userState.self().domain);
-    const dbUsers = await this.userService.loadUserFromDb();
+    const dbUsers = await this.userService.loadUsersFromDb();
     const users = this.mapUserResponse(found, failed, dbUsers);
 
     let fetchedUserEntities = this.saveUsers(users);
@@ -713,6 +725,7 @@ export class UserRepository extends TypedEventEmitter<Events> {
     if (localOnly) {
       const deletedUser = new User(userId.id, userId.domain);
       deletedUser.isDeleted = true;
+      deletedUser.name(t('deletedUser'));
       return deletedUser;
     }
     try {

src/script/user/UserService.ts

@@ -58,10 +58,20 @@ export class UserService {
    * @todo There might be more keys which are returned by this function
    * @returns Resolves with all the stored user states
    */
-  loadUserFromDb(): Promise<UserRecord[]> {
+  loadUsersFromDb(): Promise<UserRecord[]> {
     return this.storageService.getAll<UserRecord>(this.USER_STORE_NAME);
   }
 
+  /**
+   * Loads user from the local database
+   * @returns Resolves to the stored user, if it does not exists resolves to undefined
+   */
+  loadUserFromDb(qualifiedId: QualifiedId): Promise<UserRecord | undefined> {
+    const primaryKey = constructUserPrimaryKey(qualifiedId);
+
+    return this.storageService.load<UserRecord>(this.USER_STORE_NAME, primaryKey);
+  }
+
   /**
    * Will remove all the non-qualified entries in the DB
    */

src/types/i18n.d.ts

@@ -443,11 +443,11 @@ declare module 'I18n/en-US.json' {
     'conversationDetailsActionConversationParticipants': `Show all ({number})`;
     'conversationDetailsActionCreateGroup': `Create group`;
     'conversationDetailsActionDelete': `Delete group`;
+    'conversationDetailsActionDeleteForMe': `Delete for Me`;
     'conversationDetailsActionDevices': `Devices`;
     'conversationDetailsActionGuestOptions': `Guests`;
     'conversationDetailsActionLeave': `Leave group`;
     'conversationDetailsActionNotifications': `Notifications`;
-    'conversationDetailsActionDeleteForMe': `Delete for Me`;
     'conversationDetailsActionServicesOptions': `Services`;
     'conversationDetailsActionTimedMessages': `Self-deleting messages`;
     'conversationDetailsActionUnblock': `Unblock`;
@@ -654,6 +654,7 @@ declare module 'I18n/en-US.json' {
     'dataSharingModalDecline': `Decline`;
     'dataSharingModalDescription': `Help to improve Wire by sharing your usage data via a pseudonymous ID. The data is neither linked to your personal information nor shared with third parties besides Wire Group. It includes, for example, when you use a feature, your app version, device type, or your operating system. This data will be deleted at the latest after 365 days. <br /> Find further details in our [link]Privacy Policy[/link]. You can revoke your consent at any time.`;
     'dataSharingModalTitle': `Consent to share user data`;
+    'deletedUser': `Deleted User`;
     'downloadLatestMLS': `Download the latest MLS Wire version`;
     'enumerationAnd': `, and `;
     'ephemeralRemaining': `remaining`;