[FIX] Deactivated users show as offline (#18767)

pull/18932/head^2
gabriellsh 5 years ago committed by GitHub
parent b3d132fe5c
commit 38fc10ff68
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 86
      client/admin/users/UsersTable.js

@ -3,7 +3,8 @@ import { useDebouncedValue, useMediaQuery } from '@rocket.chat/fuselage-hooks';
import React, { useMemo, useCallback, useState, useEffect } from 'react'; import React, { useMemo, useCallback, useState, useEffect } from 'react';
import UserAvatar from '../../components/basic/avatar/UserAvatar'; import UserAvatar from '../../components/basic/avatar/UserAvatar';
import { GenericTable, Th } from '../../components/GenericTable'; import { GenericTable } from '../../components/GenericTable';
import { capitalize } from '../../helpers/capitalize';
import { useTranslation } from '../../contexts/TranslationContext'; import { useTranslation } from '../../contexts/TranslationContext';
import { useRoute } from '../../contexts/RouterContext'; import { useRoute } from '../../contexts/RouterContext';
import { useEndpointData } from '../../hooks/useEndpointData'; import { useEndpointData } from '../../hooks/useEndpointData';
@ -25,8 +26,34 @@ const FilterByText = ({ setFilter, ...props }) => {
const sortDir = (sortDir) => (sortDir === 'asc' ? 1 : -1); const sortDir = (sortDir) => (sortDir === 'asc' ? 1 : -1);
const UserRow = ({ emails, _id, username, name, roles, status, avatarETag, onClick, mediaQuery, active }) => {
const t = useTranslation();
const statusText = active ? t(capitalize(status)) : t('Disabled');
return <Table.Row onKeyDown={onClick(_id)} onClick={onClick(_id)} tabIndex={0} role='link' action qa-user-id={_id}>
<Table.Cell style={style}>
<Box display='flex' alignItems='center'>
<UserAvatar size={mediaQuery ? 'x28' : 'x40'} title={username} username={username} etag={avatarETag}/>
<Box display='flex' style={style} mi='x8'>
<Box display='flex' flexDirection='column' alignSelf='center' style={style}>
<Box fontScale='p2' style={style} color='default'>{name || username}</Box>
{!mediaQuery && name && <Box fontScale='p1' color='hint' style={style}> {`@${ username }`} </Box>}
</Box>
</Box>
</Box>
</Table.Cell>
{mediaQuery && <Table.Cell>
<Box fontScale='p2' style={style} color='hint'>{ username }</Box> <Box mi='x4'/>
</Table.Cell>}
<Table.Cell style={style}>{emails && emails.length && emails[0].address}</Table.Cell>
{mediaQuery && <Table.Cell style={style}>{roles && roles.join(', ')}</Table.Cell>}
<Table.Cell fontScale='p1' color='hint' style={style}>{statusText}</Table.Cell>
</Table.Row>;
};
const useQuery = ({ text, itemsPerPage, current }, [column, direction]) => useMemo(() => ({ const useQuery = ({ text, itemsPerPage, current }, [column, direction]) => useMemo(() => ({
fields: JSON.stringify({ name: 1, username: 1, emails: 1, roles: 1, status: 1, avatarETag: 1 }), fields: JSON.stringify({ name: 1, username: 1, emails: 1, roles: 1, status: 1, avatarETag: 1, active: 1 }),
query: JSON.stringify({ query: JSON.stringify({
$or: [ $or: [
{ 'emails.address': { $regex: text || '', $options: 'i' } }, { 'emails.address': { $regex: text || '', $options: 'i' } },
@ -70,35 +97,32 @@ export function UsersTable() {
const mediaQuery = useMediaQuery('(min-width: 1024px)'); const mediaQuery = useMediaQuery('(min-width: 1024px)');
const header = useMemo(() => [ return <GenericTable
<Th key={'name'} direction={sort[1]} active={sort[0] === 'name'} onClick={onHeaderClick} sort='name' w='x200'>{t('Name')}</Th>, FilterComponent={FilterByText}
mediaQuery && <Th key={'username'} direction={sort[1]} active={sort[0] === 'username'} onClick={onHeaderClick} sort='username' w='x140'>{t('Username')}</Th>, header={<>
<Th key={'email'} direction={sort[1]} active={sort[0] === 'emails.adress'} onClick={onHeaderClick} sort='emails.address' w='x120'>{t('Email')}</Th>, <GenericTable.HeaderCell key={'name'} direction={sort[1]} active={sort[0] === 'name'} onClick={onHeaderClick} sort='name' w='x200'>
mediaQuery && <Th key={'roles'} direction={sort[1]} active={sort[0] === 'roles'} onClick={onHeaderClick} sort='roles' w='x120'>{t('Roles')}</Th>, {t('Name')}
<Th key={'status'} direction={sort[1]} active={sort[0] === 'status'} onClick={onHeaderClick} sort='status' w='x100'>{t('Status')}</Th>, </GenericTable.HeaderCell>
].filter(Boolean), [sort, onHeaderClick, t, mediaQuery]); {mediaQuery && <GenericTable.HeaderCell key={'username'} direction={sort[1]} active={sort[0] === 'username'} onClick={onHeaderClick} sort='username' w='x140'>
{t('Username')}
const renderRow = useCallback(({ emails, _id, username, name, roles, status, avatarETag }) => <Table.Row key={_id} onKeyDown={onClick(_id)} onClick={onClick(_id)} tabIndex={0} role='link' action qa-user-id={_id}> </GenericTable.HeaderCell>}
<Table.Cell style={style}> <GenericTable.HeaderCell key={'email'} direction={sort[1]} active={sort[0] === 'emails.adress'} onClick={onHeaderClick} sort='emails.address' w='x120'>
<Box display='flex' alignItems='center'> {t('Email')}
<UserAvatar size={mediaQuery ? 'x28' : 'x40'} title={username} username={username} etag={avatarETag}/> </GenericTable.HeaderCell>
<Box display='flex' style={style} mi='x8'> {mediaQuery && <GenericTable.HeaderCell key={'roles'} direction={sort[1]} active={sort[0] === 'roles'} onClick={onHeaderClick} sort='roles' w='x120'>
<Box display='flex' flexDirection='column' alignSelf='center' style={style}> {t('Roles')}
<Box fontScale='p2' style={style} color='default'>{name || username}</Box> </GenericTable.HeaderCell>}
{!mediaQuery && name && <Box fontScale='p1' color='hint' style={style}> {`@${ username }`} </Box>} <GenericTable.HeaderCell key={'status'} direction={sort[1]} active={sort[0] === 'status'} onClick={onHeaderClick} sort='status' w='x100'>
</Box> {t('Status')}
</Box> </GenericTable.HeaderCell>
</Box> </>}
</Table.Cell> results={data.users}
{mediaQuery && <Table.Cell> total={data.total}
<Box fontScale='p2' style={style} color='hint'>{ username }</Box> <Box mi='x4'/> setParams={setParams}
</Table.Cell>} params={params}
<Table.Cell style={style}>{emails && emails.length && emails[0].address}</Table.Cell> >
{mediaQuery && <Table.Cell style={style}>{roles && roles.join(', ')}</Table.Cell>} {(props) => <UserRow key={props._id} onClick={onClick} mediaQuery={mediaQuery} {...props}/>}
<Table.Cell fontScale='p1' color='hint' style={style}>{status}</Table.Cell> </GenericTable>;
</Table.Row>, [mediaQuery, onClick]);
return <GenericTable FilterComponent={FilterByText} header={header} renderRow={renderRow} results={data.users} total={data.total} setParams={setParams} params={params} />;
} }
export default UsersTable; export default UsersTable;

Loading…
Cancel
Save