X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=actions%2Ffinishopenidlogin.php;h=09102e8442806b7c18b1ac1d443cc5d48e9c53d8;hb=d3679822530e351894ca1f809c97638873832a74;hp=3a7f9f250b3c60a7a4bf48d445d86e219d4295ae;hpb=171b4f72ee5a84d2a67b99aca7df9406e68f60c1;p=quix0rs-gnu-social.git

diff --git a/actions/finishopenidlogin.php b/actions/finishopenidlogin.php
index 3a7f9f250b..09102e8442 100644
--- a/actions/finishopenidlogin.php
+++ b/actions/finishopenidlogin.php
@@ -29,6 +29,11 @@ class FinishopenidloginAction extends Action {
 			common_user_error(_t('Already logged in.'));
 		} else if ($_SERVER['REQUEST_METHOD'] == 'POST') {
 			if ($this->arg('create')) {
+				if (!$this->boolean('license')) {
+					$this->show_form(_t('You can\'t register if you don\'t agree to the license.'),
+									 $this->trimmed('newname'));
+					return;
+				}
 				$this->create_new_user();
 			} else if ($this->arg('connect')) {
 				$this->connect_user();
@@ -42,8 +47,7 @@ class FinishopenidloginAction extends Action {
 		}
 	}
 
-	function show_form($error=NULL, $username=NULL) {
-		common_show_header(_t('OpenID Account Setup'));
+	function show_top($error=NULL) {
 		if ($error) {
 			common_element('div', array('class' => 'error'), $error);
 		} else {
@@ -54,7 +58,13 @@ class FinishopenidloginAction extends Action {
 						   _t(' so we must connect your OpenID to a local account. ' .
 							  ' You can either create a new account, or connect with ' .
 							  ' your existing account, if you have one.'));
-		}
+		}		
+	}
+	
+	function show_form($error=NULL, $username=NULL) {
+		common_show_header(_t('OpenID Account Setup'), NULL, $error,
+						   array($this, 'show_top'));
+
 		common_element_start('form', array('method' => 'POST',
 										   'id' => 'account_connect',
 										   'action' => common_local_url('finishopenidlogin')));
@@ -65,6 +75,16 @@ class FinishopenidloginAction extends Action {
 		common_input('newname', _t('New nickname'),
 					 ($username) ? $username : '',
 					 _t('1-64 lowercase letters or numbers, no punctuation or spaces'));
+		common_element_start('p');
+		common_element('input', array('type' => 'checkbox',
+									  'id' => 'license',
+									  'name' => 'license',
+									  'value' => 'true'));
+		common_text(_t('My text and files are available under '));
+		common_element('a', array(href => common_config('license', 'url')),
+					   common_config('license', 'title'));
+		common_text(_t(' except this private data: password, email address, IM address, phone number.'));
+		common_element_end('p');
 		common_submit('create', _t('Create'));
 		common_element('h2', NULL,
 					   'Connect existing account');
@@ -108,8 +128,9 @@ class FinishopenidloginAction extends Action {
 			
 			if ($user) {
 				oid_set_last($display);
-				$this->update_user($user, $sreg);
+				oid_update_user($user, $sreg);
 				common_set_user($user->nickname);
+				common_real_login(true);
 				$this->go_home($user->nickname);
 			} else {
 				$this->save_values($display, $canonical, $sreg);
@@ -124,46 +145,6 @@ class FinishopenidloginAction extends Action {
 		common_show_footer();
 	}
 	
-	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;
-		}
-
-		$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;
-		}
-	}
-	
 	function save_values($display, $canonical, $sreg) {
 		common_ensure_session();
 		$_SESSION['openid_display'] = $display;
@@ -187,6 +168,11 @@ class FinishopenidloginAction extends Action {
 			$this->show_form(_t('Nickname must have only letters and numbers and no spaces.'));
 			return;
 		}
+
+		if (!User::allowed_nickname($nickname)) {
+			$this->show_form(_t('Nickname not allowed.'));
+			return;
+		}
 		
 		if (User::staticGet('nickname', $nickname)) {
 			$this->show_form(_t('Nickname already in use. Try another one.'));
@@ -230,6 +216,8 @@ class FinishopenidloginAction extends Action {
 		# XXX save language if it's passed
 		# XXX save timezone if it's passed
 		
+		$profile->profileurl = common_profile_url($nickname);
+		  
 		$profile->created = DB_DataObject_Cast::dateTime(); # current time
 		
 		$id = $profile->insert();
@@ -241,7 +229,7 @@ class FinishopenidloginAction extends Action {
 		$user = new User();
 		$user->id = $id;
 		$user->nickname = $nickname;
-		$user->uri = common_mint_tag('user:'.$id);
+		$user->uri = common_user_uri($user);
 		
 		if ($sreg['email'] && Validate::email($sreg['email'], true)) {
 			$user->email = $sreg['email'];
@@ -266,6 +254,7 @@ class FinishopenidloginAction extends Action {
 		
 		oid_set_last($display);
 		common_set_user($user->nickname);
+		common_real_login(true);
 		common_redirect(common_local_url('showstream', array('nickname' => $user->nickname)));
 	}
 	
@@ -297,9 +286,10 @@ class FinishopenidloginAction extends Action {
 			return;
 		}
 		
-		$this->update_user($user, $sreg);
+		oid_update_user($user, $sreg);
 		oid_set_last($display);
 		common_set_user($user->nickname);
+		common_real_login(true);
 		$this->go_home($user->nickname);
 	}
 	
@@ -355,6 +345,9 @@ class FinishopenidloginAction extends Action {
 										  'max_length' => 64,
 										  'format' => VALIDATE_NUM . VALIDATE_ALPHA_LOWER))) {
 			return false;
+		}	
+	if (!User::allowed_nickname($str)) {
+			return false;
 		}
 		if (User::staticGet('nickname', $str)) {
 			return false;