alxarafe / alixar

dolibarr/htdocs/core/modules/modPrelevement.class.php

Indentation +113 added lines, -113 removed lines

@@ -35,94 +35,94 @@ 
 class modPrelevement extends DolibarrModules
 {
 
-	/**
-	 *   Constructor. Define names, constants, directories, boxes, permissions
-	 *
-	 *   @param      DoliDB		$db      Database handler
-	 */
-	function __construct($db)
-	{
-		global $conf;
-
-		$this->db = $db;
-		$this->numero = 57;
-
-		$this->family = "financial";
-		$this->module_position = '52';
-		// Module label (no space allowed), used if translation string 'ModuleXXXName' not found (where XXX is value of numeric property 'numero' of module)
-		$this->name = preg_replace('/^mod/i','',get_class($this));
-		$this->description = "Gestion des Prelevements";
-
-		// Possible values for version are: 'development', 'experimental', 'dolibarr' or version
-		$this->version = 'dolibarr';
-
-		$this->const_name = 'MAIN_MODULE_'.strtoupper($this->name);
-		// Name of png file (without png) used for this module
-		$this->picto='payment';
-
-		// Data directories to create when module is enabled
-		$this->dirs = array("/prelevement/temp","/prelevement/receipts");
-
-		// Dependencies
-		$this->hidden = false;			// A condition to hide module
-		$this->depends = array("modFacture","modBanque");		// List of module class names as string that must be enabled if this module is enabled
-		$this->requiredby = array();	// List of module ids to disable if this one is disabled
-		$this->conflictwith = array();	// List of module class names as string this module is in conflict with
-		$this->phpmin = array(5,4);		// Minimum version of PHP required by module
-
-		// Config pages
-		$this->config_page_url = array("prelevement.php");
-
-		// Constants
-		$this->const = array();
-		$r=0;
-
-		$this->const[$r][0] = "BANK_ADDON_PDF";
-		$this->const[$r][1] = "chaine";
-		$this->const[$r][2] = "sepamandate";
-		$this->const[$r][3] = 'Name of manager to generate SEPA mandate';
-		$this->const[$r][4] = 0;
-		$r++;
-
-
-		// Boxes
-		$this->boxes = array();
-
-		// Permissions
-		$this->rights = array();
-		$this->rights_class = 'prelevement';
-		$r=0;
-		$r++;
-		$this->rights[$r][0] = 151;
-		$this->rights[$r][1] = 'Read direct debit payment orders';
-		$this->rights[$r][2] = 'r';
-		$this->rights[$r][3] = 0;
-		$this->rights[$r][4] = 'bons';
-		$this->rights[$r][5] = 'lire';
-
-		$r++;
-		$this->rights[$r][0] = 152;
-		$this->rights[$r][1] = 'Create/modify a direct debit payment order';
-		$this->rights[$r][2] = 'w';
-		$this->rights[$r][3] = 0;
-		$this->rights[$r][4] = 'bons';
-		$this->rights[$r][5] = 'creer';
-
-		$r++;
-		$this->rights[$r][0] = 153;
-		$this->rights[$r][1] = 'Send/Transmit direct debit payment orders';
-		$this->rights[$r][2] = 'a';
-		$this->rights[$r][3] = 0;
-		$this->rights[$r][4] = 'bons';
-		$this->rights[$r][5] = 'send';
-
-		$r++;
-		$this->rights[$r][0] = 154;
-		$this->rights[$r][1] = 'Record Credits/Rejects of direct debit payment orders';
-		$this->rights[$r][2] = 'a';
-		$this->rights[$r][3] = 0;
-		$this->rights[$r][4] = 'bons';
-		$this->rights[$r][5] = 'credit';
+    /**
+     *   Constructor. Define names, constants, directories, boxes, permissions
+     *
+     *   @param      DoliDB		$db      Database handler
+     */
+    function __construct($db)
+    {
+        global $conf;
+
+        $this->db = $db;
+        $this->numero = 57;
+
+        $this->family = "financial";
+        $this->module_position = '52';
+        // Module label (no space allowed), used if translation string 'ModuleXXXName' not found (where XXX is value of numeric property 'numero' of module)
+        $this->name = preg_replace('/^mod/i','',get_class($this));
+        $this->description = "Gestion des Prelevements";
+
+        // Possible values for version are: 'development', 'experimental', 'dolibarr' or version
+        $this->version = 'dolibarr';
+
+        $this->const_name = 'MAIN_MODULE_'.strtoupper($this->name);
+        // Name of png file (without png) used for this module
+        $this->picto='payment';
+
+        // Data directories to create when module is enabled
+        $this->dirs = array("/prelevement/temp","/prelevement/receipts");
+
+        // Dependencies
+        $this->hidden = false;			// A condition to hide module
+        $this->depends = array("modFacture","modBanque");		// List of module class names as string that must be enabled if this module is enabled
+        $this->requiredby = array();	// List of module ids to disable if this one is disabled
+        $this->conflictwith = array();	// List of module class names as string this module is in conflict with
+        $this->phpmin = array(5,4);		// Minimum version of PHP required by module
+
+        // Config pages
+        $this->config_page_url = array("prelevement.php");
+
+        // Constants
+        $this->const = array();
+        $r=0;
+
+        $this->const[$r][0] = "BANK_ADDON_PDF";
+        $this->const[$r][1] = "chaine";
+        $this->const[$r][2] = "sepamandate";
+        $this->const[$r][3] = 'Name of manager to generate SEPA mandate';
+        $this->const[$r][4] = 0;
+        $r++;
+
+
+        // Boxes
+        $this->boxes = array();
+
+        // Permissions
+        $this->rights = array();
+        $this->rights_class = 'prelevement';
+        $r=0;
+        $r++;
+        $this->rights[$r][0] = 151;
+        $this->rights[$r][1] = 'Read direct debit payment orders';
+        $this->rights[$r][2] = 'r';
+        $this->rights[$r][3] = 0;
+        $this->rights[$r][4] = 'bons';
+        $this->rights[$r][5] = 'lire';
+
+        $r++;
+        $this->rights[$r][0] = 152;
+        $this->rights[$r][1] = 'Create/modify a direct debit payment order';
+        $this->rights[$r][2] = 'w';
+        $this->rights[$r][3] = 0;
+        $this->rights[$r][4] = 'bons';
+        $this->rights[$r][5] = 'creer';
+
+        $r++;
+        $this->rights[$r][0] = 153;
+        $this->rights[$r][1] = 'Send/Transmit direct debit payment orders';
+        $this->rights[$r][2] = 'a';
+        $this->rights[$r][3] = 0;
+        $this->rights[$r][4] = 'bons';
+        $this->rights[$r][5] = 'send';
+
+        $r++;
+        $this->rights[$r][0] = 154;
+        $this->rights[$r][1] = 'Record Credits/Rejects of direct debit payment orders';
+        $this->rights[$r][2] = 'a';
+        $this->rights[$r][3] = 0;
+        $this->rights[$r][4] = 'bons';
+        $this->rights[$r][5] = 'credit';
 
 /*        $this->rights[2][0] = 154;
         $this->rights[2][1] = 'Setup withdraw account';
@@ -132,32 +132,32 @@ 
         $this->rights[2][5] = 'configurer';
 */
 
-		// Menus
-		//-------
-		$this->menu = 1;        // This module add menu entries. They are coded into menu manager.
-	}
+        // Menus
+        //-------
+        $this->menu = 1;        // This module add menu entries. They are coded into menu manager.
+    }
 
 
-	/**
-	 *		Function called when module is enabled.
-	 *		The init function add constants, boxes, permissions and menus (defined in constructor) into Dolibarr database.
-	 *		It also creates data directories
-	 *
+    /**
+     *		Function called when module is enabled.
+     *		The init function add constants, boxes, permissions and menus (defined in constructor) into Dolibarr database.
+     *		It also creates data directories
+     *
      *      @param      string	$options    Options when enabling module ('', 'noboxes')
-	 *      @return     int             	1 if OK, 0 if KO
-	 */
-	function init($options='')
-	{
-		global $conf;
-
-		// Permissions
-		$this->remove($options);
-
-		$sql = array(
-		    "DELETE FROM ".MAIN_DB_PREFIX."document_model WHERE nom = '".$this->db->escape($this->const[0][2])."' AND type = 'bankaccount' AND entity = ".$conf->entity,
-		    "INSERT INTO ".MAIN_DB_PREFIX."document_model (nom, type, entity) VALUES('".$this->db->escape($this->const[0][2])."','bankaccount',".$conf->entity.")",
-		);
-
-		return $this->_init($sql,$options);
-	}
+     *      @return     int             	1 if OK, 0 if KO
+     */
+    function init($options='')
+    {
+        global $conf;
+
+        // Permissions
+        $this->remove($options);
+
+        $sql = array(
+            "DELETE FROM ".MAIN_DB_PREFIX."document_model WHERE nom = '".$this->db->escape($this->const[0][2])."' AND type = 'bankaccount' AND entity = ".$conf->entity,
+            "INSERT INTO ".MAIN_DB_PREFIX."document_model (nom, type, entity) VALUES('".$this->db->escape($this->const[0][2])."','bankaccount',".$conf->entity.")",
+        );
+
+        return $this->_init($sql,$options);
+    }
 }

Spacing +14 added lines, -14 removed lines

@@ -26,7 +26,7 @@ 
  *	\brief      	Fichier de description et activation du module Prelevement
  */
 
-include_once DOL_DOCUMENT_ROOT .'/core/modules/DolibarrModules.class.php';
+include_once DOL_DOCUMENT_ROOT.'/core/modules/DolibarrModules.class.php';
 
 
 /**
@@ -50,7 +50,7 @@ 
 		$this->family = "financial";
 		$this->module_position = '52';
 		// Module label (no space allowed), used if translation string 'ModuleXXXName' not found (where XXX is value of numeric property 'numero' of module)
-		$this->name = preg_replace('/^mod/i','',get_class($this));
+		$this->name = preg_replace('/^mod/i', '', get_class($this));
 		$this->description = "Gestion des Prelevements";
 
 		// Possible values for version are: 'development', 'experimental', 'dolibarr' or version
@@ -58,24 +58,24 @@ 
 
 		$this->const_name = 'MAIN_MODULE_'.strtoupper($this->name);
 		// Name of png file (without png) used for this module
-		$this->picto='payment';
+		$this->picto = 'payment';
 
 		// Data directories to create when module is enabled
-		$this->dirs = array("/prelevement/temp","/prelevement/receipts");
+		$this->dirs = array("/prelevement/temp", "/prelevement/receipts");
 
 		// Dependencies
-		$this->hidden = false;			// A condition to hide module
-		$this->depends = array("modFacture","modBanque");		// List of module class names as string that must be enabled if this module is enabled
-		$this->requiredby = array();	// List of module ids to disable if this one is disabled
-		$this->conflictwith = array();	// List of module class names as string this module is in conflict with
-		$this->phpmin = array(5,4);		// Minimum version of PHP required by module
+		$this->hidden = false; // A condition to hide module
+		$this->depends = array("modFacture", "modBanque"); // List of module class names as string that must be enabled if this module is enabled
+		$this->requiredby = array(); // List of module ids to disable if this one is disabled
+		$this->conflictwith = array(); // List of module class names as string this module is in conflict with
+		$this->phpmin = array(5, 4); // Minimum version of PHP required by module
 
 		// Config pages
 		$this->config_page_url = array("prelevement.php");
 
 		// Constants
 		$this->const = array();
-		$r=0;
+		$r = 0;
 
 		$this->const[$r][0] = "BANK_ADDON_PDF";
 		$this->const[$r][1] = "chaine";
@@ -91,7 +91,7 @@ 
 		// Permissions
 		$this->rights = array();
 		$this->rights_class = 'prelevement';
-		$r=0;
+		$r = 0;
 		$r++;
 		$this->rights[$r][0] = 151;
 		$this->rights[$r][1] = 'Read direct debit payment orders';
@@ -134,7 +134,7 @@ 
 
 		// Menus
 		//-------
-		$this->menu = 1;        // This module add menu entries. They are coded into menu manager.
+		$this->menu = 1; // This module add menu entries. They are coded into menu manager.
 	}
 
 
@@ -146,7 +146,7 @@ 
      *      @param      string	$options    Options when enabling module ('', 'noboxes')
 	 *      @return     int             	1 if OK, 0 if KO
 	 */
-	function init($options='')
+	function init($options = '')
 	{
 		global $conf;
 
@@ -158,6 +158,6 @@ 
 		    "INSERT INTO ".MAIN_DB_PREFIX."document_model (nom, type, entity) VALUES('".$this->db->escape($this->const[0][2])."','bankaccount',".$conf->entity.")",
 		);
 
-		return $this->_init($sql,$options);
+		return $this->_init($sql, $options);
 	}
 }

dolibarr/htdocs/core/modules/user/doc/doc_generic_user_odt.modules.php

Indentation +404 added lines, -404 removed lines

