fix: don't duplicate user values in the user child tables

Author: BreadGenie

meet/meet/doctype/sae_meeting/sae_meeting.py

@@ -12,6 +12,7 @@
 from meet.utils.user import (
 	get_guest_session,
 	get_user_info,
+	unique_users,
 )
 
 
@@ -43,6 +44,12 @@ def before_insert(self):
 		if not hasattr(self, "is_active"):
 			self.is_active = 1
 
+	def validate(self):
+		"""Ensure unique users in all child tables"""
+		self.members = unique_users(self.members) if self.members else []
+		self.waiting_room = unique_users(self.waiting_room) if self.waiting_room else []
+		self.banned_users = unique_users(self.banned_users) if self.banned_users else []
+
 	def after_insert(self):
 		self.join(frappe.session.user)
 

meet/utils/user.py

@@ -8,6 +8,17 @@
 from frappe.core.doctype.user.user import User
 
 
+def unique_users(user_list: list[dict]) -> list[str]:
+	"""Return a list of unique user IDs, preserving order."""
+	seen = set()
+	unique_list = []
+	for user in user_list:
+		if user not in seen:
+			seen.add(user)
+			unique_list.append(user)
+	return unique_list
+
+
 def assign_meet_role(user: User, method: str) -> None:
 	"""Assign the "Meet User" role to a newly created User."""
 	role_name = "Meet User"