From 5bb598f6152de2efa2ddf2090ccc39ab8199e8ec Mon Sep 17 00:00:00 2001
From: Evy Kassirer <evy@zulip.com>
Date: Tue, 6 May 2025 15:33:46 +0200
Subject: [PATCH] compose_recipient: Prefer using recipient_ids over emails for
 current narrow check.

---
 web/src/compose_recipient.ts | 21 ++++-----------------
 1 file changed, 4 insertions(+), 17 deletions(-)

diff --git a/web/src/compose_recipient.ts b/web/src/compose_recipient.ts
index 39bb6c8a87..66cea8c885 100644
--- a/web/src/compose_recipient.ts
+++ b/web/src/compose_recipient.ts
@@ -37,25 +37,12 @@ function composing_to_current_topic_narrow(): boolean {
 }
 
 function composing_to_current_private_message_narrow(): boolean {
-    const compose_state_recipient = compose_state.private_message_recipient_emails();
-    const narrow_state_recipient = narrow_state.pm_emails_string();
-    if (narrow_state_recipient === undefined) {
+    const compose_state_recipient = new Set(compose_state.private_message_recipient_ids());
+    const narrow_state_recipient = narrow_state.pm_ids_set();
+    if (narrow_state_recipient.size === 0) {
         return false;
     }
-    return (
-        Boolean(compose_state_recipient) &&
-        Boolean(narrow_state_recipient) &&
-        _.isEqual(
-            compose_state_recipient
-                .split(",")
-                .map((s) => s.trim())
-                .sort(),
-            narrow_state_recipient
-                .split(",")
-                .map((s) => s.trim())
-                .sort(),
-        )
-    );
+    return _.isEqual(narrow_state_recipient, compose_state_recipient);
 }
 
 export let update_narrow_to_recipient_visibility = (): void => {