From a6d7ed3e2727b60933b692e985f0384b1b04a0d1 Mon Sep 17 00:00:00 2001 From: seb Date: Sat, 5 Jul 2025 15:57:00 +0200 Subject: [PATCH] Refactor Sitemap component to improve context usage by replacing direct socket reference with context destructuring. This change enhances code clarity and maintains functionality for fetching category data. --- src/pages/Sitemap.js | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/pages/Sitemap.js b/src/pages/Sitemap.js index 5d420e9..452f341 100644 --- a/src/pages/Sitemap.js +++ b/src/pages/Sitemap.js @@ -36,7 +36,7 @@ const collectAllCategories = (categoryNode, categories = [], level = 0) => { const Sitemap = () => { const [categories, setCategories] = useState([]); const [loading, setLoading] = useState(true); - const {socket} = useContext(SocketContext); + const context = useContext(SocketContext); const sitemapLinks = [ @@ -67,8 +67,8 @@ const Sitemap = () => { } // Otherwise, fetch from socket if available - if (socket) { - socket.emit('categoryList', { categoryId: 209 }, (response) => { + if (context && context.socket && context.socket.connected && typeof window !== "undefined") { + context.socket.emit('categoryList', { categoryId: 209 }, (response) => { if (response && response.categoryTree) { // Store in cache try { @@ -95,7 +95,7 @@ const Sitemap = () => { }; fetchCategories(); - }, [socket]); + }, [context]); const content = ( <>