@@ -36,409 +36,409 @@ 
  */
 class doc_generic_user_odt extends ModelePDFUser
 {
-	/**
-	 * Issuer
-	 * @var Societe
-	 */
-	public $emetteur;
-
-	/**
-   * @var array() Minimum version of PHP required by module.
-	 * e.g.: PHP ≥ 5.4 = array(5, 4)
-   */
-	public $phpmin = array(5, 4);
-
-	/**
-   * Dolibarr version of the loaded document
-   * @public string
-   */
-	public $version = 'dolibarr';
-
-
-	/**
-	 *	Constructor
-	 *
-	 *  @param		DoliDB		$db      Database handler
-	 */
-	function __construct($db)
-	{
-		global $conf, $langs, $mysoc;
-
-		// Load translation files required by the page
+    /**
+     * Issuer
+     * @var Societe
+     */
+    public $emetteur;
+
+    /**
+     * @var array() Minimum version of PHP required by module.
+     * e.g.: PHP ≥ 5.4 = array(5, 4)
+     */
+    public $phpmin = array(5, 4);
+
+    /**
+     * Dolibarr version of the loaded document
+     * @public string
+     */
+    public $version = 'dolibarr';
+
+
+    /**
+     *	Constructor
+     *
+     *  @param		DoliDB		$db      Database handler
+     */
+    function __construct($db)
+    {
+        global $conf, $langs, $mysoc;
+
+        // Load translation files required by the page
     $langs->loadLangs(array("main","companies"));
 
-		$this->db = $db;
-		$this->name = "ODT templates";
-		$this->description = $langs->trans("DocumentModelOdt");
-		$this->scandir = 'USER_ADDON_PDF_ODT_PATH';	// Name of constant that is used to save list of directories to scan
-
-		// Dimension page pour format A4
-		$this->type = 'odt';
-		$this->page_largeur = 0;
-		$this->page_hauteur = 0;
-		$this->format = array($this->page_largeur,$this->page_hauteur);
-		$this->marge_gauche=0;
-		$this->marge_droite=0;
-		$this->marge_haute=0;
-		$this->marge_basse=0;
-
-		$this->option_logo = 1;                    // Affiche logo
-		$this->option_tva = 0;                     // Gere option tva USER_TVAOPTION
-		$this->option_modereg = 0;                 // Affiche mode reglement
-		$this->option_condreg = 0;                 // Affiche conditions reglement
-		$this->option_codeproduitservice = 0;      // Affiche code produit-service
-		$this->option_multilang = 1;               // Dispo en plusieurs langues
-		$this->option_escompte = 0;                // Affiche si il y a eu escompte
-		$this->option_credit_note = 0;             // Support credit notes
-		$this->option_freetext = 1;				   // Support add of a personalised text
-		$this->option_draft_watermark = 0;		   // Support add of a watermark on drafts
-
-		// Recupere emetteur
-		$this->emetteur=$mysoc;
-		if (! $this->emetteur->country_code) $this->emetteur->country_code=substr($langs->defaultlang,-2);    // By default if not defined
-	}
-
-
-	/**
-	 *	Return description of a module
-	 *
-	 *	@param	Translate	$langs      Lang object to use for output
-	 *	@return string       			Description
-	 */
-	function info($langs)
-	{
-		global $conf, $langs;
-
-		// Load translation files required by the page
+        $this->db = $db;
+        $this->name = "ODT templates";
+        $this->description = $langs->trans("DocumentModelOdt");
+        $this->scandir = 'USER_ADDON_PDF_ODT_PATH';	// Name of constant that is used to save list of directories to scan
+
+        // Dimension page pour format A4
+        $this->type = 'odt';
+        $this->page_largeur = 0;
+        $this->page_hauteur = 0;
+        $this->format = array($this->page_largeur,$this->page_hauteur);
+        $this->marge_gauche=0;
+        $this->marge_droite=0;
+        $this->marge_haute=0;
+        $this->marge_basse=0;
+
+        $this->option_logo = 1;                    // Affiche logo
+        $this->option_tva = 0;                     // Gere option tva USER_TVAOPTION
+        $this->option_modereg = 0;                 // Affiche mode reglement
+        $this->option_condreg = 0;                 // Affiche conditions reglement
+        $this->option_codeproduitservice = 0;      // Affiche code produit-service
+        $this->option_multilang = 1;               // Dispo en plusieurs langues
+        $this->option_escompte = 0;                // Affiche si il y a eu escompte
+        $this->option_credit_note = 0;             // Support credit notes
+        $this->option_freetext = 1;				   // Support add of a personalised text
+        $this->option_draft_watermark = 0;		   // Support add of a watermark on drafts
+
+        // Recupere emetteur
+        $this->emetteur=$mysoc;
+        if (! $this->emetteur->country_code) $this->emetteur->country_code=substr($langs->defaultlang,-2);    // By default if not defined
+    }
+
+
+    /**
+     *	Return description of a module
+     *
+     *	@param	Translate	$langs      Lang object to use for output
+     *	@return string       			Description
+     */
+    function info($langs)
+    {
+        global $conf, $langs;
+
+        // Load translation files required by the page
         $langs->loadLangs(array('companies', 'errors'));
 
-		$form = new Form($this->db);
-
-		$texte = $this->description.".<br>\n";
-		$texte.= '<form action="'.$_SERVER["PHP_SELF"].'" method="POST">';
-		$texte.= '<input type="hidden" name="token" value="'.$_SESSION['newtoken'].'">';
-		$texte.= '<input type="hidden" name="action" value="setModuleOptions">';
-		$texte.= '<input type="hidden" name="param1" value="USER_ADDON_PDF_ODT_PATH">';
-		if ($conf->global->MAIN_PROPAL_CHOOSE_ODT_DOCUMENT > 0)
-		{
-			$texte.= '<input type="hidden" name="param2" value="USER_ADDON_PDF_ODT_DEFAULT">';
-			$texte.= '<input type="hidden" name="param3" value="USER_ADDON_PDF_ODT_TOBILL">';
-			$texte.= '<input type="hidden" name="param4" value="USER_ADDON_PDF_ODT_CLOSED">';
-		}
-		$texte.= '<table class="nobordernopadding" width="100%">';
-
-		// List of directories area
-		$texte.= '<tr><td>';
-		$texttitle=$langs->trans("ListOfDirectories");
-		$listofdir=explode(',',preg_replace('/[\r\n]+/',',',trim($conf->global->USER_ADDON_PDF_ODT_PATH)));
-		$listoffiles=array();
-		foreach($listofdir as $key=>$tmpdir)
-		{
-			$tmpdir=trim($tmpdir);
-			$tmpdir=preg_replace('/DOL_DATA_ROOT/',DOL_DATA_ROOT,$tmpdir);
-			if (! $tmpdir) {
-				unset($listofdir[$key]); continue;
-			}
-			if (! is_dir($tmpdir)) $texttitle.=img_warning($langs->trans("ErrorDirNotFound",$tmpdir),0);
-			else
-			{
-				$tmpfiles=dol_dir_list($tmpdir,'files',0,'\.(ods|odt)');
-				if (count($tmpfiles)) $listoffiles=array_merge($listoffiles,$tmpfiles);
-			}
-		}
-		$texthelp=$langs->trans("ListOfDirectoriesForModelGenODT");
-		// Add list of substitution keys
-		$texthelp.='<br>'.$langs->trans("FollowingSubstitutionKeysCanBeUsed").'<br>';
-		$texthelp.=$langs->transnoentitiesnoconv("FullListOnOnlineDocumentation");    // This contains an url, we don't modify it
-
-		$texte.= $form->textwithpicto($texttitle,$texthelp,1,'help','',1);
-		$texte.= '<div><div style="display: inline-block; min-width: 100px; vertical-align: middle;">';
-		$texte.= '<textarea class="flat" cols="60" name="value1">';
-		$texte.=$conf->global->USER_ADDON_PDF_ODT_PATH;
-		$texte.= '</textarea>';
-		$texte.= '</div><div style="display: inline-block; vertical-align: middle;">';
-		$texte.= '<input type="submit" class="button" value="'.$langs->trans("Modify").'" name="Button">';
-		$texte.= '<br></div></div>';
-
-		// Scan directories
-		if (count($listofdir))
-		{
-			$texte.=$langs->trans("NumberOfModelFilesFound").': <b>'.count($listoffiles).'</b>';
-
-			if ($conf->global->MAIN_PROPAL_CHOOSE_ODT_DOCUMENT > 0)
-			{
-				// Model for creation
-				$liste=ModelePDFUser::liste_modeles($this->db);
-				$texte.= '<table width="50%;">';
-				$texte.= '<tr>';
-				$texte.= '<td width="60%;">'.$langs->trans("DefaultModelPropalCreate").'</td>';
-				$texte.= '<td colspan="">';
-				$texte.= $form->selectarray('value2',$liste,$conf->global->USER_ADDON_PDF_ODT_DEFAULT);
-				$texte.= "</td></tr>";
-
-				$texte.= '<tr>';
-				$texte.= '<td width="60%;">'.$langs->trans("DefaultModelPropalToBill").'</td>';
-				$texte.= '<td colspan="">';
-				$texte.= $form->selectarray('value3',$liste,$conf->global->USER_ADDON_PDF_ODT_TOBILL);
-				$texte.= "</td></tr>";
-				$texte.= '<tr>';
-
-				$texte.= '<td width="60%;">'.$langs->trans("DefaultModelPropalClosed").'</td>';
-				$texte.= '<td colspan="">';
-				$texte.= $form->selectarray('value4',$liste,$conf->global->USER_ADDON_PDF_ODT_CLOSED);
-				$texte.= "</td></tr>";
-				$texte.= '</table>';
-			}
-		}
-
-		$texte.= '</td>';
-
-		$texte.= '<td valign="top" rowspan="2" class="hideonsmartphone">';
-		$texte.= $langs->trans("ExampleOfDirectoriesForModelGen");
-		$texte.= '</td>';
-		$texte.= '</tr>';
-
-		$texte.= '</table>';
-		$texte.= '</form>';
-
-		return $texte;
-	}
+        $form = new Form($this->db);
+
+        $texte = $this->description.".<br>\n";
+        $texte.= '<form action="'.$_SERVER["PHP_SELF"].'" method="POST">';
+        $texte.= '<input type="hidden" name="token" value="'.$_SESSION['newtoken'].'">';
+        $texte.= '<input type="hidden" name="action" value="setModuleOptions">';
+        $texte.= '<input type="hidden" name="param1" value="USER_ADDON_PDF_ODT_PATH">';
+        if ($conf->global->MAIN_PROPAL_CHOOSE_ODT_DOCUMENT > 0)
+        {
+            $texte.= '<input type="hidden" name="param2" value="USER_ADDON_PDF_ODT_DEFAULT">';
+            $texte.= '<input type="hidden" name="param3" value="USER_ADDON_PDF_ODT_TOBILL">';
+            $texte.= '<input type="hidden" name="param4" value="USER_ADDON_PDF_ODT_CLOSED">';
+        }
+        $texte.= '<table class="nobordernopadding" width="100%">';
+
+        // List of directories area
+        $texte.= '<tr><td>';
+        $texttitle=$langs->trans("ListOfDirectories");
+        $listofdir=explode(',',preg_replace('/[\r\n]+/',',',trim($conf->global->USER_ADDON_PDF_ODT_PATH)));
+        $listoffiles=array();
+        foreach($listofdir as $key=>$tmpdir)
+        {
+            $tmpdir=trim($tmpdir);
+            $tmpdir=preg_replace('/DOL_DATA_ROOT/',DOL_DATA_ROOT,$tmpdir);
+            if (! $tmpdir) {
+                unset($listofdir[$key]); continue;
+            }
+            if (! is_dir($tmpdir)) $texttitle.=img_warning($langs->trans("ErrorDirNotFound",$tmpdir),0);
+            else
+            {
+                $tmpfiles=dol_dir_list($tmpdir,'files',0,'\.(ods|odt)');
+                if (count($tmpfiles)) $listoffiles=array_merge($listoffiles,$tmpfiles);
+            }
+        }
+        $texthelp=$langs->trans("ListOfDirectoriesForModelGenODT");
+        // Add list of substitution keys
+        $texthelp.='<br>'.$langs->trans("FollowingSubstitutionKeysCanBeUsed").'<br>';
+        $texthelp.=$langs->transnoentitiesnoconv("FullListOnOnlineDocumentation");    // This contains an url, we don't modify it
+
+        $texte.= $form->textwithpicto($texttitle,$texthelp,1,'help','',1);
+        $texte.= '<div><div style="display: inline-block; min-width: 100px; vertical-align: middle;">';
+        $texte.= '<textarea class="flat" cols="60" name="value1">';
+        $texte.=$conf->global->USER_ADDON_PDF_ODT_PATH;
+        $texte.= '</textarea>';
+        $texte.= '</div><div style="display: inline-block; vertical-align: middle;">';
+        $texte.= '<input type="submit" class="button" value="'.$langs->trans("Modify").'" name="Button">';
+        $texte.= '<br></div></div>';
+
+        // Scan directories
+        if (count($listofdir))
+        {
+            $texte.=$langs->trans("NumberOfModelFilesFound").': <b>'.count($listoffiles).'</b>';
+
+            if ($conf->global->MAIN_PROPAL_CHOOSE_ODT_DOCUMENT > 0)
+            {
+                // Model for creation
+                $liste=ModelePDFUser::liste_modeles($this->db);
+                $texte.= '<table width="50%;">';
+                $texte.= '<tr>';
+                $texte.= '<td width="60%;">'.$langs->trans("DefaultModelPropalCreate").'</td>';
+                $texte.= '<td colspan="">';
+                $texte.= $form->selectarray('value2',$liste,$conf->global->USER_ADDON_PDF_ODT_DEFAULT);
+                $texte.= "</td></tr>";
+
+                $texte.= '<tr>';
+                $texte.= '<td width="60%;">'.$langs->trans("DefaultModelPropalToBill").'</td>';
+                $texte.= '<td colspan="">';
+                $texte.= $form->selectarray('value3',$liste,$conf->global->USER_ADDON_PDF_ODT_TOBILL);
+                $texte.= "</td></tr>";
+                $texte.= '<tr>';
+
+                $texte.= '<td width="60%;">'.$langs->trans("DefaultModelPropalClosed").'</td>';
+                $texte.= '<td colspan="">';
+                $texte.= $form->selectarray('value4',$liste,$conf->global->USER_ADDON_PDF_ODT_CLOSED);
+                $texte.= "</td></tr>";
+                $texte.= '</table>';
+            }
+        }
+
+        $texte.= '</td>';
+
+        $texte.= '<td valign="top" rowspan="2" class="hideonsmartphone">';
+        $texte.= $langs->trans("ExampleOfDirectoriesForModelGen");
+        $texte.= '</td>';
+        $texte.= '</tr>';
+
+        $texte.= '</table>';
+        $texte.= '</form>';
+
+        return $texte;
+    }
 
     // phpcs:disable PEAR.NamingConventions.ValidFunctionName.NotCamelCaps
-	/**
-	 *	Function to build a document on disk using the generic odt module.
-	 *
-	 *	@param		User		$object				Object source to build document
-	 *	@param		Translate	$outputlangs		Lang output object
-	 * 	@param		string		$srctemplatepath	Full path of source filename for generator using a template file
-	 *  @param		int			$hidedetails		Do not show line details
-	 *  @param		int			$hidedesc			Do not show desc
-	 *  @param		int			$hideref			Do not show ref
-	 *	@return		int         					1 if OK, <=0 if KO
-	 */
-	function write_file($object,$outputlangs,$srctemplatepath,$hidedetails=0,$hidedesc=0,$hideref=0)
-	{
+    /**
+     *	Function to build a document on disk using the generic odt module.
+     *
+     *	@param		User		$object				Object source to build document
+     *	@param		Translate	$outputlangs		Lang output object
+     * 	@param		string		$srctemplatepath	Full path of source filename for generator using a template file
+     *  @param		int			$hidedetails		Do not show line details
+     *  @param		int			$hidedesc			Do not show desc
+     *  @param		int			$hideref			Do not show ref
+     *	@return		int         					1 if OK, <=0 if KO
+     */
+    function write_file($object,$outputlangs,$srctemplatepath,$hidedetails=0,$hidedesc=0,$hideref=0)
+    {
         // phpcs:enable
-		global $user, $langs, $conf, $mysoc, $hookmanager;
-
-		if (empty($srctemplatepath))
-		{
-			dol_syslog("doc_generic_odt::write_file parameter srctemplatepath empty", LOG_WARNING);
-			return -1;
-		}
-
-		// Add odtgeneration hook
-		if (! is_object($hookmanager))
-		{
-			include_once DOL_DOCUMENT_ROOT.'/core/class/hookmanager.class.php';
-			$hookmanager=new HookManager($this->db);
-		}
-		$hookmanager->initHooks(array('odtgeneration'));
-		global $action;
-
-		if (! is_object($outputlangs)) $outputlangs=$langs;
-		$sav_charset_output=$outputlangs->charset_output;
-		$outputlangs->charset_output='UTF-8';
-
-		// Load translation files required by the page
-		$outputlangs->loadLangs(array("main", "companies", "bills", "dict"));
-
-		if ($conf->user->dir_output)
-		{
-			// If $object is id instead of object
-			if (! is_object($object))
-			{
-				$id = $object;
-				$object = new User($this->db);
-				$result=$object->fetch($id);
-				if ($result < 0)
-				{
-					dol_print_error($this->db,$object->error);
-					return -1;
-				}
-			}
-
-			$dir = $conf->user->dir_output;
-			$objectref = dol_sanitizeFileName($object->ref);
-			if (! preg_match('/specimen/i',$objectref)) $dir.= "/" . $objectref;
-			$file = $dir . "/" . $objectref . ".odt";
-
-			if (! file_exists($dir))
-			{
-				if (dol_mkdir($dir) < 0)
-				{
-					$this->error=$langs->transnoentities("ErrorCanNotCreateDir",$dir);
-					return -1;
-				}
-			}
-
-			if (file_exists($dir))
-			{
-				//print "srctemplatepath=".$srctemplatepath;	// Src filename
-				$newfile=basename($srctemplatepath);
-				$newfiletmp=preg_replace('/\.od(t|s)/i','',$newfile);
-				$newfiletmp=preg_replace('/template_/i','',$newfiletmp);
-				$newfiletmp=preg_replace('/modele_/i','',$newfiletmp);
-
-				$newfiletmp=$objectref.'_'.$newfiletmp;
-
-				// Get extension (ods or odt)
-				$newfileformat=substr($newfile, strrpos($newfile, '.')+1);
-				if ( ! empty($conf->global->MAIN_DOC_USE_TIMING))
-				{
-				    $format=$conf->global->MAIN_DOC_USE_TIMING;
-				    if ($format == '1') $format='%Y%m%d%H%M%S';
-					$filename=$newfiletmp.'-'.dol_print_date(dol_now(),$format).'.'.$newfileformat;
-				}
-				else
-				{
-					$filename=$newfiletmp.'.'.$newfileformat;
-				}
-				$file=$dir.'/'.$filename;
-				//print "newdir=".$dir;
-				//print "newfile=".$newfile;
-				//print "file=".$file;
-				//print "conf->user->dir_temp=".$conf->user->dir_temp;
-
-				dol_mkdir($conf->user->dir_temp);
-
-
-				// If CUSTOMER contact defined on user, we use it
-				$usecontact=false;
-				$arrayidcontact=$object->getIdContact('external','CUSTOMER');
-				if (count($arrayidcontact) > 0)
-				{
-					$usecontact=true;
-					$result=$object->fetch_contact($arrayidcontact[0]);
-				}
-
-				// Recipient name
-				if (! empty($usecontact))
-				{
-					// On peut utiliser le nom de la societe du contact
-					if (! empty($conf->global->MAIN_USE_COMPANY_NAME_OF_CONTACT)) $socobject = $object->contact;
-					else {
-                        			$socobject = $object->thirdparty;
-                        			// if we have a CUSTOMER contact and we dont use it as recipient we store the contact object for later use
-                        			$contactobject = $object->contact;
-                    			}
-				}
-				else
-				{
-					$socobject=$object->thirdparty;
-				}
-
-				// Open and load template
-				require_once ODTPHP_PATH.'odf.php';
-				try {
-					$odfHandler = new odf(
-						$srctemplatepath,
-						array(
-							'PATH_TO_TMP'	  => $conf->user->dir_temp,
-							'ZIP_PROXY'		  => 'PclZipProxy',	// PhpZipProxy or PclZipProxy. Got "bad compression method" error when using PhpZipProxy.
-							'DELIMITER_LEFT'  => '{',
-							'DELIMITER_RIGHT' => '}'
-						)
-					);
-				}
-				catch(Exception $e)
-				{
-					$this->error=$e->getMessage();
-					dol_syslog($e->getMessage(), LOG_WARNING);
-					return -1;
-				}
-
-				// Make substitutions into odt
-				$array_user=$this->get_substitutionarray_user($object,$outputlangs);
-				$array_soc=$this->get_substitutionarray_mysoc($mysoc,$outputlangs);
-				$array_thirdparty=$this->get_substitutionarray_thirdparty($socobject,$outputlangs);
-				$array_other=$this->get_substitutionarray_other($outputlangs);
-				// retrieve contact information for use in object as contact_xxx tags
-				$array_thirdparty_contact = array();
-				if ($usecontact && is_object($contactobject)) $array_thirdparty_contact=$this->get_substitutionarray_contact($contactobject,$outputlangs,'contact');
-
-				$tmparray = array_merge($array_user,$array_soc,$array_thirdparty,$array_other,$array_thirdparty_contact);
-				complete_substitutions_array($tmparray, $outputlangs, $object);
-				$object->fetch_optionals();
-				// Call the ODTSubstitution hook
-				$parameters=array('file'=>$file,'object'=>$object,'outputlangs'=>$outputlangs,'substitutionarray'=>&$tmparray);
-				$reshook=$hookmanager->executeHooks('ODTSubstitution',$parameters,$this,$action);    // Note that $action and $object may have been modified by some hooks
-				foreach($tmparray as $key=>$value)
-				{
-					try {
-						if (preg_match('/logo$/',$key)) // Image
-						{
-							if (file_exists($value)) $odfHandler->setImage($key, $value);
-							else $odfHandler->setVars($key, 'ErrorFileNotFound', true, 'UTF-8');
-						}
-						else    // Text
-						{
-							$odfHandler->setVars($key, $value, true, 'UTF-8');
-						}
-					}
-					catch(OdfException $e)
-					{
-						dol_syslog($e->getMessage(), LOG_WARNING);
-					}
-				}
-
-				// Replace labels translated
-				$tmparray=$outputlangs->get_translations_for_substitutions();
-				foreach($tmparray as $key=>$value)
-				{
-					try {
-						$odfHandler->setVars($key, $value, true, 'UTF-8');
-					}
-					catch (OdfException $e)
-					{
-						dol_syslog($e->getMessage(), LOG_WARNING);
-					}
-				}
-
-				// Call the beforeODTSave hook
-				$parameters=array('odfHandler'=>&$odfHandler,'file'=>$file,'object'=>$object,'outputlangs'=>$outputlangs);
-				$reshook=$hookmanager->executeHooks('beforeODTSave',$parameters,$this,$action);    // Note that $action and $object may have been modified by some hooks
-
-				// Write new file
-				if (!empty($conf->global->MAIN_ODT_AS_PDF)) {
-					try {
-						$odfHandler->exportAsAttachedPDF($file);
-					} catch (Exception $e) {
-						$this->error=$e->getMessage();
-						dol_syslog($e->getMessage(), LOG_WARNING);
-						return -1;
-					}
-				}
-				else {
-					try {
-						$odfHandler->saveToDisk($file);
-					} catch (Exception $e) {
-						$this->error=$e->getMessage();
-						dol_syslog($e->getMessage(), LOG_WARNING);
-						return -1;
-					}
-				}
-
-				$reshook=$hookmanager->executeHooks('afterODTCreation',$parameters,$this,$action);    // Note that $action and $object may have been modified by some hooks
-
-				if (! empty($conf->global->MAIN_UMASK))
-					@chmod($file, octdec($conf->global->MAIN_UMASK));
-
-				$odfHandler=null;	// Destroy object
-
-				$this->result = array('fullpath'=>$file);
-
-				return 1;   // Success
-			}
-			else
-			{
-				$this->error=$langs->transnoentities("ErrorCanNotCreateDir",$dir);
-				return -1;
-			}
-		}
-
-		return -1;
-	}
+        global $user, $langs, $conf, $mysoc, $hookmanager;
+
+        if (empty($srctemplatepath))
+        {
+            dol_syslog("doc_generic_odt::write_file parameter srctemplatepath empty", LOG_WARNING);
+            return -1;
+        }
+
+        // Add odtgeneration hook
+        if (! is_object($hookmanager))
+        {
+            include_once DOL_DOCUMENT_ROOT.'/core/class/hookmanager.class.php';
+            $hookmanager=new HookManager($this->db);
+        }
+        $hookmanager->initHooks(array('odtgeneration'));
+        global $action;
+
+        if (! is_object($outputlangs)) $outputlangs=$langs;
+        $sav_charset_output=$outputlangs->charset_output;
+        $outputlangs->charset_output='UTF-8';
+
+        // Load translation files required by the page
+        $outputlangs->loadLangs(array("main", "companies", "bills", "dict"));
+
+        if ($conf->user->dir_output)
+        {
+            // If $object is id instead of object
+            if (! is_object($object))
+            {
+                $id = $object;
+                $object = new User($this->db);
+                $result=$object->fetch($id);
+                if ($result < 0)
+                {
+                    dol_print_error($this->db,$object->error);
+                    return -1;
+                }
+            }
+
+            $dir = $conf->user->dir_output;
+            $objectref = dol_sanitizeFileName($object->ref);
+            if (! preg_match('/specimen/i',$objectref)) $dir.= "/" . $objectref;
+            $file = $dir . "/" . $objectref . ".odt";
+
+            if (! file_exists($dir))
+            {
+                if (dol_mkdir($dir) < 0)
+                {
+                    $this->error=$langs->transnoentities("ErrorCanNotCreateDir",$dir);
+                    return -1;
+                }
+            }
+
+            if (file_exists($dir))
+            {
+                //print "srctemplatepath=".$srctemplatepath;	// Src filename
+                $newfile=basename($srctemplatepath);
+                $newfiletmp=preg_replace('/\.od(t|s)/i','',$newfile);
+                $newfiletmp=preg_replace('/template_/i','',$newfiletmp);
+                $newfiletmp=preg_replace('/modele_/i','',$newfiletmp);
+
+                $newfiletmp=$objectref.'_'.$newfiletmp;
+
+                // Get extension (ods or odt)
+                $newfileformat=substr($newfile, strrpos($newfile, '.')+1);
+                if ( ! empty($conf->global->MAIN_DOC_USE_TIMING))
+                {
+                    $format=$conf->global->MAIN_DOC_USE_TIMING;
+                    if ($format == '1') $format='%Y%m%d%H%M%S';
+                    $filename=$newfiletmp.'-'.dol_print_date(dol_now(),$format).'.'.$newfileformat;
+                }
+                else
+                {
+                    $filename=$newfiletmp.'.'.$newfileformat;
+                }
+                $file=$dir.'/'.$filename;
+                //print "newdir=".$dir;
+                //print "newfile=".$newfile;
+                //print "file=".$file;
+                //print "conf->user->dir_temp=".$conf->user->dir_temp;
+
+                dol_mkdir($conf->user->dir_temp);
+
+
+                // If CUSTOMER contact defined on user, we use it
+                $usecontact=false;
+                $arrayidcontact=$object->getIdContact('external','CUSTOMER');
+                if (count($arrayidcontact) > 0)
+                {
+                    $usecontact=true;
+                    $result=$object->fetch_contact($arrayidcontact[0]);
+                }
+
+                // Recipient name
+                if (! empty($usecontact))
+                {
+                    // On peut utiliser le nom de la societe du contact
+                    if (! empty($conf->global->MAIN_USE_COMPANY_NAME_OF_CONTACT)) $socobject = $object->contact;
+                    else {
+                                    $socobject = $object->thirdparty;
+                                    // if we have a CUSTOMER contact and we dont use it as recipient we store the contact object for later use
+                                    $contactobject = $object->contact;
+                                }
+                }
+                else
+                {
+                    $socobject=$object->thirdparty;
+                }
+
+                // Open and load template
+                require_once ODTPHP_PATH.'odf.php';
+                try {
+                    $odfHandler = new odf(
+                        $srctemplatepath,
+                        array(
+                            'PATH_TO_TMP'	  => $conf->user->dir_temp,
+                            'ZIP_PROXY'		  => 'PclZipProxy',	// PhpZipProxy or PclZipProxy. Got "bad compression method" error when using PhpZipProxy.
+                            'DELIMITER_LEFT'  => '{',
+                            'DELIMITER_RIGHT' => '}'
+                        )
+                    );
+                }
+                catch(Exception $e)
+                {
+                    $this->error=$e->getMessage();
+                    dol_syslog($e->getMessage(), LOG_WARNING);
+                    return -1;
+                }
+
+                // Make substitutions into odt
+                $array_user=$this->get_substitutionarray_user($object,$outputlangs);
+                $array_soc=$this->get_substitutionarray_mysoc($mysoc,$outputlangs);
+                $array_thirdparty=$this->get_substitutionarray_thirdparty($socobject,$outputlangs);
+                $array_other=$this->get_substitutionarray_other($outputlangs);
+                // retrieve contact information for use in object as contact_xxx tags
+                $array_thirdparty_contact = array();
+                if ($usecontact && is_object($contactobject)) $array_thirdparty_contact=$this->get_substitutionarray_contact($contactobject,$outputlangs,'contact');
+
+                $tmparray = array_merge($array_user,$array_soc,$array_thirdparty,$array_other,$array_thirdparty_contact);
+                complete_substitutions_array($tmparray, $outputlangs, $object);
+                $object->fetch_optionals();
+                // Call the ODTSubstitution hook
+                $parameters=array('file'=>$file,'object'=>$object,'outputlangs'=>$outputlangs,'substitutionarray'=>&$tmparray);
+                $reshook=$hookmanager->executeHooks('ODTSubstitution',$parameters,$this,$action);    // Note that $action and $object may have been modified by some hooks
+                foreach($tmparray as $key=>$value)
+                {
+                    try {
+                        if (preg_match('/logo$/',$key)) // Image
+                        {
+                            if (file_exists($value)) $odfHandler->setImage($key, $value);
+                            else $odfHandler->setVars($key, 'ErrorFileNotFound', true, 'UTF-8');
+                        }
+                        else    // Text
+                        {
+                            $odfHandler->setVars($key, $value, true, 'UTF-8');
+                        }
+                    }
+                    catch(OdfException $e)
+                    {
+                        dol_syslog($e->getMessage(), LOG_WARNING);
+                    }
+                }
+
+                // Replace labels translated
+                $tmparray=$outputlangs->get_translations_for_substitutions();
+                foreach($tmparray as $key=>$value)
+                {
+                    try {
+                        $odfHandler->setVars($key, $value, true, 'UTF-8');
+                    }
+                    catch (OdfException $e)
+                    {
+                        dol_syslog($e->getMessage(), LOG_WARNING);
+                    }
+                }
+
+                // Call the beforeODTSave hook
+                $parameters=array('odfHandler'=>&$odfHandler,'file'=>$file,'object'=>$object,'outputlangs'=>$outputlangs);
+                $reshook=$hookmanager->executeHooks('beforeODTSave',$parameters,$this,$action);    // Note that $action and $object may have been modified by some hooks
+
+                // Write new file
+                if (!empty($conf->global->MAIN_ODT_AS_PDF)) {
+                    try {
+                        $odfHandler->exportAsAttachedPDF($file);
+                    } catch (Exception $e) {
+                        $this->error=$e->getMessage();
+                        dol_syslog($e->getMessage(), LOG_WARNING);
+                        return -1;
+                    }
+                }
+                else {
+                    try {
+                        $odfHandler->saveToDisk($file);
+                    } catch (Exception $e) {
+                        $this->error=$e->getMessage();
+                        dol_syslog($e->getMessage(), LOG_WARNING);
+                        return -1;
+                    }
+                }
+
+                $reshook=$hookmanager->executeHooks('afterODTCreation',$parameters,$this,$action);    // Note that $action and $object may have been modified by some hooks
+
+                if (! empty($conf->global->MAIN_UMASK))
+                    @chmod($file, octdec($conf->global->MAIN_UMASK));
+
+                $odfHandler=null;	// Destroy object
+
+                $this->result = array('fullpath'=>$file);
+
+                return 1;   // Success
+            }
+            else
+            {
+                $this->error=$langs->transnoentities("ErrorCanNotCreateDir",$dir);
+                return -1;
+            }
+        }
+
+        return -1;
+    }
 
     // phpcs:disable PEAR.NamingConventions.ValidFunctionName.NotCamelCaps
     /**
@@ -452,12 +452,12 @@ 
     function get_substitutionarray_object($object,$outputlangs,$array_key='object')
     {
         // phpcs:enable
-		$array_other = array();
-		foreach($object as $key => $value) {
-			if (!is_array($value) && !is_object($value)) {
-				$array_other[$array_key.'_'.$key] = $value;
-			}
-		}
-		return $array_other;
-	}
+        $array_other = array();
+        foreach($object as $key => $value) {
+            if (!is_array($value) && !is_object($value)) {
+                $array_other[$array_key.'_'.$key] = $value;
+            }
+        }
+        return $array_other;
+    }
 }

Spacing +138 added lines, -138 removed lines

@@ -65,37 +65,37 @@ 
 		global $conf, $langs, $mysoc;
 
 		// Load translation files required by the page
-    $langs->loadLangs(array("main","companies"));
+    $langs->loadLangs(array("main", "companies"));
 
 		$this->db = $db;
 		$this->name = "ODT templates";
 		$this->description = $langs->trans("DocumentModelOdt");
-		$this->scandir = 'USER_ADDON_PDF_ODT_PATH';	// Name of constant that is used to save list of directories to scan
+		$this->scandir = 'USER_ADDON_PDF_ODT_PATH'; // Name of constant that is used to save list of directories to scan
 
 		// Dimension page pour format A4
 		$this->type = 'odt';
 		$this->page_largeur = 0;
 		$this->page_hauteur = 0;
-		$this->format = array($this->page_largeur,$this->page_hauteur);
-		$this->marge_gauche=0;
-		$this->marge_droite=0;
-		$this->marge_haute=0;
-		$this->marge_basse=0;
-
-		$this->option_logo = 1;                    // Affiche logo
-		$this->option_tva = 0;                     // Gere option tva USER_TVAOPTION
-		$this->option_modereg = 0;                 // Affiche mode reglement
-		$this->option_condreg = 0;                 // Affiche conditions reglement
-		$this->option_codeproduitservice = 0;      // Affiche code produit-service
-		$this->option_multilang = 1;               // Dispo en plusieurs langues
-		$this->option_escompte = 0;                // Affiche si il y a eu escompte
-		$this->option_credit_note = 0;             // Support credit notes
-		$this->option_freetext = 1;				   // Support add of a personalised text
-		$this->option_draft_watermark = 0;		   // Support add of a watermark on drafts
+		$this->format = array($this->page_largeur, $this->page_hauteur);
+		$this->marge_gauche = 0;
+		$this->marge_droite = 0;
+		$this->marge_haute = 0;
+		$this->marge_basse = 0;
+
+		$this->option_logo = 1; // Affiche logo
+		$this->option_tva = 0; // Gere option tva USER_TVAOPTION
+		$this->option_modereg = 0; // Affiche mode reglement
+		$this->option_condreg = 0; // Affiche conditions reglement
+		$this->option_codeproduitservice = 0; // Affiche code produit-service
+		$this->option_multilang = 1; // Dispo en plusieurs langues
+		$this->option_escompte = 0; // Affiche si il y a eu escompte
+		$this->option_credit_note = 0; // Support credit notes
+		$this->option_freetext = 1; // Support add of a personalised text
+		$this->option_draft_watermark = 0; // Support add of a watermark on drafts
 
 		// Recupere emetteur
-		$this->emetteur=$mysoc;
-		if (! $this->emetteur->country_code) $this->emetteur->country_code=substr($langs->defaultlang,-2);    // By default if not defined
+		$this->emetteur = $mysoc;
+		if (!$this->emetteur->country_code) $this->emetteur->country_code = substr($langs->defaultlang, -2); // By default if not defined
 	}
 
 
@@ -115,91 +115,91 @@ 
 		$form = new Form($this->db);
 
 		$texte = $this->description.".<br>\n";
-		$texte.= '<form action="'.$_SERVER["PHP_SELF"].'" method="POST">';
-		$texte.= '<input type="hidden" name="token" value="'.$_SESSION['newtoken'].'">';
-		$texte.= '<input type="hidden" name="action" value="setModuleOptions">';
-		$texte.= '<input type="hidden" name="param1" value="USER_ADDON_PDF_ODT_PATH">';
+		$texte .= '<form action="'.$_SERVER["PHP_SELF"].'" method="POST">';
+		$texte .= '<input type="hidden" name="token" value="'.$_SESSION['newtoken'].'">';
+		$texte .= '<input type="hidden" name="action" value="setModuleOptions">';
+		$texte .= '<input type="hidden" name="param1" value="USER_ADDON_PDF_ODT_PATH">';
 		if ($conf->global->MAIN_PROPAL_CHOOSE_ODT_DOCUMENT > 0)
 		{
-			$texte.= '<input type="hidden" name="param2" value="USER_ADDON_PDF_ODT_DEFAULT">';
-			$texte.= '<input type="hidden" name="param3" value="USER_ADDON_PDF_ODT_TOBILL">';
-			$texte.= '<input type="hidden" name="param4" value="USER_ADDON_PDF_ODT_CLOSED">';
+			$texte .= '<input type="hidden" name="param2" value="USER_ADDON_PDF_ODT_DEFAULT">';
+			$texte .= '<input type="hidden" name="param3" value="USER_ADDON_PDF_ODT_TOBILL">';
+			$texte .= '<input type="hidden" name="param4" value="USER_ADDON_PDF_ODT_CLOSED">';
 		}
-		$texte.= '<table class="nobordernopadding" width="100%">';
+		$texte .= '<table class="nobordernopadding" width="100%">';
 
 		// List of directories area
-		$texte.= '<tr><td>';
-		$texttitle=$langs->trans("ListOfDirectories");
-		$listofdir=explode(',',preg_replace('/[\r\n]+/',',',trim($conf->global->USER_ADDON_PDF_ODT_PATH)));
-		$listoffiles=array();
-		foreach($listofdir as $key=>$tmpdir)
+		$texte .= '<tr><td>';
+		$texttitle = $langs->trans("ListOfDirectories");
+		$listofdir = explode(',', preg_replace('/[\r\n]+/', ',', trim($conf->global->USER_ADDON_PDF_ODT_PATH)));
+		$listoffiles = array();
+		foreach ($listofdir as $key=>$tmpdir)
 		{
-			$tmpdir=trim($tmpdir);
-			$tmpdir=preg_replace('/DOL_DATA_ROOT/',DOL_DATA_ROOT,$tmpdir);
-			if (! $tmpdir) {
+			$tmpdir = trim($tmpdir);
+			$tmpdir = preg_replace('/DOL_DATA_ROOT/', DOL_DATA_ROOT, $tmpdir);
+			if (!$tmpdir) {
 				unset($listofdir[$key]); continue;
 			}
-			if (! is_dir($tmpdir)) $texttitle.=img_warning($langs->trans("ErrorDirNotFound",$tmpdir),0);
+			if (!is_dir($tmpdir)) $texttitle .= img_warning($langs->trans("ErrorDirNotFound", $tmpdir), 0);
 			else
 			{
-				$tmpfiles=dol_dir_list($tmpdir,'files',0,'\.(ods|odt)');
-				if (count($tmpfiles)) $listoffiles=array_merge($listoffiles,$tmpfiles);
+				$tmpfiles = dol_dir_list($tmpdir, 'files', 0, '\.(ods|odt)');
+				if (count($tmpfiles)) $listoffiles = array_merge($listoffiles, $tmpfiles);
 			}
 		}
-		$texthelp=$langs->trans("ListOfDirectoriesForModelGenODT");
+		$texthelp = $langs->trans("ListOfDirectoriesForModelGenODT");
 		// Add list of substitution keys
-		$texthelp.='<br>'.$langs->trans("FollowingSubstitutionKeysCanBeUsed").'<br>';
-		$texthelp.=$langs->transnoentitiesnoconv("FullListOnOnlineDocumentation");    // This contains an url, we don't modify it
-
-		$texte.= $form->textwithpicto($texttitle,$texthelp,1,'help','',1);
-		$texte.= '<div><div style="display: inline-block; min-width: 100px; vertical-align: middle;">';
-		$texte.= '<textarea class="flat" cols="60" name="value1">';
-		$texte.=$conf->global->USER_ADDON_PDF_ODT_PATH;
-		$texte.= '</textarea>';
-		$texte.= '</div><div style="display: inline-block; vertical-align: middle;">';
-		$texte.= '<input type="submit" class="button" value="'.$langs->trans("Modify").'" name="Button">';
-		$texte.= '<br></div></div>';
+		$texthelp .= '<br>'.$langs->trans("FollowingSubstitutionKeysCanBeUsed").'<br>';
+		$texthelp .= $langs->transnoentitiesnoconv("FullListOnOnlineDocumentation"); // This contains an url, we don't modify it
+
+		$texte .= $form->textwithpicto($texttitle, $texthelp, 1, 'help', '', 1);
+		$texte .= '<div><div style="display: inline-block; min-width: 100px; vertical-align: middle;">';
+		$texte .= '<textarea class="flat" cols="60" name="value1">';
+		$texte .= $conf->global->USER_ADDON_PDF_ODT_PATH;
+		$texte .= '</textarea>';
+		$texte .= '</div><div style="display: inline-block; vertical-align: middle;">';
+		$texte .= '<input type="submit" class="button" value="'.$langs->trans("Modify").'" name="Button">';
+		$texte .= '<br></div></div>';
 
 		// Scan directories
 		if (count($listofdir))
 		{
-			$texte.=$langs->trans("NumberOfModelFilesFound").': <b>'.count($listoffiles).'</b>';
+			$texte .= $langs->trans("NumberOfModelFilesFound").': <b>'.count($listoffiles).'</b>';
 
 			if ($conf->global->MAIN_PROPAL_CHOOSE_ODT_DOCUMENT > 0)
 			{
 				// Model for creation
-				$liste=ModelePDFUser::liste_modeles($this->db);
-				$texte.= '<table width="50%;">';
-				$texte.= '<tr>';
-				$texte.= '<td width="60%;">'.$langs->trans("DefaultModelPropalCreate").'</td>';
-				$texte.= '<td colspan="">';
-				$texte.= $form->selectarray('value2',$liste,$conf->global->USER_ADDON_PDF_ODT_DEFAULT);
-				$texte.= "</td></tr>";
-
-				$texte.= '<tr>';
-				$texte.= '<td width="60%;">'.$langs->trans("DefaultModelPropalToBill").'</td>';
-				$texte.= '<td colspan="">';
-				$texte.= $form->selectarray('value3',$liste,$conf->global->USER_ADDON_PDF_ODT_TOBILL);
-				$texte.= "</td></tr>";
-				$texte.= '<tr>';
-
-				$texte.= '<td width="60%;">'.$langs->trans("DefaultModelPropalClosed").'</td>';
-				$texte.= '<td colspan="">';
-				$texte.= $form->selectarray('value4',$liste,$conf->global->USER_ADDON_PDF_ODT_CLOSED);
-				$texte.= "</td></tr>";
-				$texte.= '</table>';
+				$liste = ModelePDFUser::liste_modeles($this->db);
+				$texte .= '<table width="50%;">';
+				$texte .= '<tr>';
+				$texte .= '<td width="60%;">'.$langs->trans("DefaultModelPropalCreate").'</td>';
+				$texte .= '<td colspan="">';
+				$texte .= $form->selectarray('value2', $liste, $conf->global->USER_ADDON_PDF_ODT_DEFAULT);
+				$texte .= "</td></tr>";
+
+				$texte .= '<tr>';
+				$texte .= '<td width="60%;">'.$langs->trans("DefaultModelPropalToBill").'</td>';
+				$texte .= '<td colspan="">';
+				$texte .= $form->selectarray('value3', $liste, $conf->global->USER_ADDON_PDF_ODT_TOBILL);
+				$texte .= "</td></tr>";
+				$texte .= '<tr>';
+
+				$texte .= '<td width="60%;">'.$langs->trans("DefaultModelPropalClosed").'</td>';
+				$texte .= '<td colspan="">';
+				$texte .= $form->selectarray('value4', $liste, $conf->global->USER_ADDON_PDF_ODT_CLOSED);
+				$texte .= "</td></tr>";
+				$texte .= '</table>';
 			}
 		}
 
-		$texte.= '</td>';
+		$texte .= '</td>';
 
-		$texte.= '<td valign="top" rowspan="2" class="hideonsmartphone">';
-		$texte.= $langs->trans("ExampleOfDirectoriesForModelGen");
-		$texte.= '</td>';
-		$texte.= '</tr>';
+		$texte .= '<td valign="top" rowspan="2" class="hideonsmartphone">';
+		$texte .= $langs->trans("ExampleOfDirectoriesForModelGen");
+		$texte .= '</td>';
+		$texte .= '</tr>';
 
-		$texte.= '</table>';
-		$texte.= '</form>';
+		$texte .= '</table>';
+		$texte .= '</form>';
 
 		return $texte;
 	}
@@ -216,7 +216,7 @@ 
 	 *  @param		int			$hideref			Do not show ref
 	 *	@return		int         					1 if OK, <=0 if KO
 	 */
-	function write_file($object,$outputlangs,$srctemplatepath,$hidedetails=0,$hidedesc=0,$hideref=0)
+	function write_file($object, $outputlangs, $srctemplatepath, $hidedetails = 0, $hidedesc = 0, $hideref = 0)
 	{
         // phpcs:enable
 		global $user, $langs, $conf, $mysoc, $hookmanager;
@@ -228,17 +228,17 @@ 
 		}
 
 		// Add odtgeneration hook
-		if (! is_object($hookmanager))
+		if (!is_object($hookmanager))
 		{
 			include_once DOL_DOCUMENT_ROOT.'/core/class/hookmanager.class.php';
-			$hookmanager=new HookManager($this->db);
+			$hookmanager = new HookManager($this->db);
 		}
 		$hookmanager->initHooks(array('odtgeneration'));
 		global $action;
 
-		if (! is_object($outputlangs)) $outputlangs=$langs;
-		$sav_charset_output=$outputlangs->charset_output;
-		$outputlangs->charset_output='UTF-8';
+		if (!is_object($outputlangs)) $outputlangs = $langs;
+		$sav_charset_output = $outputlangs->charset_output;
+		$outputlangs->charset_output = 'UTF-8';
 
 		// Load translation files required by the page
 		$outputlangs->loadLangs(array("main", "companies", "bills", "dict"));
@@ -246,28 +246,28 @@ 
 		if ($conf->user->dir_output)
 		{
 			// If $object is id instead of object
-			if (! is_object($object))
+			if (!is_object($object))
 			{
 				$id = $object;
 				$object = new User($this->db);
-				$result=$object->fetch($id);
+				$result = $object->fetch($id);
 				if ($result < 0)
 				{
-					dol_print_error($this->db,$object->error);
+					dol_print_error($this->db, $object->error);
 					return -1;
 				}
 			}
 
 			$dir = $conf->user->dir_output;
 			$objectref = dol_sanitizeFileName($object->ref);
-			if (! preg_match('/specimen/i',$objectref)) $dir.= "/" . $objectref;
-			$file = $dir . "/" . $objectref . ".odt";
+			if (!preg_match('/specimen/i', $objectref)) $dir .= "/".$objectref;
+			$file = $dir."/".$objectref.".odt";
 
-			if (! file_exists($dir))
+			if (!file_exists($dir))
 			{
 				if (dol_mkdir($dir) < 0)
 				{
-					$this->error=$langs->transnoentities("ErrorCanNotCreateDir",$dir);
+					$this->error = $langs->transnoentities("ErrorCanNotCreateDir", $dir);
 					return -1;
 				}
 			}
@@ -275,26 +275,26 @@ 
 			if (file_exists($dir))
 			{
 				//print "srctemplatepath=".$srctemplatepath;	// Src filename
-				$newfile=basename($srctemplatepath);
-				$newfiletmp=preg_replace('/\.od(t|s)/i','',$newfile);
-				$newfiletmp=preg_replace('/template_/i','',$newfiletmp);
-				$newfiletmp=preg_replace('/modele_/i','',$newfiletmp);
+				$newfile = basename($srctemplatepath);
+				$newfiletmp = preg_replace('/\.od(t|s)/i', '', $newfile);
+				$newfiletmp = preg_replace('/template_/i', '', $newfiletmp);
+				$newfiletmp = preg_replace('/modele_/i', '', $newfiletmp);
 
-				$newfiletmp=$objectref.'_'.$newfiletmp;
+				$newfiletmp = $objectref.'_'.$newfiletmp;
 
 				// Get extension (ods or odt)
-				$newfileformat=substr($newfile, strrpos($newfile, '.')+1);
-				if ( ! empty($conf->global->MAIN_DOC_USE_TIMING))
+				$newfileformat = substr($newfile, strrpos($newfile, '.') + 1);
+				if (!empty($conf->global->MAIN_DOC_USE_TIMING))
 				{
-				    $format=$conf->global->MAIN_DOC_USE_TIMING;
-				    if ($format == '1') $format='%Y%m%d%H%M%S';
-					$filename=$newfiletmp.'-'.dol_print_date(dol_now(),$format).'.'.$newfileformat;
+				    $format = $conf->global->MAIN_DOC_USE_TIMING;
+				    if ($format == '1') $format = '%Y%m%d%H%M%S';
+					$filename = $newfiletmp.'-'.dol_print_date(dol_now(), $format).'.'.$newfileformat;
 				}
 				else
 				{
-					$filename=$newfiletmp.'.'.$newfileformat;
+					$filename = $newfiletmp.'.'.$newfileformat;
 				}
-				$file=$dir.'/'.$filename;
+				$file = $dir.'/'.$filename;
 				//print "newdir=".$dir;
 				//print "newfile=".$newfile;
 				//print "file=".$file;
@@ -304,19 +304,19 @@ 
 
 
 				// If CUSTOMER contact defined on user, we use it
-				$usecontact=false;
-				$arrayidcontact=$object->getIdContact('external','CUSTOMER');
+				$usecontact = false;
+				$arrayidcontact = $object->getIdContact('external', 'CUSTOMER');
 				if (count($arrayidcontact) > 0)
 				{
-					$usecontact=true;
-					$result=$object->fetch_contact($arrayidcontact[0]);
+					$usecontact = true;
+					$result = $object->fetch_contact($arrayidcontact[0]);
 				}
 
 				// Recipient name
-				if (! empty($usecontact))
+				if (!empty($usecontact))
 				{
 					// On peut utiliser le nom de la societe du contact
-					if (! empty($conf->global->MAIN_USE_COMPANY_NAME_OF_CONTACT)) $socobject = $object->contact;
+					if (!empty($conf->global->MAIN_USE_COMPANY_NAME_OF_CONTACT)) $socobject = $object->contact;
 					else {
                         			$socobject = $object->thirdparty;
                         			// if we have a CUSTOMER contact and we dont use it as recipient we store the contact object for later use
@@ -325,7 +325,7 @@ 
 				}
 				else
 				{
-					$socobject=$object->thirdparty;
+					$socobject = $object->thirdparty;
 				}
 
 				// Open and load template
@@ -335,38 +335,38 @@ 
 						$srctemplatepath,
 						array(
 							'PATH_TO_TMP'	  => $conf->user->dir_temp,
-							'ZIP_PROXY'		  => 'PclZipProxy',	// PhpZipProxy or PclZipProxy. Got "bad compression method" error when using PhpZipProxy.
+							'ZIP_PROXY'		  => 'PclZipProxy', // PhpZipProxy or PclZipProxy. Got "bad compression method" error when using PhpZipProxy.
 							'DELIMITER_LEFT'  => '{',
 							'DELIMITER_RIGHT' => '}'
 						)
 					);
 				}
-				catch(Exception $e)
+				catch (Exception $e)
 				{
-					$this->error=$e->getMessage();
+					$this->error = $e->getMessage();
 					dol_syslog($e->getMessage(), LOG_WARNING);
 					return -1;
 				}
 
 				// Make substitutions into odt
-				$array_user=$this->get_substitutionarray_user($object,$outputlangs);
-				$array_soc=$this->get_substitutionarray_mysoc($mysoc,$outputlangs);
-				$array_thirdparty=$this->get_substitutionarray_thirdparty($socobject,$outputlangs);
-				$array_other=$this->get_substitutionarray_other($outputlangs);
+				$array_user = $this->get_substitutionarray_user($object, $outputlangs);
+				$array_soc = $this->get_substitutionarray_mysoc($mysoc, $outputlangs);
+				$array_thirdparty = $this->get_substitutionarray_thirdparty($socobject, $outputlangs);
+				$array_other = $this->get_substitutionarray_other($outputlangs);
 				// retrieve contact information for use in object as contact_xxx tags
 				$array_thirdparty_contact = array();
-				if ($usecontact && is_object($contactobject)) $array_thirdparty_contact=$this->get_substitutionarray_contact($contactobject,$outputlangs,'contact');
+				if ($usecontact && is_object($contactobject)) $array_thirdparty_contact = $this->get_substitutionarray_contact($contactobject, $outputlangs, 'contact');
 
-				$tmparray = array_merge($array_user,$array_soc,$array_thirdparty,$array_other,$array_thirdparty_contact);
+				$tmparray = array_merge($array_user, $array_soc, $array_thirdparty, $array_other, $array_thirdparty_contact);
 				complete_substitutions_array($tmparray, $outputlangs, $object);
 				$object->fetch_optionals();
 				// Call the ODTSubstitution hook
-				$parameters=array('file'=>$file,'object'=>$object,'outputlangs'=>$outputlangs,'substitutionarray'=>&$tmparray);
-				$reshook=$hookmanager->executeHooks('ODTSubstitution',$parameters,$this,$action);    // Note that $action and $object may have been modified by some hooks
-				foreach($tmparray as $key=>$value)
+				$parameters = array('file'=>$file, 'object'=>$object, 'outputlangs'=>$outputlangs, 'substitutionarray'=>&$tmparray);
+				$reshook = $hookmanager->executeHooks('ODTSubstitution', $parameters, $this, $action); // Note that $action and $object may have been modified by some hooks
+				foreach ($tmparray as $key=>$value)
 				{
 					try {
-						if (preg_match('/logo$/',$key)) // Image
+						if (preg_match('/logo$/', $key)) // Image
 						{
 							if (file_exists($value)) $odfHandler->setImage($key, $value);
 							else $odfHandler->setVars($key, 'ErrorFileNotFound', true, 'UTF-8');
@@ -376,15 +376,15 @@ 
 							$odfHandler->setVars($key, $value, true, 'UTF-8');
 						}
 					}
-					catch(OdfException $e)
+					catch (OdfException $e)
 					{
 						dol_syslog($e->getMessage(), LOG_WARNING);
 					}
 				}
 
 				// Replace labels translated
-				$tmparray=$outputlangs->get_translations_for_substitutions();
-				foreach($tmparray as $key=>$value)
+				$tmparray = $outputlangs->get_translations_for_substitutions();
+				foreach ($tmparray as $key=>$value)
 				{
 					try {
 						$odfHandler->setVars($key, $value, true, 'UTF-8');
@@ -396,15 +396,15 @@ 
 				}
 
 				// Call the beforeODTSave hook
-				$parameters=array('odfHandler'=>&$odfHandler,'file'=>$file,'object'=>$object,'outputlangs'=>$outputlangs);
-				$reshook=$hookmanager->executeHooks('beforeODTSave',$parameters,$this,$action);    // Note that $action and $object may have been modified by some hooks
+				$parameters = array('odfHandler'=>&$odfHandler, 'file'=>$file, 'object'=>$object, 'outputlangs'=>$outputlangs);
+				$reshook = $hookmanager->executeHooks('beforeODTSave', $parameters, $this, $action); // Note that $action and $object may have been modified by some hooks
 
 				// Write new file
 				if (!empty($conf->global->MAIN_ODT_AS_PDF)) {
 					try {
 						$odfHandler->exportAsAttachedPDF($file);
 					} catch (Exception $e) {
-						$this->error=$e->getMessage();
+						$this->error = $e->getMessage();
 						dol_syslog($e->getMessage(), LOG_WARNING);
 						return -1;
 					}
@@ -413,26 +413,26 @@ 
 					try {
 						$odfHandler->saveToDisk($file);
 					} catch (Exception $e) {
-						$this->error=$e->getMessage();
+						$this->error = $e->getMessage();
 						dol_syslog($e->getMessage(), LOG_WARNING);
 						return -1;
 					}
 				}
 
-				$reshook=$hookmanager->executeHooks('afterODTCreation',$parameters,$this,$action);    // Note that $action and $object may have been modified by some hooks
+				$reshook = $hookmanager->executeHooks('afterODTCreation', $parameters, $this, $action); // Note that $action and $object may have been modified by some hooks
 
-				if (! empty($conf->global->MAIN_UMASK))
+				if (!empty($conf->global->MAIN_UMASK))
 					@chmod($file, octdec($conf->global->MAIN_UMASK));
 
-				$odfHandler=null;	// Destroy object
+				$odfHandler = null; // Destroy object
 
 				$this->result = array('fullpath'=>$file);
 
-				return 1;   // Success
+				return 1; // Success
 			}
 			else
 			{
-				$this->error=$langs->transnoentities("ErrorCanNotCreateDir",$dir);
+				$this->error = $langs->transnoentities("ErrorCanNotCreateDir", $dir);
 				return -1;
 			}
 		}
@@ -449,11 +449,11 @@ 
      * @param string        $array_key      key for array
      * @return array                        array of substitutions
      */
-    function get_substitutionarray_object($object,$outputlangs,$array_key='object')
+    function get_substitutionarray_object($object, $outputlangs, $array_key = 'object')
     {
         // phpcs:enable
 		$array_other = array();
-		foreach($object as $key => $value) {
+		foreach ($object as $key => $value) {
 			if (!is_array($value) && !is_object($value)) {
 				$array_other[$array_key.'_'.$key] = $value;
 			}

Braces +41 added lines, -30 removed lines

@@ -95,7 +95,10 @@ 
 
 		// Recupere emetteur
 		$this->emetteur=$mysoc;
-		if (! $this->emetteur->country_code) $this->emetteur->country_code=substr($langs->defaultlang,-2);    // By default if not defined
+		if (! $this->emetteur->country_code) {
+		    $this->emetteur->country_code=substr($langs->defaultlang,-2);
+		}
+		// By default if not defined
 	}
 
 
@@ -139,11 +142,14 @@ 
 			if (! $tmpdir) {
 				unset($listofdir[$key]); continue;
 			}
-			if (! is_dir($tmpdir)) $texttitle.=img_warning($langs->trans("ErrorDirNotFound",$tmpdir),0);
-			else
+			if (! is_dir($tmpdir)) {
+			    $texttitle.=img_warning($langs->trans("ErrorDirNotFound",$tmpdir),0);
+			} else
 			{
 				$tmpfiles=dol_dir_list($tmpdir,'files',0,'\.(ods|odt)');
-				if (count($tmpfiles)) $listoffiles=array_merge($listoffiles,$tmpfiles);
+				if (count($tmpfiles)) {
+				    $listoffiles=array_merge($listoffiles,$tmpfiles);
+				}
 			}
 		}
 		$texthelp=$langs->trans("ListOfDirectoriesForModelGenODT");
@@ -236,7 +242,9 @@ 
 		$hookmanager->initHooks(array('odtgeneration'));
 		global $action;
 
-		if (! is_object($outputlangs)) $outputlangs=$langs;
+		if (! is_object($outputlangs)) {
+		    $outputlangs=$langs;
+		}
 		$sav_charset_output=$outputlangs->charset_output;
 		$outputlangs->charset_output='UTF-8';
 
@@ -260,7 +268,9 @@ 
 
 			$dir = $conf->user->dir_output;
 			$objectref = dol_sanitizeFileName($object->ref);
-			if (! preg_match('/specimen/i',$objectref)) $dir.= "/" . $objectref;
+			if (! preg_match('/specimen/i',$objectref)) {
+			    $dir.= "/" . $objectref;
+			}
 			$file = $dir . "/" . $objectref . ".odt";
 
 			if (! file_exists($dir))
@@ -287,10 +297,11 @@ 
 				if ( ! empty($conf->global->MAIN_DOC_USE_TIMING))
 				{
 				    $format=$conf->global->MAIN_DOC_USE_TIMING;
-				    if ($format == '1') $format='%Y%m%d%H%M%S';
+				    if ($format == '1') {
+				        $format='%Y%m%d%H%M%S';
+				    }
 					$filename=$newfiletmp.'-'.dol_print_date(dol_now(),$format).'.'.$newfileformat;
-				}
-				else
+				} else
 				{
 					$filename=$newfiletmp.'.'.$newfileformat;
 				}
@@ -316,14 +327,14 @@ 
 				if (! empty($usecontact))
 				{
 					// On peut utiliser le nom de la societe du contact
-					if (! empty($conf->global->MAIN_USE_COMPANY_NAME_OF_CONTACT)) $socobject = $object->contact;
-					else {
+					if (! empty($conf->global->MAIN_USE_COMPANY_NAME_OF_CONTACT)) {
+					    $socobject = $object->contact;
+					} else {
                         			$socobject = $object->thirdparty;
                         			// if we have a CUSTOMER contact and we dont use it as recipient we store the contact object for later use
                         			$contactobject = $object->contact;
                     			}
-				}
-				else
+				} else
 				{
 					$socobject=$object->thirdparty;
 				}
@@ -340,8 +351,7 @@ 
 							'DELIMITER_RIGHT' => '}'
 						)
 					);
