add vban and namechange warnings on server players

community-network · Sep 15, 2024 · 6c33a9b · 6c33a9b
1 parent 0f1d882
commit 6c33a9b
Show file tree

Hide file tree

Showing 3 changed files with 85 additions and 51 deletions.
diff --git a/src/api/GametoolsApi.tsx b/src/api/GametoolsApi.tsx
@@ -114,6 +114,9 @@ interface ServerGraphInfo {
 interface managerPlayerInfo {
   playerId: number;
 }
+interface managerPlayersInfo {
+  playerIds: number[];
+}
 
 interface managerPlayer {
   id: number;
@@ -128,6 +131,21 @@ interface managerPlayer {
   bfban: bfbanPlayers;
 }
 
+export interface managerPlayers {
+  vban: {
+    [name: string]: { [name: string]: { bannedUntil: string; reason: string } };
+  };
+  otherNames: {
+    [name: string]: {
+      updateTimestamp: string;
+      usedNames: string[];
+    };
+  };
+  bfban: { [name: string]: bfbanPlayers };
+  bfeac: number[];
+}
+name;
+
 export interface bfbanPlayer {
   apiUrl: string;
   names: { [name: string]: bfbanPlayers };
@@ -143,7 +161,7 @@ export interface bfeacPlayer {
 export interface bfbanPlayers {
   personaId?: string;
   url: string;
-  status: string;
+  status: number;
   hacker: boolean;
   originId: string;
   originPersonaId: string;
@@ -578,6 +596,14 @@ export class ApiProvider extends JsonClient {
     });
   }
 
+  async managerCheckPlayers({
+    playerIds,
+  }: managerPlayersInfo): Promise<managerPlayers> {
+    return await this.getJsonMethod("/manager/checkbans", {
+      personaids: playerIds.toString(),
+    });
+  }
+
   async bfbanCheckPlayers({
     getter,
     usernames,

diff --git a/src/components/routes/Servers/Detailed/Players.tsx b/src/components/routes/Servers/Detailed/Players.tsx
@@ -1,14 +1,11 @@
 import { useQuery } from "@tanstack/react-query";
+import { useLocalStorage } from "@uidotdev/usehooks";
 import * as React from "react";
 import { useTranslation } from "react-i18next";
 import { Link } from "react-router-dom";
 import { bfListApi, PlayerInfo, TeamInfo } from "../../../../api/bflistApi";
 import { bf1_factions, factions } from "../../../../api/Factions";
-import {
-  bfbanPlayer,
-  bfeacPlayer,
-  GametoolsApi,
-} from "../../../../api/GametoolsApi";
+import { GametoolsApi, managerPlayers } from "../../../../api/GametoolsApi";
 import {
   DetailedServerInfo,
   ScoreServerPlayer,
@@ -26,27 +23,26 @@ import { DynamicSort } from "../../Stats/Player/Main";
 import * as Mainstyles from "./Main.module.scss";
 import * as styles from "./Players.module.scss";
 
-function CheckBan(props: {
-  playerId: string;
-  bfBanList: bfbanPlayer;
-  bfbanLoading: boolean;
-  bfbanError: boolean;
-  bfeacList: bfeacPlayer;
-  bfeacLoading: boolean;
-  bfeacError: boolean;
-}) {
+function CheckBan(
+  props: Readonly<{
+    playerId: string;
+    checkBanInfo: managerPlayers;
+    checkBanLoading: boolean;
+    checkBanError: boolean;
+    adminMode: boolean;
+  }>,
+) {
   const { t } = useTranslation();
-
-  const playerInfo = props.bfBanList?.personaids[props.playerId];
-  const bfeac = props.bfeacList?.personaids?.includes(Number(props.playerId));
+  const playerInfo = props.checkBanInfo?.bfban[props.playerId];
+  const bfeac = props.checkBanInfo?.bfeac?.includes(Number(props.playerId));
   let color = "#ffffff";
 
-  if (playerInfo?.hacker || bfeac) {
+  if (playerInfo?.status === 1 || bfeac) {
     color = "#DC143C";
     return (
-      <>
-        <a style={{ color: color, lineHeight: 0 }}>{t("bfban.platoon")}: </a>
-        {playerInfo?.hacker && (
+      <p style={{ color: color, lineHeight: 0, marginTop: ".4rem" }}>
+        {t("bfban.platoon")}:
+        {playerInfo?.status === 1 && (
           <a
             style={{ color: color, lineHeight: 0 }}
             href={playerInfo?.url}
@@ -56,9 +52,7 @@ function CheckBan(props: {
             {t("bfban.main")}
           </a>
         )}
-        {playerInfo?.hacker && bfeac && (
-          <a style={{ color: color, lineHeight: 0 }}> - </a>
-        )}
+        {playerInfo?.status === 1 && bfeac && <> - </>}
         {bfeac && (
           <a
             style={{ color: color, lineHeight: 0 }}
@@ -69,9 +63,33 @@ function CheckBan(props: {
             {t("bfeac.main")}
           </a>
         )}
-      </>
+      </p>
+    );
+  }
+
+  const vbanCount = Object.keys(
+    props.checkBanInfo?.vban[props.playerId] || {},
+  )?.length;
+  if (vbanCount > 0 && props.adminMode) {
+    color = "#dc5314";
+    return (
+      <p style={{ color: color, lineHeight: 0, marginTop: ".4rem" }}>
+        {t("warn.vban", { amount: vbanCount })}
+      </p>
+    );
+  }
+
+  const usedNameCount =
+    props.checkBanInfo?.otherNames[props.playerId]?.usedNames?.length;
+  if (usedNameCount > 5 && props.adminMode) {
+    color = "#dc5314";
+    return (
+      <p style={{ color: color, lineHeight: 0, marginTop: ".4rem" }}>
+        {t("warn.nameChange", { amount: usedNameCount })}
+      </p>
     );
   }
+
   return <></>;
 }
 
@@ -85,6 +103,7 @@ function Players(props: {
   const teams = props?.stats?.teams;
   const [sortType, setSortType] = React.useState<string>("slot");
   const [copyState, setCopyState] = React.useState<string>("");
+  const [adminMode] = useLocalStorage<boolean>("adminMode", false);
 
   let playerIds = [];
   teams.forEach((teamInfo: serverTeamList) => {
@@ -96,27 +115,14 @@ function Players(props: {
   });
 
   const {
-    isLoading: bfbanLoading,
-    isError: bfbanError,
-    data: bfBanInfo,
-  } = useQuery({
-    queryKey: ["bfbanStatsServerPlayers" + props.gameid + props.game],
-    queryFn: () =>
-      GametoolsApi.bfbanCheckPlayers({
-        getter: "playerid",
-        usernames: playerIds,
-      }),
-  });
-
-  const {
-    isLoading: bfeacLoading,
-    isError: bfeacError,
-    data: bfeacInfo,
+    isLoading: checkBanLoading,
+    isError: checkBanError,
+    data: checkBanInfo,
   } = useQuery({
-    queryKey: ["bfeacStatsServerPlayers" + props.gameid + props.game],
+    queryKey: ["managerCheckPlayers" + +props.gameid + props.game],
     queryFn: () =>
-      GametoolsApi.bfeacCheckPlayers({
-        playerIds,
+      GametoolsApi.managerCheckPlayers({
+        playerIds: playerIds.map(Number),
       }),
   });
 
@@ -237,12 +243,10 @@ function Players(props: {
                                 </div>
                                 <CheckBan
                                   playerId={key?.player_id?.toString()}
-                                  bfBanList={bfBanInfo}
-                                  bfbanLoading={bfbanLoading}
-                                  bfbanError={bfbanError}
-                                  bfeacList={bfeacInfo}
-                                  bfeacLoading={bfeacLoading}
-                                  bfeacError={bfeacError}
+                                  checkBanInfo={checkBanInfo}
+                                  checkBanLoading={checkBanLoading}
+                                  checkBanError={checkBanError}
+                                  adminMode={adminMode}
                                 />
                               </div>
                               {props.game === "bf2042" ? (

diff --git a/src/locales/languages/en-US.json b/src/locales/languages/en-US.json
@@ -42,6 +42,10 @@
     "stats": "Player is marked as hacker in bfban!",
     "platoon": " Hacker"
   },
+  "warn": {
+    "nameChange": "Warning: {{ amount }} namechanges!",
+    "vban": "Warning: {{ amount }} vbans!"
+  },
   "bfeac": {
     "main": "BFEAC"
   },