alxarafe / alixar

Dolibarr/Code/Core/Classes/dolReceiptPrinter.php

Spacing +1 added lines, -1 removed lines

@@ -944,7 +944,7 @@
                         }
                         break;
                     default:
-                        $parameters = array('vals' => $vals[$tplline],'object' => $object,'nbcharactbyline' => $nbcharactbyline);
+                        $parameters = array('vals' => $vals[$tplline], 'object' => $object, 'nbcharactbyline' => $nbcharactbyline);
                         $action = '';
                         $reshook = $hookmanager->executeHooks('sendToPrinterAfter', $parameters, $this, $action); // Note that $action and $object may have been modified by some hooks
 

Dolibarr/Code/Core/Classes/Lessc.php

Spacing +31 added lines, -31 removed lines

@@ -98,8 +98,8 @@ 
 	protected function findImport($url)
 	{
 		foreach ((array) $this->importDir as $dir) {
-			$full = $dir.(substr($dir, -1) != '/' ? '/' : '').$url;
-			if ($this->fileExists($file = $full.'.less') || $this->fileExists($file = $full)) {
+			$full = $dir . (substr($dir, -1) != '/' ? '/' : '') . $url;
+			if ($this->fileExists($file = $full . '.less') || $this->fileExists($file = $full)) {
 				return $file;
 			}
 		}
@@ -250,9 +250,9 @@ 
 				$this->compileMedia($block);
 				break;
 			case "directive":
-				$name = "@".$block->name;
+				$name = "@" . $block->name;
 				if (!empty($block->value)) {
-					$name .= " ".$this->compileValue($this->reduce($block->value));
+					$name .= " " . $this->compileValue($this->reduce($block->value));
 				}
 
 				$this->compileNestedBlock($block, array($name));
@@ -424,8 +424,8 @@ 
 						break;
 					case "mediaExp":
 						if (isset($q[2])) {
-							$parts[] = "($q[1]: ".
-								$this->compileValue($this->reduce($q[2])).")";
+							$parts[] = "($q[1]: " .
+								$this->compileValue($this->reduce($q[2])) . ")";
 						} else {
 							$parts[] = "($q[1])";
 						}
@@ -443,7 +443,7 @@ 
 
 		$out = "@media";
 		if (!empty($parts)) {
-			$out .= " ".
+			$out .= " " .
 				implode($this->formatter->selectorSeparator, $compiledQueries);
 		}
 		return $out;
@@ -530,7 +530,7 @@ 
 				if ($count > 0) {
 					$out[] = trim($child);
 				} else {
-					$out[] = trim($parent.' '.$child);
+					$out[] = trim($parent . ' ' . $child);
 				}
 			}
 		}
@@ -735,7 +735,7 @@ 
 					$value = $a[2];
 				} else {
 					$value = null; // :(
-					$this->throwError("Failed to assign arg ".$a[1]); // This ends function by throwing an exception
+					$this->throwError("Failed to assign arg " . $a[1]); // This ends function by throwing an exception
 				}
 
 				$value = $this->reduce($value);
@@ -800,7 +800,7 @@ 
 							$orderedArgs[] = $this->reduce($arg[1]);
 							break;
 						default:
-							$this->throwError("Unknown arg type: ".$arg[0]);
+							$this->throwError("Unknown arg type: " . $arg[0]);
 					}
 				}
 
@@ -865,7 +865,7 @@ 
 				break;
 			case "directive":
 				list(, $name, $value) = $prop;
-				$out->lines[] = "@$name ".$this->compileValue($this->reduce($value)).';';
+				$out->lines[] = "@$name " . $this->compileValue($this->reduce($value)) . ';';
 				break;
 			case "comment":
 				$out->lines[] = $prop[1];
@@ -881,7 +881,7 @@ 
 				$result = $this->tryImport($importPath, $block, $out);
 
 				$this->env->imports[$importId] = $result === false ?
-					array(false, "@import ".$this->compileValue($importPath).";") : $result;
+					array(false, "@import " . $this->compileValue($importPath) . ";") : $result;
 
 				break;
 			case "import_mixin":
