summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAleksander Machniak <alec@alec.pl>2013-06-21 12:40:26 +0200
committerAleksander Machniak <alec@alec.pl>2013-06-21 12:41:15 +0200
commit738c446078fd4475c5e0222c2c0ab4242d1bf5b2 (patch)
treeb3ca5f13da1692384f08d590ff7a2490957fb640
parent6376aaff0c4dae5d85ca814f038ef8fbae8834ca (diff)
Fix so valid and set date.timezone is not required by installer checks (#1489180)
-rw-r--r--CHANGELOG1
-rw-r--r--installer/check.php39
2 files changed, 24 insertions, 16 deletions
diff --git a/CHANGELOG b/CHANGELOG
index b061fab19..1bab2cdd8 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -1,6 +1,7 @@
CHANGELOG Roundcube Webmail
===========================
+- Fix so valid and set date.timezone is not required by installer checks (#1489180)
- Canonize boolean ini_get() results (#1489189)
- Fix so install do not fail when one of DB driver checks fails but other drivers exist (#1489178)
- Fix so exported vCard specifies encoding in v3-compatible format (#1489183)
diff --git a/installer/check.php b/installer/check.php
index 122437b9b..84a2a3f4d 100644
--- a/installer/check.php
+++ b/installer/check.php
@@ -42,12 +42,12 @@ $ini_checks = array(
'suhosin.session.encrypt' => 0,
'magic_quotes_runtime' => 0,
'magic_quotes_sybase' => 0,
- 'date.timezone' => '-NOTEMPTY-',
);
$optional_checks = array(
// required for utils/modcss.inc, should we require this?
'allow_url_fopen' => 1,
+ 'date.timezone' => '-VALID-',
);
$source_urls = array(
@@ -189,23 +189,15 @@ foreach ($ini_checks as $var => $val) {
if ($val === '-NOTEMPTY-') {
if (empty($status)) {
$RCI->fail($var, "empty value detected");
- } else if ($var == 'date.timezone') {
- try {
- $tz = new DateTimeZone($status);
- $RCI->pass($var);
- }
- catch (Exception $e) {
- $RCI->fail($var, "invalid value detected: $status");
- }
- } else {
+ }
+ else {
$RCI->pass($var);
}
- echo '<br />';
- continue;
}
- if (filter_var($status, FILTER_VALIDATE_BOOLEAN) == $val) {
+ else if (filter_var($status, FILTER_VALIDATE_BOOLEAN) == $val) {
$RCI->pass($var);
- } else {
+ }
+ else {
$RCI->fail($var, "is '$status', should be '$val'");
}
echo '<br />';
@@ -227,9 +219,24 @@ foreach ($optional_checks as $var => $val) {
echo '<br />';
continue;
}
- if (filter_var($status, FILTER_VALIDATE_BOOLEAN) == $val) {
+ if ($val === '-VALID-') {
+ if ($var == 'date.timezone') {
+ try {
+ $tz = new DateTimeZone($status);
+ $RCI->pass($var);
+ }
+ catch (Exception $e) {
+ $RCI->optfail($var, empty($status) ? "not set" : "invalid value detected: $status");
+ }
+ }
+ else {
+ $RCI->pass($var);
+ }
+ }
+ else if (filter_var($status, FILTER_VALIDATE_BOOLEAN) == $val) {
$RCI->pass($var);
- } else {
+ }
+ else {
$RCI->optfail($var, "is '$status', could be '$val'");
}
echo '<br />';