theme_role_name is called once for each role in several forms, and each time a query is run to get the summary.

Either:
- load all summaries in to a static
- preload them in the caller and pass them in one at a time along with the role details

The second is probably better as that follows the pattern of giving a theme function what it needs to work with. Though in this case the caller is another theme function!

Setting as minor as this only affects admin edit pages, so not a performance issue really.

Comments

krknth’s picture

Issue summary: View changes
Status: Active » Closed (won't fix)

No support for 6.x versions.

Will optimize in 7.x branch.