@@ -936,7 +936,7 @@ 
 				if ($this->numberPrecision !== null) {
 					$num = round($num, $this->numberPrecision);
 				}
-				return $num.$unit;
+				return $num . $unit;
 			case 'string':
 				// [1] - contents of string (includes quotes)
 				list(, $delim, $content) = $value;
@@ -945,7 +945,7 @@ 
 						$part = $this->compileValue($part);
 					}
 				}
-				return $delim.implode($content).$delim;
+				return $delim . implode($content) . $delim;
 			case 'color':
 				// [1] - red component (either number or a %)
 				// [2] - green component
@@ -957,7 +957,7 @@ 
 				$b = round($b);
 
 				if (count($value) == 5 && $value[4] != 1) { // rgba
-					return 'rgba('.$r.','.$g.','.$b.','.$value[4].')';
+					return 'rgba(' . $r . ',' . $g . ',' . $b . ',' . $value[4] . ')';
 				}
 
 				$h = sprintf("#%02x%02x%02x", $r, $g, $b);
@@ -965,7 +965,7 @@ 
 				if (!empty($this->formatter->compressColors)) {
 					// Converting hex color to short notation (e.g. #003399 to #039)
 					if ($h[1] === $h[2] && $h[3] === $h[4] && $h[5] === $h[6]) {
-						$h = '#'.$h[1].$h[3].$h[5];
+						$h = '#' . $h[1] . $h[3] . $h[5];
 					}
 				}
 
@@ -973,7 +973,7 @@ 
 
 			case 'function':
 				list(, $name, $args) = $value;
-				return $name.'('.$this->compileValue($args).')';
+				return $name . '(' . $this->compileValue($args) . ')';
 			default: // assumed to be unit
 				$this->throwError("unknown value type: $value[0]");
 		}
@@ -1138,7 +1138,7 @@ 
 			}
 		}
 
-		return 'url("'.$url.'")';
+		return 'url("' . $url . '")';
 	}
 
 	// utility func to unquote a string
