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, 1 insertions, 4 deletions

diff --git a/core/controllers/photos.php b/core/controllers/photos.php
index 730cfd2c..4690bf58 100644
--- a/core/controllers/photos.php
+++ b/core/controllers/photos.php
@@ -25,10 +25,7 @@ class Photos_Controller extends Items_Controller {
   public function _show($photo) {
     access::required("view", $photo);
 
-    $theme_name = module::get_var("core", "active_theme", "default");
-
-    $template = new Theme_View("page.html", "photo", $theme_name);
-
+    $template = new Theme_View("page.html", "photo");
     $template->set_global('item', $photo);
     $template->set_global('children', array());
     $template->set_global('children_count', $photo->children_count());