diff --git a/actions/twittersettings.php b/actions/twittersettings.php index 0aea7d66d3..4ec1a57995 100644 --- a/actions/twittersettings.php +++ b/actions/twittersettings.php @@ -35,7 +35,7 @@ class TwittersettingsAction extends SettingsAction { $flink = Foreign_link::getForeignLink($user->id, 1); // 1 == Twitter if ($flink) { - $fuser = Foreign_user::staticGet('user_id', $flink->user_id); + $fuser = Foreign_user::getForeignUser($flink->foreign_id, 1); } $this->form_header(_('Twitter settings'), $msg, $success); @@ -54,7 +54,7 @@ class TwittersettingsAction extends SettingsAction { common_element('a', array('href' => $fuser->uri), $fuser->uri); common_element('span', 'input_instructions', _('Current verified Twitter account.')); - common_hidden('flink_user_id', $flink->user_id); + common_hidden('flink_foreign_id', $flink->foreign_id); common_element_end('p'); common_submit('remove', _('Remove')); } else { @@ -175,14 +175,14 @@ class TwittersettingsAction extends SettingsAction { // 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); - $flink_user_id = $this->arg('flink_user_id'); + $flink_foreign_id = $this->arg('flink_foreign_id'); if (!$flink) { common_debug("couldn't get flink"); } # Maybe an old tab open...? - if ($flink->user_id != $flink_user_id) { + if ($flink->foreign_id != $flink_foreign_id) { common_debug("flink user_id = " . $flink->user_id); $this->show_form(_('That is not your Twitter account.')); return; @@ -191,7 +191,7 @@ class TwittersettingsAction extends SettingsAction { $result = $fuser->delete(); if (!$result) { - common_log_db_error($flink, 'DELETE', __FILE__); + common_log_db_error($fuser, 'DELETE', __FILE__); $this->show_form(_('Couldn\'t remove Twitter user.')); return; } diff --git a/classes/Foreign_link.php b/classes/Foreign_link.php index f4f8abfc41..0c6df8ffcd 100644 --- a/classes/Foreign_link.php +++ b/classes/Foreign_link.php @@ -30,12 +30,11 @@ class Foreign_link extends DB_DataObject // a getForeignUsers() that returns more than one? --Zach static function getForeignLink($user_id, $service) { $flink = new Foreign_link(); - $flink->whereAdd("service = $service"); - $flink->whereAdd("user_id = $user_id"); + $flink->service = $service; + $flink->user_id = $user_id; $flink->limit(1); - if ($flink->find()) { - $flink->fetch(); + if ($flink->find(TRUE)) { return $flink; } @@ -43,14 +42,16 @@ class Foreign_link extends DB_DataObject } // Convenience method - function getForeignUser() { + function getForeignUser() { + $fuser = new Foreign_user(); - $fuser->whereAdd('service = ' . $this->service); - $fuser->whereAdd('id = ' . $this->foreign_id); + + $fuser->service = $this->service; + $fuser->id = $this->foreign_id; + $fuser->limit(1); - if ($fuser->find()) { - $fuser->fetch(); + if ($fuser->find(TRUE)) { return $fuser; }