Qfix: adding/deleting social ids (#9466)

Signed-off-by: Alexey Zinoviev <alexey.zinoviev@xored.com>

plugins/setting-resources/src/components/socialIds/AddEmailSocialId.svelte

@@ -14,11 +14,12 @@
 -->
 <script lang="ts">
   import { createEventDispatcher } from 'svelte'
-  import contact, { SocialIdentityProvider } from '@hcengineering/contact'
+  import contact, { getCurrentEmployee, SocialIdentityProvider, SocialIdentityRef } from '@hcengineering/contact'
   import { EditBox, Label, Button, CodeForm, TimeLeft, Status as StatusControl } from '@hcengineering/ui'
   import { OtpInfo } from '@hcengineering/account-client'
-  import { getCurrentAccount, setCurrentAccount, SocialId, Timestamp } from '@hcengineering/core'
+  import { buildSocialIdString, getCurrentAccount, setCurrentAccount, SocialId, Timestamp } from '@hcengineering/core'
   import { OK, PlatformError, Severity, Status, unknownError } from '@hcengineering/platform'
+  import { getClient } from '@hcengineering/presentation'
 
   import AddSocialId from './AddSocialId.svelte'
   import setting from '../../plugin'
@@ -31,6 +32,7 @@
   let otpInfo: OtpInfo | null = null
 
   const dispatch = createEventDispatcher()
+  const client = getClient()
   const accountClient = getAccountClient()
   const codeFields = [
     { id: 'code-1', name: 'code-1', optional: false },
@@ -89,6 +91,7 @@
 
     try {
       const newSocialIdLoginInfo = await accountClient.validateOtp(email, code, undefined, 'verify')
+      const currPerson = getCurrentEmployee()
       const currAcc = getCurrentAccount()
       const socialIds = await accountClient.getSocialIds()
 
@@ -98,6 +101,36 @@
         return
       }
 
+      // Create/update new social identity in the workspace
+      const existing = await client.findOne(contact.class.SocialIdentity, {
+        _id: newSocialIdLoginInfo.socialId as SocialIdentityRef
+      })
+      if (existing != null) {
+        // It can only exist if it were attached to an existing person and now this person was merged into the current account
+        if (existing.attachedTo !== currPerson) {
+          await client.updateDoc(contact.class.SocialIdentity, contact.space.Contacts, existing._id, {
+            attachedTo: currPerson,
+            verifiedOn: newSocialId.verifiedOn
+          })
+        }
+      } else {
+        await client.addCollection(
+          contact.class.SocialIdentity,
+          contact.space.Contacts,
+          currPerson,
+          contact.class.Person,
+          'socialIds',
+          {
+            type: newSocialId.type,
+            value: newSocialId.value,
+            key: buildSocialIdString(newSocialId), // TODO: fill it in trigger or on DB level as stored calculated column or smth?
+            verifiedOn: newSocialId.verifiedOn,
+            isDeleted: newSocialId.isDeleted
+          },
+          newSocialId._id as SocialIdentityRef
+        )
+      }
+
       setCurrentAccount({
         ...currAcc,
         fullSocialIds: socialIds,

plugins/setting-resources/src/components/socialIds/SocialIdPresenter.svelte

@@ -44,7 +44,11 @@
 
   async function doRelease (): Promise<void> {
     try {
-      const currSocialIds = currAcc.fullSocialIds
+      // Important to always get current account in the callback to avoid race conditions
+      // with the account from the reactive variable
+      // W/o it it was bringing back previously deleted social id if two are deleted in a row
+      const currentAccount = getCurrentAccount()
+      const currSocialIds = currentAccount.fullSocialIds
 
       const deletedSocialId = await accountClient.releaseSocialId(undefined, socialId.type, socialId.value, true)
 
@@ -65,12 +69,14 @@
       }
       const newPrimarySocialId = pickPrimarySocialId(newSocialIds)._id
 
-      setCurrentAccount({
-        ...currAcc,
+      // Don't need to update social ids because deleted social id retains its' _id
+      const updatedAccount = {
+        ...currentAccount,
         fullSocialIds: newSocialIds,
         primarySocialId: newPrimarySocialId
-      })
-      currAcc = getCurrentAccount()
+      }
+      setCurrentAccount(updatedAccount)
+      currAcc = updatedAccount
       dispatch('released')
     } catch (err: any) {
       console.error(err)
@@ -108,7 +114,7 @@
   <div class="icon"><Icon size="full" {icon} /></div>
 
   <div class="flex-col flex-gap-0-5">
-    <div>{socialId.value}</div>
+    <div>{socialId.displayValue ?? socialId.value}</div>
     <div class="type"><Label label={socialIdProvider.label} /></div>
   </div>
 

plugins/setting-resources/src/components/socialIds/SocialIdsEditor.svelte

@@ -79,7 +79,7 @@
             : loginSocialTypes.includes(socialId.type)
               ? !onlyLogin
               : true}
-        {#if socialIdProvider}
+        {#if socialIdProvider != null && socialId.type !== SocialIdType.HULY}
           <div class="item">
             <SocialIdPresenter {socialId} {socialIdProvider} {canRelease} on:released={handleAccountUpdated} />
           </div>