Closed (fixed)
Project:
Views (for Drupal 7)
Version:
6.x-2.x-dev
Component:
Views Data
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
26 Aug 2008 at 21:18 UTC
Updated:
14 Jul 2012 at 23:14 UTC
Jump to comment: Most recent file
Comments
Comment #1
merlinofchaos commentedmenu_get_object('node') won't always work since things like Views use their own argument handlers, and Panels will probably do the same thing. So we have to watch out for that.
I was always a little uncomfortable with the date-embedded defaults, too, though I guess that concept is basically ok, but there's a giant slippery slope here about wanting to use data from one object to seed another object, and I'm not quite sure the best way to go about doing that. Is this the road we really want to go down from here?
Comment #2
merlinofchaos commentedComment #3
eaton commentedHere's a re-rolled version of the patch that uses the 'smarter' approach used in the node-nid argument default plugin.
I definitely share the concern. There are a lot of ways that these defaults could be handled -- why not date fields on the node, or the user's last login? All could be really useful, I think, but it's easy for it to get out of control. This one "feels" like an easy win and enables a lot of nice stuff (sidebar blocks that show content from the node's week, not the current week, etc) but I don't know how we could safely/reasonably push it much farther than this...
Comment #4
eaton commentedEr, HERE is a re-rolled version. Just tested it and whipped up a nice "Twitter messages posted on the same day this node was posted" sidebar block.
Comment #5
merlinofchaos commentedCommitted.
Comment #6
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.