alxarafe / alixar

dolibarr/dev/translation/sanity_check_en_langfiles.php

Indentation +179 added lines, -179 removed lines

@@ -107,8 +107,8 @@ 
 $files = scandir($workdir);
 if (empty($files))
 {
-	echo "Can't scan workdir = ".$workdir;
-	exit;
+    echo "Can't scan workdir = ".$workdir;
+    exit;
 }
 
 $dups=array();
@@ -122,38 +122,38 @@ 
 $langstrings_3d = array();
 $langstrings_full = array();
 foreach ($files AS $file) {
-	$path_file = pathinfo($file);
-	// we're only interested in .lang files
-	if ($path_file['extension']=='lang') {
-		$content = file($workdir.$file);
-		foreach ($content AS $line => $row) {
-			// don't want comment lines
-			if (substr($row,0,1) !== '#') {
-				// don't want lines without the separator (why should those even be here, anyway...)
-				if (strpos($row,'=')!==false) {
-					$row_array = explode('=',$row);		// $row_array[0] = key
-					$langstrings_3d[$path_file['basename']][$line+1]=$row_array[0];
-					$langstrings_3dtrans[$path_file['basename']][$line+1]=$row_array[1];
-					$langstrings_full[]=$row_array[0];
-					$langstrings_dist[$row_array[0]]=$row;
-				}
-			}
-		}
-	}
+    $path_file = pathinfo($file);
+    // we're only interested in .lang files
+    if ($path_file['extension']=='lang') {
+        $content = file($workdir.$file);
+        foreach ($content AS $line => $row) {
+            // don't want comment lines
+            if (substr($row,0,1) !== '#') {
+                // don't want lines without the separator (why should those even be here, anyway...)
+                if (strpos($row,'=')!==false) {
+                    $row_array = explode('=',$row);		// $row_array[0] = key
+                    $langstrings_3d[$path_file['basename']][$line+1]=$row_array[0];
+                    $langstrings_3dtrans[$path_file['basename']][$line+1]=$row_array[1];
+                    $langstrings_full[]=$row_array[0];
+                    $langstrings_dist[$row_array[0]]=$row;
+                }
+            }
+        }
+    }
 }
 
 foreach ($langstrings_3d AS $filename => $file)
 {
-	foreach ($file AS $linenum => $value)
-	{
-		$keys = array_keys($langstrings_full, $value);
-		if (count($keys)>1)
-		{
-				foreach ($keys AS $key) {
-					$dups[$value][$filename][$linenum] = trim($langstrings_3dtrans[$filename][$linenum]);
-				}
-		}
-	}
+    foreach ($file AS $linenum => $value)
+    {
+        $keys = array_keys($langstrings_full, $value);
+        if (count($keys)>1)
+        {
+                foreach ($keys AS $key) {
+                    $dups[$value][$filename][$linenum] = trim($langstrings_3dtrans[$filename][$linenum]);
+                }
+        }
+    }
 }
 
 if ($web) print "<h2>";
@@ -175,49 +175,49 @@ 
 $count = 0;
 foreach ($dups as $string => $pages)
 {
-	$count++;
-	$s='';
-
-	// Keyword $string
-	if ($web) $s.="<tr>";
-	if ($web) $s.="<td align=\"center\">";
-	if ($web) $s.=$count;
-	if ($web) $s.="</td>";
-	if ($web) $s.="<td>";
-	$s.=$string;
-	if ($web) $s.="</td>";
-	if ($web) $s.="<td>";
-	if (! $web) $s.= ' : ';
-
-	// Loop on each files keyword was found
+    $count++;
+    $s='';
+
+    // Keyword $string
+    if ($web) $s.="<tr>";
+    if ($web) $s.="<td align=\"center\">";
+    if ($web) $s.=$count;
+    if ($web) $s.="</td>";
+    if ($web) $s.="<td>";
+    $s.=$string;
+    if ($web) $s.="</td>";
+    if ($web) $s.="<td>";
+    if (! $web) $s.= ' : ';
+
+    // Loop on each files keyword was found
     $duplicateinsamefile=0;
-	$inmain=0;
+    $inmain=0;
     $inadmin=0;
-	foreach ($pages AS $file => $lines)
-	{
+    foreach ($pages AS $file => $lines)
+    {
         if ($file == 'main.lang') { $inmain=1; $inadmin=0; }
         if ($file == 'admin.lang' && ! $inmain) { $inadmin=1; }
 
-	    $s.=$file." ";
+        $s.=$file." ";
 
-	    // Loop on each line keword was found into file.
-	    $listoffilesforthisentry=array();
-	    foreach ($lines as $line => $translatedvalue)
-		{
+        // Loop on each line keword was found into file.
+        $listoffilesforthisentry=array();
+        foreach ($lines as $line => $translatedvalue)
+        {
             if (! empty($listoffilesforthisentry[$file])) $duplicateinsamefile=1;
             $listoffilesforthisentry[$file]=1;
 
-			$s.= "(".$line." - ".htmlentities($translatedvalue).") ";
-		}
-		if ($web) $s.="<br>";
-	}
-	if ($web) $s.="</td></tr>";
-	$s.="\n";
-
-	if ($duplicateinsamefile) $sduplicateinsamefile .= $s;
-	else if ($inmain) $sinmainandother .= $s;
-	else if ($inadmin) $sininstallandadmin .= $s;
-	else $sother .= $s;
+            $s.= "(".$line." - ".htmlentities($translatedvalue).") ";
+        }
+        if ($web) $s.="<br>";
+    }
+    if ($web) $s.="</td></tr>";
+    $s.="\n";
+
+    if ($duplicateinsamefile) $sduplicateinsamefile .= $s;
+    else if ($inmain) $sinmainandother .= $s;
+    else if ($inadmin) $sininstallandadmin .= $s;
+    else $sother .= $s;
 }
 
 if (! $web) print "\n***** Entries duplicated in same file\n";
@@ -254,122 +254,122 @@ 
     print "***** Strings in en_US that are never used:\n";
 
     $unused=array();
-	foreach ($langstrings_dist AS $value => $line)
-	{
-    	$qualifiedforclean=1;
-	    // Check if we must keep this key to be into file for removal
-	    if (preg_match('/^Module\d+/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^Permission\d+/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^PermissionAdvanced\d+/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^ProfId\d+/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^Delays_/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^BarcodeDesc/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^Extrafield/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^LocalTax/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^Country/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^Civility/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^Currency/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^DemandReasonTypeSRC/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^PaperFormat/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^Duration/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^AmountLT/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^TotalLT/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^Month/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^MonthShort/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^Day\d/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^Short/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^ExportDataset_/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^ImportDataset_/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^ActionAC_/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^TypeLocaltax/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^StatusProspect/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^PL_/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^TE_/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^JuridicalStatus/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^CalcMode/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^newLT/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^LT[0-9]/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^TypeContact_contrat_/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^ErrorPriceExpression/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^Language_/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^DescADHERENT_/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^SubmitTranslation/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^ModuleCompanyCode/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/InDolibarr$/', $value)) $qualifiedforclean=0;
-	    // admin.lang
-	    if (preg_match('/^DAV_ALLOW_PUBLIC_DIR/i', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^DAV_ALLOW_ECM_DIR/i', $value)) $qualifiedforclean=0;
-	    // boxes.lang
-	    if (preg_match('/^BoxTitleLast/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^BoxTitleLatest/', $value)) $qualifiedforclean=0;
-	    // install.lang
-	    if (preg_match('/^KeepDefaultValues/', $value)) $qualifiedforclean=0;
-		// mail.lang
-	    if (preg_match('/MailingModuleDesc/i', $value)) $qualifiedforclean=0;
-	    // main.lang
-	    if (preg_match('/^Duration/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^FormatDate/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^DateFormat/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^.b$/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^.*Bytes$/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^NoteSomeFeaturesAreDisabled/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^(DoTest|Under|Limits|Cards|CurrentValue|DateLimit|DateAndHour|NbOfLines|NbOfObjects|NbOfReferes|TotalTTCShort|VATs)/', $value)) $qualifiedforclean=0;
-		// modulebuilder
-		if (preg_match('/^ModuleBuilderDesc/', $value)) $qualifiedforclean=0;
-	    // orders
-	    if (preg_match('/^OrderSource/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^TypeContact_/', $value)) $qualifiedforclean=0;
+    foreach ($langstrings_dist AS $value => $line)
+    {
+        $qualifiedforclean=1;
+        // Check if we must keep this key to be into file for removal
+        if (preg_match('/^Module\d+/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^Permission\d+/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^PermissionAdvanced\d+/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^ProfId\d+/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^Delays_/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^BarcodeDesc/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^Extrafield/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^LocalTax/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^Country/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^Civility/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^Currency/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^DemandReasonTypeSRC/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^PaperFormat/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^Duration/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^AmountLT/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^TotalLT/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^Month/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^MonthShort/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^Day\d/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^Short/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^ExportDataset_/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^ImportDataset_/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^ActionAC_/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^TypeLocaltax/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^StatusProspect/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^PL_/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^TE_/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^JuridicalStatus/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^CalcMode/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^newLT/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^LT[0-9]/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^TypeContact_contrat_/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^ErrorPriceExpression/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^Language_/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^DescADHERENT_/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^SubmitTranslation/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^ModuleCompanyCode/', $value)) $qualifiedforclean=0;
+        if (preg_match('/InDolibarr$/', $value)) $qualifiedforclean=0;
+        // admin.lang
+        if (preg_match('/^DAV_ALLOW_PUBLIC_DIR/i', $value)) $qualifiedforclean=0;
+        if (preg_match('/^DAV_ALLOW_ECM_DIR/i', $value)) $qualifiedforclean=0;
+        // boxes.lang
+        if (preg_match('/^BoxTitleLast/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^BoxTitleLatest/', $value)) $qualifiedforclean=0;
+        // install.lang
+        if (preg_match('/^KeepDefaultValues/', $value)) $qualifiedforclean=0;
+        // mail.lang
+        if (preg_match('/MailingModuleDesc/i', $value)) $qualifiedforclean=0;
+        // main.lang
+        if (preg_match('/^Duration/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^FormatDate/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^DateFormat/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^.b$/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^.*Bytes$/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^NoteSomeFeaturesAreDisabled/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^(DoTest|Under|Limits|Cards|CurrentValue|DateLimit|DateAndHour|NbOfLines|NbOfObjects|NbOfReferes|TotalTTCShort|VATs)/', $value)) $qualifiedforclean=0;
+        // modulebuilder
+        if (preg_match('/^ModuleBuilderDesc/', $value)) $qualifiedforclean=0;
+        // orders
+        if (preg_match('/^OrderSource/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^TypeContact_/', $value)) $qualifiedforclean=0;
         // other.lang
-	    if (preg_match('/^Notify_/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^PredefinedMail/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^DemoCompany/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^WeightUnit/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^LengthUnit/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^SurfaceUnit/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^VolumeUnit/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^SizeUnit/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^EMailText/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/ById$/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/ByLogin$/', $value)) $qualifiedforclean=0;
-	    // printing
-	    if (preg_match('/PrintingDriverDesc$/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/PrintTestDesc$/', $value)) $qualifiedforclean=0;
-	    // products
-	    if (preg_match('/GlobalVariableUpdaterType$/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/GlobalVariableUpdaterHelp$/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/OppStatus/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/AvailabilityType/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/CardProduct/', $value)) $qualifiedforclean=0;
-
-	    if (preg_match('/sms/i', $value)) $qualifiedforclean=0;
-	    if (preg_match('/TF_/i', $value)) $qualifiedforclean=0;
-	    if (preg_match('/WithBankUsing/i', $value)) $qualifiedforclean=0;
-	    if (preg_match('/descWORKFLOW_/i', $value)) $qualifiedforclean=0;
-
-	    if (! $qualifiedforclean)
-	    {
+        if (preg_match('/^Notify_/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^PredefinedMail/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^DemoCompany/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^WeightUnit/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^LengthUnit/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^SurfaceUnit/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^VolumeUnit/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^SizeUnit/', $value)) $qualifiedforclean=0;
+        if (preg_match('/^EMailText/', $value)) $qualifiedforclean=0;
+        if (preg_match('/ById$/', $value)) $qualifiedforclean=0;
+        if (preg_match('/ByLogin$/', $value)) $qualifiedforclean=0;
+        // printing
+        if (preg_match('/PrintingDriverDesc$/', $value)) $qualifiedforclean=0;
+        if (preg_match('/PrintTestDesc$/', $value)) $qualifiedforclean=0;
+        // products
+        if (preg_match('/GlobalVariableUpdaterType$/', $value)) $qualifiedforclean=0;
+        if (preg_match('/GlobalVariableUpdaterHelp$/', $value)) $qualifiedforclean=0;
+        if (preg_match('/OppStatus/', $value)) $qualifiedforclean=0;
+        if (preg_match('/AvailabilityType/', $value)) $qualifiedforclean=0;
+        if (preg_match('/CardProduct/', $value)) $qualifiedforclean=0;
+
+        if (preg_match('/sms/i', $value)) $qualifiedforclean=0;
+        if (preg_match('/TF_/i', $value)) $qualifiedforclean=0;
+        if (preg_match('/WithBankUsing/i', $value)) $qualifiedforclean=0;
+        if (preg_match('/descWORKFLOW_/i', $value)) $qualifiedforclean=0;
+
+        if (! $qualifiedforclean)
+        {
             continue;
-	    }
-
-	    //$search = '\'trans("'.$value.'")\'';
-	    $search = '-e "\''.$value.'\'" -e \'"'.$value.'"\' -e "('.$value.')" -e "('.$value.',"';
-		$string =  'grep -R -m 1 -F --exclude=includes/* --include=*.php '.$search.' '.$htdocs.'* '.$scripts.'*';
-		//print $string."<br>\n";
-		exec($string,$output);
-		if (empty($output)) {
-   			$unused[$value] = $line;
-       		echo $line;        // $trad contains the \n
-		}
-		else
-		{
-		    unset($output);
-		    //print 'X'.$output.'Y';
-		}
-	}
-
-	if (empty($unused)) print "No string not used found.\n";
-	else
-	{
+        }
+
+        //$search = '\'trans("'.$value.'")\'';
+        $search = '-e "\''.$value.'\'" -e \'"'.$value.'"\' -e "('.$value.')" -e "('.$value.',"';
+        $string =  'grep -R -m 1 -F --exclude=includes/* --include=*.php '.$search.' '.$htdocs.'* '.$scripts.'*';
+        //print $string."<br>\n";
+        exec($string,$output);
+        if (empty($output)) {
+                $unused[$value] = $line;
+                echo $line;        // $trad contains the \n
+        }
+        else
+        {
+            unset($output);
+            //print 'X'.$output.'Y';
+        }
+    }
+
+    if (empty($unused)) print "No string not used found.\n";
+    else
+    {
         $filetosave='/tmp/'.($argv[2]?$argv[2]:"").'notused.lang';
         print "Strings in en_US that are never used are saved into file ".$filetosave.":\n";
         file_put_contents($filetosave, implode("",$unused));
@@ -377,7 +377,7 @@ 
         if (($argv[2]?$argv[2]:"")) print 'cd htdocs/langs/en_US; mv '.($argv[2]?$argv[2]:"")." ".($argv[2]?$argv[2]:"").".tmp; ";
         print "diff ".($argv[2]?$argv[2]:"").".tmp ".$filetosave." | grep \< | cut  -b 3- > ".($argv[2]?$argv[2]:"");
         if (($argv[2]?$argv[2]:"")) print "; rm ".($argv[2]?$argv[2]:"").".tmp;\n";
-	}
+    }
 }
 
 echo "\n";

Spacing +144 added lines, -144 removed lines

@@ -20,14 +20,14 @@ 
 
 $sapi_type = php_sapi_name();
 $script_file = basename(__FILE__);
-$path=dirname(__FILE__).'/';
+$path = dirname(__FILE__).'/';
 
-$web=0;
+$web = 0;
 
 // Test if batch mode
 if (substr($sapi_type, 0, 3) == 'cgi')
 {
-    $web=1;
+    $web = 1;
 }
 
 
@@ -111,9 +111,9 @@ 
 	exit;
 }
 
