actually test and fix bug

wtfnotavailable · Dec 10, 2023 · 713c1ea · 713c1ea
1 parent 2b3e311
commit 713c1ea
Showing 1 changed file with 19 additions and 15 deletions.
diff --git a/dashboard/src/pages/servers/[id]/player.tsx b/dashboard/src/pages/servers/[id]/player.tsx
@@ -333,6 +333,24 @@ const Player: NextPageWithLayout = () => {
         });
     };
 
+    useEffect(() => {
+        sharedStateMount(sharedState);
+        seekerMount();
+
+        playerSocket.mount(serverId as string, {
+            mountHandler: {
+                close: handleSocketClose,
+            },
+            eventHandler: socketEventHandlers,
+        });
+
+        return () => {
+            sharedStateUnmount(sharedState);
+            seekerUnmount();
+            playerSocket.unmount(serverId as string);
+        };
+    }, []);
+
     const handleNavbarToggle = () => {
         if (!sharedState.setNavbarShow) return;
         if (!sharedState.navbarAbsolute) {
@@ -452,26 +470,12 @@ const Player: NextPageWithLayout = () => {
     };
 
     useEffect(() => {
-        sharedStateMount(sharedState);
-        seekerMount();
-
-        playerSocket.mount(serverId as string, {
-            mountHandler: {
-                close: handleSocketClose,
-            },
-            eventHandler: socketEventHandlers,
-        });
-
         registerAllSrct();
 
         return () => {
-            sharedStateUnmount(sharedState);
-            seekerUnmount();
-            playerSocket.unmount(serverId as string);
-
             unregisterAllSrct();
         };
-    }, []);
+    }, [registerAllSrct, unregisterAllSrct]);
 
     const mainImg = !playing?.thumbnail?.length
         ? SampleThumb.src