-				}
-				catch(Exception $e)
+				} catch(Exception $e)
 				{
 					$this->error=$e->getMessage();
 					dol_syslog($e->getMessage(), LOG_WARNING);
@@ -355,7 +365,9 @@ 
 				$array_other=$this->get_substitutionarray_other($outputlangs);
 				// retrieve contact information for use in object as contact_xxx tags
 				$array_thirdparty_contact = array();
-				if ($usecontact && is_object($contactobject)) $array_thirdparty_contact=$this->get_substitutionarray_contact($contactobject,$outputlangs,'contact');
+				if ($usecontact && is_object($contactobject)) {
+				    $array_thirdparty_contact=$this->get_substitutionarray_contact($contactobject,$outputlangs,'contact');
+				}
 
 				$tmparray = array_merge($array_user,$array_soc,$array_thirdparty,$array_other,$array_thirdparty_contact);
 				complete_substitutions_array($tmparray, $outputlangs, $object);
@@ -366,17 +378,18 @@ 
 				foreach($tmparray as $key=>$value)
 				{
 					try {
-						if (preg_match('/logo$/',$key)) // Image
+						if (preg_match('/logo$/',$key)) {
+						    // Image
 						{
 							if (file_exists($value)) $odfHandler->setImage($key, $value);
-							else $odfHandler->setVars($key, 'ErrorFileNotFound', true, 'UTF-8');
-						}
-						else    // Text
+						} else {
+							    $odfHandler->setVars($key, 'ErrorFileNotFound', true, 'UTF-8');
+							}
+						} else    // Text
 						{
 							$odfHandler->setVars($key, $value, true, 'UTF-8');
 						}
-					}
-					catch(OdfException $e)
+					} catch(OdfException $e)
 					{
 						dol_syslog($e->getMessage(), LOG_WARNING);
 					}
@@ -388,8 +401,7 @@ 
 				{
 					try {
 						$odfHandler->setVars($key, $value, true, 'UTF-8');
-					}
-					catch (OdfException $e)
+					} catch (OdfException $e)
 					{
 						dol_syslog($e->getMessage(), LOG_WARNING);
 					}
@@ -408,8 +420,7 @@ 
 						dol_syslog($e->getMessage(), LOG_WARNING);
 						return -1;
 					}
-				}
-				else {
+				} else {
 					try {
 						$odfHandler->saveToDisk($file);
 					} catch (Exception $e) {
@@ -421,16 +432,16 @@ 
 
 				$reshook=$hookmanager->executeHooks('afterODTCreation',$parameters,$this,$action);    // Note that $action and $object may have been modified by some hooks
 
-				if (! empty($conf->global->MAIN_UMASK))
-					@chmod($file, octdec($conf->global->MAIN_UMASK));
+				if (! empty($conf->global->MAIN_UMASK)) {
+									@chmod($file, octdec($conf->global->MAIN_UMASK));
+				}
 
 				$odfHandler=null;	// Destroy object
 
 				$this->result = array('fullpath'=>$file);
 
 				return 1;   // Success
-			}
-			else
+			} else
 			{
 				$this->error=$langs->transnoentities("ErrorCanNotCreateDir",$dir);
 				return -1;

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

Indentation +18 added lines, -18 removed lines

@@ -31,37 +31,37 @@
  *  \brief      File with parent class for generating contracts to PDF and File of class to manage contract numbering
  */
 
- require_once DOL_DOCUMENT_ROOT.'/core/class/commondocgenerator.class.php';
+    require_once DOL_DOCUMENT_ROOT.'/core/class/commondocgenerator.class.php';
 
 /**
  *	Parent class to manage intervention document templates
  */
 abstract class ModelePDFUser extends CommonDocGenerator
 {
-	/**
-	 * @var string Error code (or message)
-	 */
-	public $error='';
+    /**
+     * @var string Error code (or message)
+     */
+    public $error='';
 
 
     // phpcs:disable PEAR.NamingConventions.ValidFunctionName.NotCamelCaps
-	/**
-	 *	Return list of active generation modules
-	 *
+    /**
+     *	Return list of active generation modules
+     *
      *  @param	DoliDB	$db     			Database handler
      *  @param  integer	$maxfilenamelength  Max length of value to show
      *  @return	array						List of templates
-	 */
-	static function liste_modeles($db,$maxfilenamelength=0)
-	{
+     */
+    static function liste_modeles($db,$maxfilenamelength=0)
+    {
         // phpcs:enable
-		global $conf;
+        global $conf;
 
-		$type='user';
-		$liste=array();
+        $type='user';
+        $liste=array();
 
-		include_once DOL_DOCUMENT_ROOT.'/core/lib/functions2.lib.php';
-		$liste=getListOfModels($db,$type,$maxfilenamelength);
-		return $liste;
-	}
+        include_once DOL_DOCUMENT_ROOT.'/core/lib/functions2.lib.php';
+        $liste=getListOfModels($db,$type,$maxfilenamelength);
+        return $liste;
+    }
 }

Spacing +5 added lines, -5 removed lines

@@ -41,7 +41,7 @@ 
 	/**
 	 * @var string Error code (or message)
 	 */
-	public $error='';
+	public $error = '';
 
 
     // phpcs:disable PEAR.NamingConventions.ValidFunctionName.NotCamelCaps
@@ -52,16 +52,16 @@ 
      *  @param  integer	$maxfilenamelength  Max length of value to show
      *  @return	array						List of templates
 	 */
-	static function liste_modeles($db,$maxfilenamelength=0)
+	static function liste_modeles($db, $maxfilenamelength = 0)
 	{
         // phpcs:enable
 		global $conf;
 
-		$type='user';
-		$liste=array();
+		$type = 'user';
+		$liste = array();
 
 		include_once DOL_DOCUMENT_ROOT.'/core/lib/functions2.lib.php';
-		$liste=getListOfModels($db,$type,$maxfilenamelength);
+		$liste = getListOfModels($db, $type, $maxfilenamelength);
 		return $liste;
 	}
 }

