From 15c5119a79b604f80a0110f67b032e1a74ac9644 Mon Sep 17 00:00:00 2001
From: Alexander Zinchuk <alx.zinchuk@gmail.com>
Date: Tue, 20 Sep 2022 14:59:17 +0200
Subject: [PATCH] Manage Group Members: Allow removing via context menu (#2048)

---
 .../right/management/ManageGroupMembers.tsx   | 37 ++++++++++++++++++-
 1 file changed, 35 insertions(+), 2 deletions(-)

diff --git a/src/components/right/management/ManageGroupMembers.tsx b/src/components/right/management/ManageGroupMembers.tsx
index b3116ffa..0105ba41 100644
--- a/src/components/right/management/ManageGroupMembers.tsx
+++ b/src/components/right/management/ManageGroupMembers.tsx
@@ -1,6 +1,6 @@
 import type { FC } from '../../../lib/teact/teact';
 import React, {
-  memo, useCallback, useMemo, useRef,
+  memo, useCallback, useMemo, useRef, useState,
 } from '../../../lib/teact/teact';
 import { getActions, getGlobal, withGlobal } from '../../../global';
 
@@ -10,7 +10,7 @@ import { ManagementScreens } from '../../../types';
 import { unique } from '../../../util/iteratees';
 import { selectChat } from '../../../global/selectors';
 import {
-  sortUserIds, isChatChannel, filterUsersByName, sortChatIds, isUserBot,
+  sortUserIds, isChatChannel, filterUsersByName, sortChatIds, isUserBot, getHasAdminRight,
 } from '../../../global/helpers';
 import useLang from '../../../hooks/useLang';
 import useHistoryBack from '../../../hooks/useHistoryBack';
@@ -23,6 +23,7 @@ import ListItem from '../../ui/ListItem';
 import InputText from '../../ui/InputText';
 import InfiniteScroll from '../../ui/InfiniteScroll';
 import Loading from '../../ui/Loading';
+import DeleteMemberModal from '../DeleteMemberModal';
 
 type OwnProps = {
   chatId: string;
@@ -44,6 +45,8 @@ type StateProps = {
   localUserIds?: string[];
   globalUserIds?: string[];
   serverTimeOffset: number;
+  currentUserId?: string;
+  canDeleteMembers?: boolean;
 };
 
 const ManageGroupMembers: FC<OwnProps & StateProps> = ({
@@ -59,6 +62,8 @@ const ManageGroupMembers: FC<OwnProps & StateProps> = ({
   isSearching,
   searchQuery,
   serverTimeOffset,
+  currentUserId,
+  canDeleteMembers,
   onClose,
   onScreenSelect,
   onChatMemberSelect,
@@ -70,6 +75,8 @@ const ManageGroupMembers: FC<OwnProps & StateProps> = ({
   // eslint-disable-next-line no-null/no-null
   const containerRef = useRef<HTMLDivElement>(null);
 
+  const [deletingUserId, setDeletingUserId] = useState<string | undefined>();
+
   const adminIds = useMemo(() => {
     return noAdmins ? adminMembers?.map(({ userId }) => userId) || [] : [];
   }, [adminMembers, noAdmins]);
@@ -141,11 +148,25 @@ const ManageGroupMembers: FC<OwnProps & StateProps> = ({
     }
   }, '.ListItem-button', true);
 
+  const handleDeleteMembersModalClose = useCallback(() => {
+    setDeletingUserId(undefined);
+  }, []);
+
   useHistoryBack({
     isActive,
     onBack: onClose,
   });
 
+  function getMemberContextAction(memberId: string) {
+    return memberId === currentUserId || !canDeleteMembers ? undefined : [{
+      title: lang('lng_context_remove_from_group'),
+      icon: 'stop',
+      handler: () => {
+        setDeletingUserId(memberId);
+      },
+    }];
+  }
+
   function renderSearchField() {
     return (
       <div className="Management__filter" dir={lang.isRtl ? 'rtl' : undefined}>
@@ -179,6 +200,7 @@ const ManageGroupMembers: FC<OwnProps & StateProps> = ({
                   className="chat-item-clickable scroll-item"
                   // eslint-disable-next-line react/jsx-no-bind
                   onClick={() => handleMemberClick(id)}
+                  contextActions={getMemberContextAction(id)}
                 >
                   <PrivateChatInfo userId={id} forceShowSelf />
                 </ListItem>
@@ -195,6 +217,13 @@ const ManageGroupMembers: FC<OwnProps & StateProps> = ({
           )}
         </div>
       </div>
+      {canDeleteMembers && (
+        <DeleteMemberModal
+          isOpen={Boolean(deletingUserId)}
+          userId={deletingUserId}
+          onClose={handleDeleteMembersModalClose}
+        />
+      )}
     </div>
   );
 };
@@ -215,6 +244,8 @@ export default memo(withGlobal<OwnProps>(
       localUserIds,
     } = global.userSearch;
 
+    const canDeleteMembers = chat && (chat.isCreator || getHasAdminRight(chat, 'banUsers'));
+
     return {
       members,
       adminMembers,
@@ -225,7 +256,9 @@ export default memo(withGlobal<OwnProps>(
       isSearching: fetchingStatus,
       globalUserIds,
       localUserIds,
+      canDeleteMembers,
       serverTimeOffset: global.serverTimeOffset,
+      currentUserId: global.currentUserId,
     };
   },
 )(ManageGroupMembers));