-$dups=array();
-$exludefiles = array('.','..','README');
-$files = array_diff($files,$exludefiles);
+$dups = array();
+$exludefiles = array('.', '..', 'README');
+$files = array_diff($files, $exludefiles);
 // To force a file: $files=array('myfile.lang');
 if (isset($argv[2]))
 {
@@ -124,18 +124,18 @@ 
 foreach ($files AS $file) {
 	$path_file = pathinfo($file);
 	// we're only interested in .lang files
-	if ($path_file['extension']=='lang') {
+	if ($path_file['extension'] == 'lang') {
 		$content = file($workdir.$file);
 		foreach ($content AS $line => $row) {
 			// don't want comment lines
-			if (substr($row,0,1) !== '#') {
+			if (substr($row, 0, 1) !== '#') {
 				// don't want lines without the separator (why should those even be here, anyway...)
-				if (strpos($row,'=')!==false) {
-					$row_array = explode('=',$row);		// $row_array[0] = key
-					$langstrings_3d[$path_file['basename']][$line+1]=$row_array[0];
-					$langstrings_3dtrans[$path_file['basename']][$line+1]=$row_array[1];
-					$langstrings_full[]=$row_array[0];
-					$langstrings_dist[$row_array[0]]=$row;
+				if (strpos($row, '=') !== false) {
+					$row_array = explode('=', $row); // $row_array[0] = key
+					$langstrings_3d[$path_file['basename']][$line + 1] = $row_array[0];
+					$langstrings_3dtrans[$path_file['basename']][$line + 1] = $row_array[1];
+					$langstrings_full[] = $row_array[0];
+					$langstrings_dist[$row_array[0]] = $row;
 				}
 			}
 		}
@@ -147,7 +147,7 @@ 
 	foreach ($file AS $linenum => $value)
 	{
 		$keys = array_keys($langstrings_full, $value);
-		if (count($keys)>1)
+		if (count($keys) > 1)
 		{
 				foreach ($keys AS $key) {
 					$dups[$value][$filename][$linenum] = trim($langstrings_3dtrans[$filename][$linenum]);
@@ -167,52 +167,52 @@ 
     echo "<tbody>\n";
 }
 
-$sduplicateinsamefile='';
-$sinmainandother='';
-$sininstallandadmin='';
-$sother='';
+$sduplicateinsamefile = '';
+$sinmainandother = '';
+$sininstallandadmin = '';
+$sother = '';
 
 $count = 0;
 foreach ($dups as $string => $pages)
 {
 	$count++;
-	$s='';
+	$s = '';
 
 	// Keyword $string
-	if ($web) $s.="<tr>";
-	if ($web) $s.="<td align=\"center\">";
-	if ($web) $s.=$count;
-	if ($web) $s.="</td>";
-	if ($web) $s.="<td>";
-	$s.=$string;
-	if ($web) $s.="</td>";
-	if ($web) $s.="<td>";
-	if (! $web) $s.= ' : ';
+	if ($web) $s .= "<tr>";
+	if ($web) $s .= "<td align=\"center\">";
+	if ($web) $s .= $count;
+	if ($web) $s .= "</td>";
+	if ($web) $s .= "<td>";
+	$s .= $string;
+	if ($web) $s .= "</td>";
+	if ($web) $s .= "<td>";
+	if (!$web) $s .= ' : ';
 
 	// Loop on each files keyword was found
-    $duplicateinsamefile=0;
-	$inmain=0;
-    $inadmin=0;
+    $duplicateinsamefile = 0;
+	$inmain = 0;
+    $inadmin = 0;
 	foreach ($pages AS $file => $lines)
 	{
-        if ($file == 'main.lang') { $inmain=1; $inadmin=0; }
-        if ($file == 'admin.lang' && ! $inmain) { $inadmin=1; }
+        if ($file == 'main.lang') { $inmain = 1; $inadmin = 0; }
+        if ($file == 'admin.lang' && !$inmain) { $inadmin = 1; }
 
-	    $s.=$file." ";
+	    $s .= $file." ";
 
 	    // Loop on each line keword was found into file.
-	    $listoffilesforthisentry=array();
+	    $listoffilesforthisentry = array();
 	    foreach ($lines as $line => $translatedvalue)
 		{
-            if (! empty($listoffilesforthisentry[$file])) $duplicateinsamefile=1;
-            $listoffilesforthisentry[$file]=1;
+            if (!empty($listoffilesforthisentry[$file])) $duplicateinsamefile = 1;
+            $listoffilesforthisentry[$file] = 1;
 
-			$s.= "(".$line." - ".htmlentities($translatedvalue).") ";
+			$s .= "(".$line." - ".htmlentities($translatedvalue).") ";
 		}
-		if ($web) $s.="<br>";
+		if ($web) $s .= "<br>";
 	}
-	if ($web) $s.="</td></tr>";
-	$s.="\n";
+	if ($web) $s .= "</td></tr>";
+	$s .= "\n";
 
 	if ($duplicateinsamefile) $sduplicateinsamefile .= $s;
 	else if ($inmain) $sinmainandother .= $s;
@@ -220,25 +220,25 @@ 
 	else $sother .= $s;
 }
 
-if (! $web) print "\n***** Entries duplicated in same file\n";
+if (!$web) print "\n***** Entries duplicated in same file\n";
 print $sduplicateinsamefile;
-if (! $web && empty($sduplicateinsamefile)) print "None\n";
-if (! $web) print "\n";
+if (!$web && empty($sduplicateinsamefile)) print "None\n";
+if (!$web) print "\n";
 
-if (! $web) print "***** Entries in main and another (keep only entry in main)\n";
+if (!$web) print "***** Entries in main and another (keep only entry in main)\n";
 print $sinmainandother;
-if (! $web && empty($sinmainandother)) print "None\n";
-if (! $web) print "\n";
+if (!$web && empty($sinmainandother)) print "None\n";
+if (!$web) print "\n";
 
-if (! $web) print "***** Entries in admin and another\n";
+if (!$web) print "***** Entries in admin and another\n";
 print $sininstallandadmin;
-if (! $web && empty($sininstallandadmin)) print "None\n";
-if (! $web) print "\n";
+if (!$web && empty($sininstallandadmin)) print "None\n";
+if (!$web) print "\n";
 
-if (! $web) print "***** Other\n";
+if (!$web) print "***** Other\n";
 print $sother;
-if (! $web && empty($sother)) print "None\n";
-if (! $web) print "\n";
+if (!$web && empty($sother)) print "None\n";
+if (!$web) print "\n";
 
 if ($web)
 {
@@ -249,116 +249,116 @@ 
 
 // STEP 2 - Search key not used
 
-if ((! empty($_REQUEST['unused']) && $_REQUEST['unused'] == 'true') || (isset($argv[1]) && $argv[1]=='unused=true'))
+if ((!empty($_REQUEST['unused']) && $_REQUEST['unused'] == 'true') || (isset($argv[1]) && $argv[1] == 'unused=true'))
 {
     print "***** Strings in en_US that are never used:\n";
 
-    $unused=array();
+    $unused = array();
 	foreach ($langstrings_dist AS $value => $line)
 	{
-    	$qualifiedforclean=1;
+    	$qualifiedforclean = 1;
 	    // Check if we must keep this key to be into file for removal
-	    if (preg_match('/^Module\d+/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^Permission\d+/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^PermissionAdvanced\d+/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^ProfId\d+/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^Delays_/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^BarcodeDesc/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^Extrafield/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^LocalTax/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^Country/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^Civility/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^Currency/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^DemandReasonTypeSRC/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^PaperFormat/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^Duration/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^AmountLT/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^TotalLT/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^Month/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^MonthShort/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^Day\d/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^Short/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^ExportDataset_/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^ImportDataset_/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^ActionAC_/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^TypeLocaltax/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^StatusProspect/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^PL_/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^TE_/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^JuridicalStatus/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^CalcMode/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^newLT/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^LT[0-9]/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^TypeContact_contrat_/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^ErrorPriceExpression/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^Language_/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^DescADHERENT_/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^SubmitTranslation/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^ModuleCompanyCode/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/InDolibarr$/', $value)) $qualifiedforclean=0;
+	    if (preg_match('/^Module\d+/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^Permission\d+/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^PermissionAdvanced\d+/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^ProfId\d+/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^Delays_/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^BarcodeDesc/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^Extrafield/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^LocalTax/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^Country/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^Civility/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^Currency/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^DemandReasonTypeSRC/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^PaperFormat/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^Duration/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^AmountLT/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^TotalLT/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^Month/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^MonthShort/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^Day\d/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^Short/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^ExportDataset_/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^ImportDataset_/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^ActionAC_/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^TypeLocaltax/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^StatusProspect/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^PL_/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^TE_/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^JuridicalStatus/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^CalcMode/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^newLT/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^LT[0-9]/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^TypeContact_contrat_/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^ErrorPriceExpression/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^Language_/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^DescADHERENT_/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^SubmitTranslation/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^ModuleCompanyCode/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/InDolibarr$/', $value)) $qualifiedforclean = 0;
 	    // admin.lang
-	    if (preg_match('/^DAV_ALLOW_PUBLIC_DIR/i', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^DAV_ALLOW_ECM_DIR/i', $value)) $qualifiedforclean=0;
+	    if (preg_match('/^DAV_ALLOW_PUBLIC_DIR/i', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^DAV_ALLOW_ECM_DIR/i', $value)) $qualifiedforclean = 0;
 	    // boxes.lang
-	    if (preg_match('/^BoxTitleLast/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^BoxTitleLatest/', $value)) $qualifiedforclean=0;
+	    if (preg_match('/^BoxTitleLast/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^BoxTitleLatest/', $value)) $qualifiedforclean = 0;
 	    // install.lang
-	    if (preg_match('/^KeepDefaultValues/', $value)) $qualifiedforclean=0;
+	    if (preg_match('/^KeepDefaultValues/', $value)) $qualifiedforclean = 0;
 		// mail.lang
-	    if (preg_match('/MailingModuleDesc/i', $value)) $qualifiedforclean=0;
+	    if (preg_match('/MailingModuleDesc/i', $value)) $qualifiedforclean = 0;
 	    // main.lang
-	    if (preg_match('/^Duration/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^FormatDate/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^DateFormat/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^.b$/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^.*Bytes$/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^NoteSomeFeaturesAreDisabled/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^(DoTest|Under|Limits|Cards|CurrentValue|DateLimit|DateAndHour|NbOfLines|NbOfObjects|NbOfReferes|TotalTTCShort|VATs)/', $value)) $qualifiedforclean=0;
+	    if (preg_match('/^Duration/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^FormatDate/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^DateFormat/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^.b$/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^.*Bytes$/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^NoteSomeFeaturesAreDisabled/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^(DoTest|Under|Limits|Cards|CurrentValue|DateLimit|DateAndHour|NbOfLines|NbOfObjects|NbOfReferes|TotalTTCShort|VATs)/', $value)) $qualifiedforclean = 0;
 		// modulebuilder
-		if (preg_match('/^ModuleBuilderDesc/', $value)) $qualifiedforclean=0;
+		if (preg_match('/^ModuleBuilderDesc/', $value)) $qualifiedforclean = 0;
 	    // orders
-	    if (preg_match('/^OrderSource/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^TypeContact_/', $value)) $qualifiedforclean=0;
+	    if (preg_match('/^OrderSource/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^TypeContact_/', $value)) $qualifiedforclean = 0;
         // other.lang
-	    if (preg_match('/^Notify_/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^PredefinedMail/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^DemoCompany/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^WeightUnit/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^LengthUnit/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^SurfaceUnit/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^VolumeUnit/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^SizeUnit/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^EMailText/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/ById$/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/ByLogin$/', $value)) $qualifiedforclean=0;
+	    if (preg_match('/^Notify_/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^PredefinedMail/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^DemoCompany/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^WeightUnit/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^LengthUnit/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^SurfaceUnit/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^VolumeUnit/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^SizeUnit/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/^EMailText/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/ById$/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/ByLogin$/', $value)) $qualifiedforclean = 0;
 	    // printing
-	    if (preg_match('/PrintingDriverDesc$/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/PrintTestDesc$/', $value)) $qualifiedforclean=0;
+	    if (preg_match('/PrintingDriverDesc$/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/PrintTestDesc$/', $value)) $qualifiedforclean = 0;
 	    // products
-	    if (preg_match('/GlobalVariableUpdaterType$/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/GlobalVariableUpdaterHelp$/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/OppStatus/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/AvailabilityType/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/CardProduct/', $value)) $qualifiedforclean=0;
-
-	    if (preg_match('/sms/i', $value)) $qualifiedforclean=0;
-	    if (preg_match('/TF_/i', $value)) $qualifiedforclean=0;
-	    if (preg_match('/WithBankUsing/i', $value)) $qualifiedforclean=0;
-	    if (preg_match('/descWORKFLOW_/i', $value)) $qualifiedforclean=0;
-
-	    if (! $qualifiedforclean)
+	    if (preg_match('/GlobalVariableUpdaterType$/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/GlobalVariableUpdaterHelp$/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/OppStatus/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/AvailabilityType/', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/CardProduct/', $value)) $qualifiedforclean = 0;
+
+	    if (preg_match('/sms/i', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/TF_/i', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/WithBankUsing/i', $value)) $qualifiedforclean = 0;
+	    if (preg_match('/descWORKFLOW_/i', $value)) $qualifiedforclean = 0;
+
+	    if (!$qualifiedforclean)
 	    {
             continue;
 	    }
 
 	    //$search = '\'trans("'.$value.'")\'';
 	    $search = '-e "\''.$value.'\'" -e \'"'.$value.'"\' -e "('.$value.')" -e "('.$value.',"';
-		$string =  'grep -R -m 1 -F --exclude=includes/* --include=*.php '.$search.' '.$htdocs.'* '.$scripts.'*';
+		$string = 'grep -R -m 1 -F --exclude=includes/* --include=*.php '.$search.' '.$htdocs.'* '.$scripts.'*';
 		//print $string."<br>\n";
-		exec($string,$output);
+		exec($string, $output);
 		if (empty($output)) {
    			$unused[$value] = $line;
-       		echo $line;        // $trad contains the \n
+       		echo $line; // $trad contains the \n
 		}
 		else
 		{
@@ -370,13 +370,13 @@ 
 	if (empty($unused)) print "No string not used found.\n";
 	else
 	{
-        $filetosave='/tmp/'.($argv[2]?$argv[2]:"").'notused.lang';
+        $filetosave = '/tmp/'.($argv[2] ? $argv[2] : "").'notused.lang';
         print "Strings in en_US that are never used are saved into file ".$filetosave.":\n";
-        file_put_contents($filetosave, implode("",$unused));
+        file_put_contents($filetosave, implode("", $unused));
         print "To remove from original file, run command :\n";
-        if (($argv[2]?$argv[2]:"")) print 'cd htdocs/langs/en_US; mv '.($argv[2]?$argv[2]:"")." ".($argv[2]?$argv[2]:"").".tmp; ";
-        print "diff ".($argv[2]?$argv[2]:"").".tmp ".$filetosave." | grep \< | cut  -b 3- > ".($argv[2]?$argv[2]:"");
-        if (($argv[2]?$argv[2]:"")) print "; rm ".($argv[2]?$argv[2]:"").".tmp;\n";
+        if (($argv[2] ? $argv[2] : "")) print 'cd htdocs/langs/en_US; mv '.($argv[2] ? $argv[2] : "")." ".($argv[2] ? $argv[2] : "").".tmp; ";
+        print "diff ".($argv[2] ? $argv[2] : "").".tmp ".$filetosave." | grep \< | cut  -b 3- > ".($argv[2] ? $argv[2] : "");
+        if (($argv[2] ? $argv[2] : "")) print "; rm ".($argv[2] ? $argv[2] : "").".tmp;\n";
 	}
 }
 

Braces +329 added lines, -114 removed lines

@@ -87,9 +87,13 @@ 
 }
 
 echo "If you call this with argument \"unused=true\" it searches for the translation strings that exist in en_US but are never used.\n";
-if ($web) print "<br>";
+if ($web) {
+    print "<br>";
+}
 echo "IMPORTANT: that can take quite a lot of time (up to 10 minutes), you need to tune the max_execution_time on your php.ini accordingly.\n";
-if ($web) print "<br>";
+if ($web) {
+    print "<br>";
+}
 
 
 
@@ -156,9 +160,13 @@ 
 	}
 }
 
-if ($web) print "<h2>";
+if ($web) {
+    print "<h2>";
+}
 print "Duplicate strings in lang files in $workdir - ".count($dups)." found\n";
-if ($web) print "</h2>";
+if ($web) {
+    print "</h2>";
+}
 
 if ($web)
 {
@@ -179,15 +187,31 @@ 
 	$s='';
 
 	// Keyword $string
-	if ($web) $s.="<tr>";
-	if ($web) $s.="<td align=\"center\">";
-	if ($web) $s.=$count;
-	if ($web) $s.="</td>";
-	if ($web) $s.="<td>";
+	if ($web) {
+	    $s.="<tr>";
+	}
+	if ($web) {
+	    $s.="<td align=\"center\">";
+	}
+	if ($web) {
+	    $s.=$count;
+	}
+	if ($web) {
+	    $s.="</td>";
+	}
+	if ($web) {
+	    $s.="<td>";
+	}
 	$s.=$string;
-	if ($web) $s.="</td>";
-	if ($web) $s.="<td>";
-	if (! $web) $s.= ' : ';
+	if ($web) {
+	    $s.="</td>";
+	}
+	if ($web) {
+	    $s.="<td>";
+	}
+	if (! $web) {
+	    $s.= ' : ';
+	}
 
 	// Loop on each files keyword was found
     $duplicateinsamefile=0;
@@ -204,41 +228,76 @@ 
 	    $listoffilesforthisentry=array();
 	    foreach ($lines as $line => $translatedvalue)
 		{
-            if (! empty($listoffilesforthisentry[$file])) $duplicateinsamefile=1;
+            if (! empty($listoffilesforthisentry[$file])) {
+                $duplicateinsamefile=1;
+            }
             $listoffilesforthisentry[$file]=1;
 
 			$s.= "(".$line." - ".htmlentities($translatedvalue).") ";
 		}
-		if ($web) $s.="<br>";
+		if ($web) {
+		    $s.="<br>";
+		}
+	}
+	if ($web) {
+	    $s.="</td></tr>";
 	}
-	if ($web) $s.="</td></tr>";
 	$s.="\n";
 
-	if ($duplicateinsamefile) $sduplicateinsamefile .= $s;
-	else if ($inmain) $sinmainandother .= $s;
-	else if ($inadmin) $sininstallandadmin .= $s;
-	else $sother .= $s;
-}
+	if ($duplicateinsamefile) {
+	    $sduplicateinsamefile .= $s;
+	} else if ($inmain) {
+	    $sinmainandother .= $s;
+	} else if ($inadmin) {
+	    $sininstallandadmin .= $s;
+	} else {
+	    $sother .= $s;
+	}
+	}
 
-if (! $web) print "\n***** Entries duplicated in same file\n";
+if (! $web) {
+    print "\n***** Entries duplicated in same file\n";
+}
 print $sduplicateinsamefile;
-if (! $web && empty($sduplicateinsamefile)) print "None\n";
-if (! $web) print "\n";
+if (! $web && empty($sduplicateinsamefile)) {
+    print "None\n";
+}
+if (! $web) {
+    print "\n";
+}
 
-if (! $web) print "***** Entries in main and another (keep only entry in main)\n";
+if (! $web) {
+    print "***** Entries in main and another (keep only entry in main)\n";
+}
 print $sinmainandother;
-if (! $web && empty($sinmainandother)) print "None\n";
-if (! $web) print "\n";
+if (! $web && empty($sinmainandother)) {
+    print "None\n";
+}
+if (! $web) {
+    print "\n";
+}
 
-if (! $web) print "***** Entries in admin and another\n";
+if (! $web) {
+    print "***** Entries in admin and another\n";
+}
 print $sininstallandadmin;
-if (! $web && empty($sininstallandadmin)) print "None\n";
-if (! $web) print "\n";
+if (! $web && empty($sininstallandadmin)) {
+    print "None\n";
+}
+if (! $web) {
+    print "\n";
+}
 
-if (! $web) print "***** Other\n";
+if (! $web) {
+    print "***** Other\n";
+}
 print $sother;
-if (! $web && empty($sother)) print "None\n";
-if (! $web) print "\n";
+if (! $web && empty($sother)) {
+    print "None\n";
+}
+if (! $web) {
+    print "\n";
+}
 
 if ($web)
 {
@@ -258,93 +317,245 @@ 
 	{
     	$qualifiedforclean=1;
 	    // Check if we must keep this key to be into file for removal
-	    if (preg_match('/^Module\d+/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^Permission\d+/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^PermissionAdvanced\d+/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^ProfId\d+/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^Delays_/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^BarcodeDesc/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^Extrafield/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^LocalTax/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^Country/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^Civility/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^Currency/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^DemandReasonTypeSRC/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^PaperFormat/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^Duration/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^AmountLT/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^TotalLT/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^Month/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^MonthShort/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^Day\d/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^Short/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^ExportDataset_/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^ImportDataset_/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^ActionAC_/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^TypeLocaltax/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^StatusProspect/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^PL_/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^TE_/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^JuridicalStatus/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^CalcMode/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^newLT/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^LT[0-9]/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^TypeContact_contrat_/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^ErrorPriceExpression/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^Language_/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^DescADHERENT_/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^SubmitTranslation/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^ModuleCompanyCode/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/InDolibarr$/', $value)) $qualifiedforclean=0;
+	    if (preg_match('/^Module\d+/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^Permission\d+/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^PermissionAdvanced\d+/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^ProfId\d+/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^Delays_/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^BarcodeDesc/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^Extrafield/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^LocalTax/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^Country/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^Civility/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^Currency/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^DemandReasonTypeSRC/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^PaperFormat/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^Duration/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^AmountLT/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^TotalLT/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^Month/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^MonthShort/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^Day\d/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^Short/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^ExportDataset_/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^ImportDataset_/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^ActionAC_/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^TypeLocaltax/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^StatusProspect/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^PL_/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^TE_/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^JuridicalStatus/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^CalcMode/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^newLT/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^LT[0-9]/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^TypeContact_contrat_/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^ErrorPriceExpression/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^Language_/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^DescADHERENT_/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^SubmitTranslation/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^ModuleCompanyCode/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/InDolibarr$/', $value)) {
+	        $qualifiedforclean=0;
+	    }
 	    // admin.lang
-	    if (preg_match('/^DAV_ALLOW_PUBLIC_DIR/i', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^DAV_ALLOW_ECM_DIR/i', $value)) $qualifiedforclean=0;
+	    if (preg_match('/^DAV_ALLOW_PUBLIC_DIR/i', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^DAV_ALLOW_ECM_DIR/i', $value)) {
+	        $qualifiedforclean=0;
+	    }
 	    // boxes.lang
-	    if (preg_match('/^BoxTitleLast/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^BoxTitleLatest/', $value)) $qualifiedforclean=0;
+	    if (preg_match('/^BoxTitleLast/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^BoxTitleLatest/', $value)) {
+	        $qualifiedforclean=0;
+	    }
 	    // install.lang
-	    if (preg_match('/^KeepDefaultValues/', $value)) $qualifiedforclean=0;
+	    if (preg_match('/^KeepDefaultValues/', $value)) {
+	        $qualifiedforclean=0;
+	    }
 		// mail.lang
-	    if (preg_match('/MailingModuleDesc/i', $value)) $qualifiedforclean=0;
+	    if (preg_match('/MailingModuleDesc/i', $value)) {
+	        $qualifiedforclean=0;
+	    }
 	    // main.lang
-	    if (preg_match('/^Duration/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^FormatDate/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^DateFormat/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^.b$/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^.*Bytes$/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^NoteSomeFeaturesAreDisabled/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^(DoTest|Under|Limits|Cards|CurrentValue|DateLimit|DateAndHour|NbOfLines|NbOfObjects|NbOfReferes|TotalTTCShort|VATs)/', $value)) $qualifiedforclean=0;
+	    if (preg_match('/^Duration/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^FormatDate/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^DateFormat/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^.b$/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^.*Bytes$/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^NoteSomeFeaturesAreDisabled/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^(DoTest|Under|Limits|Cards|CurrentValue|DateLimit|DateAndHour|NbOfLines|NbOfObjects|NbOfReferes|TotalTTCShort|VATs)/', $value)) {
+	        $qualifiedforclean=0;
+	    }
 		// modulebuilder
-		if (preg_match('/^ModuleBuilderDesc/', $value)) $qualifiedforclean=0;
+		if (preg_match('/^ModuleBuilderDesc/', $value)) {
+		    $qualifiedforclean=0;
+		}
 	    // orders
-	    if (preg_match('/^OrderSource/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^TypeContact_/', $value)) $qualifiedforclean=0;
+	    if (preg_match('/^OrderSource/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^TypeContact_/', $value)) {
+	        $qualifiedforclean=0;
+	    }
         // other.lang
-	    if (preg_match('/^Notify_/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^PredefinedMail/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^DemoCompany/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^WeightUnit/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^LengthUnit/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^SurfaceUnit/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^VolumeUnit/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^SizeUnit/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/^EMailText/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/ById$/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/ByLogin$/', $value)) $qualifiedforclean=0;
+	    if (preg_match('/^Notify_/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^PredefinedMail/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^DemoCompany/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^WeightUnit/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^LengthUnit/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^SurfaceUnit/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^VolumeUnit/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^SizeUnit/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/^EMailText/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/ById$/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/ByLogin$/', $value)) {
+	        $qualifiedforclean=0;
+	    }
 	    // printing
-	    if (preg_match('/PrintingDriverDesc$/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/PrintTestDesc$/', $value)) $qualifiedforclean=0;
+	    if (preg_match('/PrintingDriverDesc$/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/PrintTestDesc$/', $value)) {
+	        $qualifiedforclean=0;
+	    }
 	    // products
-	    if (preg_match('/GlobalVariableUpdaterType$/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/GlobalVariableUpdaterHelp$/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/OppStatus/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/AvailabilityType/', $value)) $qualifiedforclean=0;
-	    if (preg_match('/CardProduct/', $value)) $qualifiedforclean=0;
+	    if (preg_match('/GlobalVariableUpdaterType$/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/GlobalVariableUpdaterHelp$/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/OppStatus/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/AvailabilityType/', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/CardProduct/', $value)) {
+	        $qualifiedforclean=0;
+	    }
 
-	    if (preg_match('/sms/i', $value)) $qualifiedforclean=0;
-	    if (preg_match('/TF_/i', $value)) $qualifiedforclean=0;
-	    if (preg_match('/WithBankUsing/i', $value)) $qualifiedforclean=0;
-	    if (preg_match('/descWORKFLOW_/i', $value)) $qualifiedforclean=0;
+	    if (preg_match('/sms/i', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/TF_/i', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/WithBankUsing/i', $value)) {
+	        $qualifiedforclean=0;
+	    }
+	    if (preg_match('/descWORKFLOW_/i', $value)) {
+	        $qualifiedforclean=0;
+	    }
 
 	    if (! $qualifiedforclean)
 	    {
@@ -359,24 +570,28 @@ 
 		if (empty($output)) {
    			$unused[$value] = $line;
        		echo $line;        // $trad contains the \n
-		}
-		else
+		} else
 		{
 		    unset($output);
 		    //print 'X'.$output.'Y';
 		}
 	}
 
-	if (empty($unused)) print "No string not used found.\n";
-	else
+	if (empty($unused)) {
+	    print "No string not used found.\n";
+	} else
 	{
         $filetosave='/tmp/'.($argv[2]?$argv[2]:"").'notused.lang';
         print "Strings in en_US that are never used are saved into file ".$filetosave.":\n";
         file_put_contents($filetosave, implode("",$unused));
         print "To remove from original file, run command :\n";
-        if (($argv[2]?$argv[2]:"")) print 'cd htdocs/langs/en_US; mv '.($argv[2]?$argv[2]:"")." ".($argv[2]?$argv[2]:"").".tmp; ";
+        if (($argv[2]?$argv[2]:"")) {
+            print 'cd htdocs/langs/en_US; mv '.($argv[2]?$argv[2]:"")." ".($argv[2]?$argv[2]:"").".tmp; ";
+        }
         print "diff ".($argv[2]?$argv[2]:"").".tmp ".$filetosave." | grep \< | cut  -b 3- > ".($argv[2]?$argv[2]:"");
-        if (($argv[2]?$argv[2]:"")) print "; rm ".($argv[2]?$argv[2]:"").".tmp;\n";
+        if (($argv[2]?$argv[2]:"")) {
+            print "; rm ".($argv[2]?$argv[2]:"").".tmp;\n";
+        }
 	}
 }
 

dolibarr/dev/translation/autotranslator.class.php

Indentation +299 added lines, -299 removed lines

@@ -27,17 +27,17 @@ 
  */
 class autoTranslator
 {
-	private $_translatedFiles = array();
-	private $_destlang = '';
-	private $_refLang = '';
-	private $_langDir = '';
-	private $_limittofile = '';
-	private $_time;
-	private $_time_end;
-	private $_outputpagecode = 'UTF-8';
-	private $_apikey;
-	//private $_outputpagecode = 'ISO-8859-1';
-	const DIR_SEPARATOR = '/';
+    private $_translatedFiles = array();
+    private $_destlang = '';
+    private $_refLang = '';
+    private $_langDir = '';
+    private $_limittofile = '';
+    private $_time;
+    private $_time_end;
+    private $_outputpagecode = 'UTF-8';
+    private $_apikey;
+    //private $_outputpagecode = 'ISO-8859-1';
+    const DIR_SEPARATOR = '/';
 
 
     /**
@@ -53,296 +53,296 @@ 
     function __construct($_destlang,$_refLang,$_langDir,$_limittofile,$_apikey)
     {
 
-		// Set enviorment variables
-		$this->_destlang = $_destlang;
-		$this->_refLang = $_refLang;
-		$this->_langDir = $_langDir.self::DIR_SEPARATOR;
-		$this->_time = date('Y-m-d H:i:s');
-		$this->_limittofile = $_limittofile;
+        // Set enviorment variables
+        $this->_destlang = $_destlang;
+        $this->_refLang = $_refLang;
+        $this->_langDir = $_langDir.self::DIR_SEPARATOR;
+        $this->_time = date('Y-m-d H:i:s');
+        $this->_limittofile = $_limittofile;
         $this->_apikey = $_apikey;
 
-		// Translate
-		//ini_set('default_charset','UTF-8');
-		ini_set('default_charset',$this->_outputpagecode);
-		$this->parseRefLangTranslationFiles();
-	}
-
-	/**
-	 * 	Parse file
-	 *
-	 * 	@return	void
-	 */
-	private function parseRefLangTranslationFiles()
-	{
-
-		$files = $this->getTranslationFilesArray($this->_refLang);
-		$counter = 1;
-		foreach($files as $file)
-		{
-			if ($this->_limittofile && $this->_limittofile != $file) continue;
-			$counter++;
-			$fileContent = null;
-			$refPath = $this->_langDir.$this->_refLang.self::DIR_SEPARATOR.$file;
-			$fileContent = file($refPath,FILE_IGNORE_NEW_LINES|FILE_SKIP_EMPTY_LINES);
-			print "Processing file " . $file . ", with ".count($fileContent)." lines<br>\n";
-
-			// Define target dirs
-			$targetlangs=array($this->_destlang);
-			if ($this->_destlang == 'all')
-			{
-				$targetlangs=array();
-
-				// If we must process all languages
-				$arraytmp=dol_dir_list($this->_langDir,'directories',0);
-				foreach($arraytmp as $dirtmp)
-				{
-					if ($dirtmp['name'] === $this->_refLang) continue;	// We discard source language
-					$tmppart=explode('_',$dirtmp['name']);
-					if (preg_match('/^en/i',$dirtmp['name']))  continue;	// We discard en_* languages
-					if (preg_match('/^fr/i',$dirtmp['name']))  continue;	// We discard fr_* languages
-					if (preg_match('/^es/i',$dirtmp['name']))  continue;	// We discard es_* languages
-					if (preg_match('/ca_ES/i',$dirtmp['name']))  continue;	// We discard es_CA language
-					if (preg_match('/pt_BR/i',$dirtmp['name']))  continue;	// We discard pt_BR language
+        // Translate
+        //ini_set('default_charset','UTF-8');
+        ini_set('default_charset',$this->_outputpagecode);
+        $this->parseRefLangTranslationFiles();
+    }
+
+    /**
+     * 	Parse file
+     *
+     * 	@return	void
+     */
+    private function parseRefLangTranslationFiles()
+    {
+
+        $files = $this->getTranslationFilesArray($this->_refLang);
+        $counter = 1;
+        foreach($files as $file)
+        {
+            if ($this->_limittofile && $this->_limittofile != $file) continue;
+            $counter++;
+            $fileContent = null;
+            $refPath = $this->_langDir.$this->_refLang.self::DIR_SEPARATOR.$file;
+            $fileContent = file($refPath,FILE_IGNORE_NEW_LINES|FILE_SKIP_EMPTY_LINES);
+            print "Processing file " . $file . ", with ".count($fileContent)." lines<br>\n";
+
+            // Define target dirs
+            $targetlangs=array($this->_destlang);
+            if ($this->_destlang == 'all')
+            {
+                $targetlangs=array();
+
+                // If we must process all languages
+                $arraytmp=dol_dir_list($this->_langDir,'directories',0);
+                foreach($arraytmp as $dirtmp)
+                {
+                    if ($dirtmp['name'] === $this->_refLang) continue;	// We discard source language
+                    $tmppart=explode('_',$dirtmp['name']);
+                    if (preg_match('/^en/i',$dirtmp['name']))  continue;	// We discard en_* languages
+                    if (preg_match('/^fr/i',$dirtmp['name']))  continue;	// We discard fr_* languages
+                    if (preg_match('/^es/i',$dirtmp['name']))  continue;	// We discard es_* languages
+                    if (preg_match('/ca_ES/i',$dirtmp['name']))  continue;	// We discard es_CA language
+                    if (preg_match('/pt_BR/i',$dirtmp['name']))  continue;	// We discard pt_BR language
                     if (preg_match('/nl_BE/i',$dirtmp['name']))  continue;  // We discard nl_BE language
-					if (preg_match('/^\./i',$dirtmp['name']))  continue;	// We discard files .*
-					if (preg_match('/^CVS/i',$dirtmp['name']))  continue;	// We discard CVS
-					$targetlangs[]=$dirtmp['name'];
-				}
-				//var_dump($targetlangs);
-			}
-
-			// Process translation of source file for each target languages
-			foreach($targetlangs as $my_destlang)
-			{
-				$this->_translatedFiles = array();
-
-				$destPath = $this->_langDir.$my_destlang.self::DIR_SEPARATOR.$file;
-				// Check destination file presence
-				if (! file_exists($destPath))
-				{
-					// No file present, we generate file
-					echo "File not found: " . $destPath . ". We generate it.<br>\n";
-					$this->createTranslationFile($destPath,$my_destlang);
-				}
-				else
-				{
-					echo "Updating file: " . $destPath . "<br>\n";
-				}
-
-				// Translate lines
-				$fileContentDest = file($destPath,FILE_IGNORE_NEW_LINES|FILE_SKIP_EMPTY_LINES);
-				$newlines=0;
-				foreach($fileContent as $line){
-					$key = $this->getLineKey($line);
-					$value = $this->getLineValue($line);
-					if ($key && $value)
-					{
-						$newlines+=$this->translateFileLine($fileContentDest,$file,$key,$value,$my_destlang);
-					}
-				}
-
-				$this->updateTranslationFile($destPath,$file,$my_destlang);
-				echo "New translated lines: " . $newlines . "<br>\n";
-				//if ($counter ==3) die('fim');
-			}
-		}
-	}
-
-	/**
-	 * Update file with new translations
-	 *
-	 * @param 	string	$destPath		Target path
-	 * @param 	string	$file			File
-	 * @param 	string	$my_destlang		Target language code
-	 * @return	void
-	 */
-	private function updateTranslationFile($destPath,$file,$my_destlang)
-	{
-		$this->_time_end = date('Y-m-d H:i:s');
-
-		if (isset($this->_translatedFiles[$file]) && count($this->_translatedFiles[$file])>0)
-		{
-			$fp = fopen($destPath, 'a');
-			fwrite($fp, "\n");
-			fwrite($fp, "\n");
-			fwrite($fp, "// START - Lines generated via autotranslator.php tool (".$this->_time.").\n");
-			fwrite($fp, "// Reference language: ".$this->_refLang." -> ".$my_destlang."\n");
-			foreach( $this->_translatedFiles[$file] as $line) {
-				fwrite($fp, $line . "\n");
-			}
-			fwrite($fp, "// STOP - Lines generated via autotranslator.php tool (".$this->_time_end.").\n");
-			fclose($fp);
-		}
-		return;
-	}
-
-	/**
-	 * Create a new translation file
-	 *
-	 * @param 	string	$path			Path
-	 * @param 	string	$my_destlang		Target language code
-	 * @return	void
-	 */
-	private function createTranslationFile($path,$my_destlang)
-	{
-		$fp = fopen($path, 'w+');
-		fwrite($fp, "/*\n");
-		fwrite($fp, " * Language code: {$my_destlang}\n");
-		fwrite($fp, " * Automatic generated via autotranslator.php tool\n");
-		fwrite($fp, " * Generation date " . $this->_time. "\n");
-		fwrite($fp, " */\n");
-		fclose($fp);
-		return;
-	}
-
-	/**
-	 * Put in array _translatedFiles[$file], line of a new tranlated pair
-	 *
-	 * @param 	string	$content		Existing content of dest file
-	 * @param 	string	$file			Target file name translated (xxxx.lang)
-	 * @param 	string	$key			Key to translate
-	 * @param 	string	$value			Existing value in source file
-	 * @param	string	$my_destlang	Language code (ie: fr_FR)
-	 * @return	int						0=Nothing translated, 1=Record translated
-	 */
-	private function translateFileLine($content,$file,$key,$value,$my_destlang)
-	{
-
-		//print "key    =".$key."\n";
-		foreach( $content as $line ) {
-			$destKey = $this->getLineKey($line);
-			$destValue = $this->getLineValue($line);
-			// If translated return
-			//print "destKey=".$destKey."\n";
-			if ( trim($destKey) == trim($key) )
-			{	// Found already existing translation (key already exits in dest file)
-				return 0;
-			}
-		}
-
-		if ($key == 'CHARSET') $val=$this->_outputpagecode;
-		else if (preg_match('/^Format/',$key)) $val=$value;
-		else if ($value=='-') $val=$value;
-		else
-		{
-			// If not translated then translate
-			if ($this->_outputpagecode == 'UTF-8') $val=$this->translateTexts(array($value),substr($this->_refLang,0,2),substr($my_destlang,0,2));
-			else $val=utf8_decode($this->translateTexts(array($value),substr($this->_refLang,0,2),substr($my_destlang,0,2)));
-		}
-
-		$val=trim($val);
-
-		if (empty($val)) return 0;
-
-		$this->_translatedFiles[$file][] = $key . '=' . $val ;
-		return 1;
-	}
-
-	/**
-	 * getLineKey
-	 *
-	 * @param 	string 	$line	Line found into file
-	 * @return	string	Key
-	 */
-	private function getLineKey($line)
-	{
-		$arraykey = explode('=',$line,2);
-		return trim($arraykey[0]);
-	}
-
-	/**
-	 * getLineValue
-	 *
-	 * @param 	string 	$line	Line found into file
-	 * @return	string	Value
-	 */
-	private function getLineValue($line)
-	{
-		$arraykey = explode('=',$line,2);
-		return trim(isset($arraykey[1])?$arraykey[1]:'');
-	}
-
-	/**
-	 * getTranslationFilesArray
-	 *
-	 * @param 	string 	$lang	Language code
-	 * @return	array	Array
-	 */
-	private function getTranslationFilesArray($lang)
-	{
-		$dir = new DirectoryIterator($this->_langDir.$lang);
-		while($dir->valid()) {
-			if(!$dir->isDot() && $dir->isFile() && ! preg_match('/^\./',$dir->getFilename())) {
-				$files[] =  $dir->getFilename();
-			}
-			$dir->next();
-		}
-		return $files;
-	}
-
-	/**
-	 * Return translation of a value
-	 *
-	 * @param 	array	$src_texts		Array with one value
-	 * @param 	string	$src_lang		Language code source (us, fr, it, ...)
-	 * @param 	string	$dest_lang		Language code target (es, de, ...)
-	 * @return 	string					Value translated
-	 */
-	private function translateTexts($src_texts, $src_lang, $dest_lang)
-	{
-		// We want to be sure that src_lang and dest_lang are using 2 chars only
-		$tmp=explode('_',$src_lang);
-		if (! empty($tmp[1]) && $tmp[0] == $tmp[1]) $src_lang=$tmp[0];
-		$tmp=explode('_',$dest_lang);
-		if (! empty($tmp[1]) && $tmp[0] == $tmp[1]) $dest_lang=$tmp[0];
-
-		//setting language pair
-		$lang_pair = $src_lang.'|'.$dest_lang;
-
-		$src_text_to_translate=preg_replace('/%s/','SSSSS',implode('',$src_texts));
-		$src_text_to_translate=preg_replace('/'.preg_quote('\n\n').'/',' NNNNN ',$src_text_to_translate);
-
-		// Define GET URL v1
-		//$url = "http://ajax.googleapis.com/ajax/services/language/translate?v=1.0&q=".urlencode($src_text_to_translate)."&langpair=".urlencode($lang_pair);
-		// Example: http://ajax.googleapis.com/ajax/services/language/translate?v=1.0&q=Setup%20area&langpair=en_US|fr_FR
+                    if (preg_match('/^\./i',$dirtmp['name']))  continue;	// We discard files .*
+                    if (preg_match('/^CVS/i',$dirtmp['name']))  continue;	// We discard CVS
+                    $targetlangs[]=$dirtmp['name'];
+                }
+                //var_dump($targetlangs);
+            }
+
+            // Process translation of source file for each target languages
+            foreach($targetlangs as $my_destlang)
+            {
+                $this->_translatedFiles = array();
+
+                $destPath = $this->_langDir.$my_destlang.self::DIR_SEPARATOR.$file;
+                // Check destination file presence
+                if (! file_exists($destPath))
+                {
+                    // No file present, we generate file
+                    echo "File not found: " . $destPath . ". We generate it.<br>\n";
+                    $this->createTranslationFile($destPath,$my_destlang);
+                }
+                else
+                {
+                    echo "Updating file: " . $destPath . "<br>\n";
+                }
+
+                // Translate lines
+                $fileContentDest = file($destPath,FILE_IGNORE_NEW_LINES|FILE_SKIP_EMPTY_LINES);
+                $newlines=0;
+                foreach($fileContent as $line){
+                    $key = $this->getLineKey($line);
+                    $value = $this->getLineValue($line);
+                    if ($key && $value)
+                    {
+                        $newlines+=$this->translateFileLine($fileContentDest,$file,$key,$value,$my_destlang);
+                    }
+                }
+
+                $this->updateTranslationFile($destPath,$file,$my_destlang);
+                echo "New translated lines: " . $newlines . "<br>\n";
+                //if ($counter ==3) die('fim');
+            }
+        }
+    }
+
+    /**
+     * Update file with new translations
+     *
+     * @param 	string	$destPath		Target path
+     * @param 	string	$file			File
+     * @param 	string	$my_destlang		Target language code
+     * @return	void
+     */
+    private function updateTranslationFile($destPath,$file,$my_destlang)
+    {
+        $this->_time_end = date('Y-m-d H:i:s');
+
+        if (isset($this->_translatedFiles[$file]) && count($this->_translatedFiles[$file])>0)
+        {
+            $fp = fopen($destPath, 'a');
+            fwrite($fp, "\n");
+            fwrite($fp, "\n");
+            fwrite($fp, "// START - Lines generated via autotranslator.php tool (".$this->_time.").\n");
+            fwrite($fp, "// Reference language: ".$this->_refLang." -> ".$my_destlang."\n");
+            foreach( $this->_translatedFiles[$file] as $line) {
+                fwrite($fp, $line . "\n");
+            }
+            fwrite($fp, "// STOP - Lines generated via autotranslator.php tool (".$this->_time_end.").\n");
+            fclose($fp);
+        }
+        return;
+    }
+
+    /**
+     * Create a new translation file
+     *
+     * @param 	string	$path			Path
+     * @param 	string	$my_destlang		Target language code
+     * @return	void
+     */
+    private function createTranslationFile($path,$my_destlang)
+    {
+        $fp = fopen($path, 'w+');
+        fwrite($fp, "/*\n");
+        fwrite($fp, " * Language code: {$my_destlang}\n");
+        fwrite($fp, " * Automatic generated via autotranslator.php tool\n");
+        fwrite($fp, " * Generation date " . $this->_time. "\n");
+        fwrite($fp, " */\n");
+        fclose($fp);
+        return;
+    }
+
+    /**
+     * Put in array _translatedFiles[$file], line of a new tranlated pair
+     *
+     * @param 	string	$content		Existing content of dest file
+     * @param 	string	$file			Target file name translated (xxxx.lang)
+     * @param 	string	$key			Key to translate
+     * @param 	string	$value			Existing value in source file
+     * @param	string	$my_destlang	Language code (ie: fr_FR)
+     * @return	int						0=Nothing translated, 1=Record translated
+     */
+    private function translateFileLine($content,$file,$key,$value,$my_destlang)
+    {
+
+        //print "key    =".$key."\n";
+        foreach( $content as $line ) {
+            $destKey = $this->getLineKey($line);
+            $destValue = $this->getLineValue($line);
+            // If translated return
+            //print "destKey=".$destKey."\n";
+            if ( trim($destKey) == trim($key) )
+            {	// Found already existing translation (key already exits in dest file)
+                return 0;
+            }
+        }
+
+        if ($key == 'CHARSET') $val=$this->_outputpagecode;
+        else if (preg_match('/^Format/',$key)) $val=$value;
+        else if ($value=='-') $val=$value;
+        else
+        {
+            // If not translated then translate
+            if ($this->_outputpagecode == 'UTF-8') $val=$this->translateTexts(array($value),substr($this->_refLang,0,2),substr($my_destlang,0,2));
+            else $val=utf8_decode($this->translateTexts(array($value),substr($this->_refLang,0,2),substr($my_destlang,0,2)));
+        }
+
+        $val=trim($val);
+
+        if (empty($val)) return 0;
+
+        $this->_translatedFiles[$file][] = $key . '=' . $val ;
+        return 1;
+    }
+
+    /**
+     * getLineKey
+     *
+     * @param 	string 	$line	Line found into file
+     * @return	string	Key
+     */
+    private function getLineKey($line)
+    {
+        $arraykey = explode('=',$line,2);
+        return trim($arraykey[0]);
+    }
+
+    /**
+     * getLineValue
+     *
+     * @param 	string 	$line	Line found into file
+     * @return	string	Value
+     */
+    private function getLineValue($line)
+    {
+        $arraykey = explode('=',$line,2);
+        return trim(isset($arraykey[1])?$arraykey[1]:'');
+    }
+
+    /**
+     * getTranslationFilesArray
+     *
+     * @param 	string 	$lang	Language code
+     * @return	array	Array
+     */
+    private function getTranslationFilesArray($lang)
+    {
+        $dir = new DirectoryIterator($this->_langDir.$lang);
+        while($dir->valid()) {
+            if(!$dir->isDot() && $dir->isFile() && ! preg_match('/^\./',$dir->getFilename())) {
+                $files[] =  $dir->getFilename();
+            }
+            $dir->next();
+        }
+        return $files;
+    }
+
+    /**
+     * Return translation of a value
+     *
+     * @param 	array	$src_texts		Array with one value
+     * @param 	string	$src_lang		Language code source (us, fr, it, ...)
+     * @param 	string	$dest_lang		Language code target (es, de, ...)
+     * @return 	string					Value translated
+     */
+    private function translateTexts($src_texts, $src_lang, $dest_lang)
+    {
+        // We want to be sure that src_lang and dest_lang are using 2 chars only
+        $tmp=explode('_',$src_lang);
+        if (! empty($tmp[1]) && $tmp[0] == $tmp[1]) $src_lang=$tmp[0];
+        $tmp=explode('_',$dest_lang);
+        if (! empty($tmp[1]) && $tmp[0] == $tmp[1]) $dest_lang=$tmp[0];
+
+        //setting language pair
+        $lang_pair = $src_lang.'|'.$dest_lang;
+
+        $src_text_to_translate=preg_replace('/%s/','SSSSS',implode('',$src_texts));
+        $src_text_to_translate=preg_replace('/'.preg_quote('\n\n').'/',' NNNNN ',$src_text_to_translate);
+
+        // Define GET URL v1
+        //$url = "http://ajax.googleapis.com/ajax/services/language/translate?v=1.0&q=".urlencode($src_text_to_translate)."&langpair=".urlencode($lang_pair);
+        // Example: http://ajax.googleapis.com/ajax/services/language/translate?v=1.0&q=Setup%20area&langpair=en_US|fr_FR
         // Define GET URL v2
-		$url = "https://www.googleapis.com/language/translate/v2?key=".$this->_apikey."&q=".urlencode($src_text_to_translate)."&source=".urlencode($src_lang)."&target=".urlencode($dest_lang);
-		// Example: https://www.googleapis.com/language/translate/v2?key=_apikey&q=Setup%20area&source=en_US&target=fr_FR
-
-		// Send request
-		//print "Url to translate: ".$url."\n";
-
-		if (! function_exists("curl_init"))
-		{
-		      print "Error, your PHP does not support curl functions.\n";
-		      die();
-		}
-
-		$ch = curl_init();
-		curl_setopt($ch, CURLOPT_URL, $url);
-		curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1);
-		curl_setopt($ch, CURLOPT_REFERER, "Mozilla");
-		$body = curl_exec($ch);
-		curl_close($ch);
-		//sleep(1);	// This is to avoid to overload server.
-
-		// now, process the JSON string
-		$json = json_decode($body, true);
-
-		if ((! empty($json['responseStatus']) && $json['responseStatus'] != 200)
-		|| count($json['data']['translations']) == 0)
-		{
-		    print "Error: ".$json['responseStatus']." ".$url."\n";
-			return false;
-		}
-
-		$rep=$json['data']['translations'][0]['translatedText'];
-		$rep=preg_replace('/SSSSS/i','%s',$rep);
-		$rep=preg_replace('/NNNNN/i','\n\n',$rep);
-		$rep=preg_replace('/&#39;/i','\'',$rep);
-
-		//print "OK ".join('',$src_texts).' => '.$rep."\n";
-
-		return $rep;
-	}
+        $url = "https://www.googleapis.com/language/translate/v2?key=".$this->_apikey."&q=".urlencode($src_text_to_translate)."&source=".urlencode($src_lang)."&target=".urlencode($dest_lang);
+        // Example: https://www.googleapis.com/language/translate/v2?key=_apikey&q=Setup%20area&source=en_US&target=fr_FR
+
+        // Send request
+        //print "Url to translate: ".$url."\n";
+
+        if (! function_exists("curl_init"))
+        {
+                print "Error, your PHP does not support curl functions.\n";
+                die();
+        }
+
+        $ch = curl_init();
+        curl_setopt($ch, CURLOPT_URL, $url);
+        curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1);
+        curl_setopt($ch, CURLOPT_REFERER, "Mozilla");
+        $body = curl_exec($ch);
+        curl_close($ch);
+        //sleep(1);	// This is to avoid to overload server.
+
+        // now, process the JSON string
+        $json = json_decode($body, true);
+
+        if ((! empty($json['responseStatus']) && $json['responseStatus'] != 200)
+        || count($json['data']['translations']) == 0)
+        {
+            print "Error: ".$json['responseStatus']." ".$url."\n";
+            return false;
+        }
+
+        $rep=$json['data']['translations'][0]['translatedText'];
+        $rep=preg_replace('/SSSSS/i','%s',$rep);
+        $rep=preg_replace('/NNNNN/i','\n\n',$rep);
+        $rep=preg_replace('/&#39;/i','\'',$rep);
+
+        //print "OK ".join('',$src_texts).' => '.$rep."\n";
+
+        return $rep;
+    }
 }

Spacing +65 added lines, -65 removed lines

@@ -50,7 +50,7 @@ 
      * @param   string $_apikey         Api key
      * @return void
      */
-    function __construct($_destlang,$_refLang,$_langDir,$_limittofile,$_apikey)
+    function __construct($_destlang, $_refLang, $_langDir, $_limittofile, $_apikey)
     {
 
 		// Set enviorment variables
@@ -63,7 +63,7 @@ 
 
 		// Translate
 		//ini_set('default_charset','UTF-8');
-		ini_set('default_charset',$this->_outputpagecode);
+		ini_set('default_charset', $this->_outputpagecode);
 		$this->parseRefLangTranslationFiles();
 	}
 
@@ -77,72 +77,72 @@ 
 
 		$files = $this->getTranslationFilesArray($this->_refLang);
 		$counter = 1;
-		foreach($files as $file)
+		foreach ($files as $file)
 		{
 			if ($this->_limittofile && $this->_limittofile != $file) continue;
 			$counter++;
 			$fileContent = null;
 			$refPath = $this->_langDir.$this->_refLang.self::DIR_SEPARATOR.$file;
-			$fileContent = file($refPath,FILE_IGNORE_NEW_LINES|FILE_SKIP_EMPTY_LINES);
-			print "Processing file " . $file . ", with ".count($fileContent)." lines<br>\n";
+			$fileContent = file($refPath, FILE_IGNORE_NEW_LINES | FILE_SKIP_EMPTY_LINES);
+			print "Processing file ".$file.", with ".count($fileContent)." lines<br>\n";
 
 			// Define target dirs
-			$targetlangs=array($this->_destlang);
+			$targetlangs = array($this->_destlang);
 			if ($this->_destlang == 'all')
 			{
-				$targetlangs=array();
+				$targetlangs = array();
 
 				// If we must process all languages
-				$arraytmp=dol_dir_list($this->_langDir,'directories',0);
-				foreach($arraytmp as $dirtmp)
+				$arraytmp = dol_dir_list($this->_langDir, 'directories', 0);
+				foreach ($arraytmp as $dirtmp)
 				{
-					if ($dirtmp['name'] === $this->_refLang) continue;	// We discard source language
-					$tmppart=explode('_',$dirtmp['name']);
-					if (preg_match('/^en/i',$dirtmp['name']))  continue;	// We discard en_* languages
-					if (preg_match('/^fr/i',$dirtmp['name']))  continue;	// We discard fr_* languages
-					if (preg_match('/^es/i',$dirtmp['name']))  continue;	// We discard es_* languages
-					if (preg_match('/ca_ES/i',$dirtmp['name']))  continue;	// We discard es_CA language
-					if (preg_match('/pt_BR/i',$dirtmp['name']))  continue;	// We discard pt_BR language
-                    if (preg_match('/nl_BE/i',$dirtmp['name']))  continue;  // We discard nl_BE language
-					if (preg_match('/^\./i',$dirtmp['name']))  continue;	// We discard files .*
-					if (preg_match('/^CVS/i',$dirtmp['name']))  continue;	// We discard CVS
-					$targetlangs[]=$dirtmp['name'];
+					if ($dirtmp['name'] === $this->_refLang) continue; // We discard source language
+					$tmppart = explode('_', $dirtmp['name']);
+					if (preg_match('/^en/i', $dirtmp['name']))  continue; // We discard en_* languages
+					if (preg_match('/^fr/i', $dirtmp['name']))  continue; // We discard fr_* languages
+					if (preg_match('/^es/i', $dirtmp['name']))  continue; // We discard es_* languages
+					if (preg_match('/ca_ES/i', $dirtmp['name']))  continue; // We discard es_CA language
+					if (preg_match('/pt_BR/i', $dirtmp['name']))  continue; // We discard pt_BR language
+                    if (preg_match('/nl_BE/i', $dirtmp['name']))  continue; // We discard nl_BE language
+					if (preg_match('/^\./i', $dirtmp['name']))  continue; // We discard files .*
+					if (preg_match('/^CVS/i', $dirtmp['name']))  continue; // We discard CVS
+					$targetlangs[] = $dirtmp['name'];
 				}
 				//var_dump($targetlangs);
 			}
 
 			// Process translation of source file for each target languages
-			foreach($targetlangs as $my_destlang)
+			foreach ($targetlangs as $my_destlang)
 			{
 				$this->_translatedFiles = array();
 
 				$destPath = $this->_langDir.$my_destlang.self::DIR_SEPARATOR.$file;
 				// Check destination file presence
-				if (! file_exists($destPath))
+				if (!file_exists($destPath))
 				{
 					// No file present, we generate file
-					echo "File not found: " . $destPath . ". We generate it.<br>\n";
-					$this->createTranslationFile($destPath,$my_destlang);
+					echo "File not found: ".$destPath.". We generate it.<br>\n";
+					$this->createTranslationFile($destPath, $my_destlang);
 				}
 				else
 				{
-					echo "Updating file: " . $destPath . "<br>\n";
+					echo "Updating file: ".$destPath."<br>\n";
 				}
 
 				// Translate lines
-				$fileContentDest = file($destPath,FILE_IGNORE_NEW_LINES|FILE_SKIP_EMPTY_LINES);
-				$newlines=0;
-				foreach($fileContent as $line){
+				$fileContentDest = file($destPath, FILE_IGNORE_NEW_LINES | FILE_SKIP_EMPTY_LINES);
+				$newlines = 0;
+				foreach ($fileContent as $line) {
 					$key = $this->getLineKey($line);
 					$value = $this->getLineValue($line);
 					if ($key && $value)
 					{
-						$newlines+=$this->translateFileLine($fileContentDest,$file,$key,$value,$my_destlang);
+						$newlines += $this->translateFileLine($fileContentDest, $file, $key, $value, $my_destlang);
 					}
 				}
 
-				$this->updateTranslationFile($destPath,$file,$my_destlang);
-				echo "New translated lines: " . $newlines . "<br>\n";
+				$this->updateTranslationFile($destPath, $file, $my_destlang);
+				echo "New translated lines: ".$newlines."<br>\n";
 				//if ($counter ==3) die('fim');
 			}
 		}
@@ -156,19 +156,19 @@ 
 	 * @param 	string	$my_destlang		Target language code
 	 * @return	void
 	 */
-	private function updateTranslationFile($destPath,$file,$my_destlang)
+	private function updateTranslationFile($destPath, $file, $my_destlang)
 	{
 		$this->_time_end = date('Y-m-d H:i:s');
 
-		if (isset($this->_translatedFiles[$file]) && count($this->_translatedFiles[$file])>0)
+		if (isset($this->_translatedFiles[$file]) && count($this->_translatedFiles[$file]) > 0)
 		{
 			$fp = fopen($destPath, 'a');
 			fwrite($fp, "\n");
 			fwrite($fp, "\n");
 			fwrite($fp, "// START - Lines generated via autotranslator.php tool (".$this->_time.").\n");
 			fwrite($fp, "// Reference language: ".$this->_refLang." -> ".$my_destlang."\n");
-			foreach( $this->_translatedFiles[$file] as $line) {
-				fwrite($fp, $line . "\n");
+			foreach ($this->_translatedFiles[$file] as $line) {
+				fwrite($fp, $line."\n");
 			}
 			fwrite($fp, "// STOP - Lines generated via autotranslator.php tool (".$this->_time_end.").\n");
 			fclose($fp);
@@ -183,13 +183,13 @@ 
 	 * @param 	string	$my_destlang		Target language code
 	 * @return	void
 	 */
-	private function createTranslationFile($path,$my_destlang)
+	private function createTranslationFile($path, $my_destlang)
 	{
 		$fp = fopen($path, 'w+');
 		fwrite($fp, "/*\n");
 		fwrite($fp, " * Language code: {$my_destlang}\n");
 		fwrite($fp, " * Automatic generated via autotranslator.php tool\n");
-		fwrite($fp, " * Generation date " . $this->_time. "\n");
+		fwrite($fp, " * Generation date ".$this->_time."\n");
 		fwrite($fp, " */\n");
 		fclose($fp);
 		return;
@@ -205,36 +205,36 @@ 
 	 * @param	string	$my_destlang	Language code (ie: fr_FR)
 	 * @return	int						0=Nothing translated, 1=Record translated
 	 */
-	private function translateFileLine($content,$file,$key,$value,$my_destlang)
+	private function translateFileLine($content, $file, $key, $value, $my_destlang)
 	{
 
 		//print "key    =".$key."\n";
-		foreach( $content as $line ) {
+		foreach ($content as $line) {
 			$destKey = $this->getLineKey($line);
 			$destValue = $this->getLineValue($line);
 			// If translated return
 			//print "destKey=".$destKey."\n";
-			if ( trim($destKey) == trim($key) )
+			if (trim($destKey) == trim($key))
 			{	// Found already existing translation (key already exits in dest file)
 				return 0;
 			}
 		}
 
-		if ($key == 'CHARSET') $val=$this->_outputpagecode;
-		else if (preg_match('/^Format/',$key)) $val=$value;
-		else if ($value=='-') $val=$value;
+		if ($key == 'CHARSET') $val = $this->_outputpagecode;
+		else if (preg_match('/^Format/', $key)) $val = $value;
+		else if ($value == '-') $val = $value;
 		else
 		{
 			// If not translated then translate
-			if ($this->_outputpagecode == 'UTF-8') $val=$this->translateTexts(array($value),substr($this->_refLang,0,2),substr($my_destlang,0,2));
-			else $val=utf8_decode($this->translateTexts(array($value),substr($this->_refLang,0,2),substr($my_destlang,0,2)));
+			if ($this->_outputpagecode == 'UTF-8') $val = $this->translateTexts(array($value), substr($this->_refLang, 0, 2), substr($my_destlang, 0, 2));
+			else $val = utf8_decode($this->translateTexts(array($value), substr($this->_refLang, 0, 2), substr($my_destlang, 0, 2)));
 		}
 
-		$val=trim($val);
+		$val = trim($val);
 
 		if (empty($val)) return 0;
 
-		$this->_translatedFiles[$file][] = $key . '=' . $val ;
+		$this->_translatedFiles[$file][] = $key.'='.$val;
 		return 1;
 	}
 
@@ -246,7 +246,7 @@ 
 	 */
 	private function getLineKey($line)
 	{
-		$arraykey = explode('=',$line,2);
+		$arraykey = explode('=', $line, 2);
 		return trim($arraykey[0]);
 	}
 
@@ -258,8 +258,8 @@ 
 	 */
 	private function getLineValue($line)
 	{
-		$arraykey = explode('=',$line,2);
-		return trim(isset($arraykey[1])?$arraykey[1]:'');
+		$arraykey = explode('=', $line, 2);
+		return trim(isset($arraykey[1]) ? $arraykey[1] : '');
 	}
 
 	/**
@@ -271,9 +271,9 @@ 
 	private function getTranslationFilesArray($lang)
 	{
 		$dir = new DirectoryIterator($this->_langDir.$lang);
-		while($dir->valid()) {
-			if(!$dir->isDot() && $dir->isFile() && ! preg_match('/^\./',$dir->getFilename())) {
-				$files[] =  $dir->getFilename();
+		while ($dir->valid()) {
+			if (!$dir->isDot() && $dir->isFile() && !preg_match('/^\./', $dir->getFilename())) {
+				$files[] = $dir->getFilename();
 			}
 			$dir->next();
 		}
@@ -291,16 +291,16 @@ 
 	private function translateTexts($src_texts, $src_lang, $dest_lang)
 	{
 		// We want to be sure that src_lang and dest_lang are using 2 chars only
-		$tmp=explode('_',$src_lang);
-		if (! empty($tmp[1]) && $tmp[0] == $tmp[1]) $src_lang=$tmp[0];
-		$tmp=explode('_',$dest_lang);
-		if (! empty($tmp[1]) && $tmp[0] == $tmp[1]) $dest_lang=$tmp[0];
+		$tmp = explode('_', $src_lang);
+		if (!empty($tmp[1]) && $tmp[0] == $tmp[1]) $src_lang = $tmp[0];
+		$tmp = explode('_', $dest_lang);
+		if (!empty($tmp[1]) && $tmp[0] == $tmp[1]) $dest_lang = $tmp[0];
 
 		//setting language pair
 		$lang_pair = $src_lang.'|'.$dest_lang;
 
-		$src_text_to_translate=preg_replace('/%s/','SSSSS',implode('',$src_texts));
-		$src_text_to_translate=preg_replace('/'.preg_quote('\n\n').'/',' NNNNN ',$src_text_to_translate);
+		$src_text_to_translate = preg_replace('/%s/', 'SSSSS', implode('', $src_texts));
+		$src_text_to_translate = preg_replace('/'.preg_quote('\n\n').'/', ' NNNNN ', $src_text_to_translate);
 
 		// Define GET URL v1
 		//$url = "http://ajax.googleapis.com/ajax/services/language/translate?v=1.0&q=".urlencode($src_text_to_translate)."&langpair=".urlencode($lang_pair);
@@ -312,7 +312,7 @@ 
 		// Send request
 		//print "Url to translate: ".$url."\n";
 
-		if (! function_exists("curl_init"))
+		if (!function_exists("curl_init"))
 		{
 		      print "Error, your PHP does not support curl functions.\n";
 		      die();
@@ -329,17 +329,17 @@ 
 		// now, process the JSON string
 		$json = json_decode($body, true);
 
-		if ((! empty($json['responseStatus']) && $json['responseStatus'] != 200)
+		if ((!empty($json['responseStatus']) && $json['responseStatus'] != 200)
 		|| count($json['data']['translations']) == 0)
 		{
 		    print "Error: ".$json['responseStatus']." ".$url."\n";
 			return false;
 		}
 
-		$rep=$json['data']['translations'][0]['translatedText'];
-		$rep=preg_replace('/SSSSS/i','%s',$rep);
-		$rep=preg_replace('/NNNNN/i','\n\n',$rep);
-		$rep=preg_replace('/&#39;/i','\'',$rep);
+		$rep = $json['data']['translations'][0]['translatedText'];
+		$rep = preg_replace('/SSSSS/i', '%s', $rep);
+		$rep = preg_replace('/NNNNN/i', '\n\n', $rep);
+		$rep = preg_replace('/&#39;/i', '\'', $rep);
 
 		//print "OK ".join('',$src_texts).' => '.$rep."\n";
 

Braces +61 added lines, -21 removed lines

@@ -79,7 +79,9 @@ 
 		$counter = 1;
 		foreach($files as $file)
 		{
-			if ($this->_limittofile && $this->_limittofile != $file) continue;
+			if ($this->_limittofile && $this->_limittofile != $file) {
+			    continue;
+			}
 			$counter++;
 			$fileContent = null;
 			$refPath = $this->_langDir.$this->_refLang.self::DIR_SEPARATOR.$file;
@@ -96,16 +98,43 @@ 
 				$arraytmp=dol_dir_list($this->_langDir,'directories',0);
 				foreach($arraytmp as $dirtmp)
 				{
-					if ($dirtmp['name'] === $this->_refLang) continue;	// We discard source language
+					if ($dirtmp['name'] === $this->_refLang) {
+					    continue;
+					}
+					// We discard source language
 					$tmppart=explode('_',$dirtmp['name']);
-					if (preg_match('/^en/i',$dirtmp['name']))  continue;	// We discard en_* languages
-					if (preg_match('/^fr/i',$dirtmp['name']))  continue;	// We discard fr_* languages
-					if (preg_match('/^es/i',$dirtmp['name']))  continue;	// We discard es_* languages
-					if (preg_match('/ca_ES/i',$dirtmp['name']))  continue;	// We discard es_CA language
-					if (preg_match('/pt_BR/i',$dirtmp['name']))  continue;	// We discard pt_BR language
-                    if (preg_match('/nl_BE/i',$dirtmp['name']))  continue;  // We discard nl_BE language
-					if (preg_match('/^\./i',$dirtmp['name']))  continue;	// We discard files .*
-					if (preg_match('/^CVS/i',$dirtmp['name']))  continue;	// We discard CVS
+					if (preg_match('/^en/i',$dirtmp['name'])) {
+					    continue;
+					}
+					// We discard en_* languages
+					if (preg_match('/^fr/i',$dirtmp['name'])) {
+					    continue;
+					}
+					// We discard fr_* languages
+					if (preg_match('/^es/i',$dirtmp['name'])) {
+					    continue;
+					}
+					// We discard es_* languages
+					if (preg_match('/ca_ES/i',$dirtmp['name'])) {
+					    continue;
+					}
+					// We discard es_CA language
+					if (preg_match('/pt_BR/i',$dirtmp['name'])) {
+					    continue;
+					}
+					// We discard pt_BR language
+                    if (preg_match('/nl_BE/i',$dirtmp['name'])) {
+                        continue;
+                    }
+                    // We discard nl_BE language
+					if (preg_match('/^\./i',$dirtmp['name'])) {
+					    continue;
+					}
+					// We discard files .*
+					if (preg_match('/^CVS/i',$dirtmp['name'])) {
+					    continue;
+					}
+					// We discard CVS
 					$targetlangs[]=$dirtmp['name'];
 				}
 				//var_dump($targetlangs);
@@ -123,8 +152,7 @@ 
 					// No file present, we generate file
 					echo "File not found: " . $destPath . ". We generate it.<br>\n";
 					$this->createTranslationFile($destPath,$my_destlang);
-				}
-				else
+				} else
 				{
 					echo "Updating file: " . $destPath . "<br>\n";
 				}
@@ -220,19 +248,27 @@ 
 			}
 		}
 
-		if ($key == 'CHARSET') $val=$this->_outputpagecode;
-		else if (preg_match('/^Format/',$key)) $val=$value;
-		else if ($value=='-') $val=$value;
-		else
+		if ($key == 'CHARSET') {
+		    $val=$this->_outputpagecode;
+		} else if (preg_match('/^Format/',$key)) {
+		    $val=$value;
+		} else if ($value=='-') {
+		    $val=$value;
+		} else
 		{
 			// If not translated then translate
-			if ($this->_outputpagecode == 'UTF-8') $val=$this->translateTexts(array($value),substr($this->_refLang,0,2),substr($my_destlang,0,2));
-			else $val=utf8_decode($this->translateTexts(array($value),substr($this->_refLang,0,2),substr($my_destlang,0,2)));
+			if ($this->_outputpagecode == 'UTF-8') {
+			    $val=$this->translateTexts(array($value),substr($this->_refLang,0,2),substr($my_destlang,0,2));
+			} else {
+			    $val=utf8_decode($this->translateTexts(array($value),substr($this->_refLang,0,2),substr($my_destlang,0,2)));
+			}
 		}
 
 		$val=trim($val);
 
-		if (empty($val)) return 0;
+		if (empty($val)) {
+		    return 0;
+		}
 
 		$this->_translatedFiles[$file][] = $key . '=' . $val ;
 		return 1;
@@ -292,9 +328,13 @@ 
 	{
 		// We want to be sure that src_lang and dest_lang are using 2 chars only
 		$tmp=explode('_',$src_lang);
-		if (! empty($tmp[1]) && $tmp[0] == $tmp[1]) $src_lang=$tmp[0];
+		if (! empty($tmp[1]) && $tmp[0] == $tmp[1]) {
+		    $src_lang=$tmp[0];
+		}
 		$tmp=explode('_',$dest_lang);
-		if (! empty($tmp[1]) && $tmp[0] == $tmp[1]) $dest_lang=$tmp[0];
+		if (! empty($tmp[1]) && $tmp[0] == $tmp[1]) {
+		    $dest_lang=$tmp[0];
+		}
 
 		//setting language pair
 		$lang_pair = $src_lang.'|'.$dest_lang;

dolibarr/dev/initdemo/sftpget_and_loaddump.php

Indentation +46 added lines, -46 removed lines

@@ -25,8 +25,8 @@ 
 
 // Test if batch mode
 if (substr($sapi_type, 0, 3) == 'cgi') {
-	echo "Error: You are using PHP for CGI. To execute ".$script_file." from command line, you must use PHP for CLI mode.\n";
-	exit;
+    echo "Error: You are using PHP for CGI. To execute ".$script_file." from command line, you must use PHP for CLI mode.\n";
+    exit;
 }
 
 // Global variables
@@ -60,17 +60,17 @@ 
 $server='';
 if (preg_match('/^(.*)@(.*):(.*)$/',$sourceserver,$reg))
 {
-	$login=$reg[1];
-	$server=$reg[2];
-	$sourcefile=$reg[3];
-	$targetfile=basename($sourcefile);
+    $login=$reg[1];
+    $server=$reg[2];
+    $sourcefile=$reg[3];
+    $targetfile=basename($sourcefile);
 }
 if (empty($sourceserver) || empty($server) || empty($login) || empty($sourcefile) || empty($password) || empty($database) || empty($loginbase) || empty($passwordbase))
 {
-	print "Usage: $script_file login@server:/src/file.(sql|gz|bz2) passssh databaseserver databasename loginbase passbase\n";
-	print "Return code: 0 if success, <>0 if error\n";
-	print "Warning, this script may take a long time.\n";
-	exit(-1);
+    print "Usage: $script_file login@server:/src/file.(sql|gz|bz2) passssh databaseserver databasename loginbase passbase\n";
+    print "Return code: 0 if success, <>0 if error\n";
+    print "Warning, this script may take a long time.\n";
+    exit(-1);
 }
 
 
@@ -84,21 +84,21 @@ 
 
 // SFTP connect
 if (! function_exists("ssh2_connect")) {
-	dol_print_error('','ssh2_connect function does not exists'); exit(1);
+    dol_print_error('','ssh2_connect function does not exists'); exit(1);
 }
 
 $connection = ssh2_connect($server, 22);
 if ($connection)
 {
-	if (! @ssh2_auth_password($connection, $login, $password))
-	{
-		dol_syslog("Could not authenticate with username ".$login." . and password ".preg_replace('/./', '*', $password),LOG_ERR);
-		exit(-5);
-	}
-	else
-	{
-		//$stream = ssh2_exec($connection, '/usr/bin/php -i');
-		/*
+    if (! @ssh2_auth_password($connection, $login, $password))
+    {
+        dol_syslog("Could not authenticate with username ".$login." . and password ".preg_replace('/./', '*', $password),LOG_ERR);
+        exit(-5);
+    }
+    else
+    {
+        //$stream = ssh2_exec($connection, '/usr/bin/php -i');
+        /*
 		print "Generate dump ".$filesys1.'.bz2'."\n";
 			$stream = ssh2_exec($connection, "mysqldump -u debian-sys-maint -p4k9Blxl2snq4FHXY -h 127.0.0.1 --single-transaction -K --tables -c -e --hex-blob --default-character-set=utf8 saasplex | bzip2 -1 > ".$filesys1.'.bz2');
 			stream_set_blocking($stream, true);
@@ -106,35 +106,35 @@ 
 			$output = stream_get_contents($stream);
 		*/
 
-		$sftp = ssh2_sftp($connection);
-
-		print 'Get file '.$sourcefile.' into '.$targetdir.$targetfile."\n";
-		ssh2_scp_recv($connection, $sourcefile, $targetdir.$targetfile);
-
-		$fullcommand="cat ".$targetdir.$targetfile." | mysql -h".$databaseserver." -u".$loginbase." -p".$passwordbase." -D ".$database;
-		if (preg_match('/\.bz2$/',$targetfile))
-		{
-			$fullcommand="bzip2 -c -d ".$targetdir.$targetfile." | mysql -h".$databaseserver." -u".$loginbase." -p".$passwordbase." -D ".$database;
-		}
-		if (preg_match('/\.gz$/',$targetfile))
-		{
-			$fullcommand="gzip -d ".$targetdir.$targetfile." | mysql -h".$databaseserver." -u".$loginbase." -p".$passwordbase." -D ".$database;
-		}
-		print "Load dump with ".$fullcommand."\n";
-		$output=array();
-		$return_var=0;
-		print strftime("%Y%m%d-%H%M%S").' '.$fullcommand."\n";
-		exec($fullcommand, $output, $return_var);
-		foreach($output as $line) print $line."\n";
-
-		//ssh2_sftp_unlink($sftp, $fileinstalllock);
-		//print $output;
-	}
+        $sftp = ssh2_sftp($connection);
+
+        print 'Get file '.$sourcefile.' into '.$targetdir.$targetfile."\n";
+        ssh2_scp_recv($connection, $sourcefile, $targetdir.$targetfile);
+
+        $fullcommand="cat ".$targetdir.$targetfile." | mysql -h".$databaseserver." -u".$loginbase." -p".$passwordbase." -D ".$database;
+        if (preg_match('/\.bz2$/',$targetfile))
+        {
+            $fullcommand="bzip2 -c -d ".$targetdir.$targetfile." | mysql -h".$databaseserver." -u".$loginbase." -p".$passwordbase." -D ".$database;
+        }
+        if (preg_match('/\.gz$/',$targetfile))
+        {
+            $fullcommand="gzip -d ".$targetdir.$targetfile." | mysql -h".$databaseserver." -u".$loginbase." -p".$passwordbase." -D ".$database;
+        }
+        print "Load dump with ".$fullcommand."\n";
+        $output=array();
+        $return_var=0;
+        print strftime("%Y%m%d-%H%M%S").' '.$fullcommand."\n";
+        exec($fullcommand, $output, $return_var);
+        foreach($output as $line) print $line."\n";
+
+        //ssh2_sftp_unlink($sftp, $fileinstalllock);
+        //print $output;
+    }
 }
 else
 {
-	print 'Failed to connect to ssh2 to '.$server;
-	exit(-6);
+    print 'Failed to connect to ssh2 to '.$server;
+    exit(-6);
 }
 
 

Spacing +38 added lines, -38 removed lines

@@ -21,7 +21,7 @@ 
 
 $sapi_type = php_sapi_name();
 $script_file = basename(__FILE__);
-$path=dirname(__FILE__).'/';
+$path = dirname(__FILE__).'/';
 
 // Test if batch mode
 if (substr($sapi_type, 0, 3) == 'cgi') {
@@ -30,25 +30,25 @@ 
 }
 
 // Global variables
-$error=0;
+$error = 0;
 
-$sourceserver=isset($argv[1])?$argv[1]:'';		// user@server:/src/file
-$password=isset($argv[2])?$argv[2]:'';
-$dataserver=isset($argv[3])?$argv[3]:'';
-$database=isset($argv[4])?$argv[4]:'';
-$loginbase=isset($argv[5])?$argv[5]:'';
-$passwordbase=isset($argv[6])?$argv[6]:'';
+$sourceserver = isset($argv[1]) ? $argv[1] : ''; // user@server:/src/file
+$password = isset($argv[2]) ? $argv[2] : '';
+$dataserver = isset($argv[3]) ? $argv[3] : '';
+$database = isset($argv[4]) ? $argv[4] : '';
+$loginbase = isset($argv[5]) ? $argv[5] : '';
+$passwordbase = isset($argv[6]) ? $argv[6] : '';
 
 // Include Dolibarr environment
-$res=0;
-if (! $res && file_exists($path."../../master.inc.php")) $res=@include $path."../../master.inc.php";
-if (! $res && file_exists($path."../../htdocs/master.inc.php")) $res=@include $path."../../htdocs/master.inc.php";
-if (! $res && file_exists("../master.inc.php")) $res=@include "../master.inc.php";
-if (! $res && file_exists("../../master.inc.php")) $res=@include "../../master.inc.php";
-if (! $res && file_exists("../../../master.inc.php")) $res=@include "../../../master.inc.php";
-if (! $res && preg_match('/\/nltechno([^\/]*)\//',$_SERVER["PHP_SELF"],$reg)) $res=@include $path."../../../dolibarr".$reg[1]."/htdocs/master.inc.php"; // Used on dev env only
-if (! $res && preg_match('/\/nltechno([^\/]*)\//',$_SERVER["PHP_SELF"],$reg)) $res=@include "../../../dolibarr".$reg[1]."/htdocs/master.inc.php"; // Used on dev env only
-if (! $res) die ("Failed to include master.inc.php file\n");
+$res = 0;
+if (!$res && file_exists($path."../../master.inc.php")) $res = @include $path."../../master.inc.php";
+if (!$res && file_exists($path."../../htdocs/master.inc.php")) $res = @include $path."../../htdocs/master.inc.php";
+if (!$res && file_exists("../master.inc.php")) $res = @include "../master.inc.php";
+if (!$res && file_exists("../../master.inc.php")) $res = @include "../../master.inc.php";
+if (!$res && file_exists("../../../master.inc.php")) $res = @include "../../../master.inc.php";
+if (!$res && preg_match('/\/nltechno([^\/]*)\//', $_SERVER["PHP_SELF"], $reg)) $res = @include $path."../../../dolibarr".$reg[1]."/htdocs/master.inc.php"; // Used on dev env only
+if (!$res && preg_match('/\/nltechno([^\/]*)\//', $_SERVER["PHP_SELF"], $reg)) $res = @include "../../../dolibarr".$reg[1]."/htdocs/master.inc.php"; // Used on dev env only
+if (!$res) die ("Failed to include master.inc.php file\n");
 include_once DOL_DOCUMENT_ROOT.'/core/lib/files.lib.php';
 
 
@@ -56,14 +56,14 @@ 
  *	Main
  */
 
-$login='';
-$server='';
-if (preg_match('/^(.*)@(.*):(.*)$/',$sourceserver,$reg))
+$login = '';
+$server = '';
+if (preg_match('/^(.*)@(.*):(.*)$/', $sourceserver, $reg))
 {
-	$login=$reg[1];
-	$server=$reg[2];
-	$sourcefile=$reg[3];
-	$targetfile=basename($sourcefile);
+	$login = $reg[1];
+	$server = $reg[2];
+	$sourcefile = $reg[3];
+	$targetfile = basename($sourcefile);
 }
 if (empty($sourceserver) || empty($server) || empty($login) || empty($sourcefile) || empty($password) || empty($database) || empty($loginbase) || empty($passwordbase))
 {
@@ -74,25 +74,25 @@ 
 }
 
 
-$targetdir='/tmp';
+$targetdir = '/tmp';
 print "Get dump file from server ".$server.", path ".$sourcefile.", connect with login ".$login." loaded into localhost\n";
 
-$sftpconnectstring=$sourceserver;
+$sftpconnectstring = $sourceserver;
 print 'SFTP connect string : '.$sftpconnectstring."\n";
 //print 'SFTP password '.$password."\n";
 
 
 // SFTP connect
-if (! function_exists("ssh2_connect")) {
-	dol_print_error('','ssh2_connect function does not exists'); exit(1);
+if (!function_exists("ssh2_connect")) {
+	dol_print_error('', 'ssh2_connect function does not exists'); exit(1);
 }
 
 $connection = ssh2_connect($server, 22);
 if ($connection)
 {
-	if (! @ssh2_auth_password($connection, $login, $password))
+	if (!@ssh2_auth_password($connection, $login, $password))
 	{
-		dol_syslog("Could not authenticate with username ".$login." . and password ".preg_replace('/./', '*', $password),LOG_ERR);
+		dol_syslog("Could not authenticate with username ".$login." . and password ".preg_replace('/./', '*', $password), LOG_ERR);
 		exit(-5);
 	}
 	else
@@ -111,21 +111,21 @@ 
 		print 'Get file '.$sourcefile.' into '.$targetdir.$targetfile."\n";
 		ssh2_scp_recv($connection, $sourcefile, $targetdir.$targetfile);
 
-		$fullcommand="cat ".$targetdir.$targetfile." | mysql -h".$databaseserver." -u".$loginbase." -p".$passwordbase." -D ".$database;
-		if (preg_match('/\.bz2$/',$targetfile))
+		$fullcommand = "cat ".$targetdir.$targetfile." | mysql -h".$databaseserver." -u".$loginbase." -p".$passwordbase." -D ".$database;
+		if (preg_match('/\.bz2$/', $targetfile))
 		{
-			$fullcommand="bzip2 -c -d ".$targetdir.$targetfile." | mysql -h".$databaseserver." -u".$loginbase." -p".$passwordbase." -D ".$database;
+			$fullcommand = "bzip2 -c -d ".$targetdir.$targetfile." | mysql -h".$databaseserver." -u".$loginbase." -p".$passwordbase." -D ".$database;
 		}
-		if (preg_match('/\.gz$/',$targetfile))
+		if (preg_match('/\.gz$/', $targetfile))
 		{
-			$fullcommand="gzip -d ".$targetdir.$targetfile." | mysql -h".$databaseserver." -u".$loginbase." -p".$passwordbase." -D ".$database;
+			$fullcommand = "gzip -d ".$targetdir.$targetfile." | mysql -h".$databaseserver." -u".$loginbase." -p".$passwordbase." -D ".$database;
 		}
 		print "Load dump with ".$fullcommand."\n";
-		$output=array();
-		$return_var=0;
+		$output = array();
+		$return_var = 0;
 		print strftime("%Y%m%d-%H%M%S").' '.$fullcommand."\n";
 		exec($fullcommand, $output, $return_var);
-		foreach($output as $line) print $line."\n";
+		foreach ($output as $line) print $line."\n";
 
 		//ssh2_sftp_unlink($sftp, $fileinstalllock);
 		//print $output;

Braces +31 added lines, -13 removed lines

@@ -41,14 +41,32 @@ 
 
 // Include Dolibarr environment
 $res=0;
-if (! $res && file_exists($path."../../master.inc.php")) $res=@include $path."../../master.inc.php";
-if (! $res && file_exists($path."../../htdocs/master.inc.php")) $res=@include $path."../../htdocs/master.inc.php";
-if (! $res && file_exists("../master.inc.php")) $res=@include "../master.inc.php";
-if (! $res && file_exists("../../master.inc.php")) $res=@include "../../master.inc.php";
-if (! $res && file_exists("../../../master.inc.php")) $res=@include "../../../master.inc.php";
-if (! $res && preg_match('/\/nltechno([^\/]*)\//',$_SERVER["PHP_SELF"],$reg)) $res=@include $path."../../../dolibarr".$reg[1]."/htdocs/master.inc.php"; // Used on dev env only
-if (! $res && preg_match('/\/nltechno([^\/]*)\//',$_SERVER["PHP_SELF"],$reg)) $res=@include "../../../dolibarr".$reg[1]."/htdocs/master.inc.php"; // Used on dev env only
-if (! $res) die ("Failed to include master.inc.php file\n");
+if (! $res && file_exists($path."../../master.inc.php")) {
+    $res=@include $path."../../master.inc.php";
+}
+if (! $res && file_exists($path."../../htdocs/master.inc.php")) {
+    $res=@include $path."../../htdocs/master.inc.php";
+}
+if (! $res && file_exists("../master.inc.php")) {
+    $res=@include "../master.inc.php";
+}
+if (! $res && file_exists("../../master.inc.php")) {
+    $res=@include "../../master.inc.php";
+}
+if (! $res && file_exists("../../../master.inc.php")) {
+    $res=@include "../../../master.inc.php";
+}
+if (! $res && preg_match('/\/nltechno([^\/]*)\//',$_SERVER["PHP_SELF"],$reg)) {
+    $res=@include $path."../../../dolibarr".$reg[1]."/htdocs/master.inc.php";
+}
+// Used on dev env only
+if (! $res && preg_match('/\/nltechno([^\/]*)\//',$_SERVER["PHP_SELF"],$reg)) {
+    $res=@include "../../../dolibarr".$reg[1]."/htdocs/master.inc.php";
+}
+// Used on dev env only
+if (! $res) {
+    die ("Failed to include master.inc.php file\n");
+}
 include_once DOL_DOCUMENT_ROOT.'/core/lib/files.lib.php';
 
 
@@ -94,8 +112,7 @@ 
 	{
 		dol_syslog("Could not authenticate with username ".$login." . and password ".preg_replace('/./', '*', $password),LOG_ERR);
 		exit(-5);
-	}
-	else
+	} else
 	{
 		//$stream = ssh2_exec($connection, '/usr/bin/php -i');
 		/*
@@ -125,13 +142,14 @@ 
 		$return_var=0;
 		print strftime("%Y%m%d-%H%M%S").' '.$fullcommand."\n";
 		exec($fullcommand, $output, $return_var);
-		foreach($output as $line) print $line."\n";
+		foreach($output as $line) {
+		    print $line."\n";
+		}
 
 		//ssh2_sftp_unlink($sftp, $fileinstalllock);
 		//print $output;
 	}
-}
-else
+} else
 {
 	print 'Failed to connect to ssh2 to '.$server;
 	exit(-6);

dolibarr/dev/initdemo/updatedemo.php

Indentation +5 added lines, -5 removed lines

@@ -25,8 +25,8 @@ 
 
 // Test if batch mode
 if (substr($sapi_type, 0, 3) == 'cgi') {
-	echo "Error: You are using PHP for CGI. To execute ".$script_file." from command line, you must use PHP for CLI mode.\n";
-	exit;
+    echo "Error: You are using PHP for CGI. To execute ".$script_file." from command line, you must use PHP for CLI mode.\n";
+    exit;
 }
 
 // Global variables
@@ -53,9 +53,9 @@ 
 
 if (empty($confirm))
 {
-	print "Usage: $script_file confirm\n";
-	print "Return code: 0 if success, <>0 if error\n";
-	exit(-1);
+    print "Usage: $script_file confirm\n";
+    print "Return code: 0 if success, <>0 if error\n";
+    exit(-1);
 }
 
 

Spacing +28 added lines, -28 removed lines

@@ -21,7 +21,7 @@ 
 
 $sapi_type = php_sapi_name();
 $script_file = basename(__FILE__);
-$path=dirname(__FILE__).'/';
+$path = dirname(__FILE__).'/';
 
 // Test if batch mode
 if (substr($sapi_type, 0, 3) == 'cgi') {
@@ -30,20 +30,20 @@ 
 }
 
 // Global variables
-$error=0;
+$error = 0;
 
-$confirm=isset($argv[1])?$argv[1]:'';
+$confirm = isset($argv[1]) ? $argv[1] : '';
 
 // Include Dolibarr environment
-$res=0;
-if (! $res && file_exists($path."../../master.inc.php")) $res=@include $path."../../master.inc.php";
-if (! $res && file_exists($path."../../htdocs/master.inc.php")) $res=@include $path."../../htdocs/master.inc.php";
-if (! $res && file_exists("../master.inc.php")) $res=@include "../master.inc.php";
-if (! $res && file_exists("../../master.inc.php")) $res=@include "../../master.inc.php";
-if (! $res && file_exists("../../../master.inc.php")) $res=@include "../../../master.inc.php";
-if (! $res && preg_match('/\/nltechno([^\/]*)\//',$_SERVER["PHP_SELF"],$reg)) $res=@include $path."../../../dolibarr".$reg[1]."/htdocs/master.inc.php"; // Used on dev env only
-if (! $res && preg_match('/\/nltechno([^\/]*)\//',$_SERVER["PHP_SELF"],$reg)) $res=@include "../../../dolibarr".$reg[1]."/htdocs/master.inc.php"; // Used on dev env only
-if (! $res) die ("Failed to include master.inc.php file\n");
+$res = 0;
+if (!$res && file_exists($path."../../master.inc.php")) $res = @include $path."../../master.inc.php";
+if (!$res && file_exists($path."../../htdocs/master.inc.php")) $res = @include $path."../../htdocs/master.inc.php";
+if (!$res && file_exists("../master.inc.php")) $res = @include "../master.inc.php";
+if (!$res && file_exists("../../master.inc.php")) $res = @include "../../master.inc.php";
+if (!$res && file_exists("../../../master.inc.php")) $res = @include "../../../master.inc.php";
+if (!$res && preg_match('/\/nltechno([^\/]*)\//', $_SERVER["PHP_SELF"], $reg)) $res = @include $path."../../../dolibarr".$reg[1]."/htdocs/master.inc.php"; // Used on dev env only
+if (!$res && preg_match('/\/nltechno([^\/]*)\//', $_SERVER["PHP_SELF"], $reg)) $res = @include "../../../dolibarr".$reg[1]."/htdocs/master.inc.php"; // Used on dev env only
+if (!$res) die ("Failed to include master.inc.php file\n");
 include_once DOL_DOCUMENT_ROOT.'/core/lib/files.lib.php';
 
 
@@ -59,10 +59,10 @@ 
 }
 
 
-$tmp=dol_getdate(dol_now());
+$tmp = dol_getdate(dol_now());
 
 
-$tables=array(
+$tables = array(
     'propal'=>array(0=>'datep', 1=>'fin_validite', 2=>'date_valid', 3=>'date_cloture'),
     'commande'=>array(0=>'date_commande', 1=>'date_valid', 2=>'date_cloture'),
     'facture'=>array(0=>'datef', 1=>'date_valid', 2=>'date_lim_reglement'),
@@ -72,44 +72,44 @@ 
     'supplier_proposal'=>array(0=>'datec', 1=>'date_valid', 2=>'date_cloture')
 );
 
-$year=2010;
-$currentyear=$tmp['year'];
+$year = 2010;
+$currentyear = $tmp['year'];
 while ($year <= $currentyear)
 {
     //$year=2021;
-    $delta=($currentyear - $year);
+    $delta = ($currentyear - $year);
     //$delta=-1;
 
     if ($delta)
     {
-        foreach($tables as $tablekey => $tableval)
+        foreach ($tables as $tablekey => $tableval)
         {
             print "\nCorrect ".$tablekey." for year ".$year." and move them to current year ".$currentyear." ";
-            $sql="select rowid from ".MAIN_DB_PREFIX.$tablekey." where ".$tableval[0]." between '".$year."-01-01' and '".$year."-12-31' and ".$tableval[0]." < DATE_ADD(NOW(), INTERVAL -1 YEAR)";
+            $sql = "select rowid from ".MAIN_DB_PREFIX.$tablekey." where ".$tableval[0]." between '".$year."-01-01' and '".$year."-12-31' and ".$tableval[0]." < DATE_ADD(NOW(), INTERVAL -1 YEAR)";
             //$sql="select rowid from ".MAIN_DB_PREFIX.$tablekey." where ".$tableval[0]." between '".$year."-01-01' and '".$year."-12-31' and ".$tableval[0]." > NOW()";
             $resql = $db->query($sql);
             if ($resql)
             {
                 $num = $db->num_rows($resql);
-                $i=0;
+                $i = 0;
                 while ($i < $num)
                 {
-                    $obj=$db->fetch_object($resql);
+                    $obj = $db->fetch_object($resql);
                     if ($obj)
                     {
                         print ".";
-                        $sql2="UPDATE ".MAIN_DB_PREFIX.$tablekey." set ";
-                        $j=0;
-                        foreach($tableval as $field)
+                        $sql2 = "UPDATE ".MAIN_DB_PREFIX.$tablekey." set ";
+                        $j = 0;
+                        foreach ($tableval as $field)
                         {
-                            if ($j) $sql2.=", ";
-                            $sql2.= $field." = DATE_ADD(".$field.", INTERVAL ".$delta." YEAR)";
+                            if ($j) $sql2 .= ", ";
+                            $sql2 .= $field." = DATE_ADD(".$field.", INTERVAL ".$delta." YEAR)";
                             $j++;
                         }
-                        $sql2.=" WHERE rowid = ".$obj->rowid;
+                        $sql2 .= " WHERE rowid = ".$obj->rowid;
                         //print $sql2."\n";
                         $resql2 = $db->query($sql2);
-                        if (! $resql2) dol_print_error($db);
+                        if (!$resql2) dol_print_error($db);
                     }
                     $i++;
                 }

Braces +34 added lines, -11 removed lines

@@ -36,14 +36,32 @@ 
 
 // Include Dolibarr environment
 $res=0;
-if (! $res && file_exists($path."../../master.inc.php")) $res=@include $path."../../master.inc.php";
-if (! $res && file_exists($path."../../htdocs/master.inc.php")) $res=@include $path."../../htdocs/master.inc.php";
-if (! $res && file_exists("../master.inc.php")) $res=@include "../master.inc.php";
-if (! $res && file_exists("../../master.inc.php")) $res=@include "../../master.inc.php";
-if (! $res && file_exists("../../../master.inc.php")) $res=@include "../../../master.inc.php";
-if (! $res && preg_match('/\/nltechno([^\/]*)\//',$_SERVER["PHP_SELF"],$reg)) $res=@include $path."../../../dolibarr".$reg[1]."/htdocs/master.inc.php"; // Used on dev env only
-if (! $res && preg_match('/\/nltechno([^\/]*)\//',$_SERVER["PHP_SELF"],$reg)) $res=@include "../../../dolibarr".$reg[1]."/htdocs/master.inc.php"; // Used on dev env only
-if (! $res) die ("Failed to include master.inc.php file\n");
+if (! $res && file_exists($path."../../master.inc.php")) {
+    $res=@include $path."../../master.inc.php";
+}
+if (! $res && file_exists($path."../../htdocs/master.inc.php")) {
+    $res=@include $path."../../htdocs/master.inc.php";
+}
+if (! $res && file_exists("../master.inc.php")) {
+    $res=@include "../master.inc.php";
+}
+if (! $res && file_exists("../../master.inc.php")) {
+    $res=@include "../../master.inc.php";
+}
+if (! $res && file_exists("../../../master.inc.php")) {
+    $res=@include "../../../master.inc.php";
+}
+if (! $res && preg_match('/\/nltechno([^\/]*)\//',$_SERVER["PHP_SELF"],$reg)) {
+    $res=@include $path."../../../dolibarr".$reg[1]."/htdocs/master.inc.php";
+}
+// Used on dev env only
+if (! $res && preg_match('/\/nltechno([^\/]*)\//',$_SERVER["PHP_SELF"],$reg)) {
+    $res=@include "../../../dolibarr".$reg[1]."/htdocs/master.inc.php";
+}
+// Used on dev env only
+if (! $res) {
+    die ("Failed to include master.inc.php file\n");
+}
 include_once DOL_DOCUMENT_ROOT.'/core/lib/files.lib.php';
 
 
@@ -102,19 +120,24 @@ 
                         $j=0;
                         foreach($tableval as $field)
                         {
-                            if ($j) $sql2.=", ";
+                            if ($j) {
+                                $sql2.=", ";
+                            }
                             $sql2.= $field." = DATE_ADD(".$field.", INTERVAL ".$delta." YEAR)";
                             $j++;
                         }
                         $sql2.=" WHERE rowid = ".$obj->rowid;
                         //print $sql2."\n";
                         $resql2 = $db->query($sql2);
-                        if (! $resql2) dol_print_error($db);
+                        if (! $resql2) {
+                            dol_print_error($db);
+                        }
                     }
                     $i++;
                 }
+            } else {
+                dol_print_error($db);
             }
-            else dol_print_error($db);
         }
     }
 

dolibarr/dev/initdata/generate-order.php

Indentation +37 added lines, -37 removed lines

@@ -113,13 +113,13 @@ 
 $sql = "SELECT rowid FROM ".MAIN_DB_PREFIX."societe where client in (1, 3)";
 $resql=$db->query($sql);
 if ($resql) {
-	$num_thirdparties = $db->num_rows($resql);
-	$i = 0;
-	while ($i < $num_thirdparties) {
-		$i++;
-	    $row = $db->fetch_row($resql);
-		$societesid[$i] = $row[0];
-	}
+    $num_thirdparties = $db->num_rows($resql);
+    $i = 0;
+    while ($i < $num_thirdparties) {
+        $i++;
+        $row = $db->fetch_row($resql);
+        $societesid[$i] = $row[0];
+    }
 }
 else { print "err"; }
 
@@ -127,13 +127,13 @@ 
 $sql = "SELECT rowid FROM ".MAIN_DB_PREFIX."commande";
 $resql=$db->query($sql);
 if ($resql) {
-	$num = $db->num_rows($resql);
-	$i = 0;
-	while ($i < $num) {
-		$i++;
-	    $row = $db->fetch_row($resql);
-		$commandesid[$i] = $row[0];
-	}
+    $num = $db->num_rows($resql);
+    $i = 0;
+    while ($i < $num) {
+        $i++;
+        $row = $db->fetch_row($resql);
+        $commandesid[$i] = $row[0];
+    }
 }
 else { print "err"; }
 
@@ -142,14 +142,14 @@ 
 $resql = $db->query($sql);
 if ($resql)
 {
-  $num_prods = $db->num_rows($resql);
-  $i = 0;
-  while ($i < $num_prods)
+    $num_prods = $db->num_rows($resql);
+    $i = 0;
+    while ($i < $num_prods)
     {
-      $i++;
+        $i++;
 
-      $row = $db->fetch_row($resql);
-      $prodids[$i] = $row[0];
+        $row = $db->fetch_row($resql);
+        $prodids[$i] = $row[0];
     }
 }
 
@@ -183,8 +183,8 @@ 
     $db->begin();
 
     $result=$object->create($fuser);
-	if ($result >= 0)
-	{
+    if ($result >= 0)
+    {
         $nbp = mt_rand(2, 5);
         $xnbp = 0;
         while ($xnbp < $nbp)
@@ -200,24 +200,24 @@ 
             $xnbp++;
         }
 
-	    $result=$object->valid($fuser);
-		if ($result > 0)
-		{
+        $result=$object->valid($fuser);
+        if ($result > 0)
+        {
             $db->commit();
             print " OK with ref ".$object->ref."\n";
-		}
-		else
-		{
+        }
+        else
+        {
             print " KO\n";
-		    $db->rollback();
-		    dol_print_error($db,$object->error);
-		}
-	}
-	else
-	{
+            $db->rollback();
+            dol_print_error($db,$object->error);
+        }
+    }
+    else
+    {
         print " KO\n";
-	    $db->rollback();
-	    dol_print_error($db,$object->error);
-	}
+        $db->rollback();
+        dol_print_error($db,$object->error);
+    }
 }
 

Spacing +66 added lines, -66 removed lines

@@ -33,7 +33,7 @@ 
 
 // Recupere root dolibarr
 //$path=preg_replace('/generate-commande.php/i','',$_SERVER["PHP_SELF"]);
-require __DIR__. '/../../htdocs/master.inc.php';
+require __DIR__.'/../../htdocs/master.inc.php';
 include_once DOL_DOCUMENT_ROOT.'/societe/class/societe.class.php';
 include_once DOL_DOCUMENT_ROOT.'/contact/class/contact.class.php';
 include_once DOL_DOCUMENT_ROOT.'/compta/facture/class/facture.class.php';
@@ -49,59 +49,59 @@ 
 
 define(GEN_NUMBER_COMMANDE, 10);
 $year = 2016;
-$dates = array (mktime(12,0,0,1,3,$year),
-    mktime(12,0,0,1,9,$year),
-    mktime(12,0,0,2,13,$year),
-    mktime(12,0,0,2,23,$year),
-    mktime(12,0,0,3,30,$year),
-    mktime(12,0,0,4,3,$year),
-    mktime(12,0,0,4,3,$year),
-    mktime(12,0,0,5,9,$year),
-    mktime(12,0,0,5,1,$year),
-    mktime(12,0,0,5,13,$year),
-    mktime(12,0,0,5,19,$year),
-    mktime(12,0,0,5,23,$year),
-    mktime(12,0,0,6,3,$year),
-    mktime(12,0,0,6,19,$year),
-    mktime(12,0,0,6,24,$year),
-    mktime(12,0,0,7,3,$year),
-    mktime(12,0,0,7,9,$year),
-    mktime(12,0,0,7,23,$year),
-    mktime(12,0,0,7,30,$year),
-    mktime(12,0,0,8,9,$year),
-    mktime(12,0,0,9,23,$year),
-    mktime(12,0,0,10,3,$year),
-    mktime(12,0,0,11,12,$year),
-    mktime(12,0,0,11,13,$year),
-    mktime(12,0,0,1,3,($year - 1)),
-    mktime(12,0,0,1,9,($year - 1)),
-    mktime(12,0,0,2,13,($year - 1)),
-    mktime(12,0,0,2,23,($year - 1)),
-    mktime(12,0,0,3,30,($year - 1)),
-    mktime(12,0,0,4,3,($year - 1)),
-    mktime(12,0,0,4,3,($year - 1)),
-    mktime(12,0,0,5,9,($year - 1)),
-    mktime(12,0,0,5,1,($year - 1)),
-    mktime(12,0,0,5,13,($year - 1)),
-    mktime(12,0,0,5,19,($year - 1)),
-    mktime(12,0,0,5,23,($year - 1)),
-    mktime(12,0,0,6,3,($year - 1)),
-    mktime(12,0,0,6,19,($year - 1)),
-    mktime(12,0,0,6,24,($year - 1)),
-    mktime(12,0,0,7,3,($year - 1)),
-    mktime(12,0,0,7,9,($year - 1)),
-    mktime(12,0,0,7,23,($year - 1)),
-    mktime(12,0,0,7,30,($year - 1)),
-    mktime(12,0,0,8,9,($year - 1)),
-    mktime(12,0,0,9,23,($year - 1)),
-    mktime(12,0,0,10,3,($year - 1)),
-    mktime(12,0,0,11,12,$year),
-    mktime(12,0,0,11,13,$year),
-    mktime(12,0,0,12,12,$year),
-    mktime(12,0,0,12,13,$year),
+$dates = array(mktime(12, 0, 0, 1, 3, $year),
+    mktime(12, 0, 0, 1, 9, $year),
+    mktime(12, 0, 0, 2, 13, $year),
+    mktime(12, 0, 0, 2, 23, $year),
+    mktime(12, 0, 0, 3, 30, $year),
+    mktime(12, 0, 0, 4, 3, $year),
+    mktime(12, 0, 0, 4, 3, $year),
+    mktime(12, 0, 0, 5, 9, $year),
+    mktime(12, 0, 0, 5, 1, $year),
+    mktime(12, 0, 0, 5, 13, $year),
+    mktime(12, 0, 0, 5, 19, $year),
+    mktime(12, 0, 0, 5, 23, $year),
+    mktime(12, 0, 0, 6, 3, $year),
+    mktime(12, 0, 0, 6, 19, $year),
+    mktime(12, 0, 0, 6, 24, $year),
+    mktime(12, 0, 0, 7, 3, $year),
+    mktime(12, 0, 0, 7, 9, $year),
+    mktime(12, 0, 0, 7, 23, $year),
+    mktime(12, 0, 0, 7, 30, $year),
+    mktime(12, 0, 0, 8, 9, $year),
+    mktime(12, 0, 0, 9, 23, $year),
+    mktime(12, 0, 0, 10, 3, $year),
+    mktime(12, 0, 0, 11, 12, $year),
+    mktime(12, 0, 0, 11, 13, $year),
+    mktime(12, 0, 0, 1, 3, ($year - 1)),
+    mktime(12, 0, 0, 1, 9, ($year - 1)),
+    mktime(12, 0, 0, 2, 13, ($year - 1)),
+    mktime(12, 0, 0, 2, 23, ($year - 1)),
+    mktime(12, 0, 0, 3, 30, ($year - 1)),
+    mktime(12, 0, 0, 4, 3, ($year - 1)),
+    mktime(12, 0, 0, 4, 3, ($year - 1)),
+    mktime(12, 0, 0, 5, 9, ($year - 1)),
+    mktime(12, 0, 0, 5, 1, ($year - 1)),
+    mktime(12, 0, 0, 5, 13, ($year - 1)),
+    mktime(12, 0, 0, 5, 19, ($year - 1)),
+    mktime(12, 0, 0, 5, 23, ($year - 1)),
+    mktime(12, 0, 0, 6, 3, ($year - 1)),
+    mktime(12, 0, 0, 6, 19, ($year - 1)),
+    mktime(12, 0, 0, 6, 24, ($year - 1)),
+    mktime(12, 0, 0, 7, 3, ($year - 1)),
+    mktime(12, 0, 0, 7, 9, ($year - 1)),
+    mktime(12, 0, 0, 7, 23, ($year - 1)),
+    mktime(12, 0, 0, 7, 30, ($year - 1)),
+    mktime(12, 0, 0, 8, 9, ($year - 1)),
+    mktime(12, 0, 0, 9, 23, ($year - 1)),
+    mktime(12, 0, 0, 10, 3, ($year - 1)),
+    mktime(12, 0, 0, 11, 12, $year),
+    mktime(12, 0, 0, 11, 13, $year),
+    mktime(12, 0, 0, 12, 12, $year),
+    mktime(12, 0, 0, 12, 13, $year),
 );
 
-$ret=$user->fetch('','admin');
+$ret = $user->fetch('', 'admin');
 if ($ret <= 0)
 {
     print 'A user with login "admin" and all permissions must be created to use this script.'."\n";
@@ -111,7 +111,7 @@ 
 
 $societesid = array();
 $sql = "SELECT rowid FROM ".MAIN_DB_PREFIX."societe where client in (1, 3)";
-$resql=$db->query($sql);
+$resql = $db->query($sql);
 if ($resql) {
 	$num_thirdparties = $db->num_rows($resql);
 	$i = 0;
@@ -125,7 +125,7 @@ 
 
 $commandesid = array();
 $sql = "SELECT rowid FROM ".MAIN_DB_PREFIX."commande";
-$resql=$db->query($sql);
+$resql = $db->query($sql);
 if ($resql) {
 	$num = $db->num_rows($resql);
 	$i = 0;
@@ -157,14 +157,14 @@ 
 
 
 print "Build ".GEN_NUMBER_COMMANDE." orders\n";
-for ($s = 0 ; $s < GEN_NUMBER_COMMANDE ; $s++)
+for ($s = 0; $s < GEN_NUMBER_COMMANDE; $s++)
 {
     print "Process order ".$s."\n";
 
     $object = new Commande($db);
 
     $object->socid          = $societesid[mt_rand(1, $num_thirdparties)];
-    $object->date_commande  = $dates[mt_rand(1, count($dates)-1)];
+    $object->date_commande  = $dates[mt_rand(1, count($dates) - 1)];
     $object->note           = 'My small comment about this order. Hum. Nothing.';
     $object->source         = 1;
     $object->fk_project     = 0;
@@ -174,15 +174,15 @@ 
     $object->more_reglement_id = mt_rand(0, 7);
     $object->availability_id = mt_rand(0, 1);
     
-    $listofuserid=array(12,13,16);
+    $listofuserid = array(12, 13, 16);
     
     $fuser = new User($db);
-    $fuser->fetch($listofuserid[mt_rand(0,2)]);
+    $fuser->fetch($listofuserid[mt_rand(0, 2)]);
     $fuser->getRights();
     
     $db->begin();
 
-    $result=$object->create($fuser);
+    $result = $object->create($fuser);
 	if ($result >= 0)
 	{
         $nbp = mt_rand(2, 5);
@@ -190,17 +190,17 @@ 
         while ($xnbp < $nbp)
         {
             $prodid = mt_rand(1, $num_prods);
-            $product=new Product($db);
-            $result=$product->fetch($prodids[$prodid]);
-            $result=$object->addline($product->description, $product->price, mt_rand(1,5), 0, 0, 0, $prodids[$prodid], 0, 0, 0,  $product->price_base_type, $product->price_ttc, '', '', $product->type);
+            $product = new Product($db);
+            $result = $product->fetch($prodids[$prodid]);
+            $result = $object->addline($product->description, $product->price, mt_rand(1, 5), 0, 0, 0, $prodids[$prodid], 0, 0, 0, $product->price_base_type, $product->price_ttc, '', '', $product->type);
             if ($result <= 0)
             {
-                dol_print_error($db,$object->error);
+                dol_print_error($db, $object->error);
             }
             $xnbp++;
         }
 
-	    $result=$object->valid($fuser);
+	    $result = $object->valid($fuser);
 		if ($result > 0)
 		{
             $db->commit();
@@ -210,14 +210,14 @@ 
 		{
             print " KO\n";
 		    $db->rollback();
-		    dol_print_error($db,$object->error);
+		    dol_print_error($db, $object->error);
 		}
 	}
 	else
 	{
         print " KO\n";
 	    $db->rollback();
-	    dol_print_error($db,$object->error);
+	    dol_print_error($db, $object->error);
 	}
 }
 

Braces +4 added lines, -8 removed lines

@@ -120,8 +120,7 @@ 
 	    $row = $db->fetch_row($resql);
 		$societesid[$i] = $row[0];
 	}
-}
-else { print "err"; }
+} else { print "err"; }
 
 $commandesid = array();
 $sql = "SELECT rowid FROM ".MAIN_DB_PREFIX."commande";
@@ -134,8 +133,7 @@ 
 	    $row = $db->fetch_row($resql);
 		$commandesid[$i] = $row[0];
 	}
-}
-else { print "err"; }
+} else { print "err"; }
 
 $prodids = array();
 $sql = "SELECT rowid FROM ".MAIN_DB_PREFIX."product WHERE tosell=1";
@@ -205,15 +203,13 @@ 
 		{
             $db->commit();
             print " OK with ref ".$object->ref."\n";
-		}
-		else
+		} else
 		{
             print " KO\n";
 		    $db->rollback();
 		    dol_print_error($db,$object->error);
 		}
-	}
-	else
+	} else
 	{
         print " KO\n";
 	    $db->rollback();

dolibarr/dev/initdata/purge-data.php

Indentation +11 added lines, -11 removed lines

@@ -81,11 +81,11 @@ 
         'DELETE FROM '.MAIN_DB_PREFIX.'supplier_proposaldet',
         'DELETE FROM '.MAIN_DB_PREFIX.'supplier_proposal',
     ),
-	'supplier_order'=>array(
+    'supplier_order'=>array(
         'DELETE FROM '.MAIN_DB_PREFIX.'commande_fournisseurdet',
         'DELETE FROM '.MAIN_DB_PREFIX.'commande_fournisseur',
     ),
-	'supplier_invoice'=>array(
+    'supplier_invoice'=>array(
         'DELETE FROM '.MAIN_DB_PREFIX.'facture_fourn_det',
         'DELETE FROM '.MAIN_DB_PREFIX.'facture_fourn',
     ),
@@ -116,9 +116,9 @@ 
         'DELETE FROM '.MAIN_DB_PREFIX.'product_price',
         'DELETE FROM '.MAIN_DB_PREFIX.'product_fournisseur_price',
         'DELETE FROM '.MAIN_DB_PREFIX.'product_batch',
-    	'DELETE FROM '.MAIN_DB_PREFIX.'product_stock',
+        'DELETE FROM '.MAIN_DB_PREFIX.'product_stock',
         'DELETE FROM '.MAIN_DB_PREFIX.'product_lot',
-    	'DELETE FROM '.MAIN_DB_PREFIX.'product',
+        'DELETE FROM '.MAIN_DB_PREFIX.'product',
     ),
     'project'=>array(
         'DELETE FROM '.MAIN_DB_PREFIX.'projet_task_time',
@@ -137,7 +137,7 @@ 
         'DELETE FROM '.MAIN_DB_PREFIX.'categorie_societe',
         'DELETE FROM '.MAIN_DB_PREFIX.'societe_remise_except',
         'DELETE FROM '.MAIN_DB_PREFIX.'societe_rib',
-    	'DELETE FROM '.MAIN_DB_PREFIX.'societe',
+        'DELETE FROM '.MAIN_DB_PREFIX.'societe',
     )
 );
 
@@ -172,18 +172,18 @@ 
 // Replace database handler
 if (! empty($argv[3]))
 {
-	$db->close();
-	unset($db);
-	$db=getDoliDBInstance($argv[3], $argv[4], $argv[5], $argv[6], $argv[7], $argv[8]);
-	$user=new User($db);
+    $db->close();
+    unset($db);
+    $db=getDoliDBInstance($argv[3], $argv[4], $argv[5], $argv[6], $argv[7], $argv[8]);
+    $user=new User($db);
 }
 
 //var_dump($user->db->database_name);
 $ret=$user->fetch('','admin');
 if (! $ret > 0)
 {
-	print 'An admin user with login "admin" must exists to use this script.'."\n";
-	exit;
+    print 'An admin user with login "admin" must exists to use this script.'."\n";
+    exit;
 }
 //$user->getrights();
 

Spacing +25 added lines, -25 removed lines

@@ -26,7 +26,7 @@ 
 
 $sapi_type = php_sapi_name();
 $script_file = basename(__FILE__);
-$path=dirname(__FILE__).'/';
+$path = dirname(__FILE__).'/';
 
 // Test si mode batch
 if (substr($sapi_type, 0, 3) == 'cgi') {
@@ -35,7 +35,7 @@ 
 }
 
 // Recupere root dolibarr
-$path=preg_replace('/purge-data.php/i','',$_SERVER["PHP_SELF"]);
+$path = preg_replace('/purge-data.php/i', '', $_SERVER["PHP_SELF"]);
 require $path."../../htdocs/master.inc.php";
 include_once DOL_DOCUMENT_ROOT.'/societe/class/societe.class.php';
 include_once DOL_DOCUMENT_ROOT.'/contact/class/contact.class.php';
@@ -47,11 +47,11 @@ 
 $langs->loadLangs(array("main", "errors"));
 
 // Global variables
-$version=DOL_VERSION;
-$error=0;
+$version = DOL_VERSION;
+$error = 0;
 
 // List of sql to execute
-$sqls=array(
+$sqls = array(
     'user'=>array(
         'DELETE FROM '.MAIN_DB_PREFIX."user_rights WHERE fk_user IN (SELECT rowid FROM ".MAIN_DB_PREFIX."user WHERE admin = 0 and login != 'admin')",
         'DELETE FROM '.MAIN_DB_PREFIX."user WHERE admin = 0 and login != 'admin'",
@@ -150,37 +150,37 @@ 
 
 @set_time_limit(0);
 print "***** ".$script_file." (".$version.") pid=".dol_getmypid()." *****\n";
-dol_syslog($script_file." launched with arg ".implode(',',$argv));
+dol_syslog($script_file." launched with arg ".implode(',', $argv));
 
 $mode = $argv[1];
 $option = $argv[2];
 
-if (empty($mode) || ! in_array($mode,array('test','confirm'))) {
+if (empty($mode) || !in_array($mode, array('test', 'confirm'))) {
     print "Usage:  $script_file (test|confirm) (all|option) [dbtype dbhost dbuser dbpassword dbname dbport]\n";
     print "\n";
-    print "option can be ".implode(',',array_keys($sqls))."\n";
+    print "option can be ".implode(',', array_keys($sqls))."\n";
     exit(-1);
 }
 
-if (empty($option) || ! in_array($option, array_merge(array('all'),array_keys($sqls))) ) {
+if (empty($option) || !in_array($option, array_merge(array('all'), array_keys($sqls)))) {
     print "Usage:  $script_file (test|confirm) (all|option) [dbtype dbhost dbuser dbpassword dbname dbport]\n";
     print "\n";
-    print "option can be ".implode(',',array_keys($sqls))."\n";
+    print "option can be ".implode(',', array_keys($sqls))."\n";
     exit(-1);
 }
 
 // Replace database handler
-if (! empty($argv[3]))
+if (!empty($argv[3]))
 {
 	$db->close();
 	unset($db);
-	$db=getDoliDBInstance($argv[3], $argv[4], $argv[5], $argv[6], $argv[7], $argv[8]);
-	$user=new User($db);
+	$db = getDoliDBInstance($argv[3], $argv[4], $argv[5], $argv[6], $argv[7], $argv[8]);
+	$user = new User($db);
 }
 
 //var_dump($user->db->database_name);
-$ret=$user->fetch('','admin');
-if (! $ret > 0)
+$ret = $user->fetch('', 'admin');
+if (!$ret > 0)
 {
 	print 'An admin user with login "admin" must exists to use this script.'."\n";
 	exit;
@@ -195,7 +195,7 @@ 
 print "User = ".$db->database_user."\n";
 print "\n";
 
-if (! $confirmed)
+if (!$confirmed)
 {
     print "Hit Enter to continue or CTRL+C to stop...\n";
     $input = trim(fgets(STDIN));
@@ -212,19 +212,19 @@ 
 {
     global $db, $sqls;
 
-    $error=0;
-    foreach($sqls[$family] as $sql)
+    $error = 0;
+    foreach ($sqls[$family] as $sql)
     {
-        if (preg_match('/^@/',$sql))
+        if (preg_match('/^@/', $sql))
         {
-            $newfamily=preg_replace('/@/','',$sql);
+            $newfamily = preg_replace('/@/', '', $sql);
             processfamily($newfamily);
             continue;
         }
 
         print "Run sql: ".$sql."\n";
-        $resql=$db->query($sql);
-        if (! $resql)
+        $resql = $db->query($sql);
+        if (!$resql)
         {
             if ($db->errno() != 'DB_ERROR_NOSUCHTABLE')
             {
@@ -247,15 +247,15 @@ 
 
 $db->begin();
 
-$oldfamily='';
-foreach($sqls as $family => $familysql)
+$oldfamily = '';
+foreach ($sqls as $family => $familysql)
 {
     if ($option && $option != 'all' && $option != $family) continue;
 
     if ($family != $oldfamily) print "Process action for family ".$family."\n";
     $oldfamily = $family;
 
-    $result=processfamily($family);
+    $result = processfamily($family);
     if ($result < 0)
     {
         $error++;

Braces +13 added lines, -7 removed lines

@@ -240,9 +240,12 @@ 
         }
     }
 
-    if ($error) return -1;
-    else return 1;
-}
+    if ($error) {
+        return -1;
+    } else {
+        return 1;
+    }
+    }
 
 
 $db->begin();
@@ -250,9 +253,13 @@ 
 $oldfamily='';
 foreach($sqls as $family => $familysql)
 {
-    if ($option && $option != 'all' && $option != $family) continue;
+    if ($option && $option != 'all' && $option != $family) {
+        continue;
+    }
 
-    if ($family != $oldfamily) print "Process action for family ".$family."\n";
+    if ($family != $oldfamily) {
+        print "Process action for family ".$family."\n";
+    }
     $oldfamily = $family;
 
     $result=processfamily($family);
@@ -267,8 +274,7 @@ 
 {
     print "Rollback any changes.\n";
     $db->rollback();
-}
-else
+} else
 {
     print "Commit all changes.\n";
     $db->commit();

dolibarr/dev/initdata/generate-invoice.php

Indentation +55 added lines, -55 removed lines

@@ -26,8 +26,8 @@ 
 // Test si mode batch
 $sapi_type = php_sapi_name();
 if (substr($sapi_type, 0, 3) == 'cgi') {
-	echo "Erreur: Vous utilisez l'interpreteur PHP pour le mode CGI. Pour executer mailing-send.php en ligne de commande, vous devez utiliser l'interpreteur PHP pour le mode CLI.\n";
-	exit;
+    echo "Erreur: Vous utilisez l'interpreteur PHP pour le mode CGI. Pour executer mailing-send.php en ligne de commande, vous devez utiliser l'interpreteur PHP pour le mode CLI.\n";
+    exit;
 }
 
 // Recupere root dolibarr
@@ -98,8 +98,8 @@ 
 $ret=$user->fetch('','admin');
 if (! $ret > 0)
 {
-	print 'A user with login "admin" and all permissions must be created to use this script.'."\n";
-	exit;
+    print 'A user with login "admin" and all permissions must be created to use this script.'."\n";
+    exit;
 }
 $user->getrights();
 
@@ -109,14 +109,14 @@ 
 $resql = $db->query($sql);
 if ($resql)
 {
-	$num_thirdparties = $db->num_rows($resql);
-	$i = 0;
-	while ($i < $num_thirdparties)
-	{
-		$i++;
-		$row = $db->fetch_row($resql);
-		$socids[$i] = $row[0];
-	}
+    $num_thirdparties = $db->num_rows($resql);
+    $i = 0;
+    while ($i < $num_thirdparties)
+    {
+        $i++;
+        $row = $db->fetch_row($resql);
+        $socids[$i] = $row[0];
+    }
 }
 
 $prodids = array();
@@ -124,65 +124,65 @@ 
 $resql = $db->query($sql);
 if ($resql)
 {
-	$num_prods = $db->num_rows($resql);
-	$i = 0;
-	while ($i < $num_prods)
-	{
-		$i++;
-		$row = $db->fetch_row($resql);
-		$prodids[$i] = $row[0];
-	}
+    $num_prods = $db->num_rows($resql);
+    $i = 0;
+    while ($i < $num_prods)
+    {
+        $i++;
+        $row = $db->fetch_row($resql);
+        $prodids[$i] = $row[0];
+    }
 }
 
 $i=0;
 $result=0;
 while ($i < GEN_NUMBER_FACTURE && $result >= 0)
 {
-	$i++;
-	$socid = mt_rand(1, $num_thirdparties);
+    $i++;
+    $socid = mt_rand(1, $num_thirdparties);
 
-	print "Invoice ".$i." for socid ".$socid;
+    print "Invoice ".$i." for socid ".$socid;
 
-	$object = new Facture($db);
-	$object->socid = $socids[$socid];
-	$object->date = $dates[mt_rand(1, count($dates)-1)];
-	$object->cond_reglement_id = 3;
-	$object->mode_reglement_id = 3;
+    $object = new Facture($db);
+    $object->socid = $socids[$socid];
+    $object->date = $dates[mt_rand(1, count($dates)-1)];
+    $object->cond_reglement_id = 3;
+    $object->mode_reglement_id = 3;
 
     $fuser = new User($db);
     $fuser->fetch(mt_rand(1,2));
     $fuser->getRights();
     
-	$result=$object->create($fuser);
-	if ($result >= 0)
-	{
-		$nbp = mt_rand(2, 5);
-		$xnbp = 0;
-		while ($xnbp < $nbp)
-		{
-			$prodid = mt_rand(1, $num_prods);
-			$product=new Product($db);
-			$result=$product->fetch($prodids[$prodid]);
-			$result=$object->addline($product->description, $product->price, mt_rand(1,5), 0, 0, 0, $prodids[$prodid], 0, '', '', 0, 0, '', $product->price_base_type, $product->price_ttc, $product->type);
-		    if ($result < 0)
+    $result=$object->create($fuser);
+    if ($result >= 0)
+    {
+        $nbp = mt_rand(2, 5);
+        $xnbp = 0;
+        while ($xnbp < $nbp)
+        {
+            $prodid = mt_rand(1, $num_prods);
+            $product=new Product($db);
+            $result=$product->fetch($prodids[$prodid]);
+            $result=$object->addline($product->description, $product->price, mt_rand(1,5), 0, 0, 0, $prodids[$prodid], 0, '', '', 0, 0, '', $product->price_base_type, $product->price_ttc, $product->type);
+            if ($result < 0)
             {
                 dol_print_error($db,$propal->error);
             }
             $xnbp++;
-		}
+        }
 
-	    $result=$object->validate($fuser);
-		if ($result)
-		{
-			print " OK with ref ".$object->ref."\n";;
-		}
-		else
-		{
-			dol_print_error($db,$object->error);
-		}
-	}
-	else
-	{
-		dol_print_error($db,$object->error);
-	}
+        $result=$object->validate($fuser);
+        if ($result)
+        {
+            print " OK with ref ".$object->ref."\n";;
+        }
+        else
+        {
+            dol_print_error($db,$object->error);
+        }
+    }
+    else
+    {
+        dol_print_error($db,$object->error);
+    }
 }

Spacing +66 added lines, -66 removed lines

@@ -32,7 +32,7 @@ 
 
 // Recupere root dolibarr
 //$path=preg_replace('/generate-produit.php/i','',$_SERVER["PHP_SELF"]);
-require __DIR__. '/../../htdocs/master.inc.php';
+require __DIR__.'/../../htdocs/master.inc.php';
 require_once DOL_DOCUMENT_ROOT."/compta/facture/class/facture.class.php";
 require_once DOL_DOCUMENT_ROOT."/societe/class/societe.class.php";
 
@@ -43,60 +43,60 @@ 
 
 define(GEN_NUMBER_FACTURE, 1);
 $year = 2016;
-$dates = array (mktime(12,0,0,1,3,$year),
-    mktime(12,0,0,1,9,$year),
-    mktime(12,0,0,2,13,$year),
-    mktime(12,0,0,2,23,$year),
-    mktime(12,0,0,3,30,$year),
-    mktime(12,0,0,4,3,$year),
-    mktime(12,0,0,4,3,$year),
-    mktime(12,0,0,5,9,$year),
-    mktime(12,0,0,5,1,$year),
-    mktime(12,0,0,5,13,$year),
-    mktime(12,0,0,5,19,$year),
-    mktime(12,0,0,5,23,$year),
-    mktime(12,0,0,6,3,$year),
-    mktime(12,0,0,6,19,$year),
-    mktime(12,0,0,6,24,$year),
-    mktime(12,0,0,7,3,$year),
-    mktime(12,0,0,7,9,$year),
-    mktime(12,0,0,7,23,$year),
-    mktime(12,0,0,7,30,$year),
-    mktime(12,0,0,8,9,$year),
-    mktime(12,0,0,9,23,$year),
-    mktime(12,0,0,10,3,$year),
-    mktime(12,0,0,11,12,$year),
-    mktime(12,0,0,11,13,$year),
-    mktime(12,0,0,1,3,($year - 1)),
-    mktime(12,0,0,1,9,($year - 1)),
-    mktime(12,0,0,2,13,($year - 1)),
-    mktime(12,0,0,2,23,($year - 1)),
-    mktime(12,0,0,3,30,($year - 1)),
-    mktime(12,0,0,4,3,($year - 1)),
-    mktime(12,0,0,4,3,($year - 1)),
-    mktime(12,0,0,5,9,($year - 1)),
-    mktime(12,0,0,5,1,($year - 1)),
-    mktime(12,0,0,5,13,($year - 1)),
-    mktime(12,0,0,5,19,($year - 1)),
-    mktime(12,0,0,5,23,($year - 1)),
-    mktime(12,0,0,6,3,($year - 1)),
-    mktime(12,0,0,6,19,($year - 1)),
-    mktime(12,0,0,6,24,($year - 1)),
-    mktime(12,0,0,7,3,($year - 1)),
-    mktime(12,0,0,7,9,($year - 1)),
-    mktime(12,0,0,7,23,($year - 1)),
-    mktime(12,0,0,7,30,($year - 1)),
-    mktime(12,0,0,8,9,($year - 1)),
-    mktime(12,0,0,9,23,($year - 1)),
-    mktime(12,0,0,10,3,($year - 1)),
-    mktime(12,0,0,11,12,$year),
-    mktime(12,0,0,11,13,$year),
-    mktime(12,0,0,12,12,$year),
-    mktime(12,0,0,12,13,$year),
+$dates = array(mktime(12, 0, 0, 1, 3, $year),
+    mktime(12, 0, 0, 1, 9, $year),
+    mktime(12, 0, 0, 2, 13, $year),
+    mktime(12, 0, 0, 2, 23, $year),
+    mktime(12, 0, 0, 3, 30, $year),
+    mktime(12, 0, 0, 4, 3, $year),
+    mktime(12, 0, 0, 4, 3, $year),
+    mktime(12, 0, 0, 5, 9, $year),
+    mktime(12, 0, 0, 5, 1, $year),
+    mktime(12, 0, 0, 5, 13, $year),
+    mktime(12, 0, 0, 5, 19, $year),
+    mktime(12, 0, 0, 5, 23, $year),
+    mktime(12, 0, 0, 6, 3, $year),
+    mktime(12, 0, 0, 6, 19, $year),
+    mktime(12, 0, 0, 6, 24, $year),
+    mktime(12, 0, 0, 7, 3, $year),
+    mktime(12, 0, 0, 7, 9, $year),
+    mktime(12, 0, 0, 7, 23, $year),
+    mktime(12, 0, 0, 7, 30, $year),
+    mktime(12, 0, 0, 8, 9, $year),
+    mktime(12, 0, 0, 9, 23, $year),
+    mktime(12, 0, 0, 10, 3, $year),
+    mktime(12, 0, 0, 11, 12, $year),
+    mktime(12, 0, 0, 11, 13, $year),
+    mktime(12, 0, 0, 1, 3, ($year - 1)),
+    mktime(12, 0, 0, 1, 9, ($year - 1)),
+    mktime(12, 0, 0, 2, 13, ($year - 1)),
+    mktime(12, 0, 0, 2, 23, ($year - 1)),
+    mktime(12, 0, 0, 3, 30, ($year - 1)),
+    mktime(12, 0, 0, 4, 3, ($year - 1)),
+    mktime(12, 0, 0, 4, 3, ($year - 1)),
+    mktime(12, 0, 0, 5, 9, ($year - 1)),
+    mktime(12, 0, 0, 5, 1, ($year - 1)),
+    mktime(12, 0, 0, 5, 13, ($year - 1)),
+    mktime(12, 0, 0, 5, 19, ($year - 1)),
+    mktime(12, 0, 0, 5, 23, ($year - 1)),
+    mktime(12, 0, 0, 6, 3, ($year - 1)),
+    mktime(12, 0, 0, 6, 19, ($year - 1)),
+    mktime(12, 0, 0, 6, 24, ($year - 1)),
+    mktime(12, 0, 0, 7, 3, ($year - 1)),
+    mktime(12, 0, 0, 7, 9, ($year - 1)),
+    mktime(12, 0, 0, 7, 23, ($year - 1)),
+    mktime(12, 0, 0, 7, 30, ($year - 1)),
+    mktime(12, 0, 0, 8, 9, ($year - 1)),
+    mktime(12, 0, 0, 9, 23, ($year - 1)),
+    mktime(12, 0, 0, 10, 3, ($year - 1)),
+    mktime(12, 0, 0, 11, 12, $year),
+    mktime(12, 0, 0, 11, 13, $year),
+    mktime(12, 0, 0, 12, 12, $year),
+    mktime(12, 0, 0, 12, 13, $year),
 );
 
-$ret=$user->fetch('','admin');
-if (! $ret > 0)
+$ret = $user->fetch('', 'admin');
+if (!$ret > 0)
 {
 	print 'A user with login "admin" and all permissions must be created to use this script.'."\n";
 	exit;
@@ -134,8 +134,8 @@ 
 	}
 }
 
-$i=0;
-$result=0;
+$i = 0;
+$result = 0;
 while ($i < GEN_NUMBER_FACTURE && $result >= 0)
 {
 	$i++;
@@ -145,15 +145,15 @@ 
 
 	$object = new Facture($db);
 	$object->socid = $socids[$socid];
-	$object->date = $dates[mt_rand(1, count($dates)-1)];
+	$object->date = $dates[mt_rand(1, count($dates) - 1)];
 	$object->cond_reglement_id = 3;
 	$object->mode_reglement_id = 3;
 
     $fuser = new User($db);
-    $fuser->fetch(mt_rand(1,2));
+    $fuser->fetch(mt_rand(1, 2));
     $fuser->getRights();
     
-	$result=$object->create($fuser);
+	$result = $object->create($fuser);
 	if ($result >= 0)
 	{
 		$nbp = mt_rand(2, 5);
@@ -161,28 +161,28 @@ 
 		while ($xnbp < $nbp)
 		{
 			$prodid = mt_rand(1, $num_prods);
-			$product=new Product($db);
-			$result=$product->fetch($prodids[$prodid]);
-			$result=$object->addline($product->description, $product->price, mt_rand(1,5), 0, 0, 0, $prodids[$prodid], 0, '', '', 0, 0, '', $product->price_base_type, $product->price_ttc, $product->type);
+			$product = new Product($db);
+			$result = $product->fetch($prodids[$prodid]);
+			$result = $object->addline($product->description, $product->price, mt_rand(1, 5), 0, 0, 0, $prodids[$prodid], 0, '', '', 0, 0, '', $product->price_base_type, $product->price_ttc, $product->type);
 		    if ($result < 0)
             {
-                dol_print_error($db,$propal->error);
+                dol_print_error($db, $propal->error);
             }
             $xnbp++;
 		}
 
-	    $result=$object->validate($fuser);
+	    $result = $object->validate($fuser);
 		if ($result)
 		{
-			print " OK with ref ".$object->ref."\n";;
+			print " OK with ref ".$object->ref."\n"; ;
 		}
 		else
 		{
-			dol_print_error($db,$object->error);
+			dol_print_error($db, $object->error);
 		}
 	}
 	else
 	{
-		dol_print_error($db,$object->error);
+		dol_print_error($db, $object->error);
 	}
 }

Braces +2 added lines, -4 removed lines

@@ -175,13 +175,11 @@
 		if ($result)
 		{
 			print " OK with ref ".$object->ref."\n";;
-		}
-		else
+		} else
 		{
 			dol_print_error($db,$object->error);
 		}
-	}
-	else
+	} else
 	{
 		dol_print_error($db,$object->error);
 	}

dolibarr/dev/initdata/generate-thirdparty.php

Indentation +9 added lines, -9 removed lines

@@ -55,8 +55,8 @@ 
 $ret=$user->fetch('','admin');
 if (! $ret > 0)
 {
-	print 'A user with login "admin" and all permissions must be created to use this script.'."\n";
-	exit;
+    print 'A user with login "admin" and all permissions must be created to use this script.'."\n";
+    exit;
 }
 $user->getrights();
 
@@ -64,21 +64,21 @@ 
 $sql = "SELECT rowid FROM ".MAIN_DB_PREFIX."product"; $productsid = array();
 $resql=$db->query($sql);
 if ($resql) {
-  $num = $db->num_rows($resql); $i = 0;
-  while ($i < $num) {      $row = $db->fetch_row($resql);      $productsid[$i] = $row[0];      $i++; }
+    $num = $db->num_rows($resql); $i = 0;
+    while ($i < $num) {      $row = $db->fetch_row($resql);      $productsid[$i] = $row[0];      $i++; }
 }
 
 $sql = "SELECT rowid FROM ".MAIN_DB_PREFIX."societe"; $societesid = array();
 $resql=$db->query($sql);
 if ($resql) {
-	$num = $db->num_rows($resql); $i = 0;
+    $num = $db->num_rows($resql); $i = 0;
     while ($i < $num) { $row = $db->fetch_row($resql);      $societesid[$i] = $row[0];      $i++; }
 } else { print "err"; }
 
 $sql = "SELECT rowid FROM ".MAIN_DB_PREFIX."commande"; $commandesid = array();
 $resql=$db->query($sql);
 if ($resql) {
-	$num = $db->num_rows($resql); $i = 0;
+    $num = $db->num_rows($resql); $i = 0;
     while ($i < $num) { $row = $db->fetch_row($resql);      $commandesid[$i] = $row[0];      $i++; }
 } else { print "err"; }
 
@@ -98,9 +98,9 @@ 
     $soc->tva_assuj=1;
     $soc->country_id=1;
     $soc->country_code='FR';
-	// Un client sur 3 a une remise de 5%
+    // Un client sur 3 a une remise de 5%
     $user_remise=mt_rand(1,3); if ($user_remise==3) $soc->remise_percent=5;
-	print "> client=".$soc->client.", fournisseur=".$soc->fournisseur.", remise=".$soc->remise_percent."\n";
+    print "> client=".$soc->client.", fournisseur=".$soc->fournisseur.", remise=".$soc->remise_percent."\n";
     $soc->note_private = 'Company created by the script generate-societe.php';
     $socid = $soc->create();
 
@@ -124,7 +124,7 @@ 
     }
     else
     {
-    	print "Error: ".$soc->error."\n";
+        print "Error: ".$soc->error."\n";
     }
 }
 

Spacing +25 added lines, -25 removed lines

@@ -33,7 +33,7 @@ 
 
 // Recupere root dolibarr
 //$path=preg_replace('/generate-societe.php/i','',$_SERVER["PHP_SELF"]);
-require __DIR__. '/../../htdocs/master.inc.php';
+require __DIR__.'/../../htdocs/master.inc.php';
 include_once DOL_DOCUMENT_ROOT.'/societe/class/societe.class.php';
 include_once DOL_DOCUMENT_ROOT.'/contact/class/contact.class.php';
 include_once DOL_DOCUMENT_ROOT.'/compta/facture/class/facture.class.php';
@@ -41,8 +41,8 @@ 
 include_once DOL_DOCUMENT_ROOT.'/compta/paiement/class/paiement.class.php';
 include_once DOL_DOCUMENT_ROOT.'/contrat/class/contrat.class.php';
 
-$listoftown = array("Auray","Baden","Vannes","Pirouville","Haguenau","Souffelweiersheim","Illkirch-Graffenstaden","Lauterbourg","Picauville","Sainte-Mère Eglise","Le Bono");
-$listoflastname = array("Joe","Marc","Steve","Laurent","Nico","Isabelle","Dorothee","Saby","Brigitte","Karine","Jose-Anne","Celine","Virginie");
+$listoftown = array("Auray", "Baden", "Vannes", "Pirouville", "Haguenau", "Souffelweiersheim", "Illkirch-Graffenstaden", "Lauterbourg", "Picauville", "Sainte-Mère Eglise", "Le Bono");
+$listoflastname = array("Joe", "Marc", "Steve", "Laurent", "Nico", "Isabelle", "Dorothee", "Saby", "Brigitte", "Karine", "Jose-Anne", "Celine", "Virginie");
 
 
 /*
@@ -52,8 +52,8 @@ 
 define(GEN_NUMBER_SOCIETE, 10);
 
 
-$ret=$user->fetch('','admin');
-if (! $ret > 0)
+$ret = $user->fetch('', 'admin');
+if (!$ret > 0)
 {
 	print 'A user with login "admin" and all permissions must be created to use this script.'."\n";
 	exit;
@@ -62,59 +62,59 @@ 
 
 
 $sql = "SELECT rowid FROM ".MAIN_DB_PREFIX."product"; $productsid = array();
-$resql=$db->query($sql);
+$resql = $db->query($sql);
 if ($resql) {
   $num = $db->num_rows($resql); $i = 0;
-  while ($i < $num) {      $row = $db->fetch_row($resql);      $productsid[$i] = $row[0];      $i++; }
+  while ($i < $num) {      $row = $db->fetch_row($resql); $productsid[$i] = $row[0]; $i++; }
 }
 
 $sql = "SELECT rowid FROM ".MAIN_DB_PREFIX."societe"; $societesid = array();
-$resql=$db->query($sql);
+$resql = $db->query($sql);
 if ($resql) {
 	$num = $db->num_rows($resql); $i = 0;
-    while ($i < $num) { $row = $db->fetch_row($resql);      $societesid[$i] = $row[0];      $i++; }
+    while ($i < $num) { $row = $db->fetch_row($resql); $societesid[$i] = $row[0]; $i++; }
 } else { print "err"; }
 
 $sql = "SELECT rowid FROM ".MAIN_DB_PREFIX."commande"; $commandesid = array();
-$resql=$db->query($sql);
+$resql = $db->query($sql);
 if ($resql) {
 	$num = $db->num_rows($resql); $i = 0;
-    while ($i < $num) { $row = $db->fetch_row($resql);      $commandesid[$i] = $row[0];      $i++; }
+    while ($i < $num) { $row = $db->fetch_row($resql); $commandesid[$i] = $row[0]; $i++; }
 } else { print "err"; }
 
 
 
 print "Generates ".GEN_NUMBER_SOCIETE." companies\n";
-for ($s = 0 ; $s < GEN_NUMBER_SOCIETE ; $s++)
+for ($s = 0; $s < GEN_NUMBER_SOCIETE; $s++)
 {
     print "Company $s\n";
     $soc = new Societe($db);
     $soc->name = "Company num ".time()."$s";
-    $soc->town = $listoftown[mt_rand(0, count($listoftown)-1)];
-    $soc->client = mt_rand(1,2);		// Une societe sur 2 est prospect, l'autre client
-    $soc->fournisseur = mt_rand(0,1);	// Une societe sur 2 est fournisseur
-    $soc->code_client='CU'.time()."$s";
-    $soc->code_fournisseur='SU'.time()."$s";
-    $soc->tva_assuj=1;
-    $soc->country_id=1;
-    $soc->country_code='FR';
+    $soc->town = $listoftown[mt_rand(0, count($listoftown) - 1)];
+    $soc->client = mt_rand(1, 2); // Une societe sur 2 est prospect, l'autre client
+    $soc->fournisseur = mt_rand(0, 1); // Une societe sur 2 est fournisseur
+    $soc->code_client = 'CU'.time()."$s";
+    $soc->code_fournisseur = 'SU'.time()."$s";
+    $soc->tva_assuj = 1;
+    $soc->country_id = 1;
+    $soc->country_code = 'FR';
 	// Un client sur 3 a une remise de 5%
-    $user_remise=mt_rand(1,3); if ($user_remise==3) $soc->remise_percent=5;
+    $user_remise = mt_rand(1, 3); if ($user_remise == 3) $soc->remise_percent = 5;
 	print "> client=".$soc->client.", fournisseur=".$soc->fournisseur.", remise=".$soc->remise_percent."\n";
     $soc->note_private = 'Company created by the script generate-societe.php';
     $socid = $soc->create();
 
     if ($socid >= 0)
     {
-        $rand = mt_rand(1,4);
+        $rand = mt_rand(1, 4);
         print "> Generates $rand contact(s)\n";
-        for ($c = 0 ; $c < $rand ; $c++)
+        for ($c = 0; $c < $rand; $c++)
         {
             $contact = new Contact($db);
             $contact->socid = $soc->id;
             $contact->lastname = "Lastname".$c;
-            $contact->firstname = $listoflastname[mt_rand(0, count($listoflastname)-1)];
-            if ( $contact->create($user) )
+            $contact->firstname = $listoflastname[mt_rand(0, count($listoflastname) - 1)];
+            if ($contact->create($user))
             {
 
             }

Braces +4 added lines, -3 removed lines

@@ -99,7 +99,9 @@ 
     $soc->country_id=1;
     $soc->country_code='FR';
 	// Un client sur 3 a une remise de 5%
-    $user_remise=mt_rand(1,3); if ($user_remise==3) $soc->remise_percent=5;
+    $user_remise=mt_rand(1,3); if ($user_remise==3) {
+        $soc->remise_percent=5;
+    }
 	print "> client=".$soc->client.", fournisseur=".$soc->fournisseur.", remise=".$soc->remise_percent."\n";
     $soc->note_private = 'Company created by the script generate-societe.php';
     $socid = $soc->create();
@@ -121,8 +123,7 @@ 
         }
 
         print "Company ".$s." created nom=".$soc->name."\n";
-    }
-    else
+    } else
     {
     	print "Error: ".$soc->error."\n";
     }

dolibarr/dev/initdata/import-thirdparties.php

Indentation +94 added lines, -94 removed lines

@@ -81,8 +81,8 @@ 
 $ret=$user->fetch('','admin');
 if (! $ret > 0)
 {
-	print 'A user with login "admin" and all permissions must be created to use this script.'."\n";
-	exit;
+    print 'A user with login "admin" and all permissions must be created to use this script.'."\n";
+    exit;
 }
 $user->getrights();
 
@@ -182,116 +182,116 @@ 
             print " - Error in create result code = ".$ret." - ".$object->errorsToString();
             $errorrecord++;
         }
-    	else
-    	{
-    	    print " - Creation OK with name ".$object->name." - id = ".$ret;
-    	}
+        else
+        {
+            print " - Creation OK with name ".$object->name." - id = ".$ret;
+        }
     }
 
     if (! $errorrecord)
     {
         dol_syslog("Set price level");
-	    $object->set_price_level($object->price_level, $user);
+        $object->set_price_level($object->price_level, $user);
+    }
+
+    // Assign sales representative
+    if (! $errorrecord && $fields[3])
+    {
+        $salesrep=new User($db);
+
+        $tmp=explode(' ',$fields[3],2);
+        $salesrep->firstname = trim($tmp[0]);
+        $salesrep->lastname = trim($tmp[1]);
+        if ($salesrep->lastname) $salesrep->login = strtolower(substr($salesrep->firstname, 0, 1)) . strtolower(substr($salesrep->lastname, 0));
+        else $salesrep->login=strtolower($salesrep->firstname);
+        $salesrep->login=preg_replace('/ /','',$salesrep->login);
+        $salesrep->fetch(0,$salesrep->login);
+
+        $result = $object->add_commercial($user, $salesrep->id);
+        if ($result < 0)
+        {
+            print " - Error in create link with sale representative result code = ".$result." - ".$object->errorsToString();
+            $errorrecord++;
+        }
+        else
+        {
+            print " - create link sale representative OK";
+        }
     }
 
-	// Assign sales representative
-	if (! $errorrecord && $fields[3])
-	{
-    	$salesrep=new User($db);
-
-    	$tmp=explode(' ',$fields[3],2);
-    	$salesrep->firstname = trim($tmp[0]);
-    	$salesrep->lastname = trim($tmp[1]);
-    	if ($salesrep->lastname) $salesrep->login = strtolower(substr($salesrep->firstname, 0, 1)) . strtolower(substr($salesrep->lastname, 0));
-    	else $salesrep->login=strtolower($salesrep->firstname);
-    	$salesrep->login=preg_replace('/ /','',$salesrep->login);
-    	$salesrep->fetch(0,$salesrep->login);
-
-    	$result = $object->add_commercial($user, $salesrep->id);
-    	if ($result < 0)
-    	{
-    	    print " - Error in create link with sale representative result code = ".$result." - ".$object->errorsToString();
-    	    $errorrecord++;
-    	}
-    	else
-    	{
-    	    print " - create link sale representative OK";
-    	}
-	}
-
-	dol_syslog("Add invoice contacts");
-	// Insert an invoice contact if there is an invoice email != standard email
-	if (! $errorrecord && $fields[27] && $fields[26] != $fields[27])
-	{
-	    $ret1=$ret2=0;
-
-	    $contact = new Contact($db);
-	    $contact->lastname = $object->name;
-	    $contact->address=$object->address;
-	    $contact->zip=$object->zip;
-	    $contact->town=$object->town;
-	    $contact->country_id=$object->country_id;
-	    $contact->email=$fields[27];
-	    $contact->socid=$object->id;
-
-	    $ret1=$contact->create($user);
-	    if ($ret1 > 0)
-	    {
-	        //$ret2=$contact->add_contact($object->id, 'BILLING');
-	    }
-	    if ($ret1 < 0 || $ret2 < 0)
+    dol_syslog("Add invoice contacts");
+    // Insert an invoice contact if there is an invoice email != standard email
+    if (! $errorrecord && $fields[27] && $fields[26] != $fields[27])
+    {
+        $ret1=$ret2=0;
+
+        $contact = new Contact($db);
+        $contact->lastname = $object->name;
+        $contact->address=$object->address;
+        $contact->zip=$object->zip;
+        $contact->town=$object->town;
+        $contact->country_id=$object->country_id;
+        $contact->email=$fields[27];
+        $contact->socid=$object->id;
+
+        $ret1=$contact->create($user);
+        if ($ret1 > 0)
+        {
+            //$ret2=$contact->add_contact($object->id, 'BILLING');
+        }
+        if ($ret1 < 0 || $ret2 < 0)
         {
             print " - Error in create contact result code = ".$ret1." ".$ret2." - ".$object->errorsToString();
             $errorrecord++;
         }
-    	else
-    	{
-    	    print " - create contact OK";
-    	}
-	}
-
-	dol_syslog("Add delivery contacts");
-	// Insert a delivery contact
-	if (! $errorrecord && $fields[47])
-	{
-	    $ret1=$ret2=0;
-
-	    $contact2 = new Contact($db);
-	    $contact2->lastname = 'Service livraison - '.$fields[47];
-	    $contact2->address = $fields[48];
-	    $contact2->zip = $fields[50];
-	    $contact2->town = $fields[51];
-	    $contact2->country_id=dol_getIdFromCode($db, trim($fields[52]), 'c_country', 'code', 'rowid');
-	    $contact2->note_public=$fields[54];
-	    $contact2->socid=$object->id;
-
-	    // Extrafields
-	    $contact2->array_options['options_anazoneliv']=price2num($fields[53]);
-
-	    $ret1=$contact2->create($user);
-	    if ($ret1 > 0)
-	    {
-	        //$ret2=$contact2->add_contact($object->id, 'SHIPPING');
-	    }
-	    if ($ret1 < 0 || $ret2 < 0)
+        else
+        {
+            print " - create contact OK";
+        }
+    }
+
+    dol_syslog("Add delivery contacts");
+    // Insert a delivery contact
+    if (! $errorrecord && $fields[47])
+    {
+        $ret1=$ret2=0;
+
+        $contact2 = new Contact($db);
+        $contact2->lastname = 'Service livraison - '.$fields[47];
+        $contact2->address = $fields[48];
+        $contact2->zip = $fields[50];
+        $contact2->town = $fields[51];
+        $contact2->country_id=dol_getIdFromCode($db, trim($fields[52]), 'c_country', 'code', 'rowid');
+        $contact2->note_public=$fields[54];
+        $contact2->socid=$object->id;
+
+        // Extrafields
+        $contact2->array_options['options_anazoneliv']=price2num($fields[53]);
+
+        $ret1=$contact2->create($user);
+        if ($ret1 > 0)
+        {
+            //$ret2=$contact2->add_contact($object->id, 'SHIPPING');
+        }
+        if ($ret1 < 0 || $ret2 < 0)
         {
             print " - Error in create contact result code = ".$ret1." ".$ret2." - ".$object->errorsToString();
             $errorrecord++;
         }
-    	else
-    	{
-    	    print " - create contact OK";
-    	}
-	}
+        else
+        {
+            print " - create contact OK";
+        }
+    }
 
 
-	print "\n";
+    print "\n";
 
-	if ($errorrecord)
-	{
-	    fwrite($fhandleerr, 'Error on record nb '.$i." - ".$object->errorsToString()."\n");
-	    $error++;    // $errorrecord will be reset
-	}
+    if ($errorrecord)
+    {
+        fwrite($fhandleerr, 'Error on record nb '.$i." - ".$object->errorsToString()."\n");
+        $error++;    // $errorrecord will be reset
+    }
 }
 
 

Spacing +55 added lines, -55 removed lines

@@ -28,27 +28,27 @@ 
 // Test si mode batch
 $sapi_type = php_sapi_name();
 $script_file = basename(__FILE__);
-$path=dirname(__FILE__).'/';
+$path = dirname(__FILE__).'/';
 if (substr($sapi_type, 0, 3) == 'cgi') {
     echo "Erreur: Vous utilisez l'interpreteur PHP pour le mode CGI. Pour executer mailing-send.php en ligne de commande, vous devez utiliser l'interpreteur PHP pour le mode CLI.\n";
     exit;
 }
 
 // Recupere root dolibarr
-$path=preg_replace('/import-thirdparties.php/i','',$_SERVER["PHP_SELF"]);
+$path = preg_replace('/import-thirdparties.php/i', '', $_SERVER["PHP_SELF"]);
 require $path."../../htdocs/master.inc.php";
 include_once DOL_DOCUMENT_ROOT.'/societe/class/societe.class.php';
 include_once DOL_DOCUMENT_ROOT.'/product/class/product.class.php';
 
-$delimiter=',';
-$enclosure='"';
-$linelength=10000;
-$escape='/';
+$delimiter = ',';
+$enclosure = '"';
+$linelength = 10000;
+$escape = '/';
 
 // Global variables
-$version=DOL_VERSION;
-$confirmed=1;
-$error=0;
+$version = DOL_VERSION;
+$confirmed = 1;
+$error = 0;
 
 
 /*
@@ -57,29 +57,29 @@ 
 
 @set_time_limit(0);
 print "***** ".$script_file." (".$version.") pid=".dol_getmypid()." *****\n";
-dol_syslog($script_file." launched with arg ".implode(',',$argv));
+dol_syslog($script_file." launched with arg ".implode(',', $argv));
 
 $mode = $argv[1];
 $filepath = $argv[2];
 $filepatherr = $filepath.'.err';
 //$defaultlang = empty($argv[3])?'en_US':$argv[3];
-$startlinenb = empty($argv[3])?1:$argv[3];
-$endlinenb = empty($argv[4])?0:$argv[4];
+$startlinenb = empty($argv[3]) ? 1 : $argv[3];
+$endlinenb = empty($argv[4]) ? 0 : $argv[4];
 
-if (empty($mode) || ! in_array($mode,array('test','confirm','confirmforced')) || empty($filepath)) {
+if (empty($mode) || !in_array($mode, array('test', 'confirm', 'confirmforced')) || empty($filepath)) {
     print "Usage:  $script_file (test|confirm|confirmforced) filepath.csv [startlinenb] [endlinenb]\n";
     print "Usage:  $script_file test myfilepath.csv 2 1002\n";
     print "\n";
     exit(-1);
 }
-if (! file_exists($filepath)) {
+if (!file_exists($filepath)) {
     print "Error: File ".$filepath." not found.\n";
     print "\n";
     exit(-1);
 }
 
-$ret=$user->fetch('','admin');
-if (! $ret > 0)
+$ret = $user->fetch('', 'admin');
+if (!$ret > 0)
 {
 	print 'A user with login "admin" and all permissions must be created to use this script.'."\n";
 	exit;
@@ -87,7 +87,7 @@ 
 $user->getrights();
 
 // Ask confirmation
-if (! $confirmed)
+if (!$confirmed)
 {
     print "Hit Enter to continue or CTRL+C to stop...\n";
     $input = trim(fgets(STDIN));
@@ -95,13 +95,13 @@ 
 
 // Open input and ouput files
 $fhandle = fopen($filepath, 'r');
-if (! $fhandle)
+if (!$fhandle)
 {
     print 'Error: Failed to open file '.$filepath."\n";
     exit(1);
 }
 $fhandleerr = fopen($filepatherr, 'w');
-if (! $fhandleerr)
+if (!$fhandleerr)
 {
     print 'Error: Failed to open file '.$filepatherr."\n";
     exit(1);
@@ -112,12 +112,12 @@ 
 
 $db->begin();
 
-$i=0;
-$nboflines=0;
-while ($fields=fgetcsv($fhandle, $linelength, $delimiter, $enclosure, $escape))
+$i = 0;
+$nboflines = 0;
+while ($fields = fgetcsv($fhandle, $linelength, $delimiter, $enclosure, $escape))
 {
     $i++;
-    $errorrecord=0;
+    $errorrecord = 0;
 
     if ($startlinenb && $i < $startlinenb) continue;
     if ($endlinenb && $i > $endlinenb) continue;
@@ -129,8 +129,8 @@ 
     $object->client = $fields[7];
     $object->fournisseur = $fields[8];
 
-    $object->name = $fields[13]?trim($fields[13]):$fields[0];
-    $object->name_alias = $fields[0]!=$fields[13]?trim($fields[0]):'';
+    $object->name = $fields[13] ?trim($fields[13]) : $fields[0];
+    $object->name_alias = $fields[0] != $fields[13] ?trim($fields[0]) : '';
 
     $object->address = trim($fields[14]);
     $object->zip = trim($fields[15]);
@@ -171,12 +171,12 @@ 
 
 
     // Extrafields
-    $object->array_options['options_anastate']=price2num($fields[20]);
-    $object->array_options['options_anaregion']=price2num($fields[17]);
+    $object->array_options['options_anastate'] = price2num($fields[20]);
+    $object->array_options['options_anaregion'] = price2num($fields[17]);
 
-    if (! $errorrecord)
+    if (!$errorrecord)
     {
-        $ret=$object->create($user);
+        $ret = $object->create($user);
         if ($ret < 0)
         {
             print " - Error in create result code = ".$ret." - ".$object->errorsToString();
@@ -188,24 +188,24 @@ 
     	}
     }
 
-    if (! $errorrecord)
+    if (!$errorrecord)
     {
         dol_syslog("Set price level");
 	    $object->set_price_level($object->price_level, $user);
     }
 
 	// Assign sales representative
-	if (! $errorrecord && $fields[3])
+	if (!$errorrecord && $fields[3])
 	{
-    	$salesrep=new User($db);
+    	$salesrep = new User($db);
 
-    	$tmp=explode(' ',$fields[3],2);
+    	$tmp = explode(' ', $fields[3], 2);
     	$salesrep->firstname = trim($tmp[0]);
     	$salesrep->lastname = trim($tmp[1]);
-    	if ($salesrep->lastname) $salesrep->login = strtolower(substr($salesrep->firstname, 0, 1)) . strtolower(substr($salesrep->lastname, 0));
-    	else $salesrep->login=strtolower($salesrep->firstname);
-    	$salesrep->login=preg_replace('/ /','',$salesrep->login);
-    	$salesrep->fetch(0,$salesrep->login);
+    	if ($salesrep->lastname) $salesrep->login = strtolower(substr($salesrep->firstname, 0, 1)).strtolower(substr($salesrep->lastname, 0));
+    	else $salesrep->login = strtolower($salesrep->firstname);
+    	$salesrep->login = preg_replace('/ /', '', $salesrep->login);
+    	$salesrep->fetch(0, $salesrep->login);
 
     	$result = $object->add_commercial($user, $salesrep->id);
     	if ($result < 0)
@@ -221,20 +221,20 @@ 
 
 	dol_syslog("Add invoice contacts");
 	// Insert an invoice contact if there is an invoice email != standard email
-	if (! $errorrecord && $fields[27] && $fields[26] != $fields[27])
+	if (!$errorrecord && $fields[27] && $fields[26] != $fields[27])
 	{
-	    $ret1=$ret2=0;
+	    $ret1 = $ret2 = 0;
 
 	    $contact = new Contact($db);
 	    $contact->lastname = $object->name;
-	    $contact->address=$object->address;
-	    $contact->zip=$object->zip;
-	    $contact->town=$object->town;
-	    $contact->country_id=$object->country_id;
-	    $contact->email=$fields[27];
-	    $contact->socid=$object->id;
-
-	    $ret1=$contact->create($user);
+	    $contact->address = $object->address;
+	    $contact->zip = $object->zip;
+	    $contact->town = $object->town;
+	    $contact->country_id = $object->country_id;
+	    $contact->email = $fields[27];
+	    $contact->socid = $object->id;
+
+	    $ret1 = $contact->create($user);
 	    if ($ret1 > 0)
 	    {
 	        //$ret2=$contact->add_contact($object->id, 'BILLING');
@@ -252,23 +252,23 @@ 
 
 	dol_syslog("Add delivery contacts");
 	// Insert a delivery contact
-	if (! $errorrecord && $fields[47])
+	if (!$errorrecord && $fields[47])
 	{
-	    $ret1=$ret2=0;
+	    $ret1 = $ret2 = 0;
 
 	    $contact2 = new Contact($db);
 	    $contact2->lastname = 'Service livraison - '.$fields[47];
 	    $contact2->address = $fields[48];
 	    $contact2->zip = $fields[50];
 	    $contact2->town = $fields[51];
-	    $contact2->country_id=dol_getIdFromCode($db, trim($fields[52]), 'c_country', 'code', 'rowid');
-	    $contact2->note_public=$fields[54];
-	    $contact2->socid=$object->id;
+	    $contact2->country_id = dol_getIdFromCode($db, trim($fields[52]), 'c_country', 'code', 'rowid');
+	    $contact2->note_public = $fields[54];
+	    $contact2->socid = $object->id;
 
 	    // Extrafields
-	    $contact2->array_options['options_anazoneliv']=price2num($fields[53]);
+	    $contact2->array_options['options_anazoneliv'] = price2num($fields[53]);
 
-	    $ret1=$contact2->create($user);
+	    $ret1 = $contact2->create($user);
 	    if ($ret1 > 0)
 	    {
 	        //$ret2=$contact2->add_contact($object->id, 'SHIPPING');
@@ -290,7 +290,7 @@ 
 	if ($errorrecord)
 	{
 	    fwrite($fhandleerr, 'Error on record nb '.$i." - ".$object->errorsToString()."\n");
-	    $error++;    // $errorrecord will be reset
+	    $error++; // $errorrecord will be reset
 	}
 }
 

Braces +25 added lines, -17 removed lines

@@ -119,8 +119,12 @@ 
     $i++;
     $errorrecord=0;
 
-    if ($startlinenb && $i < $startlinenb) continue;
-    if ($endlinenb && $i > $endlinenb) continue;
+    if ($startlinenb && $i < $startlinenb) {
+        continue;
+    }
+    if ($endlinenb && $i > $endlinenb) {
+        continue;
+    }
 
     $nboflines++;
 
@@ -147,8 +151,12 @@ 
     if ($fields[36])
     {
         $condpayment = trim($fields[36]);
-        if ($condpayment == 'A la commande') $condpayment = 'A réception de commande';
-        if ($condpayment == 'A reception facture') $condpayment = 'Réception de facture';
+        if ($condpayment == 'A la commande') {
+            $condpayment = 'A réception de commande';
+        }
+        if ($condpayment == 'A reception facture') {
+            $condpayment = 'Réception de facture';
+        }
         $object->cond_reglement_id = dol_getIdFromCode($db, $condpayment, 'c_payment_term', 'libelle_facture', 'rowid', 1);
         if (empty($object->cond_reglement_id))
         {
@@ -165,7 +173,9 @@ 
 
     // Set price level
     $object->price_level = 1;
-    if ($labeltype == 'Revendeur') $object->price_level = 2;
+    if ($labeltype == 'Revendeur') {
+        $object->price_level = 2;
+    }
 
     print "Process line nb ".$i.", name ".$object->name;
 
@@ -181,8 +191,7 @@ 
         {
             print " - Error in create result code = ".$ret." - ".$object->errorsToString();
             $errorrecord++;
-        }
-    	else
+        } else
     	{
     	    print " - Creation OK with name ".$object->name." - id = ".$ret;
     	}
@@ -202,8 +211,11 @@ 
     	$tmp=explode(' ',$fields[3],2);
     	$salesrep->firstname = trim($tmp[0]);
     	$salesrep->lastname = trim($tmp[1]);
-    	if ($salesrep->lastname) $salesrep->login = strtolower(substr($salesrep->firstname, 0, 1)) . strtolower(substr($salesrep->lastname, 0));
-    	else $salesrep->login=strtolower($salesrep->firstname);
+    	if ($salesrep->lastname) {
+    	    $salesrep->login = strtolower(substr($salesrep->firstname, 0, 1)) . strtolower(substr($salesrep->lastname, 0));
+    	} else {
+    	    $salesrep->login=strtolower($salesrep->firstname);
+    	}
     	$salesrep->login=preg_replace('/ /','',$salesrep->login);
     	$salesrep->fetch(0,$salesrep->login);
 
@@ -212,8 +224,7 @@ 
     	{
     	    print " - Error in create link with sale representative result code = ".$result." - ".$object->errorsToString();
     	    $errorrecord++;
-    	}
-    	else
+    	} else
     	{
     	    print " - create link sale representative OK";
     	}
@@ -243,8 +254,7 @@ 
         {
             print " - Error in create contact result code = ".$ret1." ".$ret2." - ".$object->errorsToString();
             $errorrecord++;
-        }
-    	else
+        } else
     	{
     	    print " - create contact OK";
     	}
@@ -277,8 +287,7 @@ 
         {
             print " - Error in create contact result code = ".$ret1." ".$ret2." - ".$object->errorsToString();
             $errorrecord++;
-        }
-    	else
+        } else
     	{
     	    print " - create contact OK";
     	}
@@ -305,8 +314,7 @@ 
 {
     print "Rollback any changes.\n";
     $db->rollback();
-}
-else
+} else
 {
     print "Commit all changes.\n";
     $db->commit();