Open weierophinney opened 4 years ago
In which context do you want to create and execute the chain. I think this could lead to heavy objects, because there are so much dependencies (Hydrator, Validator, Filter, ...).
And in my opinion the code is really hard to read.
But aside of this, it could be a great feature.
Originally posted by @zf2timo at https://github.com/zendframework/zend-inputfilter/issues/87#issuecomment-173200194
Chain can be created from static methods and added to input filters
$inputFilter->addInput("input name", Chain $chain);
$result = $inputFilter->resolve($inputValuesArray);
$result->isValid();
$result->getValues();
About the code legibility this is as easy as use local vars or refactor dependent components for be API compatible.
$removeWhitespaces = ...;
$validateIsFormattedAsADate = ...;
$chain
->then($removeWhitespaces)
->then($validateIsFormattedAsADate)
->then(null, $setFallbackValue)
;
About the dependencies, there is no dependencies, just callables.
Originally posted by @Maks3w at https://github.com/zendframework/zend-inputfilter/issues/87#issuecomment-173212144
Note this way of make chains could replace or remove ValidatorChain and FilterChain classes
Originally posted by @Maks3w at https://github.com/zendframework/zend-inputfilter/issues/87#issuecomment-173213893
while I like the idea of replacing *Chain
classes with pipes of callbacks, I'm not keen on using A+ promises terminology for something that is not async at all. It may be confusing for people not familiar with such concepts, and it feels like mixing apples and oranges.
Originally posted by @stefanotorresi at https://github.com/zendframework/zend-inputfilter/issues/87#issuecomment-186176094
Well, who said this things could not be async in the future or with alternative PHP engines.
Originally posted by @Maks3w at https://github.com/zendframework/zend-inputfilter/issues/87#issuecomment-186380547
If that was the actual intention, I'd rather have the input filter wrapped in a promise via a dedicated implementation like reactphp/promise, or introduce a new zend component for that purpose, but then again... why would you do that? A promise-like interface has many aspects that are not that useful in this particular case, all you want to do is just pass an array through a stack of callbacks... Why not rethink the current Chain class to be more generic and simple instead, and avoid the design overhead of things like a "fake-promise" resolution and rejection?
Originally posted by @stefanotorresi at https://github.com/zendframework/zend-inputfilter/issues/87#issuecomment-186425862
I started to think the best way of build a custom pipe of filters and validators where you can filter after validation, etc.
I propose replace
Zend\InputFilter\Input
with a custom chain of methods like A+ promises.For resume A+ promises have the following main properties along others I won't detail here:
then(callback success = null, callback error = null)
So basically the chain do two kind of things. Transformations of the input value and reject the chain if is invalid.
Options:
Reject promise using a reject method callback instead throwing exceptions.
->then($reject($errror) {}, $inputValue);
Benefits of this design:
Thoughts?
/cc @weierophinney, @Ocramius, @bakura10, @zendframework/community-review-team
Originally posted by @Maks3w at https://github.com/zendframework/zend-inputfilter/issues/87