From 479096c8d79e40abc27566551dda280be5910c32 Mon Sep 17 00:00:00 2001 From: Zach Copley Date: Mon, 25 Oct 2010 10:38:40 -0700 Subject: [PATCH] Less scary OAuth authorization messages when using anonymous consumer --- actions/apioauthauthorize.php | 18 +++++++++++++----- actions/apioauthpin.php | 2 -- 2 files changed, 13 insertions(+), 7 deletions(-) diff --git a/actions/apioauthauthorize.php b/actions/apioauthauthorize.php index f2f1071d9b..75b4540fb9 100644 --- a/actions/apioauthauthorize.php +++ b/actions/apioauthauthorize.php @@ -386,7 +386,7 @@ class ApiOauthAuthorizeAction extends Action $this->elementStart('ul', 'form_data'); $this->elementStart('li'); $this->elementStart('p'); - if (!empty($this->app->icon)) { + if (!empty($this->app->icon) && $this->app->name != 'anonymous') { $this->element('img', array('src' => $this->app->icon)); } @@ -396,11 +396,19 @@ class ApiOauthAuthorizeAction extends Action // TRANS: User notification of external application requesting account access. // TRANS: %1$s is the application name requesting access, %2$s is the organisation behind the application, // TRANS: %3$s is the access type requested, %4$s is the StatusNet sitename. - $msg = _('The application %1$s by ' . - '%2$s would like the ability ' . + if ($this->app->name == 'anonymous') { + // Special message for the anonymous app and consumer + $msg = _('An application would like the ability ' . 'to %3$s your %4$s account data. ' . 'You should only give access to your %4$s account ' . 'to third parties you trust.'); + } else { + $msg = _('The application %1$s by ' . + '%2$s would like the ability ' . + 'to %3$s your %4$s account data. ' . + 'You should only give access to your %4$s account ' . + 'to third parties you trust.'); + } $this->raw(sprintf($msg, $this->app->name, @@ -580,14 +588,14 @@ class ApiOauthAuthorizeAction extends Action // TRANS: Header of user notification after authorising an application access to a profile. // TRANS: %s is the authorised application name. _('You have successfully authorized %s.'), - $this->app->name + ($this->app->name == 'anonymous') ? 'the application' : $this->app->name ); $msg = sprintf( // TRANS: Uer notification after authorising an application access to a profile. // TRANS: %s is the authorised application name. _('Please return to %s and enter the following security code to complete the process.'), - $this->app->name + ($this->app->name == 'anonymous') ? 'the application' : $this->app->name ); if ($this->reqToken->verified_callback == 'oob') { diff --git a/actions/apioauthpin.php b/actions/apioauthpin.php index 5e6713a548..206df43217 100644 --- a/actions/apioauthpin.php +++ b/actions/apioauthpin.php @@ -31,8 +31,6 @@ if (!defined('STATUSNET') && !defined('LACONICA')) { exit(1); } -require_once INSTALLDIR . '/lib/info.php'; - /** * Class for displaying an OAuth verifier pin * -- 2.39.5