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

Ingo Renner ingo at typo3.org
Tue Mar 10 14:20:49 CET 2009


Dmitry Dulepov wrote:
> Hi!
>
> Dmitry Dulepov wrote:
>> 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".

yep, right. must have overlooked that when changing the variable name. 
just change it if you find the time.

thanks for spotting that
Ingo

-- 
Ingo Renner
TYPO3 Core Developer, Release Manager TYPO3 4.2



More information about the TYPO3-team-core mailing list