This patch refactors the theme('content_moderation_node_history') function to use a template file instead of the theme_hook() function.

Comments

eugenmayer’s picture

Well eventhough i see that your path cleans up the code a bit, moving the logic to the preprocess method.. i ask myself, if it is really worth in letting drupal load a file (tpl) just for this simple line? Thats just one more I/O call...

...

but on the other hand, this patch gives everybody the ability to change the way it looks like AND the way it gets computed, e.g. adding some bulk methods. I take that one in for 1.6, once again great work dparks, thanks!

eugenmayer’s picture

Status: Active » Fixed

included in 1.6. Thanks you!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.