Merge pull request #9264 from grandoc/new_branch_17_08_2018

standardize code

htdocs/accountancy/class/accountancysystem.class.php

@@ -29,7 +29,11 @@
 class AccountancySystem
 {
 	var $db;
-	var $error;
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
+	
 	var $rowid;
 	var $fk_pcg_version;
 	var $pcg_type;

htdocs/accountancy/class/accountingaccount.class.php

@@ -46,8 +46,18 @@ class AccountingAccount extends CommonObject
 	public $restrictiononfksoc = 1;
 
 	var $db;
-	var $error;
-	var $errors;
+	
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
+	
+	/**
+	 *
+	 * @var string[] Error codes (or messages)
+	 */
+	public $errors = array ();
+	
 	var $id;
 	var $rowid;
 	var $datec; // Creation date

htdocs/adherents/canvas/actions_adherentcard_common.class.php

@@ -41,10 +41,16 @@ abstract class ActionsAdherentCardCommon
 	var $tpl = array();
 	//! Object container
 	var $object;
-	//! Error string
-	var $error;
-	//! Error array
-	var $errors=array();
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
+	
+	/**
+	 *
+	 * @var string[] Error codes (or messages)
+	 */
+	public $errors = array ();
 
 
 	/**

htdocs/comm/action/class/cactioncomm.class.php

@@ -28,7 +28,11 @@
  */
 class CActionComm
 {
-    var $error;
+    /**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
+	
     var $db;
 
     var $id;

htdocs/comm/mailing/class/advtargetemailing.class.php

@@ -30,8 +30,17 @@ class AdvanceTargetingMailing extends CommonObject
 {
 
 	var $db; //!< To store db handler
-	var $error; //!< To return error code (or message)
-	var $errors = array(); //!< To return several error codes (or messages)
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
+	
+	/**
+	 *
+	 * @var string[] Error codes (or messages)
+	 */
+	public $errors = array ();
+	
 	var $element='advtargetemailing';			//!< Id that identify managed objects
 	var $table_element='advtargetemailing';	//!< Name of table without prefix where object is stored
 

htdocs/comm/mailing/class/html.formadvtargetemailing.class.php

@@ -27,7 +27,10 @@
 class FormAdvTargetEmailing extends Form
 {
 	var $db;
-	var $error;
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 	/**
 	 * Constructor

htdocs/compta/facture/class/paymentterm.class.php

@@ -30,7 +30,12 @@ class PaymentTerm // extends CommonObject
 {
 	var $db;							//!< To store db handler
 	var $error;							//!< To return error code (or message)
-	var $errors=array();				//!< To return several error codes (or messages)
+	/**
+	 *
+	 * @var string[] Error codes (or messages)
+	 */
+	public $errors = array ();
+	
 	//public  $element='c_payment_term';			//!< Id that identify managed objects
 	//public  $table_element='c_payment_term';	//!< Name of table without prefix where object is stored
 	var $context =array();

htdocs/contact/canvas/actions_contactcard_common.class.php

@@ -39,8 +39,12 @@ abstract class ActionsContactCardCommon
 	var $object;
 	//! Error string
 	var $error;
-	//! Error array
-	var $errors=array();
+	
+	/**
+	 *
+	 * @var string[] Error codes (or messages)
+	 */
+	public $errors = array ();
 
 
 	/**

htdocs/core/class/antivir.class.php

@@ -32,7 +32,12 @@
 class AntiVir
 {
 	var $error;
-	var $errors;
+	/**
+	 *
+	 * @var string[] Error codes (or messages)
+	 */
+	public $errors = array ();;
+	
 	var $output;
 	var $db;
 

htdocs/core/class/canvas.class.php

@@ -30,7 +30,11 @@ class Canvas
 {
 	var $db;
 	var $error;
-	var $errors=array();
+	/**
+	 *
+	 * @var string[] Error codes (or messages)
+	 */
+	public $errors = array ();
 
 	var $actiontype;
 

htdocs/core/class/ccountry.class.php

@@ -34,7 +34,12 @@ class Ccountry // extends CommonObject
 {
 	var $db;							//!< To store db handler
 	var $error;							//!< To return error code (or message)
-	var $errors=array();				//!< To return several error codes (or messages)
+	/**
+	 *
+	 * @var string[] Error codes (or messages)
+	 */
+	public $errors = array ();
+	
 	//var $element='ccountry';			//!< Id that identify managed objects
 	//var $table_element='ccountry';	//!< Name of table without prefix where object is stored
 

htdocs/core/class/cstate.class.php

@@ -34,7 +34,12 @@ class Cstate // extends CommonObject
 {
 	var $db;							//!< To store db handler
 	var $error;							//!< To return error code (or message)
-	var $errors=array();				//!< To return several error codes (or messages)
+	/**
+	 *
+	 * @var string[] Error codes (or messages)
+	 */
+	public $errors = array ();
+	
 	//var $element='cstate';			//!< Id that identify managed objects
 	//var $table_element='cstate';	    //!< Name of table without prefix where object is stored
 

htdocs/core/class/ctypent.class.php

@@ -29,7 +29,12 @@ class Ctypent // extends CommonObject
 {
 	var $db;							//!< To store db handler
 	var $error;							//!< To return error code (or message)
-	var $errors=array();				//!< To return several error codes (or messages)
+	/**
+	 *
+	 * @var string[] Error codes (or messages)
+	 */
+	public $errors = array ();
+	
 	//var $element='ctypent';			//!< Id that identify managed objects
 	//var $table_element='ctypent';	//!< Name of table without prefix where object is stored
 

htdocs/core/class/dolreceiptprinter.class.php

@@ -110,7 +110,11 @@ class dolReceiptPrinter extends Escpos
     var $printer;
     var $template;
     var $error;
-    var $errors;
+    /**
+	 *
+	 * @var string[] Error codes (or messages)
+	 */
+	public $errors = array ();
 
 
 

htdocs/core/class/hookmanager.class.php

@@ -31,7 +31,11 @@ class HookManager
 {
 	var $db;
 	var $error;
-	var $errors=array();
+	/**
+	 *
+	 * @var string[] Error codes (or messages)
+	 */
+	public $errors = array ();
 
     // Context hookmanager was created for ('thirdpartycard', 'thirdpartydao', ...)
     var $contextarray=array();

htdocs/core/class/interfaces.class.php

@@ -33,7 +33,11 @@ class Interfaces
 {
     var $db;
 	var $dir;				// Directory with all core and external triggers files
-    var $errors	= array();	// Array for errors
+    /**
+	 *
+	 * @var string[] Error codes (or messages)
+	 */
+	public $errors = array ();
 
     /**
      *	Constructor

htdocs/core/class/notify.class.php

@@ -33,7 +33,11 @@ class Notify
 	var $id;
 	var $db;
 	var $error;
-	var $errors=array();
+	/**
+	 *
+	 * @var string[] Error codes (or messages)
+	 */
+	public $errors = array ();
 
 	var $author;
 	var $ref;

htdocs/core/modules/action/modules_action.php

@@ -28,7 +28,10 @@ require_once DOL_DOCUMENT_ROOT.'/core/class/commondocgenerator.class.php';
  */
 abstract class ModeleAction extends CommonDocGenerator
 {
-    var $error='';
+    /**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
     /**
      *  Return list of active generation modules

htdocs/core/modules/bank/modules_bank.php

@@ -31,7 +31,10 @@ require_once DOL_DOCUMENT_ROOT.'/core/class/commondocgenerator.class.php';
  */
 abstract class ModeleBankAccountDoc extends CommonDocGenerator
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 
 	/**

htdocs/core/modules/barcode/doc/phpbarcode.modules.php

@@ -33,7 +33,11 @@ require_once DOL_DOCUMENT_ROOT.'/core/lib/barcode.lib.php';    // This is to inc
 class modPhpbarcode extends ModeleBarCode
 {
 	var $version='dolibarr';		// 'development', 'experimental', 'dolibarr'
-	var $error='';
+	
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 
 	/**

htdocs/core/modules/barcode/doc/tcpdfbarcode.modules.php

@@ -32,7 +32,12 @@ require_once DOL_DOCUMENT_ROOT.'/core/lib/barcode.lib.php';	   // This is to inc
 class modTcpdfbarcode extends ModeleBarCode
 {
 	var $version='dolibarr';		// 'development', 'experimental', 'dolibarr'
-	var $error='';
+	
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
+	
 	var $is2d = false;
 	
 	/**

htdocs/core/modules/barcode/modules_barcode.class.php

@@ -29,7 +29,10 @@ require_once DOL_DOCUMENT_ROOT.'/core/lib/functions.lib.php';
  */
 abstract class ModeleBarCode
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 
 	/**
@@ -50,7 +53,10 @@ abstract class ModeleBarCode
  */
 abstract class ModeleNumRefBarCode
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
     /**     Return default description of numbering model
      *

htdocs/core/modules/cheque/mod_chequereceipt_mint.php

@@ -31,7 +31,10 @@ class mod_chequereceipt_mint extends ModeleNumRefChequeReceipts
 {
 	var $version='dolibarr';		// 'development', 'experimental', 'dolibarr'
 	var $prefix='CHK';
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 	var $name='Mint';
 
 

htdocs/core/modules/cheque/modules_chequereceipts.php

@@ -36,7 +36,10 @@ require_once DOL_DOCUMENT_ROOT.'/compta/bank/class/account.class.php';   // Requ
  */
 abstract class ModeleNumRefChequeReceipts
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 	/**
 	 *	Return if a module can be used or not
@@ -119,7 +122,10 @@ abstract class ModeleNumRefChequeReceipts
  */
 abstract class ModeleChequeReceipts extends CommonDocGenerator
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 	/**
 	 *  Return list of active generation modules

htdocs/core/modules/commande/mod_commande_marbre.php

@@ -31,7 +31,10 @@ class mod_commande_marbre extends ModeleNumRefCommandes
 {
 	var $version='dolibarr';		// 'development', 'experimental', 'dolibarr'
 	var $prefix='CO';
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 	var $nom='Marbre';
 
 

htdocs/core/modules/commande/modules_commande.php

@@ -39,7 +39,10 @@ require_once DOL_DOCUMENT_ROOT.'/core/class/discount.class.php';
  */
 abstract class ModelePDFCommandes extends CommonDocGenerator
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 	/**
 	 *  Return list of active generation modules
@@ -71,7 +74,10 @@ abstract class ModelePDFCommandes extends CommonDocGenerator
 
 abstract class ModeleNumRefCommandes
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 	/**
 	 *	Return if a module can be used or not

htdocs/core/modules/contract/mod_contract_serpis.php

@@ -30,7 +30,10 @@ class mod_contract_serpis extends ModelNumRefContracts
 {
 	var $version='dolibarr';
 	var $prefix='CT';
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 	var $nom='Serpis';
 	var $code_auto=1;
 

htdocs/core/modules/contract/modules_contract.php

@@ -37,7 +37,10 @@
  */
 abstract class ModelePDFContract extends CommonDocGenerator
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 
 	/**
@@ -67,7 +70,10 @@ abstract class ModelePDFContract extends CommonDocGenerator
  */
 class ModelNumRefContracts
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 	/**
 	 *	Return if a module can be used or not

htdocs/core/modules/dons/modules_don.php

@@ -34,7 +34,10 @@ require_once DOL_DOCUMENT_ROOT.'/don/class/don.class.php';
  */
 abstract class ModeleDon extends CommonDocGenerator
 {
-    var $error='';
+    /**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
     /**
      *  Return list of active generation modules
@@ -63,7 +66,10 @@ abstract class ModeleDon extends CommonDocGenerator
  */
 abstract class ModeleNumRefDons
 {
-    var $error='';
+    /**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
     /**
      * 	Return if a module can be used or not

htdocs/core/modules/expedition/mod_expedition_safor.php

@@ -30,7 +30,10 @@ class mod_expedition_safor extends ModelNumRefExpedition
 {
 	var $version='dolibarr';
 	var $prefix='SH';
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 	var $nom='Safor';
 
 

htdocs/core/modules/expedition/modules_expedition.php

@@ -36,7 +36,10 @@
  */
 abstract class ModelePdfExpedition extends CommonDocGenerator
 {
-    var $error='';
+    /**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 
 	/**
@@ -66,7 +69,10 @@ abstract class ModelePdfExpedition extends CommonDocGenerator
  */
 abstract class ModelNumRefExpedition
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 	/** Return if a model can be used or not
 	 *

htdocs/core/modules/expensereport/mod_expensereport_jade.php

@@ -30,7 +30,10 @@ class mod_expensereport_jade extends ModeleNumRefExpenseReport
 {
 	var $version='dolibarr';		// 'development', 'experimental', 'dolibarr'
 	var $prefix='ER';
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 	var $nom='Jade';
 
 

htdocs/core/modules/expensereport/modules_expensereport.php

@@ -23,7 +23,10 @@ require_once DOL_DOCUMENT_ROOT.'/core/class/commondocgenerator.class.php';
  */
 abstract class ModeleExpenseReport extends CommonDocGenerator
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 
 	/**
@@ -73,7 +76,10 @@ function expensereport_pdf_create(DoliDB $db, ExpenseReport $object, $message, $
 
 abstract class ModeleNumRefExpenseReport
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 	/**
 	 *	Return if a module can be used or not

htdocs/core/modules/export/modules_export.php

@@ -30,7 +30,10 @@ require_once DOL_DOCUMENT_ROOT.'/core/class/commondocgenerator.class.php';
  */
 class ModeleExports extends CommonDocGenerator    // This class can't be abstract as there is instance propreties loaded by liste_modeles
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 	var $driverlabel=array();
 	var $driverversion=array();

htdocs/core/modules/facture/mod_facture_mars.php

@@ -35,7 +35,10 @@ class mod_facture_mars extends ModeleNumRefFactures
 	var $prefixreplacement='FR';
 	var $prefixdeposit='AC';
 	var $prefixcreditnote='AV';
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 
 	/**

htdocs/core/modules/facture/mod_facture_terre.php

@@ -33,7 +33,10 @@ class mod_facture_terre extends ModeleNumRefFactures
 	var $prefixinvoice='FA';
 	var $prefixcreditnote='AV';
 	var $prefixdeposit='AC';
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 
 	/**

htdocs/core/modules/facture/modules_facture.php

@@ -37,7 +37,10 @@ require_once DOL_DOCUMENT_ROOT.'/compta/bank/class/account.class.php';   // Requ
  */
 abstract class ModelePDFFactures extends CommonDocGenerator
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 	/**
 	 *  Return list of active generation modules
@@ -65,7 +68,10 @@ abstract class ModelePDFFactures extends CommonDocGenerator
  */
 abstract class ModeleNumRefFactures
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 	/**
 	 * Return if a module can be used or not

htdocs/core/modules/fichinter/mod_pacific.php

@@ -32,7 +32,10 @@ class mod_pacific extends ModeleNumRefFicheinter
 {
     var $version='dolibarr';        // 'development', 'experimental', 'dolibarr'
 	var $prefix='FI';
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 	var $nom = 'pacific';
 
 

htdocs/core/modules/fichinter/modules_fichinter.php

@@ -34,7 +34,10 @@ require_once DOL_DOCUMENT_ROOT.'/core/class/commondocgenerator.class.php';
  */
 abstract class ModelePDFFicheinter extends CommonDocGenerator
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 
 	/**
@@ -64,7 +67,10 @@ abstract class ModelePDFFicheinter extends CommonDocGenerator
  */
 abstract class ModeleNumRefFicheinter
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 	/**
 	 * 	Return if a module can be used or not

htdocs/core/modules/holiday/mod_holiday_madonna.php

@@ -31,7 +31,10 @@ class mod_holiday_madonna extends ModelNumRefHolidays
 {
 	var $version='dolibarr';
 	var $prefix='HL';
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 	var $nom='Madonna';
 	var $code_auto=1;
 

htdocs/core/modules/holiday/modules_holiday.php

@@ -38,7 +38,10 @@
  */
 abstract class ModelePDFHoliday extends CommonDocGenerator
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 
 	/**
@@ -68,7 +71,10 @@ abstract class ModelePDFHoliday extends CommonDocGenerator
  */
 class ModelNumRefHolidays
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 	/**
 	 *	Return if a module can be used or not

htdocs/core/modules/import/import_csv.modules.php

@@ -36,8 +36,16 @@ class ImportCsv extends ModeleImports
     var $db;
     var $datatoimport;
 
-	var $error='';
-	var $errors=array();
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
+	
+	/**
+	 *
+	 * @var string[] Error codes (or messages)
+	 */
+	public $errors = array ();
 
     var $id;           // Id of driver
 	var $label;        // Label of driver

htdocs/core/modules/import/import_xlsx.modules.php

@@ -36,8 +36,16 @@ class ImportXlsx extends ModeleImports
     var $db;
     var $datatoimport;
 
-	var $error='';
-	var $errors=array();
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
+	
+	/**
+	 *
+	 * @var string[] Error codes (or messages)
+	 */
+	public $errors = array ();
 
     var $id;           // Id of driver
 	var $label;        // Label of driver

htdocs/core/modules/livraison/modules_livraison.php

@@ -36,7 +36,10 @@ require_once DOL_DOCUMENT_ROOT.'/core/class/commondocgenerator.class.php';
  */
 abstract class ModelePDFDeliveryOrder extends CommonDocGenerator
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 	/**
 	 *  Return list of active generation modules
@@ -67,7 +70,10 @@ abstract class ModelePDFDeliveryOrder extends CommonDocGenerator
  */
 abstract class ModeleNumRefDeliveryOrder
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 	/**
 	 * Return if a module can be used or not

htdocs/core/modules/member/modules_cards.php

@@ -34,7 +34,10 @@ require_once DOL_DOCUMENT_ROOT.'/product/class/product.class.php';
  */
 class ModelePDFCards
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 
 	/**

htdocs/core/modules/payment/mod_payment_cicada.php

@@ -31,7 +31,10 @@ class mod_payment_cicada extends ModeleNumRefPayments
 {
 	var $version='dolibarr';		// 'development', 'experimental', 'dolibarr'
 	var $prefix='PAY';
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 	var $nom='Cicada';
 
 

htdocs/core/modules/payment/modules_payment.php

@@ -23,7 +23,10 @@
 
 abstract class ModeleNumRefPayments
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 	/**
 	 *	Return if a module can be used or not

htdocs/core/modules/printing/printgcp.modules.php

@@ -43,7 +43,12 @@ class printing_printgcp extends PrintingDriver
     var $google_id = '';
     var $google_secret = '';
     var $error;
-    var $errors = array();
+    /**
+	 *
+	 * @var string[] Error codes (or messages)
+	 */
+	public $errors = array ();
+	
     var $db;
 
     private $OAUTH_SERVICENAME_GOOGLE = 'Google';

htdocs/core/modules/printing/printipp.modules.php

@@ -41,7 +41,12 @@ class printing_printipp extends PrintingDriver
     var $user;
     var $password;
     var $error;
-    var $errors = array();
+    /**
+	 *
+	 * @var string[] Error codes (or messages)
+	 */
+	public $errors = array ();
+	
     var $db;
 
 

htdocs/core/modules/printsheet/modules_labels.php

@@ -34,7 +34,10 @@ require_once DOL_DOCUMENT_ROOT.'/product/class/product.class.php';
  */
 class ModelePDFLabels
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 
 	/**

htdocs/core/modules/product/modules_product.class.php

@@ -33,7 +33,10 @@
  */
 abstract class ModelePDFProduct extends CommonDocGenerator
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 
 	/**
@@ -58,7 +61,10 @@ abstract class ModelePDFProduct extends CommonDocGenerator
 
 abstract class ModeleProductCode
 {
-    var $error='';
+    /**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
     /**     Renvoi la description par defaut du modele de numerotation
      *

htdocs/core/modules/product_batch/modules_product_batch.class.php

@@ -38,7 +38,10 @@
  */
 abstract class ModelePDFProductBatch extends CommonDocGenerator
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 
 	/**

htdocs/core/modules/project/mod_project_simple.php

@@ -33,7 +33,10 @@ class mod_project_simple extends ModeleNumRefProjects
 {
 	var $version='dolibarr';		// 'development', 'experimental', 'dolibarr'
 	var $prefix='PJ';
-    var $error='';
+    /**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 	var $nom = "Simple";
 	var $name = "Simple";
 

htdocs/core/modules/project/modules_project.php

@@ -31,7 +31,10 @@ require_once DOL_DOCUMENT_ROOT.'/core/class/commondocgenerator.class.php';
  */
 abstract class ModelePDFProjects extends CommonDocGenerator
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 
 	/**
@@ -62,7 +65,10 @@ abstract class ModelePDFProjects extends CommonDocGenerator
  */
 abstract class ModeleNumRefProjects
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 	/**
 	 *  Return if a module can be used or not

htdocs/core/modules/project/task/mod_task_simple.php

@@ -33,7 +33,10 @@ class mod_task_simple extends ModeleNumRefTask
 {
 	var $version='dolibarr';		// 'development', 'experimental', 'dolibarr'
 	var $prefix='TK';
-    var $error='';
+    /**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 	var $nom = "Simple";
 	var $name = "Simple";
 

htdocs/core/modules/project/task/modules_task.php

@@ -32,7 +32,10 @@ require_once DOL_DOCUMENT_ROOT.'/core/class/commondocgenerator.class.php';
  */
 abstract class ModelePDFTask extends CommonDocGenerator
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 
 	/**
@@ -63,7 +66,10 @@ abstract class ModelePDFTask extends CommonDocGenerator
  */
 abstract class ModeleNumRefTask
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 	/**
 	 *  Return if a module can be used or not

htdocs/core/modules/propale/mod_propale_marbre.php

@@ -33,7 +33,10 @@ class mod_propale_marbre extends ModeleNumRefPropales
 {
 	var $version='dolibarr';		// 'development', 'experimental', 'dolibarr'
 	var $prefix='PR';
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 	var $nom = "Marbre";
 
 

htdocs/core/modules/propale/modules_propale.php

@@ -36,7 +36,10 @@ require_once DOL_DOCUMENT_ROOT.'/compta/bank/class/account.class.php';   // Requ
  */
 abstract class ModelePDFPropales extends CommonDocGenerator
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 
 	/**
@@ -66,7 +69,10 @@ abstract class ModelePDFPropales extends CommonDocGenerator
  */
 abstract class ModeleNumRefPropales
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 	/**
 	 * Return if a module can be used or not

htdocs/core/modules/security/generate/modules_genpassword.php

@@ -30,7 +30,10 @@ require_once DOL_DOCUMENT_ROOT.'/core/lib/functions.lib.php';
  */
 abstract class ModeleGenPassword
 {
-    var $error='';
+    /**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
     /**
      * 		Return if a module can be used or not

htdocs/core/modules/societe/modules_societe.class.php

@@ -33,7 +33,10 @@ require_once DOL_DOCUMENT_ROOT.'/core/class/commondocgenerator.class.php';
  */
 abstract class ModeleThirdPartyDoc extends CommonDocGenerator
 {
-    var $error='';
+    /**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
     /**
      *  Return list of active generation modules
@@ -63,7 +66,10 @@ abstract class ModeleThirdPartyDoc extends CommonDocGenerator
  */
 abstract class ModeleThirdPartyCode
 {
-    var $error='';
+    /**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
     /**     Renvoi la description par defaut du modele de numerotation
      *
@@ -255,7 +261,10 @@ abstract class ModeleThirdPartyCode
  */
 abstract class ModeleAccountancyCode
 {
-    var $error='';
+    /**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 
     /**		Return description of module

htdocs/core/modules/stock/modules_movement.php

@@ -30,7 +30,10 @@ require_once DOL_DOCUMENT_ROOT.'/core/class/commondocgenerator.class.php';
  */
 abstract class ModelePDFMovement extends CommonDocGenerator
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 
 	/**

htdocs/core/modules/stock/modules_stock.php

@@ -23,7 +23,10 @@ require_once DOL_DOCUMENT_ROOT.'/core/class/commondocgenerator.class.php';
  */
 abstract class ModelePDFStock extends CommonDocGenerator
 {
-    var $error='';
+    /**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 
     /**

htdocs/core/modules/supplier_invoice/modules_facturefournisseur.php

@@ -34,7 +34,10 @@ require_once DOL_DOCUMENT_ROOT.'/compta/bank/class/account.class.php';	// requir
  */
 abstract class ModelePDFSuppliersInvoices extends CommonDocGenerator
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 
 	/**
@@ -64,7 +67,10 @@ abstract class ModelePDFSuppliersInvoices extends CommonDocGenerator
  */
 abstract class ModeleNumRefSuppliersInvoices
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 	/**  Return if a model can be used or not
 	 *

htdocs/core/modules/supplier_order/modules_commandefournisseur.php

@@ -37,7 +37,10 @@ require_once DOL_DOCUMENT_ROOT.'/compta/bank/class/account.class.php';	// requir
  */
 abstract class ModelePDFSuppliersOrders extends CommonDocGenerator
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 
 	/**
@@ -69,7 +72,10 @@ abstract class ModelePDFSuppliersOrders extends CommonDocGenerator
  */
 abstract class ModeleNumRefSuppliersOrders
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 	/**  Return if a model can be used or not
 	 *

htdocs/core/modules/supplier_payment/mod_supplier_payment_bronan.php

@@ -31,7 +31,10 @@ class mod_supplier_payment_bronan extends ModeleNumRefSupplierPayments
 {
 	var $version='dolibarr';		// 'development', 'experimental', 'dolibarr'
 	var $prefix='SPAY';
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 	var $nom='Bronan';
 
 

htdocs/core/modules/supplier_payment/modules_supplier_payment.php

@@ -22,7 +22,10 @@ require_once DOL_DOCUMENT_ROOT.'/core/class/commondocgenerator.class.php';
  */
 abstract class ModelePDFSuppliersPayments extends CommonDocGenerator
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 
 	/**
@@ -54,7 +57,10 @@ abstract class ModelePDFSuppliersPayments extends CommonDocGenerator
 
 abstract class ModeleNumRefSupplierPayments
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 	/**
 	 *	Return if a module can be used or not

htdocs/core/modules/supplier_proposal/mod_supplier_proposal_marbre.php

@@ -33,7 +33,10 @@ class mod_supplier_proposal_marbre extends ModeleNumRefSupplierProposal
 {
 	var $version='dolibarr';		// 'development', 'experimental', 'dolibarr'
 	var $prefix='RQ';               // RQ = Request for quotation
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 	var $nom = "Marbre";
 
 

htdocs/core/modules/supplier_proposal/modules_supplier_proposal.php

@@ -36,7 +36,10 @@ require_once DOL_DOCUMENT_ROOT.'/compta/bank/class/account.class.php';   // Requ
  */
 abstract class ModelePDFSupplierProposal extends CommonDocGenerator
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 
 	/**
@@ -66,7 +69,10 @@ abstract class ModelePDFSupplierProposal extends CommonDocGenerator
  */
 abstract class ModeleNumRefSupplierProposal
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 	/**
 	 * Return if a module can be used or not

htdocs/core/modules/user/modules_user.class.php

@@ -38,7 +38,10 @@
  */
 abstract class ModelePDFUser extends CommonDocGenerator
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 
 	/**

htdocs/core/modules/usergroup/modules_usergroup.class.php

@@ -38,7 +38,10 @@
  */
 abstract class ModelePDFUserGroup extends CommonDocGenerator
 {
-	var $error='';
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
 
 
 	/**

htdocs/imports/class/import.class.php

@@ -44,7 +44,11 @@ class Import
 	var $array_import_run_sql_after;
 
 	var $error;
-	var $errors;
+	/**
+	 *
+	 * @var string[] Error codes (or messages)
+	 */
+	public $errors = array ();
 
 
 	/**

htdocs/product/dynamic_price/class/price_expression.class.php

@@ -31,7 +31,12 @@ class PriceExpression
 {
 	var $db;							//!< To store db handler
 	var $error;							//!< To return error code (or message)
-	var $errors=array();				//!< To return several error codes (or messages)
+	/**
+	 *
+	 * @var string[] Error codes (or messages)
+	 */
+	public $errors = array ();
+	
     var $id;
     var $title;
 	var $expression;

htdocs/product/dynamic_price/class/price_global_variable.class.php

@@ -31,7 +31,12 @@ class PriceGlobalVariable
 {
     var $db;							//!< To store db handler
     var $error;							//!< To return error code (or message)
-    var $errors=array();				//!< To return several error codes (or messages)
+    /**
+	 *
+	 * @var string[] Error codes (or messages)
+	 */
+	public $errors = array ();
+	
     var $id;
     var $code;
     var $description;

htdocs/product/dynamic_price/class/price_global_variable_updater.class.php

@@ -31,7 +31,12 @@ class PriceGlobalVariableUpdater
 {
     var $db;							//!< To store db handler
     var $error;							//!< To return error code (or message)
-    var $errors=array();				//!< To return several error codes (or messages)
+    /**
+	 *
+	 * @var string[] Error codes (or messages)
+	 */
+	public $errors = array ();
+	
     var $types=array(0, 1);				//!< Updater types
     var $update_min = 5;				//!< Minimal update rate
     var $id;

htdocs/projet/tasks/comment.php

@@ -278,6 +278,6 @@ if ($id > 0 || ! empty($ref))
 	}
 }
 
-
+// End of page
 llxFooter();
 $db->close();

htdocs/public/test/test_forms.php

@@ -94,6 +94,6 @@ $array=array(1=>'Value 1',2=>'Value 2',3=>'Value 3');
 $arrayselected=array(1,3);
 print $form->multiselectarray('testmulti', $array, $arrayselected, '', 0, '', 0, 250);
 
-
+// End of page
 llxFooter();
 $db->close();

htdocs/resource/card.php

@@ -401,8 +401,6 @@ else {
 	dol_print_error();
 }
 
-
-
 // End of page
 llxFooter();
 $db->close();

htdocs/resource/element_resource.php

@@ -473,6 +473,6 @@ else
 	}
 }
 
+// End of page
 llxFooter();
-
 $db->close();

htdocs/societe/canvas/actions_card_common.class.php

@@ -37,10 +37,17 @@ abstract class ActionsCardCommon
 	var $tpl = array();
 	//! Object container
 	var $object;
-	//! Error string
-	var $error;
-	//! Error array
-	var $errors=array();
+	
+	/**
+	 * @var string Error code (or message)
+	 */
+	public $error='';
+	
+	/**
+	 *
+	 * @var string[] Error codes (or messages)
+	 */
+	public $errors = array ();
 
 
 	/**

htdocs/stripe/payment.php

@@ -1113,6 +1113,6 @@ if (! GETPOST('action'))
     }
 }
 
+// End of page
 llxFooter();
-
 $db->close();

htdocs/variants/generator.php

@@ -374,5 +374,7 @@ if (! empty($id) || ! empty($ref)) {
 
 	<?php
 
-	llxFooter();
+	// End of page
+    llxFooter();
+    $db->close();
 }

htdocs/website/websiteaccount_card.php

@@ -382,7 +382,6 @@ if ($object->id > 0 && (empty($action) || ($action != 'edit' && $action != 'crea
 	include DOL_DOCUMENT_ROOT.'/core/tpl/card_presend.tpl.php';
 }
 
-
 // End of page
 llxFooter();
 $db->close();