From 6e856cf7ee2dbfac56cbd669fa1022048c5fb50f Mon Sep 17 00:00:00 2001 From: Evan Prodromou Date: Sun, 7 Dec 2008 11:59:00 -0500 Subject: [PATCH] remove favoritedrss I've removed the favoritedrss action, and removed the links from the favorited page. Most-favorited is not a good candidate for an RSS format. RSS works well for series that work sequentially -- new things always come in at the top. This just isn't the case with "most favorited". The favorited notices list on Identi.ca is sorted by how many times a message is favorited. What happens if a notice used to be on that list, then falls off because other messages get more faves, then gets back on because it gets faved again? Or if a notice moves "up" on the list? darcs-hash:20081207165900-84dde-23afe73c9c61cfb259ec5dc0cac5030643b31f2c.gz --- actions/favorited.php | 10 +---- actions/favoritedrss.php | 83 ---------------------------------------- 2 files changed, 1 insertion(+), 92 deletions(-) delete mode 100644 actions/favoritedrss.php diff --git a/actions/favorited.php b/actions/favorited.php index 035e2f9535..16bd8f2262 100644 --- a/actions/favorited.php +++ b/actions/favorited.php @@ -44,18 +44,10 @@ class FavoritedAction extends StreamAction { common_raw($output); common_element_end('div'); $this->public_views_menu(); - - $this->show_feeds_list(array(0=>array('href'=>common_local_url('favoritedrss'), - 'type' => 'rss', - 'version' => 'RSS 1.0', - 'item' => 'favoritedrss'))); } function show_header() { - common_element('link', array('rel' => 'alternate', - 'href' => common_local_url('favoritedrss'), - 'type' => 'application/rss+xml', - 'title' => _('Favorited Stream Feed'))); + return; } function get_instructions() { diff --git a/actions/favoritedrss.php b/actions/favoritedrss.php deleted file mode 100644 index 4661f48216..0000000000 --- a/actions/favoritedrss.php +++ /dev/null @@ -1,83 +0,0 @@ -. - */ - -if (!defined('LACONICA')) { exit(1); } - -require_once(INSTALLDIR.'/lib/rssaction.php'); - -// Formatting of RSS handled by Rss10Action - -class FavoritedrssAction extends Rss10Action { - - function init() { - return true; - } - - function get_notices($limit=0) { - - $qry = - 'SELECT notice_id, sum(exp(-(now() - modified)/864000)) as weight ' . - 'FROM fave GROUP BY notice_id ' . - 'ORDER BY weight DESC'; - - $offset = 0; - $total = ($limit == 0) ? 48 : $limit; - - if (common_config('db','type') == 'pgsql') { - $qry .= ' LIMIT ' . $total . ' OFFSET ' . $offset; - } else { - $qry .= ' LIMIT ' . $offset . ', ' . $limit; - } - - $fave = new Fave; - $fave->query($qry); - - $notice_list = array(); - - while ($fave->fetch()) { - array_push($notice_list, $fave->notice_id); - } - - $notice = new Notice(); - - $notice->query(sprintf('SELECT * FROM notice WHERE id in (%s)', - implode($notice_list, ','))); - - $notices = array(); - - while ($notice->fetch()) { - $notices[] = clone($notice); - } - - return $notices; - } - - function get_channel() { - global $config; - $c = array('url' => common_local_url('favoritedrss'), - 'title' => sprintf(_('%s Most Favorited Stream'), $config['site']['name']), - 'link' => common_local_url('favorited'), - 'description' => sprintf(_('Most favorited updates for %s'), $config['site']['name'])); - return $c; - } - - function get_image() { - return NULL; - } -} \ No newline at end of file