### Problem/Motivation [#3600921](https://git.drupalcode.org/project/rector/-/work_items/3600921) enhanced `HookConvertRector` to declare converted hook methods `static` when they do not reference `$this` (and to rewrite global `t()` to `$this->t()` with `StringTranslationTrait`). We should revert the `static` behavior. The reasoning behind making them `static` is a PHPStan opinion: methods that don't need `$this` shouldn't have access to it, and should communicate that by being `static`. PHPStan has a lot of such opinions, especially at higher levels. The problem is that PHPStan doesn't know that hooks are essentially interface implementations. They don't define an API, they implement it. - It's similar to unused parameters on hooks: various tools complain about them, but the point is that a hook implements a contract, and the hook definition dictates which parameters you receive — so we generally declare all parameters even if a specific implementation doesn't use one. PHPStan wouldn't complain if you implemented an actual interface; it's just that there is no language feature for what we do. - It's the same with `static`: core always calls hooks as a method on an object. Hook implementations don't get to decide that. Making hook methods `static` therefore misrepresents the contract and bakes a tool-specific opinion into generated code. `HookConvertRector` should generate plain (non-static) methods. ### Proposed resolution Revert the `static` part of [#3600921](https://git.drupalcode.org/project/rector/-/work_items/3600921): `HookConvertRector` no longer declares converted hook methods `static`, regardless of whether they reference `$this`. ### Remaining tasks - Remove the "make `$this`-free hook methods static" behavior from `HookConvertRector`. - Update fixtures/tests accordingly. ### Credit - Reported by: @berdir