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.

CommentFileSizeAuthor
#2 auto_expire-237450-2.patch736 bytesCoPut

Comments

mariuss’s picture

Assigned: Unassigned » mariuss
CoPut’s picture

Status: Active » Needs review
StatusFileSize
new736 bytes

Node expiration date is calculating from current date now.

marcvangend’s picture

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

xamount’s picture

subscribe.

Also when will the drupal version 6 be ready?

xamount’s picture

Status: Postponed » Reviewed & tested by the community

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

marcvangend’s picture

Status: Reviewed & tested by the community » Patch (to be ported)

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

marcvangend’s picture

Version: 5.x-1.0 » 6.x-1.0-rc1
Status: Patch (to be ported) » Fixed

included 6.x-1.0-rc1.

Status: Fixed » Closed (fixed)

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