read only Fields should not be validated

[gitana]

read only Fields should not be validated

[uzquiano]

Can you provide a specific example of where this is applicable?

A few thoughts -

1. If we have a JSON structure which is inherently invalid (given a schema), then it's invalid regardless of whether the fields on it are readonly or not. That is to say, all fields should always be validated and the structure as a whole should be considered invalid regardless of the user interface or the user's ability to do anything about it.
2. If a field is marked as readonly, then as you point out, the user cannot do anything about it. So perhaps what you're saying is that the user interface elements (the red box) should not appear since it is provoking the user to do something, when in fact, they can't do anything. Which is confusing... is that right?
3. If we solve #2 by removing the red box, it wouldn't change the fact that the JSON is invalid. Thus, "saving a form" or taking action with it should still be prevented. However, without the red box or some kind of indication to to the end user as to why they cannot proceed, it could also be confusing.

In the latter case, I think the issue is really the app developers then because they've coded a form that the end user cannot work with. So perhaps we need to detect that and have the warning log to console (debug) that a developer could catch.

What do you think?

[uzquiano]

I checked in a change that implements what was described. The user interface "red-box" stuff is restrained so the end user doesn't see it. However, the top object still reports that it is invalid (i.e. control.isValid(true) === false) since one of its child elements is invalid.

The fact that the child element is invalid and also readonly (meaning that the app developer has stranded the end user in an unfortunate and helpless position) is now logged to console at WARN level.

Let me know if this implementation aligns with what you were thinking?

[uzquiano]

Going to mark this as closed. Please indicate if there's something still amiss or if we should make other adjustments?

Thanks!