Qual: Uniformize constructors

ChangeLog

@@ -5,24 +5,26 @@ English Dolibarr ChangeLog
 ***** ChangeLog for 3.2 compared to 3.1 *****
 
 For users:
-- New: task #11243: Show quantity into stocks for each sub-products into the sub-product tab
+- New: Can edit customer discounts from invoice create and edit card.
-- New: task #10500: Option to choose if professional id are unique
+- New: task #11243: Show quantity into stocks for each sub-products into the sub-product tab.
-- New: Add hidden option FOURN_PRODUCT_AVAILABILITY
+- New: task #10500: Option to choose if professional id are unique.
-- New: task #11123: Add best supplier price
+- New: Add hidden option FOURN_PRODUCT_AVAILABILITY.
+- New: task #11123: Add best supplier price.
 - New: Enhancement in styles.
-- New: task #11289 : Modify third party accountancy code generator aquarium
+- New: task #11289 : Modify third party accountancy code generator aquarium.
-- New: task #10606 : more comprehensive message error
+- New: task #10606 : more comprehensive message error.
-- New: task #11278 : Option into point of sale module to add services in list
+- New: task #11278 : Option into point of sale module to add services in list.
 - New: task #11261 : Add an entry into menu called "New shipment".
-- New: Make option MAIN_GENERATE_DOCUMENTS_WITHOUT_VAT available by default
+- New: Make option MAIN_GENERATE_DOCUMENTS_WITHOUT_VAT available by default.
 - New: Can build PDF in USLetter format or canada format.
 
 For developers:
 - Qual: Removed no more used external libraries.
 - Qual: Cleaned a lot of dead code.
 - Qual: Parent templates classes were moved as abstract classes.
-- Qual: Added more "hooks"
+- Qual: Add more "hooks".
 - Qual: Add "canvas" feature to overwrite page of thirdparty, contact, product with yours.
+- More OOP (usage of "abstract", "static", ...), unifformize constructors.
 
 
 ***** ChangeLog for 3.1 compared to 3.0 *****

htdocs/accountancy/class/accountancysystem.class.php

@@ -40,21 +40,21 @@ class AccountancySystem
 
 
 	/**
-	 *    \brief  Constructor of class
+	 *	Constructor
-	 *    \param  DB          Database handler
+	 *
-	 *    \param  id          Id compte (0 by default)
+	 *  @param		DoliDB		$DB      Database handler
 	 */
