From d9e02a5d0c1a81925df33a9b25501fc90db91451 Mon Sep 17 00:00:00 2001 From: Bharat Mediratta Date: Tue, 16 Dec 2008 04:29:00 +0000 Subject: Various optimizations: o Add model_cache::get() which caches models avoiding duplicate lookups o Stop using ORM relationships for Item_Model::owner so that we can use caching o For Item_Model::xxx_edit fields, don't make them editable for guests o Other minor stuff. These optimizations reduce the number of queries for a 9-photos page from ~200 to ~45. Still way too many! --- modules/user/helpers/user.php | 2 +- modules/user/helpers/user_block.php | 6 ++++-- 2 files changed, 5 insertions(+), 3 deletions(-) (limited to 'modules/user') diff --git a/modules/user/helpers/user.php b/modules/user/helpers/user.php index e7a3e165..a04542d3 100644 --- a/modules/user/helpers/user.php +++ b/modules/user/helpers/user.php @@ -73,7 +73,7 @@ class user_Core { * @return User_Model */ static function guest() { - return ORM::factory("user", 1); + return model_cache::get("user", 1); } /** diff --git a/modules/user/helpers/user_block.php b/modules/user/helpers/user_block.php index b737cec6..51d4908d 100644 --- a/modules/user/helpers/user_block.php +++ b/modules/user/helpers/user_block.php @@ -22,8 +22,10 @@ class user_block_Core { $url = url::file("modules/user/js/user.js"); $script[] = ""; $user = user::active(); - $url = url::file("lib/jquery.jeditable.js"); - $script[] = empty($user) ? "" : ""; + if (!user::active()->guest) { + $url = url::file("lib/jquery.jeditable.js"); + $script[] = ""; + } return implode("\n", $script); } -- cgit v1.2.3