Admin

name	type	size	perms	modified	actions
[parent directory]
AddMethodCallBasedStrictParamTypeRector.php	file	3,828 B	0644	2024-11-08 13:59:10	edit
AddParamArrayDocblockBasedOnCallableNativeFuncCallRector.php	file	8,848 B	0644	2024-11-08 13:59:10	edit
AddParamTypeBasedOnPHPUnitDataProviderRector.php	file	11,687 B	0644	2024-11-08 13:59:10	edit
AddParamTypeDeclarationRector.php	file	5,752 B	0644	2024-11-08 13:59:10	edit
AddParamTypeFromPropertyTypeRector.php	file	6,315 B	0644	2024-11-08 13:59:10	edit
AddReturnArrayDocblockBasedOnArrayMapRector.php	file	5,838 B	0644	2024-11-08 13:59:10	edit
AddReturnTypeDeclarationBasedOnParentClassMethodRector.php	file	5,652 B	0644	2024-11-08 13:59:10	edit
AddReturnTypeDeclarationRector.php	file	4,770 B	0644	2024-11-08 13:59:10	edit
AddTypeFromResourceDocblockRector.php	file	6,959 B	0644	2024-11-08 13:59:10	edit
AddVoidReturnTypeWhereNoReturnRector.php	file	4,454 B	0644	2024-11-08 13:59:10	edit
BoolReturnTypeFromBooleanConstReturnsRector.php	file	4,300 B	0644	2024-11-08 13:59:10	edit
BoolReturnTypeFromBooleanStrictReturnsRector.php	file	7,695 B	0644	2024-11-08 13:59:10	edit
NumericReturnTypeFromStrictReturnsRector.php	file	5,240 B	0644	2024-11-08 13:59:10	edit
NumericReturnTypeFromStrictScalarReturnsRector.php	file	4,338 B	0644	2024-11-08 13:59:10	edit
ParamTypeByMethodCallTypeRector.php	file	6,677 B	0644	2024-11-08 13:59:10	edit
ParamTypeByParentCallTypeRector.php	file	4,787 B	0644	2024-11-08 13:59:10	edit
ReturnNeverTypeRector.php	file	1,973 B	0644	2024-11-08 13:59:10	edit
ReturnNullableTypeRector.php	file	3,772 B	0644	2024-11-08 13:59:10	edit
ReturnTypeFromMockObjectRector.php	file	4,839 B	0644	2024-11-08 13:59:10	edit
ReturnTypeFromReturnCastRector.php	file	2,271 B	0644	2024-11-08 13:59:10	edit
ReturnTypeFromReturnDirectArrayRector.php	file	3,564 B	0644	2024-11-08 13:59:10	edit
ReturnTypeFromReturnNewRector.php	file	9,510 B	0644	2024-11-08 13:59:10	edit
ReturnTypeFromStrictConstantReturnRector.php	file	4,607 B	0644	2024-11-08 13:59:10	edit
ReturnTypeFromStrictFluentReturnRector.php	file	4,607 B	0644	2024-11-08 13:59:10	edit
ReturnTypeFromStrictNativeCallRector.php	file	2,105 B	0644	2024-11-08 13:59:10	edit
ReturnTypeFromStrictNewArrayRector.php	file	9,598 B	0644	2024-11-08 13:59:10	edit
ReturnTypeFromStrictParamRector.php	file	2,020 B	0644	2024-11-08 13:59:10	edit
ReturnTypeFromStrictTypedCallRector.php	file	8,902 B	0644	2024-11-08 13:59:10	edit
ReturnTypeFromStrictTypedPropertyRector.php	file	5,589 B	0644	2024-11-08 13:59:10	edit
ReturnTypeFromSymfonySerializerRector.php	file	4,180 B	0644	2024-11-08 13:59:10	edit
ReturnUnionTypeRector.php	file	2,220 B	0644	2024-11-08 13:59:10	edit
StrictArrayParamDimFetchRector.php	file	8,913 B	0644	2024-11-08 13:59:10	edit
StrictStringParamConcatRector.php	file	6,622 B	0644	2024-11-08 13:59:10	edit
StringReturnTypeFromStrictScalarReturnsRector.php	file	3,920 B	0644	2024-11-08 13:59:10	edit
StringReturnTypeFromStrictStringReturnsRector.php	file	4,959 B	0644	2024-11-08 13:59:10	edit

editing: StrictStringParamConcatRector.php

<?php

declare (strict_types=1);
namespace Rector\TypeDeclaration\Rector\ClassMethod;

use PhpParser\Node;
use PhpParser\Node\Expr;
use PhpParser\Node\Expr\Assign;
use PhpParser\Node\Expr\AssignOp\Concat;
use PhpParser\Node\Expr\Closure;
use PhpParser\Node\Expr\Variable;
use PhpParser\Node\FunctionLike;
use PhpParser\Node\Identifier;
use PhpParser\Node\NullableType;
use PhpParser\Node\Param;
use PhpParser\Node\Stmt\Class_;
use PhpParser\Node\Stmt\ClassMethod;
use PhpParser\Node\Stmt\Function_;
use PhpParser\NodeTraverser;
use PHPStan\Type\MixedType;
use PHPStan\Type\Type;
use PHPStan\Type\TypeCombinator;
use Rector\BetterPhpDocParser\PhpDocInfo\PhpDocInfoFactory;
use Rector\Rector\AbstractRector;
use Rector\VendorLocker\ParentClassMethodTypeOverrideGuard;
use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
use Symplify\RuleDocGenerator\ValueObject\RuleDefinition;
/**
 * @see \Rector\Tests\TypeDeclaration\Rector\ClassMethod\StrictStringParamConcatRector\StrictStringParamConcatRectorTest
 */
