If you get the error: warning: Memcached::set() expects at most 3 parameters, 4 given in /var/www/sites/all/modules/memcache/dmemcache.inc on line 42., you can change lines 39 through 44 of dmemcache.inc to:

if (class_exists('Memcached')) {
  return $mc->set($full_key, $value, $exp);
}
else if (class_exists('Memcache')) {
  return $mc->set($full_key, $value, MEMCACHE_COMPRESSED, $exp);
}

This should fix it.

CommentFileSizeAuthor
#7 memcache-dmemcache.inc-respect_pecl_memcached-837758-7.patch1.87 KBsdelbosc
#1 respect-pecl-memcached.patch1.13 KBxy2

Comments

xy2’s picture

xy2 commented
StatusFileSize
new respect-pecl-memcached.patch1.13 KB

This is already fixed in D7 version.

Please apply patch to 6.x branch

Nathan Goulding’s picture

Nathan Goulding commented

Has this been applied yet?

luthien’s picture

luthien commented

the fix does not works for d6, I keep getting the error and an empty blank page for admin/reports/memcache

Note: the patch is working fine.

rjbrown99’s picture

rjbrown99 commented

Works for me, and I have been testing for a few days now. Can we go RTBC?

vacilando’s picture

vacilando commented

Same problem.

jeremy’s picture

jeremy commented
Status: Needs review » Fixed

Thanks, fix committed.
http://drupal.org/cvs?commit=412702

sdelbosc’s picture

sdelbosc commented
Status: Fixed » Needs review
StatusFileSize
new memcache-dmemcache.inc-respect_pecl_memcached-837758-7.patch1.87 KB

I think a return statement was missing in the patch given in the comment #1.

robertdouglass’s picture

robertdouglass commented
Status: Needs review » Fixed

fixed in #6 as far as I can tell.

Status: Fixed » Closed (fixed)

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