From 6eab5e77aa92f3c0febdb7f5c78072fb450b0b2c Mon Sep 17 00:00:00 2001
From: Paul <paulmakles@gmail.com>
Date: Thu, 12 Aug 2021 17:11:34 +0100
Subject: [PATCH] Remove ability to add bot as a friend.

---
 .../intermediate/popovers/UserProfile.tsx     |  15 +--
 src/lib/ContextMenus.tsx                      | 100 +++++++++---------
 2 files changed, 60 insertions(+), 55 deletions(-)

diff --git a/src/context/intermediate/popovers/UserProfile.tsx b/src/context/intermediate/popovers/UserProfile.tsx
index 12fd4e6d..fedaaeb7 100644
--- a/src/context/intermediate/popovers/UserProfile.tsx
+++ b/src/context/intermediate/popovers/UserProfile.tsx
@@ -190,12 +190,15 @@ export const UserProfile = observer(
                                 <Edit size={28} />
                             </IconButton>
                         )}
-                        {(user.relationship === RelationshipStatus.Incoming ||
-                            user.relationship === RelationshipStatus.None) && (
-                            <IconButton onClick={() => user.addFriend()}>
-                                <UserPlus size={28} />
-                            </IconButton>
-                        )}
+                        {!user.bot &&
+                            (user.relationship ===
+                                RelationshipStatus.Incoming ||
+                                user.relationship ===
+                                    RelationshipStatus.None) && (
+                                <IconButton onClick={() => user.addFriend()}>
+                                    <UserPlus size={28} />
+                                </IconButton>
+                            )}
                     </div>
                     <div className={styles.tabs}>
                         <div
diff --git a/src/lib/ContextMenus.tsx b/src/lib/ContextMenus.tsx
index 2dad2082..7448be6e 100644
--- a/src/lib/ContextMenus.tsx
+++ b/src/lib/ContextMenus.tsx
@@ -585,58 +585,60 @@ function ContextMenus(props: Props) {
                     }
 
                     if (user) {
-                        let actions: Action["action"][];
-                        switch (user.relationship) {
-                            case RelationshipStatus.User:
-                                actions = [];
-                                break;
-                            case RelationshipStatus.Friend:
-                                actions = ["remove_friend", "block_user"];
-                                break;
-                            case RelationshipStatus.Incoming:
-                                actions = [
-                                    "add_friend",
-                                    "cancel_friend",
-                                    "block_user",
-                                ];
-                                break;
-                            case RelationshipStatus.Outgoing:
-                                actions = ["cancel_friend", "block_user"];
-                                break;
-                            case RelationshipStatus.Blocked:
-                                actions = ["unblock_user"];
-                                break;
-                            case RelationshipStatus.BlockedOther:
-                                actions = ["block_user"];
-                                break;
-                            case RelationshipStatus.None:
-                            default:
-                                actions = ["add_friend", "block_user"];
-                        }
+                        if (!user.bot) {
+                            let actions: Action["action"][];
+                            switch (user.relationship) {
+                                case RelationshipStatus.User:
+                                    actions = [];
+                                    break;
+                                case RelationshipStatus.Friend:
+                                    actions = ["remove_friend", "block_user"];
+                                    break;
+                                case RelationshipStatus.Incoming:
+                                    actions = [
+                                        "add_friend",
+                                        "cancel_friend",
+                                        "block_user",
+                                    ];
+                                    break;
+                                case RelationshipStatus.Outgoing:
+                                    actions = ["cancel_friend", "block_user"];
+                                    break;
+                                case RelationshipStatus.Blocked:
+                                    actions = ["unblock_user"];
+                                    break;
+                                case RelationshipStatus.BlockedOther:
+                                    actions = ["block_user"];
+                                    break;
+                                case RelationshipStatus.None:
+                                default:
+                                    actions = ["add_friend", "block_user"];
+                            }
 
-                        if (userPermissions & UserPermission.ViewProfile) {
-                            generateAction({
-                                action: "view_profile",
-                                user,
-                            });
-                        }
+                            if (userPermissions & UserPermission.ViewProfile) {
+                                generateAction({
+                                    action: "view_profile",
+                                    user,
+                                });
+                            }
 
-                        if (
-                            user._id !== userId &&
-                            userPermissions & UserPermission.SendMessage
-                        ) {
-                            generateAction({
-                                action: "message_user",
-                                user,
-                            });
-                        }
+                            if (
+                                user._id !== userId &&
+                                userPermissions & UserPermission.SendMessage
+                            ) {
+                                generateAction({
+                                    action: "message_user",
+                                    user,
+                                });
+                            }
 
-                        for (let i = 0; i < actions.length; i++) {
-                            // Typescript can't determine that user the actions are linked together correctly
-                            generateAction({
-                                action: actions[i],
-                                user,
-                            } as unknown as Action);
+                            for (let i = 0; i < actions.length; i++) {
+                                // Typescript can't determine that user the actions are linked together correctly
+                                generateAction({
+                                    action: actions[i],
+                                    user,
+                                } as unknown as Action);
+                            }
                         }
                     }