From fcc8efa279aab75419022bd8fc1b3134023c7806 Mon Sep 17 00:00:00 2001
From: nickw <nickw@36083f99-b078-4883-b0ff-0f9b5a30f544>
Date: Mon, 17 May 2010 20:12:55 +0000
Subject: Updating RSS feeds to use proper views called with page_shells.

git-svn-id: http://code.elgg.org/elgg/trunk@6073 36083f99-b078-4883-b0ff-0f9b5a30f544
---
 views/rss/object/default.php       | 61 +++++++++++++++++++++-----------------
 views/rss/page_shells/default.php  | 44 +++++++++++++++++++++++++++
 views/rss/pageshells/pageshell.php | 44 ---------------------------
 views/rss/river/item/list.php      | 50 ++++++++++++++-----------------
 4 files changed, 99 insertions(+), 100 deletions(-)
 create mode 100644 views/rss/page_shells/default.php
 delete mode 100644 views/rss/pageshells/pageshell.php

(limited to 'views/rss')

diff --git a/views/rss/object/default.php b/views/rss/object/default.php
index 5d11599dc..7b1ab10bc 100644
--- a/views/rss/object/default.php
+++ b/views/rss/object/default.php
@@ -11,38 +11,43 @@
 $title = $vars['entity']->title;
 if (empty($title)) {
 	$subtitle = strip_tags($vars['entity']->description);
-	$title = substr($subtitle,0,32);
+	$title = substr($subtitle, 0, 32);
 	if (strlen($subtitle) > 32) {
-		$title .= " ...";
+		$title .= ' ...';
 	}
 }
 
-?>
+$permalink = htmlspecialchars($vars['entity']->getURL());
+$pubdate = date('r', $vars['entity']->time_created);
 
+$creator = '';
+if ($owner = $vars['entity']->getOwnerEntity()) {
+	$creator = "<dc:creator>{$owner->name}</dc:creator>";
+}
+
+$georss = '';
+if (
+	($vars['entity'] instanceof Locatable) &&
+	($vars['entity']->getLongitude()) &&
+	($vars['entity']->getLatitude())
+) {
+	$latitude = $vars['entity']->getLatitude();
+	$longitude = $vars['entity']->getLongitude();
+	$georss = "<georss:point>$latitude $longitude</georss:point>";
+}
+
+$extension = elgg_view('extensions/item');
+
+$item = <<<__HTML
 <item>
-<guid isPermaLink='true'><?php echo htmlspecialchars($vars['entity']->getURL()); ?></guid>
-<pubDate><?php echo date("r",$vars['entity']->time_created) ?></pubDate>
-<link><?php echo htmlspecialchars($vars['entity']->getURL()); ?></link>
-<title><![CDATA[<?php echo $title; ?>]]></title>
-<description><![CDATA[<?php echo (autop($vars['entity']->description)); ?>]]></description>
-<?php
-		$owner = $vars['entity']->getOwnerEntity();
-		if ($owner) {
-?>
-<dc:creator><?php echo $owner->name; ?></dc:creator>
-<?php
-		}
-?>
-<?php
-		if (
-			($vars['entity'] instanceof Locatable) &&
-			($vars['entity']->getLongitude()) &&
-			($vars['entity']->getLatitude())
-		) {
-			?>
-			<georss:point><?php echo $vars['entity']->getLatitude(); ?> <?php echo $vars['entity']->getLongitude(); ?></georss:point>
-			<?php
-		}
-?>
-<?php echo elgg_view('extensions/item'); ?>
+	<guid isPermaLink="true">$permalink</guid>
+	<pubDate>$pubdate</pubDate>
+	<link>$permalink</link>
+	<title><![CDATA[$title]]></title>
+	<description><![CDATA[{$vars['entity']->description}]]></description>
+	$creator$georss$extension
 </item>
