[DeadCode] Handle multiple default on RemoveArgumentFromDefaultParentCallRector

[samsonasik]

On this code:

final class MultipleDefault extends Exception
{
    public function __construct(string $message)
    {
        parent::__construct($message, 0, null);
    }
}

0 and null are default parameters, so both needs to be removed.

-        parent::__construct($message, 0, null);
+        parent::__construct($message);

[samsonasik]

The issue is on ValueResolver itself, which convert null to string "null"

rector-src/src/PhpParser/Node/Value/ValueResolver.php

Lines 94 to 96 in 39ab4b6

	if ($expr instanceof ConstFetch) {
	return $this->nodeNameResolver->getName($expr);
	}

as it returns name on ConstFetch

[samsonasik]

Fixed by show returns value on null, true, and false on ValueResolver 4276016

[samsonasik]

All checks have passed 🎉 @TomasVotruba I am merging it ;)

[github-actions]

This pull request has been automatically locked because it has been closed for 150 days. Please open a new PR if you want to continue the work.