alxarafe / alixar

public/htdocs/webhook/target_card.php

Spacing +2 added lines, -2 removed lines

@@ -36,7 +36,7 @@ 
 global $conf, $db, $hookmanager, $langs, $user;
 
 // Load translation files required by the page
-$langs->loadLangs(array('other','admin'));
+$langs->loadLangs(array('other', 'admin'));
 
 // Get parameters
 $id = GETPOSTINT('id');
@@ -47,7 +47,7 @@ 
 $contextpage = GETPOST('contextpage', 'aZ') ? GETPOST('contextpage', 'aZ') : 'targetcard'; // To manage different context of search
 $backtopage = GETPOST('backtopage', 'alpha');
 $backtopageforcancel = GETPOST('backtopageforcancel', 'alpha');
-$lineid   = GETPOSTINT('lineid');
+$lineid = GETPOSTINT('lineid');
 
 // Initialize technical objects
 $object = new Target($db);

public/htdocs/reception/card.php

Indentation +1 added lines, -1 removed lines

@@ -966,7 +966,7 @@
             /**
              * @var array $suffix2numAsked map HTTP query parameter suffixes (like '1_0') to line indices so that
              *                             extrafields from HTTP query can be assigned to the correct dispatch line
-            */
+             */
             $suffix2numAsked = array();
             $dispatchLines = array();
 

Spacing +7 added lines, -7 removed lines

