X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=actions%2Fprofilesettings.php;h=44243f310d95adcd8318b910c81f0d5b28643332;hb=f374e924f51d50a601bef4beeb138665374485b0;hp=c4c3ae0fc4410b1c856d642208744fd1d05d2c00;hpb=53eeee70e7eaba81b44f35d787b218f8ebaa69e6;p=quix0rs-gnu-social.git

diff --git a/actions/profilesettings.php b/actions/profilesettings.php
index c4c3ae0fc4..44243f310d 100644
--- a/actions/profilesettings.php
+++ b/actions/profilesettings.php
@@ -23,29 +23,47 @@ require_once(INSTALLDIR.'/lib/settingsaction.php');
 
 class ProfilesettingsAction extends SettingsAction {
 
+	function show_top($arr) {
+		$msg = $arr[0];
+		$success = $arr[1];
+		if ($msg) {
+			$this->message($msg, $success);
+		} else {
+			common_element('div', 'instructions',
+						   _t('You can update your personal profile info here '.
+							  'so people know more about you.'));
+		}
+		$this->settings_menu();
+	}
+	
 	function show_form($msg=NULL, $success=false) {
 		$user = common_current_user();
 		$profile = $user->getProfile();
-		common_show_header(_t('Profile settings'));
-		$this->settings_menu();
-		$this->message($msg, $success);
+		common_show_header(_t('Profile settings'), NULL, array($msg, $success),
+						   array($this, 'show_top'));
+
 		common_element_start('form', array('method' => 'POST',
 										   'id' => 'profilesettings',
 										   'action' =>
 										   common_local_url('profilesettings')));
 		# too much common patterns here... abstractable?
 		common_input('nickname', _t('Nickname'),
-					 ($this->arg('nickname')) ? $this->arg('nickname') : $profile->nickname);
+					 ($this->arg('nickname')) ? $this->arg('nickname') : $profile->nickname,
+					 _t('1-64 lowercase letters or numbers, no punctuation or spaces'));
 		common_input('fullname', _t('Full name'),
 					 ($this->arg('fullname')) ? $this->arg('fullname') : $profile->fullname);
 		common_input('email', _t('Email address'),
-					 ($this->arg('email')) ? $this->arg('email') : $user->email);
+					 ($this->arg('email')) ? $this->arg('email') : $user->email,
+					 _t('Used only for updates, announcements, and password recovery'));
 		common_input('homepage', _t('Homepage'),
-					 ($this->arg('homepage')) ? $this->arg('homepage') : $profile->homepage);
+					 ($this->arg('homepage')) ? $this->arg('homepage') : $profile->homepage,
+					 _t('URL of your homepage, blog, or profile on another site'));
 		common_textarea('bio', _t('Bio'),
-						($this->arg('bio')) ? $this->arg('bio') : $profile->bio);
+						($this->arg('bio')) ? $this->arg('bio') : $profile->bio,
+						_t('Describe yourself and your interests in 140 chars'));
 		common_input('location', _t('Location'),
-					 ($this->arg('location')) ? $this->arg('location') : $profile->location);
+					 ($this->arg('location')) ? $this->arg('location') : $profile->location,
+					 _t('Where you are, like "City, State (or Region), Country"'));
 		common_submit('submit', _t('Save'));
 		common_element_end('form');
 		common_show_footer();
@@ -62,7 +80,7 @@ class ProfilesettingsAction extends SettingsAction {
 
 		# Some validation
 		
-		if (!Validate::email($email, true)) {
+		if ($email && !Validate::email($email, true)) {
 			$this->show_form(_t('Not a valid email address.'));
 			return;
 		} else if (!Validate::string($nickname, array('min_length' => 1,
@@ -70,6 +88,9 @@ class ProfilesettingsAction extends SettingsAction {
 													  'format' => VALIDATE_NUM . VALIDATE_ALPHA_LOWER))) {
 			$this->show_form(_t('Nickname must have only letters and numbers and no spaces.'));
 			return;
+		} else if (!User::allowed_nickname($nickname)) {
+			$this->show_form(_t('Not a valid nickname.'));
+			return;
 		} else if (!is_null($homepage) && (strlen($homepage) > 0) &&
 				   !Validate::uri($homepage, array('allowed_schemes' => array('http', 'https')))) {
 			$this->show_form(_t('Homepage is not a valid URL.'));
@@ -92,20 +113,56 @@ class ProfilesettingsAction extends SettingsAction {
 		}
 		
 		$user = common_current_user();
-		assert(!is_null($user)); # should already be checked
 
-		# FIXME: transaction!
+		$user->query('BEGIN');
 
-		$original = clone($user);
-
-		$user->nickname = $nickname;
-		$user->email = $email;
+		if ($user->nickname != $nickname) {
+			
+			common_debug('Updating user nickname from ' . $user->nickname . ' to ' . $nickname,
+						 __FILE__);
+			
+			$original = clone($user);
+		
+			$user->nickname = $nickname;
 
-		if (!$user->update($original)) {
-			common_server_error(_t('Couldnt update user.'));
-			return;
+			$result = $user->updateKeys($original);
+		
+			if ($result === FALSE) {
+				common_log_db_error($user, 'UPDATE', __FILE__);
+				common_server_error(_t('Couldnt update user.'));
+				return;
+			}
 		}
 
+		if ($user->email != $email) {
+			
+			common_debug('Updating user email from ' . $user->email . ' to ' . $email,
+						 __FILE__);
+			
+			# We don't update email directly; it gets done by confirmemail
+
+			$confirm = new Confirm_address();
+			
+			$confirm->code = common_confirmation_code(128);
+			$confirm->user_id = $user->id;
+			$confirm->address = $email;
+			$confirm->address_type = 'email';
+			
+			$result = $confirm->insert();
+			
+			if (!$result) {
+				common_log_db_error($confirm, 'INSERT', __FILE__);
+				common_server_error(_t('Couldnt confirm email.'));
+				return FALSE;
+			}
+			
+			# XXX: try not to do this in the middle of a transaction
+		
+			mail_confirm_address($confirm->code,
+								 $profile->nickname,
+								 $email);
+		}
+		
 		$profile = $user->getProfile();
 
 		$orig_profile = clone($profile);
@@ -117,11 +174,21 @@ class ProfilesettingsAction extends SettingsAction {
 		$profile->location = $location;
 		$profile->profileurl = common_profile_url($nickname);
 
-		if (FALSE === $profile->update($orig_profile)) {
+		common_debug('Old profile: ' . common_log_objstring($orig_profile), __FILE__);
+		common_debug('New profile: ' . common_log_objstring($profile), __FILE__);
+		
+		$result = $profile->update($orig_profile);
+		
+		if (!$result) {
+			common_log_db_error($profile, 'UPDATE', __FILE__);
 			common_server_error(_t('Couldnt save profile.'));
 			return;
 		}
 
+		$user->query('COMMIT');
+
+		common_broadcast_profile($profile);
+
 		$this->show_form(_t('Settings saved.'), TRUE);
 	}
 	
@@ -144,4 +211,4 @@ class ProfilesettingsAction extends SettingsAction {
 			return $other->id != $user->id;
 		}
 	}
-}
\ No newline at end of file
+}