From c80f739782329d2bffe2a106bb4ea0a62e42b9ad Mon Sep 17 00:00:00 2001
From: Dries Buytaert <dries@buytaert.net>
Date: Fri, 14 Sep 2007 12:16:55 +0000
Subject: - Patch #168487 by douggreen: don't enable modules that depend on
 incompabitlbe modules.

---
 modules/system/system.admin.inc | 46 +++++++++++++++++++++++++++++------------
 1 file changed, 33 insertions(+), 13 deletions(-)

(limited to 'modules/system/system.admin.inc')

diff --git a/modules/system/system.admin.inc b/modules/system/system.admin.inc
index e2851cf5c..09297e1c4 100644
--- a/modules/system/system.admin.inc
+++ b/modules/system/system.admin.inc
@@ -525,6 +525,22 @@ function system_theme_settings_submit($form, &$form_state) {
   cache_clear_all();
 }
 
+/**
+ * Recursively check compatability
+ */
+function _system_is_incompatible(&$incompatible, $files, $file) {
+  if (isset($incompatible[$file->name])) {
+    return TRUE;
+  }
+  // Recursively traverse the dependencies, looking for incompatible modules
+  foreach ($file->info['dependencies'] as $dependency) {
+    if (isset($files[$dependency]) && _system_is_incompatible($incompatible, $files, $files[$dependency])) {
+      $incompatible[$file->name] = TRUE;
+      return TRUE;
+    }
+  }
+}
+
 /**
  * Menu callback; provides module enable/disable interface.
  *
@@ -557,27 +573,31 @@ function system_modules($form_state = array()) {
   // Create storage for disabled modules as browser will disable checkboxes.
   $form['disabled_modules'] = array('#type' => 'value', '#value' => array());
 
-  // Array for disabling checkboxes in callback system_module_disable.
-  $disabled = array();
-  $throttle = array();
+  // Traverse the files, checking for compatibility
   $incompatible_core = array();
   $incompatible_php = array();
-  // Traverse the files retrieved and build the form.
   foreach ($files as $filename => $file) {
-    $form['name'][$filename] = array('#value' => $file->info['name']);
-    $form['version'][$filename] = array('#value' => $file->info['version']);
-    $form['description'][$filename] = array('#value' => t($file->info['description']));
-    $options[$filename] = '';
     // Ensure this module is compatible with this version of core.
     if (!isset($file->info['core']) || $file->info['core'] != DRUPAL_CORE_COMPATIBILITY) {
-      $incompatible_core[] = $file->name;
-      $disabled[] = $file->name;
-      // Nothing else in this loop matters, so move to the next module.
-      continue;
+      $incompatible_core[$file->name] = $file->name;
     }
     // Ensure this module is compatible with the currently installed version of PHP.
     if (version_compare(phpversion(), $file->info['php']) < 0) {
       $incompatible_php[$file->name] = $file->info['php'];
+    }
+  }
+
+  // Array for disabling checkboxes in callback system_module_disable.
+  $disabled = array();
+  $throttle = array();
+  // Traverse the files retrieved and build the form.
+  foreach ($files as $filename => $file) {
+    $form['name'][$filename] = array('#value' => $file->info['name']);
+    $form['version'][$filename] = array('#value' => $file->info['version']);
+    $form['description'][$filename] = array('#value' => t($file->info['description']));
+    $options[$filename] = '';
+    // Ensure this module is compatible with this version of core and php.
+    if (_system_is_incompatible($incompatible_core, $files, $file) || _system_is_incompatible($incompatible_php, $files, $file)) {
       $disabled[] = $file->name;
       // Nothing else in this loop matters, so move to the next module.
       continue;
@@ -659,7 +679,7 @@ function system_modules($form_state = array()) {
       'system_modules_disable',
     ),
     '#disabled_modules' => $disabled,
-    '#incompatible_modules_core' => drupal_map_assoc($incompatible_core),
+    '#incompatible_modules_core' => $incompatible_core,
     '#incompatible_modules_php' => $incompatible_php,
   );
 
-- 
cgit v1.2.3