Mutualize code

This commit is contained in:
Laurent Destailleur 2011-06-24 07:25:15 +00:00
parent 8196abfb64
commit 4a81ce8f93

View File

@ -365,7 +365,7 @@ class Societe extends CommonObject
* @param call_trigger 0=non, 1=oui * @param call_trigger 0=non, 1=oui
* @param allowmodcodeclient Inclut modif code client et code compta * @param allowmodcodeclient Inclut modif code client et code compta
* @param allowmodcodefournisseur Inclut modif code fournisseur et code compta fournisseur * @param allowmodcodefournisseur Inclut modif code fournisseur et code compta fournisseur
* @return int <0 si ko, >=0 si ok * @return int <0 if KO, >=0 if OK
*/ */
function update($id, $user='', $call_trigger=1, $allowmodcodeclient=0, $allowmodcodefournisseur=0) function update($id, $user='', $call_trigger=1, $allowmodcodeclient=0, $allowmodcodefournisseur=0)
{ {
@ -517,54 +517,20 @@ class Societe extends CommonObject
$sql .= " WHERE rowid = '" . $id ."'"; $sql .= " WHERE rowid = '" . $id ."'";
dol_syslog("Societe::update sql=".$sql); dol_syslog(get_class($this)."::Update sql=".$sql);
$resql=$this->db->query($sql); $resql=$this->db->query($sql);
if ($resql) if ($resql)
{ {
// Si le fournisseur est classe on l'ajoute // Si le fournisseur est classe on l'ajoute
$this->AddFournisseurInCategory($this->fournisseur_categorie); $this->AddFournisseurInCategory($this->fournisseur_categorie);
// Add/Update extra fields $result=$this->insertExtraFields();
// TODO Run a method into commonobject if ($result < 0)
if (sizeof($this->array_options) > 0)
{ {
$sql_del = "DELETE FROM ".MAIN_DB_PREFIX."societe_extrafields WHERE fk_object = ".$this->id; $error++;
dol_syslog(get_class($this)."::update sql=".$sql_del);
$this->db->query($sql_del);
$sql = "INSERT INTO ".MAIN_DB_PREFIX."societe_extrafields (fk_object";
foreach($this->array_options as $key => $value)
{
// Add field of attribut
$sql.=",".substr($key,8); // Remove 'options_' prefix
}
$sql .= ") VALUES (".$this->id;
foreach($this->array_options as $key => $value)
{
// Add field o fattribut
if ($this->array_options[$key] != '')
{
$sql.=",'".$this->array_options[$key]."'";
}
else
{
$sql.=",null";
}
}
$sql.=")";
dol_syslog(get_class($this)."::update update option sql=".$sql);
$resql = $this->db->query($sql);
if (! $resql)
{
$this->error=$this->db->error();
dol_syslog(get_class($this)."::update ".$this->error,LOG_ERR);
$this->db->rollback();
return -2;
}
} }
if ($call_trigger) if (! $error && $call_trigger)
{ {
// Appel des triggers // Appel des triggers
include_once(DOL_DOCUMENT_ROOT . "/core/class/interfaces.class.php"); include_once(DOL_DOCUMENT_ROOT . "/core/class/interfaces.class.php");
@ -574,9 +540,17 @@ class Societe extends CommonObject
// Fin appel triggers // Fin appel triggers
} }
dol_syslog("Societe::Update success"); if (! $error)
$this->db->commit(); {
return 1; dol_syslog(get_class($this)."::Update success");
$this->db->commit();
return 1;
}
else
{
$this->db->rollback();
return -1;
}
} }
else else
{ {
@ -590,7 +564,7 @@ class Societe extends CommonObject
{ {
$this->error = $langs->trans("Error sql=".$sql); $this->error = $langs->trans("Error sql=".$sql);
dol_syslog("Societe::Update fails update sql=".$sql, LOG_ERR); dol_syslog(get_class($this)."::Update fails update sql=".$sql, LOG_ERR);
$result = -2; $result = -2;
} }
$this->db->rollback(); $this->db->rollback();
@ -600,7 +574,7 @@ class Societe extends CommonObject
else else
{ {
$this->db->rollback(); $this->db->rollback();
dol_syslog("Societe::Update fails verify ".join(',',$this->errors), LOG_WARNING); dol_syslog(get_class($this)."::Update fails verify ".join(',',$this->errors), LOG_WARNING);
return -3; return -3;
} }
} }