denozord/router-lists-ui
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: Ensure safe removal of event listeners in EditServerModal and DeleteServerModal to prevent potential memory leaks
Publish Fast Tabler Docker image / build-and-push-fast (push) Successful in 13m32s
Details

Browse Source
This commit is contained in:
denozord
2025-07-10 11:00:33 +07:00
parent cdd7963331
commit 0d53b11dc4
1 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
frontend/src/ServerManager.jsx
+4
View File
@@ -533,7 +533,9 @@ function EditServerModal({ show, server, onChange, onSave, onClose }) {
const handler = () => onClose && onClose(); const handler = () => onClose && onClose();
modalRef.current.addEventListener('hide.bs.modal', handler); modalRef.current.addEventListener('hide.bs.modal', handler);
return () => { return () => {
if (modalRef.current) {
modalRef.current.removeEventListener('hide.bs.modal', handler); modalRef.current.removeEventListener('hide.bs.modal', handler);
}
}; };
} }
}, [show, onClose]); }, [show, onClose]);
@@ -600,7 +602,9 @@ function DeleteServerModal({ show, server, onDelete, onClose }) {
const handler = () => onClose && onClose(); const handler = () => onClose && onClose();
modalRef.current.addEventListener('hide.bs.modal', handler); modalRef.current.addEventListener('hide.bs.modal', handler);
return () => { return () => {
if (modalRef.current) {
modalRef.current.removeEventListener('hide.bs.modal', handler); modalRef.current.removeEventListener('hide.bs.modal', handler);
}
}; };
} }
}, [show, onClose]); }, [show, onClose]);