Drupal 6 plans?

axel - November 1, 2007 - 13:22
Project:Memcache API and Integration
Version:5.x-1.x-dev
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:closed
Description

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

#1

zarin - March 9, 2008 - 20:17

up

#2

justinrandell - March 11, 2008 - 06:43

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?

AttachmentSize
memcache.inc_.patch 1.21 KB

#3

justinrandell - March 11, 2008 - 06:56
Status:active» needs work

setting to code needs work...

#4

kirie - March 27, 2008 - 05:49

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.

#5

justinrandell - March 27, 2008 - 06:21

@kirie: cool, looking forward to it.

#6

kirie - March 30, 2008 - 07:19
Status:needs work» needs review

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?

AttachmentSize
memcache_d6.patch 7.95 KB

#7

slantview - March 31, 2008 - 18:49
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

#8

dmgorsky - April 15, 2008 - 14:36

Up :)

#9

GoofyX - May 7, 2008 - 20:15

Subscribing.

#10

triclops - May 29, 2008 - 03:36

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

subscribes

#11

jcfiala - August 21, 2008 - 17:05

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

#12

Jeremy - September 10, 2008 - 21:28
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?

#13

patchak - September 22, 2008 - 17:00

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

#14

juan_g - October 12, 2008 - 00:14

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

#15

sidharth_k - October 13, 2008 - 16:54

subscribing

#16

jvandyk - October 19, 2008 - 19:51

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.

AttachmentSize
mcd6jv.patch 27.71 KB

#17

jvandyk - November 24, 2008 - 17:18
Status:needs work» fixed

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

#18

System Message - December 8, 2008 - 17:21
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.