[BUGFIX][Cache] Wincache backend class constructor 39/19439/2
authorChristian Kuhn <lolli@schwarzbu.ch>
Fri, 29 Mar 2013 14:42:31 +0000 (15:42 +0100)
committerChristian Kuhn <lolli@schwarzbu.ch>
Sat, 30 Mar 2013 09:54:15 +0000 (10:54 +0100)
The class constructor of wincache cache backend misses the first
parameter $context, so $options is treated as $context. Given
backend options are not used this way, for example defaultLifetime
has no effect.
The patch adds the missing constructor parameter to be in line
with the other cache backends.

Change-Id: I488b9996d5ab65420f79b2cab4c75805ae89a77a
Resolves: #46768
Releases: 6.1, 6.0
Reviewed-on: https://review.typo3.org/19439
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
typo3/sysext/core/Classes/Cache/Backend/WincacheBackend.php

index fb96a06..a14e8c6 100644 (file)
@@ -57,14 +57,15 @@ class WincacheBackend extends \TYPO3\CMS\Core\Cache\Backend\AbstractBackend impl
        /**
         * Constructs this backend
         *
-        * @param mixed $options Configuration options - unused here
+        * @param string $context FLOW3's application context
+        * @param array $options Configuration options
         * @throws \TYPO3\CMS\Core\Cache\Exception If wincache PHP extension is not loaded
         */
-       public function __construct($options = array()) {
+       public function __construct($context, array $options = array()) {
                if (!extension_loaded('wincache')) {
                        throw new \TYPO3\CMS\Core\Cache\Exception('The PHP extension "wincache" must be installed and loaded in order to use the wincache backend.', 1343331520);
                }
-               parent::__construct($options);
+               parent::__construct($context, $options);
        }
 
        /**