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

Wincache unexpected return value on internalGetItem #4552

Closed
sebdalink opened this issue May 28, 2013 · 1 comment
Closed

Wincache unexpected return value on internalGetItem #4552

sebdalink opened this issue May 28, 2013 · 1 comment
Milestone

Comments

@sebdalink
Copy link

On internalGetItem function, the expected return value, when item isn't found on cache, is null.
But wincache_ucache_get return false when $key isn't found on cache, so $result has a wrong return value in this case.

@marc-mabe
Copy link
Member

@sebdalink Thank you for reporting this!
From reading the documentation you are right -> $result needs to be set to null if $success is false.

I'll create a PR but I can't run tests for this adapter - It would be great if you can run the unit tests for this adapter.

weierophinney added a commit that referenced this issue Jun 12, 2013
fixed #4552: Wincache::getItem() have to return NULL in cases of missing items
weierophinney added a commit that referenced this issue Jun 12, 2013
gianarb pushed a commit to zendframework/zend-cache that referenced this issue May 15, 2015
…ncache::getItem() have to return NULL in cases of not found items
weierophinney added a commit to zendframework/zend-cache that referenced this issue May 15, 2015
…tfix/4552

fixed zendframework/zendframework#4552: Wincache::getItem() have to return NULL in cases of missing items
weierophinney added a commit to zendframework/zend-cache that referenced this issue May 15, 2015
boesing pushed a commit to laminas/laminas-cache-storage-adapter-apc that referenced this issue May 24, 2020
…ncache::getItem() have to return NULL in cases of not found items
boesing pushed a commit to laminas/laminas-cache-storage-adapter-wincache that referenced this issue Jul 20, 2020
…ncache::getItem() have to return NULL in cases of not found items
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants