X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=lib%2Finboxtagcloudsection.php;h=4268ee48548722093c6f67a2a799b9e891b97cdd;hb=b4271a3533bdf12329f27dd75452c1ef2a6ee3d1;hp=401983a9ded170c813af3a6e26016d6f4b69ae0a;hpb=7d64d8c78cfa102b91975598ef9e574d2ef14b8c;p=quix0rs-gnu-social.git

diff --git a/lib/inboxtagcloudsection.php b/lib/inboxtagcloudsection.php
index 401983a9de..4268ee4854 100644
--- a/lib/inboxtagcloudsection.php
+++ b/lib/inboxtagcloudsection.php
@@ -27,9 +27,7 @@
  * @link      http://status.net/
  */
 
-if (!defined('STATUSNET') && !defined('LACONICA')) {
-    exit(1);
-}
+if (!defined('GNUSOCIAL')) { exit(1); }
 
 /**
  * Personal tag cloud section
@@ -42,12 +40,14 @@ if (!defined('STATUSNET') && !defined('LACONICA')) {
  */
 class InboxTagCloudSection extends TagCloudSection
 {
-    var $user = null;
+    const MAX_NOTICES = 1024;   // legacy value for "Inbox" table size when that existed
+
+    protected $target = null;
 
-    function __construct($out=null, $user=null)
+    function __construct($out=null, Profile $target)
     {
         parent::__construct($out);
-        $this->user = $user;
+        $this->target = $target;
     }
 
     function title()
@@ -58,53 +58,44 @@ class InboxTagCloudSection extends TagCloudSection
 
     function getTags()
     {
-        $profile = Profile::current();
-
-        $keypart = sprintf('Inbox:notice_tag:%d:%d', $this->user->id, $profile->id);
-
-        $tag = Memcached_DataObject::cacheGet($keypart);
-
-        if ($tag === false) {
-
-            $stream = new InboxNoticeStream($this->user, $profile);
-
-            $ids = $stream->getNoticeIds(0, Inbox::MAX_NOTICES, null, null);
-
-            if (empty($ids)) {
-                $tag = array();
+        // FIXME: Get the Profile::current() value some other way
+        // to avoid confusion between background stuff and session.
+        $stream = new InboxNoticeStream($this->target, Profile::current());
+
+        $ids = $stream->getNoticeIds(0, self::MAX_NOTICES, null, null);
+
+        if (empty($ids)) {
+            $tag = array();
+        } else {
+            $weightexpr = common_sql_weight('notice_tag.created', common_config('tag', 'dropoff'));
+            // @fixme should we use the cutoff too? Doesn't help with indexing per-user.
+
+            $qry = 'SELECT notice_tag.tag, '.
+                $weightexpr . ' as weight ' .
+                'FROM notice_tag JOIN notice ' .
+                'ON notice_tag.notice_id = notice.id ' .
+                'WHERE notice.id in (' . implode(',', $ids) . ')'.
+                'GROUP BY notice_tag.tag ' .
+                'ORDER BY weight DESC ';
+
+            $limit = TAGS_PER_SECTION;
+            $offset = 0;
+
+            if (common_config('db','type') == 'pgsql') {
+                $qry .= ' LIMIT ' . $limit . ' OFFSET ' . $offset;
             } else {
-                $weightexpr = common_sql_weight('notice_tag.created', common_config('tag', 'dropoff'));
-                // @fixme should we use the cutoff too? Doesn't help with indexing per-user.
-
-                $qry = 'SELECT notice_tag.tag, '.
-                    $weightexpr . ' as weight ' .
-                    'FROM notice_tag JOIN notice ' .
-                    'ON notice_tag.notice_id = notice.id ' .
-                    'WHERE notice.id in (' . implode(',', $ids) . ')'.
-                    'GROUP BY notice_tag.tag ' .
-                    'ORDER BY weight DESC ';
-
-                $limit = TAGS_PER_SECTION;
-                $offset = 0;
-
-                if (common_config('db','type') == 'pgsql') {
-                    $qry .= ' LIMIT ' . $limit . ' OFFSET ' . $offset;
-                } else {
-                    $qry .= ' LIMIT ' . $offset . ', ' . $limit;
-                }
+                $qry .= ' LIMIT ' . $offset . ', ' . $limit;
+            }
 
-                $t = new Notice_tag();
+            $t = new Notice_tag();
 
-                $t->query($qry);
+            $t->query($qry);
 
-                $tag = array();
+            $tag = array();
 
-                while ($t->fetch()) {
-                    $tag[] = clone($t);
-                }
+            while ($t->fetch()) {
+                $tag[] = clone($t);
             }
-
-            Memcached_DataObject::cacheSet($keypart, $tag, 3600);
         }
 
         return new ArrayWrapper($tag);