fix: group control messages issues since strings

SES-2666

ts/models/groupUpdate.ts

@@ -29,7 +29,7 @@ export function getKickedGroupUpdateStr(kicked: Array<string>, groupName: string
     }
   }
 
-  switch (others.length) {
+  switch (othersNames.length) {
     case 0:
       throw new Error('kicked without anyone in it.');
     case 1:
@@ -46,7 +46,7 @@ export function getKickedGroupUpdateStr(kicked: Array<string>, groupName: string
       return {
         token: 'groupRemovedMultiple',
         args: {
-          name: others[0],
+          name: othersNames[0],
           count: othersNames.length - 1,
         },
       };
@@ -83,7 +83,7 @@ export function getJoinedGroupUpdateChangeStr(
   );
 
   if (us) {
-    switch (others.length) {
+    switch (othersNames.length) {
       case 0:
         return { token: 'legacyGroupMemberYouNew' };
       case 1:
@@ -92,7 +92,7 @@ export function getJoinedGroupUpdateChangeStr(
         return { token: 'legacyGroupMemberNewYouMultiple', args: { count: othersNames.length } };
     }
   }
-  switch (others.length) {
+  switch (othersNames.length) {
     case 0:
       throw new Error('joined without anyone in it.');
     case 1:
@@ -109,7 +109,7 @@ export function getJoinedGroupUpdateChangeStr(
       return {
         token: 'legacyGroupMemberNewMultiple',
         args: {
-          name: others[0],
+          name: othersNames[0],
           count: othersNames.length - 1,
         },
       };

ts/receiver/contentMessage.ts

@@ -20,7 +20,7 @@ import { removeMessagePadding } from '../session/crypto/BufferPadding';
 import { DisappearingMessages } from '../session/disappearing_messages';
 import { ReadyToDisappearMsgUpdate } from '../session/disappearing_messages/types';
 import { ProfileManager } from '../session/profile_manager/ProfileManager';
-import { GroupUtils, UserUtils } from '../session/utils';
+import { UserUtils } from '../session/utils';
 import { perfEnd, perfStart } from '../session/utils/Performance';
 import { fromHexToArray, toHex } from '../session/utils/String';
 import { isUsFromCache } from '../session/utils/User';
@@ -69,10 +69,7 @@ async function decryptForClosedGroup(envelope: EnvelopePlus) {
   window?.log?.info('received closed group message');
   try {
     const hexEncodedGroupPublicKey = envelope.source;
-    if (!GroupUtils.isClosedGroup(PubKey.cast(hexEncodedGroupPublicKey))) {
+
-      window?.log?.warn('received medium group message but not for an existing medium group');
-      throw new Error('Invalid group public key'); // invalidGroupPublicKey
-    }
     const encryptionKeyPairs = await getAllCachedECKeyPair(hexEncodedGroupPublicKey);
 
     const encryptionKeyPairsCount = encryptionKeyPairs?.length;

ts/session/utils/Groups.ts

@@ -13,16 +13,6 @@ export function getGroupMembers(groupId: PubKey): Array<PubKey> {
   return groupMembers.map(PubKey.cast);
 }
 
-export function isClosedGroup(groupId: PubKey): boolean {
-  const conversation = getConversationController().get(groupId.key);
-
-  if (!conversation) {
-    return false;
-  }
-
-  return Boolean(conversation.isClosedGroup());
-}
-
 export function encodeGroupPubKeyFromHex(hexGroupPublicKey: string | PubKey) {
   const pubkey = PubKey.cast(hexGroupPublicKey);
   return fromHexToArray(pubkey.key);

ts/util/i18n/localizedString.ts

@@ -277,7 +277,7 @@ export class LocalizedStringBuilder<
     /** Find and replace the dynamic variables in a localized string and substitute the variables with the provided values */
     return str.replace(/\{(\w+)\}/g, (match, arg: string) => {
       const matchedArg = this.args
-        ? this.args[arg as keyof StringArgsRecord<Dict[T]>].toString()
+        ? this.args[arg as keyof StringArgsRecord<Dict[T]>]?.toString()
         : undefined;
 
       return matchedArg ?? LOCALE_DEFAULTS[arg as keyof typeof LOCALE_DEFAULTS] ?? match;