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

CommentFileSizeAuthor
#3 minimal_d6.diff3.92 KBmooffie

Comments

Crell’s picture

Crell commented
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. :-)

mooffie’s picture

mooffie commented
Status: Fixed » Active

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

mooffie’s picture

mooffie commented
Status: Active » Needs review
StatusFileSize
new minimal_d6.diff3.92 KB

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.)

Crell’s picture

Crell commented

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

mooffie’s picture

mooffie commented

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.

Crell’s picture

Crell commented

... 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.

Crell’s picture

Crell commented
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.

Status: Fixed » Closed (fixed)

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