fix(ui): useEffect deps

This commit is contained in:
Fu Diwei 2025-01-01 20:52:36 +08:00
parent 67a32a98a9
commit b2417ad902
3 changed files with 7 additions and 7 deletions

View File

@ -1,4 +1,4 @@
import { memo, useCallback, useEffect, useState } from "react"; import { memo, useEffect, useState } from "react";
import { useTranslation } from "react-i18next"; import { useTranslation } from "react-i18next";
import { Link, Navigate, Outlet, useLocation, useNavigate } from "react-router-dom"; import { Link, Navigate, Outlet, useLocation, useNavigate } from "react-router-dom";
import { import {
@ -135,12 +135,12 @@ const SiderMenu = memo(({ onSelect }: { onSelect?: (key: string) => void }) => {
}); });
const [menuSelectedKey, setMenuSelectedKey] = useState<string>(); const [menuSelectedKey, setMenuSelectedKey] = useState<string>();
const getActiveMenuItem = useCallback(() => { const getActiveMenuItem = () => {
const item = const item =
menuItems.find((item) => item!.key === location.pathname) ?? menuItems.find((item) => item!.key === location.pathname) ??
menuItems.find((item) => item!.key !== MENU_KEY_HOME && location.pathname.startsWith(item!.key as string)); menuItems.find((item) => item!.key !== MENU_KEY_HOME && location.pathname.startsWith(item!.key as string));
return item; return item;
}, [location.pathname, menuItems]); };
useEffect(() => { useEffect(() => {
const item = getActiveMenuItem(); const item = getActiveMenuItem();
@ -149,13 +149,13 @@ const SiderMenu = memo(({ onSelect }: { onSelect?: (key: string) => void }) => {
} else { } else {
setMenuSelectedKey(undefined); setMenuSelectedKey(undefined);
} }
}, [location.pathname, getActiveMenuItem]); }, [location.pathname]);
useEffect(() => { useEffect(() => {
if (menuSelectedKey && menuSelectedKey !== getActiveMenuItem()?.key) { if (menuSelectedKey && menuSelectedKey !== getActiveMenuItem()?.key) {
navigate(menuSelectedKey); navigate(menuSelectedKey);
} }
}, [menuSelectedKey, navigate, getActiveMenuItem]); }, [menuSelectedKey]);
return ( return (
<> <>

View File

@ -125,7 +125,7 @@ const AccessList = () => {
console.error(err); console.error(err);
notificationApi.error({ message: t("common.text.request_error"), description: getErrMsg(err) }); notificationApi.error({ message: t("common.text.request_error"), description: getErrMsg(err) });
}); });
}, [fetchAccesses]); }, []);
const { loading } = useRequest( const { loading } = useRequest(
() => { () => {

View File

@ -25,7 +25,7 @@ const Settings = () => {
} }
setTabValue(path); setTabValue(path);
}, [location, navigate]); }, [location.pathname]);
return ( return (
<div className="p-4"> <div className="p-4">