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.
This is a start -- placing here mostly so Steef can continue work. Pay attention to comments I placed in the drupal_common_themes() function, they're of minor importance to the really core theme implementations.
Comment | File | Size | Author |
---|---|---|---|
tplify_theme.inc_.patch | 4.56 KB | merlinofchaos | |
Comments
Comment #1
merlinofchaos CreditAttribution: merlinofchaos commentedDocumenting these templates is going to be hard. I recommend we do this:
For each module that has theme functions, create a theme.doc.php (that will never be included).
We put the empty theme_* function in this file, and include the proper doxygen. It MUST contain @ingroup themeable (or put the entire file in @group themeable if that works with doxy) and if we can figure out how to do it, give it an @see that points to the proper template. Some of this might require a little delving into api.module.
Comment #2
dvessel CreditAttribution: dvessel commentedMerlin, could you provide a list to start with? I understand some functions won't need tplifying so I'd like to avoid those.
I've been working on another project so please excuse the delay. I'll start now.
Comment #3
merlinofchaos CreditAttribution: merlinofchaos commenteddvessel: I'll have to compile a list. In general a theme function in a .module is probably fair game; mostly it's the ones in core that need to be exampled.
To start, I submitted the forum theme functions here: http://drupal.org/node/158536
It contains a documentation style that should be discussed and, hopefully, adopted.
Comment #4
merlinofchaos CreditAttribution: merlinofchaos commentedIn my mind, the priorities are aggregator module, book module and comment module. I've got aggregator about half done, and I'm paused for the moment.
Other high priorities include
After that, I'm not sure where to go.
Comment #5
dvessel CreditAttribution: dvessel commentedI'm working on book.module. Actions and block is up. --Was going through it alphabetically. Started on aggregator but noticed it was broke so I skipped it.
http://drupal.org/node/159862
http://drupal.org/node/159936
All the issues I create will be named the same way.. "tplify * module".
Comment #6
dvessel CreditAttribution: dvessel commentedmerlinofchaos: Should there be a central issue cue to checkoff on converting these templates? Not sure if it's proper.
Anyway, here are a 2 more. book.module & comment.module.
http://drupal.org/node/160200
http://drupal.org/node/160589
Comment #7
merlinofchaos CreditAttribution: merlinofchaos commentedBumping -- need to do a reroll of this patch so it doesn't get forgotten.
Comment #8
dpearcefl CreditAttribution: dpearcefl commentedIs this still an active issue that needs to be fixed?