refactor: cleanup code

Author: BreadGenie

meet/api/meeting.py

@@ -245,9 +245,6 @@ def refresh_sfu_token(meeting_id: str) -> dict:
 	"""
 	meeting: SaeMeeting = frappe.get_doc("Sae Meeting", meeting_id)
 
-	if not meeting.is_active:
-		frappe.throw(_("Meeting has ended"))
-
 	if frappe.session.user not in meeting.get_members():
 		frappe.throw(_("Not a meeting member"))
 

meet/meet/doctype/sae_meeting/sae_meeting.py

@@ -40,11 +40,6 @@ def autoname(self):
 		if not self.name:
 			self.name = generate()
 
-	def before_insert(self):
-		"""Initialize meeting room"""
-		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 []
@@ -84,26 +79,6 @@ def join(self, user=None):
 
 		return {"status": "joined", "message": "Successfully joined the meeting"}
 
-	def leave(self, user=None):
-		"""
-		Leave the meeting room
-
-		Args:
-			user: User to remove (defaults to current session user)
-		"""
-		if not user:
-			user = frappe.session.user
-
-		members = self.get_members()
-
-		if user in members:
-			members.remove(user)
-			self.update_members(members)
-
-			if not members:
-				self.is_active = 0
-				self.save(ignore_permissions=True)
-
 	def get_members(self):
 		"""Get list of current members"""
 		return [row.user for row in self.members] if self.members else []
@@ -128,14 +103,6 @@ def can_join(self, user=None):
 		if self.is_user_banned(user):
 			return False
 
-		# Check if meeting is active
-		# if not self.get("is_active", True):
-		# 	return False
-
-		# Check if user has read permission on the meeting
-		# if not frappe.has_permission("Sae Meeting", "read", self):
-		# 	return False
-
 		return True
 
 	def update_members(self, members_list):