Notice: Undefined property: stdClass::$expire in /XXX/sites/all/modules/memcache/dmemcache.inc on line 74

if ($result->expire && .....

should be 

if (isset($result->expire) && .....

to avoid the warning

CommentFileSizeAuthor
#8 memcache_expire.patch802 bytesjcmarco
#6 memcache-n541136-4.patch997 bytesdamienmckenna
#3 memcache-541136-3.patch749 bytesfebbraro

Comments

febbraro’s picture

febbraro commented

Also, I only get this "notice" when I use session_inc. Using cache_inc does not generate this notice.

j4h8’s picture

j4h8 commented

Same here. Bug remains in 6.x-1.4-rc1.

Regards,
Jutta

febbraro’s picture

febbraro commented
Version: 6.x-1.3 » 6.x-1.4-rc1
Status: Active » Needs review
StatusFileSize
new memcache-541136-3.patch749 bytes

Here is a patch to fix it.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

Works for me.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

Ran into a problem.. the current solution doesn't account for when $result->expire == CACHE_PERMANENT.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
StatusFileSize
new memcache-n541136-4.patch997 bytes

Please give this a spin.

febbraro’s picture

febbraro commented

Shouldn't you use 

$result->expire !== CACHE_PERMANENT

instead of 

$result->expire !== 0
jcmarco’s picture

jcmarco commented
Version: 6.x-1.4-rc1 » 6.x-1.4
StatusFileSize
new memcache_expire.patch802 bytes

Tested and working fine,
with patch #6 plus #7

jeremy’s picture

jeremy commented
Status: Needs review » Fixed

Thanks! Patch committed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.