X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=actions%2Fprofilesettings.php;h=c4c3ae0fc4410b1c856d642208744fd1d05d2c00;hb=10f9a8a125c08dd403319aa84b4902db91253095;hp=ae060dd9086e50f82fd3a2704a130998605be724;hpb=1a15570339fa7ae5e33a98a7fe6a8645e3d3af72;p=quix0rs-gnu-social.git

diff --git a/actions/profilesettings.php b/actions/profilesettings.php
index ae060dd908..c4c3ae0fc4 100644
--- a/actions/profilesettings.php
+++ b/actions/profilesettings.php
@@ -1,87 +1,147 @@
 <?php
-/* 
+/*
  * Laconica - a distributed open-source microblogging tool
  * Copyright (C) 2008, Controlez-Vous, Inc.
- * 
+ *
  * This program is free software: you can redistribute it and/or modify
  * it under the terms of the GNU Affero General Public License as published by
  * the Free Software Foundation, either version 3 of the License, or
  * (at your option) any later version.
- * 
+ *
  * This program is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  * GNU Affero General Public License for more details.
- * 
+ *
  * You should have received a copy of the GNU Affero General Public License
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
 if (!defined('LACONICA')) { exit(1); }
 
+require_once(INSTALLDIR.'/lib/settingsaction.php');
+
 class ProfilesettingsAction extends SettingsAction {
-	
+
 	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_element_start('form', array('method' => 'POST',
 										   'id' => 'profilesettings',
-										   'action' => 
+										   'action' =>
 										   common_local_url('profilesettings')));
-		common_input('nickname', _t('Nickname'));
-		common_input('fullname', _t('Full name'));
-		common_input('email', _t('Email address'));		
-		common_input('homepage', _t('Homepage'));				
-		common_input('bio', _t('Bio'));
-		common_input('location', _t('Location'));
-		common_element('input', array('name' => 'submit',
-									  'type' => 'submit',
-									  'id' => 'submit'),
-					   _t('Login'));
-		common_element('input', array('name' => 'cancel',
-									  'type' => 'button',
-									  'id' => 'cancel'),
-					   _t('Cancel'));
+		# too much common patterns here... abstractable?
+		common_input('nickname', _t('Nickname'),
+					 ($this->arg('nickname')) ? $this->arg('nickname') : $profile->nickname);
+		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);
+		common_input('homepage', _t('Homepage'),
+					 ($this->arg('homepage')) ? $this->arg('homepage') : $profile->homepage);
+		common_textarea('bio', _t('Bio'),
+						($this->arg('bio')) ? $this->arg('bio') : $profile->bio);
+		common_input('location', _t('Location'),
+					 ($this->arg('location')) ? $this->arg('location') : $profile->location);
+		common_submit('submit', _t('Save'));
 		common_element_end('form');
 		common_show_footer();
 	}
-	
+
 	function handle_post() {
-		$nickname = $this->arg('nickname');
-		$fullname = $this->arg('fullname');
-		$email = $this->arg('email');
-		$homepage = $this->arg('homepage');
-		$bio = $this->arg('bio');
-		$location = $this->arg('location');
+		
+		$nickname = $this->trimmed('nickname');
+		$fullname = $this->trimmed('fullname');
+		$email = $this->trimmed('email');
+		$homepage = $this->trimmed('homepage');
+		$bio = $this->trimmed('bio');
+		$location = $this->trimmed('location');
 
+		# Some validation
+		
+		if (!Validate::email($email, true)) {
+			$this->show_form(_t('Not a valid email address.'));
+			return;
+		} else if (!Validate::string($nickname, array('min_length' => 1,
+													  'max_length' => 64,
+													  'format' => VALIDATE_NUM . VALIDATE_ALPHA_LOWER))) {
+			$this->show_form(_t('Nickname must have only letters and numbers and no spaces.'));
+			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.'));
+			return;
+		} else if (!is_null($fullname) && strlen($fullname) > 255) {
+			$this->show_form(_t('Fullname is too long (max 255 chars).'));
+			return;
+		} else if (!is_null($bio) && strlen($bio) > 140) {
+			$this->show_form(_t('Bio is too long (max 140 chars).'));
+			return;
+		} else if (!is_null($location) && strlen($location) > 255) {
+			$this->show_form(_t('Location is too long (max 255 chars).'));
+			return;
+		} else if ($this->nickname_exists($nickname)) {
+			$this->show_form(_t('Nickname already exists.'));
+			return;
+		} else if ($this->email_exists($email)) {
+			$this->show_form(_t('Email address already exists.'));
+			return;
+		}
+		
 		$user = common_current_user();
 		assert(!is_null($user)); # should already be checked
-		
-		# FIXME: scrub input
+
 		# FIXME: transaction!
-		
-		$user->nickname = $this->arg('nickname');
-		$user->email = $this->arg('email');
-		
-		if (!$user->update()) {
+
+		$original = clone($user);
+
+		$user->nickname = $nickname;
+		$user->email = $email;
+
+		if (!$user->update($original)) {
 			common_server_error(_t('Couldnt update user.'));
 			return;
 		}
 
 		$profile = $user->getProfile();
 
+		$orig_profile = clone($profile);
+
 		$profile->nickname = $user->nickname;
-		$profile->fullname = $this->arg('fullname');
-		$profile->homepage = $this->arg('homepage');
-		$profile->bio = $this->arg('bio');
-		$profile->location = $this->arg('location');
+		$profile->fullname = $fullname;
+		$profile->homepage = $homepage;
+		$profile->bio = $bio;
+		$profile->location = $location;
+		$profile->profileurl = common_profile_url($nickname);
 
-		if (!$profile->update()) {
+		if (FALSE === $profile->update($orig_profile)) {
 			common_server_error(_t('Couldnt save profile.'));
 			return;
 		}
-		
+
 		$this->show_form(_t('Settings saved.'), TRUE);
 	}
+	
+	function nickname_exists($nickname) {
+		$user = common_current_user();
+		$other = User::staticGet('nickname', $nickname);
+		if (!$other) {
+			return false;
+		} else {
+			return $other->id != $user->id;
+		}
+	}
+	
+	function email_exists($email) {
+		$user = common_current_user();
+		$other = User::staticGet('email', $email);
+		if (!$other) {
+			return false;
+		} else {
+			return $other->id != $user->id;
+		}
+	}
 }
\ No newline at end of file