ui/chat cannot know about relay and hardcode its separator

lichess-org · Jul 7, 2024 · 4c8aced · 4c8aced
1 parent 0031e1d
commit 4c8aced
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 3 deletions.
diff --git a/ui/analyse/src/study/relay/chatHandler.ts b/ui/analyse/src/study/relay/chatHandler.ts
@@ -6,6 +6,7 @@ export function broadcastChatHandler(ctrl: AnalyseCtrl): BroadcastChatHandler {
   const separator = '\ue666';
 
   const encodeMsg = (text: string): string => {
+    text = cleanMsg(text);
     if (ctrl.study?.relay && !ctrl.study.relay.tourShow()) {
       const chapterId = ctrl.study.currentChapter().id;
       const ply = ctrl.study.currentNode().ply;

diff --git a/ui/chat/src/ctrl.ts b/ui/chat/src/ctrl.ts
@@ -105,8 +105,6 @@ export default class ChatCtrl {
       alert('Max length: 140 chars. ' + text.length + ' chars used.');
       return false;
     }
-    if (text.includes('\ue666')) return false;
-
     if (this.broadcastChatHandler) text = this.broadcastChatHandler.encodeMsg(text);
 
     site.pubsub.emit('socket.send', 'talk', text);

diff --git a/ui/chat/src/interfaces.ts b/ui/chat/src/interfaces.ts
@@ -56,7 +56,7 @@ export interface Line {
 }
 
 export interface BroadcastChatHandler {
-  encodeMsg(text: string): string;
+  encodeMsg(msg: string): string;
   cleanMsg(msg: string): string;
   jumpToMove(msg: string): void;
   canJumpToMove(msg: string): string | null;