diff --git a/htdocs/compta/facture.php b/htdocs/compta/facture.php
index 30eab293a6e..c82678cdfe6 100644
--- a/htdocs/compta/facture.php
+++ b/htdocs/compta/facture.php
@@ -67,8 +67,27 @@ $NBLINES=4;
/* Actions */
/******************************************************************************/
+if ($_GET['action'] == 'reopen' && $user->rights->facture->creer)
+{
+ $fac = new Facture($db);
+ $result = $fac->fetch($_GET['facid']);
+ if ($fac->statut == 3 && ($fac->close_code == 'badcustomer' || $fac->close_code == 'abandon'))
+ {
+ $result = $fac->set_unpayed($user);
+ if ($result > 0)
+ {
+ Header('Location: '.$_SERVER["PHP_SELF"].'?facid='.$_GET['facid']);
+ exit;
+ }
+ else
+ {
+ $mesg=''.$fac->error.'
';
+ }
+ }
+}
+
// Suppression de la facture
-if ($_POST['action'] == 'confirm_delete' && $_POST['confirm'] == 'yes')
+if ($_POST['action'] == 'confirm_delete' && $_POST['confirm'] == 'yes' && $user->rights->facture->supprimer)
{
if ($user->rights->facture->supprimer)
{
@@ -2777,7 +2796,12 @@ else
}
}
- // Récurrente
+ // Reopen a classified invoice
+ if ($fac->statut == 3 && ($fac->close_code == 'badcustomer' || $fac->close_code == 'abandon'))
+ {
+ print ''.$langs->trans('ReOpen').'';
+ }
+ // Récurrente
if (! $conf->global->FACTURE_DISABLE_RECUR && $fac->type == 0)
{
if (! $facidnext)
diff --git a/htdocs/facture.class.php b/htdocs/facture.class.php
index debe146ba50..ea7426c828f 100644
--- a/htdocs/facture.class.php
+++ b/htdocs/facture.class.php
@@ -80,7 +80,7 @@ class Facture extends CommonObject
var $paye;
//! id facture source si facture de remplacement ou avoir
var $fk_facture_source;
- //! Fermeture apres paiement partiel: discount_vat, bad_customer, abandon
+ //! Fermeture apres paiement partiel: discount_vat, badcustomer, abandon
//! Fermeture alors que aucun paiement: replaced (si remplacé), abandon
var $close_code;
//! Commentaire si mis a paye sans paiement complet
@@ -169,7 +169,7 @@ class Facture extends CommonObject
$this->remise_absolue = $_facrec->remise_absolue;
$this->remise_percent = $_facrec->remise_percent;
$this->remise = $_facrec->remise;
-
+
// Nettoyage parametres
if (! $this->type) $this->type = 0;
$this->ref_client=trim($this->ref_client);
@@ -225,7 +225,7 @@ class Facture extends CommonObject
dolibarr_syslog("Facture::create sql=".$sql);
$resql=$this->db->query($sql);
if (! $resql) $error++;
-
+
// Mise a jour lien avec propal ou commande
if (! $error && $this->id && $this->propalid)
{
@@ -338,9 +338,10 @@ class Facture extends CommonObject
/**
- \brief Création de la facture en base depuis une autre
- \param user Object utilisateur qui crée
- \return int <0 si ko, >0 si ok
+ \brief Create a new invoice in database from current invoice
+ \param user Object user that ask creation
+ \param invertdetail Reverse sign of amounts for lines
+ \return int <0 si ko, >0 si ok
*/
function create_clone($user,$invertdetail=0)
{
@@ -367,13 +368,13 @@ class Facture extends CommonObject
if ($invertdetail)
{
foreach($facture->lignes as $i => $line)
- {
- $facture->lignes[$i]->subprice = -$facture->lignes[$i]->subprice;
- $facture->lignes[$i]->price = -$facture->lignes[$i]->price;
- $facture->lignes[$i]->total_ht = -$facture->lignes[$i]->total_ht;
- $facture->lignes[$i]->total_tva = -$facture->lignes[$i]->total_tva;
- $facture->lignes[$i]->total_ttc = -$facture->lignes[$i]->total_ttc;
- }
+ {
+ $facture->lignes[$i]->subprice = -$facture->lignes[$i]->subprice;
+ $facture->lignes[$i]->price = -$facture->lignes[$i]->price;
+ $facture->lignes[$i]->total_ht = -$facture->lignes[$i]->total_ht;
+ $facture->lignes[$i]->total_tva = -$facture->lignes[$i]->total_tva;
+ $facture->lignes[$i]->total_ttc = -$facture->lignes[$i]->total_ttc;
+ }
}
dolibarr_syslog("Facture::create_clone invertdetail=".$invertdetail." socid=".$this->socid." nboflines=".sizeof($facture->lignes));
@@ -383,8 +384,8 @@ class Facture extends CommonObject
return $facid;
}
-
-
+
+
/**
\brief Renvoie nom clicable (avec eventuellement le picto)
\param withpicto 0=Pas de picto, 1=Inclut le picto dans le lien, 2=Picto seul
@@ -628,7 +629,7 @@ class Facture extends CommonObject
$this->db->rollback();
return -5;
}
-
+
$facligne=new FactureLigne($this->db);
$facligne->fk_facture=$this->id;
$facligne->fk_remise_except=$remise->id;
@@ -913,8 +914,9 @@ class Facture extends CommonObject
/**
* \brief Tag la facture comme non payée complètement + appel trigger BILL_UNPAYED
- * Fonction utilisée quand un paiement prélevement est refusé.
- * \param user Objet utilisateur qui modifie
+ * Fonction utilisée quand un paiement prélevement est refusé,
+ * ou quand une facture annulée et réouverte.
+ * \param user Object user that change status
* \return int <0 si ok, >0 si ok
*/
function set_unpayed($user)
@@ -923,10 +925,11 @@ class Facture extends CommonObject
dolibarr_syslog("Facture::set_unpayed rowid=".$this->id, LOG_DEBUG);
$sql = 'UPDATE '.MAIN_DB_PREFIX.'facture';
- $sql.= ' SET paye=0, fk_statut=1';
+ $sql.= ' SET paye=0, fk_statut=1, close_code=null, close_note=null';
$sql.= ' WHERE rowid = '.$this->id;
$resql = $this->db->query($sql);
-
+
+ dolibarr_syslog("Facture::set_unpayed sql=".$sql);
if ($resql)
{
$this->use_webcal=($conf->global->PHPWEBCALENDAR_BILLSTATUS=='always'?1:0);
@@ -1019,7 +1022,7 @@ class Facture extends CommonObject
$this->fetch_client();
$this->fetch_lines();
-
+
// Verification paramètres
if ($this->type == 1) // si facture de remplacement
{
@@ -1118,7 +1121,7 @@ class Facture extends CommonObject
if (file_exists($dirsource))
{
dolibarr_syslog("Facture::set_valid() renommage rep ".$dirsource." en ".$dirdest);
-
+
if (@rename($dirsource, $dirdest))
{
dolibarr_syslog("Renommage ok");
@@ -1156,13 +1159,13 @@ class Facture extends CommonObject
if ($conf->stock->enabled && $conf->global->STOCK_CALCULATE_ON_BILL)
{
require_once(DOL_DOCUMENT_ROOT."/product/stock/mouvementstock.class.php");
-
+
for ($i = 0 ; $i < sizeof($this->lignes) ; $i++)
{
if ($this->lignes[$i]->fk_product && $this->lignes[$i]->product_type == 0)
{
dolibarr_syslog("Facture::set_valid() correct stock for ".$this->lignes[$i]->rowid);
-
+
// It's a product
if ($conf->global->PRODUIT_SOUSPRODUITS)
{
@@ -1332,7 +1335,7 @@ class Facture extends CommonObject
{
$pu=$pu_ttc;
}
-
+
// Calcul du total TTC et de la TVA pour la ligne a partir de
// qty, pu, remise_percent et txtva
// TRES IMPORTANT: C'est au moment de l'insertion ligne qu'on doit stocker
@@ -1351,7 +1354,7 @@ class Facture extends CommonObject
$remise = round(($pu * $remise_percent / 100),2);
$price = ($pu - $remise);
}
-
+
$product_type=0;
if ($fk_product)
{
@@ -1482,7 +1485,7 @@ class Facture extends CommonObject
$ligne->total_tva=$total_tva;
$ligne->total_ttc=$total_ttc;
$ligne->info_bits=$info_bits;
-
+
// A ne plus utiliser
$ligne->price=$price;
$ligne->remise=$remise;
@@ -1540,7 +1543,7 @@ class Facture extends CommonObject
$this->db->rollback();
return -1;
}
-
+
// Efface ligne de facture
$sql = 'DELETE FROM '.MAIN_DB_PREFIX.'facturedet WHERE rowid = '.$rowid;
dolibarr_syslog("Facture::Deleteline sql=".$sql);
@@ -1552,18 +1555,18 @@ class Facture extends CommonObject
$this->db->rollback();
return -1;
}
-
+
$result=$this->update_price();
-
+
// Appel des triggers
include_once(DOL_DOCUMENT_ROOT . "/interfaces.class.php");
$interface=new Interfaces($this->db);
$result = $interface->run_triggers('LINEBILL_DELETE',$this,$user,$langs,$conf);
if ($result < 0) { $error++; $this->errors=$interface->errors; }
// Fin appel triggers
-
+
$this->db->commit();
-
+
return 1;
}
@@ -2737,7 +2740,7 @@ class FactureLigne
return -3;
}
}
-
+
if (! $notrigger)
{
// Appel des triggers
@@ -2747,7 +2750,7 @@ class FactureLigne
if ($result < 0) { $error++; $this->errors=$interface->errors; }
// Fin appel triggers
}
-
+
$this->db->commit();
return $this->rowid;
diff --git a/htdocs/includes/modules/modFacture.class.php b/htdocs/includes/modules/modFacture.class.php
index bc1777bafcf..7be8170c6d2 100644
--- a/htdocs/includes/modules/modFacture.class.php
+++ b/htdocs/includes/modules/modFacture.class.php
@@ -114,7 +114,7 @@ class modFacture extends DolibarrModules
$r++;
$this->rights[$r][0] = 12;
- $this->rights[$r][1] = 'Cr�er les factures';
+ $this->rights[$r][1] = 'Creer les factures';
$this->rights[$r][2] = 'a';
$this->rights[$r][3] = 0;
$this->rights[$r][4] = 'creer';
diff --git a/htdocs/langs/en_US/main.lang b/htdocs/langs/en_US/main.lang
index 53e9eaf1d26..a18c71e1d81 100644
--- a/htdocs/langs/en_US/main.lang
+++ b/htdocs/langs/en_US/main.lang
@@ -103,6 +103,7 @@ Show=Show
Search=Search
Valid=Valid
Approve=Approve
+ReOpen=Re-Open
Upload=Send file
Select=Select
Choose=Choose
diff --git a/htdocs/langs/fr_BE/main.lang b/htdocs/langs/fr_BE/main.lang
index c0cd15de494..c9e1334460f 100644
--- a/htdocs/langs/fr_BE/main.lang
+++ b/htdocs/langs/fr_BE/main.lang
@@ -78,6 +78,7 @@ Show=Voir
Search=Rechercher
Valid=Valider
Approve=Approuver
+ReOpen=Réouvrir
Upload=Envoyer fichier
Select=Sélectionner
Choose=Choisir
diff --git a/htdocs/langs/fr_FR/main.lang b/htdocs/langs/fr_FR/main.lang
index 86a4cf42aee..da312d2263c 100644
--- a/htdocs/langs/fr_FR/main.lang
+++ b/htdocs/langs/fr_FR/main.lang
@@ -104,6 +104,7 @@ Show=Voir
Search=Rechercher
Valid=Valider
Approve=Approuver
+ReOpen=Réouvrir
Upload=Envoyer fichier
Select=Sélectionner
Choose=Choisir