alxarafe / alixar

Dolibarr/Code/Bom/Classes/BOM.php

Spacing +2 added lines, -2 removed lines

@@ -1430,7 +1430,7 @@ 
             foreach ($this->lines as &$line) {
                 $tmpproduct->cost_price = 0;
                 $tmpproduct->pmp = 0;
-                $result = $tmpproduct->fetch($line->fk_product, '', '', '', 0, 1, 1);   // We discard selling price and language loading
+                $result = $tmpproduct->fetch($line->fk_product, '', '', '', 0, 1, 1); // We discard selling price and language loading
 
                 if ($tmpproduct->type == $tmpproduct::TYPE_PRODUCT) {
                     if (empty($line->fk_bom_child)) {
@@ -1625,7 +1625,7 @@ 
      */
     public function getKanbanView($option = '', $arraydata = null)
     {
-        global $db,$langs;
+        global $db, $langs;
 
         $selected = (empty($arraydata['selected']) ? 0 : $arraydata['selected']);
 

Indentation +4 added lines, -5 removed lines

@@ -437,7 +437,6 @@ 
      * Load object lines in memory from the database by type of product
      *
      *  @param int    $typeproduct   0 type product, 1 type service
-
      * @return int         Return integer <0 if KO, 0 if not found, >0 if OK
      */
     public function fetchLinesbytypeproduct($typeproduct = 0)
@@ -1206,7 +1205,7 @@ 
         return $this->LibStatut($this->status, $mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return the status
      *
@@ -1216,7 +1215,7 @@ 
      */
     public function LibStatut($status, $mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         if (empty($this->labelStatus)) {
             global $langs;
             //$langs->load("mrp");
@@ -1325,7 +1324,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return if at least one photo is available
      *
@@ -1334,7 +1333,7 @@ 
      */
     public function is_photo_available($sdir)
     {
-		// phpcs:enable
+        // phpcs:enable
         include_once DOL_DOCUMENT_ROOT . '/core/lib/files.lib.php';
         include_once DOL_DOCUMENT_ROOT . '/core/lib/images.lib.php';
 

Dolibarr/Code/Bom/Api/Boms.php

Indentation +2 added lines, -2 removed lines

@@ -716,7 +716,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      * Clean sensible object datas
      *
@@ -725,7 +725,7 @@ 
      */
     protected function _cleanObjectDatas($object)
     {
-		// phpcs:enable
+        // phpcs:enable
         $object = parent::_cleanObjectDatas($object);
 
         unset($object->rowid);

Dolibarr/Code/Asset/Classes/AssetAccountancyCodes.php

Spacing +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
                 'receivable_on_assignment' => array('label' => 'AssetAccountancyCodeReceivableOnAssignment'),
                 'proceeds_from_sales' => array('label' => 'AssetAccountancyCodeProceedsFromSales'),
                 'vat_collected' => array('label' => 'AssetAccountancyCodeVatCollected'),
-                'vat_deductible' => array('label' => 'AssetAccountancyCodeVatDeductible','column_break' => true),
+                'vat_deductible' => array('label' => 'AssetAccountancyCodeVatDeductible', 'column_break' => true),
             ),
         ),
         'accelerated_depreciation' => array(

Dolibarr/Code/Asset/Classes/AssetModel.php

Indentation +2 added lines, -2 removed lines

@@ -1438,7 +1438,7 @@ 
         return $this->LibStatut($this->status, $mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return the status
      *
@@ -1448,7 +1448,7 @@ 
      */
     public function LibStatut($status, $mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         if (empty($this->labelStatus) || empty($this->labelStatusShort)) {
             global $langs;
             //$langs->load("assets");

Dolibarr/Code/Asset/Classes/Asset.php

Indentation +2 added lines, -2 removed lines

@@ -1438,7 +1438,7 @@ 
         return $this->LibStatut($this->status, $mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return the status
      *
@@ -1448,7 +1448,7 @@ 
      */
     public function LibStatut($status, $mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         if (empty($this->labelStatus) || empty($this->labelStatusShort)) {
             global $langs;
             //$langs->load("assets");

Spacing +2 added lines, -2 removed lines

@@ -63,8 +63,8 @@
      */
     public $picto = 'asset';
 
-    const STATUS_DRAFT = 0;     // In progress
-    const STATUS_DISPOSED = 9;  // Disposed
+    const STATUS_DRAFT = 0; // In progress
+    const STATUS_DISPOSED = 9; // Disposed
 
     /**
      *  'type' field format ('integer', 'integer:ObjectClass:PathToClass[:AddCreateButtonOrNot[:Filter[:Sortfield]]]', 'sellist:TableName:LabelFieldName[:KeyFieldName[:KeyFieldParent[:Filter[:Sortfield]]]]', 'varchar(x)', 'double(24,8)', 'real', 'price', 'text', 'text:none', 'html', 'date', 'datetime', 'timestamp', 'duration', 'mail', 'phone', 'url', 'password')

Dolibarr/Code/Asset/Classes/AssetDepreciationOptions.php

Spacing +2 added lines, -2 removed lines

@@ -80,7 +80,7 @@ 
             'table' => 'asset_depreciation_options_economic',
             'fields' => array(
                 'depreciation_type' => array('type' => 'smallint', 'label' => 'AssetDepreciationOptionDepreciationType', 'enabled' => '1', 'position' => 10, 'notnull' => 1, 'visible' => 1, 'default' => '0', 'arrayofkeyval' => array('0' => 'AssetDepreciationOptionDepreciationTypeLinear', '1' => 'AssetDepreciationOptionDepreciationTypeDegressive', '2' => 'AssetDepreciationOptionDepreciationTypeExceptional'), 'validate' => '1',),
-                'degressive_coefficient' => array('type' => 'double(24,8)', 'label' => 'AssetDepreciationOptionDegressiveRate', 'enabled' => '1', 'position' => 20, 'notnull' => 1, 'visible' => 1, 'default' => '0', 'isameasure' => '1', 'validate' => '1','enabled_field' => 'economic:depreciation_type:1'),
+                'degressive_coefficient' => array('type' => 'double(24,8)', 'label' => 'AssetDepreciationOptionDegressiveRate', 'enabled' => '1', 'position' => 20, 'notnull' => 1, 'visible' => 1, 'default' => '0', 'isameasure' => '1', 'validate' => '1', 'enabled_field' => 'economic:depreciation_type:1'),
                 'duration' => array('type' => 'integer', 'label' => 'AssetDepreciationOptionDuration', 'enabled' => '1', 'position' => 30, 'notnull' => 1, 'visible' => 1, 'default' => '0', 'isameasure' => '1', 'validate' => '1',),
                 'duration_type' => array('type' => 'smallint', 'label' => 'AssetDepreciationOptionDurationType', 'enabled' => '1', 'position' => 40, 'notnull' => 1, 'visible' => 1, 'default' => '0', 'arrayofkeyval' => array('0' => 'AssetDepreciationOptionDurationTypeAnnual', '1' => 'AssetDepreciationOptionDurationTypeMonthly'/*, '2'=>'AssetDepreciationOptionDurationTypeDaily'*/), 'validate' => '1',),
                 'rate' => array('type' => 'double(24,8)', 'label' => 'AssetDepreciationOptionRate', 'enabled' => '1', 'position' => 50, 'visible' => 3, 'default' => '0', 'isameasure' => '1', 'validate' => '1', 'computed' => '$object->asset_depreciation_options->getRate("economic")',),
@@ -96,7 +96,7 @@ 
             'enabled_field' => 'economic:accelerated_depreciation_option:1',
             'fields' => array(
                 'depreciation_type' => array('type' => 'smallint', 'label' => 'AssetDepreciationOptionDepreciationType', 'enabled' => '1', 'position' => 10, 'notnull' => 1, 'visible' => 1, 'default' => '0', 'arrayofkeyval' => array('0' => 'AssetDepreciationOptionDepreciationTypeLinear', '1' => 'AssetDepreciationOptionDepreciationTypeDegressive', '2' => 'AssetDepreciationOptionDepreciationTypeExceptional'), 'validate' => '1',),
-                'degressive_coefficient' => array('type' => 'double(24,8)', 'label' => 'AssetDepreciationOptionDegressiveRate', 'enabled' => '1', 'position' => 20, 'notnull' => 1, 'visible' => 1, 'default' => '0', 'isameasure' => '1', 'validate' => '1','enabled_field' => 'accelerated_depreciation:depreciation_type:1'),
+                'degressive_coefficient' => array('type' => 'double(24,8)', 'label' => 'AssetDepreciationOptionDegressiveRate', 'enabled' => '1', 'position' => 20, 'notnull' => 1, 'visible' => 1, 'default' => '0', 'isameasure' => '1', 'validate' => '1', 'enabled_field' => 'accelerated_depreciation:depreciation_type:1'),
                 'duration' => array('type' => 'integer', 'label' => 'AssetDepreciationOptionDuration', 'enabled' => '1', 'position' => 30, 'notnull' => 1, 'visible' => 1, 'default' => '0', 'isameasure' => '1', 'validate' => '1',),
                 'duration_type' => array('type' => 'smallint', 'label' => 'AssetDepreciationOptionDurationType', 'enabled' => '1', 'position' => 40, 'notnull' => 1, 'visible' => 1, 'default' => '0', 'arrayofkeyval' => array('0' => 'AssetDepreciationOptionDurationTypeAnnual', '1' => 'AssetDepreciationOptionDurationTypeMonthly'/*, '2'=>'AssetDepreciationOptionDurationTypeDaily'*/), 'validate' => '1',),
                 'rate' => array('type' => 'double(24,8)', 'label' => 'AssetDepreciationOptionRate', 'enabled' => '1', 'position' => 50, 'visible' => 3, 'default' => '0', 'isameasure' => '1', 'validate' => '1', 'computed' => '$object->asset_depreciation_options->getRate("accelerated_depreciation")',),

Dolibarr/Code/Contact/Classes/Contact.php

Indentation +14 added lines, -14 removed lines

@@ -754,8 +754,8 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      *  Return DN string complete in the LDAP directory for the object
      *
@@ -767,7 +767,7 @@ 
      */
     public function _load_ldap_dn($info, $mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
         $dn = '';
         if ($mode == 0) {
@@ -781,8 +781,8 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      *  Initialize info table (LDAP attributes table)
      *
@@ -790,7 +790,7 @@ 
      */
     public function _load_ldap_info()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs;
 
         $info = array();
@@ -891,7 +891,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Update field alert birthday
      *
@@ -902,7 +902,7 @@ 
      */
     public function update_perso($id, $user = null, $notrigger = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         $error = 0;
         $result = false;
 
@@ -1186,7 +1186,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Load number of elements the contact is used as a link for
      *  ref_facturation
@@ -1198,7 +1198,7 @@ 
      */
     public function load_ref_elements()
     {
-		// phpcs:enable
+        // phpcs:enable
         // Count the elements for which it is contact
         $sql = "SELECT tc.element, count(ec.rowid) as nb";
         $sql .= " FROM " . MAIN_DB_PREFIX . "element_contact as ec, " . MAIN_DB_PREFIX . "c_type_contact as tc";
@@ -1582,7 +1582,7 @@ 
         return $this->LibStatut($this->statut, $mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return the label of a given status
      *
@@ -1592,7 +1592,7 @@ 
      */
     public function LibStatut($status, $mode)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         $labelStatus = array(
@@ -1620,7 +1620,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return translated label of Public or Private
      *
@@ -1629,7 +1629,7 @@ 
      */
     public function LibPubPriv($status)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
         if ($status == '1') {
             return $langs->trans('ContactPrivate');

Spacing +5 added lines, -5 removed lines

@@ -215,8 +215,8 @@ 
     /**
      * @var int Thirdparty ID
      */
-    public $socid;      // both socid and fk_soc are used
-    public $fk_soc;     // both socid and fk_soc are used
+    public $socid; // both socid and fk_soc are used
+    public $fk_soc; // both socid and fk_soc are used
 
     /**
      * @var string Thirdparty name
@@ -1067,8 +1067,8 @@ 
                 $this->country_code = $obj->country_id ? $obj->country_code : '';
                 $this->country      = $obj->country_id ? ($langs->trans('Country' . $obj->country_code) != 'Country' . $obj->country_code ? $langs->transnoentities('Country' . $obj->country_code) : $obj->country) : '';
 
-                $this->fk_soc       = $obj->fk_soc;     // Both fk_soc and socid are used
-                $this->socid        = $obj->fk_soc;     // Both fk_soc and socid are used
+                $this->fk_soc       = $obj->fk_soc; // Both fk_soc and socid are used
+                $this->socid        = $obj->fk_soc; // Both fk_soc and socid are used
                 $this->socname      = $obj->socname;
                 $this->poste        = $obj->poste;
                 $this->statut       = $obj->statut;
@@ -1876,7 +1876,7 @@ 
         $error = 0;
 
         if (!isset($this->roles)) {
-            return 0;   // Avoid to loose roles when property not set
+            return 0; // Avoid to loose roles when property not set
         }
 
         $this->db->begin();

Dolibarr/Code/Contrat/Classes/ContratLigne.php

Indentation +8 added lines, -8 removed lines

@@ -260,7 +260,7 @@ 
         return $this->LibStatut($this->statut, $mode, ((!empty($this->date_end)) ? ($this->date_end < dol_now() ? 1 : 0) : -1));
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return label of a contract line status
      *
@@ -273,7 +273,7 @@ 
      */
     public static function LibStatut($status, $mode, $expired = -1, $moreatt = '', $morelabel = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
         $langs->load("contracts");
 
@@ -709,7 +709,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Update in database the fields total_xxx of lines
      *  Used by migration process
@@ -718,7 +718,7 @@ 
      */
     public function update_total()
     {
-		// phpcs:enable
+        // phpcs:enable
         $this->db->begin();
 
         // Mise a jour ligne en base
@@ -836,7 +836,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Activate a contract line
      *
@@ -848,7 +848,7 @@ 
      */
     public function active_line($user, $date, $date_end = '', $comment = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         $error = 0;
 
         $this->db->begin();
@@ -894,7 +894,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Close a contract line
      *
@@ -906,7 +906,7 @@ 
      */
     public function close_line($user, $date_end_real, $comment = '', $notrigger = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         $this->date_cloture = $date_end_real;
         $this->date_end_real = $date_end_real;
         $this->user_closing_id = $user->id;

Dolibarr/Code/Contrat/Classes/Contrat.php

Indentation +14 added lines, -14 removed lines

@@ -364,7 +364,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Activate a contract line
      *
@@ -377,7 +377,7 @@ 
      */
     public function active_line($user, $line_id, $date_start, $date_end = '', $comment = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         $result = $this->lines[$this->lines_id_index_mapper[$line_id]]->active_line($user, $date_start, $date_end, $comment);
         if ($result < 0) {
             $this->error = $this->lines[$this->lines_id_index_mapper[$line_id]]->error;
@@ -387,7 +387,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Close a contract line
      *
@@ -399,7 +399,7 @@ 
      */
     public function close_line($user, $line_id, $date_end, $comment = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         $result = $this->lines[$this->lines_id_index_mapper[$line_id]]->close_line($user, $date_end, $comment);
         if ($result < 0) {
             $this->error = $this->lines[$this->lines_id_index_mapper[$line_id]]->error;
@@ -821,7 +821,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Load lines array into this->lines.
      *  This set also nbofserviceswait, nbofservicesopened, nbofservicesexpired and nbofservicesclosed
@@ -833,7 +833,7 @@ 
      */
     public function fetch_lines($only_services = 0, $loadalsotranslation = 0, $noextrafields = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         $this->nbofservices = 0;
         $this->nbofserviceswait = 0;
         $this->nbofservicesopened = 0;
@@ -1964,7 +1964,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Update statut of contract according to services
      *
@@ -1974,7 +1974,7 @@ 
      */
     public function update_statut($user)
     {
-		// phpcs:enable
+        // phpcs:enable
         dol_syslog(__METHOD__ . " is deprecated", LOG_WARNING);
 
         // If draft, we keep it (should not happen)
@@ -2006,7 +2006,7 @@ 
         return $this->LibStatut($this->statut, $mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return the label of a given contrat status
      *
@@ -2016,7 +2016,7 @@ 
      */
     public function LibStatut($status, $mode)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         if (empty($this->labelStatus) || empty($this->labelStatusShort)) {
@@ -2226,7 +2226,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return list of line rowid
      *
@@ -2235,7 +2235,7 @@ 
      */
     public function array_detail($status = -1)
     {
-		// phpcs:enable
+        // phpcs:enable
         $tab = array();
 
         $sql = "SELECT cd.rowid";
@@ -2307,7 +2307,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *      Load indicators for dashboard (this->nbtodo and this->nbtodolate)
      *
@@ -2317,7 +2317,7 @@ 
      */
     public function load_board($user, $mode)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs;
 
         $this->from = " FROM " . MAIN_DB_PREFIX . "contrat as c";

Spacing +17 added lines, -17 removed lines

@@ -286,12 +286,12 @@ 
     /*
      * No signature
      */
-    const STATUS_NO_SIGNATURE    = 0;
+    const STATUS_NO_SIGNATURE = 0;
 
     /*
      * Signed by sender
      */
-    const STATUS_SIGNED_SENDER   = 1;
+    const STATUS_SIGNED_SENDER = 1;
 
     /*
      * Signed by receiver
@@ -301,7 +301,7 @@ 
     /*
      * Signed by all
      */
-    const STATUS_SIGNED_ALL      = 9; // To handle future kind of signature (ex: tripartite contract)
+    const STATUS_SIGNED_ALL = 9; // To handle future kind of signature (ex: tripartite contract)
 
 
     /**
@@ -437,7 +437,7 @@ 
             if ($contratline->statut != ContratLigne::STATUS_OPEN) {
                 $contratline->context = $this->context;
 
-                $result = $contratline->active_line($user, $date_start, !empty($date_end) ? $date_end : -1, $comment);  // This call trigger LINECONTRACT_ACTIVATE
+                $result = $contratline->active_line($user, $date_start, !empty($date_end) ? $date_end : -1, $comment); // This call trigger LINECONTRACT_ACTIVATE
                 if ($result < 0) {
                     $error++;
                     $this->error = $contratline->error;
@@ -487,7 +487,7 @@ 
             // Close lines not already closed
             if ($contratline->statut != ContratLigne::STATUS_CLOSED) {
                 $contratline->date_end_real = $now;
-                $contratline->date_cloture = $now;  // For backward compatibility
+                $contratline->date_cloture = $now; // For backward compatibility
                 $contratline->user_closing_id = $user->id;
                 $contratline->statut = ContratLigne::STATUS_CLOSED;
                 $result = $contratline->close_line($user, $now, $comment, $notrigger);
@@ -942,7 +942,7 @@ 
                 //$line->date_fin_prevue   = $this->db->jdate($objp->date_fin_validite);
                 //$line->date_fin_reel     = $this->db->jdate($objp->date_cloture);
 
-                $line->rang     = $objp->rang;
+                $line->rang = $objp->rang;
 
                 // Retrieve all extrafields for contract line
                 // fetch optionals attributes and labels
@@ -1454,7 +1454,7 @@ 
         }
 
         if (!$error) {
-            $result = $this->insertExtraFields();   // This delete and reinsert extrafields
+            $result = $this->insertExtraFields(); // This delete and reinsert extrafields
             if ($result < 0) {
                 $error++;
             }
@@ -2558,7 +2558,7 @@ 
         global $user;
 
         $ticket = new Ticket($this->db);
-        $nbTicket =  $ticket->fetchAll($user, 'ASC', 't.datec', '', 0, '', array('t.fk_contract' => $this->id));
+        $nbTicket = $ticket->fetchAll($user, 'ASC', 't.datec', '', 0, '', array('t.fk_contract' => $this->id));
 
         return ($nbTicket < 0 ? $nbTicket : $ticket->lines);
     }
@@ -2580,7 +2580,7 @@ 
         global $conf, $langs;
 
         if (!dol_strlen($modele)) {
-            $modele = '';   // No doc template/generation by default
+            $modele = ''; // No doc template/generation by default
 
             if (!empty($this->model_pdf)) {
                 $modele = $this->model_pdf;
@@ -2806,7 +2806,7 @@ 
 
                     // Load contract
                     $object = new Contrat($this->db);
-                    $object->fetch($obj->rowid);        // fetch also lines
+                    $object->fetch($obj->rowid); // fetch also lines
                     //$object->fetch_thirdparty();
 
                     if ($object->id <= 0) {
@@ -2853,7 +2853,7 @@ 
                     if ($expirationdate && $expirationdate < $enddatetoscan) {
                         dol_syslog("Define the newdate of end of services from expirationdate=" . $expirationdate);
                         $newdate = $expirationdate;
-                        $protecti = 0;  //$protecti is to avoid infinite loop
+                        $protecti = 0; //$protecti is to avoid infinite loop
                         while ($newdate < $enddatetoscan && $protecti < 1000) {
                             $newdate = dol_time_plus_duree($newdate, $duration_value, $duration_unit);
                             $protecti++;
@@ -2881,20 +2881,20 @@ 
 
                                 // Create an event
                                 $actioncomm = new ActionComm($this->db);
-                                $actioncomm->type_code    = 'AC_OTH_AUTO';      // Type of event ('AC_OTH', 'AC_OTH_AUTO', 'AC_XXX'...)
+                                $actioncomm->type_code    = 'AC_OTH_AUTO'; // Type of event ('AC_OTH', 'AC_OTH_AUTO', 'AC_XXX'...)
                                 $actioncomm->code         = 'AC_' . $actioncode;
                                 $actioncomm->label        = $label;
                                 $actioncomm->datep        = $now;
                                 $actioncomm->datef        = $now;
-                                $actioncomm->percentage   = -1;   // Not applicable
+                                $actioncomm->percentage   = -1; // Not applicable
                                 $actioncomm->socid        = $object->socid;
-                                $actioncomm->authorid     = $user->id;   // User saving action
-                                $actioncomm->userownerid  = $user->id;  // Owner of action
+                                $actioncomm->authorid     = $user->id; // User saving action
+                                $actioncomm->userownerid  = $user->id; // Owner of action
                                 $actioncomm->fk_element   = $object->id;
                                 $actioncomm->elementtype  = 'contract';
                                 $actioncomm->note_private = $comment;
 
-                                $ret = $actioncomm->create($user);       // User creating action
+                                $ret = $actioncomm->create($user); // User creating action
                             } else {
                                 $contracterror[$object->id] = $object->ref;
 
@@ -2903,7 +2903,7 @@ 
                                 $this->error = $this->db->lasterror();
                             }
 
-                            if (! $errorforlocaltransaction) {
+                            if (!$errorforlocaltransaction) {
                                 $this->db->commit();
                             } else {
                                 $this->db->rollback();