Normalize our Admin controllers so that functions always print out

their results, as opposed to having them return their view back
upstream.  This is a little more code in every controller, but it's
much less magical and more consistent.

Look up the active_theme and active_admin_theme inside the view
itself, no need to do that in the controllers.  This makes view
initialization easier in the controllers.

1 files changed, 3 insertions, 2 deletions

diff --git a/core/libraries/Theme_View.php b/core/libraries/Theme_View.php
index 030e564d..37507a70 100644
--- a/core/libraries/Theme_View.php
+++ b/core/libraries/Theme_View.php
@@ -29,9 +29,10 @@ class Theme_View_Core extends View {
    * @param   string  $theme_name view name
    * @return  void
    */
-  public function __construct($name, $page_type, $theme_name="default") {
+  public function __construct($name, $page_type) {
     parent::__construct($name);
-    $this->theme_name = $theme_name;
+
+    $this->theme_name = module::get_var("core", "active_theme");
     $this->set_global('theme', $this);
     $this->set_global('user', user::active());
     $this->set_global("page_type", $page_type);