[HttpFoundation] Fix incompatibility with php-memcache from Debian

[ymc-dabe]

Q	A
Bug fix?	yes
New feature?	no
BC breaks?	no
Deprecations?	no
Tests pass?	yes
Fixed tickets	-
License	MIT
Doc PR	-

The version of php-memcache (3.0.9~20151130.fdbd46b-1) in Debian makes
the test MemcacheSessionHandlerTest::testReadSession fail with the following
error message:

There was 1 error:

1) Symfony\Component\HttpFoundation\Tests\Session\Storage\Handler\MemcacheSessionHandlerTest::testReadSession
Missing argument 2 for Mock_Memcache_ddacd7a0::get(), called in /tmp/buildd/symfony/src/Symfony/Component/HttpFoundation/Session/Storage/Handler/MemcacheSessionHandler.php on line 82 and defined

/tmp/buildd/symfony/src/Symfony/Component/HttpFoundation/Session/Storage/Handler/MemcacheSessionHandler.php:82
/tmp/buildd/symfony/src/Symfony/Component/HttpFoundation/Tests/Session/Storage/Handler/MemcacheSessionHandlerTest.php:75

This PR solves the issue reported here.

[ymc-dabe]

I guess the actual problem lays in (Debian's) php-memcache. But since fixing this in Symfony itself was rather easy, I thought it was worth forwarding the patch upstream to the Symfony project. But I'm up to investigating this further and if applicable report a bug against php-memcache in Debian, if you think patching Symfony is the wrong way to deal with this issue.

[fabpot]

I'm not comfortable with this patch as it looks really weird. It should be fixed upstream instead IMHO.

[xabbuh]

I agree. This should be fixed in the Debian package instead.

[ymc-dabe]

Actually it should be fixed in https://github.com/websupport-sk/pecl-memcache, which is Debian's upstream for php-memcache. I'll do a little testing later to validate if this still exists with recent master and open an issue at websupport-sk/pecl-memcache if needed (and a Bug for Debian's php-memcache anyhow). Thanks for your feedback.