Closed (fixed)
Project:
Drupal core
Version:
6.x-dev
Component:
documentation
Priority:
Minor
Category:
Bug report
Assigned:
Reporter:
Created:
3 Jun 2008 at 23:19 UTC
Updated:
9 Jul 2008 at 08:02 UTC
I’m guessing with this, but it did confuse me when I was trying to work out how to register a theme_hook().
| Comment | File | Size | Author |
|---|---|---|---|
| themeing.patch | 674 bytes | zeta ζ |
Comments
Comment #1
steven jones commentedWrong queue.
Patch looks okay visually, though maybe more extensive changes are required to make things more clear?
Comment #2
zeta ζ commentedMaybe – though I’ll leave that to someone who knows what it is, that should be made clear. This patch makes it clear enough not to confuse me, which is all I wanted to avoid for the next person viewing the docs.
I thought it was fairly obvious that the wrong word was used unintentionally, rather than the original author was being unclear, so I didn’t attempt a rewrite.
Any suggestions?
If you would rather leave this as obviously wrong in order to encourage someone to rewrite the rest of it then I will have to wait.
I was trying to use the documentation to learn what the answer should be, rather than trying to spot errors.
Comment #3
steven jones commentedYup, fair point.
Comment #4
chx commentedThe theme hook should be deprecated because the registry already parses the file...
Comment #5
chx commentedErm no the theme hook has parameters and template/function informations, sorry.
Comment #6
dries commentedCommitted to CVS HEAD. Updating version.
Comment #7
gábor hojtsyThanks, committed to D6.
Comment #8
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.