6.x - EventManagerOnRector flips already changed signature calls

[LordSimal]

While preparing more underscore clean ups I realized, that https://github.com/cakephp/upgrade/blob/6.x/src/Rector/Cake6/EventManagerOnRector.php changes the signature of already "fixed" ->on() calls on the code if the rector is called more than once on a given folder.

It appears that custom rector needs to be extended to check, if the 2nd param is already a callable (which technically can also be an array) or not.

@dereuromark could you take a look at that with your almighty AI 😜?

[dereuromark]

How do you survive without it ;)
the 20 eur/month version for claude or codex usually suffices for medium amount work.

[LordSimal]

My company currently "only" pays for ChatGPT Pro + the Github Copilot stuff which we get for free as open-source devs.

But thats still not the same as having claude code as far as I know, maybe you can enlighten me 😛