+
+__HTML;
+
+echo $item;
diff --git a/views/rss/page_shells/default.php b/views/rss/page_shells/default.php
new file mode 100644
index 000000000..c62ac4aac
--- /dev/null
+++ b/views/rss/page_shells/default.php
@@ -0,0 +1,44 @@
+<?php
+/**
+ * Elgg RSS output pageshell
+ *
+ * @package Elgg
+ * @subpackage Core
+ * @link http://elgg.org/
+ *
+ */
+
+header("Content-Type: text/xml");
+
+// allow caching as required by stupid MS products for https feeds.
+header('Pragma: public', TRUE);
+
+echo "<?xml version='1.0'?>\n";
+
+// Set title
+if (empty($vars['title'])) {
+	$title = $vars['config']->sitename;
+} else if (empty($vars['config']->sitename)) {
+	$title = $vars['title'];
+} else {
+	$title = $vars['config']->sitename . ": " . $vars['title'];
+}
+
+// Remove RSS from URL
+$url = str_replace('?view=rss','', full_url());
+$url = str_replace('&view=rss','', $url);
+
+?>
+
+<rss version="2.0" xmlns:dc="http://purl.org/dc/elements/1.1/" xmlns:georss="http://www.georss.org/georss" <?php echo elgg_view('extensions/xmlns'); ?> >
+<channel>
+	<title><![CDATA[<?php echo $title; ?>]]></title>
+	<link><?php echo htmlentities($url); ?></link>
+	<?php echo elgg_view('extensions/channel'); ?>
+	<?php
+
+		echo $vars['body'];
+
+	?>
+</channel>
+</rss>
diff --git a/views/rss/pageshells/pageshell.php b/views/rss/pageshells/pageshell.php
deleted file mode 100644
index c62ac4aac..000000000
--- a/views/rss/pageshells/pageshell.php
+++ /dev/null
@@ -1,44 +0,0 @@
-<?php
-/**
- * Elgg RSS output pageshell
- *
- * @package Elgg
- * @subpackage Core
- * @link http://elgg.org/
- *
- */
-
-header("Content-Type: text/xml");
-
-// allow caching as required by stupid MS products for https feeds.
-header('Pragma: public', TRUE);
-
-echo "<?xml version='1.0'?>\n";
-
-// Set title
-if (empty($vars['title'])) {
-	$title = $vars['config']->sitename;
-} else if (empty($vars['config']->sitename)) {
-	$title = $vars['title'];
-} else {
-	$title = $vars['config']->sitename . ": " . $vars['title'];
-}
-
-// Remove RSS from URL
-$url = str_replace('?view=rss','', full_url());
-$url = str_replace('&view=rss','', $url);
-
-?>
-
-<rss version="2.0" xmlns:dc="http://purl.org/dc/elements/1.1/" xmlns:georss="http://www.georss.org/georss" <?php echo elgg_view('extensions/xmlns'); ?> >
-<channel>
-	<title><![CDATA[<?php echo $title; ?>]]></title>
-	<link><?php echo htmlentities($url); ?></link>
-	<?php echo elgg_view('extensions/channel'); ?>
-	<?php
-
-		echo $vars['body'];
-
-	?>
-</channel>
-</rss>
diff --git a/views/rss/river/item/list.php b/views/rss/river/item/list.php
index 5ac43c544..8047c24bf 100644
--- a/views/rss/river/item/list.php
+++ b/views/rss/river/item/list.php
@@ -6,39 +6,33 @@
  * @link http://elgg.org/
  */
 
-if (isset($vars['items']) && is_array($vars['items'])) {
-
-	$i = 0;
-	if (!empty($vars['items'])) {
-		foreach($vars['items'] as $item) {
-
-			// echo elgg_view_river_item($item);
-			if (elgg_view_exists($item->view,'default')) {
-				$body = elgg_view($item->view,array('item' => $item),false,false,'default');
-				$time = date("r",$item->posted);
-				if ($entity = get_entity($item->object_guid)) {
-					$url = htmlspecialchars($entity->getURL());
-				} else {
-					$url = $vars['url'];
-				}
-				$title = strip_tags($body);
-
-	?>
-	<item>
-		<guid isPermaLink='true'><?php echo $url; ?></guid>
-		<pubDate><?php echo $time; ?></pubDate>
-		<link><?php echo $url; ?></link>
-		<title><![CDATA[<?php echo $title; ?>]]></title>
-		<description><![CDATA[<?php echo (autop($body)); ?>]]></description>
-	</item>
-	<?php
-
+if (isset($vars['items']) && is_array($vars['items']) && !empty($vars['items'])) {
+	foreach($vars['items'] as $item) {
+		if (elgg_view_exists($item->view)) {
+			$body = elgg_view($item->view, array('item' => $item));
+			$time = date('r', $item->posted);
+			if ($entity = get_entity($item->object_guid)) {
+				$url = htmlspecialchars($entity->getURL());
+			} else {
+				$url = $vars['url'];
 			}
+			$title = strip_tags($body);
+			
+			echo <<<__HTML
+<item>
+	<guid isPermaLink="true">$url</guid>
+	<pubDate>$time</pubDate>
+	<title><![CDATA[$title]]></title>
+	<link>$url</link>
+	<description><![CDATA[$body]]></description>
+</item>
 
+__HTML;
+			
 			$i++;
 			if ($i >= $vars['limit']) {
 				break;
 			}
 		}
 	}
-}
\ No newline at end of file
+}
-- 
cgit v1.2.3