Closed (fixed)
Project:
Drush
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
14 Feb 2010 at 05:12 UTC
Updated:
3 Apr 2010 at 21:50 UTC
Jump to comment: Most recent file
This patch accomplishes a number of things:
Naming conventions are somewhat debatable, I suppose. I was also planning 'fn-module', to show all of the hooks defined in a named module. That's not implemented yet, but it would make a suite of "fn" commands. I'm open for other suggestions.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | fn-impl.patch | 4.39 KB | greg.1.anderson |
Comments
Comment #1
moshe weitzman commentedI'm tempted to move development commands like these to devel module. Any objections? We'll focus drush core on site administration
Comment #2
greg.1.anderson commentedYeah, that would be fine by me.
Comment #3
moshe weitzman commentedForgot to upload? No patch here.
Comment #4
greg.1.anderson commentedWhoops. Here you go.
I could also post this against devel, if you prefer.
Comment #5
moshe weitzman commentedyes, patch against devel would be ideal.
I'm pretty interested in showing methods here too. They are getting popular in D7. See http://www.php.net/manual/en/class.reflectionmethod.php. I think we should use same drush command for both. We could require a syntax like class::method or assume method if there are 2 arguments.
Comment #6
moshe weitzman commentedCommitted to devel. In addition to showing methods, we need some better error handling when the supplied hook has no implementations or the supplied function does not exist.