summaryrefslogtreecommitdiff
path: root/plugins/Sitemap
diff options
context:
space:
mode:
authorEvan Prodromou <evan@status.net>2010-04-12 12:00:15 -0400
committerEvan Prodromou <evan@status.net>2010-06-01 13:52:27 -0700
commit3e8172585d7146cacb44ee4543ea619f6a196561 (patch)
tree8ae72e31460bbf34a5ad66e8e3c648645df54699 /plugins/Sitemap
parent946cd15e8bc12bece4a14a07c6e109c3d8904a66 (diff)
cache user data for user sitemap
Diffstat (limited to 'plugins/Sitemap')
-rw-r--r--plugins/Sitemap/usersitemap.php75
1 files changed, 53 insertions, 22 deletions
diff --git a/plugins/Sitemap/usersitemap.php b/plugins/Sitemap/usersitemap.php
index 42cadaca7..3e5ac4652 100644
--- a/plugins/Sitemap/usersitemap.php
+++ b/plugins/Sitemap/usersitemap.php
@@ -43,7 +43,8 @@ if (!defined('STATUSNET')) {
class UsersitemapAction extends SitemapAction
{
- var $user = null;
+ var $users = null;
+ var $j = 0;
function prepare($args)
{
@@ -61,37 +62,67 @@ class UsersitemapAction extends SitemapAction
$d += 0;
$i += 0;
- $offset = ($i-1) * SitemapPlugin::USERS_PER_MAP;
- $limit = SitemapPlugin::USERS_PER_MAP;
+ $this->users = $this->getUsers($y, $m, $d, $i);
+ $this->j = 0;
+ return true;
+ }
- $this->user = new User();
+ function nextUrl()
+ {
+ if ($this->j < count($this->users)) {
+ $nickname = $this->users[$this->j];
+ $this->j++;
+ return array(common_profile_url($nickname), null, null, null);
+ } else {
+ return null;
+ }
+ }
- $begindt = sprintf('%04d-%02d-%02d 00:00:00', $y, $m, $d);
+ function getUsers($y, $m, $d, $i)
+ {
+ $u = User::cacheGet("sitemap:user:$y:$m:$d:$i");
- // XXX: estimates 1d == 24h, which screws up days
- // with leap seconds (1d == 24h + 1s). Thankfully they're
- // few and far between.
+ if ($u === false) {
- $enddt = common_sql_date(strtotime($begindt) + (24 * 60 * 60));
+ $user = new User();
- $this->user->whereAdd("created >= '$begindt'");
- $this->user->whereAdd("created < '$enddt'");
+ $begindt = sprintf('%04d-%02d-%02d 00:00:00', $y, $m, $d);
- $this->user->orderBy('created');
+ // XXX: estimates 1d == 24h, which screws up days
+ // with leap seconds (1d == 24h + 1s). Thankfully they're
+ // few and far between.
- $this->user->limit($offset, $limit);
+ $theend = strtotime($begindt) + (24 * 60 * 60);
+ $enddt = common_sql_date($theend);
- $this->user->find();
+ $user->selectAdd();
+ $user->selectAdd('nickname');
+ $user->whereAdd("created >= '$begindt'");
+ $user->whereAdd("created < '$enddt'");
- return true;
- }
+ $user->orderBy('created');
- function nextUrl()
- {
- if ($this->user->fetch()) {
- return array(common_profile_url($this->user->nickname), null, null, null);
- } else {
- return null;
+ $offset = ($i-1) * SitemapPlugin::USERS_PER_MAP;
+ $limit = SitemapPlugin::USERS_PER_MAP;
+
+ $user->limit($offset, $limit);
+
+ $user->find();
+
+ while ($user->fetch()) {
+ $u[] = $user->nickname;
+ }
+
+ $c = Cache::instance();
+
+ if (!empty($c)) {
+ $c->set(Cache::key("sitemap:user:$y:$m:$d:$i"),
+ $u,
+ Cache::COMPRESSED,
+ ((time() > $theend) ? (time() + 90 * 24 * 60 * 60) : (time() + 5 * 60)));
+ }
}
+
+ return $u;
}
}