[TYPO3-core] FYI: Fixed issue #10614: update the caching framework to the latest version as in FLOW3

Xavier Perseguers typo3 at perseguers.ch
Sun Mar 8 22:02:23 CET 2009


Hi,

>> Did you also test memcached backend? I am asking because I can test it live if you did not have time/chance :)
> 
> Btw, I think I found a error in memcached backend. There is a variable that is assigned but not used. I think it is a bug. This is near line 136:
> 
> 	if (substr($serverConfiguration, 0, 6) === 'tcp://') {
> 		$server = substr($serverConfiguration, 6);
> 	}
> 	if (strstr($serverConfiguration, ':') !== FALSE) {
> 		list($host, $port) = explode(':', $serverConfiguration, 2);
> 	} else {
> 		$host = $serverConfiguration;
> 		$port = $defaultPort;
> 	}
> 
> I think it should be $serverConfiguration, not $server because of the processing in the the second "if".
> 
> This configuration change also means that existing memcached installations should change their configuration from:
> 	'servers' => array('localhost:11211'), 
> to:
> 	'servers' => array('tcp://localhost:11211'), 

+1 after reading.

-- 
Xavier Perseguers
http://xavier.perseguers.ch/en

One contribution a day keeps the fork away


More information about the TYPO3-team-core mailing list