Revert callable type of Form::$onSuccess to accept mixed

[jiripudil]

• bug fix, related to Use phpstan callable definitions #216
• BC break? no
• doc PR: not needed

See forum thread. The import was missing, causing static analysis to report false positives.

[mabar]

https://github.com/nette/forms/blob/d9fc6d6/src/Forms/Form.php#L84

Could you also change (callable(Form, ArrayHash): void) to (callable(Form, mixed): void)? To prevent phpstan warnings for mapped types.

[mabar]

Currently best workaround is

function(Form $form, $mappedValues): void {
  /** @var MappedValues $values */
  $mappedValues = $mappedValues;
}

[jiripudil]

I've opened an issue for that (#224) but yes, mixed seems to be the way to go.

[adaamz]

@mabar I'm for ArrayHash => object, mixed is too wide.

[jiripudil]

Updated the PR to fix #224

[dg]

Thanks