From: Evan Prodromou <evan@controlyourself.ca>
Date: Sat, 21 Feb 2009 15:10:33 +0000 (-0800)
Subject: Merge branch '0.7.x' of git@gitorious.org:laconica/dev into 0.7.x
X-Git-Url: https://git.mxchange.org/?a=commitdiff_plain;h=da532bae9131b6dec8f5c0d67ce77c4ad1736f7d;hp=-c;p=quix0rs-gnu-social.git

Merge branch '0.7.x' of git@gitorious.org:laconica/dev into 0.7.x
---

da532bae9131b6dec8f5c0d67ce77c4ad1736f7d
diff --combined lib/router.php
index 85425bed29,9d0d3a3f06..e842604e9a
--- a/lib/router.php
+++ b/lib/router.php
@@@ -47,7 -47,7 +47,7 @@@ require_once 'Net/URL/Mapper.php'
  
  class Router
  {
 -    static $m = null;
 +    var $m = null;
      static $inst = null;
  
      static function get()
@@@ -98,14 -98,12 +98,14 @@@
          $main = array('login', 'logout', 'register', 'subscribe',
                        'unsubscribe', 'confirmaddress', 'recoverpassword',
                        'invite', 'favor', 'disfavor', 'sup',
 -                      'tagother', 'block');
 +                      'block');
  
          foreach ($main as $a) {
              $m->connect('main/'.$a, array('action' => $a));
          }
  
 +        $m->connect('main/tagother/:id', array('action' => 'tagother'));
 +
          // these take a code
  
          foreach (array('register', 'confirmaddress', 'recoverpassword') as $c) {
@@@ -228,20 -226,31 +228,31 @@@
  
          // direct messages
  
-         $m->connect('api/direct_messages/:method',
-                     array('action' => 'api',
-                           'apiaction' => 'direct_messages'),
-                     array('method' => '(sent|new)(\.(xml|json|atom|rss))?'));
+         foreach (array('xml', 'json') as $e) {
+             $m->connect('api/direct_messages/new.'.$e,
+                         array('action' => 'api',
+                               'apiaction' => 'direct_messages',
+                               'method' => 'create.'.$e));
+         }
+ 
+         foreach (array('xml', 'json', 'rss', 'atom') as $e) {
+             $m->connect('api/direct_messages.'.$e,
+                         array('action' => 'api',
+                               'apiaction' => 'direct_messages',
+                               'method' => 'direct_messages.'.$e));
+         }
+ 
+         foreach (array('xml', 'json', 'rss', 'atom') as $e) {
+             $m->connect('api/direct_message/sent.'.$e,
+                         array('action' => 'api',
+                         'apiaction' => 'direct_messages',
+                         'method' => 'sent.'.$e));
+         }
  
          $m->connect('api/direct_messages/destroy/:argument',
                      array('action' => 'api',
                            'apiaction' => 'direct_messages'));
  
-         $m->connect('api/:method',
-                     array('action' => 'api',
-                           'apiaction' => 'direct_messages'),
-                     array('method' => 'direct_messages(\.(xml|json|atom|rss))?'));
- 
          // friendships
  
          $m->connect('api/friendships/:method/:argument',
@@@ -271,10 -280,12 +282,12 @@@
                            'apiaction' => 'favorites',
                            'method' => 'favorites'));
  
-         $m->connect('api/:method',
-                     array('action' => 'api',
-                           'apiaction' => 'favorites'),
-                     array('method' => 'favorites(\.(xml|json|rss|atom))?'));
+         foreach (array('xml', 'json', 'rss', 'atom') as $e) {
+             $m->connect('api/favorites.'.$e,
+                 array('action' => 'api',
+                       'apiaction' => 'favorites',
+                       'method' => 'favorites.'.$e));
+         }
  
          // notifications
  
@@@ -347,10 -358,19 +360,19 @@@
  
      function map($path)
      {
-         return $this->m->match($path);
+         try {
+             $match = $this->m->match($path);
+         } catch (Net_URL_Mapper_InvalidException $e) {
+             common_log(LOG_ERR, "Problem getting route for $path - " .
+                 $e->getMessage());
+             $cac = new ClientErrorAction("Page not found.", 404);
+             $cac->showPage();
+         }
+ 
+         return $match;
      }
  
 -    function build($action, $args=null, $fragment=null)
 +    function build($action, $args=null, $params=null, $fragment=null)
      {
          $action_arg = array('action' => $action);
  
@@@ -360,6 -380,6 +382,6 @@@
              $args = $action_arg;
          }
  
 -        return $this->m->generate($args, null, $fragment);
 +        return $this->m->generate($args, $params, $fragment);
      }
  }