Twitter-compatible API - code cleanup

darcs-hash:20080716220223-ca946-e3eed117cded61eb9c2d2805fd07758f883fb85b.gz

2 files changed, 136 insertions, 137 deletions

diff --git a/actions/api.php b/actions/api.php
index badd4a42a..6a420c95e 100644
--- a/actions/api.php
+++ b/actions/api.php
@@ -38,12 +38,12 @@ class ApiAction extends Action {
 			$cmdext = explode('.', $argument);
 			$this->api_arg =  $cmdext[0];
 			$this->api_method = $method;
-			$this->content_type = $cmdext[1];
+			$this->content_type = strtolower($cmdext[1]);
 		} else {
 			#content type will be an extension on the method
 			$cmdext = explode('.', $method);
 			$this->api_method = $cmdext[0];
-			$this->content_type = $cmdext[1];
+			$this->content_type = strtolower($cmdext[1]);
 		}
 		
 		# common_debug("apiaction = $this->api_action, method = $this->api_method, argument = $this->api_arg, ctype = $this->content_type");
diff --git a/actions/twitapistatuses.php b/actions/twitapistatuses.php
index f9b804bd1..42c6313c8 100644
--- a/actions/twitapistatuses.php
+++ b/actions/twitapistatuses.php
@@ -28,148 +28,147 @@ require_once(INSTALLDIR.'/lib/twitterapi.php');
 class TwitapistatusesAction extends TwitterapiAction {
 	
 	/*
-	 *  Returns the 20 most recent statuses from non-protected users who have set a custom
-	 *  user icon. Does not require authentication.
+	 *  Returns the MAX_PUBSTATUSES most recent statuses from non-protected users who 
+	 *  have set a custom avatar. Does not require authentication.
 	 *	
-	 *	URL: http://identi.ca/api/statuses/public_timeline.format
+	 *	URL: http://server/api/statuses/public_timeline.format
      *
 	 *	Formats: xml, json, rss, atom
 	 */
 	function public_timeline($args, $apidata) {
 		parent::handle($args);
 
+		// Number of public statuses to return by default -- Twitter sends 20
+		$MAX_PUBSTATUSES = 20;
+
 		$notice = DB_DataObject::factory('notice');
 
+		// FIXME: To really live up to the spec we need to build a list
+		// of notices by users who have custom avatars, so fix this SQL -- Zach
+
 		# FIXME: bad performance
 		$notice->whereAdd('EXISTS (SELECT user.id from user where user.id = notice.profile_id)');
 		$notice->orderBy('created DESC, notice.id DESC');
-		$notice->limit(20);
+		$notice->limit($MAX_PUBSTATUSES);
 		$cnt = $notice->find();
-
-		if ($apidata['content-type'] == 'xml') {
-			
-			header('Content-Type: application/xml; charset=utf-8');		
-			
-			common_start_xml();
-
-			// XXX: To really live up to the spec we need to build a list
-			// of notices by users who have custom avatars -- Zach
-			if ($cnt > 0) {
-				common_element_start('statuses', array('type' => 'array'));
-				for ($i = 0; $i < 20; $i++) {
-					if ($notice->fetch()) {
-						$twitter_status = $this->twitter_status_array($notice);						
-						$this->render_twitter_xml_status($twitter_status);
-					} else {
-						// shouldn't happen!
-						break;
-					}
-				}
-				common_element_end('statuses');
-			}
 		
-			common_end_xml();
-		} elseif ($apidata['content-type'] == 'rss') {
+		if ($cnt > 0) {
 			
-			header("Content-Type: application/rss+xml; charset=utf-8");
-
-			$this->init_twitter_rss();
-			
-			$sitename = common_config('site', 'name');
-			$siteserver = common_config('site', 'server'); 
-			
-			common_element_start('channel');
-			common_element('title', NULL, "$sitename public timeline");
-			common_element('link', NULL, "http://$siteserver");
-			common_element('description', NULL, "$sitename updates from everyone!");
-			common_element('language', NULL, 'en-us');
-			common_element('ttl', NULL, '40');
-			
-			if ($cnt > 0) {
-				for ($i = 0; $i < 20; $i++) {
-					if ($notice->fetch()) {
-						$entry = $this->twitter_rss_entry_array($notice);						
-						$this->show_twitter_rss_item($entry);
-					} else {
-						// shouldn't happen!
-						break;
-					}
-				}
+			switch($apidata['content-type']) {
+				case 'xml': 
+					$this->show_xml_public_timeline($notice);
+					break;
+				case 'rss':
+					$this->show_rss_public_timeline($notice);
+					break;
+				case 'atom': 
+					$this->show_atom_public_timeline($notice);
+					break;
+				case 'json':
+					$this->show_json_public_timeline($notice);
+					break;
+				default:
+					common_user_error("API method not found!", $code = 404);
+					break;
 			}
-			common_element_end('channel');
-						
-			$this->end_twitter_rss();
-
-		} elseif ($apidata['content-type'] == 'atom') {
-
-			header('Content-Type: application/atom+xml; charset=utf-8');
-
-			$this->init_twitter_atom();
-			
-			$sitename = common_config('site', 'name');
-			$siteserver = common_config('site', 'server');
-
-			common_element('title', NULL, "$sitename public timeline");
-			common_element('id', NULL, "tag:$siteserver:Statuses");
-			common_element('link', array('href' => "http://$siteserver", 'rel' => 'alternate', 'type' => 'text/html'), NULL);
-			common_element('subtitle', NULL, "$sitename updates from everyone!");
 			
-			if ($cnt > 0) {
-				for ($i = 0; $i < 20; $i++) {
-					if ($notice->fetch()) {
-						$entry = $this->twitter_rss_entry_array($notice);						
-						$this->show_twitter_atom_entry($entry);
-					} else {
-						// shouldn't happen!
-						break;
-					}
-				}
-			}
+		} else {
+			common_server_error('Couldn\'t find any statuses.', $code = 503);
+		}
+ 
+		exit();
+	}	
+	
+	function show_xml_public_timeline($notice) {
+		
+		header('Content-Type: application/xml; charset=utf-8');		
+		common_start_xml();
+		common_element_start('statuses', array('type' => 'array'));
+		
+		while ($notice->fetch()) {
+			$twitter_status = $this->twitter_status_array($notice);						
+			$this->show_twitter_xml_status($twitter_status);
+		}
+		
+		common_element_end('statuses');
+		common_end_xml();
+	}
+	
+	function show_rss_public_timeline($notice) {
+		
+		header("Content-Type: application/rss+xml; charset=utf-8");
+		$this->init_twitter_rss();
+		$sitename = common_config('site', 'name');
+		$siteserver = common_config('site', 'server'); 
+		
+		common_element_start('channel');
+		common_element('title', NULL, "$sitename public timeline");
+		common_element('link', NULL, "http://$siteserver");
+		common_element('description', NULL, "$sitename updates from everyone!");
+		common_element('language', NULL, 'en-us');
+		common_element('ttl', NULL, '40');
+	
+		while ($notice->fetch()) {
+			$entry = $this->twitter_rss_entry_array($notice);						
+			$this->show_twitter_rss_item($entry);
+		}
+		
+		common_element_end('channel');			
+		$this->end_twitter_rss();
+	}
 
-			$this->end_twitter_atom();
+	function show_atom_public_timeline($notice) {
+		
+		header('Content-Type: application/atom+xml; charset=utf-8');
 
-		} elseif ($apidata['content-type'] == 'json') {
+		$this->init_twitter_atom();
+		$sitename = common_config('site', 'name');
+		$siteserver = common_config('site', 'server');
 
-			header('Content-Type: application/json; charset=utf-8');
+		common_element('title', NULL, "$sitename public timeline");
+		common_element('id', NULL, "tag:$siteserver:Statuses");
+		common_element('link', array('href' => "http://$siteserver", 'rel' => 'alternate', 'type' => 'text/html'), NULL);
+		common_element('subtitle', NULL, "$sitename updates from everyone!");
 
-			$statuses = array();
-			
-			if ($cnt > 0) {
-				for ($i = 0; $i < 20; $i++) {
-					if ($notice->fetch()) {
-						$twitter_status = $this->twitter_status_array($notice);
-						array_push($statuses, $twitter_status);						
-					} else {
-						// shouldn't happen!
-						break;
-					}
-				}				
-			}
-			$this->render_twitter_json_statuses($statuses);			
+		while ($notice->fetch()) {
+			$entry = $this->twitter_rss_entry_array($notice);						
+			$this->show_twitter_atom_entry($entry);
 		}
+		
+		$this->end_twitter_atom();
+	}
 
-		exit();
-	}	
-	
-
+	function show_json_public_timeline($notice) {
+		
+		header('Content-Type: application/json; charset=utf-8');
+		
+		$statuses = array();
+		
+		while ($notice->fetch()) {	
+			$twitter_status = $this->twitter_status_array($notice);
+			array_push($statuses, $twitter_status);						
+		}				
+		
+		$this->show_twitter_json_statuses($statuses);			
+	}
 		
 	/*
 	Returns the 20 most recent statuses posted by the authenticating user and that user's friends. 
 	This is the equivalent of /home on the Web. 
 	
-	URL: http://identi.ca/api/statuses/friends_timeline.format
+	URL: http://server/api/statuses/friends_timeline.format
 	
 	Parameters:
 
 	    * since.  Optional.  Narrows the returned results to just those statuses created after the specified 
 			HTTP-formatted date.  The same behavior is available by setting an If-Modified-Since header in 
 			your HTTP request.  
-			Ex: http://identi.ca/api/statuses/friends_timeline.rss?since=Tue%2C+27+Mar+2007+22%3A55%3A48+GMT
+			Ex: http://server/api/statuses/friends_timeline.rss?since=Tue%2C+27+Mar+2007+22%3A55%3A48+GMT
 	    * since_id.  Optional.  Returns only statuses with an ID greater than (that is, more recent than) 
-			the specified ID.  Ex: http://identi.ca/api/statuses/friends_timeline.xml?since_id=12345
+			the specified ID.  Ex: http://server/api/statuses/friends_timeline.xml?since_id=12345
 	    * count.  Optional.  Specifies the number of statuses to retrieve. May not be greater than 200.
-	  		Ex: http://identi.ca/api/statuses/friends_timeline.xml?count=5 
-	    * page. Optional. Ex: http://identi.ca/api/statuses/friends_timeline.rss?page=3
+	  		Ex: http://server/api/statuses/friends_timeline.xml?count=5 
+	    * page. Optional. Ex: http://server/api/statuses/friends_timeline.rss?page=3
 	
 	Formats: xml, json, rss, atom
 	*/
@@ -193,25 +192,25 @@ class TwitapistatusesAction extends TwitterapiAction {
         request another user's timeline via the id parameter below. This is the equivalent of the Web
         /archive page for your own user, or the profile page for a third party.
 
-		URL: http://identi.ca/api/statuses/user_timeline.format
+		URL: http://server/api/statuses/user_timeline.format
 
 		Formats: xml, json, rss, atom
 
 		Parameters:
 
 		    * id. Optional. Specifies the ID or screen name of the user for whom to return the
-            friends_timeline. Ex: http://identi.ca/api/statuses/user_timeline/12345.xml or
-            http://identi.ca/api/statuses/user_timeline/bob.json. 
+            friends_timeline. Ex: http://server/api/statuses/user_timeline/12345.xml or
+            http://server/api/statuses/user_timeline/bob.json. 
 			* count. Optional. Specifies the number of
             statuses to retrieve. May not be greater than 200. Ex:
-            http://identi.ca/api/statuses/user_timeline.xml?count=5 
+            http://server/api/statuses/user_timeline.xml?count=5 
 			* since. Optional. Narrows the returned
             results to just those statuses created after the specified HTTP-formatted date. The same
             behavior is available by setting an If-Modified-Since header in your HTTP request. Ex:
-            http://identi.ca/api/statuses/user_timeline.rss?since=Tue%2C+27+Mar+2007+22%3A55%3A48+GMT 
+            http://server/api/statuses/user_timeline.rss?since=Tue%2C+27+Mar+2007+22%3A55%3A48+GMT 
 			* since_id. Optional. Returns only statuses with an ID greater than (that is, more recent than)
-            the specified ID. Ex: http://identi.ca/api/statuses/user_timeline.xml?since_id=12345 * page.
-            Optional. Ex: http://identi.ca/api/statuses/friends_timeline.rss?page=3
+            the specified ID. Ex: http://server/api/statuses/user_timeline.xml?since_id=12345 * page.
+            Optional. Ex: http://server/api/statuses/friends_timeline.rss?page=3
 	*/
 	function user_timeline($args, $apidata) {
 		parent::handle($args);
@@ -230,14 +229,14 @@ class TwitapistatusesAction extends TwitterapiAction {
 	/*
 		Returns a single status, specified by the id parameter below. The status's author will be returned inline.
 		
-		 URL: http://identi.ca/api/statuses/show/id.format
+		 URL: http://server/api/statuses/show/id.format
 		
 		 Formats: xml, json
 		
 		 Parameters:
 		
 		 * id. Required. The numerical ID of the status you're trying to retrieve. 
-		 Ex: http://identi.ca/api/statuses/show/123.xml
+		 Ex: http://server/api/statuses/show/123.xml
 	*/
 	function show($args, $apidata) {
 		parent::handle($args);
@@ -254,7 +253,7 @@ class TwitapistatusesAction extends TwitterapiAction {
 	/*
 		Updates the authenticating user's status.  Requires the status parameter specified below.  Request must be a POST.
 
-		URL: http://identi.ca/api/statuses/update.format
+		URL: http://server/api/statuses/update.format
 
 		Formats: xml, json.  Returns the posted status in requested format when successful.
 
@@ -271,18 +270,18 @@ class TwitapistatusesAction extends TwitterapiAction {
 	
 	/*
 		Returns the 20 most recent @replies (status updates prefixed with @username) for the authenticating user.
-		URL: http://identi.ca/api/statuses/replies.format
+		URL: http://server/api/statuses/replies.format
  		
 		Formats: xml, json, rss, atom
 
  		Parameters:
 
- 		* page. Optional. Retrieves the 20 next most recent replies. Ex: http://identi.ca/api/statuses/replies.xml?page=3 
+ 		* page. Optional. Retrieves the 20 next most recent replies. Ex: http://server/api/statuses/replies.xml?page=3 
 		* since. Optional. Narrows the returned results to just those replies created after the specified HTTP-formatted date. The
         same behavior is available by setting an If-Modified-Since header in your HTTP request. Ex:
-        http://identi.ca/api/statuses/replies.xml?since=Tue%2C+27+Mar+2007+22%3A55%3A48+GMT
+        http://server/api/statuses/replies.xml?since=Tue%2C+27+Mar+2007+22%3A55%3A48+GMT
 		* since_id. Optional. Returns only statuses with an ID greater than (that is, more recent than) the specified
-		ID. Ex: http://identi.ca/api/statuses/replies.xml?since_id=12345
+		ID. Ex: http://server/api/statuses/replies.xml?since_id=12345
 	*/
 	function replies($args, $apidata) {
 		parent::handle($args);
@@ -294,15 +293,15 @@ class TwitapistatusesAction extends TwitterapiAction {
 		Destroys the status specified by the required ID parameter. The authenticating user must be
         the author of the specified status.
 		
-		 URL: http://identi.ca/api/statuses/destroy/id.format
+		 URL: http://server/api/statuses/destroy/id.format
 		
 		 Formats: xml, json
 		
 		 Parameters:
 		
 		 * id. Required. The ID of the status to destroy. Ex:
-        	http://identi.ca/api/statuses/destroy/12345.json or
-        	http://identi.ca/api/statuses/destroy/23456.xml
+        	http://server/api/statuses/destroy/12345.json or
+        	http://server/api/statuses/destroy/23456.xml
 	
 	*/
 	function destroy($args, $apidata) {
@@ -316,22 +315,22 @@ class TwitapistatusesAction extends TwitterapiAction {
 		Returns up to 100 of the authenticating user's friends who have most recently updated, each with current status inline.
         It's also possible to request another user's recent friends list via the id parameter below.
 		
-		 URL: http://identi.ca/api/statuses/friends.format
+		 URL: http://server/api/statuses/friends.format
 		
 		 Formats: xml, json
 		
 		 Parameters:
 		
 		 * id. Optional. The ID or screen name of the user for whom to request a list of friends. Ex:
-        	http://identi.ca/api/statuses/friends/12345.json 
+        	http://server/api/statuses/friends/12345.json 
 			or 
-			http://identi.ca/api/statuses/friends/bob.xml
-		 * page. Optional. Retrieves the next 100 friends. Ex: http://identi.ca/api/statuses/friends.xml?page=2
+			http://server/api/statuses/friends/bob.xml
+		 * page. Optional. Retrieves the next 100 friends. Ex: http://server/api/statuses/friends.xml?page=2
 		 * lite. Optional. Prevents the inline inclusion of current status. Must be set to a value of true. Ex:
-        	http://identi.ca/api/statuses/friends.xml?lite=true
+        	http://server/api/statuses/friends.xml?lite=true
 		 * since. Optional. Narrows the returned results to just those friendships created after the specified
   			HTTP-formatted date. The same behavior is available by setting an If-Modified-Since header in your HTTP
-  			request. Ex: http://identi.ca/api/statuses/friends.xml?since=Tue%2C+27+Mar+2007+22%3A55%3A48+GMT
+  			request. Ex: http://server/api/statuses/friends.xml?since=Tue%2C+27+Mar+2007+22%3A55%3A48+GMT
 	*/
 	function friends($args, $apidata) {
 		parent::handle($args);
@@ -342,18 +341,18 @@ class TwitapistatusesAction extends TwitterapiAction {
 		Returns the authenticating user's followers, each with current status inline. They are ordered by the
 		order in which they joined Twitter (this is going to be changed).
 		
-		URL: http://identi.ca/api/statuses/followers.format
+		URL: http://server/api/statuses/followers.format
 		Formats: xml, json
 
 		Parameters: 
 
 		    * id. Optional. The ID or screen name of the user for whom to request a list of followers. Ex:
-            	http://identi.ca/api/statuses/followers/12345.json 
+            	http://server/api/statuses/followers/12345.json 
 				or 
-				http://identi.ca/api/statuses/followers/bob.xml
-		    * page. Optional. Retrieves the next 100 followers. Ex: http://identi.ca/api/statuses/followers.xml?page=2   
+				http://server/api/statuses/followers/bob.xml
+		    * page. Optional. Retrieves the next 100 followers. Ex: http://server/api/statuses/followers.xml?page=2   
 		    * lite. Optional. Prevents the inline inclusion of current status. Must be set to a value of true.
-		 		Ex: http://identi.ca/api/statuses/followers.xml?lite=true
+		 		Ex: http://server/api/statuses/followers.xml?lite=true
 	*/
 	function followers($args, $apidata) {
 		parent::handle($args);
@@ -362,7 +361,7 @@ class TwitapistatusesAction extends TwitterapiAction {
 	
 	/*
 	Returns a list of the users currently featured on the site with their current statuses inline. 
-	URL: http://identi.ca/api/statuses/featured.format 
+	URL: http://server/api/statuses/featured.format 
 
 	Formats: xml, json
 	*/