Use updatedb.sh from update.sh + various fixes in updatedb.sh

Aleksander Machniak

2013-01-09 7e7431bddadae7802748979864ffafa73f694ede

Use updatedb.sh from update.sh + various fixes in updatedb.sh

 bin/update.sh

@@ -124,7 +124,7 @@
        }
      }
      else {
        echo "Please update your config files manually according to the above messages.\n\n";
        echo "Please update your config files manually according to the above messages.\n";
      }
    }

@@ -143,29 +143,11 @@

  // check database schema
  if ($RCI->config['db_dsnw']) {
    $DB = rcube_db::factory($RCI->config['db_dsnw'], '', false);
    $DB->db_connect('w');
    if ($db_error_msg = $DB->is_error()) {
      echo "Error connecting to database: $db_error_msg\n";
      $success = false;
    }
    else if ($err = $RCI->db_schema_check($DB, false)) {
      $updatefile = INSTALL_PATH . 'SQL/' . (isset($RCI->db_map[$DB->db_provider]) ? $RCI->db_map[$DB->db_provider] : $DB->db_provider) . '.update.sql';
      echo "WARNING: Database schema needs to be updated!\n";
      echo join("\n", $err) . "\n\n";
      $success = false;
    echo "Executing database schema update.\n";
    system(INSTALL_PATH . "bin/updatedb.sh --label=roundcube --version=" . $ops['version']
      . " --dir=" . INSTALL_PATH . DIRECTORY_SEPARATOR . "SQL", $res);
      
      if ($opts['version']) {
        echo "Do you want to run the update queries to get the schmea fixed? (y/N)\n";
        $input = trim(fgets(STDIN));
        if (strtolower($input) == 'y') {
          $success = $RCI->update_db($DB, $opts['version']);
        }
      }
      
      if (!$success)
        echo "Open $updatefile and execute all queries below the comment with the currently installed version number.\n";
    }
    $success = !$res;
  }
  
  // index contacts for fulltext searching
@@ -182,7 +164,5 @@
  echo "This instance of Roundcube is not yet configured!\n";
  echo "Open http://url-to-roundcube/installer/ in your browser and follow the instuctions.\n";
}

echo "\n";

?>

 bin/updatedb.sh

@@ -45,8 +45,27 @@
  exit(1);
}

// version is specified, use release-to-version map
if ($opts['version']) {
$RC = rcube::get_instance();
$DB = rcube_db::factory($RC->config->get('db_dsnw'));

// Connect to database
$DB->db_connect('w');
if (!$DB->is_connected()) {
    echo "Error connecting to database: " . $DB->is_error() . ".\n";
    exit(1);
}

// Read DB schema version from database
$DB->query("SELECT " . $DB->quote_identifier('value')
    ." FROM " . $DB->quote_identifier('system')
    ." WHERE " . $DB->quote_identifier('name') ." = ?",
    $opts['label'] . '-version');

$row     = $DB->fetch_array();
$version = $row[0];

// no DB version, but release version is specified
if (!$version && $opts['version']) {
    // Map old release version string to DB schema version
    // Note: This is for backward compat. only, do not need to be updated
    $map = array(
@@ -82,28 +101,7 @@
    $version = $map[$opts['version']];
}

$RC = rcube::get_instance();
$DB = rcube_db::factory($RC->config->get('db_dsnw'));

// Connect to database
$DB->db_connect('w');
if (!$DB->is_connected()) {
    echo "Error connecting to database: " . $DB->is_error() . ".\n";
    exit(1);
}

// Read DB schema version from database
if (empty($version)) {
    @$DB->query("SELECT " . $DB->quote_identifier('value')
        ." FROM " . $DB->quote_identifier('system')
        ." WHERE name = ?",
        $opts['label'] . '-version');

    $row     = $DB->fetch_array();
    $version = $row[0];
}

// Assume last version without the "system" table
// Assume last version before the system table was added
if (empty($version)) {
    $version = 2012080700;
}
@@ -162,13 +160,13 @@
    $DB->query("UPDATE " . $DB->quote_identifier('system')
        ." SET " . $DB->quote_identifier('value') . " = ?"
        ." WHERE " . $DB->quote_identifier('name') . " = ?",
        $version, $opts['label'] . '-version');
        $version, $label . '-version');

    if (!$DB->is_error() && !$DB->affected_rows()) {
        $DB->query("INSERT INTO " . $DB->quote_identifier('system')
            ." (" . $DB->quote_identifier('name') . ", " . $DB->quote_identifier('value') . ")"
            ." VALUES (?, ?)",
            $opts['label'] . '-version', $version);
            $label . '-version', $version);
    }

    return $DB->is_error();