From eb4856901b716de2539d61d7e5e713318c12e19f Mon Sep 17 00:00:00 2001
From: Alex Holliday <ajhollid@gmail.com>
Date: Fri, 1 Aug 2025 15:28:22 -0700
Subject: [PATCH] extract function

---
 .../Sidebar/components/authFooter.jsx         | 34 ++++++++++---------
 1 file changed, 18 insertions(+), 16 deletions(-)

diff --git a/client/src/Components/Sidebar/components/authFooter.jsx b/client/src/Components/Sidebar/components/authFooter.jsx
index 616754c67..6d2dfceeb 100644
--- a/client/src/Components/Sidebar/components/authFooter.jsx
+++ b/client/src/Components/Sidebar/components/authFooter.jsx
@@ -20,6 +20,22 @@ import { clearAuthState } from "../../../Features/Auth/authSlice";
 import { useDispatch } from "react-redux";
 import PropTypes from "prop-types";
 
+const getFilteredAccountMenuItems = (user, items) => {
+	if (!user) return [];
+
+	let filtered = [...items];
+
+	if (user.role?.includes("demo")) {
+		filtered = filtered.filter((item) => item.name !== "Password");
+	}
+
+	if (!user.role?.includes("superadmin")) {
+		filtered = filtered.filter((item) => item.name !== "Team");
+	}
+
+	return filtered;
+};
+
 const AuthFooter = ({ collapsed, accountMenuItems }) => {
 	const { t } = useTranslation();
 	const theme = useTheme();
@@ -43,23 +59,9 @@ const AuthFooter = ({ collapsed, accountMenuItems }) => {
 		navigate("/login");
 	};
 	const renderAccountMenuItems = (user, items) => {
-		let filteredAccountMenuItems = [...items];
+		const filteredItems = getFilteredAccountMenuItems(user, items);
 
-		// If the user is in demo mode, remove the "Password" option
-		if (user.role?.includes("demo")) {
-			filteredAccountMenuItems = filteredAccountMenuItems.filter(
-				(item) => item.name !== "Password"
-			);
-		}
-
-		// If the user is NOT a superadmin, remove the "Team" option
-		if (user.role && !user.role.includes("superadmin")) {
-			filteredAccountMenuItems = filteredAccountMenuItems.filter(
-				(item) => item.name !== "Team"
-			);
-		}
-
-		return filteredAccountMenuItems.map((item) => (
+		return filteredItems.map((item) => (
 			<MenuItem
 				key={item.name}
 				onClick={() => {