Fixed tabbing

diff --git a/lib/applicationlist.php b/lib/applicationlist.php
index 23c727bd6124225864d072d78c0fc019045dd0e0..b404767ba039736a11c9dfb05de7db1f837b0330 100644 (file)
--- a/lib/applicationlist.php
+++ b/lib/applicationlist.php
@@ -69,7 +69,7 @@ class ApplicationList extends Widget
 
     function show()
     {
-        $this->out->elementStart('ul', array('id' => 'applications'));
+        $this->out->elementStart('ul', 'applications');
 
         $cnt = 0;
 
@@ -94,69 +94,58 @@ class ApplicationList extends Widget
         $this->out->elementStart('li', array('class' => 'application',
                                              'id' => 'oauthclient-' . $this->application->id));
 
-       if (!empty($this->application->icon)) {
-           $this->out->element('img', array('src' => $this->application->icon));
-       }
-
-       if (!$this->connections) {
-
-           $this->out->elementStart('a',
-                                    array('href' =>
-                                          common_local_url('showapplication',
-                                                           array('nickname' => $user->nickname,
-                                                                 'id' => $this->application->id)),
-                                          'class' => 'url')
-                                    );
-
-           $this->out->raw($this->application->name);
-           $this->out->elementEnd('a');
-       } else {
-           $this->out->elementStart('a',
-                                    array('href' =>  $this->application->source_url,
-                                          'class' => 'url'));
-
-           $this->out->raw($this->application->name);
-           $this->out->elementEnd('a');
-       }
-
-       $this->out->raw(' by ');
-
-       $this->out->elementStart('a',
-            array(
-                 'href' => $this->application->homepage,
-                 'class' => 'url'
-                )
-                                );
-       $this->out->raw($this->application->organization);
-       $this->out->elementEnd('a');
-
-       $this->out->elementStart('p', 'note');
-        $this->out->raw($this->application->description);
-        $this->out->elementEnd('p');
+        if (!empty($this->application->icon)) {
+            $this->out->element('img', array('src' => $this->application->icon));
+        }
+
+        if (!$this->connections) {
+            $this->out->elementStart('a',
+                            array('href' => common_local_url('showapplication',
+                                                array('nickname' => $user->nickname,
+                                                      'id' => $this->application->id)),
+                                  'class' => 'url'));
+
+            $this->out->raw($this->application->name);
+            $this->out->elementEnd('a');
+        } else {
+            $this->out->elementStart('a', array('href' =>  $this->application->source_url,
+                                                'class' => 'url'));
+
+            $this->out->raw($this->application->name);
+            $this->out->elementEnd('a');
+        }
 
-       $this->out->elementEnd('li');
+        $this->out->raw(' by ');
 
-       if ($this->connections) {
+        $this->out->elementStart('a', array('href' => $this->application->homepage,
+                                            'class' => 'url'));
+        $this->out->raw($this->application->organization);
+        $this->out->elementEnd('a');
 
-           $appUser = Oauth_application_user::getByKeys($this->owner, $this->application);
+        $this->out->elementStart('p', 'note');
+        $this->out->raw($this->application->description);
+        $this->out->elementEnd('p');
 
-           if (empty($appUser)) {
-               common_debug("empty appUser!");
-           }
+        if ($this->connections) {
+            $appUser = Oauth_application_user::getByKeys($this->owner, $this->application);
 
-           $this->out->elementStart('li');
+            if (empty($appUser)) {
+                common_debug("empty appUser!");
+            }
 
-           $access = ($this->application->access_type & Oauth_application::$writeAccess)
-             ? 'read-write' : 'read-only';
+            $this->out->elementStart('li');
 
-           $txt = 'Approved ' . common_exact_date($appUser->modified) .
-             " $access for access.";
+            $access = ($this->application->access_type & Oauth_application::$writeAccess)
+              ? 'read-write' : 'read-only';
 
-           $this->out->raw($txt);
-           $this->out->elementEnd('li');
+            $txt = 'Approved ' . common_exact_date($appUser->modified) .
+              " $access for access.";
 
-           // XXX: Add revoke access button
-       }
+            $this->out->raw($txt);
+            $this->out->elementEnd('li');
+
+            // XXX: Add revoke access button
+        }
     }
 
     /* Override this in subclasses. */