From c7193f9b2ede6ed913d16a9a8047b6867b8afada Mon Sep 17 00:00:00 2001 From: Bharat Mediratta Date: Wed, 24 Dec 2008 04:22:22 +0000 Subject: 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. --- modules/user/controllers/admin_users.php | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) (limited to 'modules/user/controllers/admin_users.php') diff --git a/modules/user/controllers/admin_users.php b/modules/user/controllers/admin_users.php index 38e4e43a..ac328780 100644 --- a/modules/user/controllers/admin_users.php +++ b/modules/user/controllers/admin_users.php @@ -19,9 +19,10 @@ */ class Admin_Users_Controller extends Controller { public function index() { - $view = new View("admin_users.html"); - $view->users = ORM::factory("user")->find_all(); - return $view; + $view = new Admin_View("admin.html"); + $view->content = new View("admin_users.html"); + $view->content->users = ORM::factory("user")->find_all(); + print $view; } public function edit($id) { @@ -40,6 +41,8 @@ class Admin_Users_Controller extends Controller { url::redirect("admin/users/edit/$id"); } - return $form; + $view = new Admin_View("admin.html"); + $view->content = $form; + print $view; } } -- cgit v1.2.3