[Fusionforge-commits] r14812 - in branches/Branch_5_1: . src/common/include

Roland Mas lolando at fusionforge.org
Wed Nov 16 15:52:44 CET 2011


Author: lolando
Date: 2011-11-16 15:52:44 +0100 (Wed, 16 Nov 2011)
New Revision: 14812

Modified:
   branches/Branch_5_1/
   branches/Branch_5_1/src/common/include/User.class.php
Log:
Merged from Evolvis: make {set,delete}Preference method?\226?\128?\153s return value actually match its docs

Modified: branches/Branch_5_1/src/common/include/User.class.php
===================================================================
--- branches/Branch_5_1/src/common/include/User.class.php	2011-11-16 14:40:23 UTC (rev 14811)
+++ branches/Branch_5_1/src/common/include/User.class.php	2011-11-16 14:52:44 UTC (rev 14812)
@@ -1222,7 +1222,7 @@
 		$res = db_query_params ('DELETE FROM user_preferences WHERE user_id=$1 AND preference_name=$2',
 					array ($this->getID(),
 					       $preference_name)) ;
-		return $res;
+		return ((!$res || db_affected_rows($res) < 1) ? false : true);
 	}
 
 	/**
@@ -1253,9 +1253,9 @@
 								  $preference_name,
 								  $value,
 								  time())) ;
-				return $result;
 			}
 		}
+		return ((!$result || db_affected_rows($result) < 1) ? false : true);
 	}
 
 	/**




More information about the Fusionforge-commits mailing list