`<PasswordInput/>` or `<TextInput/>` has corrupted `validate` prop, if field is handled with `disabled` prop about react-admin HOT 4 OPEN

Hi @silversonicaxel, I'm not sure to understand when, where or how you want to use this code snippet. Could you apply it a Stackblitz please

from react-admin.

@erwanMarmelab here my example:
https://stackblitz.com/edit/github-eqexed-qzclqx?file=src%2Fposts%2FPostEdit.tsx

If you go to that link, follow the steps to see the two behaviours I am mentioning:

• click on a Post item in the Posts Lists page
• now in the Posts Edit page click on "Activate password change" - this toggle enables or disables the password field
• write in the password field two characters
• save the form

Wrong implementation (missing the disabled password prop), without the bug
With the current and wrong implementation of the Stackblitz, in file src/posts/PostEdit.tsx at line 140 with disabled prop totally commented out.
The outcome of the previous steps is the error, with the validate message requiring 8 characters to set a correct password field.

Correct implementation (with the disabled password prop), but WITH the bug
Now, in order to see the bug, uncomment in file src/posts/PostEdit.tsx at line 140 the disabled prop.
The outcome of the previous steps is a submitted form, with the element saved, and the validate rule ignored.

I hope things are more clear now.

from react-admin.

Confirmed, thanks for the report.

For the moment, you can validate your edit page by adding a form validation like this:

const validatePost = values => {
    const errors = {};
    if (values.activator && values.password.length < 8) {
        errors.password = 'Password at least 8 characters';
    }
    // other validations...
    return errors;
};

const PostEdit = () => {
    ...
    return (
        <Edit>
            <TabbedForm validate={validatePost}>
                ...
            </TabbedForm>
        </Edit>
    );
};

from react-admin.

Thanks!

from react-admin.