validate passwordLength and add fields to validator manager (ITISFoundation#2478)

Author: odeimaiz

services/web/client/source/class/osparc/auth/core/Utils.js

@@ -20,6 +20,14 @@ qx.Class.define("osparc.auth.core.Utils", {
 
   statics:
   {
+    passwordLengthValidator: function(value, item) {
+      const valid = value != null && value.length > 3;
+      if (!valid) {
+        item.setInvalidMessage("Please enter a password at with least 4 characters.");
+      }
+      return valid;
+    },
+
     checkSamePasswords: function(passwordField1, passwordField2) {
       const isValid = passwordField1.getValue() == passwordField2.getValue();
       if (!isValid) {

services/web/client/source/class/osparc/auth/ui/RegistrationView.js

@@ -31,7 +31,6 @@ qx.Class.define("osparc.auth.ui.RegistrationView", {
   */
 
   members: {
-    __email: null,
     __submitBtn: null,
     __cancelBtn: null,
 
@@ -48,7 +47,6 @@ qx.Class.define("osparc.auth.ui.RegistrationView", {
       });
       this.add(email);
       osparc.utils.Utils.setIdToWidget(email, "registrationEmailFld");
-      this.__email = email;
       this.addListener("appear", () => {
         email.focus();
         email.activate();
@@ -78,9 +76,9 @@ qx.Class.define("osparc.auth.ui.RegistrationView", {
 
       // validation
       validator.add(email, qx.util.Validate.email());
-      validator.setValidator(function(_itemForms) {
-        return osparc.auth.core.Utils.checkSamePasswords(pass1, pass2);
-      });
+      validator.add(pass1, osparc.auth.core.Utils.passwordLengthValidator);
+      validator.add(pass2, osparc.auth.core.Utils.passwordLengthValidator);
+      validator.setValidator(() => osparc.auth.core.Utils.checkSamePasswords(pass1, pass2));
 
       // submit & cancel buttons
       const grp = new qx.ui.container.Composite(new qx.ui.layout.HBox(10));

services/web/client/source/class/osparc/auth/ui/ResetPassView.js

@@ -56,6 +56,8 @@ qx.Class.define("osparc.auth.ui.ResetPassView", {
       });
       this.add(code);
 
+      validator.add(password, osparc.auth.core.Utils.passwordLengthValidator);
+      validator.add(confirm, osparc.auth.core.Utils.passwordLengthValidator);
       validator.setValidator(function(_itemForms) {
         return osparc.auth.core.Utils.checkSamePasswords(password, confirm);
       });

services/web/client/source/class/osparc/desktop/preferences/pages/SecurityPage.js

@@ -57,6 +57,8 @@ qx.Class.define("osparc.desktop.preferences.pages.SecurityPage", {
       box.add(confirm);
 
       const manager = new qx.ui.form.validation.Manager();
+      manager.add(newPassword, osparc.auth.core.Utils.passwordLengthValidator);
+      manager.add(confirm, osparc.auth.core.Utils.passwordLengthValidator);
       manager.setValidator(function(_itemForms) {
         return osparc.auth.core.Utils.checkSamePasswords(newPassword, confirm);
       });