From 1610c8416a530c917053c03d2d1b48e1ea683b2f Mon Sep 17 00:00:00 2001 From: Sofiya Tepikin Date: Tue, 13 Sep 2022 19:47:11 +0200 Subject: [PATCH] Change the API to remove nodejs listeners --- components/password-prompt.js | 4 ++-- components/settings/common/confirm-with-password.js | 10 ++++++---- 2 files changed, 8 insertions(+), 6 deletions(-) diff --git a/components/password-prompt.js b/components/password-prompt.js index 8e8e86a..55236ba 100644 --- a/components/password-prompt.js +++ b/components/password-prompt.js @@ -36,9 +36,9 @@ const PasswordPrompt = ({ enableShowApp }) => { } useEffect(() => { - nodejs.channel.addListener('check-pw', passHashToDb, this) + const listener = nodejs.channel.addListener('check-pw', passHashToDb, this) - return () => nodejs.channel.remove('check-pw', passHashToDb) + return () => listener.remove() }, []) const onDeleteDataConfirmation = async () => { diff --git a/components/settings/common/confirm-with-password.js b/components/settings/common/confirm-with-password.js index 4fceacb..ee3bb81 100644 --- a/components/settings/common/confirm-with-password.js +++ b/components/settings/common/confirm-with-password.js @@ -24,10 +24,12 @@ const ConfirmWithPassword = ({ onSuccess, onCancel }) => { } useEffect(() => { - nodejs.channel.addListener('password-check', checkPassword, this) - return () => { - nodejs.channel.removeListener('password-check', checkPassword) - } + const listener = nodejs.channel.addListener( + 'password-check', + checkPassword, + this + ) + return () => listener.remove() }, []) const onIncorrectPassword = () => {