Memcached adapter store bug in internalSetItems method for PHP 7.0 about zend-cache HOT 7 CLOSED

hi @malinink

As you describe it looks like you the value gets serialized which has nothing to do with the variable by reference you linked.

Please provide a code example incl. how you setup the storage adapter and call to write/read with data as var_dump.

from zend-cache.

@DCMNMarc
I add test into MemcachedTest
All connection options are default (from phpunit.xml.dist)

    public function testMultipleKeySetIncrement()
    {
        $this->_storage->setItems(['key' => 0]);
        $value = $this->_storage->getItem('key');
        var_dump($value);
        $this->_storage->incrementItem('key', 1);
    }

That is test screen:

./vendor/bin/phpunit --filter testMultipleKeySetIncrement
PHPUnit 4.8.22 by Sebastian Bergmann and contributors.

Eint(0)


Time: 167 ms, Memory: 8.00Mb

There was 1 error:

1) ZendTest\Cache\Storage\Adapter\MemcachedTest::testMultipleKeySetIncrement
Zend\Cache\Exception\RuntimeException: CLIENT ERROR

zend-cache/src/Storage/Adapter/Memcached.php:700
zend-cache/src/Storage/Adapter/Memcached.php:578
zend-cache/src/Storage/Adapter/AbstractAdapter.php:1272
zend-cache/test/Storage/Adapter/MemcachedTest.php:206

FAILURES!
Tests: 1, Assertions: 2, Errors: 1.

It seems like adapter could get and set such kind of values but it store them as none integer values, so increment doesn't work.

That is real value stored in memcached

echo "get zfcache:key" | nc localhost 11211
VALUE zfcache:key 4 4
i:0;
END

Normally that value must be (and that happens in setItem() method)

echo "get zfcache:key" | nc localhost 11211
VALUE zfcache:key 1 1
0
END

That is error itself

echo "incr zfcache:key 1" | nc localhost 11211
CLIENT_ERROR cannot increment or decrement non-numeric value

from zend-cache.

@malinink Thanks for your help. I can reproduce the bug with Memcached and Apc and created a fix with #67
It's probably related the internal changes how PHP-7 handles variables by reference.

from zend-cache.

@marc-mabe
That PR completely fixes the issue. Thanks!

from zend-cache.

@malinink nice but the issue should be closed on merging that PR - thanks

from zend-cache.

link #74

@malinink I just noticed that currently there is no release (no stable, no beta, no alpha) for php memcached that supports PHP-7 (see #74) how did you install that?

from zend-cache.

@marc-mabe
I have installed php7.0-fpm package from that ppa https://launchpad.net/~ondrej/+archive/ubuntu/php
But as you can see there is no php7.0-memcached package present there.
I could install it as usual with php-memcached name.
I also install php-redis and other packages that are not prefixed with php7.0 (I could provide all list), and have not found any problems (So I decided that these packages are fully compatible).

That is one thing I don't think about.

from zend-cache.