com.vaadin.data.BinderValidationStatus bug?

Vaadin 6 / 7 / 8 Framework Data Binding
1

Hello there,
I’m very early in the process of migrating my V7 app to V8 and so I’m still learning the new data/fields binding mechanism and I might be wrong here but, aren’t lines 93-95 from BinderValidationStatus constructor an obvious copy+paste mistake?

This snippet is taken from the master repo on Github, but 8.4 branch is the same: shouldn’t be bindingStatuses (instead of binderStatus) the argument passed to the first Objects.requireNonNull call on line 93?

2

Yep, I’d say you are correct.

-Olli

3

Pull request to fix this issue here: https://github.com/vaadin/framework/pull/10964

-Olli