Qual: On effectue la creation des contrats en transactions

This commit is contained in:
Laurent Destailleur 2005-09-09 21:25:53 +00:00
parent 23e83aa0b1
commit b5d5fb9485
2 changed files with 53 additions and 43 deletions

View File

@ -46,6 +46,7 @@ class Contrat
var $user_service;
var $user_cloture;
var $date_creation;
var $date_validation;
var $date_cloture;
var $commercial_signature_id;
@ -218,7 +219,7 @@ class Contrat
/**
* \brief Chargement depuis la base des données du contrat
* \param id Id du contrat à charger
* \return int <0 si KO, >0 si OK
* \return int <0 si ko, id du contrat chargé si ok
*/
function fetch($id)
{
@ -235,6 +236,8 @@ class Contrat
{
$result = $this->db->fetch_array($resql);
if ($result)
{
$this->id = $result["rowid"];
$this->ref = $result["rowid"];
$this->statut = $result["statut"];
@ -258,7 +261,14 @@ class Contrat
$this->db->free($resql);
return 1;
return $this->id;
}
else
{
dolibarr_syslog("Contrat::Fetch Erreur contrat non trouve");
$this->error="Contrat non trouve";
return -2;
}
}
else
{
@ -382,14 +392,12 @@ class Contrat
/**
* \brief Crée un contrat vierge en base
* \param user Utilisateur qui crée
* \param lang Environnement langue de l'utilisateur
* \param langs Environnement langue de l'utilisateur
* \param conf Environnement de configuration lors de l'opération
* \return int < 0 si erreur, id contrat créé sinon
* \return int <0 si erreur, id contrat créé sinon
*/
function create($user,$lang='',$conf='')
function create($user,$langs='',$conf='')
{
global $langs;
// Controle validité des paramètres
$paramsok=1;
if ($this->commercial_signature_id <= 0)
@ -407,6 +415,8 @@ class Contrat
}
if (! $paramsok) return -1;
$this->db->begin();
// Insère contrat
$sql = "INSERT INTO ".MAIN_DB_PREFIX."contrat (datec, fk_soc, fk_user_author, fk_commercial_signature, fk_commercial_suivi, date_contrat)";
$sql.= " VALUES (now(),".$this->soc_id.",".$user->id;
@ -424,16 +434,18 @@ class Contrat
$interface->run_triggers('CONTRACT_CREATE',$this,$user,$lang,$conf);
// Fin appel triggers
$result = $this->id;
$this->db->commit();
return $this->id;
}
else
{
$this->error=$lang->trans("UnknownError: ".$this->db->error()." - sql=".$sql);
dolibarr_syslog("Contrat::create - 10");
$result = -1;
}
$this->db->rollback();
return $result;
dolibarr_syslog("Contrat::create - 10");
$this->error=$lang->trans("UnknownError: ".$this->db->error()." - sql=".$sql);
return -1;
}
}
@ -456,14 +468,12 @@ class Contrat
$interface->run_triggers('CONTRACT_DELETE',$this,$user,$lang,$conf);
// Fin appel triggers
$result = 1;
return 1;
}
else
{
$result = -1;
return -1;
}
return $result;
}

View File

@ -94,7 +94,7 @@ if ($_POST["action"] == 'add')
$contrat->add_product($_POST["idprod4"],$_POST["qty4"],$_POST["remise_percent4"]);
*/
$result = $contrat->create($user,$langs,$conf);
if ($result >= 0)
if ($result > 0)
{
Header("Location: fiche.php?id=".$contrat->id);
exit;
@ -378,7 +378,7 @@ if ($_GET["action"] == 'create')
print '<tr><td>'.$langs->trans("Comment").'</td><td valign="top">';
print '<textarea name="note" wrap="soft" cols="60" rows="3"></textarea></td></tr>';
print '<tr><td colspan="2" align="center"><input type="submit" value="'.$langs->trans("Create").'"></td></tr>';
print '<tr><td colspan="2" align="center"><input type="submit" class="button" value="'.$langs->trans("Create").'"></td></tr>';
print "</form>\n";
print "</table>\n";
@ -464,9 +464,9 @@ else
{
$contrat = New Contrat($db);
$result=$contrat->fetch($id);
if ($result <= 0)
if ($result < 0)
{
dolibarr_print_error($db);
dolibarr_print_error($db,$contrat->error);
exit;
}