From 4f1c30fd882211186a1e8609e500a6d2683098dc Mon Sep 17 00:00:00 2001 From: Zach Copley Date: Fri, 5 Sep 2008 15:47:32 -0400 Subject: Conflated Avatar and Profile settings + removed a bunch of unnecessary whitespace darcs-hash:20080905194732-7b5ce-2797acd4d04f3b02c4ad67be629badbae469e8ed.gz --- actions/twittersettings.php | 106 ++++++++++++++++++++++---------------------- 1 file changed, 54 insertions(+), 52 deletions(-) (limited to 'actions/twittersettings.php') diff --git a/actions/twittersettings.php b/actions/twittersettings.php index dac0ce200..24438723f 100644 --- a/actions/twittersettings.php +++ b/actions/twittersettings.php @@ -33,11 +33,11 @@ class TwittersettingsAction extends SettingsAction { $profile = $user->getProfile(); $fuser = NULL; $flink = Foreign_link::getForeignLink($user->id, 1); // 1 == Twitter - + if ($flink) { $fuser = Foreign_user::staticGet('user_id', $flink->user_id); } - + $this->form_header(_('Twitter settings'), $msg, $success); common_element_start('form', array('method' => 'post', 'id' => 'twittersettings', @@ -45,9 +45,11 @@ class TwittersettingsAction extends SettingsAction { common_local_url('twittersettings'))); common_hidden('token', common_session_token()); + common_element('h2', NULL, _('Twitter Account')); + if ($fuser) { common_element_start('p'); - + common_element('span', 'twitter_user', $fuser->nickname); common_element('a', array('href' => $fuser->uri), $fuser->uri); common_element('span', 'input_instructions', @@ -62,36 +64,36 @@ class TwittersettingsAction extends SettingsAction { ($this->arg('twitter_username')) ? $this->arg('twitter_username') : $profile->nickname, _('No spaces, please.')); // hey, it's what Twitter says - common_password('twitter_password', _('Twitter Password')); + common_password('twitter_password', _('Twitter Password')); } - + common_element('h2', NULL, _('Preferences')); - - if ($flink) { - common_checkbox('noticesync', _('Automatically send my notices to Twitter.'), + + if ($flink) { + common_checkbox('noticesync', _('Automatically send my notices to Twitter.'), ($flink->noticesync) ? true : false); common_checkbox('friendsync', _('Subscribe to my Twitter friends here.'), - ($flink->friendsync) ? true : false); - common_submit('save', _('Save')); + ($flink->friendsync) ? true : false); + common_submit('save', _('Save')); } else { common_checkbox('noticesync', _('Automatically send my notices to Twitter.'), true); common_checkbox('friendsync', _('Subscribe to my Twitter friends here.'), true); common_submit('add', _('Add')); } - + common_element_end('form'); common_show_footer(); } function handle_post() { - + # CSRF protection $token = $this->trimmed('token'); if (!$token || $token != common_session_token()) { $this->show_form(_('There was a problem with your session token. Try again, please.')); return; } - + if ($this->arg('save')) { $this->save_preferences(); } else if ($this->arg('add')) { @@ -102,43 +104,43 @@ class TwittersettingsAction extends SettingsAction { $this->show_form(_('Unexpected form submission.')); } } - + function add_twitter_acct() { $twitter_username = $this->trimmed('twitter_username'); $twitter_password = $this->trimmed('twitter_password'); $noticesync = $this->boolean('noticesync'); $friendsync = $this->boolean('friendsync'); - + if (!Validate::string($twitter_username, array('min_length' => 1, 'max_length' => 64, 'format' => VALIDATE_NUM . VALIDATE_ALPHA_LOWER))) { $this->show_form(_('Username must have only lowercase letters and numbers and no spaces.')); return; } - + // Verify this is a real Twitter user. if (!$this->verify_credentials($twitter_username, $twitter_password)) { $this->show_form(_('Could not verify your Twitter credentials!')); return; } - - // Now that we have a valid Twitter user, we have to make another api call to + + // Now that we have a valid Twitter user, we have to make another api call to // find its Twitter ID. Dumb, but true. $twitter_id = $this->get_twitter_id($twitter_username); - + if (!$twitter_id) { $this->show_form(sprintf(_('Unable to retrieve account information for "%s" from Twitter.'), $twitter_username)); return; } - + $fuser = DB_DataObject::factory('foreign_user'); $fuser->id = $twitter_id; - $fuser->service = 1; // Twitter + $fuser->service = 1; // Twitter $fuser->uri = "http://www.twitter.com/$twitter_username"; $fuser->nickname = $twitter_username; $fuser->created = common_sql_now(); $result = $fuser->insert(); - + if (!$result) { common_log_db_error($fuser, 'INSERT', __FILE__); $this->show_form(_('Unable to save your Twitter settings!')); @@ -146,7 +148,7 @@ class TwittersettingsAction extends SettingsAction { } $user = common_current_user(); - + $flink = DB_DataObject::factory('foreign_link'); $flink->user_id = $user->id; $flink->foreign_id = $fuser->id; @@ -157,19 +159,19 @@ class TwittersettingsAction extends SettingsAction { $flink->friendsync = ($friendsync) ? 2 : 0; $flink->profilesync = 0; // XXX: leave as default? $flink_id = $flink->insert(); - + if (!$flink_id) { common_log_db_error($flink, 'INSERT', __FILE__); $this->show_form(_('Unable to save your Twitter settings!')); return; } - + $this->show_form(_('Twitter settings saved.'), true); } function remove_twitter_acct() { $user = common_current_user(); - + // For now we assume one Twitter acct per Laconica acct $flink = Foreign_link::getForeignLink($user->id, 1); $fuser = Foreign_user::getForeignUser($flink->foreign_id, 1); @@ -187,13 +189,13 @@ class TwittersettingsAction extends SettingsAction { } $result = $fuser->delete(); - + if (!$result) { common_log_db_error($flink, 'DELETE', __FILE__); $this->show_form(_('Couldn\'t remove Twitter user.')); return; } - + $result = $flink->delete(); if (!$result) { @@ -204,73 +206,73 @@ class TwittersettingsAction extends SettingsAction { $this->show_form(_('Twitter account removed.'), TRUE); } - + function save_preferences() { $noticesync = $this->boolean('noticesync'); $friendsync = $this->boolean('friendsync'); $user = common_current_user(); $flink = Foreign_link::getForeignLink($user->id, 1); - + if (!$flink) { common_log_db_error($flink, 'SELECT', __FILE__); $this->show_form(_('Couldn\'t save Twitter preferences.')); return; - } - + } + $flink->noticesync = ($noticesync) ? 1 : 0; $flink->friendsync = ($friendsync) ? 2 : 0; // $flink->profilesync = 0; // XXX: leave as default? $result = $flink->update(); - + if (!$result) { common_log_db_error($flink, 'UPDATE', __FILE__); $this->show_form(_('Couldn\'t save Twitter preferences.')); return; } - + $this->show_form(_('Twitter preferences saved.')); - + return; } - function get_twitter_id($twitter_username) { - $uri = "http://twitter.com/users/show/$twitter_username.json"; + function get_twitter_id($twitter_username) { + $uri = "http://twitter.com/users/show/$twitter_username.json"; $data = $this->get_twitter_data($uri); - + if (!$data) { return NULL; } - + $user = json_decode($data); if (!$user) { return NULL; } - + return $user->id; } - function verify_credentials($user, $password) { + function verify_credentials($user, $password) { $uri = 'http://twitter.com/account/verify_credentials.json'; $data = $this->get_twitter_data($uri, $user, $password); - + if (!$data) { return false; } - - $creds = json_decode($data); - + + $creds = json_decode($data); + if (!$creds) { return false; } - - if ($creds->authorized == 1) { + + if ($creds->authorized == 1) { return true; } - - return false; + + return false; } - + // PHP's cURL the best thing to use here? -- Zach function get_twitter_data($uri, $user=NULL, $password=NULL) { $options = array( @@ -283,7 +285,7 @@ class TwittersettingsAction extends SettingsAction { CURLOPT_CONNECTTIMEOUT => 120, CURLOPT_TIMEOUT => 120 ); - + $ch = curl_init($uri); curl_setopt_array($ch, $options); $data = curl_exec($ch); @@ -294,7 +296,7 @@ class TwittersettingsAction extends SettingsAction { } curl_close($ch); - + return $data; } -- cgit v1.2.3-54-g00ecf