From 32ea4406c87049b25d3d82351097e69f13168311 Mon Sep 17 00:00:00 2001 From: Bharat Mediratta Date: Sun, 14 Jun 2009 21:51:54 -0700 Subject: Refactor the way that the rss module works so that we're not allowing the url to dictate arbitrary static method calls. * Each xxx_rss helper has a single feed() call which takes an id as the argument * xxx_rss::available_feedS() only returns feeds when they're applicable (ie if you're viewing a tag, it won't show you an item feed). * Feed urls are now in the module/feed_id form so that we can bind a feed id to a given module * Tightened up the Rss_Controller by using url::merge and some other tricks. * Made the slideshow module express its own feed. --- modules/rss/controllers/rss.php | 38 +++++++++++++++++++++--------------- modules/rss/helpers/rss.php | 34 +++++++++----------------------- modules/rss/helpers/rss_theme.php | 19 +++++++++++------- modules/rss/views/rss_block.html.php | 4 ++-- 4 files changed, 45 insertions(+), 50 deletions(-) (limited to 'modules/rss') diff --git a/modules/rss/controllers/rss.php b/modules/rss/controllers/rss.php index 29300d58..e9dd9fff 100644 --- a/modules/rss/controllers/rss.php +++ b/modules/rss/controllers/rss.php @@ -20,37 +20,43 @@ class Rss_Controller extends Controller { public static $page_size = 20; - public function feed($method, $id=null) { + public function feed($module_id, $feed_id, $id=null) { $page = $this->input->get("page", 1); - $feed_uri = "rss/feed/$method" . (empty($id) ? "" : "/$id"); if ($page < 1) { - url::redirect($feed_uri); + url::redirect(url::merge(array("page" => 1))); } - $feed = rss::feed_data($method, ($page - 1) * self::$page_size, self::$page_size, $id); - $max_pages = $feed->max_pages; - if ($max_pages && $page > $max_pages) { - url::redirect("$feed_uri?page={$max_pages}"); + // Run the appropriate feed callback + if (module::is_active($module_id)) { + $class_name = "{$module_id}_rss"; + if (method_exists($class_name, "feed")) { + $feed = call_user_func( + array($class_name, "feed"), $feed_id, + ($page - 1) * self::$page_size, self::$page_size, $id); + } + } + if (empty($feed)) { + Kohana::show_404(); + } + + if ($feed->max_pages && $page > $feed->max_pages) { + url::redirect(url::merge(array("page" => $feed->max_pages))); } $view = new View(empty($feed->view) ? "feed.mrss" : $feed->view); unset($feed->view); - $feed->uri = url::abs_site($feed_uri); $view->feed = $feed; + $view->pub_date = date("D, d M Y H:i:s T"); + $feed->uri = url::abs_site(Router::$current_uri); if ($page > 1) { - $previous_page = $page - 1; - $feed->previous_page_uri = url::site("$feed_uri?page={$previous_page}"); + $feed->previous_page_uri = url::abs_site(url::merge(array("page" => $page - 1))); } - - if ($page < $max_pages) { - $next_page = $page + 1; - $feed->next_page_uri = url::site("$feed_uri?page={$next_page}"); + if ($page < $feed->max_pages) { + $feed->next_page_uri = url::abs_site(url::merge(array("page" => $page + 1))); } - $view->pub_date = date("D, d M Y H:i:s T"); - rest::http_content_type(rest::RSS); print $view; } diff --git a/modules/rss/helpers/rss.php b/modules/rss/helpers/rss.php index 8a7d0c97..81ff175f 100644 --- a/modules/rss/helpers/rss.php +++ b/modules/rss/helpers/rss.php @@ -19,34 +19,18 @@ */ class rss_Core { - static function feed_link($uri) { - $url = url::site("rss/feed/$uri"); - return ""; - } - /** - * Get all available rss feeds + * Convert a rss feed id into a rss feed url. */ - static function available_feeds($item) { - $feeds = array(); - foreach (module::active() as $module) { - $class_name = "{$module->name}_rss"; - if (method_exists($class_name, "available_feeds")) { - foreach (call_user_func(array($class_name, "available_feeds"), $item) as $feed) { - $feeds[$feed["description"]] = url::site("rss/feed/{$feed['uri']}"); - } - } - } - - return $feeds; + static function url($uri) { + return url::site("rss/feed/$uri"); } - static function feed_data($method, $offset, $limit, $id) { - foreach (module::active() as $module) { - $class_name = "{$module->name}_rss"; - if (method_exists($class_name, $feed)) { - return call_user_func(array($class_name, $feed), $offset, $limit, $id); - } - } + /** + * Return a element for a given rss feed id. + */ + static function feed_link($uri) { + $url = url::site("rss/feed/$uri"); + return ""; } } \ No newline at end of file diff --git a/modules/rss/helpers/rss_theme.php b/modules/rss/helpers/rss_theme.php index 52d988bf..3d1b9a29 100644 --- a/modules/rss/helpers/rss_theme.php +++ b/modules/rss/helpers/rss_theme.php @@ -19,17 +19,22 @@ */ class rss_theme_Core { static function sidebar_blocks($theme) { - // @todo this needs to be data driven - if (!$theme->item()) { - return; - } - $block = new Block(); $block->css_id = "gRss"; $block->title = t("Available RSS Feeds"); $block->content = new View("rss_block.html"); - $block->content->feeds = rss::available_feeds($theme->item()); + $block->content->feeds = array(); + foreach (module::active() as $module) { + $class_name = "{$module->name}_rss"; + if (method_exists($class_name, "available_feeds")) { + $block->content->feeds = array_merge( + $block->content->feeds, + call_user_func(array($class_name, "available_feeds"), $theme->item(), $theme->tag())); + } + } - return $block; + if ($block->content->feeds) { + return $block; + } } } diff --git a/modules/rss/views/rss_block.html.php b/modules/rss/views/rss_block.html.php index f964329c..39921d7d 100644 --- a/modules/rss/views/rss_block.html.php +++ b/modules/rss/views/rss_block.html.php @@ -1,9 +1,9 @@