Binder writeBeanIfValid writes invalid values to the bean

See example below, typing a number > 1000 or a number < -1000 in the field and pressing “validate” to store it, should NEVER store the invalid values in the bean.

The javadoc of setMax also tells me:

Sets the maximum value of the field. Entering a value which is greater than code max invalidates the field.

Vaadin Version: 14.1.18 Browser: FF 74

@Route("bug-min-max")
public class IntegerFieldMinMax extends Composite<VerticalLayout> {

  public IntegerFieldMinMax() {
    IntegerField field = new IntegerField("Type here");
    field.setMax(1000);
    field.setMin(-1000);
    field.addValueChangeListener(evt -> Notification.show("Changed value to: " + evt.getValue(), 3000, Position.TOP_START));

    IntergerObject bean = new IntergerObject(1);
    Binder<IntergerObject> binder = new Binder<>();
    binder.forField(field).asRequired("Integer is required")
      .bind(IntergerObject::getValue, IntergerObject::setValue);
    binder.readBean(bean);

    Button validate = new Button("Validate", evt -> {
      if (binder.writeBeanIfValid(bean)) {
        if(bean.getValue() > 1000) {
          Notification.show("This is not valid - should NEVER been STORED. Field is configured with max = " + field.getMax()).addThemeVariants(NotificationVariant.LUMO_ERROR);
        }else if(bean.getValue() < -1000) {
          Notification.show("This is not valid - should NEVER been STORED. Field is configured with min = " + field.getMin()).addThemeVariants(NotificationVariant.LUMO_ERROR);
        }
        Notification.show("Stored valid value: " + bean.getValue(), 3000, Position.TOP_START);
      }
    });

    getContent().add(field, validate);
  }

  @Data // Lombok Annotation
  @AllArgsConstructor // Lombok Annotation
  private static class IntergerObject {

    private Integer value;
  }
}

[Legioth]

This is a missing feature that should be ported from Vaadin 8: https://github.com/vaadin/flow/issues/6803

[pleku]

I’ve asked to transfer the issue or make that repository public but until something happens