[BUGFIX] Presets cause fatal error during last installation step 37/24837/3
authorNicole Cordes <typo3@cordes.co>
Wed, 16 Oct 2013 11:06:26 +0000 (13:06 +0200)
committerJigal van Hemert <jigal.van.hemert@typo3.org>
Wed, 16 Oct 2013 14:15:02 +0000 (16:15 +0200)
This patch restructures the PresetInterface and AbstractPreset usage to
prevent any inheritance which causes a fatal error in some PHP versions.

Resolves: #52886
Releases: 6.2
Change-Id: I2505234d41f0a2da2ba49a895fa23c7a8ffa89ac
Reviewed-on: https://review.typo3.org/24837
Reviewed-by: Markus Klein
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel
Reviewed-by: Jigal van Hemert
Tested-by: Jigal van Hemert
typo3/sysext/install/Classes/Configuration/AbstractPreset.php
typo3/sysext/install/Classes/Configuration/Charset/CoreInternalPreset.php
typo3/sysext/install/Classes/Configuration/Charset/IconvPreset.php
typo3/sysext/install/Classes/Configuration/Charset/MbstringPreset.php
typo3/sysext/install/Classes/Configuration/Context/DevelopmentPreset.php
typo3/sysext/install/Classes/Configuration/Context/ProductionPreset.php
typo3/sysext/install/Classes/Configuration/ExtbaseObjectCache/ApcPreset.php
typo3/sysext/install/Classes/Configuration/ExtbaseObjectCache/DatabasePreset.php

index 6e79179..91950f0 100644 (file)
@@ -29,7 +29,7 @@ use TYPO3\CMS\Core\Utility\GeneralUtility;
 /**
  * Abstract preset class implements common preset code
  */
-abstract class AbstractPreset {
+abstract class AbstractPreset implements PresetInterface {
 
        /**
         * @var \TYPO3\CMS\Core\Configuration\ConfigurationManager
@@ -68,13 +68,6 @@ abstract class AbstractPreset {
        }
 
        /**
-        * Check if preset is available on the system
-        *
-        * @return boolean TRUE if preset is available
-        */
-       abstract public function isAvailable();
-
-       /**
         * Wrapper for isAvailable, used in fluid
         *
         * @return boolean TRUE if preset is available
index b6230a0..9e84e15 100644 (file)
@@ -29,7 +29,7 @@ use TYPO3\CMS\Install\Configuration;
 /**
  * Internal core charset handling preset
  */
-class CoreInternalPreset extends Configuration\AbstractPreset implements Configuration\PresetInterface {
+class CoreInternalPreset extends Configuration\AbstractPreset {
 
        /**
         * @var string Name of preset
index fb16b77..d67a54e 100644 (file)
@@ -29,7 +29,7 @@ use TYPO3\CMS\Install\Configuration;
 /**
  * Iconv charset preset
  */
-class IconvPreset extends Configuration\AbstractPreset implements Configuration\PresetInterface {
+class IconvPreset extends Configuration\AbstractPreset {
 
        /**
         * @var string Name of preset
index f553319..ecfac5b 100644 (file)
@@ -29,7 +29,7 @@ use TYPO3\CMS\Install\Configuration;
 /**
  * Mbstring charset preset
  */
-class MbstringPreset extends Configuration\AbstractPreset implements Configuration\PresetInterface {
+class MbstringPreset extends Configuration\AbstractPreset {
 
        /**
         * @var string Name of preset
index 088955e..6a85e69 100644 (file)
@@ -29,7 +29,7 @@ use TYPO3\CMS\Install\Configuration;
 /**
  * Development preset
  */
-class DevelopmentPreset extends Configuration\AbstractPreset implements Configuration\PresetInterface {
+class DevelopmentPreset extends Configuration\AbstractPreset {
 
        /**
         * @var string Name of preset
index 7d31074..7b79153 100644 (file)
@@ -29,7 +29,7 @@ use TYPO3\CMS\Install\Configuration;
 /**
  * Production preset
  */
-class ProductionPreset extends Configuration\AbstractPreset implements Configuration\PresetInterface {
+class ProductionPreset extends Configuration\AbstractPreset {
 
        /**
         * @var string Name of preset
index 239c98d..3550f04 100644 (file)
@@ -29,7 +29,7 @@ use TYPO3\CMS\Install\Configuration;
 /**
  * APC preset
  */
-class ApcPreset extends Configuration\AbstractPreset implements Configuration\PresetInterface {
+class ApcPreset extends Configuration\AbstractPreset {
 
        /**
         * @var string Name of preset
index 2f9d24d..3fb79c8 100644 (file)
@@ -29,7 +29,7 @@ use TYPO3\CMS\Install\Configuration;
 /**
  * Database preset
  */
-class DatabasePreset extends Configuration\AbstractPreset implements Configuration\PresetInterface {
+class DatabasePreset extends Configuration\AbstractPreset {
 
        /**
         * @var string Name of preset