Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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 CreditAttribution: Steven Jones commentedWrong queue.
Patch looks okay visually, though maybe more extensive changes are required to make things more clear?
Comment #2
zeta ζ CreditAttribution: 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 CreditAttribution: Steven Jones commentedYup, fair point.
Comment #4
chx CreditAttribution: chx commentedThe theme hook should be deprecated because the registry already parses the file...
Comment #5
chx CreditAttribution: chx commentedErm no the theme hook has parameters and template/function informations, sorry.
Comment #6
Dries CreditAttribution: Dries commentedCommitted to CVS HEAD. Updating version.
Comment #7
Gábor HojtsyThanks, committed to D6.
Comment #8
Anonymous (not verified) CreditAttribution: Anonymous commentedAutomatically closed -- issue fixed for two weeks with no activity.