When user tries to extend expired node, its expiration time is calculated starting from expiration date, but not from now.

CoPut - March 22, 2008 - 11:40
Project:Auto Expire
Version:6.x-1.0-rc1
Component:Code
Category:bug report
Priority:normal
Assigned:marcvangend
Status:closed
Description

When user tries to extend expired node, its expiration time is calculated starting from expiration date, but not from current date.

For example:
If my expiration period is 2 days and node expired 5 days ago, I have to extend it minimum 3 times to return it into not expired state.

It would be good if expiration time be calculated with offset from current date.

#1

mariuss - March 25, 2008 - 01:30
Assigned to:Anonymous» mariuss

#2

CoPut - May 19, 2008 - 10:20
Status:active» needs review

Node expiration date is calculating from current date now.

AttachmentSize
auto_expire-237450-2.patch 736 bytes

#3

marcvangend - June 6, 2009 - 19:42
Assigned to:mariuss» marcvangend
Status:needs review» postponed

I agree, the new expiry date of an expired nodes should be calculated as now + $days. However I think that nodes that have not yet expired, should still be calculated as expire + $days, otherwise, the user 'loses' a couple of days when he extends before the expiry date. I intend to implement this behavior in the upcoming D6 version.

#4

xamount - June 8, 2009 - 03:04

subscribe.

Also when will the drupal version 6 be ready?

#5

xamount - June 8, 2009 - 03:14
Status:postponed» reviewed & tested by the community

patch tested...looks good to be committed to version 5 at least.

#6

marcvangend - June 9, 2009 - 22:33
Status:reviewed & tested by the community» patch (to be ported)

Committed to the Drupal5 branch, still to be ported to D6.

#7

marcvangend - June 28, 2009 - 23:21
Version:5.x-1.0» 6.x-1.0-rc1
Status:patch (to be ported)» fixed

included 6.x-1.0-rc1.

#8

System Message - July 12, 2009 - 23:30
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.