Fix: test phpunit error

This commit is contained in:
Regis Houssin 2011-12-31 11:30:49 +01:00
parent 700c83125e
commit a3a3db58e3
2 changed files with 10 additions and 10 deletions

View File

@ -229,7 +229,7 @@ class Contact extends CommonObject
$sql = "UPDATE ".MAIN_DB_PREFIX."socpeople SET "; $sql = "UPDATE ".MAIN_DB_PREFIX."socpeople SET ";
if ($this->socid > 0) $sql .= " fk_soc='".$this->db->escape($this->socid)."',"; 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 .= " civilite='".$this->db->escape($this->civilite_id)."'";
$sql .= ", name='".$this->db->escape($this->lastname)."'"; $sql .= ", name='".$this->db->escape($this->lastname)."'";
$sql .= ", firstname='".$this->db->escape($this->firstname)."'"; $sql .= ", firstname='".$this->db->escape($this->firstname)."'";
@ -415,13 +415,13 @@ class Contact extends CommonObject
//check existing //check existing
$sql_check = "SELECT * FROM ".MAIN_DB_PREFIX."user_alert WHERE type=1 AND fk_contact=".$id." AND fk_user=".$user->id; $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); $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 //insert
$sql = "INSERT into ".MAIN_DB_PREFIX."user_alert(type,fk_contact,fk_user) "; $sql = "INSERT INTO ".MAIN_DB_PREFIX."user_alert(type,fk_contact,fk_user) ";
$sql.= "values (1,".$id.",".$user->id.")"; $sql.= "VALUES (1,".$id.",".$user->id.")";
$result = $this->db->query($sql); $result = $this->db->query($sql);
if (!$result) if (! $result)
{ {
$error++; $error++;
$this->error=$this->db->lasterror(); $this->error=$this->db->lasterror();
@ -434,8 +434,8 @@ class Contact extends CommonObject
} }
else else
{ {
$sql = "DELETE from ".MAIN_DB_PREFIX."user_alert "; $sql = "DELETE FROM ".MAIN_DB_PREFIX."user_alert ";
$sql.= "where type=1 AND fk_contact=".$id." AND fk_user=".$user->id; $sql.= "WHERE type=1 AND fk_contact=".$id." AND fk_user=".$user->id;
$result = $this->db->query($sql); $result = $this->db->query($sql);
if (! $result) if (! $result)
{ {

View File

@ -179,10 +179,10 @@ class ContactTest extends PHPUnit_Framework_TestCase
$localobject->default_lang='es_ES'; $localobject->default_lang='es_ES';
$result=$localobject->update($localobject->id,$user); $result=$localobject->update($localobject->id,$user);
print __METHOD__." id=".$localobject->id." result=".$result."\n"; 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); $result=$localobject->update_note($localobject->note);
print __METHOD__." id=".$localobject->id." result=".$result."\n"; 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); //$result=$localobject->update_note_public($localobject->note_public);
//print __METHOD__." id=".$localobject->id." result=".$result."\n"; //print __METHOD__." id=".$localobject->id." result=".$result."\n";
//$this->assertLessThan($result, 0); //$this->assertLessThan($result, 0);
@ -190,7 +190,7 @@ class ContactTest extends PHPUnit_Framework_TestCase
$newobject=new Contact($this->savdb); $newobject=new Contact($this->savdb);
$result=$newobject->fetch($localobject->id); $result=$newobject->fetch($localobject->id);
print __METHOD__." id=".$localobject->id." result=".$result."\n"; 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, $newobject->note);
//$this->assertEquals($localobject->note_public, $newobject->note_public); //$this->assertEquals($localobject->note_public, $newobject->note_public);