}
function try_login() {
-
- $consumer = oid_consumer();
+
+ $consumer =& oid_consumer();
$response = $consumer->complete(common_local_url('finishaddopenid'));
// Authentication failed; display the error message.
$this->message(_t('OpenID authentication failed: ') . $response->message);
} else if ($response->status == Auth_OpenID_SUCCESS) {
- // This means the authentication succeeded; extract the
- // identity URL and Simple Registration data (if it was
- // returned).
+
$display = $response->getDisplayIdentifier();
- $canonical = ($response->endpoint->canonicalID) ?
- $response->endpoint->canonicalID : $response->getDisplayIdentifier();
+ $canonical = ($response->endpoint && $response->endpoint->canonicalID) ?
+ $response->endpoint->canonicalID : $display;
$sreg_resp = Auth_OpenID_SRegResponse::fromSuccessResponse($response);
if ($sreg_resp) {
$sreg = $sreg_resp->contents();
}
-
- $user = $this->get_user($canonical);
- if ($user) {
- $this->message(_t('This OpenID is already associated with user "') . $user->nickname . _t('"'));
- } else {
- $user = common_current_user();
- $this->connect_user($user, $display, $canonical);
- if ($sreg) {
- $this->update_user($user, $sreg);
+ $cur =& common_current_user();
+ common_debug('cur = ' .print_r($cur, TRUE), __FILE__);
+ $result = oid_link_user($cur->id, $display, $canonical);
+
+ if (!$result) {
+ $this->message(_t('Error connecting user.'));
+ return;
+ }
+ if ($sreg) {
+ if (!$this->update_user($cur, $sreg)) {
+ $this->message(_t('Error updating profile'));
+ return;
}
- common_redirect(common_local_url('openidsettings'));
}
+ # success!
+ common_redirect(common_local_url('openidsettings'));
}
}
common_element('p', NULL, $msg);
common_show_footer();
}
-
+
function get_user($canonical) {
$user = NULL;
$oid = User_openid::staticGet('canonical', $canonical);
return $user;
}
- function update_user($user, $sreg) {
-
- $profile = $user->getProfile();
+ function update_user(&$user, $sreg) {
+
+ $profile =& $user->getProfile();
$orig_profile = clone($profile);
-
+
if ($sreg['fullname'] && strlen($sreg['fullname']) <= 255) {
$profile->fullname = $sreg['fullname'];
}
-
+
if ($sreg['country']) {
if ($sreg['postcode']) {
# XXX: use postcode to get city and region
# XXX save language if it's passed
# XXX save timezone if it's passed
-
+
if (!$profile->update($orig_profile)) {
common_server_error(_t('Error saving the profile.'));
- return;
+ return false;
}
$orig_user = clone($user);
-
+
if ($sreg['email'] && Validate::email($sreg['email'], true)) {
$user->email = $sreg['email'];
}
-
+
if (!$user->update($orig_user)) {
common_server_error(_t('Error saving the user.'));
- return;
+ return false;
}
- }
-
- function connect_user($user, $display, $canonical) {
- $oid = new User_openid();
- $oid->display = $display;
- $oid->canonical = $canonical;
- $oid->user_id = $user->id;
- $oid->created = DB_DataObject_Cast::dateTime();
-
- if (!$oid->insert()) {
- common_server_error(_t('Error connecting OpenID.'));
- return;
- }
+ return true;
}
}