diff --git a/classes/Fave.php b/classes/Fave.php index 5c42d8b483..2823a3833d 100644 --- a/classes/Fave.php +++ b/classes/Fave.php @@ -30,4 +30,8 @@ class Fave extends Memcached_DataObject } return $fave; } + + function &pkeyGet($kv) { + return Memcached_DataObject('Fave', $kv); + } } diff --git a/classes/Subscription.php b/classes/Subscription.php index deb01eb47c..ace2fa4c42 100644 --- a/classes/Subscription.php +++ b/classes/Subscription.php @@ -42,4 +42,8 @@ class Subscription extends Memcached_DataObject /* the code above is auto generated do not remove the tag below */ ###END_AUTOCODE + + function &pkeyGet($kv) { + return Memcached_DataObject::pkeyGet('Subscription', $kv); + } } diff --git a/classes/User.php b/classes/User.php index a8862ddad2..8c9ffbb6d6 100644 --- a/classes/User.php +++ b/classes/User.php @@ -65,21 +65,15 @@ class User extends Memcached_DataObject ###END_AUTOCODE function getProfile() { - $profile = DB_DataObject::factory('profile'); - $profile->id = $this->id; - if ($profile->find()) { - $profile->fetch(); - return $profile; - } - return NULL; + return Profile::staticGet('id', $this->id); } function isSubscribed($other) { assert(!is_null($other)); - $sub = DB_DataObject::factory('subscription'); - $sub->subscriber = $this->id; - $sub->subscribed = $other->id; - return $sub->find(); + # XXX: cache results of this query + $sub = Subscription::pkeyGet(array('subscriber' => $this->id, + 'subscribed' => $other->id)); + return (is_null($sub)) ? false : true; } # 'update' won't write key columns, so we have to do it ourselves. @@ -130,7 +124,7 @@ class User extends Memcached_DataObject } function getCarrier() { - return Sms_carrier::staticGet($this->carrier); + return Sms_carrier::staticGet('id', $this->carrier); } function subscribeTo($other) { @@ -271,10 +265,9 @@ class User extends Memcached_DataObject } function hasFave($notice) { - $fave = new Fave(); - $fave->user_id = $this->id; - $fave->notice_id = $notice->id; - if ($fave->find()) { + $fave = Fave::pkeyGet(array('user_id' => $this->id, + 'notice_id' => $notice->id)); + if (!is_null($fave)) { $result = true; } else { $result = false;