Fix #3081: maintain p-filled properly

[kalinkrustev]

Fix #3081

[melloware]

Instead of

    const currentValue = elementRef.current && elementRef.current.value;
    const isFilled = React.useMemo(() => (
        ObjectUtils.isNotEmpty(props.value) || ObjectUtils.isNotEmpty(props.defaultValue) || ObjectUtils.isNotEmpty(currentValue)
    ), [props.value, props.defaultValue, currentValue]);

Would it be better to lazy evaluate the current rather than store it in a const?

    const isFilled = React.useMemo(() => (
        ObjectUtils.isNotEmpty(props.value) || 
        ObjectUtils.isNotEmpty(props.defaultValue) || 
        (elementRef.current && ObjectUtils.isNotEmpty(elementRef.current.value))
    ), [props.value, props.defaultValue, elementRef.current]);

[kalinkrustev]

The problem is that elementRef.current.value must be in the dependencies, otherwise it might change (without a change to elementRef.current) without triggering reevaluation of the memoized value.

[melloware]

Ahh yes.