Drupal 6 Please

Flying Drupalist - August 30, 2008 - 17:13
Project:Component
Version:6.x-1.x-dev
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:closed
Description

Can this be updated for drupal 6 please? I'm looking for just such a module.

#1

Crell - November 13, 2008 - 09:26
Version:5.x-1.x-dev» 6.x-1.x-dev
Status:active» fixed

The views form function doesn't work yet and panels is still pending panels figuring out what it's doing, but most of it seems to work. :-)

#2

mooffie - November 26, 2008 - 13:53
Status:fixed» active

The module's hook_menu() isn't compatible with D6.

#3

mooffie - November 26, 2008 - 14:44
Status:active» needs review

Here's a minimal port to D6. (sans the Views stuff, which I might work on some other time.)

(Note that I'm doing a node_access('view') access check. I'm not sure the D5 version has it.)

AttachmentSize
minimal_d6.diff 3.92 KB

#4

Crell - November 26, 2008 - 15:55

The hook_menu() in the 6.x branch should be converted to D6. I remember doing so. Are you sure?

#5

mooffie - November 26, 2008 - 16:46

Oh, my!

You made your changes to HEAD. But the '6.x-dev' package is pointing to DRUPAL-6--1, which is still D5 code. I was looking at this '6.x-dev', that's why I didn't see your D6 code.

#6

Crell - November 26, 2008 - 22:06

... Oh crap. I see what the problem is. I keep forgetting after I branch in CVS that it does not switch to that branch. *sigh* I'll fix this in CVS shortly. Sorry about the mixup.

#7

Crell - December 2, 2008 - 04:38
Status:needs review» fixed

OK, I think I got CVS sorted out now. Give the tarball a few hours to rebuild. Sorry for the mixup.

#8

System Message - December 16, 2008 - 04:41
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.