@@ -1186,7 +1186,7 @@ 
 				$i++;
 				$rep = $this->compileValue($this->lib_e($val));
 				$template = preg_replace(
-					'/'.self::preg_quote($match).'/',
+					'/' . self::preg_quote($match) . '/',
 					$rep,
 					$template,
 					1
@@ -1539,7 +1539,7 @@ 
 			$numValues = count($values);
 			if ($expectedArgs != $numValues) {
 				if ($name) {
-					$name = $name.": ";
+					$name = $name . ": ";
 				}
 
 				$this->throwError("{$name}expecting $expectedArgs arguments, got $numValues");
@@ -1731,7 +1731,7 @@ 
 			case "interpolate":
 				$reduced = $this->reduce($value[1]);
 				$var = $this->compileValue($reduced);
-				$res = $this->reduce(array("variable", $this->vPrefix.$var));
+				$res = $this->reduce(array("variable", $this->vPrefix . $var));
 
 				if ($res[0] == "raw_color") {
 					$res = $this->coerceColor($res);
@@ -1746,7 +1746,7 @@ 
 				$key = $value[1];
 				if (is_array($key)) {
 					$key = $this->reduce($key);
-					$key = $this->vPrefix.$this->compileValue($this->lib_e($key));
+					$key = $this->vPrefix . $this->compileValue($this->lib_e($key));
 				}
 
 				$seen = & $this->env->seenNames;
@@ -1792,7 +1792,7 @@ 
 				}
 
 				$f = isset($this->libFunctions[$name]) ?
-					$this->libFunctions[$name] : array($this, 'lib_'.str_replace('-', '_', $name));
+					$this->libFunctions[$name] : array($this, 'lib_' . str_replace('-', '_', $name));
 
 				if (is_callable($f)) {
 					if ($args[0] == 'list') {
@@ -1868,7 +1868,7 @@ 
 					$t = intval($num) % $width;
 					$num /= $width;
 
-					$c[$i] = $t * (256 / $width) + $t * floor(16/$width);
+					$c[$i] = $t * (256 / $width) + $t * floor(16 / $width);
 				}
 
 				return $c;
@@ -1957,7 +1957,7 @@ 
 		// make the expression look it did before being parsed
 		$paddedOp = $op;
 		if ($whiteBefore) {
-			$paddedOp = " ".$paddedOp;
+			$paddedOp = " " . $paddedOp;
 		}
 		if ($whiteAfter) {
 			$paddedOp .= " ";
@@ -2047,7 +2047,7 @@ 
 					$out[] = $lval / $rval;
 					break;
 				default:
-					$this->throwError('evaluate error: color op number failed on op '.$op);
+					$this->throwError('evaluate error: color op number failed on op ' . $op);
 			}
 		}
 		return $this->fixColor($out);
@@ -2118,7 +2118,7 @@ 
 			case '=<':
 				return $this->toBool($left[1] <= $right[1]);
 			default:
-				$this->throwError('parse error: unknown number operator: '.$op);
+				$this->throwError('parse error: unknown number operator: ' . $op);
 		}
 
 		return array("number", $value, $unit);
@@ -2170,7 +2170,7 @@ 
 	{
 		$current = $this->env;
 
-		$isArguments = $name == $this->vPrefix.'arguments';
+		$isArguments = $name == $this->vPrefix . 'arguments';
 		while ($current) {
 			if ($isArguments && isset($current->arguments)) {
 				return array('list', ' ', $current->arguments);
@@ -2195,7 +2195,7 @@ 
 		$value = null;
 		foreach ($args as $name => $strValue) {
 			if ($name[0] !== '@') {
-				$name = '@'.$name;
+				$name = '@' . $name;
 			}
 			$parser->count = 0;
 			$parser->buffer = (string) $strValue;
@@ -2249,7 +2249,7 @@ 
 	public function compileFile($fname, $outFname = null)
 	{
 		if (!is_readable($fname)) {
-			throw new Exception('load error: failed to find '.$fname);
+			throw new Exception('load error: failed to find ' . $fname);
 		}
 
 		$pi = pathinfo($fname);
@@ -2257,7 +2257,7 @@ 
 		$oldImport = $this->importDir;
 
 		$this->importDir = (array) $this->importDir;
-		$this->importDir[] = $pi['dirname'].'/';
+		$this->importDir[] = $pi['dirname'] . '/';
 
 		$this->addParsedFile($fname);
 

Dolibarr/Code/Core/Classes/FileUpload.php

Spacing +1 added lines, -1 removed lines

@@ -428,7 +428,7 @@
         // Sanitize to avoid stream execution when calling file_size(). Not that this is a second security because
         // most streams are already disabled by stream_wrapper_unregister() in filefunc.inc.php
         $uploaded_file = preg_replace('/\s*(http|ftp)s?:/i', '', $uploaded_file);
-        $uploaded_file = realpath($uploaded_file);  // A hack to be sure the file point to an existing file on disk (and is not a SSRF attack)
+        $uploaded_file = realpath($uploaded_file); // A hack to be sure the file point to an existing file on disk (and is not a SSRF attack)
 
         $validate = $this->validate($uploaded_file, $file, $error, $index);
 

Dolibarr/Code/Core/Classes/FormSetupItem.php

Spacing +3 added lines, -3 removed lines

@@ -898,7 +898,7 @@ 
         } elseif ($this->type == 'html') {
             $out .= $this->generateInputFieldHtml();
         } elseif ($this->type == 'color') {
-            $out .=  $this->generateInputFieldColor();
+            $out .= $this->generateInputFieldColor();
         } elseif ($this->type == 'yesno') {
             if (!empty($conf->use_javascript_ajax)) {
                 $out .= ajax_constantonoff($this->confKey);
@@ -1209,9 +1209,9 @@ 
         } elseif ($this->type == 'selectUser') {
             $out .= $this->generateOutputFieldSelectUser();
         } elseif ($this->type == 'html') {
-            $out .=  $this->fieldValue;
+            $out .= $this->fieldValue;
         } elseif ($this->type == 'color') {
-            $out .=  $this->generateOutputFieldColor();
+            $out .= $this->generateOutputFieldColor();
         } elseif ($this->type == 'yesno') {
             if (!empty($conf->use_javascript_ajax)) {
                 $out .= ajax_constantonoff($this->confKey, array(), $this->entity); // TODO possibility to add $input parameter

Dolibarr/Code/Core/Classes/FormIntervention.php

Spacing +1 added lines, -1 removed lines

@@ -113,7 +113,7 @@
                             $out .= '<option value="' . $obj->rowid . '" selected>' . $labeltoshow . '</option>';
                         } else {
                             $disabled = 0;
-                            if (!$obj->fk_statut > 0 && ! $draftonly) {
+                            if (!$obj->fk_statut > 0 && !$draftonly) {
                                 $disabled = 1;
                                 $labeltoshow .= ' (' . $langs->trans("Draft") . ')';
                             }

Dolibarr/Code/Core/Classes/FormWebsite.php

Spacing +1 added lines, -1 removed lines

@@ -387,7 +387,7 @@
 
             $out .= '<div class="template-option" data-template="' . $template . '" data-content="' . htmlentities($contentHtml) . '">';
             $out .= '<img class="maillayout" alt="' . $template . '" src="' . constant('DOL_URL_ROOT') . '/theme/common/maillayout/' . $template . '.png" />';
-            $out .= '<span class="template-option-text">' . ($template != 'text'  ? ucfirst($template) : ucfirst($templateFunction)) . '</span>';
+            $out .= '<span class="template-option-text">' . ($template != 'text' ? ucfirst($template) : ucfirst($templateFunction)) . '</span>';
             $out .= '</div>';
         }
         $out .= '<input type="hidden" name="sample" value="" />';

Dolibarr/Code/Core/Classes/FormMail.php

Spacing +1 added lines, -1 removed lines

@@ -886,7 +886,7 @@
                         $maxfilesizearray = getMaxFileSizeArray();
                         $maxmin = $maxfilesizearray['maxmin'];
                         if ($maxmin > 0) {
-                            $out .= '<input type="hidden" name="MAX_FILE_SIZE" value="' . ($maxmin * 1024) . '">';  // MAX_FILE_SIZE must precede the field type=file
+                            $out .= '<input type="hidden" name="MAX_FILE_SIZE" value="' . ($maxmin * 1024) . '">'; // MAX_FILE_SIZE must precede the field type=file
                         }
                         // Can add other files
                         if (!getDolGlobalString('FROM_MAIL_DONT_USE_INPUT_FILE_MULTIPLE')) {

Dolibarr/Code/Core/Classes/FormExpenseReport.php

Spacing +1 added lines, -1 removed lines

@@ -78,7 +78,7 @@
             if (!empty($selected) && $selected == $key) {
                 $html .= '<option value="' . $key . '" selected>';
             } else {
-                $html .=  '<option value="' . $key . '">';
+                $html .= '<option value="' . $key . '">';
             }
             $html .= $langs->trans($val);
             $html .= '</option>';

Dolibarr/Code/Core/Classes/HookManager.php

Spacing +1 added lines, -1 removed lines

@@ -290,7 +290,7 @@
                 '@phan-var-force array<string,CommonHookActions> $modules';
                 // Loop on each active hooks of module for this context
                 foreach ($modules as $module => $actionclassinstance) {
-                    $module = preg_replace('/^\d+:/', '', $module);     // $module string is 'priority:module'
+                    $module = preg_replace('/^\d+:/', '', $module); // $module string is 'priority:module'
                     //print "Before hook ".get_class($actionclassinstance)." method=".$method." module=".$module." hooktype=".$hooktype." results=".count($actionclassinstance->results)." resprints=".count($actionclassinstance->resprints)." resaction=".$resaction."<br>\n";
 
                     // test to avoid running twice a hook, when a module implements several active contexts