PropertyDefinition.getType() silently converts primitive class to its boxed counterpart

Description

I’d expect PropertyDefinition.getType() to return the actual type of the field (e.g. int.class) and not the boxed counterpart (Integer.class). Please either make getType() return the actual type, or document that it will return the class of the boxed counterpart (+provide a function named getActualType() that returns the original type, e.g. primitive int.class).

Expected outcome

Minimal reproducible example

Steps to reproduce

Environment

Vaadin version(s): 14.8.11 OS: Windows

Browsers

No response

[mvysny]

Thank you Leif. My use-case: calculating default values. The default value of int.class is 0 while of Integer.class is null. I was building a FilterBar showing closable filter “pills” for filter fields with different than the default value.

In this example, the “Active Only” filter bean is primitive boolean with the default value of false; the FilterBar only displays the “Active Only” “pill” when it’s different from the default value, which in this case is false. For booleans it probably doesn’t make much sense (3-state vs 2-state boolean logic is not a sound example 😄 ) but it makes much more sense for, say, int.class.

Note the red cross button which resets the filter to the default value, which is null for Objects but a non-null value for primitives. I was using the PropertyDefinition API to figure out the default value and setting nulls via reflection, which gave me exceptions… hence the issue.

[DiegoCardoso]

Sorry, can we move this to vaadin/flow?

Done.