-	function AccountancySystem($DB, $id=0)
+	function AccountancySystem($DB)
 	{
 		$this->db = $DB;
-		$this->id   = $id ;
 	}
 
 
 	/**
-	 *    \brief  	Insert accountancy system name into database
+	 *  Insert accountancy system name into database
-	 *    \param  	user 	User making insert
+	 *
-	 *    \return	int		<0 if KO, Id of line if OK
+	 *  @param  	User	$user 	User making insert
+	 *  @return		int				<0 if KO, Id of line if OK
 	 */
 	function create($user)
 	{

htdocs/bookmarks/class/bookmark.class.php

@@ -43,13 +43,11 @@ class Bookmark
     /**
 	 *	Constructor
 	 *
-     *    @param      DoliDB	$db          Handler for Database Access
+	 *  @param		DoliDB		$DB      Database handler
-     *    @param      int		$id          Bookmark Id
      */
-    function Bookmark($db, $id=-1)
+    function Bookmark($db)
     {
         $this->db = $db;
-        $this->id = $id;
     }
 
     /**

htdocs/cashdesk/facturation_verif.php

@@ -72,7 +72,8 @@ switch ( $_GET['action'] )
 						$societe = new Societe($db);
 						$societe->fetch($thirdpartyid);
 
-						$product = new Product($db, $productid);
+						$product = new Product($db);
+                        $product->fetch($productid);
 
 						if(isset($product->multiprices[$societe->price_level]))
 						{

htdocs/cashdesk/tpl/liste_articles.tpl.php

@@ -64,7 +64,8 @@ along with this program. If not, see <http://www.gnu.org/licenses/>.
 			/** add Ditto for MultiPrix*/
 			if($conf->global->PRODUIT_MULTIPRICES)
 			{
-				$product = new Product($db, $ret[$i]['id']);
+				$product = new Product($db);
+                $product->fetch($ret[$i]['id']);
 
 				if(isset($product->multiprices[$societe->price_level]))
 				{

htdocs/categories/class/categorie.class.php

@@ -57,8 +57,8 @@ class Categorie
 	/**
 	 *	Constructor
 	 *
-	 * 	@param	DB		acces base de donnees
+	 *  @param		DoliDB		$DB     Database handler
-	 * 	@param	id		id de la categorie
+	 *  @param		int			$id		Id of category to fetch during init
 	 */
 	function Categorie($DB, $id=-1)
 	{
@@ -71,7 +71,7 @@ class Categorie
 	/**
 	 * 	Load category into memory from database
 	 *
-	 * 	@param		id		id of category
+	 * 	@param		int		$id		Id of category
 	 */
 	function fetch($id)
 	{

htdocs/categories/edit.php

@@ -105,12 +105,8 @@ llxHeader("","",$langs->trans("Categories"));
 print_fiche_titre($langs->trans("ModifCat"));
 
 
-if ($mesg)
+dol_htmloutput_errors($mesg);
-{
+
-	print '<div class="error">';
-	print $mesg;
-	print '</div>';
-}
 
 $categorie = new Categorie($db, $id);
 $html = new Form($db);

htdocs/comm/fiche.php

@@ -84,14 +84,16 @@ if ($_POST['action'] == 'setcustomeraccountancycode')
 
 if ($_GET["action"] == 'attribute_prefix' && $user->rights->societe->creer)
 {
-	$societe = new Societe($db, $_GET["socid"]);
+	$societe = new Societe($db);
+	$societe->fetch($_GET["socid"]);
 	$societe->attribute_prefix($db, $_GET["socid"]);
 }
 // conditions de reglement
 if ($_POST["action"] == 'setconditions' && $user->rights->societe->creer)
 {
 
-	$societe = new Societe($db, $_GET["socid"]);
+	$societe = new Societe($db);
+	$societe->fetch($_GET["socid"]);
 	$societe->cond_reglement=$_POST['cond_reglement_id'];
 	$sql = "UPDATE ".MAIN_DB_PREFIX."societe SET cond_reglement='".$_POST['cond_reglement_id'];
 	$sql.= "' WHERE rowid='".$_GET["socid"]."'";
@@ -101,7 +103,8 @@ if ($_POST["action"] == 'setconditions' && $user->rights->societe->creer)
 // mode de reglement
 if ($_POST["action"] == 'setmode' && $user->rights->societe->creer)
 {
-	$societe = new Societe($db, $_GET["socid"]);
+	$societe = new Societe($db);
+	$societe->fetch($_GET["socid"]);
 	$societe->mode_reglement=$_POST['mode_reglement_id'];
 	$sql = "UPDATE ".MAIN_DB_PREFIX."societe SET mode_reglement='".$_POST['mode_reglement_id'];
 	$sql.= "' WHERE rowid='".$_GET["socid"]."'";
@@ -111,7 +114,8 @@ if ($_POST["action"] == 'setmode' && $user->rights->societe->creer)
 // assujetissement a la TVA
 if ($_POST["action"] == 'setassujtva' && $user->rights->societe->creer)
 {
-	$societe = new Societe($db, $_GET["socid"]);
+	$societe = new Societe($db);
+	$societe->fetch($_GET["socid"]);
 	$societe->tva_assuj=$_POST['assujtva_value'];
 	$sql = "UPDATE ".MAIN_DB_PREFIX."societe SET tva_assuj='".$_POST['assujtva_value']."' WHERE rowid='".$socid."'";
 	$result = $db->query($sql);

htdocs/comm/propal.php

@@ -661,7 +661,7 @@ if ($_POST['action'] == "addline" && $user->rights->propale->creer)
 		// Ecrase $txtva par celui du produit
 		if ($_POST['idprod'])
 		{
-			$prod = new Product($db, $_POST['idprod']);
+			$prod = new Product($db);
 			$prod->fetch($_POST['idprod']);
 
 			$tva_tx = get_default_tva($mysoc,$object->client,$prod->id);

htdocs/comm/propal/apercu.php

@@ -62,7 +62,7 @@ if ($id > 0 || ! empty($ref))
 
 	if ($object->fetch($id,$ref) > 0)
 	{
-		$soc = new Societe($db, $object->socid);
+		$soc = new Societe($db);
 		$soc->fetch($object->socid);
 
 		$head = propal_prepare_head($object);

htdocs/comm/propal/contact.php

@@ -138,7 +138,7 @@ if ($id > 0 || ! empty($ref))
 	$propal = New Propal($db);
 	if ( $propal->fetch($id,$ref) > 0)
 	{
-		$soc = new Societe($db, $propal->socid);
+		$soc = new Societe($db);
 		$soc->fetch($propal->socid);
 
 

htdocs/comm/prospect/fiche.php

@@ -55,7 +55,8 @@ if ($_GET["action"] == 'cstc')
 // set prospect level
 if ($_POST["action"] == 'setprospectlevel' && $user->rights->societe->creer)
 {
-	$societe = new Societe($db, $_GET["socid"]);
+	$societe = new Societe($db);
+	$societe->fetch($_GET["socid"]);
 	$societe->fk_prospectlevel=$_POST['prospect_level_id'];
 	$sql = "UPDATE ".MAIN_DB_PREFIX."societe SET fk_prospectlevel='".$_POST['prospect_level_id'];
 	$sql.= "' WHERE rowid='".$_GET["socid"]."'";

htdocs/commande/apercu.php

@@ -65,7 +65,7 @@ if ($id > 0 || ! empty($ref))
 
 	if ($object->fetch($id,$ref) > 0)
 	{
-		$soc = new Societe($db, $object->socid);
+		$soc = new Societe($db);
 		$soc->fetch($object->socid);
 
 

htdocs/commande/class/commande.class.php

@@ -99,10 +99,11 @@ class Commande extends CommonObject
 
 
 	/**
-	 *        \brief      Constructeur
+	 *	Constructor
-	 *        \param      DB      Handler d'acces base
+	 *
+	 *  @param		DoliDB		$DB      Database handler
 	 */
-	function Commande($DB, $socid="", $commandeid=0)
+	function Commande($DB)
 	{
 		global $langs;
 		$langs->load('orders');
@@ -1152,7 +1153,8 @@ class Commande extends CommonObject
 			/** POUR AJOUTER AUTOMATIQUEMENT LES SOUSPRODUITS a LA COMMANDE
 			 if (! empty($conf->global->PRODUIT_SOUSPRODUITS))
 			 {
-			 $prod = new Product($this->db, $idproduct);
+			 $prod = new Product($this->db);
+			 $prod->fetch($idproduct);
 			 $prod -> get_sousproduits_arbo ();
 			 $prods_arbo = $prod->get_each_prod();
 			 if(sizeof($prods_arbo) > 0)

htdocs/commande/contact.php

@@ -134,7 +134,7 @@ if ($id > 0 || ! empty($ref))
 	$commande = new Commande($db);
 	if ( $commande->fetch($_GET['id'],$_GET['ref']) > 0)
 	{
-		$soc = new Societe($db, $commande->socid);
+		$soc = new Societe($db);
 		$soc->fetch($commande->socid);
 
 

htdocs/commande/fiche.php

@@ -510,7 +510,7 @@ if ($action == 'addline' && $user->rights->commande->creer)
         // Ecrase $base_price_type par celui du produit
         if ($_POST['idprod'])
         {
-            $prod = new Product($db, $_POST['idprod']);
+            $prod = new Product($db);
             $prod->fetch($_POST['idprod']);
 
             $tva_tx = get_default_tva($mysoc,$object->client,$prod->id);

htdocs/commande/info.php

@@ -49,7 +49,7 @@ llxHeader('',$langs->trans('Order'),'EN:Customers_Orders|FR:Commandes_Clients|ES
 $commande = new Commande($db);
 $commande->fetch($_GET["id"]);
 $commande->info($_GET["id"]);
-$soc = new Societe($db, $commande->socid);
+$soc = new Societe($db);
 $soc->fetch($commande->socid);
 
 $head = commande_prepare_head($commande);

htdocs/commande/note.php

@@ -85,7 +85,7 @@ $html = new Form($db);
 
 if ($id > 0 || ! empty($ref))
 {
-	$soc = new Societe($db, $commande->socid);
+	$soc = new Societe($db);
 	$soc->fetch($commande->socid);
 
 	$head = commande_prepare_head($commande);

htdocs/compta/bank/class/bankcateg.class.php

@@ -48,8 +48,9 @@ class BankCateg // extends CommonObject
 
 
     /**
-     *      \brief      Constructor
+     *	Constructor
-     *      \param      DB      Database handler
+     *
+     *  @param		DoliDB		$DB      Database handler
      */
     function BankCateg($DB)
     {
@@ -59,10 +60,11 @@ class BankCateg // extends CommonObject
 
 
     /**
-     *      \brief      Create in database
+     *  Create in database
-     *      \param      user        	User that create
+     *
-     *      \param      notrigger	    0=launch triggers after, 1=disable triggers
+     *  @param      user        	User that create
-     *      \return     int         	<0 if KO, Id of created object if OK
+     *  @param      notrigger	    0=launch triggers after, 1=disable triggers
+     *  @return     int         	<0 if KO, Id of created object if OK
      */
     function create($user, $notrigger=0)
     {

htdocs/compta/clients.php

@@ -63,7 +63,8 @@ $thirdpartystatic=new Societe($db);
 
 if ($action == 'attribute_prefix')
 {
-	$societe = new Societe($db, $socid);
+	$societe = new Societe($db);
+	$societe->fetch($socid);
 	$societe->attribute_prefix($db, $socid);
 }
 

htdocs/compta/deplacement/note.php

@@ -102,7 +102,7 @@ if ($id > 0 || ! empty($ref))
 	$trip = new Deplacement($db);
 	$trip->fetch($id,$ref);
 
-	$soc = new Societe($db, $trip->socid);
+	$soc = new Societe($db);
     $soc->fetch($trip->socid);
 
 	$h=0;

htdocs/compta/facture.php

@@ -903,7 +903,7 @@ if (($action == 'addline' || $action == 'addline_predef') && $user->rights->fact
         // Ecrase $base_price_type par celui du produit
         if ($_POST['idprod'])
         {
-            $prod = new Product($db, $_POST['idprod']);
+            $prod = new Product($db);
             $prod->fetch($_POST['idprod']);
 
             $tva_tx = get_default_tva($mysoc,$object->client,$prod->id);
@@ -1884,7 +1884,7 @@ else
 
             $result=$object->fetch_thirdparty();
 
-            $soc = new Societe($db, $object->socid);
+            $soc = new Societe($db);
             $soc->fetch($object->socid);
 
             $totalpaye  = $object->getSommePaiement();

htdocs/compta/facture/apercu.php

@@ -63,7 +63,7 @@ if ($id > 0 || ! empty($ref))
 
     if ($object->fetch($id,$ref) > 0)
     {
-        $soc = new Societe($db, $object->socid);
+        $soc = new Societe($db);
         $soc->fetch($object->socid);
 
         $author = new User($db);

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

@@ -372,7 +372,7 @@ class Facture extends CommonObject
                 {
                     if ($_facrec->lines[$i]->fk_product)
                     {
-                        $prod = new Product($this->db, $_facrec->lines[$i]->fk_product);
+                        $prod = new Product($this->db);
                         $res=$prod->fetch($_facrec->lines[$i]->fk_product);
                     }
                     $tva_tx = get_default_tva($mysoc,$soc,$prod->id);

htdocs/compta/facture/fiche-rec.php

@@ -337,7 +337,7 @@ else
 
 		if ($fac->fetch($facid, $user->societe_id) > 0)
 		{
-			$soc = new Societe($db, $fac->socid);
+			$soc = new Societe($db);
 			$soc->fetch($fac->socid);
 			$author = new User($db);
 			$author->fetch($fac->user_author);

htdocs/compta/facture/info.php

@@ -41,7 +41,7 @@ $fac = new Facture($db);
 $fac->fetch($_GET["facid"]);
 $fac->info($_GET["facid"]);
 
-$soc = new Societe($db, $fac->socid);
+$soc = new Societe($db);
 $soc->fetch($fac->socid);
 
 $head = facture_prepare_head($fac);

htdocs/compta/facture/note.php

@@ -100,7 +100,7 @@ if ($id > 0 || ! empty($ref))
 	$fac = new Facture($db);
 	$fac->fetch($id,$ref);
 
-	$soc = new Societe($db, $fac->socid);
+	$soc = new Societe($db);
     $soc->fetch($fac->socid);
 
     $head = facture_prepare_head($fac);

htdocs/compta/facture/prelevement.php

@@ -105,7 +105,7 @@ if ($_REQUEST["facid"] > 0 || $_REQUEST["ref"])
     {
         if ($mesg) print $mesg.'<br>';
 
-        $soc = new Societe($db, $fac->socid);
+        $soc = new Societe($db);
         $soc->fetch($fac->socid);
 
         $totalpaye  = $fac->getSommePaiement();

htdocs/compta/prelevement/class/bon-prelevement.class.php

@@ -50,10 +50,12 @@ class BonPrelevement extends CommonObject
     var $statut;    // 0-Wait, 1-Trans, 2-Done
     var $labelstatut=array();
 
+
     /**
-   *	Class constructor
+     *	Constructor
-   *    @param  DB          data base handler access
+     *
-   *    @param  filename    filename
+     *  @param		DoliDB		$DB      	Database handler
+     *  @param		string		$filename	Filename of withdraw receipt
      */
     function BonPrelevement($DB, $filename='')
     {

htdocs/compta/sociales/class/paymentsocialcontribution.class.php

@@ -53,8 +53,9 @@ class PaymentSocialContribution extends CommonObject
 	var $fk_user_modif;
 
 	/**
-	 *      \brief      Constructor
+	 *	Constructor
-	 *      \param      DB      Database handler
+	 *
+	 *  @param		DoliDB		$DB      Database handler
 	 */
 	function PaymentSocialContribution($DB)
 	{
@@ -65,7 +66,8 @@ class PaymentSocialContribution extends CommonObject
 	/**
 	 *  Create payment of social contribution into database.
      *  Use this->amounts to have list of lines for the payment
-	 *      @param      user    User making payment
+     *
+	 *  @param      User		$user   User making payment
 	 *  @return     int     			<0 if KO, id of payment if OK
 	 */
 	function create($user)

htdocs/compta/tva/class/tva.class.php

@@ -56,8 +56,9 @@ class Tva extends CommonObject
 
 
     /**
-     *      \brief      Constructor
+	 *	Constructor
-     *      \param      DB      Database handler
+	 *
+	 *  @param		DoliDB		$DB      Database handler
      */
     function Tva($DB)
     {
@@ -67,9 +68,10 @@ class Tva extends CommonObject
 
 
     /**
-     *      \brief      Create in database
+     *  Create in database
-     *      \param      user        User that create
+     *
-     *      \return     int         <0 si ko, >0 si ok
+     *  @param      User	$user       User that create
+     *  @return     int      			<0 if KO, >0 if OK
      */
     function create($user)
     {

htdocs/contact/canvas/actions_contactcard_common.class.php

@@ -45,10 +45,11 @@ abstract class ActionsContactCardCommon
 
     /**
 	 *    Constructor
-     *    @param   DB              Handler acces base de donnees
+	 *
-     *    @param   targmetmodule   Name of directory of module where canvas is stored
+     *    @param   DoliDB	$DB              Handler acces base de donnees
-     *    @param   canvas          Name of canvas
+     *    @param   string	$targmetmodule   Name of directory of module where canvas is stored
-     *    @param   card            Name of tab (sub-canvas)
+     *    @param   string	$canvas          Name of canvas
+     *    @param   streing	$card            Name of tab (sub-canvas)
 	 */
 	function ActionsContactCardCommon($DB,$targetmodule,$canvas,$card)
 	{

htdocs/contact/canvas/default/actions_contactcard_default.class.php

@@ -36,10 +36,11 @@ class ActionsContactCardDefault extends ActionsContactCardCommon
 
 	/**
      *    Constructor
-     *    @param   DB              Handler acces base de donnees
+     *
-     *    @param   targmetmodule   Name of directory of module where canvas is stored
+     *    @param   DoliDB	$DB              Handler acces base de donnees
-     *    @param   canvas          Name of canvas
+     *    @param   string	$targmetmodule   Name of directory of module where canvas is stored
-     *    @param   card            Name of tab (sub-canvas)
+     *    @param   string	$canvas          Name of canvas
+     *    @param   string	$card            Name of tab (sub-canvas)
 	 */
 	function ActionsContactCardDefault($DB,$targetmodule,$canvas,$card)
 	{

htdocs/contact/class/contact.class.php

@@ -80,8 +80,9 @@ class Contact extends CommonObject
 
 
 	/**
-	 *      Constructor of class Contact
+	 *	Constructor
-	 *      @param      DB      Habler d'acces base
+	 *
+	 *  @param		DoliDB		$DB      Database handler
 	 */
 	function Contact($DB)
 	{
@@ -90,7 +91,8 @@ class Contact extends CommonObject
 
 	/**
 	 *  Add a contact into database
-	 *      @param      user        Object user that create
+	 *
+	 *  @param      User	$user       Object user that create
 	 *  @return     int      			<0 if KO, >0 if OK
 	 */
 	function create($user)

htdocs/contrat/class/contrat.class.php

@@ -72,8 +72,9 @@ class Contrat extends CommonObject
 
 
 	/**
-	 *    Constructor of classe
+	 *	Constructor
-	 *    @param      DB          Databse handler
+	 *
+	 *  @param		DoliDB		$DB      Database handler
 	 */
 	function Contrat($DB)
 	{

htdocs/contrat/document.php

@@ -109,7 +109,7 @@ llxHeader("","",$langs->trans("CardProduct".$product->type));
 
 if ($contrat->id)
 {
-    $soc = new Societe($db, $contrat->societe->id);
+    $soc = new Societe($db);
     $soc->fetch($contrat->societe->id);
 
 	if ( $error_msg )

htdocs/contrat/fiche.php

@@ -233,7 +233,7 @@ if ($action == 'addline' && $user->rights->contrat->creer)
 		// Ecrase $base_price_type par celui du produit
         if ($_POST['idprod'])
         {
-            $prod = new Product($db, $_POST['idprod']);
+            $prod = new Product($db);
             $prod->fetch($_POST['idprod']);
 
             $tva_tx = get_default_tva($mysoc,$object->client,$prod->id);

htdocs/contrat/note.php

@@ -92,9 +92,9 @@ $html = new Form($db);
 
 if ($_GET["id"])
 {
-	if ($mesg) print $mesg;
+	dol_htmloutput_mesg($mesg);
 
-    $soc = new Societe($db, $contrat->societe->id);
+    $soc = new Societe($db);
     $soc->fetch($contrat->societe->id);
 
     $head = contract_prepare_head($contrat);

htdocs/core/class/canvas.class.php

@@ -48,8 +48,8 @@ class Canvas
    /**
 	*   Constructor
 	*
-	*   @param     DB          Database handler
+	*   @param     DoliDB	$DB          Database handler
-	*   @param     action      Action ('create', 'view', 'edit')
+	*   @param     string	$action      Action ('create', 'view', 'edit')
 	*/
 	function Canvas($DB, $action='view')
 	{

htdocs/core/class/cookie.class.php

@@ -39,8 +39,9 @@ class DolCookie
 	var $cookie;
 
 	/**
-	 *      \brief      Constructor
+	 *  Constructor
-	 *      \param      key      Personnal key
+	 *
+	 *  @param      string	$key      Personnal key
 	 */
 	function DolCookie($key = '')
 	{
@@ -53,7 +54,7 @@ class DolCookie
 
 
 	/**
-	 *      \brief      Encrypt en create the cookie
+	 *   Encrypt en create the cookie
 	 */
 	function cryptCookie()
 	{

htdocs/core/class/events.class.php

@@ -89,8 +89,9 @@ class Events // extends CommonObject
 
 
 	/**
-	 *      \brief      Constructor
+	 *	Constructor
-	 *      \param      DB      Database handler
+	 *
+	 *  @param		DoliDB		$DB      Database handler
 	 */
 	function Events($DB)
 	{
@@ -100,9 +101,9 @@ class Events // extends CommonObject
 
 
 	/**
-	 *      \brief      Create in database
+	 *   Create in database
-	 *      \param      user        User that create
+	 *   @param      User	$user       User that create
-	 *      \return     int         <0 si ko, >0 si ok
+	 *   @return     int     		    <0 if KO, >0 if OK
 	 */
 	function create($user)
 	{

htdocs/core/class/extrafields.class.php

@@ -45,7 +45,8 @@ class ExtraFields
 
 	/**
 	 *	Constructor
-	 *  @param DB			base de donnees
+	 *
+	 *  @param		DoliDB		$DB      Database handler
 	 */
 	function ExtraFields($DB)
 	{

htdocs/core/class/genericobject.class.php

@@ -33,9 +33,9 @@ class GenericObject extends CommonObject
 	var $db;
 
 	/**
-	 *	Class constructor
+	 *	Constructor
 	 *
-	 *	@param		DoliDB	$db		Data base handler access
+	 *  @param		DoliDB		$DB      Database handler
 	 */
 	function GenericObject($db)
 	{

htdocs/core/class/html.form.class.php

@@ -56,7 +56,8 @@ class Form
 
     /**
 	 *	Constructor
-     * @param      $DB      Database handler
+	 *
+	 *  @param		DoliDB		$DB      Database handler
      */
     function Form($DB)
     {
@@ -65,6 +66,7 @@ class Form
 
     /**
      * Output key field for an editable field
+     *
      * @param      text            Text of label
      * @param      htmlname        Name of select field
      * @param      preselected     Preselected value for parameter

htdocs/core/class/html.formactions.class.php

@@ -36,7 +36,8 @@ class FormActions
 
     /**
 	 *	Constructor
-     *		@param     DB      Database access handler
+	 *
+	 *  @param		DoliDB		$DB      Database handler
      */
     function FormActions($DB)
     {
@@ -48,6 +49,7 @@ class FormActions
 
     /**
      *      Show list of action status
+     *
      * 		@param		formname	Name of form where select in included
      * 		@param		selected	Preselected value
      * 		@param		canedit		1=can edit, 0=read only

htdocs/core/class/html.formadmin.class.php

@@ -35,8 +35,9 @@ class FormAdmin
 
 
 	/**
-	 *	\brief     Constructor
+	 *	Constructor
-	 *	\param     DB      handler d'acces base de donnee
+	 *
+	 *  @param		DoliDB		$DB      Database handler
 	 */
 	function FormAdmin($DB)
 	{
@@ -46,7 +47,8 @@ class FormAdmin
 	}
 
 	/**
-	 *    	Output list with available languages.
+	 *    	Output list with available languages
+	 *
 	 *      @deprecated                 Use select_language instead
 	 *    	@param      selected        Langue pre-selectionnee
 	 *    	@param      htmlname        Nom de la zone select

htdocs/core/class/html.formfile.class.php

@@ -38,7 +38,8 @@ class FormFile
 
     /**
 	 *	Constructor
-     *		@param     DB      handler d'acces base de donnee
+	 *
+	 *  @param		DoliDB		$DB      Database handler
      */
     function FormFile($DB)
     {

htdocs/core/class/html.formsms.class.php

@@ -61,7 +61,8 @@ class FormSms
 
 	/**
 	 *	Constructor
-	 *  @param     DB      handler d'acces base de donnee
+	 *
+	 *  @param		DoliDB		$DB      Database handler
 	 */
 	function FormSms($DB)
 	{

htdocs/core/class/interfaces.class.php

@@ -35,8 +35,9 @@ class Interfaces
     var $errors	= array();	// Array for errors
 
     /**
-	*   \brief      Constructor.
+     *	Constructor
-	*   \param      DB      Database handler
+     *
+     *  @param		DoliDB		$DB      Database handler
      */
     function Interfaces($DB)
     {

htdocs/core/class/menu.class.php

@@ -32,7 +32,7 @@ class Menu {
     var $liste;
 
     /**
-     *  \brief      Constructor for class menu
+	 *	Constructor
      */
     function Menu()
     {

htdocs/core/class/menubase.class.php

@@ -57,9 +57,10 @@ class Menubase
 
     /**
 	 *	Constructor
-     *      @param     DB      Database handler
+	 *
-     *      @param     menu_handler
+	 *  @param		DoliDB		$DB      Database handler
-     *      @param     type
+     *  @param     	string		$menu_handler
+     *  @param     	string		$type
      */
     function Menubase($DB,$menu_handler='',$type='')
     {
@@ -72,7 +73,8 @@ class Menubase
 
     /**
      *      Create menu entry into database
-     *      @param      user        User that create
+     *
+     *      @param      User	$user       User that create
      *      @return     int      			<0 if KO, Id of record if OK
      */
     function create($user=0)

htdocs/core/class/notify.class.php

@@ -55,11 +55,12 @@ class Notify
 
 
     /**
-     *    	\brief      Renvoie le message signalant les notifications qui auront lieu sur
+     *  Renvoie le message signalant les notifications qui auront lieu sur
-     *					un evenement pour affichage dans texte de confirmation evenement.
+     *	un evenement pour affichage dans texte de confirmation evenement
-     * 		\param		action		Id of action in llx_c_action_trigger
+     *
-     * 		\param		socid		Id of third party
+     * 	@param		action		Id of action in llx_c_action_trigger
-     *		\return		string		Message
+     * 	@param		socid		Id of third party
+     *	@return		string		Message
      */
 	function confirmMessage($action,$socid)
 	{

htdocs/core/class/rssparser.class.php

@@ -60,6 +60,8 @@ class RssParser
 
 	/**
 	 *	Constructor
+	 *
+	 *  @param		DoliDB		$DB      Database handler
 	 */
 	public function RssParser($db)
 	{

htdocs/core/class/translate.class.php

@@ -49,8 +49,7 @@ class Translate {
 	/**
 	 *	Constructor
 	 *
-	 *  @param      dir             Force directory that contains /langs subdirectory (value is sometine '..' like into install/* pages
+	 *  @param      dir             Force directory that contains /langs subdirectory (value is sometine '..' like into install/* pages or support/* pages).
-	 *                              or support/* pages).
 	 *  @param      conf			Object with Dolibarr configuration
 	 */
 	function Translate($dir = "",$conf)

htdocs/ecm/class/ecmdirectory.class.php

@@ -52,8 +52,9 @@ class EcmDirectory // extends CommonObject
 
 
 	/**
-	 *      \brief      Constructor
+	 *	Constructor
-	 *      \param      DB      Database handler
+	 *
+	 *  @param		DoliDB		$DB      Database handler
 	 */
 	function EcmDirectory($DB)
 	{
@@ -64,7 +65,8 @@ class EcmDirectory // extends CommonObject
 
 	/**
 	 *  Create record into database
-	 *      @param      user        User that create
+	 *
+	 *  @param      User	$user       User that create
 	 *  @return     int      			<0 if KO, >0 if OK
 	 */
 	function create($user)

htdocs/fichinter/apercu.php

@@ -55,7 +55,7 @@ if ($id > 0 || ! empty($ref))
 
 	if ($object->fetch($id,$ref) > 0)
 	{
-		$soc = new Societe($db, $object->socid);
+		$soc = new Societe($db);
 		$soc->fetch($object->socid);
 
 		$head = fichinter_prepare_head($object);

htdocs/fichinter/class/fichinter.class.php

@@ -59,9 +59,9 @@ class Fichinter extends CommonObject
 	var $lines = array();
 
 	/**
-	 *	Class constructor
+	 *	Constructor
 	 *
-	 *	@param		DoliDB	$DB		Data base handler access
+	 *  @param		DoliDB		$DB      Database handler
 	 */
 	function Fichinter($DB)
 	{

htdocs/fichinter/contact.php

@@ -128,10 +128,10 @@ dol_htmloutput_mesg($mesg);
 $id = $_GET["id"];
 if ($id > 0)
 {
-	$fichinter = New Fichinter($db);
+	$fichinter = new Fichinter($db);
 	if ($fichinter->fetch($_GET['id']) > 0)
 	{
-		$soc = new Societe($db, $fichinter->socid);
+		$soc = new Societe($db);
 		$soc->fetch($fichinter->socid);
 
 

htdocs/fichinter/document.php

@@ -111,7 +111,7 @@ if ($object->id)
 {
 	$object->fetch_thirdparty();
 
-    $soc = new Societe($db, $object->societe->id);
+    $soc = new Societe($db);
     $soc->fetch($object->societe->id);
 
 	if ( $error_msg )

htdocs/fourn/class/fournisseur.commande.class.php

@@ -1406,7 +1406,7 @@ class CommandeFournisseur extends Commande
 
         for ($i = 0 ; $i < sizeof($comclient->lines) ; $i++)
         {
-			$prod = new Product($this->db, $comclient->lines[$i]->fk_product);
+            $prod = new Product($this->db);
             if ($prod->fetch($comclient->lines[$i]->fk_product) > 0)
             {
                 $libelle  = $prod->libelle;
@@ -1824,6 +1824,8 @@ class CommandeFournisseurLigne extends OrderLine
 
     /**
      *	Constructor
+     *
+     *  @param		DoliDB		$DB      Database handler
      */
     function CommandeFournisseurLigne($DB)
     {
@@ -1832,7 +1834,8 @@ class CommandeFournisseurLigne extends OrderLine
 
     /**
      *  Load line order
-	 *  @param     rowid           id line order
+     *
+     *  @param     int		$rowid           id line order
      */
     function fetch($rowid)
     {

htdocs/fourn/class/fournisseur.facture.class.php

@@ -85,11 +85,10 @@ class FactureFournisseur extends Facture
 
     /**
 	 *	Constructor
-     *    @param  DB          	Database access handler
+	 *
-     *    @param  socid			Id societe ('' par defaut)
+	 *  @param		DoliDB		$DB      Database handler
-     *    @param  facid       	Id facture ('' par defaut)
      */
-    function FactureFournisseur($DB, $socid='', $facid='')
+    function FactureFournisseur($DB)
     {
         $this->db = $DB ;
 
@@ -112,7 +111,8 @@ class FactureFournisseur extends Facture
 
     /**
      *    Create supplier invoice into database
-     *    @param      user        object utilisateur qui cree
+     *
+     *    @param      User		$user       object utilisateur qui cree
      *    @return     int    	     		id facture si ok, < 0 si erreur
      */
     function create($user)

htdocs/fourn/class/fournisseur.product.class.php

@@ -50,6 +50,8 @@ class ProductFournisseur extends Product
 
     /**
 	 *	Constructor
+	 *
+	 *  @param		DoliDB		$DB      Database handler
      */
     function ProductFournisseur($db)
     {

htdocs/fourn/commande/contact.php

@@ -135,7 +135,7 @@ if ($id > 0 || ! empty($ref))
 	$commande = new CommandeFournisseur($db);
 	if ( $commande->fetch($_GET['id'],$_GET['ref']) > 0)
 	{
-		$soc = new Societe($db, $commande->socid);
+		$soc = new Societe($db);
 		$soc->fetch($commande->socid);
 
 

htdocs/fourn/facture/fiche.php

@@ -374,7 +374,8 @@ if ($_POST['action'] == 'add' && $user->rights->fournisseur->facture->creer)
 
 if ($_GET['action'] == 'del_ligne')
 {
-    $facfou = new FactureFournisseur($db,'',$_GET['facid']);
+    $facfou = new FactureFournisseur($db);
+    $facfou->fetch($_GET['facid']);
     $facfou->deleteline($_GET['lineid']);
     $_GET['action'] = 'edit';
 }
@@ -432,7 +433,7 @@ if ($_REQUEST['action'] == 'update_line')
 
 if ($_GET['action'] == 'addline')
 {
-    $facfou = new FactureFournisseur($db, '', $_GET['facid']);
+    $facfou = new FactureFournisseur($db);
     $ret=$facfou->fetch($_GET['facid']);
     if ($ret < 0)
     {
@@ -540,7 +541,7 @@ if ($_GET['action'] == 'addline')
 
 if ($_POST['action'] == 'classin')
 {
-    $facture = new FactureFournisseur($db,'',$_GET['facid']);
+    $facture = new FactureFournisseur($db);
     $facture->fetch($_GET['facid']);
     $result=$facture->setProject($_POST['projectid']);
 }

htdocs/fourn/facture/info.php

@@ -48,7 +48,7 @@ llxHeader();
 $fac = new FactureFournisseur($db);
 $fac->fetch($_GET["facid"]);
 $fac->info($_GET["facid"]);
-$soc = new Societe($db, $fac->socid);
+$soc = new Societe($db);
 $soc->fetch($fac->socid);
 
 $head = facturefourn_prepare_head($fac);

htdocs/includes/boxes/box_comptes.php

@@ -41,7 +41,7 @@ class box_comptes extends ModeleBoxes {
 
 
 	/**
-	 *      \brief      Constructor for class
+	 *  Constructor
 	 */
 	function box_comptes()
 	{
@@ -52,8 +52,9 @@ class box_comptes extends ModeleBoxes {
 	}
 
 	/**
-	 *      \brief      Load data into info_box_contents array to show array later.
+	 *  Load data into info_box_contents array to show array later.
-	 *      \param      $max        Maximum number of records to load
+	 *
+	 *  @param      $max        Maximum number of records to load
 	 */
 	function loadBox($max=5)
 	{

htdocs/includes/menus/smartphone/smartphone_frontoffice.php

@@ -35,7 +35,8 @@ class MenuSmart {
 
 	/**
 	 *	Constructor
-	 *    @param      db      Database handler
+	 *
+	 *  @param		DoliDB		$DB      Database handler
 	 */
 	function MenuSmart($db)
 	{
@@ -45,6 +46,7 @@ class MenuSmart {
 
 	/**
 	 *    Show menu
+	 *
 	 *    @param		limitmenuto		To limit menu to a top or left menu value
 	 */
 	function showmenu($limitmenuto)

htdocs/includes/menus/standard/auguria_backoffice.php

@@ -77,7 +77,8 @@ class MenuLeft {
 
     /**
 	 *	Constructor
-     *  @param      db                  Database handler
+	 *
+	 *  @param		DoliDB				$DB     Database handler
      *  @param      menu_array          		Table of menu entries to show before entries of menu handler
      *  @param      menu_array_after    		Table of menu entries to show after entries of menu handler
      *
@@ -91,8 +92,9 @@ class MenuLeft {
 
 
     /**
-     *      \brief      Show menu
+     *   Show menu
-     *      \return     int     Number of menu entries shown
+     *
+     *   @return     int     					Number of menu entries shown
      */
     function showmenu()
     {

htdocs/includes/menus/standard/auguria_frontoffice.php

@@ -78,6 +78,7 @@ class MenuLeft {
 
     /**
      *  Constructor
+     *
      *  @param      db                  Database handler
      *  @param      menu_array          Table of menu entries to show before entries of menu handler
      *  @param      menu_array_after    Table of menu entries to show after entries of menu handler
@@ -91,8 +92,9 @@ class MenuLeft {
 
 
     /**
-     *      \brief      Show menu
+     *  Show menu
-     *      \return     int     Number of menu entries shown
+     *
+     *  @return     int     Number of menu entries shown
      */
     function showmenu()
     {

htdocs/includes/menus/standard/eldy_backoffice.php

@@ -75,6 +75,7 @@ class MenuLeft {
 
     /**
      *  Constructor
+     *
      *  @param      db                  Database handler
      *  @param      menu_array          Table of menu entries to show before entries of menu handler
      *  @param      menu_array_after    Table of menu entries to show after entries of menu handler
@@ -88,8 +89,9 @@ class MenuLeft {
 
 
     /**
-     *      \brief      Show menu
+     *  Show menu
-     *      \return     int     Number of menu entries shown
+     *
+     *  @return     int     Number of menu entries shown
      */
     function showmenu()
     {

htdocs/includes/menus/standard/eldy_frontoffice.php

@@ -75,6 +75,7 @@ class MenuLeft {
 
     /**
      *  Constructor
+     *
      *  @param      db                  Database handler
      *  @param      menu_array          Table of menu entries to show before entries of menu handler
      *  @param      menu_array_after    Table of menu entries to show after entries of menu handler
@@ -88,8 +89,9 @@ class MenuLeft {
 
 
     /**
-     *      \brief      Show menu
+     *  Show menu
-     *      \return     int     Number of menu entries shown
+     *
+     *  @return     int     Number of menu entries shown
      */
     function showmenu()
     {

htdocs/includes/menus/standard/empty.php

@@ -34,7 +34,8 @@ class MenuTop {
 
     /**
 	 *	Constructor
-     *    @param      db      Dabatase handler
+	 *
+	 *  @param		DoliDB		$DB      Database handler
      */
     function MenuTop($db)
     {
@@ -122,6 +123,7 @@ class MenuLeft {
 
     /**
      *  Constructor
+     *
      *  @param      db                  Database handler
      *  @param      menu_array          Table of menu entries to show before entries of menu handler
      *  @param      menu_array_after    Table of menu entries to show after entries of menu handler
@@ -135,7 +137,7 @@ class MenuLeft {
 
 
     /**
-     *    \brief      Show menu
+     *  Show menu
      */
     function showmenu()
     {

htdocs/includes/modules/DolibarrModules.class.php

@@ -55,7 +55,7 @@ abstract class DolibarrModules
     /**
 	 *	Constructor
 	 *
-     *      @param      DB      Database access handler
+	 *  @param		DoliDB		$DB      Database handler
      */
     function DolibarrModules($DB)
     {

htdocs/includes/modules/commande/modules_commande.php

@@ -239,7 +239,7 @@ function commande_delete_preview($db, $commandeid, $commanderef='')
 
 	if (!$commanderef)
 	{
-		$com = new Commande($db,"",$commandeid);
+		$com = new Commande($db);
 		$com->fetch($commandeid);
 		$commanderef = $com->ref;
 	}

htdocs/includes/modules/commande/pdf_einstein.modules.php

@@ -61,7 +61,7 @@ class pdf_einstein extends ModelePDFCommandes
 	/**
 	 *	Constructor
 	 *
-	 *		@param		DoliDb	$db		Database access handler
+	 *  @param		DoliDB		$DB      Database handler
 	 */
 	function pdf_einstein($db)
 	{

htdocs/includes/modules/expedition/pdf/pdf_expedition_merou.modules.php

@@ -40,8 +40,9 @@ Class pdf_expedition_merou extends ModelePdfExpedition
 
 
 	/**
-	 *	\brief  Constructor
+	 *	Constructor
-	 *	\param	db		Database handler
+	 *
+	 *  @param		DoliDB		$DB      Database handler
 	 */
 	function pdf_expedition_merou($db=0)
 	{

htdocs/includes/modules/facture/doc/doc_generic_invoice_odt.modules.php

@@ -43,8 +43,9 @@ class doc_generic_invoice_odt extends ModelePDFFactures
 
 
 	/**
-	 *		\brief  Constructor
+	 *	Constructor
-	 *		\param	db		Database handler
+	 *
+	 *  @param		DoliDB		$DB      Database handler
 	 */
 	function doc_generic_invoice_odt($db)
 	{

htdocs/includes/modules/facture/doc/pdf_crabe.modules.php

@@ -61,7 +61,8 @@ class pdf_crabe extends ModelePDFFactures
 
 	/**
 	 *	Constructor
-	 *		@param		db		Database access handler
+	 *
+	 *  @param		DoliDB		$DB      Database handler
 	 */
 	function pdf_crabe($db)
 	{

htdocs/includes/modules/facture/doc/pdf_oursin.modules.php

@@ -50,7 +50,7 @@ class pdf_oursin extends ModelePDFFactures
 	/**
 	 *	Constructor
 	 *
-	 *	@param	    DoliDB	$db		Handler access data base
+	 *  @param		DoliDB		$DB      Database handler
 	 */
 	function pdf_oursin($db)
 	{

htdocs/includes/modules/livraison/pdf/pdf_sirocco.modules.php

@@ -37,8 +37,10 @@ require_once(DOL_DOCUMENT_ROOT.'/lib/pdf.lib.php');
 class pdf_sirocco extends ModelePDFDeliveryOrder
 {
 
-	/**		\brief      Constructor
+	/**
-	 *		\param	    db	    Database handler
+	 *	Constructor
+	 *
+	 *  @param		DoliDB		$DB      Database handler
 	 */
 	function pdf_sirocco($db)
 	{

htdocs/includes/modules/livraison/pdf/pdf_typhon.modules.php

@@ -42,8 +42,9 @@ class pdf_typhon extends ModelePDFDeliveryOrder
 	var $emetteur;	// Objet societe qui emet
 
 	/**
-	 *		\brief  Constructor
+	 *	Constructor
-	 *		\param	db		Database handler
+	 *
+	 *  @param		DoliDB		$DB      Database handler
 	 */
 	function pdf_typhon($db)
 	{

htdocs/includes/modules/mailings/example.modules.php

@@ -39,6 +39,11 @@ class mailing_example extends MailingTargets
 
 
     // CHANGE THIS: Constructor name must be called mailing_xxx with xxx=name of your selector
+	/**
+	 *	Constructor
+	 *
+	 *  @param		DoliDB		$DB      Database handler
+	 */
     function mailing_example($DB)
     {
         $this->db=$DB;

htdocs/includes/modules/mailings/framboise.modules.php

@@ -37,7 +37,11 @@ class mailing_framboise extends MailingTargets
 	var $db;
 
 
-	// CHANGE THIS: Constructor name must be called mailing_xxx with xxx=name of your selector
+	/**
+	 *	Constructor
+	 *
+	 *  @param		DoliDB		$DB      Database handler
+	 */
 	function mailing_framboise($DB)
 	{
 		$this->db=$DB;

htdocs/includes/modules/mailings/thirdparties_services_expired.modules.php

@@ -36,7 +36,7 @@ class mailing_thirdparties_services_expired extends MailingTargets
 	/**
 	 *	Constructor
 	 *
-	 * @param  $DB
+	 *  @param		DoliDB		$DB      Database handler
 	 *  @return 	mailing_thirdparties_services_expired
 	 */
 	function mailing_thirdparties_services_expired($DB)

htdocs/includes/modules/member/cards/pdf_standard.class.php

@@ -97,6 +97,8 @@ class pdf_standard {
 
 	/**
 	 *	Constructor
+	 *
+	 *  @param		DoliDB		$DB      Database handler
 	 */
 	function pdf_standard($db)
 	{

htdocs/includes/modules/member/labels/pdf_standardlabel.class.php

@@ -98,6 +98,8 @@ class pdf_standardlabel {
 
 	/**
 	 *	Constructor
+	 *
+	 *  @param		DoliDB		$DB      Database handler
 	 */
 	function pdf_standard($db)
 	{

htdocs/includes/modules/modPaybox.class.php

@@ -32,8 +32,9 @@ include_once(DOL_DOCUMENT_ROOT ."/includes/modules/DolibarrModules.class.php");
 class modPayBox extends DolibarrModules
 {
     /**
-    *   \brief      Constructor. Define names, constants, directories, boxes, permissions
+     *   Constructor. Define names, constants, directories, boxes, permissions
-    *   \param      DB      Database handler
+     *
+     *   @param      DoliDB		$DB      Database handler
      */
     function modPayBox($DB)
     {

htdocs/includes/modules/modPaypal.class.php

@@ -33,8 +33,9 @@ include_once(DOL_DOCUMENT_ROOT ."/includes/modules/DolibarrModules.class.php");
 class modPaypal extends DolibarrModules
 {
     /**
-    *   \brief      Constructor. Define names, constants, directories, boxes, permissions
+     *   Constructor. Define names, constants, directories, boxes, permissions
-    *   \param      DB      Database handler
+     *
+     *   @param      DoliDB		$DB      Database handler
      */
     function modPaypal($DB)
     {

htdocs/includes/modules/project/pdf/pdf_baleine.modules.php

@@ -40,8 +40,9 @@ class pdf_baleine extends ModelePDFProjects
 	var $emetteur;	// Objet societe qui emet
 
 	/**
-	 *		\brief  Constructor
+	 *	Constructor
-	 *		\param	db		Database handler
+	 *
+	 *  @param		DoliDB		$DB      Database handler
 	 */
 	function pdf_baleine($db)
 	{

htdocs/includes/modules/propale/pdf_propale_azur.modules.php

@@ -44,7 +44,7 @@ class pdf_propale_azur extends ModelePDFPropales
 	/**
 	 *	Constructor
 	 *
-	 *	@param	    db		Database access handler
+	 *  @param		DoliDB		$DB      Database handler
 	 */
 	function pdf_propale_azur($db)
 	{

htdocs/includes/modules/propale/pdf_propale_jaune.modules.php

@@ -44,7 +44,7 @@ class pdf_propale_jaune extends ModelePDFPropales
 	/**
 	 *	Constructor
 	 *
-	 *	@param	    DoliDB	$db		Handler access data base
+	 *  @param		DoliDB		$DB      Database handler
 	 */
 	function pdf_propale_jaune($db)
 	{

htdocs/includes/modules/security/generate/modGeneratePassNone.class.php

@@ -41,11 +41,12 @@ class modGeneratePassNone extends ModeleGenPassword
 
 
 	/**
-	 *		\brief      Constructor
+	 *	Constructor
-	 *		\param		db			Database handler
+	 *
-	 *		\param		conf		Handler de conf
+	 *  @param		DoliDB		$DB      Database handler
-	 *		\param		lang		Handler de langue
+	 *	@param		conf		Handler de conf
-	 *		\param		user		Handler du user connecte
+	 *	@param		lang		Handler de langue
+	 *	@param		user		Handler du user connecte
 	 */
 	function modGeneratePassNone($db, $conf, $langs, $user)
 	{

htdocs/includes/modules/security/generate/modGeneratePassStandard.class.php

@@ -41,11 +41,12 @@ class modGeneratePassStandard extends ModeleGenPassword
 
 
 	/**
-	 *		\brief      Constructor
+	 *	Constructor
-	 *		\param		db			Database handler
+	 *
-	 *		\param		conf		Handler de conf
+	 *  @param		DoliDB		$DB      Database handler
-	 *		\param		lang		Handler de langue
+	 *	@param		conf		Handler de conf
-	 *		\param		user		Handler du user connecte
+	 *	@param		lang		Handler de langue
+	 *	@param		user		Handler du user connecte
 	 */
 	function modGeneratePassStandard($db, $conf, $langs, $user)
 	{

htdocs/includes/modules/societe/doc/doc_generic_odt.modules.php

@@ -41,8 +41,9 @@ class doc_generic_odt extends ModeleThirdPartyDoc
 
 
 	/**
-	 *		\brief  Constructor
+	 *	Constructor
-	 *		\param	db		Database handler
+	 *
+	 *  @param		DoliDB		$DB      Database handler
 	 */
 	function doc_generic_odt($db)
 	{

htdocs/includes/modules/societe/mod_codeclient_elephant.php

@@ -47,7 +47,8 @@ class mod_codeclient_elephant extends ModeleThirdPartyCode
 	var $prefixIsRequired; // Le champ prefix du tiers doit etre renseigne quand on utilise {pre}
 
 
-	/**		Constructor
+	/**
+	 *	Constructor
 	 */
 	function mod_codeclient_elephant()
 	{

htdocs/includes/modules/societe/mod_codeclient_leopard.php

@@ -48,7 +48,8 @@ class mod_codeclient_leopard extends ModeleThirdPartyCode
 	var $code_auto; 	                // Numerotation automatique
 
 
-	/**		Constructor
+	/**
+	 *	Constructor
 	 */
 	function mod_codeclient_leopard()
 	{

htdocs/includes/modules/societe/mod_codeclient_monkey.php

@@ -44,7 +44,8 @@ class mod_codeclient_monkey extends ModeleThirdPartyCode
 	var $prefixsupplier='SU';
 
 
-	/**		Constructor classe
+	/**
+	 * 	Constructor
 	 */
 	function mod_codeclient_monkey()
 	{

htdocs/includes/modules/supplier_invoice/modules_facturefournisseur.php

@@ -128,7 +128,7 @@ function supplier_invoice_delete_preview($db, $objectid)
 	global $langs,$conf;
     require_once(DOL_DOCUMENT_ROOT."/lib/files.lib.php");
 
-	$comfourn = new FactureFournisseur($db,"",$objectid);
+	$comfourn = new FactureFournisseur($db);
 	$comfourn->fetch($objectid);
 	$client = new Societe($db);
 	$client->fetch($comfourn->socid);

htdocs/includes/modules/supplier_invoice/pdf/pdf_canelle.modules.php

@@ -58,7 +58,7 @@ class pdf_canelle extends ModelePDFSuppliersInvoices
 	/**
 	 *	Constructor
 	 *
-	 *	@param	    DoliDB	$db		Handler access data base
+	 *  @param		DoliDB		$DB      Database handler
 	 */
 	function pdf_canelle($db,$object)
 	{

htdocs/includes/modules/supplier_order/pdf/pdf_muscadet.modules.php

@@ -60,7 +60,7 @@ class pdf_muscadet extends ModelePDFSuppliersOrders
 	/**
 	 *	Constructor
 	 *
-	 *	@param	    DoliDB	$db		Handler access data base
+	 *  @param		DoliDB		$DB      Database handler
 	 */
 	function pdf_muscadet($db)
 	{