dolibarr/htdocs/core/modules/modBlockedLog.class.php

Indentation +92 added lines, -92 removed lines

@@ -30,21 +30,21 @@ 
 class modBlockedLog extends DolibarrModules
 {
     /**
-	 *   Constructor. Define names, constants, directories, boxes, permissions
-	 *
-	 *   @param      DoliDB		$db      Database handler
+     *   Constructor. Define names, constants, directories, boxes, permissions
+     *
+     *   @param      DoliDB		$db      Database handler
      */
     function __construct($db)
     {
-    	global $langs, $conf, $mysoc;
+        global $langs, $conf, $mysoc;
 
         $this->db = $db;
         $this->numero = 3200;
         // Key text used to identify module (for permissions, menus, etc...)
         $this->rights_class = 'blockedlog';
 
-		// Family can be 'crm','financial','hr','projects','products','ecm','technic','other'
-		// It is used to group modules in module setup page
+        // Family can be 'crm','financial','hr','projects','products','ecm','technic','other'
+        // It is used to group modules in module setup page
         $this->family = "base";
         // Module position in the family on 2 digits ('01', '10', '20', ...)
         $this->module_position = '75';
@@ -67,10 +67,10 @@ 
 
         // Dependancies
         //-------------
-	    $this->hidden = false;	// A condition to disable module
-	    $this->depends = array('always'=>'modFacture');	   // List of modules id that must be enabled if this module is enabled
+        $this->hidden = false;	// A condition to disable module
+        $this->depends = array('always'=>'modFacture');	   // List of modules id that must be enabled if this module is enabled
         $this->requiredby = array();	                   // List of modules id to disable if this one is disabled
-	    $this->conflictwith = array();	                   // List of modules id this module is in conflict with
+        $this->conflictwith = array();	                   // List of modules id this module is in conflict with
         $this->langfiles = array('blockedlog');
 
         $this->warnings_activation = array();                     // Warning to show when we activate module. array('always'='text') or array('FR'='textfr','ES'='textes'...)
@@ -91,14 +91,14 @@ 
         //var_dump($this->automatic_activation);
 
         $this->always_enabled = (!empty($conf->blockedlog->enabled)
-        	&& !empty($conf->global->BLOCKEDLOG_DISABLE_NOT_ALLOWED_FOR_COUNTRY)
-        	&& in_array($mysoc->country_code, explode(',', $conf->global->BLOCKEDLOG_DISABLE_NOT_ALLOWED_FOR_COUNTRY))
-        	&& $this->alreadyUsed(1));
+            && !empty($conf->global->BLOCKEDLOG_DISABLE_NOT_ALLOWED_FOR_COUNTRY)
+            && in_array($mysoc->country_code, explode(',', $conf->global->BLOCKEDLOG_DISABLE_NOT_ALLOWED_FOR_COUNTRY))
+            && $this->alreadyUsed(1));
 
         // Constants
         //-----------
         $this->const = array(
-        	1=>array('BLOCKEDLOG_DISABLE_NOT_ALLOWED_FOR_COUNTRY', 'chaine', 'FR', 'This is list of country code where the module may be mandatory', 0, 'current', 0)
+            1=>array('BLOCKEDLOG_DISABLE_NOT_ALLOWED_FOR_COUNTRY', 'chaine', 'FR', 'This is list of country code where the module may be mandatory', 0, 'current', 0)
         );
 
         // New pages on tabs
@@ -148,9 +148,9 @@ 
     function alreadyUsed()
     {
 
-    	require_once DOL_DOCUMENT_ROOT.'/blockedlog/class/blockedlog.class.php';
-    	$b=new BlockedLog($this->db);
-    	return $b->alreadyUsed(1);
+        require_once DOL_DOCUMENT_ROOT.'/blockedlog/class/blockedlog.class.php';
+        $b=new BlockedLog($this->db);
+        return $b->alreadyUsed(1);
     }
 
 
@@ -164,87 +164,87 @@ 
      */
     function init($options='')
     {
-    	global $conf, $user;
-
-    	$sql = array();
-
-    	// If already used, we add an entry to show we enable module
-   		require_once DOL_DOCUMENT_ROOT.'/blockedlog/class/blockedlog.class.php';
-
-   		$object=new stdClass();
-    	$object->id = 1;
-    	$object->element = 'module';
-    	$object->ref = 'systemevent';
-    	$object->entity = $conf->entity;
-    	$object->date = dol_now();
-
-    	$b=new BlockedLog($this->db);
-    	$result = $b->setObjectData($object, 'MODULE_SET', 0);
-    	if ($result < 0)
-    	{
-    		$this->error = $b->error;
-    		$this->errors = $b->erros;
-    		return 0;
-    	}
-
-    	$res = $b->create($user);
-    	if ($res<=0) {
-    		$this->error = $b->error;
-    		$this->errors = $b->errors;
-    		return $res;
-    	}
-
-    	return $this->_init($sql, $options);
+        global $conf, $user;
+
+        $sql = array();
+
+        // If already used, we add an entry to show we enable module
+            require_once DOL_DOCUMENT_ROOT.'/blockedlog/class/blockedlog.class.php';
+
+            $object=new stdClass();
+        $object->id = 1;
+        $object->element = 'module';
+        $object->ref = 'systemevent';
+        $object->entity = $conf->entity;
+        $object->date = dol_now();
+
+        $b=new BlockedLog($this->db);
+        $result = $b->setObjectData($object, 'MODULE_SET', 0);
+        if ($result < 0)
+        {
+            $this->error = $b->error;
+            $this->errors = $b->erros;
+            return 0;
+        }
+
+        $res = $b->create($user);
+        if ($res<=0) {
+            $this->error = $b->error;
+            $this->errors = $b->errors;
+            return $res;
+        }
+
+        return $this->_init($sql, $options);
     }
 
     /**
-	 * Function called when module is disabled.
-	 * The remove function removes tabs, constants, boxes, permissions and menus from Dolibarr database.
-	 * Data directories are not deleted
-	 *
-	 * @param      string	$options    Options when enabling module ('', 'noboxes')
-	 * @return     int             		1 if OK, 0 if KO
-	 */
+     * Function called when module is disabled.
+     * The remove function removes tabs, constants, boxes, permissions and menus from Dolibarr database.
+     * Data directories are not deleted
+     *
+     * @param      string	$options    Options when enabling module ('', 'noboxes')
+     * @return     int             		1 if OK, 0 if KO
+     */
     function remove($options = '')
     {
 
-    	global $conf, $user;
-
-    	$sql = array();
-
-		// If already used, we add an entry to show we enable module
-		require_once DOL_DOCUMENT_ROOT.'/blockedlog/class/blockedlog.class.php';
-
-		$object=new stdClass();
-    	$object->id = 1;
-    	$object->element = 'module';
-    	$object->ref = 'systemevent';
-    	$object->entity = $conf->entity;
-    	$object->date = dol_now();
-
-    	$b=new BlockedLog($this->db);
-    	$result = $b->setObjectData($object, 'MODULE_RESET', 0);
-    	if ($result < 0)
-    	{
-    		$this->error = $b->error;
-    		$this->errors = $b->erros;
-    		return 0;
-    	}
-
-    	if ($b->alreadyUsed(1))
-    	{
-    		$res = $b->create($user, '0000000000');		// If already used for something else than SET or UNSET, we log with error
-    	}
-    	else
-    	{
-    		$res = $b->create($user);
-    	}
-	    if ($res<=0) {
-	    	$this->error = $b->error;
-	    	$this->errors = $b->errors;
-	    	return $res;
-    	}
-
-    	return $this->_remove($sql, $options);
+        global $conf, $user;
+
+        $sql = array();
+
+        // If already used, we add an entry to show we enable module
+        require_once DOL_DOCUMENT_ROOT.'/blockedlog/class/blockedlog.class.php';
+
+        $object=new stdClass();
+        $object->id = 1;
+        $object->element = 'module';
+        $object->ref = 'systemevent';
+        $object->entity = $conf->entity;
+        $object->date = dol_now();
+
+        $b=new BlockedLog($this->db);
+        $result = $b->setObjectData($object, 'MODULE_RESET', 0);
+        if ($result < 0)
+        {
+            $this->error = $b->error;
+            $this->errors = $b->erros;
+            return 0;
+        }
+
+        if ($b->alreadyUsed(1))
+        {
+            $res = $b->create($user, '0000000000');		// If already used for something else than SET or UNSET, we log with error
+        }
+        else
+        {
+            $res = $b->create($user);
+        }
+        if ($res<=0) {
+            $this->error = $b->error;
+            $this->errors = $b->errors;
+            return $res;
+        }
+
+        return $this->_remove($sql, $options);
     }
 }

Spacing +32 added lines, -32 removed lines

@@ -22,7 +22,7 @@ 
  *  \ingroup    blockedlog
  *  \brief      Description and activation file for module BlockedLog
  */
-include_once DOL_DOCUMENT_ROOT .'/core/modules/DolibarrModules.class.php';
+include_once DOL_DOCUMENT_ROOT.'/core/modules/DolibarrModules.class.php';
 
 /**
  *	Class to describe a BlockedLog module
@@ -49,14 +49,14 @@ 
         // Module position in the family on 2 digits ('01', '10', '20', ...)
         $this->module_position = '75';
         // Module label (no space allowed), used if translation string 'ModuleXXXName' not found (where XXX is value of numeric property 'numero' of module)
-        $this->name = preg_replace('/^mod/i','',get_class($this));
+        $this->name = preg_replace('/^mod/i', '', get_class($this));
         $this->description = "Enable a log on some business events into a non reversible log. This module may be mandatory for some countries.";
         // Possible values for version are: 'development', 'experimental', 'dolibarr' or version
         $this->version = 'dolibarr';
         // Key used in llx_const table to save module status enabled/disabled (where MYMODULE is value of property name of module in uppercase)
         $this->const_name = 'MAIN_MODULE_'.strtoupper($this->name);
         // Name of image file used for this module.
-        $this->picto='technic';
+        $this->picto = 'technic';
 
         // Data directories to create when module is enabled
         $this->dirs = array();
@@ -67,14 +67,14 @@ 
 
         // Dependancies
         //-------------
-	    $this->hidden = false;	// A condition to disable module
-	    $this->depends = array('always'=>'modFacture');	   // List of modules id that must be enabled if this module is enabled
-        $this->requiredby = array();	                   // List of modules id to disable if this one is disabled
-	    $this->conflictwith = array();	                   // List of modules id this module is in conflict with
+	    $this->hidden = false; // A condition to disable module
+	    $this->depends = array('always'=>'modFacture'); // List of modules id that must be enabled if this module is enabled
+        $this->requiredby = array(); // List of modules id to disable if this one is disabled
+	    $this->conflictwith = array(); // List of modules id this module is in conflict with
         $this->langfiles = array('blockedlog');
 
-        $this->warnings_activation = array();                     // Warning to show when we activate module. array('always'='text') or array('FR'='textfr','ES'='textes'...)
-        $this->warnings_activation_ext = array();                 // Warning to show when we activate an external module. array('always'='text') or array('FR'='textfr','ES'='textes'...)
+        $this->warnings_activation = array(); // Warning to show when we activate module. array('always'='text') or array('FR'='textfr','ES'='textes'...)
+        $this->warnings_activation_ext = array(); // Warning to show when we activate an external module. array('always'='text') or array('FR'='textfr','ES'='textes'...)
         $this->warnings_unactivation = array('FR'=>'BlockedLogAreRequiredByYourCountryLegislation');
 
         // Currently, activation is not automatic because only companies (in France) making invoices to non business customers must
@@ -111,31 +111,31 @@ 
 
         // Permissions
         // -----------------
-        $this->rights = array();		// Permission array used by this module
+        $this->rights = array(); // Permission array used by this module
 
-        $r=0;
-        $this->rights[$r][0] = $this->numero + $r;	// Permission id (must not be already used)
-        $this->rights[$r][1] = 'Read archived events and fingerprints';	// Permission label
-        $this->rights[$r][3] = 0; 					// Permission by default for new user (0/1)
-        $this->rights[$r][4] = 'read';				// In php code, permission will be checked by test if ($user->rights->mymodule->level1->level2)
+        $r = 0;
+        $this->rights[$r][0] = $this->numero + $r; // Permission id (must not be already used)
+        $this->rights[$r][1] = 'Read archived events and fingerprints'; // Permission label
+        $this->rights[$r][3] = 0; // Permission by default for new user (0/1)
+        $this->rights[$r][4] = 'read'; // In php code, permission will be checked by test if ($user->rights->mymodule->level1->level2)
         $this->rights[$r][5] = '';
 
         // Main menu entries
         // -----------------
-        $r=0;
-        $this->menu[$r]=array(
-        'fk_menu'=>'fk_mainmenu=tools',		    // Use 'fk_mainmenu=xxx' or 'fk_mainmenu=xxx,fk_leftmenu=yyy' where xxx is mainmenucode and yyy is a leftmenucode
+        $r = 0;
+        $this->menu[$r] = array(
+        'fk_menu'=>'fk_mainmenu=tools', // Use 'fk_mainmenu=xxx' or 'fk_mainmenu=xxx,fk_leftmenu=yyy' where xxx is mainmenucode and yyy is a leftmenucode
         'mainmenu'=>'tools',
         'leftmenu'=>'blockedlogbrowser',
-        'type'=>'left',			                // This is a Left menu entry
+        'type'=>'left', // This is a Left menu entry
         'titre'=>'BrowseBlockedLog',
         'url'=>'/blockedlog/admin/blockedlog_list.php?mainmenu=tools&leftmenu=blockedlogbrowser',
-        'langs'=>'blockedlog',	        // Lang file to use (without .lang) by module. File must be in langs/code_CODE/ directory.
+        'langs'=>'blockedlog', // Lang file to use (without .lang) by module. File must be in langs/code_CODE/ directory.
         'position'=>200,
-        'enabled'=>'$conf->blockedlog->enabled',  // Define condition to show or hide menu entry. Use '$conf->mymodule->enabled' if entry must be visible if module is enabled. Use '$leftmenu==\'system\'' to show if leftmenu system is selected.
-        'perms'=>'$user->rights->blockedlog->read',			    // Use 'perms'=>'$user->rights->mymodule->level1->level2' if you want your menu with a permission rules
+        'enabled'=>'$conf->blockedlog->enabled', // Define condition to show or hide menu entry. Use '$conf->mymodule->enabled' if entry must be visible if module is enabled. Use '$leftmenu==\'system\'' to show if leftmenu system is selected.
+        'perms'=>'$user->rights->blockedlog->read', // Use 'perms'=>'$user->rights->mymodule->level1->level2' if you want your menu with a permission rules
         'target'=>'',
-        'user'=>2);				                // 0=Menu for internal users, 1=external users, 2=both
+        'user'=>2); // 0=Menu for internal users, 1=external users, 2=both
         $r++;
     }
 
@@ -149,7 +149,7 @@ 
     {
 
     	require_once DOL_DOCUMENT_ROOT.'/blockedlog/class/blockedlog.class.php';
-    	$b=new BlockedLog($this->db);
+    	$b = new BlockedLog($this->db);
     	return $b->alreadyUsed(1);
     }
 
@@ -162,7 +162,7 @@ 
      *      @param      string	$options    Options when enabling module ('', 'noboxes')
      *      @return     int             	1 if OK, 0 if KO
      */
