[Php81] Skip param variable used in next stmt after coalesce on NewInInitializerRector

[samsonasik]

Fixes rectorphp/rector#8673

[samsonasik]

Fixed 🎉 /cc @VincentLanglet

[samsonasik]

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

[VincentLanglet]

I'm not familiar with rector code, so what does nextstmt means ? @samsonasik

Does it also fix the bug for code like

public function __construct(?\DateTime $time = null)
    {
        $this->timeWasSet = null !== $time; // Here the statement is before 
        $this->time = $time ?? new \DateTime();
    }

or

public function __construct(?\DateTime $time = null)
    {
        $this->time = $time ?? new \DateTime();

        // Some code...

        $a = random_int(0, 1);
        if ($a === 1) {
            $this->timeWasSet = null !== $time; // Here it's not the direct next statement
        }
    }

[samsonasik]

That mean variable used by next stmt start by key + 1 until end of stmts.

Yes, that should cover that, feel free to try latest dev-main