$context -> loadTheme() setupThemeContext()
There is so much mixing with $context on where to load it.
I think all of this base $context that doesn't rely on individual actions should be moved to loadThemeContext() and then be called from loadTheme().
Thoughts?
Before you say that setupThemeContext() isn't used by all themes, well, then... I say it (loadThemeContext) should be in the init template (along with a bunch of other things that are theme dependent).
Re: $context -> loadTheme() setupThemeContext()
Reply #1 –
loadTheme() does indeed (still, I think) too much. Speaking off-hand at the moment, I think it does user related stuff which would be better elsewhere... Could try some ways to make it into more manageable chunks. Not sure atm about setupThemeContext(), will need to look.