remove swiss dep to chat

lichess-org · Apr 11, 2024 · 1f9464c · 1f9464c
1 parent f408121
commit 1f9464c
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 8 deletions.
diff --git a/build.sbt b/build.sbt
@@ -300,7 +300,7 @@ lazy val tournament = module("tournament",
 )
 
 lazy val swiss = module("swiss",
-  Seq(gathering, user, game, chat, room),
+  Seq(gathering, user, game, room),
   Seq(scalatags, lettuce) ++ reactivemongo.bundle ++ tests.bundle
 )
 

diff --git a/modules/chat/src/main/ChatApi.scala b/modules/chat/src/main/ChatApi.scala
@@ -123,9 +123,6 @@ final class ChatApi(
       val line = UserLine(UserName.lichess, None, false, flair = true, text, troll = false, deleted = false)
       publishLine(chatId, line, busChan)
 
-    def service(chatId: ChatId, text: String, busChan: BusChan.Select, isVolatile: Boolean): Unit =
-      (if isVolatile then volatile else system) (chatId, text, busChan)
-
     def timeout(
         chatId: ChatId,
         userId: UserId,

diff --git a/modules/swiss/src/main/Env.scala b/modules/swiss/src/main/Env.scala
@@ -18,7 +18,7 @@ final class Env(
     userApi: lila.user.UserApi,
     onStart: lila.core.game.OnStart,
     socketKit: lila.core.socket.SocketKit,
-    chatApi: lila.chat.ChatApi,
+    chat: lila.core.chat.ChatApi,
     cacheApi: lila.memo.CacheApi,
     lightUserApi: lila.user.LightUserApi,
     historyApi: lila.core.history.HistoryApi,

diff --git a/modules/swiss/src/main/SwissApi.scala b/modules/swiss/src/main/SwissApi.scala
@@ -36,7 +36,7 @@ final class SwissApi(
     banApi: SwissBanApi,
     boardApi: SwissBoardApi,
     verify: SwissCondition.Verify,
-    chatApi: lila.chat.ChatApi,
+    chatApi: lila.core.chat.ChatApi,
     lightUserApi: lila.user.LightUserApi,
     roundApi: lila.game.core.RoundApi
 )(using scheduler: Scheduler)(using Executor, akka.stream.Materializer)
@@ -613,7 +613,9 @@ final class SwissApi(
         }
 
   private def systemChat(id: SwissId, text: String, volatile: Boolean = false): Unit =
-    chatApi.userChat.service(id.into(ChatId), text, _.swiss, isVolatile = volatile)
+    if volatile
+    then chatApi.volatile(id.into(ChatId), text, _.swiss)
+    else chatApi.system(id.into(ChatId), text, _.swiss)
 
   def withdrawAll(user: User, teamIds: List[TeamId]): Funit =
     mongo.swiss

diff --git a/modules/swiss/src/main/SwissSocket.scala b/modules/swiss/src/main/SwissSocket.scala
@@ -7,7 +7,7 @@ import lila.core.socket.{ protocol as P, * }
 
 final private class SwissSocket(
     socketKit: SocketKit,
-    chat: lila.chat.ChatApi,
+    chat: lila.core.chat.ChatApi,
     teamOf: SwissId => Fu[Option[TeamId]]
 )(using Executor, akka.actor.ActorSystem, Scheduler, lila.core.user.FlairGet):