From 1e4a248643eedccf5df0aa169194161bfe79aff9 Mon Sep 17 00:00:00 2001
From: zkldi <20380519+zkldi@users.noreply.github.com>
Date: Sat, 4 Feb 2023 04:29:52 +0000
Subject: [PATCH] fix: no signout if logged out

---
 .../components/layout/header/HeaderMenu.tsx   | 68 ++++++++++---------
 1 file changed, 35 insertions(+), 33 deletions(-)

diff --git a/client/src/components/layout/header/HeaderMenu.tsx b/client/src/components/layout/header/HeaderMenu.tsx
index 5cc241a6f..8a35ef063 100644
--- a/client/src/components/layout/header/HeaderMenu.tsx
+++ b/client/src/components/layout/header/HeaderMenu.tsx
@@ -129,40 +129,42 @@ export function HeaderMenu({
 						{settings?.preferences.developerMode && <UtilsDropdown />}
 					</ul>
 				</div>
-				<div
-					className="d-flex d-lg-none"
-					style={{ paddingLeft: "30px", marginBottom: "2rem", flex: "1 1 auto" }}
-				>
-					<div>
-						<Divider />
-					</div>
-					<Button
-						style={{ alignSelf: "flex-end" }}
-						variant="outline-danger"
-						onClick={async () => {
-							if (confirm("Are you sure you want to sign out?")) {
-								const rj = await APIFetchV1("/auth/logout", {
-									method: "POST",
-								});
-
-								if (rj.success) {
-									toast.success("Logged out.");
-									setTimeout(() => {
-										setUser(null);
-										localStorage.removeItem("isLoggedIn");
-										// This has to be the case.
-										// Otherwise, react just ruins its own
-										// state. I hate react state.
-										window.location.href = "/";
-									}, 500);
-								}
-							}
-						}}
+				{user && (
+					<div
+						className="d-flex d-lg-none"
+						style={{ paddingLeft: "30px", marginBottom: "2rem", flex: "1 1 auto" }}
 					>
-						<Icon type="sign-out-alt" />
-						Sign Out
-					</Button>
-				</div>
+						<div>
+							<Divider />
+						</div>
+						<Button
+							style={{ alignSelf: "flex-end" }}
+							variant="outline-danger"
+							onClick={async () => {
+								if (confirm("Are you sure you want to sign out?")) {
+									const rj = await APIFetchV1("/auth/logout", {
+										method: "POST",
+									});
+
+									if (rj.success) {
+										toast.success("Logged out.");
+										setTimeout(() => {
+											setUser(null);
+											localStorage.removeItem("isLoggedIn");
+											// This has to be the case.
+											// Otherwise, react just ruins its own
+											// state. I hate react state.
+											window.location.href = "/";
+										}, 500);
+									}
+								}
+							}}
+						>
+							<Icon type="sign-out-alt" />
+							Sign Out
+						</Button>
+					</div>
+				)}
 			</div>
 		</div>
 	);