Please port this Module to D6

CommentFileSizeAuthor
#27 auto_expire.patch13.02 KBpobster
#26 auto_expire.patch9.9 KBpobster
#13 auto_expire_6.x-1.0.tar_.gz14.83 KBEvanDonovan
#13 auto_expire_d6.patch14.71 KBEvanDonovan

Comments

mariuss’s picture

mariuss commented
Assigned: Unassigned » mariuss
Category: support » feature
JuLearn’s picture

JuLearn commented

That would be great! AE, we miss you in 6th heaven ^^

phdhiren’s picture

phdhiren commented

Any progress for D6 port?

bbeyer’s picture

bbeyer commented

bump

Muskwa’s picture

Muskwa commented

any news?

nquocbao’s picture

nquocbao commented

Subscribe

matsta’s picture

matsta commented

Subscribing

FWE’s picture

FWE commented

Subscribing...

green monkey’s picture

green monkey commented

Subscribing...

EvanDonovan’s picture

EvanDonovan commented

Subscribing.

jdm843’s picture

jdm843 commented

This has been a very helpful module...would be great to get clarification as to whether or not there are plans to port to 6.x.... so we can plan / move ahead accordingly. Thanks.

Heilong’s picture

Heilong commented

I'd also like to have a try of this module on a D6 environment, I'm currently working on a project in which I need to use expiration and notification on nodes... If there is any plan to port this to D6, it'll be very helpful.
Subscribing...

EvanDonovan’s picture

EvanDonovan commented
Status: Active » Needs review
StatusFileSize
new auto_expire_d6.patch14.71 KB
new auto_expire_6.x-1.0.tar_.gz14.83 KB

Here's a version of the Auto Expire module for Drupal 6. I've attached it as both a patch and a tar.gz. My apologies for the dirtyness of the patch; I'm not the best at making them.

Heilong’s picture

Heilong commented

Thanks for this release, I'll have a look and report as soon as possible.

Heilong’s picture

Heilong commented

I used the tar.gz you posted and it's working great on my installation D6.11.

I'd like to know how to add more warning notifications email at different periods of time.

Heilong’s picture

Heilong commented

I also want to find a way to let the user click on a link or a button in the email that let him extend the node from the email.

I think it could be a good feature to add in the following versions.

marcvangend’s picture

marcvangend
he/him
Primary language Dutch
Location Amsterdam
commented

Thanks for the D6 version, it seems to work great (after a couple of hours playing with it).

Please have a look at my views integration patch: #464776: Views integration in D6 version.

marcvangend’s picture

marcvangend
he/him
Primary language Dutch
Location Amsterdam
commented
Category: feature » task
Status: Needs review » Reviewed & tested by the community

I'm setting this to RTBC since the patch is reported to work well and no issues have come up.

yrre7’s picture

yrre7 commented

subscribing.

marcvangend’s picture

marcvangend
he/him
Primary language Dutch
Location Amsterdam
commented
Version: 5.x-1.x-dev » 6.x-1.0-rc1
Status: Reviewed & tested by the community » Fixed

6.x-1.0-rc1 is released.

Status: Fixed » Closed (fixed)

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

pobster’s picture

pobster commented
Status: Closed (fixed) » Active

There are some coding issues with the current 6.x release, I've only looked over a dozen lines of code and there's a little mess and misunderstanding with the conversion from the 5.x branch.

BTW I've only looked over this module as I'd like to aid with http://drupal.org/node/660300 which tbh, is actually a whole lot easier than you probably imagine... However, there is no way I'm going to implement anything new until the code is shipshape!

Anyways, I'll submit a patch a bit later...

Pobster

marcvangend’s picture

marcvangend
he/him
Primary language Dutch
Location Amsterdam
commented
Status: Active » Closed (fixed)

Pobster, your input and patches are very welcome, but I'd prefer a new issue for the problems you encounter. Thanks in advance.

pobster’s picture

pobster commented

Well it's relating to the port to the D6.x branch and I imagine there probably are people who've installed it from #13 in this issue queue as opposed to the normal project page - reopening this ticket at least alerts them in their trackers that the codebase should be changed...

Pobster

marcvangend’s picture

marcvangend
he/him
Primary language Dutch
Location Amsterdam
commented

That's true. Well, they have been alerted now!

pobster’s picture

pobster commented
StatusFileSize
new auto_expire.patch9.9 KB

Meh, well anyways - please find attached a quick patch, I haven't tested it fully yet - so it does need reviewing.

Afraid I'm off to bed right now so I'll have another look over it before I attempt anything with vbo actions.

Pobster
edit: Erk... NOTE: You'll need to rebuild the menu (i.e. flush your cache) as I used a proper menu loader for gaining a node object. Night!

pobster’s picture

pobster commented
StatusFileSize
new auto_expire.patch13.02 KB

Okay I think that should be everything... Although I didn't notice your HEAD branch has newer code in it (this patch is against DRUPAL-6--1-0-RC1) I'll perhaps merge that in when I take a look at the vbo action stuff tomorrow. Is there any guidance on requirements here? What actions are actually needed?

Incidentally, hope you don't mind but I added an update hook in nodeapi as the first thing I noticed after enabling the module was that you cannot get it to work with existing nodes.

Thanks,

Pobster