It was suggested in another issue to add caching. Making an issue for it so I don't forget.

Michelle

Comments

crea’s picture

Subscribing ;)

michelle’s picture

Title: Cache AP variables » Cache Author Pane
Project: Author Pane » Advanced Forum

Moving this to AF's queue. There isn't a way to cache AP from within AP since there's no template post process and it's really only needed where you have multiple APs, like AF.

Michelle

michelle’s picture

Project: Advanced Forum » Author Pane

Spoke too soon... Should have finished the IRC conversation first. :)

Michelle: so you could call hook_theme_registry_alter() and add a template_process_HOOK function to the end of the list for your module's own theme hooks.

That would make it work with anything else that showed multiples of the same AP on a page as well. Probably not much outside of AF will do that but you never know.

Michelle

michelle’s picture

Status: Active » Fixed

I decided against the whole pane caching because it's very messy and is already handled by AF with Nodecomment anyway due to the Views integration. Instead, I implemented static variable caching on the integrations maintained in AP that have queries. Committed but needs thorough testing.

Michelle

Status: Fixed » Closed (fixed)

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