Make /msg <message> param optional for more flexibility (#7028)

matrix-org · Oct 25, 2021 · e3d1615 · e3d1615
1 parent ead2a51
commit e3d1615
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/src/SlashCommands.tsx b/src/SlashCommands.tsx
@@ -1014,22 +1014,24 @@ export const Commands = [
     new Command({
         command: "msg",
         description: _td("Sends a message to the given user"),
-        args: "<user-id> <message>",
+        args: "<user-id> [<message>]",
         runFn: function(roomId, args) {
             if (args) {
                 // matches the first whitespace delimited group and then the rest of the string
                 const matches = args.match(/^(\S+?)(?: +(.*))?$/s);
                 if (matches) {
                     const [userId, msg] = matches.slice(1);
-                    if (msg && userId && userId.startsWith("@") && userId.includes(":")) {
+                    if (userId && userId.startsWith("@") && userId.includes(":")) {
                         return success((async () => {
                             const cli = MatrixClientPeg.get();
                             const roomId = await ensureDMExists(cli, userId);
                             dis.dispatch({
                                 action: 'view_room',
                                 room_id: roomId,
                             });
-                            cli.sendTextMessage(roomId, msg);
+                            if (msg) {
+                                cli.sendTextMessage(roomId, msg);
+                            }
                         })());
                     }
                 }