alxarafe / alixar

public/htdocs/knowledgemanagement/knowledgerecord_card.php

Spacing +1 added lines, -1 removed lines

@@ -426,7 +426,7 @@
     print '<table class="border centpercent tableforfield">' . "\n";
 
     // Common attributes
-    $keyforbreak = 'fk_c_ticket_category';    // We change column just before this field
+    $keyforbreak = 'fk_c_ticket_category'; // We change column just before this field
     //unset($object->fields['fk_project']);             // Hide field already shown in banner
     //unset($object->fields['fk_soc']);                 // Hide field already shown in banner
     $object->fields['answer']['enabled'] = 0;

public/htdocs/hrm/evaluation_card.php

Spacing +3 added lines, -3 removed lines

@@ -45,7 +45,7 @@ 
 require_once constant('DOL_DOCUMENT_ROOT') . '/hrm/lib/hrm_skillrank.lib.php';
 
 // Load translation files required by the page
-$langs->loadLangs(array('hrm', 'other', 'products'));  // why products?
+$langs->loadLangs(array('hrm', 'other', 'products')); // why products?
 
 // Get parameters
 $id = GETPOSTINT('id');
@@ -175,7 +175,7 @@ 
     if ($action == 'close') {
         // save evaldet lines to user;
         $sk = new SkillRank($db);
-        $SkillrecordsForActiveUser = $sk->fetchAll('ASC', 'fk_skill', 0, 0, "(fk_object:=:" . ((int)$object->fk_user) . ") AND (objecttype:=:'" . $db->escape(SkillRank::SKILLRANK_TYPE_USER) . "')", 'AND');
+        $SkillrecordsForActiveUser = $sk->fetchAll('ASC', 'fk_skill', 0, 0, "(fk_object:=:" . ((int) $object->fk_user) . ") AND (objecttype:=:'" . $db->escape(SkillRank::SKILLRANK_TYPE_USER) . "')", 'AND');
 
         $errors = 0;
         // we go through the evaldets of the eval
@@ -550,7 +550,7 @@ 
         $sql .= '  INNER JOIN ' . MAIN_DB_PREFIX . 'hrm_skilldet as skdet_user ON (skdet_user.fk_skill = sk.rowid AND skdet_user.rankorder = ed.rankorder)';
         //$sql .= "  LEFT JOIN " . MAIN_DB_PREFIX . "hrm_skillrank as skr ON (j.rowid = skr.fk_object AND skr.fk_skill = ed.fk_skill AND skr.objecttype = 'job')";
         $sql .= '  LEFT JOIN ' . MAIN_DB_PREFIX . 'hrm_skilldet as skdet_required ON (skdet_required.fk_skill = sk.rowid AND skdet_required.rankorder = ed.required_rank)';
-        $sql .= " WHERE e.rowid =" . ((int)$object->id);
+        $sql .= " WHERE e.rowid =" . ((int) $object->id);
 
         //      echo $sql;
 

public/htdocs/hrm/job_card.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
 require_once constant('DOL_DOCUMENT_ROOT') . '/hrm/lib/hrm_job.lib.php';
 
 // Load translation files required by the page
-$langs->loadLangs(array('hrm', 'other', 'products'));   // why products?
+$langs->loadLangs(array('hrm', 'other', 'products')); // why products?
 
 // Get parameters
 $id = GETPOSTINT('id');

public/htdocs/hrm/skill_card.php

Spacing +5 added lines, -5 removed lines

@@ -41,7 +41,7 @@ 
 require_once constant('DOL_DOCUMENT_ROOT') . '/hrm/lib/hrm_skill.lib.php';
 
 // Load translation files required by the page
-$langs->loadLangs(array('hrm', 'other', 'products'));  // why products?
+$langs->loadLangs(array('hrm', 'other', 'products')); // why products?
 
 // Get parameters
 $id = GETPOSTINT('id');
@@ -529,7 +529,7 @@ 
 
     // Default sort order (if not yet defined by previous GETPOST)
     if (!$sortfield) {
-        reset($objectline->fields);                    // Reset is required to avoid key() to return null.
+        reset($objectline->fields); // Reset is required to avoid key() to return null.
         $sortfield = "t." . key($objectline->fields); // Set here default search field. By default 1st field in definition.
     }
     if (!$sortorder) {
@@ -562,11 +562,11 @@ 
     foreach ($objectline->fields as $key => $val) {
         // If $val['visible']==0, then we never show the field
         if (!empty($val['visible'])) {
-            $visible = (int)dol_eval($val['visible'], 1, 1, '1');
+            $visible = (int) dol_eval($val['visible'], 1, 1, '1');
             $arrayfields['t.' . $key] = array(
                 'label' => $val['label'],
                 'checked' => (($visible < 0) ? 0 : 1),
-                'enabled' => (abs($visible) != 3 && (int)dol_eval($val['enabled'], 1, 1, '1')),
+                'enabled' => (abs($visible) != 3 && (int) dol_eval($val['enabled'], 1, 1, '1')),
                 'position' => $val['position'],
                 'help' => isset($val['help']) ? $val['help'] : ''
             );
@@ -601,7 +601,7 @@ 
     } else {
         $sql .= " WHERE 1 = 1 ";
     }
-    $sql .= " AND fk_skill = " . ((int)$id);
+    $sql .= " AND fk_skill = " . ((int) $id);
 
     $resql = $db->query($sql);
     $nbtotalofrecords = $db->num_rows($resql);

public/htdocs/imports/emptyexample.php

Spacing +2 added lines, -2 removed lines

@@ -99,8 +99,8 @@
 }
 
 // List of targets fields
-$headerlinefields = array();    // Array of fields (label to show)
-$contentlinevalues = array();   // Array of example values
+$headerlinefields = array(); // Array of fields (label to show)
+$contentlinevalues = array(); // Array of example values
 $i = 0;
 foreach ($fieldstarget as $code => $label) {
     $withoutstar = preg_replace('/\*/', '', $fieldstarget[$code]);

public/htdocs/imports/import.php

Spacing +18 added lines, -18 removed lines

@@ -148,7 +148,7 @@ 
 $endatlinenb = (GETPOST('endatlinenb') ? GETPOST('endatlinenb') : '');
 $updatekeys = (GETPOST('updatekeys', 'array') ? GETPOST('updatekeys', 'array') : array());
 $separator = (GETPOST('separator', 'nohtml') ? GETPOST('separator', 'nohtml', 3) : '');
-$enclosure = (GETPOST('enclosure', 'nohtml') ? GETPOST('enclosure', 'nohtml') : '"');  // We must use 'nohtml' and not 'alphanohtml' because we must accept "
+$enclosure = (GETPOST('enclosure', 'nohtml') ? GETPOST('enclosure', 'nohtml') : '"'); // We must use 'nohtml' and not 'alphanohtml' because we must accept "
 $charset = GETPOST('charset', 'aZ09');
 $separator_used = str_replace('\t', "\t", $separator);
 
@@ -615,7 +615,7 @@ 
     $maxfilesizearray = getMaxFileSizeArray();
     $maxmin = $maxfilesizearray['maxmin'];
     if ($maxmin > 0) {
-        print '<input type="hidden" name="MAX_FILE_SIZE" value="' . ($maxmin * 1024) . '">';    // MAX_FILE_SIZE must precede the field type=file
+        print '<input type="hidden" name="MAX_FILE_SIZE" value="' . ($maxmin * 1024) . '">'; // MAX_FILE_SIZE must precede the field type=file
     }
     print '<input type="file" name="userfile" size="20" maxlength="80"> &nbsp; &nbsp; ';
     $out = (!getDolGlobalString('MAIN_UPLOAD_DOC') ? ' disabled' : '');
@@ -625,29 +625,29 @@ 
         $max = getDolGlobalString('MAIN_UPLOAD_DOC'); // In Kb
         $maxphp = @ini_get('upload_max_filesize'); // In unknown
         if (preg_match('/k$/i', $maxphp)) {
-            $maxphp = (int)substr($maxphp, 0, -1);
+            $maxphp = (int) substr($maxphp, 0, -1);
         }
         if (preg_match('/m$/i', $maxphp)) {
-            $maxphp = (int)substr($maxphp, 0, -1) * 1024;
+            $maxphp = (int) substr($maxphp, 0, -1) * 1024;
         }
         if (preg_match('/g$/i', $maxphp)) {
-            $maxphp = (int)substr($maxphp, 0, -1) * 1024 * 1024;
+            $maxphp = (int) substr($maxphp, 0, -1) * 1024 * 1024;
         }
         if (preg_match('/t$/i', $maxphp)) {
-            $maxphp = (int)substr($maxphp, 0, -1) * 1024 * 1024 * 1024;
+            $maxphp = (int) substr($maxphp, 0, -1) * 1024 * 1024 * 1024;
         }
         $maxphp2 = @ini_get('post_max_size'); // In unknown
         if (preg_match('/k$/i', $maxphp2)) {
-            $maxphp2 = (int)substr($maxphp2, 0, -1);
+            $maxphp2 = (int) substr($maxphp2, 0, -1);
         }
         if (preg_match('/m$/i', $maxphp2)) {
-            $maxphp2 = (int)substr($maxphp2, 0, -1) * 1024;
+            $maxphp2 = (int) substr($maxphp2, 0, -1) * 1024;
         }
         if (preg_match('/g$/i', $maxphp2)) {
-            $maxphp2 = (int)substr($maxphp2, 0, -1) * 1024 * 1024;
+            $maxphp2 = (int) substr($maxphp2, 0, -1) * 1024 * 1024;
         }
         if (preg_match('/t$/i', $maxphp2)) {
-            $maxphp2 = (int)substr($maxphp2, 0, -1) * 1024 * 1024 * 1024;
+            $maxphp2 = (int) substr($maxphp2, 0, -1) * 1024 * 1024 * 1024;
         }
         // Now $max and $maxphp and $maxphp2 are in Kb
         $maxmin = $max;
@@ -734,7 +734,7 @@ 
     //var_dump($_SESSION["dol_array_match_file_to_database_select"]);
     $serialized_array_match_file_to_database = isset($_SESSION["dol_array_match_file_to_database_select"]) ? $_SESSION["dol_array_match_file_to_database_select"] : '';
     $fieldsarray = explode(',', $serialized_array_match_file_to_database);
-    $array_match_file_to_database = array();        // Same than $fieldsarray but with mapped value only  (col1 => 's.fielda', col2 => 's.fieldb'...)
+    $array_match_file_to_database = array(); // Same than $fieldsarray but with mapped value only  (col1 => 's.fielda', col2 => 's.fieldb'...)
     foreach ($fieldsarray as $elem) {
         $tabelem = explode('=', $elem, 2);
         $key = $tabelem[0];
@@ -1088,7 +1088,7 @@ 
     foreach ($fieldstarget as $code => $line) {
         //var_dump($line);
 
-        $tmparray = explode('|', $line["label"]);   // If label of field is several translation keys separated with |
+        $tmparray = explode('|', $line["label"]); // If label of field is several translation keys separated with |
         $labeltoshow = '';
         foreach ($tmparray as $tmpkey => $tmpval) {
             $labeltoshow .= ($labeltoshow ? ' ' . $langs->trans('or') . ' ' : '') . $langs->transnoentities($tmpval);
@@ -1118,7 +1118,7 @@ 
     //var_dump($fieldssource);
     //var_dump($fieldstarget);
 
-    $modetoautofillmapping = 'session';     // Use setup in session
+    $modetoautofillmapping = 'session'; // Use setup in session
     if ($initialloadofstep4) {
         $modetoautofillmapping = 'guess';
     }
@@ -1250,10 +1250,10 @@ 
             } elseif ($modetoautofillmapping == 'session' && !empty($_SESSION['dol_array_match_file_to_database_select'])) {
                 $tmpselectioninsession = dolExplodeIntoArray($_SESSION['dol_array_match_file_to_database_select'], ',', '=');
                 //var_dump($code);
-                if (!empty($tmpselectioninsession[(string)($i + 1)]) && $tmpselectioninsession[(string)($i + 1)] == $tmpcode) {
+                if (!empty($tmpselectioninsession[(string) ($i + 1)]) && $tmpselectioninsession[(string) ($i + 1)] == $tmpcode) {
                     $selectforline .= ' selected';
                 }
-                $selectforline .= ' data-debug="' . $tmpcode . '-' . $code . '-' . $j . '-' . (!empty($tmpselectioninsession[(string)($i + 1)]) ? $tmpselectioninsession[(string)($i + 1)] : "") . '"';
+                $selectforline .= ' data-debug="' . $tmpcode . '-' . $code . '-' . $j . '-' . (!empty($tmpselectioninsession[(string) ($i + 1)]) ? $tmpselectioninsession[(string) ($i + 1)] : "") . '"';
             }
             $selectforline .= ' data-html="' . dol_escape_htmltag($labelhtml) . '"';
             $selectforline .= '>';
@@ -1330,7 +1330,7 @@ 
         // - Then we set to disabled all fields that are selected
         print 'function setOptionsToDisabled() {' . "\n";
         print '		console.log("Remove the disabled flag everywhere");' . "\n";
-        print '		$("select.targetselectchange").not($( this )).find(\'option\').prop("disabled", false);' . "\n";  // Enable all options
+        print '		$("select.targetselectchange").not($( this )).find(\'option\').prop("disabled", false);' . "\n"; // Enable all options
         print '		arrayofselectedvalues = [];' . "\n";
 
         print '		$("select.targetselectchange").each(function(){' . "\n";
@@ -1347,7 +1347,7 @@ 
         print '     $.each(arrayofselectedvalues, function(key, value) {' . "\n"; // Loop on each selected value
         print '         if (value != -1) {' . "\n";
         print '     		console.log("Process key="+key+" value="+value+" to disable.");' . "\n";
-        print '				$("select.targetselectchange").find(\'option[value="\'+value+\'"]:not(:selected)\').prop("disabled", true);' . "\n";  // Set to disabled except if currently selected
+        print '				$("select.targetselectchange").find(\'option[value="\'+value+\'"]:not(:selected)\').prop("disabled", true);' . "\n"; // Set to disabled except if currently selected
         print '         }' . "\n";
         print '     });' . "\n";
         print '}' . "\n";
@@ -1490,7 +1490,7 @@ 
         $sql .= " FROM " . MAIN_DB_PREFIX . "import_model";
         $sql .= " WHERE type = '" . $db->escape($datatoimport) . "'";
         if (!getDolGlobalString('EXPORTS_SHARE_MODELS')) {  // EXPORTS_SHARE_MODELS means all templates are visible, whatever is owner.
-            $sql .= " AND fk_user IN (0, " . ((int)$user->id) . ")";
+            $sql .= " AND fk_user IN (0, " . ((int) $user->id) . ")";
         }
         $sql .= " ORDER BY rowid";
 

public/htdocs/fichinter/card.php

Spacing +3 added lines, -3 removed lines

@@ -387,7 +387,7 @@ 
                                                     $mult = 3600 * 24 * 7;
                                                     break;
                                                 case 'm':
-                                                    $mult = (int)3600 * 24 * (365 / 12); // Average month duration
+                                                    $mult = (int) 3600 * 24 * (365 / 12); // Average month duration
                                                     break;
                                                 case 'y':
                                                     $mult = 3600 * 24 * 365;
@@ -1076,7 +1076,7 @@ 
     } else {
         print '<form name="fichinter" action="' . $_SERVER['PHP_SELF'] . '" method="POST">';
         print '<input type="hidden" name="token" value="' . newToken() . '">';
-        print '<input type="hidden" name="action" value="create">';     // We go back to create action
+        print '<input type="hidden" name="action" value="create">'; // We go back to create action
         print '<input type="hidden" name="backtopage" value="' . $backtopage . '">';
 
         print dol_get_fiche_head('');
@@ -1365,7 +1365,7 @@ 
         $sql = 'SELECT ft.rowid, ft.description, ft.fk_fichinter, ft.duree, ft.rang,';
         $sql .= ' ft.date as date_intervention';
         $sql .= ' FROM ' . MAIN_DB_PREFIX . 'fichinterdet as ft';
-        $sql .= ' WHERE ft.fk_fichinter = ' . ((int)$object->id);
+        $sql .= ' WHERE ft.fk_fichinter = ' . ((int) $object->id);
         if (getDolGlobalString('FICHINTER_HIDE_EMPTY_DURATION')) {
             $sql .= ' AND ft.duree <> 0';
         }

public/htdocs/mrp/mo_card.php

Spacing +2 added lines, -2 removed lines

@@ -162,7 +162,7 @@ 
         $objectbomchildline = new BOMLine($db);
 
         foreach ($TBomLineId as $id_bom_line) {
-            $object = new Mo($db);  // modified by the actions_addupdatedelete.inc.php
+            $object = new Mo($db); // modified by the actions_addupdatedelete.inc.php
 
             $objectbomchildline->fetch($id_bom_line);
 
@@ -180,7 +180,7 @@ 
             $res = $object->add_object_linked('mo', $mo_parent->id);
         }
 
-        header("Location: " . dol_buildpath('/mrp/mo_card.php?id=' . ((int)$mo_parent->id), 1));
+        header("Location: " . dol_buildpath('/mrp/mo_card.php?id=' . ((int) $mo_parent->id), 1));
         exit;
     } elseif ($action == 'confirm_cancel' && $confirm == 'yes' && !empty($permissiontoadd)) {
         $also_cancel_consumed_and_produced_lines = (GETPOST('alsoCancelConsumedAndProducedLines', 'alpha') ? 1 : 0);

public/htdocs/core/modules/import/import_xlsx.modules.php

Indentation +20 added lines, -20 removed lines

@@ -132,7 +132,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Output header of an example file for this format
      *
@@ -141,7 +141,7 @@ 
      */
     public function write_header_example($outputlangs)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $user, $conf, $langs, $file;
         // create a temporary object, the final output will be generated in footer
         $this->workbook->getProperties()->setCreator($user->getFullName($outputlangs) . ' - Dolibarr ' . DOL_VERSION);
@@ -156,7 +156,7 @@ 
         return '';
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Output title line of an example file for this format
      *
@@ -166,7 +166,7 @@ 
      */
     public function write_title_example($outputlangs, $headerlinefields)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
         $this->workbook->getActiveSheet()->getStyle('1')->getFont()->setBold(true);
         $this->workbook->getActiveSheet()->getStyle('1')->getAlignment()->setHorizontal(Alignment::HORIZONTAL_LEFT);
@@ -182,7 +182,7 @@ 
         return ''; // final output will be generated in footer
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Output record of an example file for this format
      *
@@ -192,7 +192,7 @@ 
      */
     public function write_record_example($outputlangs, $contentlinevalues)
     {
-		// phpcs:enable
+        // phpcs:enable
         $col = 1;
         $row = 2;
         foreach ($contentlinevalues as $cell) {
@@ -203,7 +203,7 @@ 
         return ''; // final output will be generated in footer
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Output footer of an example file for this format
      *
@@ -212,7 +212,7 @@ 
      */
     public function write_footer_example($outputlangs)
     {
-		// phpcs:enable
+        // phpcs:enable
         // return the file content as a string
         $tempfile = tempnam(sys_get_temp_dir(), 'dol');
         $objWriter = new PhpOffice\PhpSpreadsheet\Writer\Xlsx($this->workbook);
@@ -227,7 +227,7 @@ 
 
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Open input file
      *
@@ -236,7 +236,7 @@ 
      */
     public function import_open_file($file)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
         $ret = 1;
 
@@ -251,7 +251,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return nb of records. File must be closed.
      *
@@ -260,7 +260,7 @@ 
      */
     public function import_get_nb_of_lines($file)
     {
-		// phpcs:enable
+        // phpcs:enable
         $reader = new Xlsx();
         $this->workbook = $reader->load($file);
 
@@ -273,7 +273,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Input header line from file
      *
@@ -281,7 +281,7 @@ 
      */
     public function import_read_header()
     {
-		// phpcs:enable
+        // phpcs:enable
         // This is not called by the import code !!!
         $this->headers = array();
         $xlsx = new Xlsx();
@@ -294,7 +294,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return array of next record in input file.
      *
@@ -302,7 +302,7 @@ 
      */
     public function import_read_record()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $rowcount = $this->workbook->getActiveSheet()->getHighestDataRow();
@@ -322,7 +322,7 @@ 
         return $array;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Close file handle
      *
@@ -330,7 +330,7 @@ 
      */
     public function import_close_file()
     {
-		// phpcs:enable
+        // phpcs:enable
         $this->workbook->disconnectWorksheets();
         unset($this->workbook);
         return 0;
@@ -338,7 +338,7 @@ 
 
 
     // What is this doing here ? it is common to all imports, is should be in the parent class
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Insert a record into database
      *
@@ -352,7 +352,7 @@ 
      */
     public function import_insert($arrayrecord, $array_match_file_to_database, $objimport, $maxfields, $importid, $updatekeys)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs, $conf, $user;
         global $thirdparty_static; // Specific to thirdparty import
         global $tablewithentity_cache; // Cache to avoid to call  desc at each rows on tables