X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=actions%2Ffinishaddopenid.php;h=b1dce14747153b8c8104bf9dd8940c54292dd210;hb=596365672a9627dbcab0f1aeddc13e4fb9c3b18b;hp=237f0454f7ded23a67486bf89b3142e72d6f6670;hpb=7258969cd48d9e11e500db56665e190d05d1e4c0;p=quix0rs-gnu-social.git diff --git a/actions/finishaddopenid.php b/actions/finishaddopenid.php index 237f0454f7..b1dce14747 100644 --- a/actions/finishaddopenid.php +++ b/actions/finishaddopenid.php @@ -57,7 +57,7 @@ class FinishaddopenidAction extends Action { } $cur =& common_current_user(); - $other = $this->get_user($canonical); + $other = oid_get_user($canonical); if ($other) { if ($other->id == $cur->id) { @@ -72,14 +72,14 @@ class FinishaddopenidAction extends Action { $cur->query('BEGIN'); - $result = oid_link_user($cur->id, $display, $canonical); + $result = oid_link_user($cur->id, $canonical, $display); if (!$result) { $this->message(_t('Error connecting user.')); return; } if ($sreg) { - if (!$this->update_user($cur, $sreg)) { + if (!oid_update_user($cur, $sreg)) { $this->message(_t('Error updating profile')); return; } @@ -88,6 +88,8 @@ class FinishaddopenidAction extends Action { # success! $cur->query('COMMIT'); + + oid_set_last($display); common_redirect(common_local_url('openidsettings')); } @@ -98,55 +100,4 @@ class FinishaddopenidAction extends Action { common_element('p', NULL, $msg); common_show_footer(); } - - function get_user($canonical) { - $user = NULL; - $oid = User_openid::staticGet('canonical', $canonical); - if ($oid) { - $user = User::staticGet('id', $oid->user_id); - } - return $user; - } - - 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: also, store postcode somewhere -- it's valuable! - $profile->location = $sreg['postcode'] . ', ' . $sreg['country']; - } else { - $profile->location = $sreg['country']; - } - } - - # 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 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 false; - } - - return true; - } }