From a3a3db58e3d946a7bbd2cc52fede413aa320e47f Mon Sep 17 00:00:00 2001
From: Regis Houssin <regis@dolibarr.fr>
Date: Sat, 31 Dec 2011 11:30:49 +0100
Subject: [PATCH] Fix: test phpunit error

---
 htdocs/contact/class/contact.class.php | 14 +++++++-------
 test/phpunit/ContactTest.php           |  6 +++---
 2 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/htdocs/contact/class/contact.class.php b/htdocs/contact/class/contact.class.php
index 4b481e93902..0a9dae0c131 100644
--- a/htdocs/contact/class/contact.class.php
+++ b/htdocs/contact/class/contact.class.php
@@ -229,7 +229,7 @@ class Contact extends CommonObject
 
 		$sql = "UPDATE ".MAIN_DB_PREFIX."socpeople SET ";
 		if ($this->socid > 0) $sql .= " fk_soc='".$this->db->escape($this->socid)."',";
-		if ($this->socid == -1) $sql .= " fk_soc=null,";
+		else if ($this->socid == -1) $sql .= " fk_soc=null,";
 		$sql .= "  civilite='".$this->db->escape($this->civilite_id)."'";
 		$sql .= ", name='".$this->db->escape($this->lastname)."'";
 		$sql .= ", firstname='".$this->db->escape($this->firstname)."'";
@@ -415,13 +415,13 @@ class Contact extends CommonObject
 			//check existing
 			$sql_check = "SELECT * FROM ".MAIN_DB_PREFIX."user_alert WHERE type=1 AND fk_contact=".$id." AND fk_user=".$user->id;
 			$result_check = $this->db->query($sql_check);
-			if (!$result_check or ($this->db->num_rows($result_check)<1))
+			if (! $result_check || ($this->db->num_rows($result_check)<1))
 			{
 				//insert
-				$sql = "INSERT into ".MAIN_DB_PREFIX."user_alert(type,fk_contact,fk_user) ";
-				$sql.= "values (1,".$id.",".$user->id.")";
+				$sql = "INSERT INTO ".MAIN_DB_PREFIX."user_alert(type,fk_contact,fk_user) ";
+				$sql.= "VALUES (1,".$id.",".$user->id.")";
 				$result = $this->db->query($sql);
-				if (!$result)
+				if (! $result)
 				{
                     $error++;
                     $this->error=$this->db->lasterror();
@@ -434,8 +434,8 @@ class Contact extends CommonObject
 		}
 		else
 		{
-			$sql = "DELETE from ".MAIN_DB_PREFIX."user_alert ";
-			$sql.= "where type=1 AND fk_contact=".$id." AND fk_user=".$user->id;
+			$sql = "DELETE FROM ".MAIN_DB_PREFIX."user_alert ";
+			$sql.= "WHERE type=1 AND fk_contact=".$id." AND fk_user=".$user->id;
 			$result = $this->db->query($sql);
 			if (! $result)
 			{
diff --git a/test/phpunit/ContactTest.php b/test/phpunit/ContactTest.php
index f5bb8938ca6..51dba473862 100755
--- a/test/phpunit/ContactTest.php
+++ b/test/phpunit/ContactTest.php
@@ -179,10 +179,10 @@ class ContactTest extends PHPUnit_Framework_TestCase
 		$localobject->default_lang='es_ES';
 		$result=$localobject->update($localobject->id,$user);
     	print __METHOD__." id=".$localobject->id." result=".$result."\n";
-    	$this->assertLessThan($result, 0);
+    	$this->assertLessThan($result, 0, 'Contact::update error');
 		$result=$localobject->update_note($localobject->note);
     	print __METHOD__." id=".$localobject->id." result=".$result."\n";
-    	$this->assertLessThan($result, 0);
+    	$this->assertLessThan($result, 0, 'Contact::update_note error');
 		//$result=$localobject->update_note_public($localobject->note_public);
     	//print __METHOD__." id=".$localobject->id." result=".$result."\n";
     	//$this->assertLessThan($result, 0);
@@ -190,7 +190,7 @@ class ContactTest extends PHPUnit_Framework_TestCase
 		$newobject=new Contact($this->savdb);
     	$result=$newobject->fetch($localobject->id);
         print __METHOD__." id=".$localobject->id." result=".$result."\n";
-    	$this->assertLessThan($result, 0);
+    	$this->assertLessThan($result, 0, 'Contact::fetch error');
 
     	$this->assertEquals($localobject->note, $newobject->note);
     	//$this->assertEquals($localobject->note_public, $newobject->note_public);