Replace all module_invoke_all() deprecated function calls in OO code

Do we think this might be better broken up into smaller bite size chunks?

Rerolled patch attached.

Re-roll.

This is OO code, so it should be using a module_handler object directly, rather than via \Drupal()

We'll deal with dependency injection in different issues. This is just about removing the procedural wrappers, and facilitating PHPUnit tests by making code depend on auto-loadable code.

Fair enough about doing DI in other issues, this is at least a step in the right direction. This patch looks good - it's mostly find and replace, plus creating a mock ModuleHandler for the only affected PHPUnit test.

The patch also removes the function itself, I've seen conflicting requests from core committers about whether they want patches to do that, so I'll leave it to them to say if they want to keep it for now.

This change is already covered by a change notice that does not need updating: https://drupal.org/node/1894902

We'll deal with dependency injection in different issues.

I don't really agree that we should replace the OO code if we're not going going to inject the module handler but at the very least can you create a follow up now so we have somewhere to track the OO updates.

Re-roll.

This version leaves the function itself in, as core maintainers have been requesting on other issues.

The context for invokeAll() needs to be passed as a single array, there were a few examples of multiple parameters of the wrong type being passed, which caused the installation failure seen above.

One more that needs the parameters in an array, not sure how I missed it.

Committed/pushed to 8.x, thanks!