In the forms API, all forms are supposed to be themeable.
However, forms.inc checks to see if a theme_ function exists for a given formid; if it doesn't, it never calls a specific themed function.
The answer is to create a new theme function, which I called 'theme_get_function' which is mostly the code from theme(). It checks for a theme function and, if it finds it, returns it; otherwise it returns false.
theme() then just calls that and executes the function it returns, if one was found.
Patch attached.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | theme_get_function2_0.patch | 2.39 KB | merlinofchaos |
| #3 | theme_get_function2.patch | 2.57 KB | merlinofchaos |
| #1 | theme_get_function_0.patch | 1.93 KB | chx |
| theme_get_function.patch | 1.61 KB | merlinofchaos |
Comments
Comment #1
chx commentedRerolled.
Comment #2
chx commentedComment #3
merlinofchaos commentedNew patch has better doxy, but should be same otherwise.
Comment #4
merlinofchaos commentedUgh cvs diff hates me
Comment #5
dries commentedNot sure I like this patch. Has this been benchmarked?
Comment #6
merlinofchaos commentedAfter fiddling with xdebug:
The call added 15ms in 161 calls to theme() on my server (testing just the front page, which had 10 nodes on it, plus some blocks, etc). My server runs slow, too, and was taking 1200ms to run the entire page.
Comment #7
dries commentedCommitted to HEAD. Thanks.
Comment #8
raema commented