mirror of
https://github.com/Respect/Validation.git
synced 2024-06-29 02:30:06 +02:00
1b844763a9
When using nested "Not" and "AllOf" rules Validation does not fetch the message of the rule that failed, and instead it fetches the default message in "AllOfException." That is because "Not" cannot reach the rules inside "AllOf," making it impossible to fetch the correct message. This commit will improve that a bit but making "Not" deal directly with the rule inside "AllOf" when it has only one rule. Unfortunately, it will not fix all the issues. For example, when the negated rule is an "AllOf" with multiple rules and it fails "Not" will fetch only the first message. Signed-off-by: Henrique Moody <henriquemoody@gmail.com>
119 lines
2.6 KiB
PHP
119 lines
2.6 KiB
PHP
<?php
|
|
|
|
/*
|
|
* This file is part of Respect/Validation.
|
|
*
|
|
* (c) Alexandre Gomes Gaigalas <alexandre@gaigalas.net>
|
|
*
|
|
* For the full copyright and license information, please view the "LICENSE.md"
|
|
* file that was distributed with this source code.
|
|
*/
|
|
|
|
declare(strict_types=1);
|
|
|
|
namespace Respect\Validation\Rules;
|
|
|
|
use Respect\Validation\Exceptions\ValidationException;
|
|
use Respect\Validation\Validatable;
|
|
use function array_shift;
|
|
use function count;
|
|
use function current;
|
|
|
|
/**
|
|
* @author Alexandre Gomes Gaigalas <alexandre@gaigalas.net>
|
|
* @author Caio César Tavares <caiotava@gmail.com>
|
|
* @author Henrique Moody <henriquemoody@gmail.com>
|
|
*/
|
|
final class Not extends AbstractRule
|
|
{
|
|
/**
|
|
* @var Validatable
|
|
*/
|
|
private $rule;
|
|
|
|
public function __construct(Validatable $rule)
|
|
{
|
|
$this->rule = $this->extractNegatedRule($rule);
|
|
}
|
|
|
|
public function getNegatedRule(): Validatable
|
|
{
|
|
return $this->rule;
|
|
}
|
|
|
|
public function setName(string $name): Validatable
|
|
{
|
|
$this->rule->setName($name);
|
|
|
|
return parent::setName($name);
|
|
}
|
|
|
|
/**
|
|
* {@inheritDoc}
|
|
*/
|
|
public function validate($input): bool
|
|
{
|
|
return $this->rule->validate($input) === false;
|
|
}
|
|
|
|
/**
|
|
* {@inheritDoc}
|
|
*/
|
|
public function assert($input): void
|
|
{
|
|
if ($this->validate($input)) {
|
|
return;
|
|
}
|
|
|
|
$rule = $this->rule;
|
|
if ($rule instanceof AllOf) {
|
|
$rule = $this->absorbAllOf($rule, $input);
|
|
}
|
|
|
|
$exception = $rule->reportError($input);
|
|
$exception->updateMode(ValidationException::MODE_NEGATIVE);
|
|
|
|
throw $exception;
|
|
}
|
|
|
|
/**
|
|
* @param mixed $input
|
|
*/
|
|
private function absorbAllOf(AllOf $rule, $input): Validatable
|
|
{
|
|
$rules = $rule->getRules();
|
|
while (($current = array_shift($rules))) {
|
|
$rule = $current;
|
|
if (!$rule instanceof AllOf) {
|
|
continue;
|
|
}
|
|
|
|
if (!$rule->validate($input)) {
|
|
continue;
|
|
}
|
|
|
|
$rules = $rule->getRules();
|
|
}
|
|
|
|
return $rule;
|
|
}
|
|
|
|
private function extractNegatedRule(Validatable $rule): Validatable
|
|
{
|
|
if ($rule instanceof self && $rule->getNegatedRule() instanceof self) {
|
|
return $this->extractNegatedRule($rule->getNegatedRule()->getNegatedRule());
|
|
}
|
|
|
|
if (!$rule instanceof AllOf) {
|
|
return $rule;
|
|
}
|
|
|
|
$rules = $rule->getRules();
|
|
if (count($rules) === 1) {
|
|
return $this->extractNegatedRule(current($rules));
|
|
}
|
|
|
|
return $rule;
|
|
}
|
|
}
|