better logic in profile settings, missing method in all

darcs-hash:20080622152306-34904-3d3ab7c02c33efac17d02692a10b4fee2fb013f1.gz

1 files changed, 13 insertions, 14 deletions

diff --git a/actions/profilesettings.php b/actions/profilesettings.php
index bca86a85f..19157e9ca 100644
--- a/actions/profilesettings.php
+++ b/actions/profilesettings.php
@@ -114,22 +114,21 @@ class ProfilesettingsAction extends SettingsAction {
 
 		$user->query('BEGIN');
 
-		if ($user->nickname != $nickname) {
-			
-			$original = clone($user);
-			
-			$user->nickname = $nickname;
-			
-			$result = $user->updateKeys($original);
-			
-			if (!$result) {
-				common_log_db_error($user, 'UPDATE', __FILE__);
-				common_server_error(_t('Couldnt update user.'));
-				return;
-			}
+		$original = clone($user);
+		
+		$user->nickname = $nickname;
+		
+		$result = $user->updateKeys($original);
+		
+		if ($result === FALSE) {
+			common_log_db_error($user, 'UPDATE', __FILE__);
+			common_server_error(_t('Couldnt update user.'));
+			return;
 		}
 
-		if ($email != $user->email) {
+		if ($user->email != $email) {
+			
+			# We don't update email directly; it gets done by confirmemail
 			
 			$confirm = new Confirm_email();
 			$confirm->code = common_good_rand(16);