Index: modules/update/update.report.inc
===================================================================
RCS file: /cvs/drupal/drupal/modules/update/update.report.inc,v
retrieving revision 1.32
diff -u -r1.32 update.report.inc
--- modules/update/update.report.inc	17 Aug 2010 13:50:52 -0000	1.32
+++ modules/update/update.report.inc	22 Aug 2010 20:13:43 -0000
@@ -23,7 +23,7 @@
 /**
  * Returns HTML for the project status report.
  *
- * @param $variables
+ * @param array $variables
  *   An associative array containing:
  *   - data: An array of data about each project's status.
  *
@@ -104,8 +104,7 @@
     }
     $row .= "</div>\n";
 
-    $row .= "<div class=\"versions\">\n";
-
+    $versionsinner = '';
     if (isset($project['recommended'])) {
       if ($project['status'] != UPDATE_CURRENT || $project['existing_version'] !== $project['recommended']) {
 
@@ -114,7 +113,7 @@
         // recommending, give it the same CSS class as if it was recommended,
         // but don't print out a separate "Recommended" line for this project.
         if (!empty($project['security updates']) && count($project['security updates']) == 1 && $project['security updates'][0]['version'] === $project['recommended']) {
-          $security_class = ' version-recommended version-recommended-strong';
+          $security_class[] = 'version-recommended version-recommended-strong';
         }
         else {
           $security_class = array();
@@ -132,35 +131,36 @@
               ) {
             $version_class[] = 'version-recommended-strong';
           }
-          $row .= theme('update_version', array('version' => $project['releases'][$project['recommended']], 'tag' => t('Recommended version:'), 'class' => $version_class));
+          $versionsinner .= theme('update_version', array('version' => $project['releases'][$project['recommended']], 'tag' => t('Recommended version:'), 'class' => $version_class));
         }
 
         // Now, print any security updates.
         if (!empty($project['security updates'])) {
+          $security_class[] = 'version-security';
           foreach ($project['security updates'] as $security_update) {
-            $row .= theme('update_version', array('version' => $security_update, 'tag' => t('Security update:'), 'class' => 'version-security' . $security_class));
+            $versionsinner .= theme('update_version', array('version' => $security_update, 'tag' => t('Security update:'), 'class' => $security_class));
           }
         }
       }
 
       if ($project['recommended'] !== $project['latest_version']) {
-        $row .= theme('update_version', array('version' => $project['releases'][$project['latest_version']], 'tag' => t('Latest version:'), 'class' => array('version-latest')));
+        $versionsinner .= theme('update_version', array('version' => $project['releases'][$project['latest_version']], 'tag' => t('Latest version:'), 'class' => array('version-latest')));
       }
       if ($project['install_type'] == 'dev'
           && $project['status'] != UPDATE_CURRENT
           && isset($project['dev_version'])
           && $project['recommended'] !== $project['dev_version']) {
-        $row .= theme('update_version', array('version' => $project['releases'][$project['dev_version']], 'tag' => t('Development version:'), 'class' => array('version-latest')));
+        $versionsinner .= theme('update_version', array('version' => $project['releases'][$project['dev_version']], 'tag' => t('Development version:'), 'class' => array('version-latest')));
       }
     }
 
     if (isset($project['also'])) {
       foreach ($project['also'] as $also) {
-        $row .= theme('update_version', array('version' => $project['releases'][$also], 'tag' => t('Also available:'), 'class' => array('version-also-available')));
+        $versionsinner .= theme('update_version', array('version' => $project['releases'][$also], 'tag' => t('Also available:'), 'class' => array('version-also-available')));
       }
     }
 
-    $row .= "</div>\n"; // versions div.
+    $row .= !empty($versionsinner) ? "<div class=\"versions\">\n" . $versionsinner . "</div>\n" : ''; // versions div.
 
     $row .= "<div class=\"info\">\n";
     if (!empty($project['extra'])) {
@@ -250,7 +250,7 @@
 /**
  * Returns HTML for a label to display for a project's update status.
  *
- * @param $variables
+ * @param array $variables
  *   An associative array containing:
  *   - status: The integer code for a project's current update status.
  *
@@ -279,7 +279,7 @@
 /**
  * Returns HTML for the version display of a project.
  *
- * @param $variables
+ * @param array $variables
  *   An associative array containing:
  *   - version: An array of data about the latest released version, containing:
  *     - version: The version number.
@@ -294,7 +294,7 @@
 function theme_update_version($variables) {
   $version = $variables['version'];
   $tag = $variables['tag'];
-  $class = $variables['class'];
+  $class = implode(' ', $variables['class']);
 
   $output = '';
   $output .= '<table class="version ' . $class . '">';