-    function init($options='')
+    function init($options = '')
     {
     	global $conf, $user;
 
@@ -171,14 +171,14 @@ 
     	// If already used, we add an entry to show we enable module
    		require_once DOL_DOCUMENT_ROOT.'/blockedlog/class/blockedlog.class.php';
 
-   		$object=new stdClass();
+   		$object = new stdClass();
     	$object->id = 1;
     	$object->element = 'module';
     	$object->ref = 'systemevent';
     	$object->entity = $conf->entity;
     	$object->date = dol_now();
 
-    	$b=new BlockedLog($this->db);
+    	$b = new BlockedLog($this->db);
     	$result = $b->setObjectData($object, 'MODULE_SET', 0);
     	if ($result < 0)
     	{
@@ -188,7 +188,7 @@ 
     	}
 
     	$res = $b->create($user);
-    	if ($res<=0) {
+    	if ($res <= 0) {
     		$this->error = $b->error;
     		$this->errors = $b->errors;
     		return $res;
@@ -215,14 +215,14 @@ 
 		// If already used, we add an entry to show we enable module
 		require_once DOL_DOCUMENT_ROOT.'/blockedlog/class/blockedlog.class.php';
 
-		$object=new stdClass();
+		$object = new stdClass();
     	$object->id = 1;
     	$object->element = 'module';
     	$object->ref = 'systemevent';
     	$object->entity = $conf->entity;
     	$object->date = dol_now();
 
-    	$b=new BlockedLog($this->db);
+    	$b = new BlockedLog($this->db);
     	$result = $b->setObjectData($object, 'MODULE_RESET', 0);
     	if ($result < 0)
     	{
@@ -233,13 +233,13 @@ 
 
     	if ($b->alreadyUsed(1))
     	{
-    		$res = $b->create($user, '0000000000');		// If already used for something else than SET or UNSET, we log with error
+    		$res = $b->create($user, '0000000000'); // If already used for something else than SET or UNSET, we log with error
     	}
     	else
     	{
     		$res = $b->create($user);
     	}
-	    if ($res<=0) {
+	    if ($res <= 0) {
 	    	$this->error = $b->error;
 	    	$this->errors = $b->errors;
 	    	return $res;

Braces +1 added lines, -2 removed lines

@@ -234,8 +234,7 @@
     	if ($b->alreadyUsed(1))
     	{
     		$res = $b->create($user, '0000000000');		// If already used for something else than SET or UNSET, we log with error
-    	}
-    	else
+    	} else
     	{
     		$res = $b->create($user);
     	}

dolibarr/htdocs/core/modules/mailings/fraise.modules.php

Indentation +17 added lines, -17 removed lines

@@ -34,7 +34,7 @@ 
 class mailing_fraise extends MailingTargets
 {
     var $name='FundationMembers';                    // Identifiant du module mailing
-	// This label is used if no translation is found for key XXX neither MailingModuleDescXXX where XXX=name is found
+    // This label is used if no translation is found for key XXX neither MailingModuleDescXXX where XXX=name is found
     var $desc='Foundation members with emails';
     // Set to 1 if selector is available for admin users only
     var $require_admin=0;
@@ -175,24 +175,24 @@ 
         $resql = $this->db->query($sql);
         if ($resql)
         {
-        	$num = $this->db->num_rows($resql);
+            $num = $this->db->num_rows($resql);
 
-        	$s.='<option value="0">&nbsp;</option>';
-        	if (! $num) $s.='<option value="0" disabled>'.$langs->trans("NoCategoriesDefined").'</option>';
+            $s.='<option value="0">&nbsp;</option>';
+            if (! $num) $s.='<option value="0" disabled>'.$langs->trans("NoCategoriesDefined").'</option>';
 
-        	$i = 0;
-        	while ($i < $num)
-        	{
-        		$obj = $this->db->fetch_object($resql);
+            $i = 0;
+            while ($i < $num)
+            {
+                $obj = $this->db->fetch_object($resql);
 
-        		$s.='<option value="'.$obj->rowid.'">'.dol_trunc($obj->label,38,'middle');
-        		$s.='</option>';
-        		$i++;
-        	}
+                $s.='<option value="'.$obj->rowid.'">'.dol_trunc($obj->label,38,'middle');
+                $s.='</option>';
+                $i++;
+            }
         }
         else
         {
-        	dol_print_error($this->db);
+            dol_print_error($this->db);
         }
 
         $s.='</select>';
@@ -230,9 +230,9 @@ 
     function add_to_target($mailing_id)
     {
         // phpcs:enable
-    	global $langs,$_POST;
+        global $langs,$_POST;
 
-    	// Load translation files required by the page
+        // Load translation files required by the page
         $langs->loadLangs(array("members","companies"));
 
         $cibles = array();
@@ -248,8 +248,8 @@ 
         $sql.= " FROM ".MAIN_DB_PREFIX."adherent as a";
         if ($_POST['filter_category'])
         {
-        	$sql.= " LEFT JOIN ".MAIN_DB_PREFIX."categorie_member as cm ON cm.fk_member = a.rowid";
-        	$sql.= " LEFT JOIN ".MAIN_DB_PREFIX."categorie as c ON c.rowid = cm.fk_categorie";
+            $sql.= " LEFT JOIN ".MAIN_DB_PREFIX."categorie_member as cm ON cm.fk_member = a.rowid";
+            $sql.= " LEFT JOIN ".MAIN_DB_PREFIX."categorie as c ON c.rowid = cm.fk_categorie";
         }
         $sql.= " , ".MAIN_DB_PREFIX."adherent_type as ta";
         $sql.= " WHERE a.entity IN (".getEntity('member').") AND a.email <> ''";     // Note that null != '' is false

Spacing +75 added lines, -75 removed lines

@@ -33,14 +33,14 @@ 
  */
 class mailing_fraise extends MailingTargets
 {
-    var $name='FundationMembers';                    // Identifiant du module mailing
+    var $name = 'FundationMembers'; // Identifiant du module mailing
 	// This label is used if no translation is found for key XXX neither MailingModuleDescXXX where XXX=name is found
-    var $desc='Foundation members with emails';
+    var $desc = 'Foundation members with emails';
     // Set to 1 if selector is available for admin users only
-    var $require_admin=0;
+    var $require_admin = 0;
 
-    var $require_module=array('adherent');
-    var $picto='user';
+    var $require_module = array('adherent');
+    var $picto = 'user';
 
     /**
      * @var DoliDB Database handler.
@@ -73,10 +73,10 @@ 
         $langs->load("members");
 
         // Array for requests for statistics board
-        $statssql=array();
+        $statssql = array();
 
-        $statssql[0] ="SELECT '".$this->db->escape($langs->trans("FundationMembers"))."' as label, count(*) as nb";
-        $statssql[0].=" FROM ".MAIN_DB_PREFIX."adherent where statut = 1 and entity IN (".getEntity('member').")";
+        $statssql[0] = "SELECT '".$this->db->escape($langs->trans("FundationMembers"))."' as label, count(*) as nb";
+        $statssql[0] .= " FROM ".MAIN_DB_PREFIX."adherent where statut = 1 and entity IN (".getEntity('member').")";
 
         return $statssql;
     }
@@ -90,7 +90,7 @@ 
      *  @param    string    $sql        Requete sql de comptage
      *    @return        int            Nb of recipients
      */
-    function getNbOfRecipients($sql='')
+    function getNbOfRecipients($sql = '')
     {
         $sql  = "SELECT count(distinct(a.email)) as nb";
         $sql .= " FROM ".MAIN_DB_PREFIX."adherent as a";
@@ -112,42 +112,42 @@ 
         global $conf, $langs;
 
         // Load translation files required by the page
-        $langs->loadLangs(array("members","companies","categories"));
+        $langs->loadLangs(array("members", "companies", "categories"));
 
-        $form=new Form($this->db);
+        $form = new Form($this->db);
 
-        $s='';
+        $s = '';
 
         // Status
-        $s.=$langs->trans("Status").': ';
-        $s.='<select name="filter" class="flat">';
-        $s.='<option value="none">&nbsp;</option>';
-        $s.='<option value="-1">'.$langs->trans("MemberStatusDraft").'</option>';
-        $s.='<option value="1a">'.$langs->trans("MemberStatusActiveShort").' ('.$langs->trans("MemberStatusPaidShort").')</option>';
-        $s.='<option value="1b">'.$langs->trans("MemberStatusActiveShort").' ('.$langs->trans("MemberStatusActiveLateShort").')</option>';
-        $s.='<option value="0">'.$langs->trans("MemberStatusResiliatedShort").'</option>';
-        $s.='</select> ';
-        $s.=$langs->trans("Type").': ';
-        $s.='<select name="filter_type" class="flat">';
+        $s .= $langs->trans("Status").': ';
+        $s .= '<select name="filter" class="flat">';
+        $s .= '<option value="none">&nbsp;</option>';
+        $s .= '<option value="-1">'.$langs->trans("MemberStatusDraft").'</option>';
+        $s .= '<option value="1a">'.$langs->trans("MemberStatusActiveShort").' ('.$langs->trans("MemberStatusPaidShort").')</option>';
+        $s .= '<option value="1b">'.$langs->trans("MemberStatusActiveShort").' ('.$langs->trans("MemberStatusActiveLateShort").')</option>';
+        $s .= '<option value="0">'.$langs->trans("MemberStatusResiliatedShort").'</option>';
+        $s .= '</select> ';
+        $s .= $langs->trans("Type").': ';
+        $s .= '<select name="filter_type" class="flat">';
         $sql = "SELECT rowid, libelle, statut";
-        $sql.= " FROM ".MAIN_DB_PREFIX."adherent_type";
-        $sql.= " WHERE entity IN (".getEntity('member_type').")";
-        $sql.= " ORDER BY rowid";
+        $sql .= " FROM ".MAIN_DB_PREFIX."adherent_type";
+        $sql .= " WHERE entity IN (".getEntity('member_type').")";
+        $sql .= " ORDER BY rowid";
         $resql = $this->db->query($sql);
         if ($resql)
         {
             $num = $this->db->num_rows($resql);
 
-            $s.='<option value="0">&nbsp;</option>';
-            if (! $num) $s.='<option value="0" disabled="disabled">'.$langs->trans("NoCategoriesDefined").'</option>';
+            $s .= '<option value="0">&nbsp;</option>';
+            if (!$num) $s .= '<option value="0" disabled="disabled">'.$langs->trans("NoCategoriesDefined").'</option>';
 
             $i = 0;
             while ($i < $num)
             {
                 $obj = $this->db->fetch_object($resql);
 
-                $s.='<option value="'.$obj->rowid.'">'.dol_trunc($obj->libelle,38,'middle');
-                $s.='</option>';
+                $s .= '<option value="'.$obj->rowid.'">'.dol_trunc($obj->libelle, 38, 'middle');
+                $s .= '</option>';
                 $i++;
             }
         }
@@ -156,20 +156,20 @@ 
             dol_print_error($this->db);
         }
 
-        $s.='</select>';
+        $s .= '</select>';
 
-        $s.=' ';
+        $s .= ' ';
 
-        $s.=$langs->trans("Category").': ';
-        $s.='<select name="filter_category" class="flat">';
+        $s .= $langs->trans("Category").': ';
+        $s .= '<select name="filter_category" class="flat">';
 
         // Show categories
         $sql = "SELECT rowid, label, type, visible";
-        $sql.= " FROM ".MAIN_DB_PREFIX."categorie";
-        $sql.= " WHERE type = 3";	// We keep only categories for members
+        $sql .= " FROM ".MAIN_DB_PREFIX."categorie";
+        $sql .= " WHERE type = 3"; // We keep only categories for members
         // $sql.= " AND visible > 0";	// We ignore the property visible because member's categories does not use this property (only products categories use it).
-        $sql.= " AND entity = ".$conf->entity;
-        $sql.= " ORDER BY label";
+        $sql .= " AND entity = ".$conf->entity;
+        $sql .= " ORDER BY label";
 
         //print $sql;
         $resql = $this->db->query($sql);
@@ -177,16 +177,16 @@ 
         {
         	$num = $this->db->num_rows($resql);
 
-        	$s.='<option value="0">&nbsp;</option>';
-        	if (! $num) $s.='<option value="0" disabled>'.$langs->trans("NoCategoriesDefined").'</option>';
+        	$s .= '<option value="0">&nbsp;</option>';
+        	if (!$num) $s .= '<option value="0" disabled>'.$langs->trans("NoCategoriesDefined").'</option>';
 
         	$i = 0;
         	while ($i < $num)
         	{
         		$obj = $this->db->fetch_object($resql);
 
-        		$s.='<option value="'.$obj->rowid.'">'.dol_trunc($obj->label,38,'middle');
-        		$s.='</option>';
+        		$s .= '<option value="'.$obj->rowid.'">'.dol_trunc($obj->label, 38, 'middle');
+        		$s .= '</option>';
         		$i++;
         	}
         }
@@ -195,14 +195,14 @@ 
         	dol_print_error($this->db);
         }
 
-        $s.='</select>';
+        $s .= '</select>';
 
 
-        $s.='<br>';
-        $s.=$langs->trans("DateEndSubscription").': &nbsp;';
-        $s.=$langs->trans("After").' > '.$form->selectDate(-1,'subscriptionafter',0,0,1,'fraise',1,0,0);
-        $s.=' &nbsp; ';
-        $s.=$langs->trans("Before").' < '.$form->selectDate(-1,'subscriptionbefore',0,0,1,'fraise',1,0,0);
+        $s .= '<br>';
+        $s .= $langs->trans("DateEndSubscription").': &nbsp;';
+        $s .= $langs->trans("After").' > '.$form->selectDate(-1, 'subscriptionafter', 0, 0, 1, 'fraise', 1, 0, 0);
+        $s .= ' &nbsp; ';
+        $s .= $langs->trans("Before").' < '.$form->selectDate(-1, 'subscriptionbefore', 0, 0, 1, 'fraise', 1, 0, 0);
 
         return $s;
     }
@@ -216,7 +216,7 @@ 
      */
     function url($id)
     {
-        return '<a href="'.DOL_URL_ROOT.'/adherents/card.php?rowid='.$id.'">'.img_object('',"user").'</a>';
+        return '<a href="'.DOL_URL_ROOT.'/adherents/card.php?rowid='.$id.'">'.img_object('', "user").'</a>';
     }
 
 
@@ -230,49 +230,49 @@ 
     function add_to_target($mailing_id)
     {
         // phpcs:enable
-    	global $langs,$_POST;
+    	global $langs, $_POST;
 
     	// Load translation files required by the page
-        $langs->loadLangs(array("members","companies"));
+        $langs->loadLangs(array("members", "companies"));
 
         $cibles = array();
-        $now=dol_now();
+        $now = dol_now();
 
-        $dateendsubscriptionafter=dol_mktime($_POST['subscriptionafterhour'],$_POST['subscriptionaftermin'],$_POST['subscriptionaftersec'],$_POST['subscriptionaftermonth'],$_POST['subscriptionafterday'],$_POST['subscriptionafteryear']);
-        $dateendsubscriptionbefore=dol_mktime($_POST['subscriptionbeforehour'],$_POST['subscriptionbeforemin'],$_POST['subscriptionbeforesec'],$_POST['subscriptionbeforemonth'],$_POST['subscriptionbeforeday'],$_POST['subscriptionbeforeyear']);
+        $dateendsubscriptionafter = dol_mktime($_POST['subscriptionafterhour'], $_POST['subscriptionaftermin'], $_POST['subscriptionaftersec'], $_POST['subscriptionaftermonth'], $_POST['subscriptionafterday'], $_POST['subscriptionafteryear']);
+        $dateendsubscriptionbefore = dol_mktime($_POST['subscriptionbeforehour'], $_POST['subscriptionbeforemin'], $_POST['subscriptionbeforesec'], $_POST['subscriptionbeforemonth'], $_POST['subscriptionbeforeday'], $_POST['subscriptionbeforeyear']);
 
         // La requete doit retourner: id, email, fk_contact, name, firstname
         $sql = "SELECT a.rowid as id, a.email as email, null as fk_contact, ";
-        $sql.= " a.lastname, a.firstname,";
-        $sql.= " a.datefin, a.civility as civility_id, a.login, a.societe";    // Other fields
-        $sql.= " FROM ".MAIN_DB_PREFIX."adherent as a";
+        $sql .= " a.lastname, a.firstname,";
+        $sql .= " a.datefin, a.civility as civility_id, a.login, a.societe"; // Other fields
+        $sql .= " FROM ".MAIN_DB_PREFIX."adherent as a";
         if ($_POST['filter_category'])
         {
-        	$sql.= " LEFT JOIN ".MAIN_DB_PREFIX."categorie_member as cm ON cm.fk_member = a.rowid";
-        	$sql.= " LEFT JOIN ".MAIN_DB_PREFIX."categorie as c ON c.rowid = cm.fk_categorie";
+        	$sql .= " LEFT JOIN ".MAIN_DB_PREFIX."categorie_member as cm ON cm.fk_member = a.rowid";
+        	$sql .= " LEFT JOIN ".MAIN_DB_PREFIX."categorie as c ON c.rowid = cm.fk_categorie";
         }
-        $sql.= " , ".MAIN_DB_PREFIX."adherent_type as ta";
-        $sql.= " WHERE a.entity IN (".getEntity('member').") AND a.email <> ''";     // Note that null != '' is false
-        $sql.= " AND a.email NOT IN (SELECT email FROM ".MAIN_DB_PREFIX."mailing_cibles WHERE fk_mailing=".$this->db->escape($mailing_id).")";
+        $sql .= " , ".MAIN_DB_PREFIX."adherent_type as ta";
+        $sql .= " WHERE a.entity IN (".getEntity('member').") AND a.email <> ''"; // Note that null != '' is false
+        $sql .= " AND a.email NOT IN (SELECT email FROM ".MAIN_DB_PREFIX."mailing_cibles WHERE fk_mailing=".$this->db->escape($mailing_id).")";
         // Filter on status
-        if (isset($_POST["filter"]) && $_POST["filter"] == '-1') $sql.= " AND a.statut=-1";
-        if (isset($_POST["filter"]) && $_POST["filter"] == '1a') $sql.= " AND a.statut=1 AND a.datefin >= '".$this->db->idate($now)."'";
-        if (isset($_POST["filter"]) && $_POST["filter"] == '1b') $sql.= " AND a.statut=1 AND (a.datefin IS NULL or a.datefin < '".$this->db->idate($now)."')";
-        if (isset($_POST["filter"]) && $_POST["filter"] == '0')  $sql.= " AND a.statut=0";
+        if (isset($_POST["filter"]) && $_POST["filter"] == '-1') $sql .= " AND a.statut=-1";
+        if (isset($_POST["filter"]) && $_POST["filter"] == '1a') $sql .= " AND a.statut=1 AND a.datefin >= '".$this->db->idate($now)."'";
+        if (isset($_POST["filter"]) && $_POST["filter"] == '1b') $sql .= " AND a.statut=1 AND (a.datefin IS NULL or a.datefin < '".$this->db->idate($now)."')";
+        if (isset($_POST["filter"]) && $_POST["filter"] == '0')  $sql .= " AND a.statut=0";
         // Filter on date
-        if ($dateendsubscriptionafter > 0)  $sql.=" AND datefin > '".$this->db->idate($dateendsubscriptionafter)."'";
-        if ($dateendsubscriptionbefore > 0) $sql.=" AND datefin < '".$this->db->idate($dateendsubscriptionbefore)."'";
-        $sql.= " AND a.fk_adherent_type = ta.rowid";
+        if ($dateendsubscriptionafter > 0)  $sql .= " AND datefin > '".$this->db->idate($dateendsubscriptionafter)."'";
+        if ($dateendsubscriptionbefore > 0) $sql .= " AND datefin < '".$this->db->idate($dateendsubscriptionbefore)."'";
+        $sql .= " AND a.fk_adherent_type = ta.rowid";
         // Filter on type
-        if ($_POST['filter_type']) $sql.= " AND ta.rowid='".$_POST['filter_type']."'";
+        if ($_POST['filter_type']) $sql .= " AND ta.rowid='".$_POST['filter_type']."'";
         // Filter on category
-        if ($_POST['filter_category']) $sql.= " AND c.rowid='".$_POST['filter_category']."'";
-        $sql.= " ORDER BY a.email";
+        if ($_POST['filter_category']) $sql .= " AND c.rowid='".$_POST['filter_category']."'";
+        $sql .= " ORDER BY a.email";
         //print $sql;
 
         // Add targets into table
         dol_syslog(get_class($this)."::add_to_target", LOG_DEBUG);
-        $result=$this->db->query($sql);
+        $result = $this->db->query($sql);
         if ($result)
         {
             $num = $this->db->num_rows($result);
@@ -294,8 +294,8 @@ 
                                 'firstname' => $obj->firstname,
                                 'other' =>
                                 ($langs->transnoentities("Login").'='.$obj->login).';'.
-                                ($langs->transnoentities("UserTitle").'='.($obj->civility_id?$langs->transnoentities("Civility".$obj->civility_id):'')).';'.
-                                ($langs->transnoentities("DateEnd").'='.dol_print_date($this->db->jdate($obj->datefin),'day')).';'.
+                                ($langs->transnoentities("UserTitle").'='.($obj->civility_id ? $langs->transnoentities("Civility".$obj->civility_id) : '')).';'.
+                                ($langs->transnoentities("DateEnd").'='.dol_print_date($this->db->jdate($obj->datefin), 'day')).';'.
                                 ($langs->transnoentities("Company").'='.$obj->societe),
                                 'source_url' => $this->url($obj->id),
                                 'source_id' => $obj->id,
@@ -311,7 +311,7 @@ 
         else
         {
             dol_syslog($this->db->error());
-            $this->error=$this->db->error();
+            $this->error = $this->db->error();
             return -1;
         }
 

Braces +33 added lines, -16 removed lines

@@ -139,7 +139,9 @@ 
             $num = $this->db->num_rows($resql);
 
             $s.='<option value="0">&nbsp;</option>';
-            if (! $num) $s.='<option value="0" disabled="disabled">'.$langs->trans("NoCategoriesDefined").'</option>';
+            if (! $num) {
+                $s.='<option value="0" disabled="disabled">'.$langs->trans("NoCategoriesDefined").'</option>';
+            }
 
             $i = 0;
             while ($i < $num)
@@ -150,8 +152,7 @@ 
                 $s.='</option>';
                 $i++;
             }
-        }
-        else
+        } else
         {
             dol_print_error($this->db);
         }
@@ -178,7 +179,9 @@ 
         	$num = $this->db->num_rows($resql);
 
         	$s.='<option value="0">&nbsp;</option>';
-        	if (! $num) $s.='<option value="0" disabled>'.$langs->trans("NoCategoriesDefined").'</option>';
+        	if (! $num) {
+        	    $s.='<option value="0" disabled>'.$langs->trans("NoCategoriesDefined").'</option>';
+        	}
 
         	$i = 0;
         	while ($i < $num)
@@ -189,8 +192,7 @@ 
         		$s.='</option>';
         		$i++;
         	}
-        }
-        else
+        } else
         {
         	dol_print_error($this->db);
         }
@@ -255,18 +257,34 @@ 
         $sql.= " WHERE a.entity IN (".getEntity('member').") AND a.email <> ''";     // Note that null != '' is false
         $sql.= " AND a.email NOT IN (SELECT email FROM ".MAIN_DB_PREFIX."mailing_cibles WHERE fk_mailing=".$this->db->escape($mailing_id).")";
         // Filter on status
-        if (isset($_POST["filter"]) && $_POST["filter"] == '-1') $sql.= " AND a.statut=-1";
-        if (isset($_POST["filter"]) && $_POST["filter"] == '1a') $sql.= " AND a.statut=1 AND a.datefin >= '".$this->db->idate($now)."'";
-        if (isset($_POST["filter"]) && $_POST["filter"] == '1b') $sql.= " AND a.statut=1 AND (a.datefin IS NULL or a.datefin < '".$this->db->idate($now)."')";
-        if (isset($_POST["filter"]) && $_POST["filter"] == '0')  $sql.= " AND a.statut=0";
+        if (isset($_POST["filter"]) && $_POST["filter"] == '-1') {
+            $sql.= " AND a.statut=-1";
+        }
+        if (isset($_POST["filter"]) && $_POST["filter"] == '1a') {
+            $sql.= " AND a.statut=1 AND a.datefin >= '".$this->db->idate($now)."'";
+        }
+        if (isset($_POST["filter"]) && $_POST["filter"] == '1b') {
+            $sql.= " AND a.statut=1 AND (a.datefin IS NULL or a.datefin < '".$this->db->idate($now)."')";
+        }
+        if (isset($_POST["filter"]) && $_POST["filter"] == '0') {
+            $sql.= " AND a.statut=0";
+        }
         // Filter on date
-        if ($dateendsubscriptionafter > 0)  $sql.=" AND datefin > '".$this->db->idate($dateendsubscriptionafter)."'";
-        if ($dateendsubscriptionbefore > 0) $sql.=" AND datefin < '".$this->db->idate($dateendsubscriptionbefore)."'";
+        if ($dateendsubscriptionafter > 0) {
+            $sql.=" AND datefin > '".$this->db->idate($dateendsubscriptionafter)."'";
+        }
+        if ($dateendsubscriptionbefore > 0) {
+            $sql.=" AND datefin < '".$this->db->idate($dateendsubscriptionbefore)."'";
+        }
         $sql.= " AND a.fk_adherent_type = ta.rowid";
         // Filter on type
-        if ($_POST['filter_type']) $sql.= " AND ta.rowid='".$_POST['filter_type']."'";
+        if ($_POST['filter_type']) {
+            $sql.= " AND ta.rowid='".$_POST['filter_type']."'";
+        }
         // Filter on category
-        if ($_POST['filter_category']) $sql.= " AND c.rowid='".$_POST['filter_category']."'";
+        if ($_POST['filter_category']) {
+            $sql.= " AND c.rowid='".$_POST['filter_category']."'";
+        }
         $sql.= " ORDER BY a.email";
         //print $sql;
 
@@ -307,8 +325,7 @@ 
 
                 $i++;
             }
-        }
-        else
+        } else
         {
             dol_syslog($this->db->error());
             $this->error=$this->db->error();

dolibarr/htdocs/core/modules/mailings/modules_mailings.php

Indentation +51 added lines, -51 removed lines

@@ -37,22 +37,22 @@ 
     public $db;
 
     /**
-	 * @var string Error code (or message)
-	 */
-	public $error='';
+     * @var string Error code (or message)
+     */
+    public $error='';
 
     public $tooltip='';
 
 
     /**
-	 *	Constructor
-	 *
-	 *  @param		DoliDB		$db      Database handler
-	 */
-	function __construct($db)
-	{
+     *	Constructor
+     *
+     *  @param		DoliDB		$db      Database handler
+     */
+    function __construct($db)
+    {
         $this->db = $db;
-	}
+    }
 
     /**
      * Return description of email selector
@@ -76,7 +76,7 @@ 
     }
 
     /**
-	 *	Return number of records for email selector
+     *	Return number of records for email selector
      *
      *  @return     integer      Example
      */
@@ -101,7 +101,7 @@ 
         }
         else
         {
-        	$this->error=$this->db->lasterror();
+            $this->error=$this->db->lasterror();
             return -1;
         }
     }
