summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorEvan Prodromou <evan@controlyourself.ca>2009-05-22 10:21:22 -0400
committerEvan Prodromou <evan@controlyourself.ca>2009-05-22 10:21:22 -0400
commit677434b113ac58297a32135823c3fc2fc865105c (patch)
treebda8c003aec267e8ff0e3138cf037f1a83b3cd2c
parentebc16062ed2ba6a68d96fbe564c54a3b92b4ef68 (diff)
parent9f23807af4495889a96713c798c0007d5bd6cba7 (diff)
Merge branch '0.8.x' of git@gitorious.org:laconica/dev into 0.8.x
-rw-r--r--lib/noticelist.php28
-rw-r--r--plugins/FBConnect/FBCLoginGroupNav.php12
-rw-r--r--plugins/FBConnect/FBConnectAuth.php69
-rw-r--r--plugins/FBConnect/FBConnectPlugin.css37
-rw-r--r--plugins/FBConnect/FBConnectPlugin.php141
-rw-r--r--plugins/FBConnect/FBConnectSettings.php14
-rw-r--r--plugins/FBConnect/fbfavicon.icobin0 -> 1150 bytes
-rw-r--r--theme/base/css/display.css7
-rw-r--r--theme/default/css/display.css17
-rw-r--r--theme/identica/css/display.css17
10 files changed, 225 insertions, 117 deletions
diff --git a/lib/noticelist.php b/lib/noticelist.php
index 004905056..a52132171 100644
--- a/lib/noticelist.php
+++ b/lib/noticelist.php
@@ -180,25 +180,14 @@ class NoticeListItem extends Widget
$this->showStart();
$this->showNotice();
$this->showNoticeAttachments();
- $this->showNoticeOptions();
$this->showNoticeInfo();
+ $this->showNoticeOptions();
$this->showEnd();
}
function showNotice()
{
-if(0)
- $this->out->elementStart('entry-title');
-else
-
- if ('shownotice' === $this->out->args['action']) {
- $width = '85%';
- } else {
- $width = '90%';
- }
-
-
- $this->out->elementStart('div', array('class' => 'entry-title', 'style' => "float: left; width: $width;"));
+ $this->out->elementStart('div', 'entry-title');
$this->showAuthor();
$this->showContent();
$this->out->elementEnd('div');
@@ -248,17 +237,7 @@ else
function showNoticeInfo()
{
-if(0)
$this->out->elementStart('div', 'entry-content');
-else
-
- if ('shownotice' === $this->out->args['action']) {
- $width = '85%';
- } else {
- $width = '90%';
- }
-
- $this->out->elementStart('div', array('class' => 'entry-content', 'style' => "float: left; width: $width;"));
$this->showNoticeLink();
$this->showNoticeSource();
$this->showContext();
@@ -269,10 +248,7 @@ else
{
$user = common_current_user();
if ($user) {
-if(0)
$this->out->elementStart('div', 'notice-options');
-else
- $this->out->elementStart('div', array('class' => 'notice-options', 'style' => 'float: right; width: 16%;'));
$this->showFaveForm();
$this->showReplyLink();
$this->showDeleteLink();
diff --git a/plugins/FBConnect/FBCLoginGroupNav.php b/plugins/FBConnect/FBCLoginGroupNav.php
index 9aa01a094..6eb09c3c0 100644
--- a/plugins/FBConnect/FBCLoginGroupNav.php
+++ b/plugins/FBConnect/FBCLoginGroupNav.php
@@ -71,26 +71,24 @@ class FBCLoginGroupNav extends Widget
function show()
{
- common_debug('FBCLoginGroupNav');
-
$this->action->elementStart('dl', array('id' => 'site_nav_local_views'));
$this->action->element('dt', null, _('Local views'));
$this->action->elementStart('dd');
-
+
// action => array('prompt', 'title')
$menu = array();
$menu['login'] = array(_('Login'),
_('Login with a username and password'));
-
+
if (!(common_config('site','closed') || common_config('site','inviteonly'))) {
$menu['register'] = array(_('Register'),
_('Sign up for a new account'));
}
-
+
$menu['openidlogin'] = array(_('OpenID'),
_('Login or register with OpenID'));
-
+
$menu['FBConnectLogin'] = array(_('Facebook'),
_('Login or register using Facebook'));
@@ -105,7 +103,7 @@ class FBCLoginGroupNav extends Widget
}
$this->action->elementEnd('ul');
-
+
$this->action->elementEnd('dd');
$this->action->elementEnd('dl');
}
diff --git a/plugins/FBConnect/FBConnectAuth.php b/plugins/FBConnect/FBConnectAuth.php
index e8724cdf9..233eb83ab 100644
--- a/plugins/FBConnect/FBConnectAuth.php
+++ b/plugins/FBConnect/FBConnectAuth.php
@@ -50,7 +50,8 @@ class FBConnectauthAction extends Action
}
} catch (Exception $e) {
- common_debug("Problem getting fbuid.");
+ common_log(LOG_WARNING, 'Problem getting Facebook uid: ' .
+ $e->getMessage());
}
return true;
@@ -124,34 +125,60 @@ class FBConnectauthAction extends Action
}
$this->elementStart('form', array('method' => 'post',
- 'id' => 'account_connect',
+ 'id' => 'form_settings_facebook_connect',
+ 'class' => 'form_settings',
'action' => common_local_url('FBConnectAuth')));
- $this->hidden('token', common_session_token());
- $this->element('h2', null,
- _('Create new account'));
- $this->element('p', null,
- _('Create a new user with this nickname.'));
- $this->input('newname', _('New nickname'),
- ($this->username) ? $this->username : '',
- _('1-64 lowercase letters or numbers, no punctuation or spaces'));
- $this->elementStart('p');
+ $this->elementStart('fieldset', array('id' => 'settings_facebook_connect_options'));
+ $this->element('legend', null, _('Connection options'));
+ $this->elementStart('ul', 'form_data');
+ $this->elementStart('li');
$this->element('input', array('type' => 'checkbox',
'id' => 'license',
+ 'class' => 'checkbox',
'name' => 'license',
'value' => 'true'));
+ $this->elementStart('label', array('class' => 'checkbox', 'for' => 'license'));
$this->text(_('My text and files are available under '));
$this->element('a', array('href' => common_config('license', 'url')),
common_config('license', 'title'));
$this->text(_(' except this private data: password, email address, IM address, phone number.'));
- $this->elementEnd('p');
+ $this->elementEnd('label');
+ $this->elementEnd('li');
+ $this->elementEnd('ul');
+
+ $this->elementStart('fieldset');
+ $this->hidden('token', common_session_token());
+ $this->element('legend', null,
+ _('Create new account'));
+ $this->element('p', null,
+ _('Create a new user with this nickname.'));
+ $this->elementStart('ul', 'form_data');
+ $this->elementStart('li');
+ $this->input('newname', _('New nickname'),
+ ($this->username) ? $this->username : '',
+ _('1-64 lowercase letters or numbers, no punctuation or spaces'));
+ $this->elementEnd('li');
+ $this->elementEnd('ul');
$this->submit('create', _('Create'));
- $this->element('h2', null,
+ $this->elementEnd('fieldset');
+
+ $this->elementStart('fieldset');
+ $this->element('legend', null,
_('Connect existing account'));
$this->element('p', null,
_('If you already have an account, login with your username and password to connect it to your Facebook.'));
+ $this->elementStart('ul', 'form_data');
+ $this->elementStart('li');
$this->input('nickname', _('Existing nickname'));
+ $this->elementEnd('li');
+ $this->elementStart('li');
$this->password('password', _('Password'));
+ $this->elementEnd('li');
+ $this->elementEnd('ul');
$this->submit('connect', _('Connect'));
+ $this->elementEnd('fieldset');
+
+ $this->elementEnd('fieldset');
$this->elementEnd('form');
}
@@ -283,7 +310,7 @@ class FBConnectauthAction extends Action
} else {
- common_debug("no flink found for fbuid: $this->fbuid");
+ common_debug("No flink found for fbuid: $this->fbuid");
$this->showForm(null, $this->bestNewNickname());
}
@@ -306,8 +333,6 @@ class FBConnectauthAction extends Action
function flinkUser($user_id, $fbuid)
{
- common_debug("flinkUser()");
-
$flink = new Foreign_link();
$flink->user_id = $user_id;
$flink->foreign_id = $fbuid;
@@ -370,7 +395,10 @@ class FBConnectauthAction extends Action
// XXX: Consider moving this to lib/facebookutil.php
function getFacebookFields($fb_uid, $fields) {
try {
- $infos = getFacebook()->api_client->users_getInfo($fb_uid, $fields);
+
+ $facebook = getFacebook();
+
+ $infos = $facebook->api_client->users_getInfo($fb_uid, $fields);
if (empty($infos)) {
return null;
@@ -378,9 +406,10 @@ class FBConnectauthAction extends Action
return reset($infos);
} catch (Exception $e) {
- error_log("Failure in the api when requesting " . join(",", $fields)
- ." on uid " . $fb_uid . " : ". $e->getMessage());
- return null;
+ common_log(LOG_WARNING, "Facebook client failure when requesting " .
+ join(",", $fields) . " on uid " . $fb_uid .
+ " : ". $e->getMessage());
+ return null;
}
}
diff --git a/plugins/FBConnect/FBConnectPlugin.css b/plugins/FBConnect/FBConnectPlugin.css
new file mode 100644
index 000000000..564fdaee9
--- /dev/null
+++ b/plugins/FBConnect/FBConnectPlugin.css
@@ -0,0 +1,37 @@
+/** Styles for Facebook logo and Facebook user profile avatar.
+ *
+ * @package Laconica
+ * @author Sarven Capadisli <csarven@controlyourself.ca>
+ * @copyright 2009 Control Yourself, Inc.
+ * @license http://www.fsf.org/licensing/licenses/agpl-3.0.html GNU Affero General Public License version 3.0
+ * @link http://laconi.ca/
+ */
+
+#site_nav_global_primary #nav_fb {
+position:relative;
+margin-left:18px;
+margin-right:-7px;
+}
+
+#nav_fb .fb_profile_pic_rendered img {
+position:relative;
+top:3px;
+left:0;
+display:inline;
+border:1px solid #3B5998;
+padding:1px;
+}
+
+#nav_fb img {
+position:absolute;
+top:-13px;
+left:-11px;
+display:inline;
+}
+
+#settings_facebook_connect_options legend {
+display:none;
+}
+#form_settings_facebook_connect fieldset fieldset legend {
+display:block;
+}
diff --git a/plugins/FBConnect/FBConnectPlugin.php b/plugins/FBConnect/FBConnectPlugin.php
index 36dee0448..ad5e47e47 100644
--- a/plugins/FBConnect/FBConnectPlugin.php
+++ b/plugins/FBConnect/FBConnectPlugin.php
@@ -40,7 +40,6 @@ require_once INSTALLDIR . '/plugins/FBConnect/FBConnectSettings.php';
require_once INSTALLDIR . '/plugins/FBConnect/FBCLoginGroupNav.php';
require_once INSTALLDIR . '/plugins/FBConnect/FBCSettingsNav.php';
-
/**
* Plugin to enable Facebook Connect
*
@@ -53,7 +52,6 @@ require_once INSTALLDIR . '/plugins/FBConnect/FBCSettingsNav.php';
class FBConnectPlugin extends Plugin
{
-
function __construct()
{
parent::__construct();
@@ -84,29 +82,27 @@ class FBConnectPlugin extends Plugin
'media type you accept'), 406);
}
-
header('Content-Type: '.$type);
$action->extraHeaders();
$action->startXML('html',
- '-//W3C//DTD XHTML 1.0 Strict//EN',
- 'http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd');
+ '-//W3C//DTD XHTML 1.0 Strict//EN',
+ 'http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd');
$language = $action->getLanguage();
- $action->elementStart('html', array('xmlns' => 'http://www.w3.org/1999/xhtml',
- 'xmlns:fb' => 'http://www.facebook.com/2008/fbml',
- 'xml:lang' => $language,
- 'lang' => $language));
+ $action->elementStart('html',
+ array('xmlns' => 'http://www.w3.org/1999/xhtml',
+ 'xmlns:fb' => 'http://www.facebook.com/2008/fbml',
+ 'xml:lang' => $language,
+ 'lang' => $language));
return false;
-
}
function onEndShowLaconicaScripts($action)
{
-
$action->element('script',
array('type' => 'text/javascript',
'src' => 'http://static.ak.connect.facebook.com/js/api_lib/v0.4/FeatureLoader.js.php'),
@@ -134,83 +130,105 @@ class FBConnectPlugin extends Plugin
$action->raw($html);
}
+ function onEndShowLaconicaStyles($action)
+ {
+ $action->element('link', array('rel' => 'stylesheet',
+ 'type' => 'text/css',
+ 'href' => common_path('plugins/FBConnect/FBConnectPlugin.css')));
+ }
+
function onStartPrimaryNav($action)
{
$user = common_current_user();
if ($user) {
+ $action->menuItem(common_local_url('all', array('nickname' => $user->nickname)),
+ _('Home'), _('Personal profile and friends timeline'), false, 'nav_home');
+ $action->menuItem(common_local_url('profilesettings'),
+ _('Account'), _('Change your email, avatar, password, profile'), false, 'nav_account');
+ if (common_config('xmpp', 'enabled')) {
+ $action->menuItem(common_local_url('imsettings'),
+ _('Connect'), _('Connect to IM, SMS, Twitter'), false, 'nav_connect');
+ } else {
+ $action->menuItem(common_local_url('smssettings'),
+ _('Connect'), _('Connect to SMS, Twitter'), false, 'nav_connect');
+ }
+ $action->menuItem(common_local_url('invite'),
+ _('Invite'),
+ sprintf(_('Invite friends and colleagues to join you on %s'),
+ common_config('site', 'name')),
+ false, 'nav_invitecontact');
+
$flink = Foreign_link::getByUserId($user->id, FACEBOOK_CONNECT_SERVICE);
+ $fbuid = 0;
if ($flink) {
- $facebook = getFacebook();
+ try {
+
+ $facebook = getFacebook();
+ $fbuid = getFacebook()->get_loggedin_user();
- if ($facebook->api_client->users_isAppUser($flink->foreign_id) ||
- $facebook->api_client->added) {
+ } catch (Exception $e) {
+ common_log(LOG_WARNING,
+ 'Problem getting Facebook client: ' .
+ $e->getMessage());
+ }
- // XXX: We need to replace this with a proper mini-icon and only after
- // checing the FB Connect JavaScript lib method to see what the Connect
- // status is. Checking Connect status looks to be impossible with the
- // PHP client.
+ // Display Facebook Logged in indicator w/Facebook favicon
- $action->elementStart('li');
+ if ($fbuid > 0) {
+
+ $action->elementStart('li', array('id' => 'nav_fb'));
$action->elementStart('fb:profile-pic', array('uid' => $flink->foreign_id,
- 'facebook-logo' => 'true',
'linked' => 'false',
- 'width' => 32,
- 'height' => 32));
+ 'width' => 16,
+ 'height' => 16));
$action->elementEnd('fb:profile-pic');
+
+ $iconurl = common_path('/plugins/FBConnect/fbfavicon.ico');
+ $action->element('img', array('src' => $iconurl));
+
$action->elementEnd('li');
- }
+ }
}
-
- $action->menuItem(common_local_url('all', array('nickname' => $user->nickname)),
- _('Home'), _('Personal profile and friends timeline'), false, 'nav_home');
- $action->menuItem(common_local_url('profilesettings'),
- _('Account'), _('Change your email, avatar, password, profile'), false, 'nav_account');
- if (common_config('xmpp', 'enabled')) {
- $action->menuItem(common_local_url('imsettings'),
- _('Connect'), _('Connect to IM, SMS, Twitter'), false, 'nav_connect');
- } else {
- $action->menuItem(common_local_url('smssettings'),
- _('Connect'), _('Connect to SMS, Twitter'), false, 'nav_connect');
- }
- $action->menuItem(common_local_url('invite'),
- _('Invite'),
- sprintf(_('Invite friends and colleagues to join you on %s'),
- common_config('site', 'name')),
- false, 'nav_invitecontact');
- // Need to override the Logout link to make it do FB stuff
+ // Need to override the Logout link to make it do FB stuff
- $logout_url = common_local_url('logout');
- $title = _('Logout from the site');
- $text = _('Logout');
+ if ($fbuid > 0) {
- $html = sprintf('<li id="nav_logout"><a href="%s" title="%s" ' .
- 'onclick="FB.Connect.logout(function() { goto_logout() })">%s</a></li>',
+ $logout_url = common_local_url('logout');
+ $title = _('Logout from the site');
+ $text = _('Logout');
+
+ $html = sprintf('<li id="nav_logout"><a href="%s" title="%s" ' .
+ 'onclick="FB.Connect.logout(function() { goto_logout() })">%s</a></li>',
$logout_url, $title, $text);
- $action->raw($html);
+ $action->raw($html);
+ } else {
+ $action->menuItem(common_local_url('logout'),
+ _('Logout'), _('Logout from the site'), false, 'nav_logout');
+ }
}
else {
if (!common_config('site', 'closed')) {
$action->menuItem(common_local_url('register'),
- _('Register'), _('Create an account'), false, 'nav_register');
+ _('Register'), _('Create an account'), false, 'nav_register');
}
$action->menuItem(common_local_url('openidlogin'),
- _('OpenID'), _('Login with OpenID'), false, 'nav_openid');
+ _('OpenID'), _('Login with OpenID'), false, 'nav_openid');
$action->menuItem(common_local_url('login'),
- _('Login'), _('Login to the site'), false, 'nav_login');
+ _('Login'), _('Login to the site'), false, 'nav_login');
}
$action->menuItem(common_local_url('doc', array('title' => 'help')),
- _('Help'), _('Help me!'), false, 'nav_help');
+ _('Help'), _('Help me!'), false, 'nav_help');
$action->menuItem(common_local_url('peoplesearch'),
- _('Search'), _('Search for people or text'), false, 'nav_search');
+ _('Search'), _('Search for people or text'), false, 'nav_search');
return false;
}
@@ -220,7 +238,7 @@ class FBConnectPlugin extends Plugin
$action_name = get_class($action);
$login_actions = array('LoginAction', 'RegisterAction',
- 'OpenidloginAction', 'FacebookStart');
+ 'OpenidloginAction', 'FBConnectLoginAction');
if (in_array($action_name, $login_actions)) {
$nav = new FBCLoginGroupNav($action);
@@ -240,6 +258,21 @@ class FBConnectPlugin extends Plugin
return true;
}
-}
+ function onEndLogout($action)
+ {
+ try {
+
+ $facebook = getFacebook();
+ $fbuid = $facebook->get_loggedin_user();
+ if ($fbuid > 0) {
+ $facebook->logout(common_local_url('public'));
+ }
+ } catch (Exception $e) {
+ common_log(LOG_WARNING, 'Could\'t logout of Facebook: ' .
+ $e->getMessage());
+ }
+ }
+
+}
diff --git a/plugins/FBConnect/FBConnectSettings.php b/plugins/FBConnect/FBConnectSettings.php
index b7831269c..7e255f43a 100644
--- a/plugins/FBConnect/FBConnectSettings.php
+++ b/plugins/FBConnect/FBConnectSettings.php
@@ -169,8 +169,18 @@ class FBConnectSettingsAction extends ConnectSettingsAction
return;
}
- $facebook = getFacebook();
- $facebook->logout();
+ try {
+
+ // XXX: not sure what exactly to do here
+
+ $facebook = getFacebook();
+ $facebook->clear_cookie_state();
+
+ } catch (Exception $e) {
+ common_log(LOG_WARNING,
+ 'Couldn\'t clear Facebook cookies: ' .
+ $e->getMessage());
+ }
$this->showForm(_('Facebook user disconnected.'), true);
diff --git a/plugins/FBConnect/fbfavicon.ico b/plugins/FBConnect/fbfavicon.ico
new file mode 100644
index 000000000..c57c0342f
--- /dev/null
+++ b/plugins/FBConnect/fbfavicon.ico
Binary files differ
diff --git a/theme/base/css/display.css b/theme/base/css/display.css
index edec14163..5d2b5231c 100644
--- a/theme/base/css/display.css
+++ b/theme/base/css/display.css
@@ -747,6 +747,13 @@ border-radius:4px;
-moz-border-radius:4px;
-webkit-border-radius:4px;
}
+.notices .notices {
+margin-top:7px;
+margin-left:3%;
+width:97%;
+float:left;
+}
+
/* NOTICES */
#notices_primary {
diff --git a/theme/default/css/display.css b/theme/default/css/display.css
index fce2e7e29..16c9322a5 100644
--- a/theme/default/css/display.css
+++ b/theme/default/css/display.css
@@ -175,10 +175,6 @@ background-image:url(../../base/images/icons/twotone/green/shield.gif);
}
/* NOTICES */
-.notices li.over {
-background-color:#fcfcfc;
-}
-
.notice-options .notice_reply a,
.notice-options form input.submit {
background-color:transparent;
@@ -214,6 +210,19 @@ font-family:sans-serif;
.notices li.hover {
background-color:#fcfcfc;
}
+
+.notices .notices {
+background-color:rgba(200, 200, 200, 0.025);
+}
+.notices .notices .notices {
+background-color:rgba(200, 200, 200, 0.050);
+}
+.notices .notices .notices .notices {
+background-color:rgba(200, 200, 200, 0.075);
+}
+.notices .notices .notices .notices .notices {
+background-color:rgba(200, 200, 200, 0.100);
+}
/*END: NOTICES */
#new_group a {
diff --git a/theme/identica/css/display.css b/theme/identica/css/display.css
index df114417e..2fb123a20 100644
--- a/theme/identica/css/display.css
+++ b/theme/identica/css/display.css
@@ -175,10 +175,6 @@ background-image:url(../../base/images/icons/twotone/green/shield.gif);
}
/* NOTICES */
-.notices li.over {
-background-color:#fcfcfc;
-}
-
.notice-options .notice_reply a,
.notice-options form input.submit {
background-color:transparent;
@@ -214,6 +210,19 @@ font-family:sans-serif;
.notices li.hover {
background-color:#fcfcfc;
}
+
+.notices .notices {
+background-color:rgba(200, 200, 200, 0.025);
+}
+.notices .notices .notices {
+background-color:rgba(200, 200, 200, 0.050);
+}
+.notices .notices .notices .notices {
+background-color:rgba(200, 200, 200, 0.075);
+}
+.notices .notices .notices .notices .notices {
+background-color:rgba(200, 200, 200, 0.100);
+}
/*END: NOTICES */
#new_group a {