[Fusionforge-commits] r7612 - trunk/gforge/common/include

Julien HEYMAN jheyman at libremir.placard.fr.eu.org
Wed May 6 10:41:29 CEST 2009


Author: jheyman
Date: 2009-05-06 10:41:28 +0200 (Wed, 06 May 2009)
New Revision: 7612

Modified:
   trunk/gforge/common/include/User.class.php
Log:
Fixed bug when you have a quote in email.

Modified: trunk/gforge/common/include/User.class.php
===================================================================
--- trunk/gforge/common/include/User.class.php	2009-05-05 13:12:58 UTC (rev 7611)
+++ trunk/gforge/common/include/User.class.php	2009-05-06 08:41:28 UTC (rev 7612)
@@ -824,7 +824,7 @@
 	 */
 	function setEmail($email) {
 
-		if (!strcasecmp($this->getEmail(), $email)) {
+		if (!strcasecmp($this->getEmail(), stripslashes($email))) {
 			return true;
 		}
 
@@ -857,7 +857,11 @@
 			$hook_params['user_email'] = $email;
 			plugin_hook ("user_setemail", $hook_params);
 			
-			$this->data_array['email'] = $email;
+			if (!$this->fetchData($this->getId())) {
+				db_rollback();
+				return false;
+			}
+
 			db_commit();
 			return true;
 		}




More information about the Fusionforge-commits mailing list