Description Alexander Kläser 2013-05-10 17:22:12 CEST

This fix leads to a problem with the general validation of form elements. Upon validation, form elements are not anymore marked with a little exclamation mark to indicated that the chosen input is invalid. Workaround could be the following redefinition of umc/widgets/_FormWidgetMixin:validate():

validate: function() {
    var val = this.inherited(arguments);
    if (val === undefined) {
        return this.isValid();
    }
    return val;
}


+++ This bug was initially created as a clone of Bug #30109 +++

A form.validate() iterates over all form widgets and validates them. This is
done via widget.validate(). If this function is not found, the widget is
silently skipped, producing Bugs like the cloned one.

Every widget should implement a validate() function. At least PasswordInputBox
and TimeBox lack this functionality. In general, Widgets that encapsulate
dijit.form.Widgets through ContainerWidgets are prone to this error.

In addition, the form.validate() function should not skip a widget without
validate(), but instead fall back to isValid() and console.warn() the developer
(if not throw an exception...). This way we can also handle future Widgets that
forgot to implement the function.

+++ This bug was initially created as a clone of Bug #30091 +++

* It is possible to set an empty password for new users.
* It is not validated whether the first password matches the second. A red
exclamation mark may appear but that does not prevent the form from being
submitted. The first password input is chosen as the password.