I reckon that depending on date_api is a fairly major thing, given the extra functionality you actually get if you have it on. Can we look at factoring out the code so that the expiry stuff just doesn't appear if you don't have date_api.

Comments

jamsilver’s picture

jamsilver commented
Category: task » feature
Status: Active » Needs work

Good idea. If you have a spare moment, feel free to give this a go yourself Steven.

steven jones’s picture

steven jones commented

Do you reckon this is better as an add-on module that uses the content levels API to change the content levels when it needs to, or just some nice if statements around bits in the main module?

steven jones’s picture

steven jones commented
Assigned: Unassigned » steven jones
jamsilver’s picture

jamsilver commented
Version: » 6.x-1.x-dev
Category: feature » task

I think if-statements will be good for the time being. And a little note explaining that more can be gained through the use of date_api. Changed this from a feature request since it's not really as significant as that and appropriate to get in at the release candidate stage I would say =p.

jamsilver’s picture

jamsilver commented
Assigned: steven jones » jamsilver
Status: Needs work » Fixed

Fixed in rc5.

Status: Fixed » Closed (fixed)

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