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 '';
+ foreach ($messages['dependencies'] as $msg) {
+ echo html::tag('li', null, html::span('propname', $msg['prop']) . ': ' . $msg['explain']);
+ }
+ 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