From 871ca9adfedfa0aedf1994af579a5ea6715cff5f Mon Sep 17 00:00:00 2001 From: thomascube Date: Tue, 14 Oct 2008 12:40:11 +0000 Subject: Add dependency checks for config options + update database schema with MDB2_Schema (doesn't work correctly yet) --- installer/config.php | 2 +- installer/rcube_install.php | 105 +++++++++++++++++++++++++++++++++++++++++++- installer/test.php | 29 ++++++++++-- 3 files changed, 130 insertions(+), 6 deletions(-) (limited to 'installer') diff --git a/installer/config.php b/installer/config.php index b0f56889d..52991242c 100644 --- a/installer/config.php +++ b/installer/config.php @@ -6,7 +6,7 @@ $RCI->load_defaults(); // register these boolean fields -$RCI->config_props = array( +$RCI->bool_config_props = array( 'ip_check' => 1, 'enable_caching' => 1, 'enable_spellcheck' => 1, diff --git a/installer/rcube_install.php b/installer/rcube_install.php index 21e353c9c..808994dca 100644 --- a/installer/rcube_install.php +++ b/installer/rcube_install.php @@ -30,7 +30,7 @@ class rcube_install var $configured = false; var $last_error = null; var $email_pattern = '([a-z0-9][a-z0-9\-\.\+\_]*@[a-z0-9]([a-z0-9\-][.]?)*[a-z0-9])'; - var $config_props = array(); + var $bool_config_props = array(); var $obsolete_config = array('db_backend'); var $replaced_config = array('skin_path' => 'skin', 'locale_string' => 'language'); @@ -136,7 +136,7 @@ class rcube_install return '[Warning: could not read the template file]'; foreach ($this->config as $prop => $default) { - $value = (isset($_POST["_$prop"]) || $this->config_props[$prop]) ? $_POST["_$prop"] : $default; + $value = (isset($_POST["_$prop"]) || $this->bool_config_props[$prop]) ? $_POST["_$prop"] : $default; // convert some form data if ($prop == 'debug_level') { @@ -232,6 +232,30 @@ class rcube_install $out['missing'][] = array('prop' => $prop); } + // check config dependencies and contradictions + if ($this->config['enable_spellcheck'] && $this->config['spellcheck_engine'] == 'pspell') { + if (!extension_loaded('pspell')) { + $out['dependencies'][] = array('prop' => 'spellcheck_engine', + 'explain' => 'This requires the pspell extension which could not be loaded.'); + } + if (empty($this->config['spellcheck_languages'])) { + $out['dependencies'][] = array('prop' => 'spellcheck_languages', + 'explain' => 'You should specify the list of languages supported by your local pspell installation.'); + } + } + + if ($this->config['log_driver'] == 'syslog') { + if (!function_exists('openlog')) { + $out['dependencies'][] = array('prop' => 'log_driver', + 'explain' => 'This requires the sylog extension which could not be loaded.'); + } + if (empty($this->config['syslog_id'])) { + $out['dependencies'][] = array('prop' => 'syslog_id', + 'explain' => 'Using syslog for logging requires a syslog ID to be configured'); + } + + } + return $out; } @@ -264,6 +288,83 @@ class rcube_install } + /** + * Compare the local database schema with the reference schema + * required for this version of RoundCube + * + * @param boolean True if the schema schould be updated + * @return boolean True if the schema is up-to-date, false if not or an error occured + */ + function db_schema_check($update = false) + { + if (!$this->configured) + return false; + + $options = array( + 'use_transactions' => false, + 'log_line_break' => "\n", + 'idxname_format' => '%s', + 'debug' => false, + 'quote_identifier' => true, + 'force_defaults' => false, + 'portability' => true + ); + + $schema =& MDB2_Schema::factory($this->config['db_dsnw'], $options); + $schema->db->supported['transactions'] = false; + + if (PEAR::isError($schema)) { + $this->raise_error(array('code' => $schema->getCode(), 'message' => $schema->getMessage() . ' ' . $schema->getUserInfo())); + return false; + } + else { + $definition = $schema->getDefinitionFromDatabase(); + $definition['charset'] = 'utf8'; + + if (PEAR::isError($definition)) { + $this->raise_error(array('code' => $definition->getCode(), 'message' => $definition->getMessage() . ' ' . $definition->getUserInfo())); + return false; + } + + // load reference schema + $dsn = MDB2::parseDSN($this->config['db_dsnw']); + $ref_schema = INSTALL_PATH . 'SQL/' . $dsn['phptype'] . '.schema.xml'; + + if (is_file($ref_schema)) { + $reference = $schema->parseDatabaseDefinition($ref_schema, false, array(), $schema->options['fail_on_invalid_names']); + + if (PEAR::isError($reference)) { + $this->raise_error(array('code' => $reference->getCode(), 'message' => $reference->getMessage() . ' ' . $reference->getUserInfo())); + } + else { + $diff = $schema->compareDefinitions($reference, $definition); + + if (empty($diff)) { + return true; + } + else if ($update) { + // update database schema with the diff from the above check + $success = $schema->alterDatabase($reference, $definition, $diff); + + if (PEAR::isError($success)) { + $this->raise_error(array('code' => $success->getCode(), 'message' => $success->getMessage() . ' ' . $success->getUserInfo())); + } + else + return true; + } + echo '
'; var_dump($diff); echo '
'; + return false; + } + } + else + $this->raise_error(array('message' => "Could not find reference schema file ($ref_schema)")); + return false; + } + + return false; + } + + /** * Getter for the last error message * diff --git a/installer/test.php b/installer/test.php index edf77325a..d66fe34f2 100644 --- a/installer/test.php +++ b/installer/test.php @@ -69,6 +69,20 @@ if ($RCI->configured && ($messages = $RCI->check_config())) { echo html::a(array('href' => './?_mergeconfig=main'), 'main.inc.php') . '  '; echo html::a(array('href' => './?_mergeconfig=db'), 'db.inc.php'); echo "

"; + + + if (is_array($messages['dependencies'])) { + echo '

Dependency check failed

'; + echo '

Some of your configuration settings require other options to be configured or additional PHP modules to be installed

'; + + echo ''; + } + + } ?> @@ -147,13 +161,22 @@ if ($db_working) { $db_read = $DB->query("SELECT count(*) FROM {$RCI->config['db_table_users']}"); if (!$db_read) { $RCI->fail('DB Schema', "Database not initialized"); - $db_working = false; echo '

'; + $db_working = false; + } + /* + else if (!$RCI->db_schema_check($update = !empty($_POST['updatedb']))) { + $RCI->fail('DB Schema', "Database schema differs"); + + echo $update ? '

Failed to update the database schema! Please manually execute the SQL statements from the SQL/*.update.sql file on your database

' : + '

'; + $db_working = false; } + */ else { $RCI->pass('DB Schema'); + echo '
'; } - echo '
'; } // more database tests @@ -169,7 +192,7 @@ if ($db_working) { else { $RCI->fail('DB Write', $RCI->get_error()); } - echo '
'; + echo '
'; // check timezone settings $tz_db = 'SELECT ' . $DB->unixtimestamp($DB->now()) . ' AS tz_db'; -- cgit v1.2.3