[BUGFIX] PHP Module pgsql not supported
authorNicole Cordes <nicole@cordes.co>
Tue, 20 Nov 2012 16:42:03 +0000 (17:42 +0100)
committerXavier Perseguers <xavier@typo3.org>
Thu, 22 Nov 2012 08:57:06 +0000 (09:57 +0100)
The return value of the function setRequiredPhpModules does not contain
the php modules but some pseudo name. Therefore the detection of necessary
modules can fail on systems where only the pgsql module is installed.

Change-Id: Id55170cefef6df32ae616c05696bec8fa59b9cb2
Fixes: #43163
Releases: 6.0, 4.7, 4.5
Reviewed-on: http://review.typo3.org/16626
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
Reviewed-by: Jigal van Hemert
Reviewed-by: Xavier Perseguers
Tested-by: Xavier Perseguers
typo3/sysext/dbal/Classes/Hooks/InstallToolHooks.php

index e0e4b66..db676d3 100644 (file)
@@ -70,16 +70,18 @@ class InstallToolHooks {
         * Hooks into Installer to set required PHP modules.
         *
         * @param array $modules
-        * @param tx_install|tx_reports_reports_status_SystemStatus $instObj
+        * @param \TYPO3\CMS\Install\Installer|\TYPO3\CMS\Reports\Report\Status\SystemStatus $instObj
         * @return array modules
         */
-       public function setRequiredPhpModules(array &$modules, $instObj) {
+       public function setRequiredPhpModules(array $modules, $instObj) {
                $modifiedModules = array();
                foreach ($modules as $key => $module) {
                        if ($module === 'mysql') {
                                $dbModules = array();
                                foreach ($this->supportedDrivers as $abstractionLayer => $drivers) {
-                                       $dbModules = array_merge($dbModules, array_keys($drivers));
+                                       foreach ($drivers as $driver) {
+                                               $dbModules = array_merge($dbModules, $driver['extensions']);
+                                       }
                                }
                                $module = $dbModules;
                        }
@@ -497,4 +499,4 @@ class InstallToolHooks {
 }
 
 
-?>
\ No newline at end of file
+?>