Skip to content

Commit f1b5543

Browse files
committed
deprecate ReplaceArgumentDefaultValue value object, move shared sentinel to interface
1 parent 6253d93 commit f1b5543

3 files changed

Lines changed: 4 additions & 5 deletions

File tree

‎rules/Arguments/ArgumentDefaultValueReplacer.php‎

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@
2020
use PHPStan\Reflection\MethodReflection;
2121
use PHPStan\Reflection\ParametersAcceptorSelector;
2222
use Rector\Arguments\Contract\ReplaceArgumentDefaultValueInterface;
23-
use Rector\Arguments\ValueObject\ReplaceArgumentDefaultValue;
2423
use Rector\NodeAnalyzer\ArgsAnalyzer;
2524
use Rector\NodeTypeResolver\NodeTypeResolver;
2625
use Rector\PhpParser\Node\NodeFactory;
@@ -63,7 +62,7 @@ public function processReplaces(
6362
private function isDefaultValueMatched(?Expr $expr, mixed $value): bool
6463
{
6564
// allow any values before, also allow param without default value
66-
if ($value === ReplaceArgumentDefaultValue::ANY_VALUE_BEFORE) {
65+
if ($value === ReplaceArgumentDefaultValueInterface::ANY_VALUE_BEFORE) {
6766
return true;
6867
}
6968

‎rules/Arguments/Contract/ReplaceArgumentDefaultValueInterface.php‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,8 @@
66

77
interface ReplaceArgumentDefaultValueInterface
88
{
9+
public const string ANY_VALUE_BEFORE = '*ANY_VALUE_BEFORE*';
10+
911
public function getPosition(): int;
1012

1113
public function getValueBefore(): mixed;

‎rules/Arguments/ValueObject/ReplaceArgumentDefaultValue.php‎

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,12 +9,10 @@
99
use Rector\Validation\RectorAssert;
1010

1111
/**
12-
* @api used in rector-symfony
12+
* @deprecated Use custom rule instead, see ReplaceArgumentDefaultValueRector deprecation.
1313
*/
1414
final readonly class ReplaceArgumentDefaultValue implements ReplaceArgumentDefaultValueInterface
1515
{
16-
public const string ANY_VALUE_BEFORE = '*ANY_VALUE_BEFORE*';
17-
1816
/**
1917
* @param int<0, max> $position
2018
*/

0 commit comments

Comments
 (0)