@@ -162,51 +162,51 @@ 
     function add_to_target($mailing_id, $cibles)
     {
         // phpcs:enable
-    	global $conf;
+        global $conf;
 
-    	$this->db->begin();
+        $this->db->begin();
 
         // Insert emailing targets from array into database
         $j = 0;
         $num = count($cibles);
         foreach ($cibles as $targetarray)
         {
-        	if (! empty($targetarray['email'])) // avoid empty email address
-        	{
-        		$sql = "INSERT INTO ".MAIN_DB_PREFIX."mailing_cibles";
-        		$sql.= " (fk_mailing,";
-        		$sql.= " fk_contact,";
-        		$sql.= " lastname, firstname, email, other, source_url, source_id,";
-       			$sql.= " tag,";
-        		$sql.= " source_type)";
-        		$sql.= " VALUES (".$mailing_id.",";
-        		$sql.= (empty($targetarray['fk_contact']) ? '0' : "'".$targetarray['fk_contact']."'") .",";
-        		$sql.= "'".$this->db->escape($targetarray['lastname'])."',";
-        		$sql.= "'".$this->db->escape($targetarray['firstname'])."',";
-        		$sql.= "'".$this->db->escape($targetarray['email'])."',";
-        		$sql.= "'".$this->db->escape($targetarray['other'])."',";
-        		$sql.= "'".$this->db->escape($targetarray['source_url'])."',";
-        		$sql.= (empty($targetarray['source_id']) ? 'null' : "'".$this->db->escape($targetarray['source_id'])."'").",";
-       			$sql .= "'".$this->db->escape(dol_hash($targetarray['email'].';'.$targetarray['lastname'].';'.$mailing_id.';'.$conf->global->MAILING_EMAIL_UNSUBSCRIBE_KEY))."',";
-        		$sql .= "'".$this->db->escape($targetarray['source_type'])."')";
-        		dol_syslog(get_class($this)."::".__METHOD__,LOG_DEBUG);
-        		$result=$this->db->query($sql);
-        		if ($result)
-        		{
-        			$j++;
-        		}
-        		else
-        		{
-        			if ($this->db->errno() != 'DB_ERROR_RECORD_ALREADY_EXISTS')
-        			{
-        				// Si erreur autre que doublon
-        				dol_syslog($this->db->error());
-        				$this->error=$this->db->error();
-        				$this->db->rollback();
-        				return -1;
-        			}
-        		}
-        	}
+            if (! empty($targetarray['email'])) // avoid empty email address
+            {
+                $sql = "INSERT INTO ".MAIN_DB_PREFIX."mailing_cibles";
+                $sql.= " (fk_mailing,";
+                $sql.= " fk_contact,";
+                $sql.= " lastname, firstname, email, other, source_url, source_id,";
+                    $sql.= " tag,";
+                $sql.= " source_type)";
+                $sql.= " VALUES (".$mailing_id.",";
+                $sql.= (empty($targetarray['fk_contact']) ? '0' : "'".$targetarray['fk_contact']."'") .",";
+                $sql.= "'".$this->db->escape($targetarray['lastname'])."',";
+                $sql.= "'".$this->db->escape($targetarray['firstname'])."',";
+                $sql.= "'".$this->db->escape($targetarray['email'])."',";
+                $sql.= "'".$this->db->escape($targetarray['other'])."',";
+                $sql.= "'".$this->db->escape($targetarray['source_url'])."',";
+                $sql.= (empty($targetarray['source_id']) ? 'null' : "'".$this->db->escape($targetarray['source_id'])."'").",";
+                    $sql .= "'".$this->db->escape(dol_hash($targetarray['email'].';'.$targetarray['lastname'].';'.$mailing_id.';'.$conf->global->MAILING_EMAIL_UNSUBSCRIBE_KEY))."',";
+                $sql .= "'".$this->db->escape($targetarray['source_type'])."')";
+                dol_syslog(get_class($this)."::".__METHOD__,LOG_DEBUG);
+                $result=$this->db->query($sql);
+                if ($result)
+                {
+                    $j++;
+                }
+                else
+                {
+                    if ($this->db->errno() != 'DB_ERROR_RECORD_ALREADY_EXISTS')
+                    {
+                        // Si erreur autre que doublon
+                        dol_syslog($this->db->error());
+                        $this->error=$this->db->error();
+                        $this->db->rollback();
+                        return -1;
+                    }
+                }
+            }
         }
 
         dol_syslog(get_class($this)."::".__METHOD__.": mailing ".$j." targets added");
@@ -237,7 +237,7 @@ 
         $result=$this->db->query($sql);
         if (! $result)
         {
-        	dol_print_error($this->db);
+            dol_print_error($this->db);
         }
 
         $this->update_nb($mailing_id);

Spacing +33 added lines, -33 removed lines

@@ -39,9 +39,9 @@ 
     /**
 	 * @var string Error code (or message)
 	 */
-	public $error='';
+	public $error = '';
 
-    public $tooltip='';
+    public $tooltip = '';
 
 
     /**
@@ -64,12 +64,12 @@ 
         global $langs, $form;
 
         $langs->load("mails");
-        $transstring="MailingModuleDesc".$this->name;
-        $s='';
+        $transstring = "MailingModuleDesc".$this->name;
+        $s = '';
 
-        if ($langs->trans($this->name) != $this->name) $s=$langs->trans($this->name);
-        elseif ($langs->trans($transstring) != $transstring) $s=$langs->trans($transstring);
-        else $s=$this->desc;
+        if ($langs->trans($this->name) != $this->name) $s = $langs->trans($this->name);
+        elseif ($langs->trans($transstring) != $transstring) $s = $langs->trans($transstring);
+        else $s = $this->desc;
 
         if ($this->tooltip && is_object($form)) $s .= ' '.$form->textwithpicto('', $langs->trans($this->tooltip), 1, 1);
         return $s;
@@ -93,7 +93,7 @@ 
      */
     function getNbOfRecipients($sql)
     {
-        $result=$this->db->query($sql);
+        $result = $this->db->query($sql);
         if ($result)
         {
             $obj = $this->db->fetch_object($result);
@@ -101,7 +101,7 @@ 
         }
         else
         {
-        	$this->error=$this->db->lasterror();
+        	$this->error = $this->db->lasterror();
             return -1;
         }
     }
@@ -130,18 +130,18 @@ 
         // Mise a jour nombre de destinataire dans table des mailings
         $sql = "SELECT COUNT(*) nb FROM ".MAIN_DB_PREFIX."mailing_cibles";
         $sql .= " WHERE fk_mailing = ".$mailing_id;
-        $result=$this->db->query($sql);
+        $result = $this->db->query($sql);
         if ($result)
         {
-            $obj=$this->db->fetch_object($result);
-            $nb=$obj->nb;
+            $obj = $this->db->fetch_object($result);
+            $nb = $obj->nb;
 
             $sql = "UPDATE ".MAIN_DB_PREFIX."mailing";
             $sql .= " SET nbemail = ".$nb." WHERE rowid = ".$mailing_id;
             if (!$this->db->query($sql))
             {
                 dol_syslog($this->db->error());
-                $this->error=$this->db->error();
+                $this->error = $this->db->error();
                 return -1;
             }
         }
@@ -171,26 +171,26 @@ 
         $num = count($cibles);
         foreach ($cibles as $targetarray)
         {
-        	if (! empty($targetarray['email'])) // avoid empty email address
+        	if (!empty($targetarray['email'])) // avoid empty email address
         	{
         		$sql = "INSERT INTO ".MAIN_DB_PREFIX."mailing_cibles";
-        		$sql.= " (fk_mailing,";
-        		$sql.= " fk_contact,";
-        		$sql.= " lastname, firstname, email, other, source_url, source_id,";
-       			$sql.= " tag,";
-        		$sql.= " source_type)";
-        		$sql.= " VALUES (".$mailing_id.",";
-        		$sql.= (empty($targetarray['fk_contact']) ? '0' : "'".$targetarray['fk_contact']."'") .",";
-        		$sql.= "'".$this->db->escape($targetarray['lastname'])."',";
-        		$sql.= "'".$this->db->escape($targetarray['firstname'])."',";
-        		$sql.= "'".$this->db->escape($targetarray['email'])."',";
-        		$sql.= "'".$this->db->escape($targetarray['other'])."',";
-        		$sql.= "'".$this->db->escape($targetarray['source_url'])."',";
-        		$sql.= (empty($targetarray['source_id']) ? 'null' : "'".$this->db->escape($targetarray['source_id'])."'").",";
+        		$sql .= " (fk_mailing,";
+        		$sql .= " fk_contact,";
+        		$sql .= " lastname, firstname, email, other, source_url, source_id,";
+       			$sql .= " tag,";
+        		$sql .= " source_type)";
+        		$sql .= " VALUES (".$mailing_id.",";
+        		$sql .= (empty($targetarray['fk_contact']) ? '0' : "'".$targetarray['fk_contact']."'").",";
+        		$sql .= "'".$this->db->escape($targetarray['lastname'])."',";
+        		$sql .= "'".$this->db->escape($targetarray['firstname'])."',";
+        		$sql .= "'".$this->db->escape($targetarray['email'])."',";
+        		$sql .= "'".$this->db->escape($targetarray['other'])."',";
+        		$sql .= "'".$this->db->escape($targetarray['source_url'])."',";
+        		$sql .= (empty($targetarray['source_id']) ? 'null' : "'".$this->db->escape($targetarray['source_id'])."'").",";
        			$sql .= "'".$this->db->escape(dol_hash($targetarray['email'].';'.$targetarray['lastname'].';'.$mailing_id.';'.$conf->global->MAILING_EMAIL_UNSUBSCRIBE_KEY))."',";
         		$sql .= "'".$this->db->escape($targetarray['source_type'])."')";
-        		dol_syslog(get_class($this)."::".__METHOD__,LOG_DEBUG);
-        		$result=$this->db->query($sql);
+        		dol_syslog(get_class($this)."::".__METHOD__, LOG_DEBUG);
+        		$result = $this->db->query($sql);
         		if ($result)
         		{
         			$j++;
@@ -201,7 +201,7 @@ 
         			{
         				// Si erreur autre que doublon
         				dol_syslog($this->db->error());
-        				$this->error=$this->db->error();
+        				$this->error = $this->db->error();
         				$this->db->rollback();
         				return -1;
         			}
@@ -234,8 +234,8 @@ 
         $sql .= " WHERE fk_mailing=".$mailing_id." AND email IN (SELECT mu.email FROM ".MAIN_DB_PREFIX."mailing_unsubscribe AS mu WHERE mu.entity IN ('".getEntity('mailing')."'))";
 
         dol_syslog(get_class($this)."::".__METHOD__.":mailing update status to display emails that do not want to be contacted anymore", LOG_DEBUG);
-        $result=$this->db->query($sql);
-        if (! $result)
+        $result = $this->db->query($sql);
+        if (!$result)
         {
         	dol_print_error($this->db);
         }
@@ -260,7 +260,7 @@ 
         $sql = "DELETE FROM ".MAIN_DB_PREFIX."mailing_cibles";
         $sql .= " WHERE fk_mailing = ".$mailing_id;
 
-        if (! $this->db->query($sql))
+        if (!$this->db->query($sql))
         {
             dol_syslog($this->db->error());
         }

Braces +16 added lines, -11 removed lines

@@ -67,11 +67,17 @@ 
         $transstring="MailingModuleDesc".$this->name;
         $s='';
 
-        if ($langs->trans($this->name) != $this->name) $s=$langs->trans($this->name);
-        elseif ($langs->trans($transstring) != $transstring) $s=$langs->trans($transstring);
-        else $s=$this->desc;
+        if ($langs->trans($this->name) != $this->name) {
+            $s=$langs->trans($this->name);
+        } elseif ($langs->trans($transstring) != $transstring) {
+            $s=$langs->trans($transstring);
+        } else {
+            $s=$this->desc;
+        }
 
-        if ($this->tooltip && is_object($form)) $s .= ' '.$form->textwithpicto('', $langs->trans($this->tooltip), 1, 1);
+        if ($this->tooltip && is_object($form)) {
+            $s .= ' '.$form->textwithpicto('', $langs->trans($this->tooltip), 1, 1);
+        }
         return $s;
     }
 
@@ -98,8 +104,7 @@ 
         {
             $obj = $this->db->fetch_object($result);
             return $obj->nb;
-        }
-        else
+        } else
         {
         	$this->error=$this->db->lasterror();
             return -1;
@@ -144,8 +149,7 @@ 
                 $this->error=$this->db->error();
                 return -1;
             }
-        }
-        else {
+        } else {
             return -1;
         }
         return $nb;
@@ -171,9 +175,11 @@ 
         $num = count($cibles);
         foreach ($cibles as $targetarray)
         {
-        	if (! empty($targetarray['email'])) // avoid empty email address
+        	if (! empty($targetarray['email'])) {
+        	    // avoid empty email address
         	{
         		$sql = "INSERT INTO ".MAIN_DB_PREFIX."mailing_cibles";
+        	}
         		$sql.= " (fk_mailing,";
         		$sql.= " fk_contact,";
         		$sql.= " lastname, firstname, email, other, source_url, source_id,";
@@ -194,8 +200,7 @@ 
         		if ($result)
         		{
         			$j++;
-        		}
-        		else
+        		} else
         		{
         			if ($this->db->errno() != 'DB_ERROR_RECORD_ALREADY_EXISTS')
         			{

dolibarr/htdocs/core/modules/mailings/thirdparties.modules.php

Indentation +204 added lines, -204 removed lines

@@ -24,77 +24,77 @@ 
  */
 class mailing_thirdparties extends MailingTargets
 {
-	var $name='ThirdPartiesByCategories';
-	// This label is used if no translation is found for key XXX neither MailingModuleDescXXX where XXX=name is found
-	var $desc="Third parties (by categories)";
-	var $require_admin=0;
+    var $name='ThirdPartiesByCategories';
+    // This label is used if no translation is found for key XXX neither MailingModuleDescXXX where XXX=name is found
+    var $desc="Third parties (by categories)";
+    var $require_admin=0;
 
-	var $require_module=array("societe");	// This module allows to select by categories must be also enabled if category module is not activated
-	var $picto='company';
+    var $require_module=array("societe");	// This module allows to select by categories must be also enabled if category module is not activated
+    var $picto='company';
 
-	/**
+    /**
      * @var DoliDB Database handler.
      */
     public $db;
 
 
-	/**
-	 *	Constructor
-	 *
-	 *  @param		DoliDB		$db      Database handler
-	 */
-	function __construct($db)
-	{
-		global $conf, $langs;
+    /**
+     *	Constructor
+     *
+     *  @param		DoliDB		$db      Database handler
+     */
+    function __construct($db)
+    {
+        global $conf, $langs;
         $langs->load("companies");
 
-		$this->db=$db;
-	}
+        $this->db=$db;
+    }
 
 
     // phpcs:disable PEAR.NamingConventions.ValidFunctionName.NotCamelCaps
-	/**
-	 *    This is the main function that returns the array of emails
-	 *
-	 *    @param	int		$mailing_id    	Id of mailing. No need to use it.
-	 *    @return   int 					<0 if error, number of emails added if ok
-	 */
-	function add_to_target($mailing_id)
-	{
+    /**
+     *    This is the main function that returns the array of emails
+     *
+     *    @param	int		$mailing_id    	Id of mailing. No need to use it.
+     *    @return   int 					<0 if error, number of emails added if ok
+     */
+    function add_to_target($mailing_id)
+    {
         // phpcs:enable
-		global $conf, $langs;
-
-		$cibles = array();
-
-		// Select the third parties from category
-		if (empty($_POST['filter']))
-		{
-		    $sql = "SELECT s.rowid as id, s.email as email, s.nom as name, null as fk_contact, null as firstname, null as label";
-		    $sql.= " FROM ".MAIN_DB_PREFIX."societe as s";
-		    $sql.= " WHERE s.email <> ''";
-		    $sql.= " AND s.entity IN (".getEntity('societe').")";
-		    $sql.= " AND s.email NOT IN (SELECT email FROM ".MAIN_DB_PREFIX."mailing_cibles WHERE fk_mailing=".$mailing_id.")";
-		}
-		else
-		{
-		    $sql = "SELECT s.rowid as id, s.email as email, s.nom as name, null as fk_contact, null as firstname, c.label as label";
-		    $sql.= " FROM ".MAIN_DB_PREFIX."societe as s, ".MAIN_DB_PREFIX."categorie_societe as cs, ".MAIN_DB_PREFIX."categorie as c";
-		    $sql.= " WHERE s.email <> ''";
-		    $sql.= " AND s.entity IN (".getEntity('societe').")";
-		    $sql.= " AND s.email NOT IN (SELECT email FROM ".MAIN_DB_PREFIX."mailing_cibles WHERE fk_mailing=".$mailing_id.")";
-		    $sql.= " AND cs.fk_soc = s.rowid";
-		    $sql.= " AND c.rowid = cs.fk_categorie";
-		    $sql.= " AND c.rowid='".$this->db->escape($_POST['filter'])."'";
-		    $sql.= " UNION ";
-		    $sql.= "SELECT s.rowid as id, s.email as email, s.nom as name, null as fk_contact, null as firstname, c.label as label";
-		    $sql.= " FROM ".MAIN_DB_PREFIX."societe as s, ".MAIN_DB_PREFIX."categorie_fournisseur as cs, ".MAIN_DB_PREFIX."categorie as c";
-		    $sql.= " WHERE s.email <> ''";
-		    $sql.= " AND s.entity IN (".getEntity('societe').")";
-		    $sql.= " AND s.email NOT IN (SELECT email FROM ".MAIN_DB_PREFIX."mailing_cibles WHERE fk_mailing=".$mailing_id.")";
-		    $sql.= " AND cs.fk_soc = s.rowid";
-		    $sql.= " AND c.rowid = cs.fk_categorie";
-		    $sql.= " AND c.rowid='".$this->db->escape($_POST['filter'])."'";
-		}
+        global $conf, $langs;
+
+        $cibles = array();
+
+        // Select the third parties from category
+        if (empty($_POST['filter']))
+        {
+            $sql = "SELECT s.rowid as id, s.email as email, s.nom as name, null as fk_contact, null as firstname, null as label";
+            $sql.= " FROM ".MAIN_DB_PREFIX."societe as s";
+            $sql.= " WHERE s.email <> ''";
+            $sql.= " AND s.entity IN (".getEntity('societe').")";
+            $sql.= " AND s.email NOT IN (SELECT email FROM ".MAIN_DB_PREFIX."mailing_cibles WHERE fk_mailing=".$mailing_id.")";
+        }
+        else
+        {
+            $sql = "SELECT s.rowid as id, s.email as email, s.nom as name, null as fk_contact, null as firstname, c.label as label";
+            $sql.= " FROM ".MAIN_DB_PREFIX."societe as s, ".MAIN_DB_PREFIX."categorie_societe as cs, ".MAIN_DB_PREFIX."categorie as c";
+            $sql.= " WHERE s.email <> ''";
+            $sql.= " AND s.entity IN (".getEntity('societe').")";
+            $sql.= " AND s.email NOT IN (SELECT email FROM ".MAIN_DB_PREFIX."mailing_cibles WHERE fk_mailing=".$mailing_id.")";
+            $sql.= " AND cs.fk_soc = s.rowid";
+            $sql.= " AND c.rowid = cs.fk_categorie";
+            $sql.= " AND c.rowid='".$this->db->escape($_POST['filter'])."'";
+            $sql.= " UNION ";
+            $sql.= "SELECT s.rowid as id, s.email as email, s.nom as name, null as fk_contact, null as firstname, c.label as label";
+            $sql.= " FROM ".MAIN_DB_PREFIX."societe as s, ".MAIN_DB_PREFIX."categorie_fournisseur as cs, ".MAIN_DB_PREFIX."categorie as c";
+            $sql.= " WHERE s.email <> ''";
+            $sql.= " AND s.entity IN (".getEntity('societe').")";
+            $sql.= " AND s.email NOT IN (SELECT email FROM ".MAIN_DB_PREFIX."mailing_cibles WHERE fk_mailing=".$mailing_id.")";
+            $sql.= " AND cs.fk_soc = s.rowid";
+            $sql.= " AND c.rowid = cs.fk_categorie";
+            $sql.= " AND c.rowid='".$this->db->escape($_POST['filter'])."'";
+        }
 
         $addDescription= "";
         if (isset($_POST["filter_client"]) && $_POST["filter_client"] <> '-1')
@@ -143,149 +143,149 @@ 
         $sql.= " ORDER BY email";
 
         // Stock recipients emails into targets table
-		$result=$this->db->query($sql);
-		if ($result)
-		{
-			$num = $this->db->num_rows($result);
-			$i = 0;
-			$j = 0;
-
-			dol_syslog(get_class($this)."::add_to_target mailing ".$num." targets found");
-
-			$old = '';
-			while ($i < $num)
-			{
-				$obj = $this->db->fetch_object($result);
-				if ($old <> $obj->email)
-				{
-					$otherTxt= ($obj->label?$langs->transnoentities("Category").'='.$obj->label:'');
-					if (strlen($addDescription) > 0 && strlen($otherTxt) > 0)
-					{
-						$otherTxt.= ";";
-					}
-					$otherTxt.= $addDescription;
-					$cibles[$j] = array(
-                    			'email' => $obj->email,
-                    			'fk_contact' => $obj->fk_contact,
-                    			'lastname' => $obj->name,	// For a thirdparty, we must use name
-                    			'firstname' => '',			// For a thirdparty, lastname is ''
-                    			'other' => $otherTxt,
+        $result=$this->db->query($sql);
+        if ($result)
+        {
+            $num = $this->db->num_rows($result);
+            $i = 0;
+            $j = 0;
+
+            dol_syslog(get_class($this)."::add_to_target mailing ".$num." targets found");
+
+            $old = '';
+            while ($i < $num)
+            {
+                $obj = $this->db->fetch_object($result);
+                if ($old <> $obj->email)
+                {
+                    $otherTxt= ($obj->label?$langs->transnoentities("Category").'='.$obj->label:'');
+                    if (strlen($addDescription) > 0 && strlen($otherTxt) > 0)
+                    {
+                        $otherTxt.= ";";
+                    }
+                    $otherTxt.= $addDescription;
+                    $cibles[$j] = array(
+                                'email' => $obj->email,
+                                'fk_contact' => $obj->fk_contact,
+                                'lastname' => $obj->name,	// For a thirdparty, we must use name
+                                'firstname' => '',			// For a thirdparty, lastname is ''
+                                'other' => $otherTxt,
                                 'source_url' => $this->url($obj->id),
                                 'source_id' => $obj->id,
                                 'source_type' => 'thirdparty'
-					);
-					$old = $obj->email;
-					$j++;
-				}
+                    );
+                    $old = $obj->email;
+                    $j++;
+                }
+
+                $i++;
+            }
+        }
+        else
+        {
+            dol_syslog($this->db->error());
+            $this->error=$this->db->error();
+            return -1;
+        }
+
+        return parent::add_to_target($mailing_id, $cibles);
+    }
+
 
-				$i++;
-			}
-		}
-		else
-		{
-			dol_syslog($this->db->error());
-			$this->error=$this->db->error();
-			return -1;
-		}
+    /**
+     *	On the main mailing area, there is a box with statistics.
+     *	If you want to add a line in this report you must provide an
+     *	array of SQL request that returns two field:
+     *	One called "label", One called "nb".
+     *
+     *	@return		array		Array with SQL requests
+     */
+    function getSqlArrayForStats()
+    {
+        // CHANGE THIS: Optionnal
+
+        //var $statssql=array();
+        //$this->statssql[0]="SELECT field1 as label, count(distinct(email)) as nb FROM mytable WHERE email IS NOT NULL";
+        return array();
+    }
+
+
+    /**
+     *	Return here number of distinct emails returned by your selector.
+     *	For example if this selector is used to extract 500 different
+     *	emails from a text file, this function must return 500.
+     *
+     *  @param      string	$sql        Requete sql de comptage
+     *	@return		int					Nb of recipients
+     */
+    function getNbOfRecipients($sql='')
+    {
+        global $conf;
 
-		return parent::add_to_target($mailing_id, $cibles);
-	}
+        $sql = "SELECT count(distinct(s.email)) as nb";
+        $sql.= " FROM ".MAIN_DB_PREFIX."societe as s";
+        $sql.= " WHERE s.email != ''";
+        $sql.= " AND s.entity IN (".getEntity('societe').")";
 
+        // La requete doit retourner un champ "nb" pour etre comprise
+        // par parent::getNbOfRecipients
+        return parent::getNbOfRecipients($sql);
+    }
 
     /**
-	 *	On the main mailing area, there is a box with statistics.
-	 *	If you want to add a line in this report you must provide an
-	 *	array of SQL request that returns two field:
-	 *	One called "label", One called "nb".
-	 *
-	 *	@return		array		Array with SQL requests
-	 */
-	function getSqlArrayForStats()
-	{
-		// CHANGE THIS: Optionnal
-
-		//var $statssql=array();
-		//$this->statssql[0]="SELECT field1 as label, count(distinct(email)) as nb FROM mytable WHERE email IS NOT NULL";
-		return array();
-	}
-
-
-	/**
-	 *	Return here number of distinct emails returned by your selector.
-	 *	For example if this selector is used to extract 500 different
-	 *	emails from a text file, this function must return 500.
-	 *
-	 *  @param      string	$sql        Requete sql de comptage
-	 *	@return		int					Nb of recipients
-	 */
-	function getNbOfRecipients($sql='')
-	{
-		global $conf;
-
-		$sql = "SELECT count(distinct(s.email)) as nb";
-		$sql.= " FROM ".MAIN_DB_PREFIX."societe as s";
-		$sql.= " WHERE s.email != ''";
-		$sql.= " AND s.entity IN (".getEntity('societe').")";
-
-		// La requete doit retourner un champ "nb" pour etre comprise
-		// par parent::getNbOfRecipients
-		return parent::getNbOfRecipients($sql);
-	}
-
-	/**
-	 *  This is to add a form filter to provide variant of selector
-	 *	If used, the HTML select must be called "filter"
-	 *
-	 *  @return     string      A html select zone
-	 */
-	function formFilter()
-	{
-		global $conf, $langs;
-
-		$langs->load("companies");
-
-		$s=$langs->trans("Categories").': ';
-		$s.='<select name="filter" class="flat">';
-
-		// Show categories
-		$sql = "SELECT rowid, label, type, visible";
-		$sql.= " FROM ".MAIN_DB_PREFIX."categorie";
-		$sql.= " WHERE type in (1,2)";	// We keep only categories for suppliers and customers/prospects
-		// $sql.= " AND visible > 0";	// We ignore the property visible because third party's categories does not use this property (only products categories use it).
-		$sql.= " AND entity = ".$conf->entity;
-		$sql.= " ORDER BY label";
-
-		//print $sql;
-		$resql = $this->db->query($sql);
-		if ($resql)
-		{
-			$num = $this->db->num_rows($resql);
-
-			if (empty($conf->categorie->enabled)) $num=0;	// Force empty list if category module is not enabled
-
-			if ($num) $s.='<option value="0">&nbsp;</option>';
-			else $s.='<option value="0">'.$langs->trans("ContactsAllShort").'</option>';
-
-			$i = 0;
-			while ($i < $num)
-			{
-				$obj = $this->db->fetch_object($resql);
-
-				$type='';
-				if ($obj->type == 1) $type=$langs->trans("Supplier");
-				if ($obj->type == 2) $type=$langs->trans("Customer");
-				$s.='<option value="'.$obj->rowid.'">'.dol_trunc($obj->label,38,'middle');
-				if ($type) $s.=' ('.$type.')';
-				$s.='</option>';
-				$i++;
-			}
-		}
-		else
-		{
-			dol_print_error($this->db);
-		}
-
-		$s.='</select> ';
+     *  This is to add a form filter to provide variant of selector
+     *	If used, the HTML select must be called "filter"
+     *
+     *  @return     string      A html select zone
+     */
+    function formFilter()
+    {
+        global $conf, $langs;
+
+        $langs->load("companies");
+
+        $s=$langs->trans("Categories").': ';
+        $s.='<select name="filter" class="flat">';
+
+        // Show categories
+        $sql = "SELECT rowid, label, type, visible";
+        $sql.= " FROM ".MAIN_DB_PREFIX."categorie";
+        $sql.= " WHERE type in (1,2)";	// We keep only categories for suppliers and customers/prospects
+        // $sql.= " AND visible > 0";	// We ignore the property visible because third party's categories does not use this property (only products categories use it).
+        $sql.= " AND entity = ".$conf->entity;
+        $sql.= " ORDER BY label";
+
+        //print $sql;
+        $resql = $this->db->query($sql);
+        if ($resql)
+        {
+            $num = $this->db->num_rows($resql);
+
+            if (empty($conf->categorie->enabled)) $num=0;	// Force empty list if category module is not enabled
+
+            if ($num) $s.='<option value="0">&nbsp;</option>';
+            else $s.='<option value="0">'.$langs->trans("ContactsAllShort").'</option>';
+
+            $i = 0;
+            while ($i < $num)
+            {
+                $obj = $this->db->fetch_object($resql);
+
+                $type='';
+                if ($obj->type == 1) $type=$langs->trans("Supplier");
+                if ($obj->type == 2) $type=$langs->trans("Customer");
+                $s.='<option value="'.$obj->rowid.'">'.dol_trunc($obj->label,38,'middle');
+                if ($type) $s.=' ('.$type.')';
+                $s.='</option>';
+                $i++;
+            }
+        }
+        else
+        {
+            dol_print_error($this->db);
+        }
+
+        $s.='</select> ';
                 $s.= $langs->trans('ProspectCustomer');
                 $s.=': <select name="filter_client" class="flat">';
                 $s.= '<option value="-1">&nbsp;</option>';
@@ -308,19 +308,19 @@ 
                 $s.='<option value="-1">&nbsp;</option>';
                 $s.='<option value="1" selected>'.$langs->trans("Enabled").'</option>';
                 $s.='<option value="0">'.$langs->trans("Disabled").'</option>';
-		$s.='</select>';
-		return $s;
-	}
+        $s.='</select>';
+        return $s;
+    }
 
 
     /**
-	 *  Can include an URL link on each record provided by selector shown on target page.
-	 *
+     *  Can include an URL link on each record provided by selector shown on target page.
+     *
      *  @param	int		$id		ID
-	 *  @return string      	Url link
-	 */
-	function url($id)
-	{
-		return '<a href="'.DOL_URL_ROOT.'/societe/card.php?socid='.$id.'">'.img_object('',"company").'</a>';
-	}
+     *  @return string      	Url link
+     */
+    function url($id)
+    {
+        return '<a href="'.DOL_URL_ROOT.'/societe/card.php?socid='.$id.'">'.img_object('',"company").'</a>';
+    }
 }

Spacing +83 added lines, -83 removed lines

@@ -24,13 +24,13 @@ 
  */
 class mailing_thirdparties extends MailingTargets
 {
-	var $name='ThirdPartiesByCategories';
+	var $name = 'ThirdPartiesByCategories';
 	// This label is used if no translation is found for key XXX neither MailingModuleDescXXX where XXX=name is found
-	var $desc="Third parties (by categories)";
-	var $require_admin=0;
+	var $desc = "Third parties (by categories)";
+	var $require_admin = 0;
 
-	var $require_module=array("societe");	// This module allows to select by categories must be also enabled if category module is not activated
-	var $picto='company';
+	var $require_module = array("societe"); // This module allows to select by categories must be also enabled if category module is not activated
+	var $picto = 'company';
 
 	/**
      * @var DoliDB Database handler.
@@ -48,7 +48,7 @@ 
 		global $conf, $langs;
         $langs->load("companies");
 
-		$this->db=$db;
+		$this->db = $db;
 	}
 
 
@@ -70,80 +70,80 @@ 
 		if (empty($_POST['filter']))
 		{
 		    $sql = "SELECT s.rowid as id, s.email as email, s.nom as name, null as fk_contact, null as firstname, null as label";
-		    $sql.= " FROM ".MAIN_DB_PREFIX."societe as s";
-		    $sql.= " WHERE s.email <> ''";
-		    $sql.= " AND s.entity IN (".getEntity('societe').")";
-		    $sql.= " AND s.email NOT IN (SELECT email FROM ".MAIN_DB_PREFIX."mailing_cibles WHERE fk_mailing=".$mailing_id.")";
+		    $sql .= " FROM ".MAIN_DB_PREFIX."societe as s";
+		    $sql .= " WHERE s.email <> ''";
+		    $sql .= " AND s.entity IN (".getEntity('societe').")";
+		    $sql .= " AND s.email NOT IN (SELECT email FROM ".MAIN_DB_PREFIX."mailing_cibles WHERE fk_mailing=".$mailing_id.")";
 		}
 		else
 		{
 		    $sql = "SELECT s.rowid as id, s.email as email, s.nom as name, null as fk_contact, null as firstname, c.label as label";
-		    $sql.= " FROM ".MAIN_DB_PREFIX."societe as s, ".MAIN_DB_PREFIX."categorie_societe as cs, ".MAIN_DB_PREFIX."categorie as c";
-		    $sql.= " WHERE s.email <> ''";
-		    $sql.= " AND s.entity IN (".getEntity('societe').")";
-		    $sql.= " AND s.email NOT IN (SELECT email FROM ".MAIN_DB_PREFIX."mailing_cibles WHERE fk_mailing=".$mailing_id.")";
-		    $sql.= " AND cs.fk_soc = s.rowid";
-		    $sql.= " AND c.rowid = cs.fk_categorie";
-		    $sql.= " AND c.rowid='".$this->db->escape($_POST['filter'])."'";
-		    $sql.= " UNION ";
-		    $sql.= "SELECT s.rowid as id, s.email as email, s.nom as name, null as fk_contact, null as firstname, c.label as label";
-		    $sql.= " FROM ".MAIN_DB_PREFIX."societe as s, ".MAIN_DB_PREFIX."categorie_fournisseur as cs, ".MAIN_DB_PREFIX."categorie as c";
-		    $sql.= " WHERE s.email <> ''";
-		    $sql.= " AND s.entity IN (".getEntity('societe').")";
-		    $sql.= " AND s.email NOT IN (SELECT email FROM ".MAIN_DB_PREFIX."mailing_cibles WHERE fk_mailing=".$mailing_id.")";
-		    $sql.= " AND cs.fk_soc = s.rowid";
-		    $sql.= " AND c.rowid = cs.fk_categorie";
-		    $sql.= " AND c.rowid='".$this->db->escape($_POST['filter'])."'";
+		    $sql .= " FROM ".MAIN_DB_PREFIX."societe as s, ".MAIN_DB_PREFIX."categorie_societe as cs, ".MAIN_DB_PREFIX."categorie as c";
+		    $sql .= " WHERE s.email <> ''";
+		    $sql .= " AND s.entity IN (".getEntity('societe').")";
+		    $sql .= " AND s.email NOT IN (SELECT email FROM ".MAIN_DB_PREFIX."mailing_cibles WHERE fk_mailing=".$mailing_id.")";
+		    $sql .= " AND cs.fk_soc = s.rowid";
+		    $sql .= " AND c.rowid = cs.fk_categorie";
+		    $sql .= " AND c.rowid='".$this->db->escape($_POST['filter'])."'";
+		    $sql .= " UNION ";
+		    $sql .= "SELECT s.rowid as id, s.email as email, s.nom as name, null as fk_contact, null as firstname, c.label as label";
+		    $sql .= " FROM ".MAIN_DB_PREFIX."societe as s, ".MAIN_DB_PREFIX."categorie_fournisseur as cs, ".MAIN_DB_PREFIX."categorie as c";
+		    $sql .= " WHERE s.email <> ''";
+		    $sql .= " AND s.entity IN (".getEntity('societe').")";
+		    $sql .= " AND s.email NOT IN (SELECT email FROM ".MAIN_DB_PREFIX."mailing_cibles WHERE fk_mailing=".$mailing_id.")";
+		    $sql .= " AND cs.fk_soc = s.rowid";
+		    $sql .= " AND c.rowid = cs.fk_categorie";
+		    $sql .= " AND c.rowid='".$this->db->escape($_POST['filter'])."'";
 		}
 
-        $addDescription= "";
+        $addDescription = "";
         if (isset($_POST["filter_client"]) && $_POST["filter_client"] <> '-1')
         {
-            $sql.= " AND s.client=" . $_POST["filter_client"];
-            $addDescription= $langs->trans('ProspectCustomer')."=";
+            $sql .= " AND s.client=".$_POST["filter_client"];
+            $addDescription = $langs->trans('ProspectCustomer')."=";
             if ($_POST["filter_client"] == 0)
             {
-                $addDescription.= $langs->trans('NorProspectNorCustomer');
+                $addDescription .= $langs->trans('NorProspectNorCustomer');
             }
             elseif ($_POST["filter_client"] == 1)
             {
-                $addDescription.= $langs->trans('Customer');
+                $addDescription .= $langs->trans('Customer');
             }
             elseif ($_POST["filter_client"] == 2)
             {
-                $addDescription.= $langs->trans('Prospect');
+                $addDescription .= $langs->trans('Prospect');
             }
             elseif ($_POST["filter_client"] == 3)
             {
-                $addDescription.= $langs->trans('ProspectCustomer');
+                $addDescription .= $langs->trans('ProspectCustomer');
             }
             else
             {
-                $addDescription.= "Unknown status ".$_POST["filter_client"];
+                $addDescription .= "Unknown status ".$_POST["filter_client"];
             }
         }
         if (isset($_POST["filter_status"]))
         {
             if (strlen($addDescription) > 0)
             {
-                $addDescription.= ";";
+                $addDescription .= ";";
             }
-            $addDescription.= $langs->trans("Status")."=";
+            $addDescription .= $langs->trans("Status")."=";
             if ($_POST["filter_status"] == '1')
             {
-                $sql.= " AND s.status=1";
-                $addDescription.= $langs->trans("Enabled");
+                $sql .= " AND s.status=1";
+                $addDescription .= $langs->trans("Enabled");
             }
             else
             {
-                $sql.= " AND s.status=0";
-                $addDescription.= $langs->trans("Disabled");
+                $sql .= " AND s.status=0";
+                $addDescription .= $langs->trans("Disabled");
             }
         }
-        $sql.= " ORDER BY email";
+        $sql .= " ORDER BY email";
 
         // Stock recipients emails into targets table
-		$result=$this->db->query($sql);
+		$result = $this->db->query($sql);
 		if ($result)
 		{
 			$num = $this->db->num_rows($result);
@@ -158,17 +158,17 @@ 
 				$obj = $this->db->fetch_object($result);
 				if ($old <> $obj->email)
 				{
-					$otherTxt= ($obj->label?$langs->transnoentities("Category").'='.$obj->label:'');
+					$otherTxt = ($obj->label ? $langs->transnoentities("Category").'='.$obj->label : '');
 					if (strlen($addDescription) > 0 && strlen($otherTxt) > 0)
 					{
-						$otherTxt.= ";";
+						$otherTxt .= ";";
 					}
-					$otherTxt.= $addDescription;
+					$otherTxt .= $addDescription;
 					$cibles[$j] = array(
                     			'email' => $obj->email,
                     			'fk_contact' => $obj->fk_contact,
-                    			'lastname' => $obj->name,	// For a thirdparty, we must use name
-                    			'firstname' => '',			// For a thirdparty, lastname is ''
+                    			'lastname' => $obj->name, // For a thirdparty, we must use name
+                    			'firstname' => '', // For a thirdparty, lastname is ''
                     			'other' => $otherTxt,
                                 'source_url' => $this->url($obj->id),
                                 'source_id' => $obj->id,
@@ -184,7 +184,7 @@ 
 		else
 		{
 			dol_syslog($this->db->error());
-			$this->error=$this->db->error();
+			$this->error = $this->db->error();
 			return -1;
 		}
 
@@ -218,14 +218,14 @@ 
 	 *  @param      string	$sql        Requete sql de comptage
 	 *	@return		int					Nb of recipients
 	 */
-	function getNbOfRecipients($sql='')
+	function getNbOfRecipients($sql = '')
 	{
 		global $conf;
 
 		$sql = "SELECT count(distinct(s.email)) as nb";
-		$sql.= " FROM ".MAIN_DB_PREFIX."societe as s";
-		$sql.= " WHERE s.email != ''";
-		$sql.= " AND s.entity IN (".getEntity('societe').")";
+		$sql .= " FROM ".MAIN_DB_PREFIX."societe as s";
+		$sql .= " WHERE s.email != ''";
+		$sql .= " AND s.entity IN (".getEntity('societe').")";
 
 		// La requete doit retourner un champ "nb" pour etre comprise
 		// par parent::getNbOfRecipients
@@ -244,16 +244,16 @@ 
 
 		$langs->load("companies");
 
-		$s=$langs->trans("Categories").': ';
-		$s.='<select name="filter" class="flat">';
+		$s = $langs->trans("Categories").': ';
+		$s .= '<select name="filter" class="flat">';
 
 		// Show categories
 		$sql = "SELECT rowid, label, type, visible";
-		$sql.= " FROM ".MAIN_DB_PREFIX."categorie";
-		$sql.= " WHERE type in (1,2)";	// We keep only categories for suppliers and customers/prospects
+		$sql .= " FROM ".MAIN_DB_PREFIX."categorie";
+		$sql .= " WHERE type in (1,2)"; // We keep only categories for suppliers and customers/prospects
 		// $sql.= " AND visible > 0";	// We ignore the property visible because third party's categories does not use this property (only products categories use it).
-		$sql.= " AND entity = ".$conf->entity;
-		$sql.= " ORDER BY label";
+		$sql .= " AND entity = ".$conf->entity;
+		$sql .= " ORDER BY label";
 
 		//print $sql;
 		$resql = $this->db->query($sql);
@@ -261,22 +261,22 @@ 
 		{
 			$num = $this->db->num_rows($resql);
 
-			if (empty($conf->categorie->enabled)) $num=0;	// Force empty list if category module is not enabled
+			if (empty($conf->categorie->enabled)) $num = 0; // Force empty list if category module is not enabled
 
-			if ($num) $s.='<option value="0">&nbsp;</option>';
-			else $s.='<option value="0">'.$langs->trans("ContactsAllShort").'</option>';
+			if ($num) $s .= '<option value="0">&nbsp;</option>';
+			else $s .= '<option value="0">'.$langs->trans("ContactsAllShort").'</option>';
 
 			$i = 0;
 			while ($i < $num)
 			{
 				$obj = $this->db->fetch_object($resql);
 
-				$type='';
-				if ($obj->type == 1) $type=$langs->trans("Supplier");
-				if ($obj->type == 2) $type=$langs->trans("Customer");
-				$s.='<option value="'.$obj->rowid.'">'.dol_trunc($obj->label,38,'middle');
-				if ($type) $s.=' ('.$type.')';
-				$s.='</option>';
+				$type = '';
+				if ($obj->type == 1) $type = $langs->trans("Supplier");
+				if ($obj->type == 2) $type = $langs->trans("Customer");
+				$s .= '<option value="'.$obj->rowid.'">'.dol_trunc($obj->label, 38, 'middle');
+				if ($type) $s .= ' ('.$type.')';
+				$s .= '</option>';
 				$i++;
 			}
 		}
@@ -285,30 +285,30 @@ 
 			dol_print_error($this->db);
 		}
 
-		$s.='</select> ';
-                $s.= $langs->trans('ProspectCustomer');
-                $s.=': <select name="filter_client" class="flat">';
-                $s.= '<option value="-1">&nbsp;</option>';
+		$s .= '</select> ';
+                $s .= $langs->trans('ProspectCustomer');
+                $s .= ': <select name="filter_client" class="flat">';
+                $s .= '<option value="-1">&nbsp;</option>';
                 if (empty($conf->global->SOCIETE_DISABLE_PROSPECTS))
                 {
-                    $s.= '<option value="2">'.$langs->trans('Prospect').'</option>';
+                    $s .= '<option value="2">'.$langs->trans('Prospect').'</option>';
                 }
                 if (empty($conf->global->SOCIETE_DISABLE_PROSPECTS) && empty($conf->global->SOCIETE_DISABLE_CUSTOMERS) && empty($conf->global->SOCIETE_DISABLE_PROSPECTSCUSTOMERS)) {
-                    $s.= '<option value="3">'.$langs->trans('ProspectCustomer').'</option>';
+                    $s .= '<option value="3">'.$langs->trans('ProspectCustomer').'</option>';
                 }
                 if (empty($conf->global->SOCIETE_DISABLE_CUSTOMERS)) {
-                    $s.= '<option value="1">'.$langs->trans('Customer').'</option>';
+                    $s .= '<option value="1">'.$langs->trans('Customer').'</option>';
                 }
-                $s.= '<option value="0">'.$langs->trans('NorProspectNorCustomer').'</option>';
+                $s .= '<option value="0">'.$langs->trans('NorProspectNorCustomer').'</option>';
 
-                $s.='</select> ';
+                $s .= '</select> ';
 
-                $s.=$langs->trans("Status");
-                $s.=': <select name="filter_status" class="flat">';
-                $s.='<option value="-1">&nbsp;</option>';
-                $s.='<option value="1" selected>'.$langs->trans("Enabled").'</option>';
-                $s.='<option value="0">'.$langs->trans("Disabled").'</option>';
-		$s.='</select>';
+                $s .= $langs->trans("Status");
+                $s .= ': <select name="filter_status" class="flat">';
+                $s .= '<option value="-1">&nbsp;</option>';
+                $s .= '<option value="1" selected>'.$langs->trans("Enabled").'</option>';
+                $s .= '<option value="0">'.$langs->trans("Disabled").'</option>';
+		$s .= '</select>';
 		return $s;
 	}
 
@@ -321,6 +321,6 @@ 
 	 */
 	function url($id)
 	{
-		return '<a href="'.DOL_URL_ROOT.'/societe/card.php?socid='.$id.'">'.img_object('',"company").'</a>';
+		return '<a href="'.DOL_URL_ROOT.'/societe/card.php?socid='.$id.'">'.img_object('', "company").'</a>';
 	}
 }

Braces +26 added lines, -22 removed lines

@@ -74,8 +74,7 @@ 
 		    $sql.= " WHERE s.email <> ''";
 		    $sql.= " AND s.entity IN (".getEntity('societe').")";
 		    $sql.= " AND s.email NOT IN (SELECT email FROM ".MAIN_DB_PREFIX."mailing_cibles WHERE fk_mailing=".$mailing_id.")";
-		}
-		else
+		} else
 		{
 		    $sql = "SELECT s.rowid as id, s.email as email, s.nom as name, null as fk_contact, null as firstname, c.label as label";
 		    $sql.= " FROM ".MAIN_DB_PREFIX."societe as s, ".MAIN_DB_PREFIX."categorie_societe as cs, ".MAIN_DB_PREFIX."categorie as c";
@@ -104,20 +103,16 @@ 
             if ($_POST["filter_client"] == 0)
             {
                 $addDescription.= $langs->trans('NorProspectNorCustomer');
-            }
-            elseif ($_POST["filter_client"] == 1)
+            } elseif ($_POST["filter_client"] == 1)
             {
                 $addDescription.= $langs->trans('Customer');
-            }
-            elseif ($_POST["filter_client"] == 2)
+            } elseif ($_POST["filter_client"] == 2)
             {
                 $addDescription.= $langs->trans('Prospect');
-            }
-            elseif ($_POST["filter_client"] == 3)
+            } elseif ($_POST["filter_client"] == 3)
             {
                 $addDescription.= $langs->trans('ProspectCustomer');
-            }
-            else
+            } else
             {
                 $addDescription.= "Unknown status ".$_POST["filter_client"];
             }
