X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=actions%2Ffinishaddopenid.php;h=9ba3189b33fdeff3bd9492eb39b6dae27ccf0063;hb=4aedce6864eb27a0c37b3dcae22fee0aa16ae537;hp=6a7bb3d2484cc703dea65e4948d725ec824581e5;hpb=4a0b1a9ea9d288e42ac03c43ef904b0565a7087f;p=quix0rs-gnu-social.git

diff --git a/actions/finishaddopenid.php b/actions/finishaddopenid.php
index 6a7bb3d248..9ba3189b33 100644
--- a/actions/finishaddopenid.php
+++ b/actions/finishaddopenid.php
@@ -34,7 +34,7 @@ class FinishaddopenidAction extends Action {
 
 	function try_login() {
 
-		$consumer = oid_consumer();
+		$consumer =& oid_consumer();
 
 		$response = $consumer->complete(common_local_url('finishaddopenid'));
 
@@ -45,39 +45,33 @@ class FinishaddopenidAction extends Action {
 			// 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();
 			}
-
-			$other = $this->get_user($canonical);
-
-			if ($other) {
-				$this->message(_t('This OpenID is already associated with user "') . $user->nickname . _t('"'));
-			} else {
-				$cur = common_current_user();
-				$result = oid_link_user($cur->id, $display, $canonical);
-				if (!$result) {
-					$this->message(_t('Error connecting user'));
+			
+			$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;
 				}
-				if ($sreg) {
-					if (!$this->update_user($cur, $sreg)) {
-						$this->message(_t('Error updating profile'));
-						return;
-					}
-				}
-				# success!
-				common_redirect(common_local_url('openidsettings'));
 			}
+			# success!
+			common_redirect(common_local_url('openidsettings'));
 		}
 	}
 
@@ -96,9 +90,9 @@ class FinishaddopenidAction extends Action {
 		return $user;
 	}
 
-	function update_user($user, $sreg) {
+	function update_user(&$user, $sreg) {
 
-		$profile = $user->getProfile();
+		$profile =& $user->getProfile();
 
 		$orig_profile = clone($profile);