diff --git a/htdocs/livraison/fiche.php b/htdocs/livraison/fiche.php
index 1e26f4837c4..d55d16b01a8 100644
--- a/htdocs/livraison/fiche.php
+++ b/htdocs/livraison/fiche.php
@@ -23,7 +23,7 @@
/**
* \file htdocs/livraison/fiche.php
* \ingroup livraison
- * \brief Fiche descriptive d'un bon de livraison
+ * \brief Fiche descriptive d'un bon de livraison=reception
* \version $Id$
*/
@@ -98,7 +98,7 @@ if ($_POST["action"] == 'add')
}
}
-if ($_POST["action"] == 'confirm_valid' && $_POST["confirm"] == 'yes' && $user->rights->expedition->livraison->valider)
+if ($_REQUEST["action"] == 'confirm_valid' && $_REQUEST["confirm"] == 'yes' && $user->rights->expedition->livraison->valider)
{
$livraison = new Livraison($db);
$livraison->fetch($_GET["id"]);
@@ -106,14 +106,18 @@ if ($_POST["action"] == 'confirm_valid' && $_POST["confirm"] == 'yes' && $user->
//$livraison->PdfWrite();
}
-if ($_POST["action"] == 'confirm_delete' && $_POST["confirm"] == 'yes')
+if ($_REQUEST["action"] == 'confirm_delete' && $_REQUEST["confirm"] == 'yes' && $user->rights->expedition->livraison->supprimer)
{
- if ($user->rights->expedition->livraison->supprimer )
+ $livraison = new Livraison($db);
+ $livraison->fetch($_GET["id"]);
+ $expedition_id = $livraison->expedition_id;
+
+ $db->begin();
+ $result=$livraison->delete();
+
+ if ($result > 0)
{
- $livraison = new Livraison($db);
- $livraison->fetch($_GET["id"]);
- $expedition_id = $_GET["expid"];
- $livraison->delete();
+ $db->commit();
if ($conf->expedition_bon->enabled)
{
Header("Location: ".DOL_URL_ROOT.'/expedition/fiche.php?id='.$expedition_id);
@@ -124,6 +128,10 @@ if ($_POST["action"] == 'confirm_delete' && $_POST["confirm"] == 'yes')
}
exit;
}
+ else
+ {
+ $db->rollback();
+ }
}
/*
@@ -396,7 +404,7 @@ else
if ($_GET["action"] == 'delete')
{
$expedition_id = $_GET["expid"];
- $ret=$html->form_confirm($_SERVER['PHP_SELF'].'?id='.$livraison->id.'&expid='.$expedition_id,$langs->trans("DeleteDeliveryReceipt"),$langs->trans("DeleteDeliveryReceiptConfirm"),'confirm_delete');
+ $ret=$html->form_confirm($_SERVER['PHP_SELF'].'?id='.$livraison->id.'&expid='.$expedition_id,$langs->trans("DeleteDeliveryReceipt"),$langs->trans("DeleteDeliveryReceiptConfirm"),'confirm_delete','','',1);
if ($ret == 'html') print '
';
}
@@ -406,7 +414,7 @@ else
*/
if ($_GET["action"] == 'valid')
{
- $ret=$html->form_confirm($_SERVER['PHP_SELF'].'?id='.$livraison->id,$langs->trans("ValidateDeliveryReceipt"),$langs->trans("ValidateDeliveryReceiptConfirm"),'confirm_valid');
+ $ret=$html->form_confirm($_SERVER['PHP_SELF'].'?id='.$livraison->id,$langs->trans("ValidateDeliveryReceipt"),$langs->trans("ValidateDeliveryReceiptConfirm"),'confirm_valid','','',1);
if ($ret == 'html') print '
';
}
@@ -552,14 +560,14 @@ else
{
print '