ref: Remove the "robot" flag.

jitsi · Dec 10, 2024 · 9eb6c5e · 9eb6c5e
1 parent 707414f
commit 9eb6c5e
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 19 deletions.
diff --git a/jicofo-common/src/main/kotlin/org/jitsi/jicofo/xmpp/muc/ChatRoomMember.kt b/jicofo-common/src/main/kotlin/org/jitsi/jicofo/xmpp/muc/ChatRoomMember.kt
@@ -53,7 +53,6 @@ interface ChatRoomMember {
     /** The last [Presence] packet received for this member (or null it if no presence has been received yet) */
     val presence: Presence?
 
-    val isRobot: Boolean
     val isJigasi: Boolean
     val isTranscriber: Boolean
     val isJibri: Boolean

diff --git a/jicofo-common/src/main/kotlin/org/jitsi/jicofo/xmpp/muc/ChatRoomMemberImpl.kt b/jicofo-common/src/main/kotlin/org/jitsi/jicofo/xmpp/muc/ChatRoomMemberImpl.kt
@@ -98,13 +98,6 @@ class ChatRoomMemberImpl(
         }
         private set
 
-    private var robot = false
-    override val isRobot: Boolean
-        get() {
-            // Jigasi and Jibri do not use the "robot" signaling, but semantically they should be considered "robots".
-            return robot || isJigasi || isJibri
-        }
-
     private fun updateSourceInfo(presence: Presence) {
         val sourceInfo = presence.getExtension<StandardExtensionElement>("SourceInfo", "jabber:client")
         if (sourceInfo == null) {
@@ -156,13 +149,6 @@ class ChatRoomMemberImpl(
 
         val firstPresence = (this.presence == null)
         this.presence = presence
-        presence.getExtension(UserInfoPacketExt::class.java)?.let {
-            val newStatus = it.isRobot
-            if (newStatus != null && robot != newStatus) {
-                logger.debug { "robot: $robot" }
-                robot = newStatus
-            }
-        }
 
         presence.getExtension(CapsExtension::class.java)?.let {
             capsNodeVer = "${it.node}#${it.ver}"
@@ -273,9 +259,9 @@ class ChatRoomMemberImpl(
             this["region"] = region.toString()
             this["occupant_jid"] = occupantJid.toString()
             this["jid"] = jid.toString()
-            this["robot"] = robot
             this["is_jibri"] = isJibri
             this["is_jigasi"] = isJigasi
+            this["is_transcriber"] = isTranscriber
             this["role"] = role.toString()
             this["video_codecs"] = JSONArray().apply { videoCodecs?.let { addAll(it) } }
             this["stats_id"] = statsId.toString()

diff --git a/jicofo/src/main/kotlin/org/jitsi/jicofo/xmpp/muc/ChatRoomRoleManager.kt b/jicofo/src/main/kotlin/org/jitsi/jicofo/xmpp/muc/ChatRoomRoleManager.kt
@@ -91,12 +91,12 @@ class AutoOwnerRoleManager(chatRoom: ChatRoom) : ChatRoomRoleManager(chatRoom) {
                 return@add
             }
 
-            owner = chatRoom.members.find { !it.isRobot && it.role.hasOwnerRights() }
+            owner = chatRoom.members.find { !(it.isJibri || it.isJigasi) && it.role.hasOwnerRights() }
             if (owner != null) {
                 return@add
             }
 
-            val newOwner = chatRoom.members.find { !it.isRobot && it.role != MemberRole.VISITOR }
+            val newOwner = chatRoom.members.find { !(it.isJibri || it.isJigasi) && it.role != MemberRole.VISITOR }
             if (newOwner != null) {
                 logger.info("Electing new owner: $newOwner")
                 chatRoom.grantOwnership(newOwner)

diff --git a/jicofo/src/test/kotlin/org/jitsi/jicofo/jibri/JibriDetectorTest.kt b/jicofo/src/test/kotlin/org/jitsi/jicofo/jibri/JibriDetectorTest.kt
@@ -136,7 +136,6 @@ class JibriChatRoomMember(
     override val name: String get() = TODO("Not yet implemented")
     override val jid: Jid? get() = TODO("Not yet implemented")
     override val sourceInfos: Set<SourceInfo> get() = TODO("Not yet implemented")
-    override val isRobot: Boolean get() = TODO("Not yet implemented")
     override val isJigasi: Boolean get() = TODO("Not yet implemented")
     override val isTranscriber: Boolean get() = TODO("Not yet implemented")
     override val isJibri: Boolean get() = TODO("Not yet implemented")