From ef03c1b03299dc5d82c57823aeb8930395793624 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Maxime=20=E2=80=9Cpep=E2=80=9D=20Buquet?= <pep@bouah.net>
Date: Sat, 17 Sep 2022 16:53:33 +0200
Subject: [PATCH] Return SessionAlreadyExists error in Occupant::add_session
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: Maxime “pep” Buquet <pep@bouah.net>
---
 src/error.rs | 2 ++
 src/room.rs  | 7 +++++++
 2 files changed, 9 insertions(+)

diff --git a/src/error.rs b/src/error.rs
index 8923f4d..9486a3b 100644
--- a/src/error.rs
+++ b/src/error.rs
@@ -24,6 +24,7 @@ pub enum Error {
     MismatchJids(Jid, Jid),
     NickAlreadyAssigned(String),
     NonexistantSession(FullJid),
+    SessionAlreadyExists(FullJid),
     XMPPError(TokioXMPPError),
 }
 
@@ -35,6 +36,7 @@ impl fmt::Display for Error {
             Error::MismatchJids(jid1, jid2) => write!(f, "Mismatch Jids: {}, {}", jid1, jid2),
             Error::NickAlreadyAssigned(err) => write!(f, "Nickname already assigned: {}", err),
             Error::NonexistantSession(err) => write!(f, "Session doesn't exist: {}", err),
+            Error::SessionAlreadyExists(err) => write!(f, "Session already exist: {}", err),
             Error::XMPPError(err) => write!(f, "XMPP error: {}", err),
         }
     }
diff --git a/src/room.rs b/src/room.rs
index 84a7a46..bb50182 100644
--- a/src/room.rs
+++ b/src/room.rs
@@ -224,6 +224,13 @@ impl Occupant {
             return Err(Error::MismatchJids(Jid::from(self.real.clone()), Jid::from(real.clone())));
         }
 
+        for session in &self.sessions {
+            if &real == session {
+                return Err(Error::SessionAlreadyExists(real))
+            }
+        }
+
+        self.sessions.push(real);
         Ok(())
     }