ezqms-730: better check for roles when changing members (#5527)

Signed-off-by: Alexey Zinoviev <[email protected]>

Author: lexiv0re

Diff for: plugins/document-resources/src/components/teamspace/CreateTeamspace.svelte

@@ -242,8 +242,7 @@
 
     if (removedMembersSet.size > 0 && rolesAssignment !== undefined) {
       for (const [key, value] of Object.entries(rolesAssignment)) {
-        rolesAssignment[key as Ref<Role>] =
-          value !== undefined ? value.filter((m) => !removedMembersSet.has(m)) : undefined
+        rolesAssignment[key as Ref<Role>] = value != null ? value.filter((m) => !removedMembersSet.has(m)) : undefined
       }
     }
 

Diff for: plugins/lead-resources/src/components/CreateFunnel.svelte

@@ -140,8 +140,7 @@
 
     if (removedMembersSet.size > 0 && rolesAssignment !== undefined) {
       for (const [key, value] of Object.entries(rolesAssignment)) {
-        rolesAssignment[key as Ref<Role>] =
-          value !== undefined ? value.filter((m) => !removedMembersSet.has(m)) : undefined
+        rolesAssignment[key as Ref<Role>] = value != null ? value.filter((m) => !removedMembersSet.has(m)) : undefined
       }
     }
 

Diff for: plugins/tracker-resources/src/components/projects/CreateProject.svelte

@@ -302,8 +302,7 @@
 
     if (removedMembersSet.size > 0 && rolesAssignment !== undefined) {
       for (const [key, value] of Object.entries(rolesAssignment)) {
-        rolesAssignment[key as Ref<Role>] =
-          value !== undefined ? value.filter((m) => !removedMembersSet.has(m)) : undefined
+        rolesAssignment[key as Ref<Role>] = value != null ? value.filter((m) => !removedMembersSet.has(m)) : undefined
       }
     }