diff options
Diffstat (limited to 'modules/rss')
| -rw-r--r-- | modules/rss/controllers/rss.php | 38 | ||||
| -rw-r--r-- | modules/rss/helpers/rss.php | 34 | ||||
| -rw-r--r-- | modules/rss/helpers/rss_theme.php | 19 | ||||
| -rw-r--r-- | modules/rss/views/rss_block.html.php | 4 |
4 files changed, 45 insertions, 50 deletions
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 "<link rel=\"alternate\" type=\"" . rest::RSS . "\" href=\"$url\" />"; - } - /** - * 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 <link> element for a given rss feed id. + */ + static function feed_link($uri) { + $url = url::site("rss/feed/$uri"); + return "<link rel=\"alternate\" type=\"" . rest::RSS . "\" href=\"$url\" />"; } }
\ 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 @@ <?php defined("SYSPATH") or die("No direct script access.") ?> <ul id="gFeeds"> -<? foreach($feeds as $title => $url): ?> +<? foreach($feeds as $url => $title): ?> <li style="clear: both;"> <span class="ui-icon-left"> - <a href="<?= $url ?>"> + <a href="<?= rss::url($url) ?>"> <span class="ui-icon ui-icon-signal-diag"></span> <?= $title ?> </a> |