@@ -343,7 +343,7 @@ 
 
         // Loop lines to calculate $totalqty
         for ($i = 1; $i <= $num; $i++) {
-            $idl = "idl" . $i;    // id line source
+            $idl = "idl" . $i; // id line source
 
             //$sub_qty = array();
             //$subtotalqty = 0;
@@ -351,7 +351,7 @@ 
             //$j = 0;
             //$batch = "batchl".$i."_0";
             //$stockLocation = "ent1".$i."_0";
-            $qty = "qtyl" . $i;   // qty
+            $qty = "qtyl" . $i; // qty
 
             //reception line for product with no batch management and no multiple stock location
             if (GETPOST($qty, 'alpha') > 0) {
@@ -365,7 +365,7 @@ 
 
         if ($totalqty > 0) {  // There is at least one thing to ship
             for ($i = 1; $i <= $num; $i++) {
-                $idl = "idl" . $i;    // id line source
+                $idl = "idl" . $i; // id line source
                 $lineToTest = '';
                 $lineId = GETPOSTINT($idl);
                 foreach ($objectsrc->lines as $linesrc) {
@@ -1144,7 +1144,7 @@ 
 
                     print '<input type="hidden" name="productl' . $indiceAsked . '" value="' . $line->fk_product . '">';
 
-                    if (! array_key_exists($line->id, $arrayofpurchaselinealreadyoutput)) { // Add test to avoid to show qty twice
+                    if (!array_key_exists($line->id, $arrayofpurchaselinealreadyoutput)) { // Add test to avoid to show qty twice
                         print '<input type="hidden" name="productid' . $indiceAsked . '" value="' . $line->fk_product . '">';
 
                         // Show product and description
@@ -1166,7 +1166,7 @@ 
                     print '</td>';
                 } else {
                     print "<td>";
-                    if (! array_key_exists($line->id, $arrayofpurchaselinealreadyoutput)) { // Add test to avoid to show qty twice
+                    if (!array_key_exists($line->id, $arrayofpurchaselinealreadyoutput)) { // Add test to avoid to show qty twice
                         if ($type == 1) {
                             $text = img_object($langs->trans('Service'), 'service');
                         } else {
@@ -1195,7 +1195,7 @@ 
 
                 // Qty in source purchase order line
                 print '<td class="center">';
-                if (! array_key_exists($line->id, $arrayofpurchaselinealreadyoutput)) { // Add test to avoid to show qty twice
+                if (!array_key_exists($line->id, $arrayofpurchaselinealreadyoutput)) { // Add test to avoid to show qty twice
                     print $line->qty;
                 }
                 print '<input type="hidden" name="fk_commandefournisseurdet' . $indiceAsked . '" value="' . $line->id . '">';
@@ -1207,7 +1207,7 @@ 
                 // Qty already received
                 print '<td class="center">';
                 $quantityDelivered = $objectsrc->receptions[$line->id];
-                if (! array_key_exists($line->id, $arrayofpurchaselinealreadyoutput)) { // Add test to avoid to show qty twice
+                if (!array_key_exists($line->id, $arrayofpurchaselinealreadyoutput)) { // Add test to avoid to show qty twice
                     print $quantityDelivered;
                 }
                 print '<input name="qtydelivered' . $indiceAsked . '" id="qtydelivered' . $indiceAsked . '" type="hidden" value="' . $quantityDelivered . '">';

public/htdocs/reception/dispatch.php

Spacing +2 added lines, -2 removed lines

@@ -442,7 +442,7 @@ 
     print '<center>';
 
     print '<br>';
-    $disabled = 0;  // This is used to disable or not the bulk selection of target warehouse. No reason to have it disabled so forced to 0.
+    $disabled = 0; // This is used to disable or not the bulk selection of target warehouse. No reason to have it disabled so forced to 0.
 
     if ($object->statut == Reception::STATUS_DRAFT || ($object->statut == Reception::STATUS_VALIDATED && !getDolGlobalString('STOCK_CALCULATE_ON_RECEPTION'))) {
         require_once constant('DOL_DOCUMENT_ROOT') . '/product/class/html.formproduct.class.php';
@@ -546,7 +546,7 @@ 
                 print '<td class="right">' . $langs->trans("SupplierRef") . '</td>';
                 print '<td class="right">' . $langs->trans("QtyOrdered") . '</td>';
                 if ($object->status == Reception::STATUS_DRAFT) {
-                    print '<td class="right">' . $langs->trans("QtyToReceive");   // Qty to dispatch (sum for all lines of batch detail if there is)
+                    print '<td class="right">' . $langs->trans("QtyToReceive"); // Qty to dispatch (sum for all lines of batch detail if there is)
                 } else {
                     print '<td class="right">' . $langs->trans("QtyDispatchedShort") . '</td>';
                 }

public/htdocs/reception/list.php

Spacing +6 added lines, -6 removed lines

@@ -68,7 +68,7 @@ 
 $search_date_delivery_endday = GETPOSTINT('search_date_delivery_endday');
 $search_date_delivery_endmonth = GETPOSTINT('search_date_delivery_endmonth');
 $search_date_delivery_endyear = GETPOSTINT('search_date_delivery_endyear');
-$search_date_delivery_start = dol_mktime(0, 0, 0, $search_date_delivery_startmonth, $search_date_delivery_startday, $search_date_delivery_startyear);   // Use tzserver
+$search_date_delivery_start = dol_mktime(0, 0, 0, $search_date_delivery_startmonth, $search_date_delivery_startday, $search_date_delivery_startyear); // Use tzserver
 $search_date_delivery_end = dol_mktime(23, 59, 59, $search_date_delivery_endmonth, $search_date_delivery_endday, $search_date_delivery_endyear);
 $search_date_create_startday = GETPOSTINT('search_date_create_startday');
 $search_date_create_startmonth = GETPOSTINT('search_date_create_startmonth');
@@ -76,7 +76,7 @@ 
 $search_date_create_endday = GETPOSTINT('search_date_create_endday');
 $search_date_create_endmonth = GETPOSTINT('search_date_create_endmonth');
 $search_date_create_endyear = GETPOSTINT('search_date_create_endyear');
-$search_date_create_start = dol_mktime(0, 0, 0, $search_date_create_startmonth, $search_date_create_startday, $search_date_create_startyear);   // Use tzserver
+$search_date_create_start = dol_mktime(0, 0, 0, $search_date_create_startmonth, $search_date_create_startday, $search_date_create_startyear); // Use tzserver
 $search_date_create_end = dol_mktime(23, 59, 59, $search_date_create_endmonth, $search_date_create_endday, $search_date_create_endyear);
 $search_billed = GETPOST("search_billed", 'intcomma');
 $search_status = GETPOST('search_status', 'intcomma');
@@ -144,7 +144,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
 
 $error = 0;
 
@@ -336,7 +336,7 @@ 
                 // if the VAT reverse-charge is activated by default in supplier card to resume the information
                 $objecttmp->vat_reverse_charge = $soc->vat_reverse_charge;
 
-                $objecttmp->fk_project          = $rcp->fk_project;
+                $objecttmp->fk_project = $rcp->fk_project;
                 //$objecttmp->multicurrency_code = $rcp->multicurrency_code;
                 if (empty($createbills_onebythird)) {
                     $objecttmp->ref_supplier = $rcp->ref;
@@ -369,7 +369,7 @@ 
                     }
                 }
 
-                $res = $objecttmp->create($user);       // This should create the supplier invoice + links into $objecttmp->linked_objects + add a link to ->origin_id
+                $res = $objecttmp->create($user); // This should create the supplier invoice + links into $objecttmp->linked_objects + add a link to ->origin_id
 
                 //var_dump($objecttmp->error);exit;
                 if ($res > 0) {
@@ -576,7 +576,7 @@ 
             $db->rollback();
 
             $action = 'create';
-            $_GET["origin"] = $_POST["origin"];     // Keep this ?
+            $_GET["origin"] = $_POST["origin"]; // Keep this ?
             $_GET["originid"] = $_POST["originid"]; // Keep this ?
             setEventMessages($object->error, $errors, 'errors');
             $error++;

public/htdocs/reception/class/api_receptions.class.php

Indentation +12 added lines, -12 removed lines

@@ -45,7 +45,7 @@ 
     public $reception;
 
     /**
-        * Constructor
+     * Constructor
      */
     public function __construct()
     {
@@ -623,15 +623,15 @@ 
     */
 
     /**
-    * Close a reception (Classify it as "Delivered")
-    *
-    * @param    int     $id             Reception ID
-    * @param    int     $notrigger      Disabled triggers
-    *
-    * @url POST    {id}/close
-    *
-    * @return  Object
-    */
+     * Close a reception (Classify it as "Delivered")
+     *
+     * @param    int     $id             Reception ID
+     * @param    int     $notrigger      Disabled triggers
+     *
+     * @url POST    {id}/close
+     *
+     * @return  Object
+     */
     public function close($id, $notrigger = 0)
     {
         if (!DolibarrApiAccess::$user->hasRight('reception', 'creer')) {
@@ -663,7 +663,7 @@ 
         return $this->_cleanObjectDatas($this->reception);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      * Clean sensible object datas
      *
@@ -672,7 +672,7 @@ 
      */
     protected function _cleanObjectDatas($object)
     {
-		// phpcs:enable
+        // phpcs:enable
         $object = parent::_cleanObjectDatas($object);
 
         unset($object->thirdparty); // id already returned

Spacing +4 added lines, -4 removed lines

@@ -204,11 +204,11 @@
 
                 $receptionline->fk_product = $line['fk_product'];
                 $receptionline->fk_entrepot = $line['fk_entrepot'];
-                $receptionline->fk_element = $line['fk_element'] ?? $line['origin_id'];             // example: purchase order id.  this->origin is 'supplier_order'
+                $receptionline->fk_element = $line['fk_element'] ?? $line['origin_id']; // example: purchase order id.  this->origin is 'supplier_order'
                 $receptionline->origin_line_id = $line['fk_elementdet'] ?? $line['origin_line_id']; // example: purchase order id
-                $receptionline->fk_elementdet = $line['fk_elementdet'] ?? $line['origin_line_id'];  // example: purchase order line id
-                $receptionline->origin_type = $line['element_type'] ?? $line['origin_type'];        // example 'supplier_order'
-                $receptionline->element_type = $line['element_type'] ?? $line['origin_type'];       // example 'supplier_order'
+                $receptionline->fk_elementdet = $line['fk_elementdet'] ?? $line['origin_line_id']; // example: purchase order line id
+                $receptionline->origin_type = $line['element_type'] ?? $line['origin_type']; // example 'supplier_order'
+                $receptionline->element_type = $line['element_type'] ?? $line['origin_type']; // example 'supplier_order'
                 $receptionline->qty = $line['qty'];
                 //$receptionline->rang = $line['rang'];
                 $receptionline->array_options = $line['array_options'];

public/htdocs/reception/class/reception.class.php

Indentation +8 added lines, -8 removed lines

@@ -1181,7 +1181,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Load lines
      *
@@ -1189,7 +1189,7 @@ 
      */
     public function fetch_lines()
     {
-		// phpcs:enable
+        // phpcs:enable
         $this->lines = array();
 
         require_once constant('DOL_DOCUMENT_ROOT') . '/fourn/class/fournisseur.commande.dispatch.class.php';
@@ -1334,7 +1334,7 @@ 
         return $this->LibStatut($this->statut, $mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Return label of a status
      *
@@ -1344,7 +1344,7 @@ 
      */
     public function LibStatut($status, $mode)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         // List of long language codes for status
@@ -1510,7 +1510,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Fetch deliveries method and return an array. Load array this->meths(rowid=>label).
      *
@@ -1518,7 +1518,7 @@ 
      */
     public function fetch_delivery_methods()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
         $this->meths = array();
 
@@ -1536,7 +1536,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Fetch all deliveries method and return an array. Load array this->listmeths.
      *
@@ -1545,7 +1545,7 @@ 
      */
     public function list_delivery_methods($id = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         $this->listmeths = array();

Spacing +4 added lines, -4 removed lines

@@ -734,9 +734,9 @@ 
                 if ($this->origin_object instanceof CommonObject && empty($this->origin_object->lines)) {
                     $res = $this->origin_object->fetch_lines();
                     if ($this->origin_object instanceof CommandeFournisseur) {
-                        $this->commandeFournisseur = $this->origin_object;  // deprecated
+                        $this->commandeFournisseur = $this->origin_object; // deprecated
                     } else {
-                        $this->commandeFournisseur = null;  // deprecated
+                        $this->commandeFournisseur = null; // deprecated
                     }
                     if ($res < 0) {
                         return $res;
@@ -1358,7 +1358,7 @@ 
         if (getDolGlobalInt("STOCK_CALCULATE_ON_RECEPTION_CLOSE")) {
             $this->labelStatus[1]  = 'StatusReceptionValidatedToReceive';
         }
-        $this->labelStatus[2]  = 'StatusReceptionProcessed';
+        $this->labelStatus[2] = 'StatusReceptionProcessed';
 
         // List of short language codes for status
         $this->labelStatusShort[-1] = 'StatusReceptionCanceledShort';
@@ -1468,7 +1468,7 @@ 
         while ($xnbp < $nbp) {
             $line = new CommandeFournisseurDispatch($this->db);
             $line->desc = $langs->trans("Description") . " " . $xnbp;
-            $line->libelle = $langs->trans("Description") . " " . $xnbp;    // deprecated
+            $line->libelle = $langs->trans("Description") . " " . $xnbp; // deprecated
             $line->label = $langs->trans("Description") . " " . $xnbp;
             $line->qty = 10;
 

public/htdocs/reception/class/receptionlinebatch.class.php

Indentation +2 added lines, -2 removed lines

@@ -568,7 +568,7 @@ 
         return $this->LibStatut($this->status, $mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return label of a status
      *
@@ -578,7 +578,7 @@ 
      */
     public function LibStatut($status, $mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
         $langs->load('orders');
 

public/htdocs/website/websiteaccount_card.php

Spacing +3 added lines, -3 removed lines

@@ -91,9 +91,9 @@
 } else {
     $permissiontocreate = isModEnabled('website') && $user->hasRight('website', 'write') || isModEnabled('webportal') && $user->hasRight('webportal', 'write');
 }
-$permissionnote    = $permissiontocreate;   //  Used by the include of actions_setnotes.inc.php
-$permissiondellink = $permissiontocreate;   //  Used by the include of actions_dellink.inc.php
-$permissiontoadd   = $permissiontocreate;   //  Used by the include of actions_addupdatedelete.inc.php and actions_lineupdown.inc.php
+$permissionnote    = $permissiontocreate; //  Used by the include of actions_setnotes.inc.php
+$permissiondellink = $permissiontocreate; //  Used by the include of actions_dellink.inc.php
+$permissiontoadd   = $permissiontocreate; //  Used by the include of actions_addupdatedelete.inc.php and actions_lineupdown.inc.php
 
 // check access from type of site on create, edit, delete (other than view)
 $site_type_js = '';

public/htdocs/website/class/website.class.php

Indentation +2 added lines, -2 removed lines

@@ -826,7 +826,7 @@ 
         return $this->LibStatut($this->status, $mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return the label of a given status
      *
@@ -836,7 +836,7 @@ 
      */
     public function LibStatut($status, $mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         if (empty($this->labelStatus) || empty($this->labelStatusShort)) {

Spacing +4 added lines, -4 removed lines

@@ -1126,7 +1126,7 @@ 
             $stringtoexport = str_replace('medias/image/' . $website->ref . '/', "medias/image/__WEBSITE_KEY__/", $stringtoexport);
             $stringtoexport = str_replace('medias/js/' . $website->ref . '/', "medias/js/__WEBSITE_KEY__/", $stringtoexport);
             $stringtoexport = str_replace('"image/' . $website->ref . '/', '"image/__WEBSITE_KEY__/', $stringtoexport); // When we have a link src="image/websiteref/file.png" into html content
-            $stringtoexport = str_replace('"/image/' . $website->ref . '/', '"/image/__WEBSITE_KEY__/', $stringtoexport);   // When we have a link src="/image/websiteref/file.png" into html content
+            $stringtoexport = str_replace('"/image/' . $website->ref . '/', '"/image/__WEBSITE_KEY__/', $stringtoexport); // When we have a link src="/image/websiteref/file.png" into html content
             $stringtoexport = str_replace('"js/' . $website->ref . '/', '"js/__WEBSITE_KEY__/', $stringtoexport);
             $stringtoexport = str_replace('"/js/' . $website->ref . '/', '"/js/__WEBSITE_KEY__/', $stringtoexport);
 
@@ -1455,7 +1455,7 @@ 
                 $filetpl = $pathofwebsite . '/page' . $object->fk_default_home . '.tpl.php';
             }
             $filewrapper = $pathofwebsite . '/wrapper.php';
-            dolSaveIndexPage($pathofwebsite, $fileindex, $filetpl, $filewrapper, $object);  // This includes also a version of index.php into sublanguage directories
+            dolSaveIndexPage($pathofwebsite, $fileindex, $filetpl, $filewrapper, $object); // This includes also a version of index.php into sublanguage directories
         }
 
         if ($error) {
@@ -1901,7 +1901,7 @@ 
                 $destContent = file_get_contents($fileNeeded['fullname']);
 
                 $numOfPageDest = $this->extractNumberFromFilename($fileNeeded['name']);
-                $differences = $this->showDifferences($destContent, $sourceContent, array($numOfPageDest,$numOfPageSource));
+                $differences = $this->showDifferences($destContent, $sourceContent, array($numOfPageDest, $numOfPageSource));
                 $differences['file_destination'] = $fileNeeded;
             } else {
                 $differences = array();
@@ -1944,7 +1944,7 @@ 
         $linefound = array();
         $countNumPage = count($exceptNumPge);
 
-        for ($i = 0;$i < $countNumPage; $i++) {
+        for ($i = 0; $i < $countNumPage; $i++) {
             $linefound[$i] = array();
             $linefound[$i]['meta'] = '/content="' . preg_quote($exceptNumPge[$i], '/') . '" \/>/';
             $linefound[$i]['output'] = '/dolWebsiteOutput\(\$tmp, "html", ' . preg_quote($exceptNumPge[$i], '/') . '\);/';