Replace all module_invoke_all() deprecated function calls in procedural code.

Everything looks good apart from module_invoke_all() is still referenced in a few comments. Can we fix those up also.

I'm not sure if this should be absolute or not, i know namespaces are meant to be absolute in comments but not sure about the global class thing. Ill try find someone else to review.

+++ b/core/includes/update.incundefined
@@ -1247,22 +1247,22 @@ function update_already_performed($module, $number) {
- * @see module_invoke_all()
+ * @see Drupal::moduleHandler()->invokeAll()
 

@drupalrv discussed in IRC and the rule I'm told is to use absolute so please update the comment to be:

@see \Drupal::moduleHandler()->invokeAll()

and then we should be good to go!

Sorry, I spotted a couple more.

This comment should be absolute reference.

+++ b/core/includes/update.incundefined
@@ -1247,22 +1247,22 @@ function update_already_performed($module, $number) {
+ * This function is similar to Drupal::moduleHandler()->invokeAll(), with the

This should be absolute also.

+++ b/core/modules/simpletest/lib/Drupal/simpletest/UnitTestBase.phpundefined
@@ -15,7 +15,7 @@
+ * watchdog(), module_implements(), Drupal::moduleHandler()->invokeAll() etc.

+++ b/core/modules/rdf/rdf.moduleundefined
@@ -109,7 +109,7 @@ function rdf_rdf_namespaces() {
+  // namespace, do not use Drupal::moduleHandler()->invokeAll().

Should be absolute.

Looks good.

as per https://drupal.org/node/2045931#comment-7697279 we should be injecting the container in OO code.

OK sounds good :)