diff options
author | Brion Vibber <brion@pobox.com> | 2010-06-07 10:19:40 -0700 |
---|---|---|
committer | Brion Vibber <brion@pobox.com> | 2010-06-07 10:19:40 -0700 |
commit | d88b208edcb75ec864e09bb3ab29785b35064400 (patch) | |
tree | 6a3b2eb35fb792bffb7ebaf3b4f81fb1131466d1 /classes/Inbox.php | |
parent | a7e33ac89df9f05b7497bfb34c6e69b3329a87e5 (diff) | |
parent | 41e9dba7297d43b7de0cb7665901869910d1047a (diff) |
Merge branch 'testing' of gitorious.org:statusnet/mainline into 0.9.x
Conflicts:
plugins/OpenID/openid.php
Diffstat (limited to 'classes/Inbox.php')
-rw-r--r-- | classes/Inbox.php | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/classes/Inbox.php b/classes/Inbox.php index 2533210b7..430419ba5 100644 --- a/classes/Inbox.php +++ b/classes/Inbox.php @@ -115,9 +115,12 @@ class Inbox extends Memcached_DataObject */ static function insertNotice($user_id, $notice_id) { - $inbox = DB_DataObject::staticGet('inbox', 'user_id', $user_id); - - if (empty($inbox)) { + // Going straight to the DB rather than trusting our caching + // during an update. Note: not using DB_DataObject::staticGet, + // which is unsafe to use directly (in-process caching causes + // memory leaks, which accumulate in queue processes). + $inbox = new Inbox(); + if (!$inbox->get('user_id', $user_id)) { $inbox = Inbox::initialize($user_id); } |