Node creation dates (and parts of it such as year, month, day, monthwithyear ...) can be used as arguments in Views, so I can create a beautiful view (I can't say how much I love this module :-) that will give me something like this:
posts --> list of all the posts
posts/dates ---> list of all years with posts
posts/dates/2009 ---> display all the post in a year (and an attachment letting me choose months of that year with posts)
posts/dates/2009/01 --> display all the posts in that month (and an attachment letting me choose days of that month with posts)
posts/dates/2009/01/12 --> display all the posts in that day
What I cannot get is the same with comments (i.e., lists of comments by date):
comments/dates/[YYYY]/[MM]/[DD]
The key thing is dates of creation (in its diverse forms) are not available as arguments for comments, so there is no way you can split comments by date.
I think that would be very useful, but wanted to know OPPs opinion. I am no coder, so I cannot foresee how hard or easy to implement this would be. However, the same is implemented for nodes already, so I hope that is at least a base.
Thanks!
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | views-373615-6.patch | 2.14 KB | lonelyrobot |
| #1 | views-comment-timestamp-arguments.patch | 6.83 KB | dawehner |
Comments
Comment #1
dawehnerthis is a first patch which should solve this issue
TODO: provide this kind of stuff for every timestamp field
Comment #2
merlinofchaos commentedIs there any need to copy the node_created classes? The code didn't change, right? We could just use them and declare them misnamed.
Comment #3
dawehneryes the code is exactly the same
so there has to be a abstract class foreach kind of handlers(month/day/week...)?
i'm not very familiar with OOP
Comment #4
merlinofchaos commentedOnly for historical reasons. If you can reroll the patch to just use the existing ones, we can declare them misnamed and move on from there. Changing the names risks breaking things so we'll live with incorrect names.
Comment #5
merlinofchaos commentedStill needs a reroll.
Comment #6
lonelyrobot commentedReroll of the patch to just use the existing argument handlers.
Comment #7
merlinofchaos commentedApplied to the 3.x branch!