final class StrictStringParamConcatRector extends AbstractRector
{
    /**
     * @readonly
     * @var \Rector\VendorLocker\ParentClassMethodTypeOverrideGuard
     */
    private $parentClassMethodTypeOverrideGuard;
    /**
     * @readonly
     * @var \Rector\BetterPhpDocParser\PhpDocInfo\PhpDocInfoFactory
     */
    private $phpDocInfoFactory;
    public function __construct(ParentClassMethodTypeOverrideGuard $parentClassMethodTypeOverrideGuard, PhpDocInfoFactory $phpDocInfoFactory)
    {
        $this->parentClassMethodTypeOverrideGuard = $parentClassMethodTypeOverrideGuard;
        $this->phpDocInfoFactory = $phpDocInfoFactory;
    }
    public function getRuleDefinition() : RuleDefinition
    {
        return new RuleDefinition('Add string type based on concat use', [new CodeSample(<<<'CODE_SAMPLE'
class SomeClass
{
    public function resolve($item)
    {
        return $item . ' world';
    }
}
CODE_SAMPLE
, <<<'CODE_SAMPLE'
class SomeClass
{
    public function resolve(string $item)
    {
        return $item . ' world';
    }
}
CODE_SAMPLE
)]);
    }
    /**
     * @return array<class-string<Node>>
     */
    public function getNodeTypes() : array
    {
        return [ClassMethod::class, Function_::class, Closure::class];
    }
    /**
     * @param ClassMethod|Function_|Closure $node
     */
    public function refactor(Node $node) : ?Node
    {
        if ($node instanceof ClassMethod && $this->parentClassMethodTypeOverrideGuard->hasParentClassMethod($node)) {
            return null;
        }
        $hasChanged = \false;
        $phpDocInfo = $this->phpDocInfoFactory->createFromNodeOrEmpty($node);
        foreach ($node->getParams() as $param) {
            if ($param->type instanceof Node) {
                continue;
            }
            $variableConcattedFromParam = $this->resolveVariableConcattedFromParam($param, $node);
            if (!$variableConcattedFromParam instanceof Variable) {
                continue;
            }
            $paramDocType = $phpDocInfo->getParamType($this->getName($param));
            if (!$paramDocType instanceof MixedType && !$paramDocType->isString()->yes()) {
                continue;
            }
            $nativeType = $this->nodeTypeResolver->getNativeType($variableConcattedFromParam);
            if (!$nativeType instanceof MixedType) {
                continue;
            }
            $subtractedType = $nativeType->getSubtractedType();
            if (!$subtractedType instanceof Type) {
                $param->type = new Identifier('string');
                $hasChanged = \true;
                continue;
            }
            if (TypeCombinator::containsNull($subtractedType)) {
                $param->type = new NullableType(new Identifier('string'));
                $hasChanged = \true;
            }
        }
        if ($hasChanged) {
            return $node;
        }
        return null;
    }
    /**
     * @param \PhpParser\Node\Stmt\ClassMethod|\PhpParser\Node\Stmt\Function_|\PhpParser\Node\Expr\Closure $functionLike
     */
    private function resolveVariableConcattedFromParam(Param $param, $functionLike) : ?Variable
    {
        if ($functionLike->stmts === null) {
            return null;
        }
        if ($param->default instanceof Expr && !$this->getType($param->default)->isString()->yes()) {
            return null;
        }
        $paramName = $this->getName($param);
        $variableConcatted = null;
        $this->traverseNodesWithCallable($functionLike->stmts, function (Node $node) use($paramName, &$variableConcatted) : ?int {
            // skip nested class and function nodes
            if ($node instanceof FunctionLike || $node instanceof Class_) {
                return NodeTraverser::DONT_TRAVERSE_CURRENT_AND_CHILDREN;
            }
            if ($node instanceof Assign && $node->var instanceof Variable && $this->isName($node->var, $paramName)) {
                $variableConcatted = null;
                return NodeTraverser::STOP_TRAVERSAL;
            }
            $expr = $this->resolveAssignConcatVariable($node, $paramName);
            if ($expr instanceof Variable) {
                $variableConcatted = $expr;
            }
            $variableBinaryConcat = $this->resolveBinaryConcatVariable($node, $paramName);
            if ($variableBinaryConcat instanceof Variable) {
                $variableConcatted = $variableBinaryConcat;
            }
            return null;
        });
        return $variableConcatted;
    }
    private function isVariableWithSameParam(Expr $expr, string $paramName) : bool
    {
        if (!$expr instanceof Variable) {
            return \false;
        }
        return $this->isName($expr, $paramName);
    }
    private function resolveAssignConcatVariable(Node $node, string $paramName) : ?Expr
    {
        if (!$node instanceof Concat) {
            return null;
        }
        if ($this->isVariableWithSameParam($node->var, $paramName)) {
            return $node->var;
        }
        if ($this->isVariableWithSameParam($node->expr, $paramName)) {
            return $node->expr;
        }
        return null;
    }
    private function resolveBinaryConcatVariable(Node $node, string $paramName) : ?Expr
    {
        if (!$node instanceof Expr\BinaryOp\Concat) {
            return null;
        }
        if ($this->isVariableWithSameParam($node->left, $paramName)) {
            return $node->left;
        }
        if ($this->isVariableWithSameParam($node->right, $paramName)) {
            return $node->right;
        }
        return null;
    }
}

cancel