PHP 7 compatibility

[mevdschee]

Does anyone understand why memcache.so is not available for PHP 7 on Travis?

https://travis-ci.org/LeaseWeb/LswMemcacheBundle/builds/69521319

Thanks!

[hanikesn]

Apparently the memcache extension has not been ported to PHP 7, yet. I'm currently trying to get our code base to run with PHP 7 and this whole memcache/memcahed mess is quite hard to handle.

[jimaek]

@mevdschee
It doesnt look like the memcache extension is going to be updated. Someone made a fork with PHP7 support https://github.com/websupport-sk/pecl-memcache (didnt test it)

But most switched to memcached that supports PHP7 as far as I know.
Any plans to switch to the other better supported extension?

[mevdschee]

Version 2 is already switched to memcached. (edit: I'm wrong, it is switched from memcached to memcache)

[jimaek]

From the readme: "NB: This bundle no longer uses the PHP "memcached" extension that uses "libmemcached", see "Considerations". "

Version 2 switched away from the PHP7 supported extension to the currently broken one.

[mevdschee]

Ah.. sorry, even I was confused (one letter, but such a big difference). PSR compliant PHP projects can better avoid memcached, due to the problems that project has with inheritance of it's base class. See: #21

Let's hope that @szabolcsbalogh succeeds 👍

[hanikesn]

@jimaek I'm kinda mainting a memcached compatible version:
https://github.com/hanikesn/memcached-bundle/commits/memcached-compat
The problem is that for newer memcached versions, it's not extending \Memcached anymore. There are workaround for DoctrineCache in there, but if you have custom code you need to change that to use \MemcachedInterface.

[mevdschee]

Good work @hanikesn !

[jimaek]

@mevdschee Ok, so I guess this module wont get PHP7 support until that extension gets updated?
I dont think this will happen any time soon if at all. Maybe you could link people to the fork https://github.com/websupport-sk/pecl-memcache to at least have the option to compile it themselves if they want to

[mevdschee]

I added a link to the readme. Let's hope he does a PR and it gets merged.