Is there time plans for update project to upcoming Drupal 6 API? Thanks.

CommentFileSizeAuthor
#16 mcd6jv.patch27.71 KBjvandyk
#6 memcache_d6.patch7.95 KBkirie
#2 memcache.inc_.patch1.21 KBAnonymous (not verified)

Comments

zarin’s picture

zarin commented

up

Anonymous’s picture

Anonymous (not verified) commented
StatusFileSize
new memcache.inc_.patch1.21 KB

lightly tested patch attached which changes the function signature of cache_set in memcache.inc as per the changes from cache_set in drupal 5 to cache_set in drupal 6.

i'm interested in doing some real work here - has anyone else already started on this?

Anonymous’s picture

Anonymous (not verified) commented
Status: Active » Needs work

setting to code needs work...

kirie’s picture

kirie commented

We have a couple of guys looking into making a basic workable package for D6. We'll update this thread with files/comments when there are any updates - we'll hopefully have something over the weekend.

Anonymous’s picture

Anonymous (not verified) commented

@kirie: cool, looking forward to it.

kirie’s picture

kirie commented
Status: Needs work » Needs review
StatusFileSize
new memcache_d6.patch7.95 KB

This patch (rolled against HEAD) updates both memcache.inc and memcache.db.inc. There is not a lot of change here, the biggest being the cache_get and cache_get function signatures. I've also changed a parameter name ($key to $cid), a variable name ($time to $created) and some function documentation to bring it more in line with D6. Also added in cache_clear_all(NULL, 'cache_block'); as per D6.

The total number of required files to make memcache work correctly in D6 have been drastically reduced, as there is no need to patch core or update tables:
dmemcache.inc
memcache.db.inc
memcache.inc

As there is not need to use the existing memcache.module (actually its .install file) to update tables, I think the functionality in memcache_admin could be moved here?

slantview’s picture

slantview commented
Status: Needs review » Needs work

I will have a D6 branch later this week, worse case, end of next week. Thanks for submitting this, but I will be combining some of the work I already put into Cache Router - http://drupal.org/project/cacherouter

dmgorsky’s picture

dmgorsky commented

Up :)

GoofyX’s picture

GoofyX commented

Subscribing.

triclops’s picture

triclops commented

woops. why didn't I see this before I did this... http://drupal.org/node/264015

subscribes

jcfiala’s picture

jcfiala commented

Any updates on the Drupal 6 port? I need this for a project, so I'm willing to donate some time.

jeremy’s picture

jeremy commented
Priority: Minor » Normal

What's the status on this porting effort? How can we best pitch in? @slantview, you suggest you've got your own changes you want to integrate rather than the posted patches, any luck with that?

patchak’s picture

patchak commented

Will there be any d6 version of this project anytime soon? I'm waiting for this to upgrade some sites, since the patch has been posted so long ago would be nice to revisit this issue and keep it up to date?

Thanks a lot for the work,
Patchak

juan_g’s picture

juan_g commented

A Drupal 6 possibility for those in a hurry is the memcache option of the Cache Router mentioned above by slantview.

sidharth_k’s picture

sidharth_k commented

subscribing

jvandyk’s picture

jvandyk commented
StatusFileSize
new mcd6jv.patch27.71 KB

This patch is against the 6--1 branch (though there is no 6.x-1.x-dev option in the Version dropdown, so I'm leaving it as 5.x-1.x-dev). It is based on kirie's changes in #6 but also ports the memcache_admin module to Drupal 6 so that the state of memcache bins can be inspected.

jvandyk’s picture

jvandyk commented
Status: Needs work » Fixed

6.x-1.0 release is out, so closing this issue.

Status: Fixed » Closed (fixed)

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