alxarafe / alixar

public/htdocs/core/modules/export/export_tsv.modules.php

Indentation +14 added lines, -14 removed lines

@@ -151,7 +151,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *   Open output file
      *
@@ -161,7 +161,7 @@ 
      */
     public function open_file($file, $outputlangs)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         dol_syslog("ExportTsv::open_file file=" . $file);
@@ -179,7 +179,7 @@ 
         return $ret;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Output header into file
      *
@@ -188,12 +188,12 @@ 
      */
     public function write_header($outputlangs)
     {
-		// phpcs:enable
+        // phpcs:enable
         return 0;
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Output title line into file
      *
@@ -205,7 +205,7 @@ 
      */
     public function write_title($array_export_fields_label, $array_selected_sorted, $outputlangs, $array_types)
     {
-		// phpcs:enable
+        // phpcs:enable
         $selectlabel = array();
         foreach ($array_selected_sorted as $code => $value) {
             $newvalue = $outputlangs->transnoentities($array_export_fields_label[$code]); // newvalue is now $outputlangs->charset_output encoded
@@ -226,7 +226,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Output record line into file
      *
@@ -238,7 +238,7 @@ 
      */
     public function write_record($array_selected_sorted, $objp, $outputlangs, $array_types)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $this->col = 0;
@@ -284,7 +284,7 @@ 
         return 0;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Output footer into file
      *
@@ -293,11 +293,11 @@ 
      */
     public function write_footer($outputlangs)
     {
-		// phpcs:enable
+        // phpcs:enable
         return 0;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Close file handle
      *
@@ -305,12 +305,12 @@ 
      */
     public function close_file()
     {
-		// phpcs:enable
+        // phpcs:enable
         fclose($this->handle);
         return 0;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Clean a cell to respect rules of TSV file cells
      *
@@ -320,7 +320,7 @@ 
      */
     public function tsv_clean($newvalue, $charset)
     {
-		// phpcs:enable
+        // phpcs:enable
         // Rule Dolibarr: No HTML
         $newvalue = dol_string_nohtmltag($newvalue, 1, $charset);
 

public/htdocs/core/modules/export/exportcsv.class.php

Indentation +12 added lines, -12 removed lines

@@ -129,7 +129,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Open output file
      *
@@ -139,7 +139,7 @@ 
      */
     public function open_file($file, $outputlangs)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         dol_syslog("ExportCsv::open_file file=" . $file);
@@ -157,7 +157,7 @@ 
         return $ret;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Output header into file
      *
@@ -166,12 +166,12 @@ 
      */
     public function write_header($outputlangs)
     {
-		// phpcs:enable
+        // phpcs:enable
         return 0;
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Output title line into file
      *
@@ -183,7 +183,7 @@ 
      */
     public function write_title($array_export_fields_label, $array_selected_sorted, $outputlangs, $array_types)
     {
-		// phpcs:enable
+        // phpcs:enable
         $outputlangs->charset_output = getDolGlobalString('EXPORT_CSV_FORCE_CHARSET');
 
         $selectlabel = array();
@@ -207,7 +207,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Output record line into file
      *
@@ -219,7 +219,7 @@ 
      */
     public function write_record($array_selected_sorted, $objp, $outputlangs, $array_types)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $outputlangs->charset_output = getDolGlobalString('EXPORT_CSV_FORCE_CHARSET');
@@ -271,7 +271,7 @@ 
         return 0;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Output footer into file
      *
@@ -280,11 +280,11 @@ 
      */
     public function write_footer($outputlangs)
     {
-		// phpcs:enable
+        // phpcs:enable
         return 0;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Close file handle
      *
@@ -292,7 +292,7 @@ 
      */
     public function close_file()
     {
-		// phpcs:enable
+        // phpcs:enable
         fclose($this->handle);
         return 0;
     }

public/htdocs/core/modules/export/export_csviso.modules.php

Indentation +2 added lines, -2 removed lines

@@ -61,7 +61,7 @@ 
         $this->version_lib = DOL_VERSION;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Output title line into file
      *
@@ -79,7 +79,7 @@ 
         return parent::write_title($array_export_fields_label, $array_selected_sorted, $outputlangs, $array_types);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Output record line into file
      *

public/htdocs/core/modules/export/export_excel2007.modules.php

Indentation +14 added lines, -14 removed lines

@@ -186,7 +186,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Open output file
      *
@@ -196,7 +196,7 @@ 
      */
     public function open_file($file, $outputlangs)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $user, $langs;
 
         dol_syslog(get_class($this) . "::open_file file=" . $file);
@@ -232,7 +232,7 @@ 
         return $ret;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Write header
      *
@@ -241,14 +241,14 @@ 
      */
     public function write_header($outputlangs)
     {
-		// phpcs:enable
+        // phpcs:enable
         //$outputlangs->charset_output='ISO-8859-1';    // Because Excel 5 format is ISO
 
         return 0;
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Output title line into file
      *
@@ -260,7 +260,7 @@ 
      */
     public function write_title($array_export_fields_label, $array_selected_sorted, $outputlangs, $array_types)
     {
-		// phpcs:enable
+        // phpcs:enable
 
         // Create a format for the column headings
         $this->workbook->getActiveSheet()->getStyle('1')->getFont()->setBold(true);
@@ -301,7 +301,7 @@ 
         return 0;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Output record line into file
      *
@@ -313,7 +313,7 @@ 
      */
     public function write_record($array_selected_sorted, $objp, $outputlangs, $array_types)
     {
-		// phpcs:enable
+        // phpcs:enable
 
         // Define first row
         $this->col = 1;
@@ -411,7 +411,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Write footer
      *
@@ -420,12 +420,12 @@ 
      */
     public function write_footer($outputlangs)
     {
-		// phpcs:enable
+        // phpcs:enable
         return 0;
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Close Excel file
      *
@@ -433,7 +433,7 @@ 
      */
     public function close_file()
     {
-		// phpcs:enable
+        // phpcs:enable
 
         $objWriter = new Xlsx($this->workbook);
         $objWriter->save($this->file);
@@ -444,7 +444,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Clean a cell to respect rules of Excel file cells
      *
@@ -453,7 +453,7 @@ 
      */
     public function excel_clean($newvalue)
     {
-		// phpcs:enable
+        // phpcs:enable
         // Rule Dolibarr: No HTML
         $newvalue = dol_string_nohtmltag($newvalue);
 

public/htdocs/core/modules/member/doc/pdf_standard.class.php

Indentation +4 added lines, -4 removed lines

@@ -75,7 +75,7 @@ 
         // use this method in future refactoring
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Output a sticker on page at position _COUNTX, _COUNTY (_COUNTX and _COUNTY start from 0)
      * - __LOGO__ is replace with company logo
@@ -93,7 +93,7 @@ 
      */
     public function Add_PDF_card(&$pdf, $textleft, $header, $footer, $outputlangs, $textright = '', $idmember = 0, $photo = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $db, $mysoc, $conf, $langs;
         global $forceimgscalewidth, $forceimgscaleheight;
 
@@ -260,7 +260,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Function to build PDF on disk, then output on HTTP stream.
      *
@@ -274,7 +274,7 @@ 
      */
     public function write_file($object, $outputlangs, $srctemplatepath, $mode = 'member', $nooutput = 0, $filename = 'tmp_cards')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $user, $conf, $langs, $mysoc, $_Avery_Labels;
 
         $this->code = $srctemplatepath;

public/htdocs/core/modules/member/doc/doc_generic_member_odt.class.php

Indentation +4 added lines, -4 removed lines

@@ -183,7 +183,7 @@ 
         return $texte;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Function to build a document on disk using the generic odt module.
      *
@@ -197,7 +197,7 @@ 
      */
     public function write_file($object, $outputlangs, $srctemplatepath, $mode = 'member', $nooutput = 0, $filename = 'tmp_cards')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $user, $langs, $conf, $mysoc, $hookmanager;
 
         if (empty($srctemplatepath)) {
@@ -415,7 +415,7 @@ 
         return -1;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * get substitution array for object
      *
@@ -426,7 +426,7 @@ 
      */
     public function get_substitutionarray_object($object, $outputlangs, $array_key = 'object')
     {
-		// phpcs:enable
+        // phpcs:enable
         if (!$object instanceof Adherent) {
             dol_syslog("Expected Adherent object, got " . gettype($object), LOG_ERR);
             return array();

Spacing +1 added lines, -1 removed lines

@@ -167,7 +167,7 @@
         $maxfilesizearray = getMaxFileSizeArray();
         $maxmin = $maxfilesizearray['maxmin'];
         if ($maxmin > 0) {
-            $texte .= '<input type="hidden" name="MAX_FILE_SIZE" value="' . ($maxmin * 1024) . '">';    // MAX_FILE_SIZE must precede the field type=file
+            $texte .= '<input type="hidden" name="MAX_FILE_SIZE" value="' . ($maxmin * 1024) . '">'; // MAX_FILE_SIZE must precede the field type=file
         }
         $texte .= ' <input type="file" name="uploadfile">';
         $texte .= '<input type="hidden" value="MEMBER_ADDON_PDF_ODT_PATH" name="keyforuploaddir">';

public/htdocs/core/modules/member/modules_cards.php

Indentation +3 added lines, -3 removed lines

@@ -42,7 +42,7 @@ 
     public $error = '';
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return list of active generation modules
      *
@@ -52,7 +52,7 @@ 
      */
     public static function liste_modeles($db, $maxfilenamelength = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         $type = 'member';
         $list = array();
 
@@ -79,7 +79,7 @@ 
  */
 function members_card_pdf_create($db, $arrayofmembers, $modele, $outputlangs, $outputdir = '', $template = 'standard', $filename = 'tmp_cards')
 {
-	// phpcs:enable
+    // phpcs:enable
     global $conf, $langs;
     $langs->load("members");
 

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

Indentation +20 added lines, -20 removed lines

@@ -130,7 +130,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Output header of an example file for this format
      *
@@ -139,7 +139,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);
@@ -154,7 +154,7 @@ 
         return '';
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Output title line of an example file for this format
      *
@@ -164,7 +164,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);
@@ -180,7 +180,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
      *
@@ -190,7 +190,7 @@ 
      */
     public function write_record_example($outputlangs, $contentlinevalues)
     {
-		// phpcs:enable
+        // phpcs:enable
         $col = 1;
         $row = 2;
         foreach ($contentlinevalues as $cell) {
@@ -201,7 +201,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
      *
@@ -210,7 +210,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);
@@ -225,7 +225,7 @@ 
 
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Open input file
      *
@@ -234,7 +234,7 @@ 
      */
     public function import_open_file($file)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
         $ret = 1;
 
@@ -249,7 +249,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return nb of records. File must be closed.
      *
@@ -258,7 +258,7 @@ 
      */
     public function import_get_nb_of_lines($file)
     {
-		// phpcs:enable
+        // phpcs:enable
         $reader = new Xlsx();
         $this->workbook = $reader->load($file);
 
@@ -271,7 +271,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Input header line from file
      *
@@ -279,7 +279,7 @@ 
      */
     public function import_read_header()
     {
-		// phpcs:enable
+        // phpcs:enable
         // This is not called by the import code !!!
         $this->headers = array();
         $xlsx = new Xlsx();
@@ -292,7 +292,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return array of next record in input file.
      *
@@ -300,7 +300,7 @@ 
      */
     public function import_read_record()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $rowcount = $this->workbook->getActiveSheet()->getHighestDataRow();
@@ -320,7 +320,7 @@ 
         return $array;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Close file handle
      *
@@ -328,7 +328,7 @@ 
      */
     public function import_close_file()
     {
-		// phpcs:enable
+        // phpcs:enable
         $this->workbook->disconnectWorksheets();
         unset($this->workbook);
         return 0;
@@ -336,7 +336,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
      *
@@ -350,7 +350,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

Spacing +8 added lines, -8 removed lines

@@ -520,7 +520,7 @@ 
                                             //print 'We have made a '.$class.'->'.$method.' to get id from code '.$newval.'. ';
                                             if ($classinstance->id != '') { // id may be 0, it is a found value
                                                 $newval = $classinstance->id;
-                                            } elseif (! $error) {
+                                            } elseif (!$error) {
                                                 if (!empty($objimport->array_import_convertvalue[0][$val]['dict'])) {
                                                     // @phan-suppress-next-line PhanPluginSuspiciousParamPosition
                                                     $this->errors[$error]['lib'] = $langs->trans('ErrorFieldValueNotIn', $key, $newval, 'code', $langs->transnoentitiesnoconv($objimport->array_import_convertvalue[0][$val]['dict']));
@@ -697,7 +697,7 @@ 
                                     $classinstance = new $class($this->db);
                                     $res = call_user_func_array(array($classinstance, $method), array(&$arrayrecord, $arrayfield, $key));
                                     if (empty($classinstance->error) && empty($classinstance->errors)) {
-                                        $newval = $res;     // We get new value computed.
+                                        $newval = $res; // We get new value computed.
                                     } else {
                                         $this->errors[$error]['type'] = 'CLASSERROR';
                                         $this->errors[$error]['lib'] = implode(
@@ -798,11 +798,11 @@ 
                         if (isModEnabled("socialnetworks") && strpos($fieldname, "socialnetworks") !== false) {
                             if (!in_array("socialnetworks", $listfields)) {
                                 $listfields[] = "socialnetworks";
-                                $socialkey = array_search("socialnetworks", $listfields);   // Return position of 'socialnetworks' key in array. Example socialkey=19
+                                $socialkey = array_search("socialnetworks", $listfields); // Return position of 'socialnetworks' key in array. Example socialkey=19
                                 $listvalues[$socialkey] = '';
                             }
                             if (!empty($newval) && $arrayrecord[($key)]['type'] > 0) {
-                                $socialkey = array_search("socialnetworks", $listfields);   // Return position of 'socialnetworks' key in array. Example socialkey=19
+                                $socialkey = array_search("socialnetworks", $listfields); // Return position of 'socialnetworks' key in array. Example socialkey=19
                                 $socialnetwork = explode("_", $fieldname)[1];
                                 if (empty($listvalues[$socialkey]) || $listvalues[$socialkey] == "null") {
                                     $json = new stdClass();
@@ -928,7 +928,7 @@ 
 
                                 $data = array_combine($listfields, $listvalues);
 
-                                $where = array();   // filters to forge SQL request
+                                $where = array(); // filters to forge SQL request
                                 // @phpstan-ignore-next-line
                                 '@phan-var string[] $where';
                                 $filters = array(); // filters to forge output error message
@@ -1021,7 +1021,7 @@ 
                                 // We db escape social network field because he isn't in field creation
                                 if (in_array("socialnetworks", $listfields)) {
                                     $socialkey = array_search("socialnetworks", $listfields);
-                                    $tmpsql =  $listvalues[$socialkey];
+                                    $tmpsql = $listvalues[$socialkey];
                                     $listvalues[$socialkey] = "'" . $this->db->escape($tmpsql) . "'";
                                 }
 
@@ -1031,7 +1031,7 @@ 
                                 $data = array_combine($listfields, $listvalues);
                                 $set = array();
                                 foreach ($data as $key => $val) {
-                                    $set[] = $key . " = " . $val;   // $val was escaped/sanitized previously
+                                    $set[] = $key . " = " . $val; // $val was escaped/sanitized previously
                                 }
                                 $sqlstart .= " SET " . implode(', ', $set) . ", import_key = '" . $this->db->escape($importid) . "'";
 
@@ -1070,7 +1070,7 @@ 
                             // We db escape social network field because he isn't in field creation
                             if (in_array("socialnetworks", $listfields)) {
                                 $socialkey = array_search("socialnetworks", $listfields);
-                                $tmpsql =  $listvalues[$socialkey];
+                                $tmpsql = $listvalues[$socialkey];
                                 $listvalues[$socialkey] = "'" . $this->db->escape($tmpsql) . "'";
                             }
 

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

Indentation +20 added lines, -20 removed lines

@@ -115,7 +115,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Output header of an example file for this format
      *
@@ -124,11 +124,11 @@ 
      */
     public function write_header_example($outputlangs)
     {
-		// phpcs:enable
+        // phpcs:enable
         return '';
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Output title line of an example file for this format
      *
@@ -138,12 +138,12 @@ 
      */
     public function write_title_example($outputlangs, $headerlinefields)
     {
-		// phpcs:enable
+        // phpcs:enable
         $s = implode($this->separator, array_map('cleansep', $headerlinefields));
         return $s . "\n";
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Output record of an example file for this format
      *
@@ -153,12 +153,12 @@ 
      */
     public function write_record_example($outputlangs, $contentlinevalues)
     {
-		// phpcs:enable
+        // phpcs:enable
         $s = implode($this->separator, array_map('cleansep', $contentlinevalues));
         return $s . "\n";
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Output footer of an example file for this format
      *
@@ -167,12 +167,12 @@ 
      */
     public function write_footer_example($outputlangs)
     {
-		// phpcs:enable
+        // phpcs:enable
         return '';
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Open input file
      *
@@ -181,7 +181,7 @@ 
      */
     public function import_open_file($file)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
         $ret = 1;
 
@@ -202,7 +202,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return nb of records. File must be closed.
      *
@@ -211,12 +211,12 @@ 
      */
     public function import_get_nb_of_lines($file)
     {
-		// phpcs:enable
+        // phpcs:enable
         return dol_count_nb_of_line($file);
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Input header line from file
      *
@@ -224,12 +224,12 @@ 
      */
     public function import_read_header()
     {
-		// phpcs:enable
+        // phpcs:enable
         return 0;
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return array of next record in input file.
      *
@@ -237,7 +237,7 @@ 
      */
     public function import_read_record()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $arrayres = fgetcsv($this->handle, 100000, $this->separator, $this->enclosure, $this->escape);
@@ -277,7 +277,7 @@ 
         return $newarrayres;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Close file handle
      *
@@ -285,13 +285,13 @@ 
      */
     public function import_close_file()
     {
-		// phpcs:enable
+        // phpcs:enable
         fclose($this->handle);
         return 0;
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Insert a record into database
      *
@@ -305,7 +305,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

Spacing +8 added lines, -8 removed lines

@@ -476,7 +476,7 @@ 
                                             //print 'We have made a '.$class.'->'.$method.' to get id from code '.$newval.'. ';
                                             if ($classinstance->id != '') { // id may be 0, it is a found value
                                                 $newval = $classinstance->id;
-                                            } elseif (! $error) {
+                                            } elseif (!$error) {
                                                 if (!empty($objimport->array_import_convertvalue[0][$val]['dict'])) {
                                                     $this->errors[$error]['lib'] = $langs->trans('ErrorFieldValueNotIn', num2Alpha($key - 1), $newval, 'code', $langs->transnoentitiesnoconv($objimport->array_import_convertvalue[0][$val]['dict']));
                                                 } elseif (!empty($objimport->array_import_convertvalue[0][$val]['element'])) {
@@ -649,7 +649,7 @@ 
                                     $classinstance = new $class($this->db);
                                     $res = call_user_func_array(array($classinstance, $method), array(&$arrayrecord, $arrayfield, ($key - 1)));
                                     if (empty($classinstance->error) && empty($classinstance->errors)) {
-                                        $newval = $res;     // We get new value computed.
+                                        $newval = $res; // We get new value computed.
                                     } else {
                                         $this->errors[$error]['type'] = 'CLASSERROR';
                                         $this->errors[$error]['lib'] = implode(
@@ -747,12 +747,12 @@ 
                         if (isModEnabled("socialnetworks") && strpos($fieldname, "socialnetworks") !== false) {
                             if (!in_array("socialnetworks", $listfields)) {
                                 $listfields[] = "socialnetworks";
-                                $socialkey = array_search("socialnetworks", $listfields);   // Return position of 'socialnetworks' key in array
+                                $socialkey = array_search("socialnetworks", $listfields); // Return position of 'socialnetworks' key in array
                                 $listvalues[$socialkey] = '';
                             }
                             //var_dump($newval); var_dump($arrayrecord[($key - 1)]['type']);
                             if (!empty($newval) && $arrayrecord[($key - 1)]['type'] > 0) {
-                                $socialkey = array_search("socialnetworks", $listfields);   // Return position of 'socialnetworks' key in array
+                                $socialkey = array_search("socialnetworks", $listfields); // Return position of 'socialnetworks' key in array
                                 //var_dump('sk='.$socialkey);   // socialkey=19
                                 $socialnetwork = explode("_", $fieldname)[1];
                                 if (empty($listvalues[$socialkey]) || $listvalues[$socialkey] == "null") {
@@ -875,7 +875,7 @@ 
                             if (empty($lastinsertid)) { // No insert done yet for a parent table
                                 $sqlSelect = "SELECT " . $fname . " FROM " . $tablename;
                                 $data = array_combine($listfields, $listvalues);
-                                $where = array();   // filters to forge SQL request
+                                $where = array(); // filters to forge SQL request
                                 $filters = array(); // filters to forge output error message
                                 foreach ($updatekeys as $key) {
                                     $col = $objimport->array_import_updatekeys[0][$key];
@@ -965,7 +965,7 @@ 
                                 // We db escape social network field because he isn't in field creation
                                 if (in_array("socialnetworks", $listfields)) {
                                     $socialkey = array_search("socialnetworks", $listfields);
-                                    $tmpsql =  $listvalues[$socialkey];
+                                    $tmpsql = $listvalues[$socialkey];
                                     $listvalues[$socialkey] = "'" . $this->db->escape($tmpsql) . "'";
                                 }
 
@@ -975,7 +975,7 @@ 
                                 $data = array_combine($listfields, $listvalues);
                                 $set = array();
                                 foreach ($data as $key => $val) {
-                                    $set[] = $key . " = " . $val;   // $val was escaped/sanitized previously
+                                    $set[] = $key . " = " . $val; // $val was escaped/sanitized previously
                                 }
                                 $sqlstart .= " SET " . implode(', ', $set) . ", import_key = '" . $this->db->escape($importid) . "'";
 
@@ -1014,7 +1014,7 @@ 
                             // We db escape social network field because he isn't in field creation
                             if (in_array("socialnetworks", $listfields)) {
                                 $socialkey = array_search("socialnetworks", $listfields);
-                                $tmpsql =  $listvalues[$socialkey];
+                                $tmpsql = $listvalues[$socialkey];
                                 $listvalues[$socialkey] = "'" . $this->db->escape($tmpsql) . "'";
                             }