@@ -133,8 +128,7 @@ 
             {
                 $sql.= " AND s.status=1";
                 $addDescription.= $langs->trans("Enabled");
-            }
-            else
+            } else
             {
                 $sql.= " AND s.status=0";
                 $addDescription.= $langs->trans("Disabled");
@@ -180,8 +174,7 @@ 
 
 				$i++;
 			}
-		}
-		else
+		} else
 		{
 			dol_syslog($this->db->error());
 			$this->error=$this->db->error();
@@ -261,10 +254,16 @@ 
 		{
 			$num = $this->db->num_rows($resql);
 
-			if (empty($conf->categorie->enabled)) $num=0;	// Force empty list if category module is not enabled
+			if (empty($conf->categorie->enabled)) {
+			    $num=0;
+			}
+			// Force empty list if category module is not enabled
 
-			if ($num) $s.='<option value="0">&nbsp;</option>';
-			else $s.='<option value="0">'.$langs->trans("ContactsAllShort").'</option>';
+			if ($num) {
+			    $s.='<option value="0">&nbsp;</option>';
+			} else {
+			    $s.='<option value="0">'.$langs->trans("ContactsAllShort").'</option>';
+			}
 
 			$i = 0;
 			while ($i < $num)
@@ -272,15 +271,20 @@ 
 				$obj = $this->db->fetch_object($resql);
 
 				$type='';
-				if ($obj->type == 1) $type=$langs->trans("Supplier");
-				if ($obj->type == 2) $type=$langs->trans("Customer");
+				if ($obj->type == 1) {
+				    $type=$langs->trans("Supplier");
+				}
+				if ($obj->type == 2) {
+				    $type=$langs->trans("Customer");
+				}
 				$s.='<option value="'.$obj->rowid.'">'.dol_trunc($obj->label,38,'middle');
-				if ($type) $s.=' ('.$type.')';
+				if ($type) {
+				    $s.=' ('.$type.')';
+				}
 				$s.='</option>';
 				$i++;
 			}
-		}
-		else
+		} else
 		{
 			dol_print_error($this->db);
 		}

dolibarr/htdocs/core/modules/mailings/advthirdparties.modules.php

Indentation +263 added lines, -263 removed lines

@@ -25,278 +25,278 @@
  */
 class mailing_advthirdparties extends MailingTargets
 {
-	var $name='ThirdPartyAdvancedTargeting';
-	// This label is used if no translation is found for key XXX neither MailingModuleDescXXX where XXX=name is found
-	var $desc="Third parties";
-	var $require_admin=0;
+    var $name='ThirdPartyAdvancedTargeting';
+    // This label is used if no translation is found for key XXX neither MailingModuleDescXXX where XXX=name is found
+    var $desc="Third parties";
+    var $require_admin=0;
 
-	var $require_module=array("none");	// This module should not be displayed as Selector in mailling
-	var $picto='company';
+    var $require_module=array("none");	// This module should not be displayed as Selector in mailling
+    var $picto='company';
 
-	/**
+    /**
      * @var DoliDB Database handler.
      */
     public $db;
 
 
-	/**
-	 *	Constructor
-	 *
-	 *  @param		DoliDB		$db      Database handler
-	 */
-	function __construct($db)
-	{
-		$this->db=$db;
-	}
+    /**
+     *	Constructor
+     *
+     *  @param		DoliDB		$db      Database handler
+     */
+    function __construct($db)
+    {
+        $this->db=$db;
+    }
 
 
     // phpcs:disable PEAR.NamingConventions.ValidFunctionName.NotCamelCaps
-	/**
-	 *    This is the main function that returns the array of emails
-	 *
-	 *    @param	int		$mailing_id    	Id of mailing. No need to use it.
-	 *    @param	array	$socid  		Array of id soc to add
-	 *    @param	int		$type_of_target	Defined in advtargetemailing.class.php
-	 *    @param	array	$contactid 		Array of contact id to add
-	 *    @return   int 					<0 if error, number of emails added if ok
-	 */
-	function add_to_target_spec($mailing_id,$socid,$type_of_target, $contactid)
-	{
+    /**
+     *    This is the main function that returns the array of emails
+     *
+     *    @param	int		$mailing_id    	Id of mailing. No need to use it.
+     *    @param	array	$socid  		Array of id soc to add
+     *    @param	int		$type_of_target	Defined in advtargetemailing.class.php
+     *    @param	array	$contactid 		Array of contact id to add
+     *    @return   int 					<0 if error, number of emails added if ok
+     */
+    function add_to_target_spec($mailing_id,$socid,$type_of_target, $contactid)
+    {
         // phpcs:enable
-		global $conf, $langs;
-
-		dol_syslog(get_class($this)."::add_to_target socid=".var_export($socid,true).' contactid='.var_export($contactid,true));
-
-		$cibles = array();
-
-		if (($type_of_target==1) || ($type_of_target==3)) {
-			// Select the third parties from category
-			if (count($socid)>0)
-			{
-				$sql= "SELECT s.rowid as id, s.email as email, s.nom as name, null as fk_contact";
-				$sql.= " FROM ".MAIN_DB_PREFIX."societe as s LEFT OUTER JOIN ".MAIN_DB_PREFIX."societe_extrafields se ON se.fk_object=s.rowid";
-				$sql.= " WHERE s.entity IN (".getEntity('societe').")";
-				$sql.= " AND s.rowid IN (".implode(',',$socid).")";
-				$sql.= " ORDER BY email";
-
-    			// Stock recipients emails into targets table
-    			$result=$this->db->query($sql);
-    			if ($result)
-    			{
-    				$num = $this->db->num_rows($result);
-    				$i = 0;
-
-    				dol_syslog(get_class($this)."::add_to_target mailing ".$num." targets found", LOG_DEBUG);
-
-    				$old = '';
-    				while ($i < $num)
-    				{
-    					$obj = $this->db->fetch_object($result);
-
-    					if (!empty($obj->email) && filter_var($obj->email, FILTER_VALIDATE_EMAIL)) {
-    						if (!array_key_exists($obj->email, $cibles)) {
-    							$cibles[$obj->email] = array(
-    								'email' => $obj->email,
-    								'fk_contact' => $obj->fk_contact,
-    								'name' => $obj->name,
-    								'firstname' => $obj->firstname,
-    								'other' => '',
-    								'source_url' => $this->url($obj->id,'thirdparty'),
-    								'source_id' => $obj->id,
-    								'source_type' => 'thirdparty'
-    							);
-    						}
-    					}
-
-    					$i++;
-    				}
-    			}
-    			else
-    			{
-    				dol_syslog($this->db->error());
-    				$this->error=$this->db->error();
-    				return -1;
-    			}
-			}
-		}
-
-		if  (($type_of_target==1) || ($type_of_target==2) || ($type_of_target==4)) {
-			// Select the third parties from category
-			if (count($socid)>0 || count($contactid)>0)
-			{
-				$sql= "SELECT socp.rowid as id, socp.email as email, socp.lastname as lastname, socp.firstname as firstname";
-				$sql.= " FROM ".MAIN_DB_PREFIX."socpeople as socp";
-				$sql.= " WHERE socp.entity IN (".getEntity('socpeople').")";
-				if (count($contactid)>0) {
-					$sql.= " AND socp.rowid IN (".implode(',',$contactid).")";
-				}
-				if (count($socid)>0) {
-					$sql.= " AND socp.fk_soc IN (".implode(',',$socid).")";
-				}
-				$sql.= " ORDER BY email";
-
-    			// Stock recipients emails into targets table
-    			$result=$this->db->query($sql);
-    			if ($result)
-    			{
-    				$num = $this->db->num_rows($result);
-    				$i = 0;
-
-    				dol_syslog(get_class($this)."::add_to_target mailing ".$num." targets found");
-
-    				$old = '';
-    				while ($i < $num)
-    				{
-    					$obj = $this->db->fetch_object($result);
-
-    					if (!empty($obj->email) && filter_var($obj->email, FILTER_VALIDATE_EMAIL)) {
-    						if (!array_key_exists($obj->email, $cibles)) {
-    							$cibles[$obj->email] = array(
-    								'email' => $obj->email,
-    								'fk_contact' =>$obj->id,
-    								'lastname' => $obj->lastname,
-    								'firstname' => $obj->firstname,
-    								'other' => '',
-    								'source_url' => $this->url($obj->id,'contact'),
-    								'source_id' => $obj->id,
-    								'source_type' => 'contact'
-    							);
-    						}
-    					}
-
-    					$i++;
-    				}
-    			}
-    			else
-    			{
-    				dol_syslog($this->db->error());
-    				$this->error=$this->db->error();
-    				return -1;
-    			}
-			}
-		}
-
-
-		dol_syslog(get_class($this)."::add_to_target mailing cibles=".var_export($cibles,true), LOG_DEBUG);
-		return parent::add_to_target($mailing_id, $cibles);
-	}
-
-
-	/**
-	 *	On the main mailing area, there is a box with statistics.
-	 *	If you want to add a line in this report you must provide an
-	 *	array of SQL request that returns two field:
-	 *	One called "label", One called "nb".
-	 *
-	 *	@return		array		Array with SQL requests
-	 */
-	function getSqlArrayForStats()
-	{
-		// CHANGE THIS: Optionnal
-
-		//var $statssql=array();
-		//$this->statssql[0]="SELECT field1 as label, count(distinct(email)) as nb FROM mytable WHERE email IS NOT NULL";
-		return array();
-	}
-
-
-	/**
-	 *	Return here number of distinct emails returned by your selector.
-	 *	For example if this selector is used to extract 500 different
-	 *	emails from a text file, this function must return 500.
-	 *
-	 *  @param	string	$sql 		Not use here
-	 *	@return	    int			          Nb of recipients
-	 */
-	function getNbOfRecipients($sql='')
-	{
-		global $conf;
-
-		$sql = "SELECT count(distinct(s.email)) as nb";
-		$sql.= " FROM ".MAIN_DB_PREFIX."societe as s";
-		$sql.= " WHERE s.email != ''";
-		$sql.= " AND s.entity IN (".getEntity('societe').")";
-
-		// La requete doit retourner un champ "nb" pour etre comprise
-		// par parent::getNbOfRecipients
-		return parent::getNbOfRecipients($sql);
-	}
-
-	/**
-	 *  This is to add a form filter to provide variant of selector
-	 *	If used, the HTML select must be called "filter"
-	 *
-	 *  @return     string      A html select zone
-	 */
-	function formFilter()
-	{
-		global $conf, $langs;
-
-		$langs->load("companies");
-
-		$s='';
-		$s.='<select name="filter" class="flat">';
-
-		// Show categories
-		$sql = "SELECT rowid, label, type, visible";
-		$sql.= " FROM ".MAIN_DB_PREFIX."categorie";
-		$sql.= " WHERE type in (1,2)";	// We keep only categories for suppliers and customers/prospects
-		// $sql.= " AND visible > 0";	// We ignore the property visible because third party's categories does not use this property (only products categories use it).
-		$sql.= " AND entity = ".$conf->entity;
-		$sql.= " ORDER BY label";
-
-		//print $sql;
-		$resql = $this->db->query($sql);
-		if ($resql)
-		{
-			$num = $this->db->num_rows($resql);
-
-			if (empty($conf->categorie->enabled)) $num=0;	// Force empty list if category module is not enabled
-
-			if ($num) $s.='<option value="0">&nbsp;</option>';
-			else $s.='<option value="0">'.$langs->trans("ContactsAllShort").'</option>';
-
-			$i = 0;
-			while ($i < $num)
-			{
-				$obj = $this->db->fetch_object($resql);
-
-				$type='';
-				if ($obj->type == 1) $type=$langs->trans("Supplier");
-				if ($obj->type == 2) $type=$langs->trans("Customer");
-				$s.='<option value="'.$obj->rowid.'">'.dol_trunc($obj->label,38,'middle');
-				if ($type) $s.=' ('.$type.')';
-				$s.='</option>';
-				$i++;
-			}
-		}
-		else
-		{
-			dol_print_error($this->db);
-		}
-
-		$s.='</select>';
-		return $s;
-	}
-
-
-	/**
-	 *  Can include an URL link on each record provided by selector shown on target page.
-	 *
-	 *  @param	int		$id		ID
-	 *  @param	string		$type	type
-	 *  @return string      	Url link
-	 */
-	function url($id,$type)
-	{
-		if ($type=='thirdparty') {
-			$companystatic=new Societe($this->db);
-			$companystatic->fetch($id);
-			return $companystatic->getNomUrl(0, '', 0, 1);
-		} elseif ($type=='contact') {
-			$contactstatic=new Contact($this->db);
-			$contactstatic->fetch($id);
-			return $contactstatic->getNomUrl(0, '', 0, '', -1, 1);
-		}
-	}
+        global $conf, $langs;
+
+        dol_syslog(get_class($this)."::add_to_target socid=".var_export($socid,true).' contactid='.var_export($contactid,true));
+
+        $cibles = array();
+
+        if (($type_of_target==1) || ($type_of_target==3)) {
+            // Select the third parties from category
+            if (count($socid)>0)
+            {
+                $sql= "SELECT s.rowid as id, s.email as email, s.nom as name, null as fk_contact";
+                $sql.= " FROM ".MAIN_DB_PREFIX."societe as s LEFT OUTER JOIN ".MAIN_DB_PREFIX."societe_extrafields se ON se.fk_object=s.rowid";
+                $sql.= " WHERE s.entity IN (".getEntity('societe').")";
+                $sql.= " AND s.rowid IN (".implode(',',$socid).")";
+                $sql.= " ORDER BY email";
+
+                // Stock recipients emails into targets table
+                $result=$this->db->query($sql);
+                if ($result)
+                {
+                    $num = $this->db->num_rows($result);
+                    $i = 0;
+
+                    dol_syslog(get_class($this)."::add_to_target mailing ".$num." targets found", LOG_DEBUG);
+
+                    $old = '';
+                    while ($i < $num)
+                    {
+                        $obj = $this->db->fetch_object($result);
+
+                        if (!empty($obj->email) && filter_var($obj->email, FILTER_VALIDATE_EMAIL)) {
+                            if (!array_key_exists($obj->email, $cibles)) {
+                                $cibles[$obj->email] = array(
+                                    'email' => $obj->email,
+                                    'fk_contact' => $obj->fk_contact,
+                                    'name' => $obj->name,
+                                    'firstname' => $obj->firstname,
+                                    'other' => '',
+                                    'source_url' => $this->url($obj->id,'thirdparty'),
+                                    'source_id' => $obj->id,
+                                    'source_type' => 'thirdparty'
+                                );
+                            }
+                        }
+
+                        $i++;
+                    }
+                }
+                else
+                {
+                    dol_syslog($this->db->error());
+                    $this->error=$this->db->error();
+                    return -1;
+                }
+            }
+        }
+
+        if  (($type_of_target==1) || ($type_of_target==2) || ($type_of_target==4)) {
+            // Select the third parties from category
+            if (count($socid)>0 || count($contactid)>0)
+            {
+                $sql= "SELECT socp.rowid as id, socp.email as email, socp.lastname as lastname, socp.firstname as firstname";
+                $sql.= " FROM ".MAIN_DB_PREFIX."socpeople as socp";
+                $sql.= " WHERE socp.entity IN (".getEntity('socpeople').")";
+                if (count($contactid)>0) {
+                    $sql.= " AND socp.rowid IN (".implode(',',$contactid).")";
+                }
+                if (count($socid)>0) {
+                    $sql.= " AND socp.fk_soc IN (".implode(',',$socid).")";
+                }
+                $sql.= " ORDER BY email";
+
+                // Stock recipients emails into targets table
+                $result=$this->db->query($sql);
+                if ($result)
+                {
+                    $num = $this->db->num_rows($result);
+                    $i = 0;
+
+                    dol_syslog(get_class($this)."::add_to_target mailing ".$num." targets found");
+
+                    $old = '';
+                    while ($i < $num)
+                    {
+                        $obj = $this->db->fetch_object($result);
+
+                        if (!empty($obj->email) && filter_var($obj->email, FILTER_VALIDATE_EMAIL)) {
+                            if (!array_key_exists($obj->email, $cibles)) {
+                                $cibles[$obj->email] = array(
+                                    'email' => $obj->email,
+                                    'fk_contact' =>$obj->id,
+                                    'lastname' => $obj->lastname,
+                                    'firstname' => $obj->firstname,
+                                    'other' => '',
+                                    'source_url' => $this->url($obj->id,'contact'),
+                                    'source_id' => $obj->id,
+                                    'source_type' => 'contact'
+                                );
+                            }
+                        }
+
+                        $i++;
+                    }
+                }
+                else
+                {
+                    dol_syslog($this->db->error());
+                    $this->error=$this->db->error();
+                    return -1;
+                }
+            }
+        }
+
+
+        dol_syslog(get_class($this)."::add_to_target mailing cibles=".var_export($cibles,true), LOG_DEBUG);
+        return parent::add_to_target($mailing_id, $cibles);
+    }
+
+
+    /**
+     *	On the main mailing area, there is a box with statistics.
+     *	If you want to add a line in this report you must provide an
+     *	array of SQL request that returns two field:
+     *	One called "label", One called "nb".
+     *
+     *	@return		array		Array with SQL requests
+     */
+    function getSqlArrayForStats()
+    {
+        // CHANGE THIS: Optionnal
+
+        //var $statssql=array();
+        //$this->statssql[0]="SELECT field1 as label, count(distinct(email)) as nb FROM mytable WHERE email IS NOT NULL";
+        return array();
+    }
+
+
+    /**
+     *	Return here number of distinct emails returned by your selector.
+     *	For example if this selector is used to extract 500 different
+     *	emails from a text file, this function must return 500.
+     *
+     *  @param	string	$sql 		Not use here
+     *	@return	    int			          Nb of recipients
+     */
+    function getNbOfRecipients($sql='')
+    {
+        global $conf;
+
+        $sql = "SELECT count(distinct(s.email)) as nb";
+        $sql.= " FROM ".MAIN_DB_PREFIX."societe as s";
+        $sql.= " WHERE s.email != ''";
+        $sql.= " AND s.entity IN (".getEntity('societe').")";
+
+        // La requete doit retourner un champ "nb" pour etre comprise
+        // par parent::getNbOfRecipients
+        return parent::getNbOfRecipients($sql);
+    }
+
+    /**
+     *  This is to add a form filter to provide variant of selector
+     *	If used, the HTML select must be called "filter"
+     *
+     *  @return     string      A html select zone
+     */
+    function formFilter()
+    {
+        global $conf, $langs;
+
+        $langs->load("companies");
+
+        $s='';
+        $s.='<select name="filter" class="flat">';
+
+        // Show categories
+        $sql = "SELECT rowid, label, type, visible";
+        $sql.= " FROM ".MAIN_DB_PREFIX."categorie";
+        $sql.= " WHERE type in (1,2)";	// We keep only categories for suppliers and customers/prospects
+        // $sql.= " AND visible > 0";	// We ignore the property visible because third party's categories does not use this property (only products categories use it).
+        $sql.= " AND entity = ".$conf->entity;
+        $sql.= " ORDER BY label";
+
+        //print $sql;
+        $resql = $this->db->query($sql);
+        if ($resql)
+        {
+            $num = $this->db->num_rows($resql);
+
+            if (empty($conf->categorie->enabled)) $num=0;	// Force empty list if category module is not enabled
+
+            if ($num) $s.='<option value="0">&nbsp;</option>';
+            else $s.='<option value="0">'.$langs->trans("ContactsAllShort").'</option>';
+
+            $i = 0;
+            while ($i < $num)
+            {
+                $obj = $this->db->fetch_object($resql);
+
+                $type='';
+                if ($obj->type == 1) $type=$langs->trans("Supplier");
+                if ($obj->type == 2) $type=$langs->trans("Customer");
+                $s.='<option value="'.$obj->rowid.'">'.dol_trunc($obj->label,38,'middle');
+                if ($type) $s.=' ('.$type.')';
+                $s.='</option>';
+                $i++;
+            }
+        }
+        else
+        {
+            dol_print_error($this->db);
+        }
+
+        $s.='</select>';
+        return $s;
+    }
+
+
+    /**
+     *  Can include an URL link on each record provided by selector shown on target page.
+     *
+     *  @param	int		$id		ID
+     *  @param	string		$type	type
+     *  @return string      	Url link
+     */
+    function url($id,$type)
+    {
+        if ($type=='thirdparty') {
+            $companystatic=new Societe($this->db);
+            $companystatic->fetch($id);
+            return $companystatic->getNomUrl(0, '', 0, 1);
+        } elseif ($type=='contact') {
+            $contactstatic=new Contact($this->db);
+            $contactstatic->fetch($id);
+            return $contactstatic->getNomUrl(0, '', 0, '', -1, 1);
+        }
+    }
 }

