- Patch #81740 by merlinofchaos, webchick, moshe, neclimdul et al: added...

- Patch #81740 by merlinofchaos, webchick, moshe, neclimdul et al: added package support to the modules page.

includes/module.inc

@@ -175,11 +175,13 @@ function _module_build_dependents($files) {
  * name - The real name of the module for display purposes.
  * description - A brief description of the module.
  * dependencies - A space delimited list of the short names (shortname) of other modules this module depends on.
+ * package - The name of the package of modules this module belongs to.
  *
  * Example of .info file:
  *   name = Forum
  *   description = Enables threaded discussions about general topics.
  *   dependencies = taxonomy comment
+ *   package = Core - optional
  *
  * @param $filename
  *   The file we are parsing. Accepts file with relative or absolute path.

modules/aggregator/aggregator.info

 ; $Id$
 name = Aggregator
 description = "Aggregates syndicated content (RSS, RDF, and Atom feeds)."
+package = Core - optional

modules/block/block.info

 ; $Id$
 name = Block
 description = Controls the boxes that are displayed around the main content.
-
+package = Core - required

modules/blog/blog.info

 ; $Id$
 name = Blog
 description = Enables keeping an easily and regularly updated web page or a blog.
-
+package = Core - optional

modules/blogapi/blogapi.info

 ; $Id$
 name = Blog API
 description = Allows users to post content using applications that support XML-RPC blog APIs.
-
+package = Core - optional

modules/book/book.info

 ; $Id$
 name = Book
 description = Allows users to collaboratively author a book.
-
+package = Core - optional

modules/comment/comment.info

 ; $Id$
 name = Comment
 description = Allows users to comment on and discuss published content.
-
+package = Core - optional

modules/contact/contact.info

 ; $Id$
 name = Contact
 description = Enables the use of both personal and site-wide contact forms.
-
+package = Core - optional

modules/drupal/drupal.info

 ; $Id$
 name = Drupal
 description = Lets you register your site with a central server and improve ranking of Drupal projects by posting information on your installed modules and themes; also enables users to log in using a Drupal ID.
-
+package = Core - optional

modules/filter/filter.info

 ; $Id$
 name = Filter
 description = Handles the filtering of content in preparation for display.
-
+package = Core - required

modules/forum/forum.info

@@ -2,3 +2,4 @@
 name = Forum
 description = Enables threaded discussions about general topics.
 dependencies = taxonomy comment
+package = Core - optional

modules/help/help.info

 ; $Id$
 name = Help
 description = Manages the display of online help.
+package = Core - optional

modules/legacy/legacy.info

 ; $Id$
 name = Legacy
 description = Provides legacy handlers for upgrades from older Drupal installations.
-
+package = Core - optional

modules/locale/locale.info

 ; $Id$
 name = Locale
 description = Enables the translation of the user interface to languages other than English.
-
+package = Core - optional

modules/menu/menu.info

 ; $Id$
 name = Menu
 description = Allows administrators to customize the site navigation menu.
-
+package = Core - optional

modules/node/node.info

 ; $Id$
 name = Node
 description = Allows content to be submitted to the site and displayed on pages.
+package = Core - required
 

modules/path/path.info

 ; $Id$
 name = Path
 description = Allows users to rename URLs.
-
+package = Core - optional

modules/ping/ping.info

 ; $Id$
 name = Ping
 description = Alerts other sites when your site has been updated.
-
+package = Core - optional

modules/poll/poll.info

 ; $Id$
 name = Poll
 description = Allows your site to capture votes on different topics in the form of multiple choice questions.
-
+package = Core - optional

modules/profile/profile.info

 ; $Id$
 name = Profile
 description = Supports configurable user profiles.
-
+package = Core - optional

modules/search/search.info

 ; $Id$
 name = Search
 description = Enables site-wide keyword searching.
-
+package = Core - optional

modules/statistics/statistics.info

 ; $Id$
 name = Statistics
 description = Logs access statistics for your site.
-
+package = Core - optional

modules/system/admin.css

@@ -39,6 +39,12 @@ div.admin .expert-link {
   padding-right: 4px;
 }
 
+table.package {
+  width: 100%;
+}
+table.package .description {
+  width: 100%;
+}
 div.admin-dependencies, div.admin-required {
   font-size: 0.9em;
   color: #444;

modules/system/system.info

 ; $Id$
 name = System
 description = Handles general site configuration for administrators.
-
+package = Core - required

modules/system/system.module

@@ -1333,11 +1333,6 @@ function system_modules($form_values = NULL) {
   // Merge in required modules.
   $modules_required = array('block', 'filter', 'node', 'system', 'user', 'watchdog');
   foreach ($modules_required as $required) {
-    $form['description'][$required]['core'] = array(
-      '#value' => t('Required for Drupal core'),
-      '#prefix' => '<div class="admin-required">',
-      '#suffix' => '</div>',
-    );
     $disabled[] = $required;
     $form['disabled_modules']['#value'][$required] = TRUE;
   }
@@ -1526,18 +1521,7 @@ function theme_system_modules($form) {
     return drupal_render($form);
   }
 
-  foreach (element_children($form['name']) as $key) {
-    $row = array();
-    $row[] = array('data' => drupal_render($form['status'][$key]), 'align' => 'center');
-
-    if (module_exists('throttle')) {
-      $row[] = array('data' => drupal_render($form['throttle'][$key]), 'align' => 'center');
-    }
-    $row[] = '<strong>'. drupal_render($form['name'][$key]) .'</strong>';
-    $row[] = drupal_render($form['description'][$key]);
-    $rows[] = $row;
-  }
-
+  // Individual table headers.
   $header = array(t('Enabled'));
   if (module_exists('throttle')) {
     $header[] = t('Throttle');
@@ -1545,7 +1529,35 @@ function theme_system_modules($form) {
   $header[] = t('Name');
   $header[] = t('Description');
 
-  $output = theme('table', $header, $rows);
+  // Pull package information from module list and start grouping modules.
+  $modules = $form['validation_modules']['#value'];
+  foreach ($modules as $module) {
+    if (!isset($module->info['package'])) {
+      $module->info['package'] = 'Uncategorized';
+    }
+    $packages[$module->info['package']][$module->name] = $module->info;
+  }
+  ksort($packages);
+
+  // Display packages.
+  $output = '';
+  foreach ($packages as $package => $modules) {
+    $rows = array();
+    foreach ($modules as $key => $module) {
+      $row = array();
+      $row[] = array('data' => drupal_render($form['status'][$key]), 'align' => 'center');
+
+      if (module_exists('throttle')) {
+        $row[] = array('data' => drupal_render($form['throttle'][$key]), 'align' => 'center');
+      }
+      $row[] = '<strong>'. drupal_render($form['name'][$key]) .'</strong>';
+      $row[] = array('data' => drupal_render($form['description'][$key]), 'class' => 'description');
+      $rows[] = $row;
+    }
+    $output .= '<h2>'. t($package) .'</h2>';
+    $output .= theme('table', $header, $rows, array('class' => 'package'));
+  }
+
   $output .= drupal_render($form);
   return $output;
 }

modules/taxonomy/taxonomy.info

 ; $Id$
 name = Taxonomy
 description = Enables the categorization of content.
-
+package = Core - optional

modules/throttle/throttle.info

 ; $Id$
 name = Throttle
 description = Handles the auto-throttling mechanism, to control site congestion.
-
+package = Core - optional

modules/tracker/tracker.info

 ; $Id$
 name = Tracker
 description = Enables tracking of recent posts for users.
-
+package = Core - optional

modules/upload/upload.info

 ; $Id$
 name = Upload
 description = Allows users to upload and attach files to content.
-
+package = Core - optional

modules/user/user.info

 ; $Id$
 name = User
 description = Manages the user registration and login system.
-
+package = Core - required

modules/watchdog/watchdog.info

 ; $Id$
 name = Watchdog
 description = Logs and records system events.
-
+package = Core - required