mirror of
https://github.com/Respect/Validation.git
synced 2024-06-04 14:52:17 +02:00
97b243daa1
Although helpful, the changes in the Min, Max, and Length rules made using those rules more verbose. This commit will simplify their use by allowing users to use them as prefixes. Because I was creating prefixes for those rules, I made other cool prefixes. Doing that is scary because it will generate more code to support, and I would have liked to avoid that. However, that's a valuable addition, and it's worth the risk. I might reconsider that in the future, but for now, that looks like a good idea. Signed-off-by: Henrique Moody <henriquemoody@gmail.com>
82 lines
2.4 KiB
PHP
82 lines
2.4 KiB
PHP
<?php
|
|
|
|
/*
|
|
* Copyright (c) Alexandre Gomes Gaigalas <alganet@gmail.com>
|
|
* SPDX-License-Identifier: MIT
|
|
*/
|
|
|
|
declare(strict_types=1);
|
|
|
|
namespace Respect\Validation\Transformers;
|
|
|
|
use Respect\Validation\Validatable;
|
|
|
|
use function Respect\Stringifier\stringify;
|
|
use function sprintf;
|
|
use function trigger_error;
|
|
|
|
use const E_USER_DEPRECATED;
|
|
|
|
final class DeprecatedLength implements Transformer
|
|
{
|
|
public function __construct(
|
|
private readonly Transformer $next
|
|
) {
|
|
}
|
|
|
|
public function transform(RuleSpec $ruleSpec): RuleSpec
|
|
{
|
|
if ($ruleSpec->name !== 'length' || $ruleSpec->arguments === []) {
|
|
return $this->next->transform($ruleSpec);
|
|
}
|
|
|
|
if (isset($ruleSpec->arguments[0]) && $ruleSpec->arguments[0] instanceof Validatable) {
|
|
return $this->next->transform($ruleSpec);
|
|
}
|
|
|
|
$minValue = $ruleSpec->arguments[0] ?? null;
|
|
$maxValue = $ruleSpec->arguments[1] ?? null;
|
|
$inclusive = $ruleSpec->arguments[2] ?? true;
|
|
|
|
$message = 'Calling length() with scalar values has been deprecated, ' .
|
|
'and will not be allowed in the next major version. ';
|
|
|
|
if (!$minValue) {
|
|
$name = $inclusive ? 'LessThanOrEqual' : 'LessThan';
|
|
trigger_error(
|
|
sprintf($message . 'Use length%s(%s) instead.', $name, stringify($maxValue)),
|
|
E_USER_DEPRECATED
|
|
);
|
|
|
|
return new RuleSpec($name, [$maxValue], new RuleSpec('length'));
|
|
}
|
|
|
|
if (!$maxValue) {
|
|
$name = $inclusive ? 'GreaterThanOrEqual' : 'GreaterThan';
|
|
trigger_error(
|
|
sprintf($message . 'Use length%s(%s) instead.', $name, stringify($minValue)),
|
|
E_USER_DEPRECATED
|
|
);
|
|
|
|
return new RuleSpec($name, [$minValue], new RuleSpec('length'));
|
|
}
|
|
|
|
if ($minValue === $maxValue) {
|
|
trigger_error(
|
|
sprintf($message . 'Use lengthEquals(%s) instead.', stringify($minValue)),
|
|
E_USER_DEPRECATED
|
|
);
|
|
|
|
return new RuleSpec('equals', [$minValue], new RuleSpec('length'));
|
|
}
|
|
|
|
$name = $inclusive ? 'Between' : 'BetweenExclusive';
|
|
trigger_error(
|
|
sprintf($message . 'Use length%s(%s, %s) instead.', $name, stringify($minValue), stringify($maxValue)),
|
|
E_USER_DEPRECATED
|
|
);
|
|
|
|
return new RuleSpec($name, [$minValue, $maxValue], new RuleSpec('length'));
|
|
}
|
|
}
|