alxarafe / alixar

public/htdocs/contrat/card.php

Spacing +8 added lines, -8 removed lines

@@ -110,7 +110,7 @@ 
 $permissionnote = $user->hasRight('contrat', 'creer'); // Used by the include of actions_setnotes.inc.php
 $permissiondellink = $user->hasRight('contrat', 'creer'); // Used by the include of actions_dellink.inc.php
 $permissiontodelete = ($user->hasRight('contrat', 'creer') && $object->statut == $object::STATUS_DRAFT) || $user->hasRight('contrat', 'supprimer');
-$permissiontoadd   = $user->hasRight('contrat', 'creer');     //  Used by the include of actions_addupdatedelete.inc.php and actions_lineupdown.inc.php
+$permissiontoadd   = $user->hasRight('contrat', 'creer'); //  Used by the include of actions_addupdatedelete.inc.php and actions_lineupdown.inc.php
 $permissiontoedit = $permissiontoadd;
 $permissiontoactivate = $user->hasRight('contrat', 'activer');
 $error = 0;
@@ -155,7 +155,7 @@ 
 
     include DOL_DOCUMENT_ROOT . '/core/actions_dellink.inc.php'; // Must be include, not include_once
 
-    include DOL_DOCUMENT_ROOT . '/core/actions_lineupdown.inc.php';  // Must be include, not include_once
+    include DOL_DOCUMENT_ROOT . '/core/actions_lineupdown.inc.php'; // Must be include, not include_once
 
     if ($action == 'confirm_active' && $confirm == 'yes' && $permissiontoactivate) {
         $date_start = '';
@@ -533,8 +533,8 @@ 
                     $result = $prodcustprice->fetchAll('', '', 0, 0, $filter);
                     if ($result) {
                         if (count($prodcustprice->lines) > 0) {
-                            $price_min =  price($prodcustprice->lines[0]->price_min);
-                            $price_min_ttc =  price($prodcustprice->lines[0]->price_min_ttc);
+                            $price_min = price($prodcustprice->lines[0]->price_min);
+                            $price_min_ttc = price($prodcustprice->lines[0]->price_min_ttc);
                             /*$tva_tx = $prodcustprice->lines[0]->tva_tx;
                             if ($prodcustprice->lines[0]->default_vat_code && !preg_match('/\(.*\)/', $tva_tx)) {
                                 $tva_tx .= ' ('.$prodcustprice->lines[0]->default_vat_code.')';
@@ -711,7 +711,7 @@ 
         if (!empty($date_start_update) && !empty($date_end_update) && $date_start_update > $date_end_update) {
             setEventMessages($langs->trans("Error") . ': ' . $langs->trans("DateStartPlanned") . ' > ' . $langs->trans("DateEndPlanned"), null, 'errors');
             $action = 'editline';
-            $_GET['rowid'] = GETPOST('elrowid');    // Keep $_GET here. Used by GETPOST('rowid') later
+            $_GET['rowid'] = GETPOST('elrowid'); // Keep $_GET here. Used by GETPOST('rowid') later
             $error++;
         }
 
@@ -770,7 +770,7 @@ 
             // update price_ht with discount
             // TODO Use object->updateline instead objectline->update
 
-            $price_ht =  price2num(GETPOST('elprice'), 'MU');
+            $price_ht = price2num(GETPOST('elprice'), 'MU');
             $remise_percent = price2num(GETPOST('elremise_percent'), '', 2);
             if ($remise_percent > 0) {
                 $remise = round(((float) $price_ht * (float) $remise_percent / 100), 2);
@@ -875,7 +875,7 @@ 
     } elseif ($action == 'confirm_activate' && $confirm == 'yes' && $user->hasRight('contrat', 'creer')) {
         $date_start = dol_mktime(12, 0, 0, GETPOST('d_startmonth'), GETPOST('d_startday'), GETPOST('d_startyear'));
         $date_end   = dol_mktime(12, 0, 0, GETPOST('d_endmonth'), GETPOST('d_endday'), GETPOST('d_endyear'));
-        $comment      = GETPOST('comment', 'alpha');
+        $comment = GETPOST('comment', 'alpha');
         $result = $object->activateAll($user, $date_start, 0, $comment, $date_end);
         if ($result < 0) {
             setEventMessages($object->error, $object->errors, 'errors');
@@ -1504,7 +1504,7 @@ 
         }
 
 
-        $arrayothercontracts = $object->getListOfContracts('others');   // array or -1 if technical error
+        $arrayothercontracts = $object->getListOfContracts('others'); // array or -1 if technical error
 
         /*
          * Lines of contracts

public/htdocs/contrat/ticket.php

Spacing +3 added lines, -3 removed lines

@@ -75,7 +75,7 @@ 
 
 dol_get_fiche_head($head, 'ticket', $langs->trans("Contract"), -1, 'contract');
 
-$linkback = '<a href="' . constant('BASE_URL') . '/contrat/list.php' . (! empty($socid) ? '?socid=' . $socid : '') . '">';
+$linkback = '<a href="' . constant('BASE_URL') . '/contrat/list.php' . (!empty($socid) ? '?socid=' . $socid : '') . '">';
 $linkback .= $langs->trans("BackToList") . '</a>';
 
 $morehtmlref = '';
@@ -136,12 +136,12 @@ 
 // Thirdparty
 $morehtmlref .= '<br>' . $langs->trans('ThirdParty') . ' : ' . $object->thirdparty->getNomUrl(1);
 // Project
-if (! empty($conf->projet->enabled)) {
+if (!empty($conf->projet->enabled)) {
     require_once constant('DOL_DOCUMENT_ROOT') . '/projet/class/task.class.php';
 
     $langs->load("projects");
     $morehtmlref .= '<br>' . $langs->trans('Project') . ' : ';
-    if (! empty($object->fk_project)) {
+    if (!empty($object->fk_project)) {
         $proj = new Project($db);
         $proj->fetch($object->fk_project);
         $morehtmlref .= '<a href="' . constant('BASE_URL') . '/projet/card.php?id=';

public/htdocs/contrat/services_list.php

Spacing +2 added lines, -2 removed lines

@@ -153,7 +153,7 @@ 
 
 $object->fields = dol_sort_array($object->fields, 'position');
 $arrayfields = dol_sort_array($arrayfields, 'position');
-'@phan-var-force array<string,array{label:string,checked?:int<0,1>,position?:int,help?:string}> $arrayfields';  // dol_sort_array looses type for Phan
+'@phan-var-force array<string,array{label:string,checked?:int<0,1>,position?:int,help?:string}> $arrayfields'; // dol_sort_array looses type for Phan
 
 $permissiontoread = $user->hasRight('contrat', 'lire');
 $permissiontoadd = $user->hasRight('contrat', 'creer');
@@ -573,7 +573,7 @@ 
 }
 
 $varpage = empty($contextpage) ? $_SERVER["PHP_SELF"] : $contextpage;
-$htmlofselectarray = $form->multiSelectArrayWithCheckbox('selectedfields', $arrayfields, $varpage, getDolGlobalString('MAIN_CHECKBOX_LEFT_COLUMN'));  // This also change content of $arrayfields with user setup
+$htmlofselectarray = $form->multiSelectArrayWithCheckbox('selectedfields', $arrayfields, $varpage, getDolGlobalString('MAIN_CHECKBOX_LEFT_COLUMN')); // This also change content of $arrayfields with user setup
 $selectedfields = ($mode != 'kanban' ? $htmlofselectarray : '');
 $selectedfields .= (count($arrayofmassactions) ? $form->showCheckAddButtons('checkforselect', 1) : '');
 

public/htdocs/contrat/agenda.php

Spacing +1 added lines, -1 removed lines

@@ -89,7 +89,7 @@
 // Initialize technical object to manage hooks of page. Note that conf->hooks_modules contains array of hook context
 $hookmanager->initHooks(array('agendacontract', 'globalcard'));
 
-$permissiontoadd = $user->hasRight('contrat', 'creer');     //  Used by the include of actions_addupdatedelete.inc.php and actions_lineupdown.inc.php
+$permissiontoadd = $user->hasRight('contrat', 'creer'); //  Used by the include of actions_addupdatedelete.inc.php and actions_lineupdown.inc.php
 
 $result = restrictedArea($user, 'contrat', $object->id);
 

public/htdocs/contrat/contact.php

Spacing +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
 // Initialize technical object to manage hooks of page. Note that conf->hooks_modules contains array of hook context
 $hookmanager->initHooks(array('contractcontactcard', 'globalcard'));
 
-$permissiontoadd   = $user->hasRight('contrat', 'creer');     //  Used by the include of actions_addupdatedelete.inc.php and actions_lineupdown.inc.php
+$permissiontoadd = $user->hasRight('contrat', 'creer'); //  Used by the include of actions_addupdatedelete.inc.php and actions_lineupdown.inc.php
 
 $result = restrictedArea($user, 'contrat', $object->id);
 

public/htdocs/contrat/list.php

Spacing +29 added lines, -29 removed lines

@@ -85,7 +85,7 @@ 
 $search_date_endday = GETPOSTINT('search_date_endday');
 $search_date_endmonth = GETPOSTINT('search_date_endmonth');
 $search_date_endyear = GETPOSTINT('search_date_endyear');
-$search_date_start = dol_mktime(0, 0, 0, $search_date_startmonth, $search_date_startday, $search_date_startyear);   // Use tzserver
+$search_date_start = dol_mktime(0, 0, 0, $search_date_startmonth, $search_date_startday, $search_date_startyear); // Use tzserver
 $search_date_end = dol_mktime(23, 59, 59, $search_date_endmonth, $search_date_endday, $search_date_endyear);
 $searchCategoryCustomerOperator = 0;
 if (GETPOSTISSET('formfilteraction')) {
@@ -98,20 +98,20 @@ 
 $search_date_creation_startmonth = GETPOSTINT('search_date_creation_startmonth');
 $search_date_creation_startyear = GETPOSTINT('search_date_creation_startyear');
 $search_date_creation_startday = GETPOSTINT('search_date_creation_startday');
-$search_date_creation_start = dol_mktime(0, 0, 0, $search_date_creation_startmonth, $search_date_creation_startday, $search_date_creation_startyear);   // Use tzserver
+$search_date_creation_start = dol_mktime(0, 0, 0, $search_date_creation_startmonth, $search_date_creation_startday, $search_date_creation_startyear); // Use tzserver
 $search_date_creation_endmonth = GETPOSTINT('search_date_creation_endmonth');
 $search_date_creation_endyear = GETPOSTINT('search_date_creation_endyear');
 $search_date_creation_endday = GETPOSTINT('search_date_creation_endday');
-$search_date_creation_end = dol_mktime(23, 59, 59, $search_date_creation_endmonth, $search_date_creation_endday, $search_date_creation_endyear);    // Use tzserver
+$search_date_creation_end = dol_mktime(23, 59, 59, $search_date_creation_endmonth, $search_date_creation_endday, $search_date_creation_endyear); // Use tzserver
 
 $search_date_modif_startmonth = GETPOSTINT('search_date_modif_startmonth');
 $search_date_modif_startyear = GETPOSTINT('search_date_modif_startyear');
 $search_date_modif_startday = GETPOSTINT('search_date_modif_startday');
-$search_date_modif_start = dol_mktime(0, 0, 0, $search_date_modif_startmonth, $search_date_modif_startday, $search_date_modif_startyear);   // Use tzserver
+$search_date_modif_start = dol_mktime(0, 0, 0, $search_date_modif_startmonth, $search_date_modif_startday, $search_date_modif_startyear); // Use tzserver
 $search_date_modif_endmonth = GETPOSTINT('search_date_modif_endmonth');
 $search_date_modif_endyear = GETPOSTINT('search_date_modif_endyear');
 $search_date_modif_endday = GETPOSTINT('search_date_modif_endday');
-$search_date_modif_end = dol_mktime(23, 59, 59, $search_date_modif_endmonth, $search_date_modif_endday, $search_date_modif_endyear);    // Use tzserver
+$search_date_modif_end = dol_mktime(23, 59, 59, $search_date_modif_endmonth, $search_date_modif_endday, $search_date_modif_endyear); // Use tzserver
 
 // Load variable for pagination
 $limit = GETPOSTINT('limit') ? GETPOSTINT('limit') : $conf->liste_limit;
@@ -200,7 +200,7 @@ 
 
 $object->fields = dol_sort_array($object->fields, 'position');
 $arrayfields = dol_sort_array($arrayfields, 'position');
-'@phan-var-force array<string,array{label:string,checked?:int<0,1>,position?:int,help?:string}> $arrayfields';  // dol_sort_array looses type for Phan
+'@phan-var-force array<string,array{label:string,checked?:int<0,1>,position?:int,help?:string}> $arrayfields'; // dol_sort_array looses type for Phan
 
 if (!$user->hasRight('societe', 'client', 'voir')) {
     $search_sale = $user->id;
@@ -534,7 +534,7 @@ 
         $resql = $db->query($sql, 0, 'auto', 1);
         while ($db->fetch_object($resql)) {
             if (empty($nbtotalofrecords)) {
-                $nbtotalofrecords = 1;    // We can't make +1 because init value is ''
+                $nbtotalofrecords = 1; // We can't make +1 because init value is ''
             } else {
                 $nbtotalofrecords++;
             }
@@ -986,55 +986,55 @@ 
 print '<tr class="liste_titre">';
 if (getDolGlobalString('MAIN_CHECKBOX_LEFT_COLUMN')) {
     print_liste_field_titre($selectedfields, $_SERVER["PHP_SELF"], "", '', '', '', $sortfield, $sortorder, 'center maxwidthsearch ');
-    $totalarray['nbfield']++;   // For the column action
+    $totalarray['nbfield']++; // For the column action
 }
 if (!empty($arrayfields['c.ref']['checked'])) {
     print_liste_field_titre($arrayfields['c.ref']['label'], $_SERVER["PHP_SELF"], "c.ref", "", $param, '', $sortfield, $sortorder);
-    $totalarray['nbfield']++;   // For the column action
+    $totalarray['nbfield']++; // For the column action
 }
 if (!empty($arrayfields['c.ref_customer']['checked'])) {
     print_liste_field_titre($arrayfields['c.ref_customer']['label'], $_SERVER["PHP_SELF"], "c.ref_customer", "", $param, '', $sortfield, $sortorder);
-    $totalarray['nbfield']++;   // For the column action
+    $totalarray['nbfield']++; // For the column action
 }
 if (!empty($arrayfields['c.ref_supplier']['checked'])) {
     print_liste_field_titre($arrayfields['c.ref_supplier']['label'], $_SERVER["PHP_SELF"], "c.ref_supplier", "", $param, '', $sortfield, $sortorder);
-    $totalarray['nbfield']++;   // For the column action
+    $totalarray['nbfield']++; // For the column action
 }
 if (!empty($arrayfields['s.nom']['checked'])) {
     print_liste_field_titre($arrayfields['s.nom']['label'], $_SERVER["PHP_SELF"], "s.nom", "", $param, '', $sortfield, $sortorder);
-    $totalarray['nbfield']++;   // For the column action
+    $totalarray['nbfield']++; // For the column action
 }
 if (!empty($arrayfields['s.email']['checked'])) {
     print_liste_field_titre($arrayfields['s.email']['label'], $_SERVER["PHP_SELF"], "s.email", "", $param, '', $sortfield, $sortorder);
-    $totalarray['nbfield']++;   // For the column action
+    $totalarray['nbfield']++; // For the column action
 }
 if (!empty($arrayfields['s.town']['checked'])) {
     print_liste_field_titre($arrayfields['s.town']['label'], $_SERVER["PHP_SELF"], 's.town', '', $param, '', $sortfield, $sortorder);
-    $totalarray['nbfield']++;   // For the column action
+    $totalarray['nbfield']++; // For the column action
 }
 if (!empty($arrayfields['s.zip']['checked'])) {
     print_liste_field_titre($arrayfields['s.zip']['label'], $_SERVER["PHP_SELF"], 's.zip', '', $param, '', $sortfield, $sortorder);
-    $totalarray['nbfield']++;   // For the column action
+    $totalarray['nbfield']++; // For the column action
 }
 if (!empty($arrayfields['state.nom']['checked'])) {
     print_liste_field_titre($arrayfields['state.nom']['label'], $_SERVER["PHP_SELF"], "state.nom", "", $param, '', $sortfield, $sortorder);
-    $totalarray['nbfield']++;   // For the column action
+    $totalarray['nbfield']++; // For the column action
 }
 if (!empty($arrayfields['country.code_iso']['checked'])) {
     print_liste_field_titre($arrayfields['country.code_iso']['label'], $_SERVER["PHP_SELF"], "country.code_iso", "", $param, '', $sortfield, $sortorder, 'center ');
-    $totalarray['nbfield']++;   // For the column action
+    $totalarray['nbfield']++; // For the column action
 }
 if (!empty($arrayfields['typent.code']['checked'])) {
     print_liste_field_titre($arrayfields['typent.code']['label'], $_SERVER["PHP_SELF"], "typent.code", "", $param, '', $sortfield, $sortorder, 'center ');
-    $totalarray['nbfield']++;   // For the column action
+    $totalarray['nbfield']++; // For the column action
 }
 if (!empty($arrayfields['sale_representative']['checked'])) {
     print_liste_field_titre($arrayfields['sale_representative']['label'], $_SERVER["PHP_SELF"], "", "", $param, '', $sortfield, $sortorder);
-    $totalarray['nbfield']++;   // For the column action
+    $totalarray['nbfield']++; // For the column action
 }
 if (!empty($arrayfields['c.date_contrat']['checked'])) {
     print_liste_field_titre($arrayfields['c.date_contrat']['label'], $_SERVER["PHP_SELF"], "c.date_contrat", "", $param, '', $sortfield, $sortorder, 'center ');
-    $totalarray['nbfield']++;   // For the column action
+    $totalarray['nbfield']++; // For the column action
 }
 // Extra fields
 include DOL_DOCUMENT_ROOT . '/core/tpl/extrafields_list_search_title.tpl.php';
@@ -1044,29 +1044,29 @@ 
 print $hookmanager->resPrint;
 if (!empty($arrayfields['c.datec']['checked'])) {
     print_liste_field_titre($arrayfields['c.datec']['label'], $_SERVER["PHP_SELF"], "c.datec", "", $param, '', $sortfield, $sortorder, 'center nowrap ');
-    $totalarray['nbfield']++;   // For the column action
+    $totalarray['nbfield']++; // For the column action
 }
 if (!empty($arrayfields['c.tms']['checked'])) {
     print_liste_field_titre($arrayfields['c.tms']['label'], $_SERVER["PHP_SELF"], "c.tms", "", $param, '', $sortfield, $sortorder, 'center nowrap ');
-    $totalarray['nbfield']++;   // For the column action
+    $totalarray['nbfield']++; // For the column action
 }
 if (!empty($arrayfields['lower_planned_end_date']['checked'])) {
     print_liste_field_titre($arrayfields['lower_planned_end_date']['label'], $_SERVER["PHP_SELF"], "lower_planned_end_date", "", $param, '', $sortfield, $sortorder, 'center ');
-    $totalarray['nbfield']++;   // For the column action
+    $totalarray['nbfield']++; // For the column action
 }
 if (!empty($arrayfields['status']['checked'])) {
     print_liste_field_titre($staticcontratligne->LibStatut(0, 3, -1, 'class="nochangebackground"'), '', '', '', '', 'width="16"');
-    $totalarray['nbfield']++;   // For the column action
+    $totalarray['nbfield']++; // For the column action
     print_liste_field_titre($staticcontratligne->LibStatut(4, 3, 0, 'class="nochangebackground"'), '', '', '', '', 'width="16"');
-    $totalarray['nbfield']++;   // For the column action
+    $totalarray['nbfield']++; // For the column action
     print_liste_field_titre($staticcontratligne->LibStatut(4, 3, 1, 'class="nochangebackground"'), '', '', '', '', 'width="16"');
-    $totalarray['nbfield']++;   // For the column action
+    $totalarray['nbfield']++; // For the column action
     print_liste_field_titre($staticcontratligne->LibStatut(5, 3, -1, 'class="nochangebackground"'), '', '', '', '', 'width="16"');
-    $totalarray['nbfield']++;   // For the column action
+    $totalarray['nbfield']++; // For the column action
 }
 if (!getDolGlobalString('MAIN_CHECKBOX_LEFT_COLUMN')) {
     print_liste_field_titre($selectedfields, $_SERVER["PHP_SELF"], "", '', '', '', $sortfield, $sortorder, 'center maxwidthsearch ');
-    $totalarray['nbfield']++;   // For the column action
+    $totalarray['nbfield']++; // For the column action
 }
 print "</tr>\n";
 
@@ -1356,7 +1356,7 @@ 
 
 // If no record found
 if ($num == 0) {
-    $colspan = 4;   // Include the 4 columns of status
+    $colspan = 4; // Include the 4 columns of status
     foreach ($arrayfields as $key => $val) {
         if (!empty($val['checked'])) {
             $colspan++;

public/htdocs/contrat/class/contrat.class.php

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();

public/htdocs/contrat/class/api_contracts.class.php

Spacing +1 added lines, -1 removed lines

@@ -511,7 +511,7 @@
             }
             if ($field == 'array_options' && is_array($value)) {
                 foreach ($value as $index => $val) {
-                    $this->contract->array_options[$index] = $this->_checkValForAPI($field, $val, $this->contract);;
+                    $this->contract->array_options[$index] = $this->_checkValForAPI($field, $val, $this->contract); ;
                 }
                 continue;
             }

public/htdocs/contrat/note.php

Spacing +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
 $object = new Contrat($db);
 $object->fetch($id, $ref);
 
-$permissiontoadd   = $user->hasRight('contrat', 'creer');     //  Used by the include of actions_addupdatedelete.inc.php and actions_lineupdown.inc.php
+$permissiontoadd = $user->hasRight('contrat', 'creer'); //  Used by the include of actions_addupdatedelete.inc.php and actions_lineupdown.inc.php
 $permissionnote = $user->hasRight('contrat', 'creer'); // Used by the include of actions_setnotes.inc.php
 
 $result = restrictedArea($user, 'contrat', $object->id);