Closed (outdated)
Project:
Chaos Tool Suite (ctools)
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
12 Jun 2009 at 00:24 UTC
Updated:
25 Feb 2016 at 04:43 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
moonray commentedComment #2
merlinofchaos commentedThis totally makes sense, though it can be a rather complex operation to do right (see the fun in Views there) -- patches welcome!
Comment #3
moonray commentedAttached is a patch that adds default argument handling for the delegator module. I didn't apply it anywhere else, since I wanted feedback on it first.
Also, if you could provide a list of files I would need to add this code to, it would be appreciated. That way I wont miss any.
Comment #4
merlinofchaos commentedWow. Ok, so this patch is awesome, and it totally won't apply since as of last night, delegator.module is history. I like this enough I may try to manually recreate it, though. This is good stuff!
Comment #5
merlinofchaos commentedI'm thinking about the utility of this in terms of the new UI. And '404' is an unnecessary option. If you want to 404 if the argument is not there, then it's not an optional argument.
So what this really is for is to provide defaults if the argument is not specified. To do that we probably need to hook into the context UI so that the defaults can be provided by whatever UI is there. That makes it easy to provide default terms by selection rather than just entering a number for example. It's possible we'll need some way to do PHP code selection of a default, as well, though the need for that is a little less than it is with Views where you often have to extract defaults from URLs for blocks. This one you always have the URLs. Still, this bears some thinking about. We should consider as many use cases, as possible, of what we might want to do if arguments do not exist.
Comment #6
japerryClosing this issue as outdated as Drupal 6 ctools is not supported. If this issue is relevant for Drupal 7, feel free to re-open and mark for Drupal 7 (or 8)