[internal] Remove duplicated enterNode() type check already handled in node traverser

[TomasVotruba]

When Rector traversers nodes, it only applies related rules. E.g. when a rule hooks into Class_, its only run on a Class_ node. This already happens in a node visitor since #6232

Yet, we still have a duplicate check on every single node enter:

  final public function enterNode(Node $node): int|Node|null
    {
        if (! $this->isMatchingNodeType($node)) {
            return null;
        }
        
        // ...
   }

This is redundant and now removed 👍

[TomasVotruba]

Let's give this a go 👍

[samsonasik]

I think this should continue 2; as well, as currentNodeVisitors is inside inner loop, so next subnode should also allow to be visited, to ensure nothing left behind, while keep the performance :)

[samsonasik]

I created PR:

• [internal] Make consistent to continue outer loop when subnode type changed on AbstractImmutableNodeTraverser #7766

for it :)