Ordering of alter "extra hooks" is a gigantic mess

1 small nit and a question.

Applied and replied, let me know what you think!

Test failure is probably unrelated and needs a re-run. Fine to go to RTBC after test passes.

Tests passed rtbc per 22

I rebased and fixed the deprecation test. I'll keep an eye on the tests.
With 🐛 ModuleHandler->alterDeprecated does not properly handle deprecated OOP hooks Active the alter on these hooks is respected so the order in the deprecation is inverted.

Leaving RTBC since that's the only change and it was expected once the mentioned issue was merged. If it needs another review let me know.

I think this is good to get us to a point of minimal change for 11.1.x and then we can figure out hook ordering for all-OOP hooks will look like a bit further along.

Committed/pushed to 11.x and cherry-picked to 11.1.x, thanks!

catch → closed merge request !10039

RTBC + 1 from me - post merge

This implementation is much cleaner.