From 10ea10a3177fb59caac13adc44cfbc4b5e6ff8dd Mon Sep 17 00:00:00 2001
From: thomascube <thomas@roundcube.net>
Date: Fri, 20 May 2011 14:35:08 -0400
Subject: [PATCH] Catch write-errors on database (yet untested)
---
installer/check.php | 144 +++++++++++++++++++++++++++++++----------------
1 files changed, 95 insertions(+), 49 deletions(-)
diff --git a/installer/check.php b/installer/check.php
index 694455a..944d384 100644
--- a/installer/check.php
+++ b/installer/check.php
@@ -1,39 +1,66 @@
<form action="index.php" method="get">
<?php
-$required_php_exts = array('PCRE' => 'pcre', 'DOM' => 'dom',
- 'Session' => 'session', 'XML' => 'xml');
+$required_php_exts = array(
+ 'PCRE' => 'pcre',
+ 'DOM' => 'dom',
+ 'Session' => 'session',
+ 'XML' => 'xml',
+ 'JSON' => 'json'
+);
-$optional_php_exts = array('FileInfo' => 'fileinfo', 'Libiconv' => 'iconv',
- 'Multibyte' => 'mbstring', 'OpenSSL' => 'openssl', 'Mcrypt' => 'mcrypt',
- 'GD' => 'gd');
+$optional_php_exts = array(
+ 'FileInfo' => 'fileinfo',
+ 'Libiconv' => 'iconv',
+ 'Multibyte' => 'mbstring',
+ 'OpenSSL' => 'openssl',
+ 'Mcrypt' => 'mcrypt',
+ 'Intl' => 'intl',
+);
-$required_libs = array('PEAR' => 'PEAR.php', 'MDB2' => 'MDB2.php',
- 'Net_SMTP' => 'Net/SMTP.php', 'Mail_mime' => 'Mail/mime.php',
- 'iilConnection' => 'lib/imap.inc');
+$required_libs = array(
+ 'PEAR' => 'PEAR.php',
+ 'MDB2' => 'MDB2.php',
+ 'Net_SMTP' => 'Net/SMTP.php',
+ 'Net_IDNA2' => 'Net/IDNA2.php',
+ 'Mail_mime' => 'Mail/mime.php',
+);
-$supported_dbs = array('MySQL' => 'mysql', 'MySQLi' => 'mysqli',
- 'PostgreSQL' => 'pgsql', 'SQLite (v2)' => 'sqlite');
+$supported_dbs = array(
+ 'MySQL' => 'mysql',
+ 'MySQLi' => 'mysqli',
+ 'PostgreSQL' => 'pgsql',
+ 'SQLite (v2)' => 'sqlite',
+);
-$ini_checks = array('file_uploads' => 1, 'session.auto_start' => 0,
- 'zend.ze1_compatibility_mode' => 0, 'mbstring.func_overload' => 0,
- 'suhosin.session.encrypt' => 0, 'date.timezone' => '-NOTEMPTY-');
+$ini_checks = array(
+ 'file_uploads' => 1,
+ 'session.auto_start' => 0,
+ 'zend.ze1_compatibility_mode' => 0,
+ 'mbstring.func_overload' => 0,
+ 'suhosin.session.encrypt' => 0,
+);
+
+$optional_checks = array(
+ 'date.timezone' => '-NOTEMPTY-',
+);
$source_urls = array(
- 'Sockets' => 'http://www.php.net/manual/en/ref.sockets.php',
- 'Session' => 'http://www.php.net/manual/en/ref.session.php',
- 'PCRE' => 'http://www.php.net/manual/en/ref.pcre.php',
- 'FileInfo' => 'http://www.php.net/manual/en/ref.fileinfo.php',
- 'Libiconv' => 'http://www.php.net/manual/en/ref.iconv.php',
- 'Multibyte' => 'http://www.php.net/manual/en/ref.mbstring.php',
- 'Mcrypt' => 'http://www.php.net/manual/en/ref.mcrypt.php',
- 'OpenSSL' => 'http://www.php.net/manual/en/ref.openssl.php',
- 'GD' => 'http://www.php.net/manual/en/ref.image.php',
- 'PEAR' => 'http://pear.php.net',
- 'MDB2' => 'http://pear.php.net/package/MDB2',
- 'Net_SMTP' => 'http://pear.php.net/package/Net_SMTP',
+ 'Sockets' => 'http://www.php.net/manual/en/book.sockets.php',
+ 'Session' => 'http://www.php.net/manual/en/book.session.php',
+ 'PCRE' => 'http://www.php.net/manual/en/book.pcre.php',
+ 'FileInfo' => 'http://www.php.net/manual/en/book.fileinfo.php',
+ 'Libiconv' => 'http://www.php.net/manual/en/book.iconv.php',
+ 'Multibyte' => 'http://www.php.net/manual/en/book.mbstring.php',
+ 'Mcrypt' => 'http://www.php.net/manual/en/book.mcrypt.php',
+ 'OpenSSL' => 'http://www.php.net/manual/en/book.openssl.php',
+ 'JSON' => 'http://www.php.net/manual/en/book.json.php',
+ 'DOM' => 'http://www.php.net/manual/en/book.dom.php',
+ 'Intl' => 'http://www.php.net/manual/en/book.intl.php',
+ 'PEAR' => 'http://pear.php.net',
+ 'MDB2' => 'http://pear.php.net/package/MDB2',
+ 'Net_SMTP' => 'http://pear.php.net/package/Net_SMTP',
'Mail_mime' => 'http://pear.php.net/package/Mail_mime',
- 'DOM' => 'http://www.php.net/manual/en/intro.dom.php'
);
echo '<input type="hidden" name="_step" value="' . ($RCI->configured ? 3 : 2) . '" />';
@@ -42,7 +69,7 @@
<h3>Checking PHP version</h3>
<?php
-define('MIN_PHP_VERSION', '5.2.0');
+define('MIN_PHP_VERSION', '5.2.1');
if (version_compare(PHP_VERSION, MIN_PHP_VERSION, '>=')) {
$RCI->pass('Version', 'PHP ' . PHP_VERSION . ' detected');
} else {
@@ -51,16 +78,19 @@
?>
<h3>Checking PHP extensions</h3>
-<p class="hint">The following modules/extensions are <em>required</em> to run RoundCube:</p>
+<p class="hint">The following modules/extensions are <em>required</em> to run Roundcube:</p>
<?php
-
+
+// get extensions location
+$ext_dir = ini_get('extension_dir');
+
$prefix = (PHP_SHLIB_SUFFIX === 'dll') ? 'php_' : '';
-foreach ($required_php_exts AS $name => $ext) {
+foreach ($required_php_exts as $name => $ext) {
if (extension_loaded($ext)) {
$RCI->pass($name);
} else {
- $_ext = $prefix . $ext . '.' . PHP_SHLIB_SUFFIX;
- $msg = @dl($_ext) ? 'Could be loaded. Please add in php.ini' : '';
+ $_ext = $ext_dir . '/' . $prefix . $ext . '.' . PHP_SHLIB_SUFFIX;
+ $msg = @is_readable($_ext) ? 'Could be loaded. Please add in php.ini' : '';
$RCI->fail($name, $msg, $source_urls[$name]);
}
echo '<br />';
@@ -68,16 +98,16 @@
?>
-<p class="hint">The next couple of extensions are <em>optional</em> but recommended to get the best performance:</p>
+<p class="hint">The next couple of extensions are <em>optional</em> and recommended to get the best performance:</p>
<?php
-foreach ($optional_php_exts AS $name => $ext) {
+foreach ($optional_php_exts as $name => $ext) {
if (extension_loaded($ext)) {
$RCI->pass($name);
}
else {
- $_ext = $prefix . $ext . '.' . PHP_SHLIB_SUFFIX;
- $msg = @dl($_ext) ? 'Could be loaded. Please add in php.ini' : '';
+ $_ext = $ext_dir . '/' . $prefix . $ext . '.' . PHP_SHLIB_SUFFIX;
+ $msg = @is_readable($_ext) ? 'Could be loaded. Please add in php.ini' : '';
$RCI->na($name, $msg, $source_urls[$name]);
}
echo '<br />';
@@ -92,13 +122,13 @@
<?php
$prefix = (PHP_SHLIB_SUFFIX === 'dll') ? 'php_' : '';
-foreach ($supported_dbs AS $database => $ext) {
+foreach ($supported_dbs as $database => $ext) {
if (extension_loaded($ext)) {
$RCI->pass($database);
}
else {
- $_ext = $prefix . $ext . '.' . PHP_SHLIB_SUFFIX;
- $msg = @dl($_ext) ? 'Could be loaded. Please add in php.ini' : 'Not installed';
+ $_ext = $ext_dir . '/' . $prefix . $ext . '.' . PHP_SHLIB_SUFFIX;
+ $msg = @is_readable($_ext) ? 'Could be loaded. Please add in php.ini' : 'Not installed';
$RCI->na($database, $msg, $source_urls[$database]);
}
echo '<br />';
@@ -127,6 +157,7 @@
?>
<h3>Checking php.ini/.htaccess settings</h3>
+<p class="hint">The following settings are <em>required</em> to run Roundcube:</p>
<?php
@@ -136,15 +167,6 @@
if (empty($status)) {
$RCI->fail($var, "cannot be empty and needs to be set");
} else {
- switch ($var) {
- case 'date.timezone':
- if (date_default_timezone_set($status) === false) {
- $RCI->fail($var, "is '$status', but the settings is wrong");
- echo '<br />';
- continue;
- }
- break;
- }
$RCI->pass($var);
}
echo '<br />';
@@ -159,10 +181,34 @@
}
?>
+<p class="hint">The following settings are <em>optional</em> and recommended:</p>
+
+<?php
+
+foreach ($optional_checks as $var => $val) {
+ $status = ini_get($var);
+ if ($val === '-NOTEMPTY-') {
+ if (empty($status)) {
+ $RCI->optfail($var, "Could be set");
+ } else {
+ $RCI->pass($var);
+ }
+ echo '<br />';
+ continue;
+ }
+ if ($status == $val) {
+ $RCI->pass($var);
+ } else {
+ $RCI->optfail($var, "is '$status', could be '$val'");
+ }
+ echo '<br />';
+}
+?>
+
<?php
if ($RCI->failures) {
- echo '<p class="warning">Sorry but your webserver does not meet the requirements for RoundCube!<br />
+ echo '<p class="warning">Sorry but your webserver does not meet the requirements for Roundcube!<br />
Please install the missing modules or fix the php.ini settings according to the above check results.<br />
Hint: only checks showing <span class="fail">NOT OK</span> need to be fixed.</p>';
}
--
Gitblit v1.9.1