[Naming][Php56] Do not add default value init for renamed variable from param on RenameParamToMatchTypeRector+AddDefaultValueForUndefinedVariableRector

[samsonasik]

Given the following code:

use Rector\BetterPhpDocParser\ValueObject\Type\BracketsAwareUnionTypeNode;

class Fixture
{
    public function hasTruePseudoType(BracketsAwareUnionTypeNode $type): bool
    {
        $unionTypes = $type->types;

        foreach ($unionTypes as $unionType) {}
    }
}

It currently produce define init null value which invalid as variable is from renamed param:

     public function hasTruePseudoType(BracketsAwareUnionTypeNode $bracketsAwareUnionTypeNode): bool
     {
+        $bracketsAwareUnionTypeNode = null;
         $unionTypes = $bracketsAwareUnionTypeNode->types;

Applied rules:

Rector\Naming\Rector\ClassMethod\RenameParamToMatchTypeRector;
Rector\Php56\Rector\FunctionLike\AddDefaultValueForUndefinedVariableRector;

[samsonasik]

Fixed 🎉

[samsonasik]

I am merging it, the functionality was removed at #1697 that was still working due to direct use of renamed parameter, while if it is assigned back on this PR use case, it will failure.

This patch rolled back the detection functionality with the assigned back use case.