From acf562b3ddaa6fbe1727fe301a50f1e9b5f26fd2 Mon Sep 17 00:00:00 2001
From: Evan Prodromou <evan@controlezvous.ca>
Date: Sun, 22 Jun 2008 11:16:11 -0400
Subject: use updateKeys() for updating the user

darcs-hash:20080622151611-34904-1d952df08b5247ab391bb76fd50d7b90498f0c96.gz
---
 actions/profilesettings.php | 26 +++++++++++++++-----------
 1 file changed, 15 insertions(+), 11 deletions(-)

(limited to 'actions/profilesettings.php')

diff --git a/actions/profilesettings.php b/actions/profilesettings.php
index 1b7c75d47..bca86a85f 100644
--- a/actions/profilesettings.php
+++ b/actions/profilesettings.php
@@ -113,20 +113,23 @@ class ProfilesettingsAction extends SettingsAction {
 		assert(!is_null($user)); # should already be checked
 
 		$user->query('BEGIN');
-		
-		$original = clone($user);
-
-		$user->nickname = $nickname;
 
-		$result = $user->update($original);
-		
-		if (!$result) {
-			common_log_db_error($user, 'UPDATE', __FILE__);
-			common_server_error(_t('Couldnt update user.'));
-			return;
+		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;
+			}
 		}
 
-		if ($email != $original->email) {
+		if ($email != $user->email) {
 			
 			$confirm = new Confirm_email();
 			$confirm->code = common_good_rand(16);
@@ -134,6 +137,7 @@ class ProfilesettingsAction extends SettingsAction {
 			$confirm->email = $email;
 			
 			$result = $confirm->insert();
+			
 			if (!$result) {
 				common_log_db_error($confirm, 'INSERT', __FILE__);
 				common_server_error(_t('Couldnt confirm email.'));
-- 
cgit v1.2.3-54-g00ecf