It would be nice if it was possible to set a custom number of days for when comments should be closed.

In combination with my other suggestion, individual settings for content type or node, this would make it very powerful and save me a lot of manual work for content types I am planning for my sites.

CommentFileSizeAuthor
#5 comment_closer_granularity.patch7.23 KBDavid Stosik

Comments

rmiddle’s picture

rmiddle commented

You can set a custom number of days to close. It is 1 number for all types you want to close.

Thanks
Robert

tsvenson’s picture

tsvenson commented

OK, I think you misunderstood me a little. What I mean is that I would like to be able to set for example 10 days as custom time. Now I can select week, month, quarterly and year only.

rmiddle’s picture

rmiddle commented

I might need to look at the code again. The actually query is done buy number of days. It is just a form change then to do that.

Thanks
Robert

rmiddle’s picture

rmiddle commented
Title: Custom number of days for closing » Change select pull down to use number of days instead.

Yea i just checked you are right.

Thanks
Robert

David Stosik’s picture

David Stosik commented
Assigned: Unassigned » David Stosik
Status: Active » Needs review
StatusFileSize
new comment_closer_granularity.patch7.23 KB

Here is a patch that transforms the select "day|week|quarterly|year" in a text field. It is still possible to use the four previous values, but it is possible to put a number in days (so if you put "10", comments will be closed 10 days after create).
I also improved the display of node types, to use human readable labels.

rmiddle’s picture

rmiddle commented

David,

Thanks for the patch. Let me look at it and get it into the code base.

Thanks
Robert

rmiddle’s picture

rmiddle commented
Status: Needs review » Fixed

Committed to CVS. Will be in the next release.

Thanks
Robert

David Stosik’s picture

David Stosik commented

Thank you too!
Do you have an idea of next release date?

rmiddle’s picture

rmiddle commented

At this point. No. I want to do a little testing 1st probably in the next few weeks.

Status: Fixed » Closed (fixed)

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