Fix ReturnBinaryOrToEarlyReturnRector mixing up returned nodes of array, hook into StmtsAwareInterface instead to keep next Rector rules updated

[TomasVotruba]

• fixes [CodeQuality][EarlyReturn] Handle crash Replace Stmt to Expr on FlipTypeControlToUseExclusiveTypeRector+FlipTypeControlToUseExclusiveTypeRector+ReturnBinaryOrToEarlyReturnRector #4474
• fixes System error: "Trying to replace statement (Stmt_Return) with expression (Expr_BooleanNot). Are you missing a Stmt_Expression wrapper? rector#8044

After 3 hours of debugging, I've found out the issue is in 1 node being replaced by many nodes. Espcially return with many ifs. Lot of rules hook into the If_ node then and php-parser incorrectly resolved original position. Instead of working with new array, it still operates with original tree. Then the node tree gets messed up as some rules work with one node and another with array of many nodes.

This is trail of making the rule work with parent node - StmtsAwareInterface, to always provide child node with final rules.

In short: replace one node with array of nodes only on a stmts aware. Better do not change node structure from 1 to many :)

Let's see how it works, we might want to update few more rules in the future.

[TomasVotruba]

Let's see if this helps with the build :)

[samsonasik]

@TomasVotruba thank you 🙏

[TomasVotruba]

Thanks!

That was a journey deep into AST world 😆