#72467 by profix898. Make the modules page fieldsets less useless.

And there was a missing </p>.

1 files changed, 4 insertions, 4 deletions

diff --git a/modules/system/system.module b/modules/system/system.module
index 711c919f3..1e2d9e938 100644
--- a/modules/system/system.module
+++ b/modules/system/system.module
@@ -46,7 +46,7 @@ function system_help($section) {
       return t('<p>These options control the display settings for the <code>%template</code> theme. When your site is displayed using this theme, these settings will be used. By clicking "Reset to defaults," you can choose to use the <a href="@global">global settings</a> for this theme.</p>', array('%template' => $theme, '@global' => url('admin/build/themes/settings')));
     case 'admin/build/modules':
       return t('<p>Modules are plugins for Drupal that extend its core functionality. Here you can select which modules are enabled. Click on the name of the module in the navigation menu for their individual configuration pages. Once a module is enabled, new <a href="@permissions">permissions</a> might be made available. Modules can automatically be temporarily disabled to reduce server load when your site becomes extremely busy by enabling the throttle.module and checking throttle. The auto-throttle functionality must be enabled on the <a href="@throttle">throttle configuration page</a> after having enabled the throttle module.</p>
-<p>It is important that <a href="@update-php">update.php</a> is run every time a module is updated to a newer version.</p><p>You can find all administration tasks belonging to a particular module on the <a href="@by-module">administration by module page</a>.', array('@permissions' => url('admin/user/access'), '@throttle' => url('admin/settings/throttle'), '@update-php' => $base_url .'/update.php', '@by-module' => url('admin/by-module')));
+<p>It is important that <a href="@update-php">update.php</a> is run every time a module is updated to a newer version.</p><p>You can find all administration tasks belonging to a particular module on the <a href="@by-module">administration by module page</a>.</p>', array('@permissions' => url('admin/user/access'), '@throttle' => url('admin/settings/throttle'), '@update-php' => $base_url .'/update.php', '@by-module' => url('admin/by-module')));
     case 'admin/build/modules/uninstall':
       return (t('<p>The uninstall process removes all data related to a module. To uninstall a module, you must first disable it. Not all modules support this feature.</p>'));
     case 'admin/logs/status':
@@ -1527,11 +1527,11 @@ function theme_system_modules($form) {
       $row[] = array('data' => drupal_render($form['description'][$key]), 'class' => 'description');
       $rows[] = $row;
     }
-    $fieldset = array (
+    $fieldset = array(
       '#title' => t($package),
       '#collapsible' => TRUE,
-      '#collapsed' => TRUE,
-      '#value' => theme('table', $header, $rows, array('class' => 'package'))
+      '#collapsed' => ($package == 'Core - required'),
+      '#value' => theme('table', $header, $rows, array('class' => 'package')),
     );
     $output .= theme('fieldset', $fieldset);
   }