Skip to content
This repository has been archived by the owner on Jan 29, 2020. It is now read-only.

StorageInterface::hasItem issue with ext-redis 4.0.1+ #174

Closed
boesing opened this issue Sep 7, 2018 · 1 comment
Closed

StorageInterface::hasItem issue with ext-redis 4.0.1+ #174

boesing opened this issue Sep 7, 2018 · 1 comment

Comments

@boesing
Copy link
Member

boesing commented Sep 7, 2018

I am actually working on zend-cache v3 and therefore a zend-cache-redis extension, where I am running into unit test errors.

I narrowed the issue down to ext-redis which introduced a huge bc break in a minor release.
Starting with ext-redis v4.0.1, they changed the return value of exists from boolean to integer.

I've created an issue phpredis/phpredis#1401 to see what can be done.
However, actually, zend-cache is not working with ext-redis 4.0.1+ (at least if someone uses strict type checks).

I just wanted you guys to be on track with that issue.
Not sure if we should provide a patch for those versions until they fix that issue?

Code to reproduce the issue

Just execute unit tests on local machine with ext-redis 4.0.1+.

Expected results

StorageInterface::hasItem returns boolean

Actual results

StorageInterface::hasItem returns integer

@boesing boesing closed this as completed Sep 7, 2018
@boesing
Copy link
Member Author

boesing commented Sep 7, 2018

Never mind, the project I've forked had old code.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant