* changed variable list as per comment on r79954 left only wgDBtype
authorJure Kajzer <freakolowsky@users.mediawiki.org>
Fri, 14 Jan 2011 07:05:21 +0000 (07:05 +0000)
committerJure Kajzer <freakolowsky@users.mediawiki.org>
Fri, 14 Jan 2011 07:05:21 +0000 (07:05 +0000)
* all other variables get set trough ->getGlobalNames
* could somone test this on mysql, postgres and sqlite ... it should work, but to be sure

includes/installer/OracleInstaller.php
includes/installer/WebInstallerPage.php

index 4223402..8a95711 100644 (file)
@@ -15,7 +15,7 @@
 class OracleInstaller extends DatabaseInstaller {
 
        protected $globalNames = array(
-               'wgDBport',
+               'wgDBserver',
                'wgDBname',
                'wgDBuser',
                'wgDBpassword',
index 453aa27..0e2b089 100644 (file)
@@ -300,7 +300,7 @@ class WebInstaller_ExistingWiki extends WebInstallerPage {
                }
 
                // Set the relevant variables from LocalSettings.php
-               $requiredVars = array( 'wgDBtype', 'wgDBuser', 'wgDBpassword', 'wgDBname', 'wgDBserver' );
+               $requiredVars = array( 'wgDBtype' );
                $status = $this->importVariables( $requiredVars , $vars );
                $installer = $this->parent->getDBInstaller();
                $status->merge( $this->importVariables( $installer->getGlobalNames(), $vars ) );