Make the verifier pin display a little nicer

actions/apioauthauthorize.php

@@ -449,21 +449,28 @@ class ApiOauthAuthorizeAction extends Action
 
     function showAuthorized()
     {
+        $title = sprintf(
+            _("You have successfully authorized %s."),
+            $this->app->name
+        );
+
+        $msg = sprintf(
+            _('Please return to %s and enter the following security code to complete the process.'),
+            $this->app->name
+        );
 
         if ($this->reqToken->verified_callback == 'oob') {
-
+            $pin = new ApiOauthPinAction($title, $msg, $this->reqToken->verifier);
-            $pin = new ApiOauthPinAction($this->reqToken->verifier);
             $pin->showPage();
-
         } else {
 
+            // NOTE: This should probably never happen; trhow an error instead?
+
             $info = new InfoAction(
-                _("Authorization succeeded."),
+                $title,
-                sprintf(
+                $msg,
-                    _('The request token %s has been authorized. Please exchange it for an access token using this verifier: %s'),
+                $this->oauthTokenParam,
-                    $this->oauthTokenParam,
+                $this->reqToken->verifier
-                    $this->reqToken->verifier
-                )
             );
 
             $info->showPage();

actions/apioauthpin.php

@@ -36,6 +36,8 @@ require_once INSTALLDIR . '/lib/info.php';
 /**
  * Class for displaying an OAuth verifier pin
  *
+ * XXX: I'm pretty sure we don't need to check the logged in state here. -- Zach
+ *
  * @category Action
  * @package  StatusNet
  * @author   Zach Copley <zach@status.net>
@@ -45,16 +47,13 @@ require_once INSTALLDIR . '/lib/info.php';
 
 class ApiOauthPinAction extends InfoAction
 {
-
+    function __construct($title, $message, $verifier)
-    function __construct($verifier)
     {
         $this->verifier = $verifier;
-        $title = _('Authorization succeeded.');
+        $this->title    = $title;
-        parent::__construct($title, $title);
+        parent::__construct($title, $message);
     }
 
-    // TODO: Check for logged in state!
-
     /**
      * Display content.
      *
@@ -62,8 +61,7 @@ class ApiOauthPinAction extends InfoAction
      */
     function showContent()
     {
-        // XXX: make this much nicer
+        $this->element('div', array('class' => 'info'), $this->message);
-        $this->element('div', array('class' => 'info'), $this->verifier);
+        $this->element('div', array('id' => 'oauth_pin'), $this->verifier);
     }
-
 }

tests/oauth/fetch_temp_creds.php

@@ -56,7 +56,7 @@ try {
 } catch (Exception $e) {
     // oh noez
     print $e->getMessage();
-    print "OAuth Request:\n";
+    print "\nOAuth Request:\n";
     var_dump($req);
     exit(1);
 }