Spacing +57 added lines, -57 removed lines

@@ -25,13 +25,13 @@ 
  */
 class mailing_advthirdparties extends MailingTargets
 {
-	var $name='ThirdPartyAdvancedTargeting';
+	var $name = 'ThirdPartyAdvancedTargeting';
 	// This label is used if no translation is found for key XXX neither MailingModuleDescXXX where XXX=name is found
-	var $desc="Third parties";
-	var $require_admin=0;
+	var $desc = "Third parties";
+	var $require_admin = 0;
 
-	var $require_module=array("none");	// This module should not be displayed as Selector in mailling
-	var $picto='company';
+	var $require_module = array("none"); // This module should not be displayed as Selector in mailling
+	var $picto = 'company';
 
 	/**
      * @var DoliDB Database handler.
@@ -46,7 +46,7 @@ 
 	 */
 	function __construct($db)
 	{
-		$this->db=$db;
+		$this->db = $db;
 	}
 
 
@@ -60,27 +60,27 @@ 
 	 *    @param	array	$contactid 		Array of contact id to add
 	 *    @return   int 					<0 if error, number of emails added if ok
 	 */
-	function add_to_target_spec($mailing_id,$socid,$type_of_target, $contactid)
+	function add_to_target_spec($mailing_id, $socid, $type_of_target, $contactid)
 	{
         // phpcs:enable
 		global $conf, $langs;
 
-		dol_syslog(get_class($this)."::add_to_target socid=".var_export($socid,true).' contactid='.var_export($contactid,true));
+		dol_syslog(get_class($this)."::add_to_target socid=".var_export($socid, true).' contactid='.var_export($contactid, true));
 
 		$cibles = array();
 
-		if (($type_of_target==1) || ($type_of_target==3)) {
+		if (($type_of_target == 1) || ($type_of_target == 3)) {
 			// Select the third parties from category
-			if (count($socid)>0)
+			if (count($socid) > 0)
 			{
-				$sql= "SELECT s.rowid as id, s.email as email, s.nom as name, null as fk_contact";
-				$sql.= " FROM ".MAIN_DB_PREFIX."societe as s LEFT OUTER JOIN ".MAIN_DB_PREFIX."societe_extrafields se ON se.fk_object=s.rowid";
-				$sql.= " WHERE s.entity IN (".getEntity('societe').")";
-				$sql.= " AND s.rowid IN (".implode(',',$socid).")";
-				$sql.= " ORDER BY email";
+				$sql = "SELECT s.rowid as id, s.email as email, s.nom as name, null as fk_contact";
+				$sql .= " FROM ".MAIN_DB_PREFIX."societe as s LEFT OUTER JOIN ".MAIN_DB_PREFIX."societe_extrafields se ON se.fk_object=s.rowid";
+				$sql .= " WHERE s.entity IN (".getEntity('societe').")";
+				$sql .= " AND s.rowid IN (".implode(',', $socid).")";
+				$sql .= " ORDER BY email";
 
     			// Stock recipients emails into targets table
-    			$result=$this->db->query($sql);
+    			$result = $this->db->query($sql);
     			if ($result)
     			{
     				$num = $this->db->num_rows($result);
@@ -101,7 +101,7 @@ 
     								'name' => $obj->name,
     								'firstname' => $obj->firstname,
     								'other' => '',
-    								'source_url' => $this->url($obj->id,'thirdparty'),
+    								'source_url' => $this->url($obj->id, 'thirdparty'),
     								'source_id' => $obj->id,
     								'source_type' => 'thirdparty'
     							);
@@ -114,29 +114,29 @@ 
     			else
     			{
     				dol_syslog($this->db->error());
-    				$this->error=$this->db->error();
+    				$this->error = $this->db->error();
     				return -1;
     			}
 			}
 		}
 
-		if  (($type_of_target==1) || ($type_of_target==2) || ($type_of_target==4)) {
+		if (($type_of_target == 1) || ($type_of_target == 2) || ($type_of_target == 4)) {
 			// Select the third parties from category
-			if (count($socid)>0 || count($contactid)>0)
+			if (count($socid) > 0 || count($contactid) > 0)
 			{
-				$sql= "SELECT socp.rowid as id, socp.email as email, socp.lastname as lastname, socp.firstname as firstname";
-				$sql.= " FROM ".MAIN_DB_PREFIX."socpeople as socp";
-				$sql.= " WHERE socp.entity IN (".getEntity('socpeople').")";
-				if (count($contactid)>0) {
-					$sql.= " AND socp.rowid IN (".implode(',',$contactid).")";
+				$sql = "SELECT socp.rowid as id, socp.email as email, socp.lastname as lastname, socp.firstname as firstname";
+				$sql .= " FROM ".MAIN_DB_PREFIX."socpeople as socp";
+				$sql .= " WHERE socp.entity IN (".getEntity('socpeople').")";
+				if (count($contactid) > 0) {
+					$sql .= " AND socp.rowid IN (".implode(',', $contactid).")";
 				}
-				if (count($socid)>0) {
-					$sql.= " AND socp.fk_soc IN (".implode(',',$socid).")";
+				if (count($socid) > 0) {
+					$sql .= " AND socp.fk_soc IN (".implode(',', $socid).")";
 				}
-				$sql.= " ORDER BY email";
+				$sql .= " ORDER BY email";
 
     			// Stock recipients emails into targets table
-    			$result=$this->db->query($sql);
+    			$result = $this->db->query($sql);
     			if ($result)
     			{
     				$num = $this->db->num_rows($result);
@@ -157,7 +157,7 @@ 
     								'lastname' => $obj->lastname,
     								'firstname' => $obj->firstname,
     								'other' => '',
-    								'source_url' => $this->url($obj->id,'contact'),
+    								'source_url' => $this->url($obj->id, 'contact'),
     								'source_id' => $obj->id,
     								'source_type' => 'contact'
     							);
@@ -170,14 +170,14 @@ 
     			else
     			{
     				dol_syslog($this->db->error());
-    				$this->error=$this->db->error();
+    				$this->error = $this->db->error();
     				return -1;
     			}
 			}
 		}
 
 
-		dol_syslog(get_class($this)."::add_to_target mailing cibles=".var_export($cibles,true), LOG_DEBUG);
+		dol_syslog(get_class($this)."::add_to_target mailing cibles=".var_export($cibles, true), LOG_DEBUG);
 		return parent::add_to_target($mailing_id, $cibles);
 	}
 
@@ -208,14 +208,14 @@ 
 	 *  @param	string	$sql 		Not use here
 	 *	@return	    int			          Nb of recipients
 	 */
-	function getNbOfRecipients($sql='')
+	function getNbOfRecipients($sql = '')
 	{
 		global $conf;
 
 		$sql = "SELECT count(distinct(s.email)) as nb";
-		$sql.= " FROM ".MAIN_DB_PREFIX."societe as s";
-		$sql.= " WHERE s.email != ''";
-		$sql.= " AND s.entity IN (".getEntity('societe').")";
+		$sql .= " FROM ".MAIN_DB_PREFIX."societe as s";
+		$sql .= " WHERE s.email != ''";
+		$sql .= " AND s.entity IN (".getEntity('societe').")";
 
 		// La requete doit retourner un champ "nb" pour etre comprise
 		// par parent::getNbOfRecipients
@@ -234,16 +234,16 @@ 
 
 		$langs->load("companies");
 
-		$s='';
-		$s.='<select name="filter" class="flat">';
+		$s = '';
+		$s .= '<select name="filter" class="flat">';
 
 		// Show categories
 		$sql = "SELECT rowid, label, type, visible";
-		$sql.= " FROM ".MAIN_DB_PREFIX."categorie";
-		$sql.= " WHERE type in (1,2)";	// We keep only categories for suppliers and customers/prospects
+		$sql .= " FROM ".MAIN_DB_PREFIX."categorie";
+		$sql .= " WHERE type in (1,2)"; // We keep only categories for suppliers and customers/prospects
 		// $sql.= " AND visible > 0";	// We ignore the property visible because third party's categories does not use this property (only products categories use it).
-		$sql.= " AND entity = ".$conf->entity;
-		$sql.= " ORDER BY label";
+		$sql .= " AND entity = ".$conf->entity;
+		$sql .= " ORDER BY label";
 
 		//print $sql;
 		$resql = $this->db->query($sql);
@@ -251,22 +251,22 @@ 
 		{
 			$num = $this->db->num_rows($resql);
 
-			if (empty($conf->categorie->enabled)) $num=0;	// Force empty list if category module is not enabled
+			if (empty($conf->categorie->enabled)) $num = 0; // Force empty list if category module is not enabled
 
-			if ($num) $s.='<option value="0">&nbsp;</option>';
-			else $s.='<option value="0">'.$langs->trans("ContactsAllShort").'</option>';
+			if ($num) $s .= '<option value="0">&nbsp;</option>';
+			else $s .= '<option value="0">'.$langs->trans("ContactsAllShort").'</option>';
 
 			$i = 0;
 			while ($i < $num)
 			{
 				$obj = $this->db->fetch_object($resql);
 
-				$type='';
-				if ($obj->type == 1) $type=$langs->trans("Supplier");
-				if ($obj->type == 2) $type=$langs->trans("Customer");
-				$s.='<option value="'.$obj->rowid.'">'.dol_trunc($obj->label,38,'middle');
-				if ($type) $s.=' ('.$type.')';
-				$s.='</option>';
+				$type = '';
+				if ($obj->type == 1) $type = $langs->trans("Supplier");
+				if ($obj->type == 2) $type = $langs->trans("Customer");
+				$s .= '<option value="'.$obj->rowid.'">'.dol_trunc($obj->label, 38, 'middle');
+				if ($type) $s .= ' ('.$type.')';
+				$s .= '</option>';
 				$i++;
 			}
 		}
@@ -275,7 +275,7 @@ 
 			dol_print_error($this->db);
 		}
 
-		$s.='</select>';
+		$s .= '</select>';
 		return $s;
 	}
 
@@ -287,14 +287,14 @@ 
 	 *  @param	string		$type	type
 	 *  @return string      	Url link
 	 */
-	function url($id,$type)
+	function url($id, $type)
 	{
-		if ($type=='thirdparty') {
-			$companystatic=new Societe($this->db);
+		if ($type == 'thirdparty') {
+			$companystatic = new Societe($this->db);
 			$companystatic->fetch($id);
 			return $companystatic->getNomUrl(0, '', 0, 1);
-		} elseif ($type=='contact') {
-			$contactstatic=new Contact($this->db);
+		} elseif ($type == 'contact') {
+			$contactstatic = new Contact($this->db);
 			$contactstatic->fetch($id);
 			return $contactstatic->getNomUrl(0, '', 0, '', -1, 1);
 		}

Braces +21 added lines, -12 removed lines

@@ -110,8 +110,7 @@ 
 
     					$i++;
     				}
-    			}
-    			else
+    			} else
     			{
     				dol_syslog($this->db->error());
     				$this->error=$this->db->error();
@@ -166,8 +165,7 @@ 
 
     					$i++;
     				}
-    			}
-    			else
+    			} else
     			{
     				dol_syslog($this->db->error());
     				$this->error=$this->db->error();
@@ -251,10 +249,16 @@ 
 		{
 			$num = $this->db->num_rows($resql);
 
-			if (empty($conf->categorie->enabled)) $num=0;	// Force empty list if category module is not enabled
+			if (empty($conf->categorie->enabled)) {
+			    $num=0;
+			}
+			// Force empty list if category module is not enabled
 
-			if ($num) $s.='<option value="0">&nbsp;</option>';
-			else $s.='<option value="0">'.$langs->trans("ContactsAllShort").'</option>';
+			if ($num) {
+			    $s.='<option value="0">&nbsp;</option>';
+			} else {
+			    $s.='<option value="0">'.$langs->trans("ContactsAllShort").'</option>';
+			}
 
 			$i = 0;
 			while ($i < $num)
@@ -262,15 +266,20 @@ 
 				$obj = $this->db->fetch_object($resql);
 
 				$type='';
-				if ($obj->type == 1) $type=$langs->trans("Supplier");
-				if ($obj->type == 2) $type=$langs->trans("Customer");
+				if ($obj->type == 1) {
+				    $type=$langs->trans("Supplier");
+				}
+				if ($obj->type == 2) {
+				    $type=$langs->trans("Customer");
+				}
 				$s.='<option value="'.$obj->rowid.'">'.dol_trunc($obj->label,38,'middle');
-				if ($type) $s.=' ('.$type.')';
+				if ($type) {
+				    $s.=' ('.$type.')';
+				}
 				$s.='</option>';
 				$i++;
 			}
-		}
-		else
+		} else
 		{
 			dol_print_error($this->db);
 		}

dolibarr/htdocs/core/modules/mailings/xinputuser.modules.php

Indentation +112 added lines, -112 removed lines

@@ -30,133 +30,133 @@
  */
 class mailing_xinputuser extends MailingTargets
 {
-	var $name='EmailsFromUser';              // Identifiant du module mailing
-	// This label is used if no translation is found for key XXX neither MailingModuleDescXXX where XXX=name is found
-	var $desc='EMails input by user';        // Libelle utilise si aucune traduction pour MailingModuleDescXXX ou XXX=name trouv�e
-	var $require_module=array();            // Module mailing actif si modules require_module actifs
-	var $require_admin=0;                    // Module mailing actif pour user admin ou non
-	var $picto='generic';
-	var $tooltip='UseFormatInputEmailToTarget';
-
-
-	/**
-	 *	Constructor
-	 *
-	 *  @param		DoliDB		$db      Database handler
-	 */
-	function __construct($db)
-	{
-		$this->db=$db;
-	}
+    var $name='EmailsFromUser';              // Identifiant du module mailing
+    // This label is used if no translation is found for key XXX neither MailingModuleDescXXX where XXX=name is found
+    var $desc='EMails input by user';        // Libelle utilise si aucune traduction pour MailingModuleDescXXX ou XXX=name trouv�e
+    var $require_module=array();            // Module mailing actif si modules require_module actifs
+    var $require_admin=0;                    // Module mailing actif pour user admin ou non
+    var $picto='generic';
+    var $tooltip='UseFormatInputEmailToTarget';
 
 
     /**
-	 *	On the main mailing area, there is a box with statistics.
-	 *	If you want to add a line in this report you must provide an
-	 *	array of SQL request that returns two field:
-	 *	One called "label", One called "nb".
-	 *
-	 *	@return		array		Array with SQL requests
-	 */
-	function getSqlArrayForStats()
-	{
-		global $langs;
-		$langs->load("users");
-
-		$statssql=array();
-		return $statssql;
-	}
-
-
-	/**
-	 *	Return here number of distinct emails returned by your selector.
-	 *	For example if this selector is used to extract 500 different
-	 *	emails from a text file, this function must return 500.
-	 *
-	 *  @param      string	$sql   	Sql request to count
-	 *	@return		string			'' means NA
-	 */
-	function getNbOfRecipients($sql='')
-	{
-		return '';
-	}
-
-
-	/**
-	 *  Renvoie url lien vers fiche de la source du destinataire du mailing
-	 *
+     *	Constructor
+     *
+     *  @param		DoliDB		$db      Database handler
+     */
+    function __construct($db)
+    {
+        $this->db=$db;
+    }
+
+
+    /**
+     *	On the main mailing area, there is a box with statistics.
+     *	If you want to add a line in this report you must provide an
+     *	array of SQL request that returns two field:
+     *	One called "label", One called "nb".
+     *
+     *	@return		array		Array with SQL requests
+     */
+    function getSqlArrayForStats()
+    {
+        global $langs;
+        $langs->load("users");
+
+        $statssql=array();
+        return $statssql;
+    }
+
+
+    /**
+     *	Return here number of distinct emails returned by your selector.
+     *	For example if this selector is used to extract 500 different
+     *	emails from a text file, this function must return 500.
+     *
+     *  @param      string	$sql   	Sql request to count
+     *	@return		string			'' means NA
+     */
+    function getNbOfRecipients($sql='')
+    {
+        return '';
+    }
+
+
+    /**
+     *  Renvoie url lien vers fiche de la source du destinataire du mailing
+     *
      *  @param	int		$id		ID
-	 *  @return string      	Url lien
-	 */
-	function url($id)
-	{
-		return '';
-	}
-
-
-	/**
-	 *   Affiche formulaire de filtre qui apparait dans page de selection des destinataires de mailings
-	 *
-	 *   @return     string      Retourne zone select
-	 */
-	function formFilter()
-	{
-		global $langs;
-
-		$s='';
-		$s.='<input type="text" name="xinputuser" class="flat minwidth300" value="'.GETPOST("xinputuser").'">';
-		return $s;
-	}
+     *  @return string      	Url lien
+     */
+    function url($id)
+    {
+        return '';
+    }
+
+
+    /**
+     *   Affiche formulaire de filtre qui apparait dans page de selection des destinataires de mailings
+     *
+     *   @return     string      Retourne zone select
+     */
+    function formFilter()
+    {
+        global $langs;
+
+        $s='';
+        $s.='<input type="text" name="xinputuser" class="flat minwidth300" value="'.GETPOST("xinputuser").'">';
+        return $s;
+    }
 
     // phpcs:disable PEAR.NamingConventions.ValidFunctionName.NotCamelCaps
-	/**
-	 *  Ajoute destinataires dans table des cibles
-	 *
-	 *  @param	int		$mailing_id    	Id of emailing
-	 *  @return int           			< 0 si erreur, nb ajout si ok
-	 */
-	function add_to_target($mailing_id)
-	{
+    /**
+     *  Ajoute destinataires dans table des cibles
+     *
+     *  @param	int		$mailing_id    	Id of emailing
+     *  @return int           			< 0 si erreur, nb ajout si ok
+     */
+    function add_to_target($mailing_id)
+    {
         // phpcs:enable
-		global $conf,$langs,$_FILES;
+        global $conf,$langs,$_FILES;
 
-		require_once DOL_DOCUMENT_ROOT.'/core/lib/files.lib.php';
+        require_once DOL_DOCUMENT_ROOT.'/core/lib/files.lib.php';
 
-		$tmparray=explode(';',GETPOST('xinputuser'));
-		$email=$tmparray[0];
-		$lastname=$tmparray[1];
-		$firstname=$tmparray[2];
-		$other=$tmparray[3];
+        $tmparray=explode(';',GETPOST('xinputuser'));
+        $email=$tmparray[0];
+        $lastname=$tmparray[1];
+        $firstname=$tmparray[2];
+        $other=$tmparray[3];
 
-		$cibles=array();
+        $cibles=array();
         if (! empty($email))
         {
-			if (isValidEMail($email))
-			{
-				$cibles[] = array(
-           			'email' => $email,
-           			'lastname' => $lastname,
-           			'firstname' => $firstname,
-					'other' => $other,
+            if (isValidEMail($email))
+            {
+                $cibles[] = array(
+                        'email' => $email,
+                        'lastname' => $lastname,
+                        'firstname' => $firstname,
+                    'other' => $other,
                     'source_url' => '',
                     'source_id' => '',
                     'source_type' => 'file'
-				);
-
-				return parent::add_to_target($mailing_id, $cibles);
-			}
-			else
-			{
-				$langs->load("errors");
-				$this->error = $langs->trans("ErrorBadEMail",$email);
-				return -1;
-			}
-		}
-		else
-		{
+                );
+
+                return parent::add_to_target($mailing_id, $cibles);
+            }
+            else
+            {
+                $langs->load("errors");
+                $this->error = $langs->trans("ErrorBadEMail",$email);
+                return -1;
+            }
+        }
+        else
+        {
             $langs->load("errors");
             $this->error = $langs->trans("ErrorBadEmail",$email);
-			return -1;
-		}
-	}
+            return -1;
+        }
+    }
 }

Spacing +21 added lines, -21 removed lines

@@ -30,13 +30,13 @@ 
  */
 class mailing_xinputuser extends MailingTargets
 {
-	var $name='EmailsFromUser';              // Identifiant du module mailing
+	var $name = 'EmailsFromUser'; // Identifiant du module mailing
 	// This label is used if no translation is found for key XXX neither MailingModuleDescXXX where XXX=name is found
-	var $desc='EMails input by user';        // Libelle utilise si aucune traduction pour MailingModuleDescXXX ou XXX=name trouv�e
-	var $require_module=array();            // Module mailing actif si modules require_module actifs
-	var $require_admin=0;                    // Module mailing actif pour user admin ou non
-	var $picto='generic';
-	var $tooltip='UseFormatInputEmailToTarget';
+	var $desc = 'EMails input by user'; // Libelle utilise si aucune traduction pour MailingModuleDescXXX ou XXX=name trouv�e
+	var $require_module = array(); // Module mailing actif si modules require_module actifs
+	var $require_admin = 0; // Module mailing actif pour user admin ou non
+	var $picto = 'generic';
+	var $tooltip = 'UseFormatInputEmailToTarget';
 
 
 	/**
@@ -46,7 +46,7 @@ 
 	 */
 	function __construct($db)
 	{
-		$this->db=$db;
+		$this->db = $db;
 	}
 
 
@@ -63,7 +63,7 @@ 
 		global $langs;
 		$langs->load("users");
 
-		$statssql=array();
+		$statssql = array();
 		return $statssql;
 	}
 
@@ -76,7 +76,7 @@ 
 	 *  @param      string	$sql   	Sql request to count
 	 *	@return		string			'' means NA
 	 */
-	function getNbOfRecipients($sql='')
+	function getNbOfRecipients($sql = '')
 	{
 		return '';
 	}
@@ -103,8 +103,8 @@ 
 	{
 		global $langs;
 
-		$s='';
-		$s.='<input type="text" name="xinputuser" class="flat minwidth300" value="'.GETPOST("xinputuser").'">';
+		$s = '';
+		$s .= '<input type="text" name="xinputuser" class="flat minwidth300" value="'.GETPOST("xinputuser").'">';
 		return $s;
 	}
 
@@ -118,18 +118,18 @@ 
 	function add_to_target($mailing_id)
 	{
         // phpcs:enable
-		global $conf,$langs,$_FILES;
+		global $conf, $langs, $_FILES;
 
 		require_once DOL_DOCUMENT_ROOT.'/core/lib/files.lib.php';
 
-		$tmparray=explode(';',GETPOST('xinputuser'));
-		$email=$tmparray[0];
-		$lastname=$tmparray[1];
-		$firstname=$tmparray[2];
-		$other=$tmparray[3];
+		$tmparray = explode(';', GETPOST('xinputuser'));
+		$email = $tmparray[0];
+		$lastname = $tmparray[1];
+		$firstname = $tmparray[2];
+		$other = $tmparray[3];
 
-		$cibles=array();
-        if (! empty($email))
+		$cibles = array();
+        if (!empty($email))
         {
 			if (isValidEMail($email))
 			{
@@ -148,14 +148,14 @@ 
 			else
 			{
 				$langs->load("errors");
-				$this->error = $langs->trans("ErrorBadEMail",$email);
+				$this->error = $langs->trans("ErrorBadEMail", $email);
 				return -1;
 			}
 		}
 		else
 		{
             $langs->load("errors");
-            $this->error = $langs->trans("ErrorBadEmail",$email);
+            $this->error = $langs->trans("ErrorBadEmail", $email);
 			return -1;
 		}
 	}

Braces +2 added lines, -4 removed lines

@@ -144,15 +144,13 @@
 				);
 
 				return parent::add_to_target($mailing_id, $cibles);
-			}
-			else
+			} else
 			{
 				$langs->load("errors");
 				$this->error = $langs->trans("ErrorBadEMail",$email);
 				return -1;
 			}
-		}
-		else
+		} else
 		{
             $langs->load("errors");
             $this->error = $langs->trans("ErrorBadEmail",$email);