Filled validation in custom Nette\Forms\Control instance

[hrach]

I have a custom form control implementing Nette\Forms\Control interface.

I want to add a common rule to check if that control is filled. Sadly, the default implementation delegates to the static Validator class that requires a BaseControl subtype.

forms/src/Forms/Validator.php

Line 162 in fcfc86c

public static function validateFilled(Controls\BaseControl $control): bool

I worked around this by providing custom rules that override the filled check:

class CustomControlRules extends Rules
{
	public static function validateRule(Rule $rule): bool
	{
		if ($rule->validator === Form::Filled) {
			$control = $rule->control;
			if (!$control instanceof \MyCustomImplementation) {
				throw new InvalidStateException();
			}
			return $control->isFilled();
		} else {
			return parent::validateRule($rule);
		}
	}
}

But, this is getting forbidden by the recent (Forms 3.2) changes that are making the Rules class final.

The simple solution is to remove the final once again, but since I got to report this, I'd ask you to consider some proper LSP solution -> ideally moving the required methods to the interface.

[dg]

Yeah, feel free to suggest some proper solutions.

[hrach]

Which solution do you prefer?

1. Add isFilled() to Control (BC break)
2. Add isFilled() to Control but commented out (not a BC break, at least for now), add use method_exists
3. Add ValidatedControl interface and implement in BaseControl
   a. add there just validateFilled & validateMinMax (to extract this datetime logic)
   b. add there more

[hrach]

Another option implemented in #347.

[hrach]

@dg any opinion on this? thx.

[hrach]

@dg gentle reminder