From: Chad Horohoe Date: Sun, 6 Jun 2010 14:03:15 +0000 (+0000) Subject: (bug 23021) Mediawiki should not warn if the sesion path is not writable and should... X-Git-Tag: 1.31.0-rc.0~36606 X-Git-Url: http://git.cyclocoop.org/%40spipnet%40?a=commitdiff_plain;h=0172ccbe06dbb46d0b58e00d533fe328c1d02bf5;p=lhc%2Fweb%2Fwiklou.git (bug 23021) Mediawiki should not warn if the sesion path is not writable and should not suggest putting it in open_basedir --- diff --git a/includes/installer/WebInstaller.php b/includes/installer/WebInstaller.php index 44c5b528d2..95e1f81782 100644 --- a/includes/installer/WebInstaller.php +++ b/includes/installer/WebInstaller.php @@ -207,7 +207,11 @@ class WebInstaller extends Installer { function startSession() { $sessPath = $this->getSessionSavePath(); if( $sessPath != '' ) { - if( !is_dir( $sessPath ) || !is_writeable( $sessPath ) ) { + if( strval( ini_get( 'open_basedir' ) ) != '' ) { + // we need to skip the following check when open_basedir is on. + // The session path probably *wont* be writable by the current + // user, and telling them to change it is bad. Bug 23021. + } elseif( !is_dir( $sessPath ) || !is_writeable( $sessPath ) ) { $this->showError( 'config-session-path-bad', $sessPath ); return false; }