hide password option is possible when password has correct length for… (#103)

* hide password option is possible when password has correct length for WEP and WPA

* refactor, logic is factored out into a function instead of in-lined in the markup

* refactor of function because Sonar warning

* improvement after code review - move function

1 files changed, 13 insertions, 0 deletions

diff --git a/src/components/Card.js b/src/components/Card.js
index ab0ec3f..4e7b0de 100644
--- a/src/components/Card.js
+++ b/src/components/Card.js
@@ -47,6 +47,18 @@ export const Card = ({ direction = 'ltr' }) => {
     }
   };
 
+  const disableHidePassword = () => {
+    const isWEPWithPasswordLengthShorterThat5Characters = () => {
+      return network.encryptionMode === 'WEP' && network.password.length < 5
+        ? true
+        : false;
+    };
+
+    return network.encryptionMode === 'WPA' && network.password.length < 8
+      ? true
+      : isWEPWithPasswordLengthShorterThat5Characters();
+  };
+
   useEffect(() => {
     if (firstLoad.current && window.innerWidth < 500) {
       firstLoad.current = false;
@@ -134,6 +146,7 @@ export const Card = ({ direction = 'ltr' }) => {
               <input
                 type="checkbox"
                 id="hide-password-checkbox"
+                disabled={disableHidePassword()}
                 className={network.encryptionMode === 'nopass' ? 'hidden' : ''}
                 onChange={() =>
                   setNetwork({