Logic error in migrate/hosting_migrate.module theme function

From 0.4a, at around line 122 (theme_hosting_task_migrate_form()):

  foreach ($packages as $platform => $package_instances) {
    $row = array();
    foreach ($packages[$node->platform] as $key => $platform_instance) {
      if ($instance = $package_instances[$key]) {
        $comparison = hosting_package_instance_version_compare($platform_instance, $instance);
        $display = $instance->version;
        if ($instance->schema_version) {
          $display .= " (" . $instance->schema_version . ")";
        }
        if ($comparison > 0) {
          $row[] = array('class' => 'hosting-package-upgrade', 'data' => $display);
        }
        elseif ($comparison == 0) {
          $row[] = array('class' => 'hosting-package-same', 'data' => $display);
        }
        else {
          $row[] = array('class' => 'hosting-package-downgrade', 'data' => $display);
          $error = TRUE;
        }
      }

Background: I was unable to migrate any sites, despite the fact that at least one platform was returning no errors.

Problem: It looks like there's a logic error in the task_migrate form for errors. See line 139 -

$error = TRUE;

Since this is inside a FOREACH, $error = FALSE; needs to be declared at the beginning of the FOREACH, otherwise any row that comes after one that has an error will have an error as well. Please see my suggested patch, attached.

Thanks!
-pl