Twitter-compatible API - rss and atom feeds for direct messages

darcs-hash:20080926012015-7b5ce-da94465ff9c027183e0e674b6152902c302d0131.gz

2 files changed, 87 insertions, 28 deletions

diff --git a/actions/twitapidirect_messages.php b/actions/twitapidirect_messages.php
index 39780258a..8eee6a753 100644
--- a/actions/twitapidirect_messages.php
+++ b/actions/twitapidirect_messages.php
@@ -63,18 +63,24 @@ class Twitapidirect_messagesAction extends TwitterapiAction {
 
 		$message->find();
 
+		$title = 'Direct messages to ' . $user->nickname;
+		$subtitle = 'All the direct messages sent to ' . $user->nickname;
+
+		$server = common_root_url();
+		$link = $server . $user->nickname . '/inbox';
+
 		switch($apidata['content-type']) {
 		 case 'xml':
-			$this->show_xml_direct_messages($message);
+			$this->show_xml_dmsgs($message);
 			break;
 		 case 'rss':
-			//$this->show_rss_timeline($notice, $title, $id, $link, $subtitle);
+			$this->show_rss_dmsgs($message, $title, $link, $subtitle);
 			break;
 		 case 'atom':
-			//$this->show_atom_timeline($notice, $title, $id, $link, $subtitle);
+			$this->show_atom_dmsgs($message, $title, $link, $subtitle);
 			break;
 		 case 'json':
-			$this->show_json_direct_messages($message);
+			$this->show_json_dmsgs($message);
 			break;
 		 default:
 			common_user_error(_('API method not found!'), $code = 404);
@@ -102,7 +108,7 @@ class Twitapidirect_messagesAction extends TwitterapiAction {
 		exit();
 	}
 
-	function show_xml_direct_messages($message) {
+	function show_xml_dmsgs($message) {
 
 		$this->init_document('xml');
 		common_element_start('direct-messages', array('type' => 'array'));
@@ -123,7 +129,7 @@ class Twitapidirect_messagesAction extends TwitterapiAction {
 		$this->end_document('xml');
 	}
 
-	function show_json_direct_messages($message) {
+	function show_json_dmsgs($message) {
 
 		$this->init_document('json');
 
@@ -146,6 +152,59 @@ class Twitapidirect_messagesAction extends TwitterapiAction {
 		$this->end_document('json');
 	}
 
+	function show_rss_dmsgs($message, $title, $link, $subtitle) {
+
+		$this->init_document('rss');
+
+		common_element_start('channel');
+		common_element('title', NULL, $title);
+
+		common_element('link', NULL, $link);
+		common_element('description', NULL, $subtitle);
+		common_element('language', NULL, 'en-us');
+		common_element('ttl', NULL, '40');
+
+		if (is_array($message)) {
+			foreach ($message as $m) {
+				$entry = $this->twitter_rss_dmsg_array($m);
+				$this->show_twitter_rss_item($entry);
+			}
+		} else {
+			while ($message->fetch()) {
+				$entry = $this->twitter_rss_dmsg_array($message);
+				$this->show_twitter_rss_item($entry);
+			}
+		}
+
+		common_element_end('channel');
+		$this->end_twitter_rss();
+	}
+
+	function show_atom_dmsgs($message, $title, $link, $subtitle) {
+
+		$this->init_document('atom');
+
+		common_element('title', NULL, $title);
+		$siteserver = common_config('site', 'server');
+		common_element('id', NULL, "tag:$siteserver,2008:DirectMessage");
+		common_element('link', array('href' => $link, 'rel' => 'alternate', 'type' => 'text/html'), NULL);
+		common_element('updated', NULL, common_date_iso8601(strftime('%c')));
+		common_element('subtitle', NULL, $subtitle);
+
+		if (is_array($message)) {
+			foreach ($message as $m) {
+				$entry = $this->twitter_rss_dmsg_array($m);
+				$this->show_twitter_atom_entry($entry);
+			}
+		} else {
+			while ($message->fetch()) {
+				$entry = $this->twitter_rss_dmsg_array($message);
+				$this->show_twitter_atom_entry($entry);
+			}
+		}
+
+		$this->end_document('atom');
 
+	}
 
 }
\ No newline at end of file
diff --git a/actions/twitapistatuses.php b/actions/twitapistatuses.php
index 723c19499..c2511b431 100644
--- a/actions/twitapistatuses.php
+++ b/actions/twitapistatuses.php
@@ -66,7 +66,7 @@ class TwitapistatusesAction extends TwitterapiAction {
 					$this->show_xml_timeline($notice);
 					break;
 				case 'rss':
-					$this->show_rss_timeline($notice, $title, $id, $link, $subtitle);
+					$this->show_rss_timeline($notice, $title, $link, $subtitle);
 					break;
 				case 'atom':
 					$this->show_atom_timeline($notice, $title, $id, $link, $subtitle);
@@ -107,7 +107,7 @@ class TwitapistatusesAction extends TwitterapiAction {
 		$this->end_document('xml');
 	}
 
-	function show_rss_timeline($notice, $title, $id, $link, $subtitle) {
+	function show_rss_timeline($notice, $title, $link, $subtitle) {
 
 		$this->init_document('rss');
 
@@ -395,10 +395,10 @@ class TwitapistatusesAction extends TwitterapiAction {
 		$reply_to = NULL;
 
 		if ($in_reply_to_status_id) {
-						
+
 			// check whether notice actually exists
 			$reply = Notice::staticGet($in_reply_to_status_id);
-			
+
 			if ($reply) {
 				$reply_to = $in_reply_to_status_id;
 			} else {
@@ -406,7 +406,7 @@ class TwitapistatusesAction extends TwitterapiAction {
 				exit();
 			}
 		}
-			
+
 		$notice = Notice::saveNew($user->id, $status, $source, 1, $reply_to);
 
 		if (is_string($notice)) {
@@ -504,12 +504,12 @@ class TwitapistatusesAction extends TwitterapiAction {
 
 	function show($args, $apidata) {
 		parent::handle($args);
-		
-		$notice_id = $apidata['api_arg'];		
+
+		$notice_id = $apidata['api_arg'];
 		$notice = Notice::staticGet($notice_id);
 
 		if ($notice) {
-			if ($apidata['content-type'] == 'xml') { 
+			if ($apidata['content-type'] == 'xml') {
 				$this->show_single_xml_status($notice);
 			} elseif ($apidata['content-type'] == 'json') {
 				$this->show_single_json_status($notice);
@@ -518,7 +518,7 @@ class TwitapistatusesAction extends TwitterapiAction {
 			// XXX: Twitter just sets a 404 header and doens't bother to return an err msg
 			$this->client_error(_('No status with that ID found.'), 404, $apidata['content-type']);
 		}
-		
+
 		exit();
 	}
 
@@ -539,43 +539,43 @@ class TwitapistatusesAction extends TwitterapiAction {
 
 	*/
 	function destroy($args, $apidata) {
-	
+
 		parent::handle($args);
 
 		common_debug($_SERVER['REQUEST_METHOD']);
-		
-		// Check for RESTfulness  
+
+		// Check for RESTfulness
 		if (!in_array($_SERVER['REQUEST_METHOD'], array('POST', 'DELETE'))) {
 			// XXX: Twitter just prints the err msg, no XML / JSON.
 			$this->client_error(_('This method requires a POST or DELETE.'), 400, $apidata['content-type']);
 			exit();
-		} 
-		
-		$user = $apidata['user'];				
-		$notice_id = $apidata['api_arg'];		
+		}
+
+		$user = $apidata['user'];
+		$notice_id = $apidata['api_arg'];
 		$notice = Notice::staticGet($notice_id);
-		
+
 		if (!$notice) {
 			$this->client_error(_('No status found with that ID.'), 404, $apidata['content-type']);
 			exit();
 		}
-				
+
 		if ($user->id == $notice->profile_id) {
 			$replies = new Reply;
 			$replies->get('notice_id', $notice_id);
 			common_dequeue_notice($notice);
 			$replies->delete();
 			$notice->delete();
-			
-			if ($apidata['content-type'] == 'xml') { 
+
+			if ($apidata['content-type'] == 'xml') {
 				$this->show_single_xml_status($notice);
 			} elseif ($apidata['content-type'] == 'json') {
 				$this->show_single_json_status($notice);
-			}	
+			}
 		} else {
 			$this->client_error(_('You may not delete another user\'s status.'), 403, $apidata['content-type']);
 		}
-		
+
 		exit();
 	}