- Patch #27645 by Neil: refactor away module_load_all().

  "This function is called in one place, so it can be rolled into the calling function. The return value isn't used so we can remove handling of it. This is executed for every non-cached page view, so the smaller code should save a smallish ammount of memory and time."

1 files changed, 4 insertions, 20 deletions

diff --git a/includes/module.inc b/includes/module.inc
index 1eb4aebe7..0fee1cf43 100644
--- a/includes/module.inc
+++ b/includes/module.inc
@@ -8,10 +8,12 @@
 
 /**
  * Initialize all modules.
- *
  */
 function module_init() {
-  module_load_all();
+  // Load all the modules that have been enabled in the system table.
+  foreach (module_list(TRUE, FALSE) as $module) {
+    drupal_load('module', $module);
+  }
   module_invoke_all('init');
 }
 
@@ -71,24 +73,6 @@ function module_list($refresh = FALSE, $bootstrap = TRUE) {
 }
 
 /**
- * Load all the modules that have been enabled in the system table.
- *
- * @return
- *   TRUE if all modules were loaded successfully.
- */
-function module_load_all() {
-  $list = module_list(TRUE, FALSE);
-  $status = TRUE;
-
-  foreach ($list as $module) {
-    $status = (drupal_load('module', $module) && $status);
-  }
-
-  return $status;
-}
-
-
-/**
  * Determine whether a given module exists.
  *
  * @param $module