From c0dc4f3a48edcf1a01da1779b72c730b177319fd Mon Sep 17 00:00:00 2001 From: Philippe GRAND Date: Thu, 20 Sep 2018 13:52:57 +0200 Subject: [PATCH] Standardize and update code --- .../modules/livraison/mod_livraison_jade.php | 14 +++++--- .../livraison/mod_livraison_saphir.php | 12 +++++-- .../core/modules/payment/mod_payment_ant.php | 14 +++++--- .../modules/payment/mod_payment_cicada.php | 13 +++++-- .../product/mod_codeproduct_elephant.php | 35 +++++++++++++------ .../product/mod_codeproduct_leopard.php | 27 +++++++++----- 6 files changed, 82 insertions(+), 33 deletions(-) diff --git a/htdocs/core/modules/livraison/mod_livraison_jade.php b/htdocs/core/modules/livraison/mod_livraison_jade.php index 2bc64bdcb63..7578ed2728f 100644 --- a/htdocs/core/modules/livraison/mod_livraison_jade.php +++ b/htdocs/core/modules/livraison/mod_livraison_jade.php @@ -34,11 +34,17 @@ require_once DOL_DOCUMENT_ROOT .'/core/modules/livraison/modules_livraison.php'; class mod_livraison_jade extends ModeleNumRefDeliveryOrder { - var $version='dolibarr'; // 'development', 'experimental', 'dolibarr' - var $error = ''; - var $nom = "Jade"; + /** + * Dolibarr version of the loaded document + * @public string + */ + public $version = 'dolibarr'; // 'development', 'experimental', 'dolibarr' - var $prefix='BL'; + public $error = ''; + + public $nom = "Jade"; + + public $prefix='BL'; /** diff --git a/htdocs/core/modules/livraison/mod_livraison_saphir.php b/htdocs/core/modules/livraison/mod_livraison_saphir.php index bad82554ff3..4c43f8c6136 100644 --- a/htdocs/core/modules/livraison/mod_livraison_saphir.php +++ b/htdocs/core/modules/livraison/mod_livraison_saphir.php @@ -31,9 +31,15 @@ require_once DOL_DOCUMENT_ROOT .'/core/modules/livraison/modules_livraison.php'; */ class mod_livraison_saphir extends ModeleNumRefDeliveryOrder { - var $version='dolibarr'; // 'development', 'experimental', 'dolibarr' - var $error = ''; - var $nom = 'Saphir'; + /** + * Dolibarr version of the loaded document + * @public string + */ + public $version = 'dolibarr'; // 'development', 'experimental', 'dolibarr' + + public $error = ''; + + public $nom = 'Saphir'; /** diff --git a/htdocs/core/modules/payment/mod_payment_ant.php b/htdocs/core/modules/payment/mod_payment_ant.php index d059f0ccebc..1c6bcdc4150 100644 --- a/htdocs/core/modules/payment/mod_payment_ant.php +++ b/htdocs/core/modules/payment/mod_payment_ant.php @@ -30,9 +30,15 @@ require_once DOL_DOCUMENT_ROOT .'/core/modules/payment/modules_payment.php'; */ class mod_payment_ant extends ModeleNumRefPayments { - var $version='dolibarr'; // 'development', 'experimental', 'dolibarr' - var $error = ''; - var $nom = 'Ant'; + /** + * Dolibarr version of the loaded document + * @public string + */ + public $version = 'dolibarr'; // 'development', 'experimental', 'dolibarr' + + public $error = ''; + + public $nom = 'Ant'; /** @@ -42,7 +48,7 @@ class mod_payment_ant extends ModeleNumRefPayments */ function info() { - global $conf,$langs; + global $conf, $langs; $langs->load("bills"); diff --git a/htdocs/core/modules/payment/mod_payment_cicada.php b/htdocs/core/modules/payment/mod_payment_cicada.php index 5e87f33f8c1..1fce35f93cb 100644 --- a/htdocs/core/modules/payment/mod_payment_cicada.php +++ b/htdocs/core/modules/payment/mod_payment_cicada.php @@ -29,13 +29,20 @@ require_once DOL_DOCUMENT_ROOT .'/core/modules/payment/modules_payment.php'; */ class mod_payment_cicada extends ModeleNumRefPayments { - var $version='dolibarr'; // 'development', 'experimental', 'dolibarr' - var $prefix='PAY'; + /** + * Dolibarr version of the loaded document + * @public string + */ + public $version = 'dolibarr'; // 'development', 'experimental', 'dolibarr' + + public $prefix='PAY'; + /** * @var string Error code (or message) */ public $error=''; - var $nom='Cicada'; + + public $nom='Cicada'; /** diff --git a/htdocs/core/modules/product/mod_codeproduct_elephant.php b/htdocs/core/modules/product/mod_codeproduct_elephant.php index f55595a63d5..8910805b3cb 100644 --- a/htdocs/core/modules/product/mod_codeproduct_elephant.php +++ b/htdocs/core/modules/product/mod_codeproduct_elephant.php @@ -34,18 +34,31 @@ require_once DOL_DOCUMENT_ROOT.'/core/modules/product/modules_product.class.php' */ class mod_codeproduct_elephant extends ModeleProductCode { - var $nom='Elephant'; // Nom du modele - var $name='Elephant'; // Nom du modele - var $code_modifiable; // Code modifiable - var $code_modifiable_invalide; // Code modifiable si il est invalide - var $code_modifiable_null; // Code modifiables si il est null - var $code_null; // Code facultatif - var $version='dolibarr'; // 'development', 'experimental', 'dolibarr' - var $code_auto; // Numerotation automatique + public $nom='Elephant'; // Nom du modele - var $searchcode; // String de recherche - var $numbitcounter; // Nombre de chiffres du compteur - var $prefixIsRequired; // Le champ prefix du tiers doit etre renseigne quand on utilise {pre} + public $name='Elephant'; // Nom du modele + + public $code_modifiable; // Code modifiable + + public $code_modifiable_invalide; // Code modifiable si il est invalide + + public $code_modifiable_null; // Code modifiables si il est null + + public $code_null; // Code facultatif + + /** + * Dolibarr version of the loaded document + * @public string + */ + public $version = 'dolibarr'; // 'development', 'experimental', 'dolibarr' + + public $code_auto; // Numerotation automatique + + public $searchcode; // String de recherche + + public $numbitcounter; // Nombre de chiffres du compteur + + public $prefixIsRequired; // Le champ prefix du tiers doit etre renseigne quand on utilise {pre} /** diff --git a/htdocs/core/modules/product/mod_codeproduct_leopard.php b/htdocs/core/modules/product/mod_codeproduct_leopard.php index 3b850e1ba91..de9d281c92f 100644 --- a/htdocs/core/modules/product/mod_codeproduct_leopard.php +++ b/htdocs/core/modules/product/mod_codeproduct_leopard.php @@ -39,14 +39,25 @@ class mod_codeproduct_leopard extends ModeleProductCode * Le fonctionnement de celui-ci doit donc rester le plus ouvert possible */ - var $nom='Leopard'; // Nom du modele - var $name='Leopard'; // Nom du modele - var $code_modifiable; // Code modifiable - var $code_modifiable_invalide; // Code modifiable si il est invalide - var $code_modifiable_null; // Code modifiables si il est null - var $code_null; // Code facultatif - var $version='dolibarr'; // 'development', 'experimental', 'dolibarr' - var $code_auto; // Numerotation automatique + public $nom='Leopard'; // Nom du modele + + public $name='Leopard'; // Nom du modele + + public $code_modifiable; // Code modifiable + + public $code_modifiable_invalide; // Code modifiable si il est invalide + + public $code_modifiable_null; // Code modifiables si il est null + + public $code_null; // Code facultatif + + /** + * Dolibarr version of the loaded document + * @public string + */ + public $version = 'dolibarr'; // 'development', 'experimental', 'dolibarr' + + public $code_auto; // Numerotation automatique /**