Dolibarr / dolibarr

htdocs/includes/odtphp/zip/PclZipProxy.php

Braces +7 added lines, -2 removed lines

@@ -1,5 +1,7 @@ 
 <?php
-if (! defined('ODTPHP_PATHTOPCLZIP')) define('ODTPHP_PATHTOPCLZIP','pclzip/');
+if (! defined('ODTPHP_PATHTOPCLZIP')) {
+	define('ODTPHP_PATHTOPCLZIP','pclzip/');
+}
 require_once ODTPHP_PATHTOPCLZIP.'pclzip.lib.php';
 require_once 'ZipInterface.php';
 class PclZipProxyException extends Exception
@@ -32,7 +34,10 @@ 
 			throw new PclZipProxyException('PclZip class not loaded - PclZip library
 			 is required for using PclZipProxy'); ;
 		}
-		if ($forcedir) $this->tmpdir=preg_replace('|[//\/]$|','',$forcedir);	// $this->tmpdir must not contains / at the end
+		if ($forcedir) {
+			$this->tmpdir=preg_replace('|[//\/]$|','',$forcedir);
+		}
+		// $this->tmpdir must not contains / at the end
 	}
 
 	/**

htdocs/includes/odtphp/odf.php

Braces +31 added lines, -10 removed lines

@@ -72,7 +72,10 @@ 
 		}
 
 		$md5uniqid = md5(uniqid());
-		if ($this->config['PATH_TO_TMP']) $this->tmpdir = preg_replace('|[\/]$|', '', $this->config['PATH_TO_TMP']);	// Remove last \ or /
+		if ($this->config['PATH_TO_TMP']) {
+			$this->tmpdir = preg_replace('|[\/]$|', '', $this->config['PATH_TO_TMP']);
+		}
+		// Remove last \ or /
 		$this->tmpdir .= ($this->tmpdir?'/':'').$md5uniqid;
 		$this->tmpfile = $this->tmpdir.'/'.$md5uniqid.'.odt';	// We keep .odt extension to allow OpenOffice usage during debug.
 
@@ -88,7 +91,9 @@ 
 
 		// Load zip proxy
 		$zipHandler = $this->config['ZIP_PROXY'];
-		if (!defined('PCLZIP_TEMPORARY_DIR')) define('PCLZIP_TEMPORARY_DIR', $this->tmpdir);
+		if (!defined('PCLZIP_TEMPORARY_DIR')) {
+			define('PCLZIP_TEMPORARY_DIR', $this->tmpdir);
+		}
 		include_once 'zip/'.$zipHandler.'.php';
 		if (! class_exists($this->config['ZIP_PROXY'])) {
 			throw new OdfException($this->config['ZIP_PROXY'] . ' class not found - check your php settings');
@@ -230,8 +235,12 @@ 
      */
 	private function _replaceHtmlWithOdtTag($tags, &$customStyles, &$fontDeclarations, $encode = false, $charset = '')
 	{
-		if ($customStyles == null) $customStyles = array();
-		if ($fontDeclarations == null) $fontDeclarations = array();
+		if ($customStyles == null) {
+			$customStyles = array();
+		}
+		if ($fontDeclarations == null) {
+			$fontDeclarations = array();
+		}
 
 		$odtResult = '';
 
@@ -589,7 +598,9 @@ 
 				$reg = '@\[!--\sIF\s' . $key . '\s--\](.*)(\[!--\sELSE\s' . $key . '\s--\](.*))?\[!--\sENDIF\s' . $key . '\s--\]@smU'; // U modifier = all quantifiers are non-greedy
 				preg_match_all($reg, $this->contentXml, $matches, PREG_SET_ORDER);
 				foreach ($matches as $match) { // For each match, if there is an ELSE clause, we replace the whole block by the value in the ELSE clause
-					if (!empty($match[3])) $this->contentXml = str_replace($match[0], $match[3], $this->contentXml);
+					if (!empty($match[3])) {
+						$this->contentXml = str_replace($match[0], $match[3], $this->contentXml);
+					}
 				}
 				// Cleanup the other conditional blocks (all the others where there were no ELSE clause, we can just remove them altogether)
 				$this->contentXml = preg_replace($reg, '', $this->contentXml);
@@ -602,9 +613,15 @@ 
 		}
 
 		// Static substitution
-		if ($type == 'content')	$this->contentXml = str_replace(array_keys($this->vars), array_values($this->vars), $this->contentXml);
-		if ($type == 'styles')	$this->stylesXml = str_replace(array_keys($this->vars), array_values($this->vars), $this->stylesXml);
-		if ($type == 'meta')	$this->metaXml = str_replace(array_keys($this->vars), array_values($this->vars), $this->metaXml);
+		if ($type == 'content') {
+			$this->contentXml = str_replace(array_keys($this->vars), array_values($this->vars), $this->contentXml);
+		}
+		if ($type == 'styles') {
+			$this->stylesXml = str_replace(array_keys($this->vars), array_values($this->vars), $this->stylesXml);
+		}
+		if ($type == 'meta') {
+			$this->metaXml = str_replace(array_keys($this->vars), array_values($this->vars), $this->metaXml);
+		}
 	}
 
 	/**
@@ -745,7 +762,9 @@ 
 	 */
 	public function setMetaData()
 	{
-		if (empty($this->creator)) $this->creator='';
+		if (empty($this->creator)) {
+			$this->creator='';
+		}
 
 		$this->metaXml = preg_replace('/<dc:date>.*<\/dc:date>/', '<dc:date>'.gmdate("Y-m-d\TH:i:s").'</dc:date>', $this->metaXml);
 		$this->metaXml = preg_replace('/<dc:creator>.*<\/dc:creator>/', '<dc:creator>'.htmlspecialchars($this->creator).'</dc:creator>', $this->metaXml);
@@ -813,7 +832,9 @@ 
 	{
 		global $conf;
 
-		if ( $name == "" ) $name = "temp".md5(uniqid());
+		if ( $name == "" ) {
+			$name = "temp".md5(uniqid());
+		}
 
 		dol_syslog(get_class($this).'::exportAsAttachedPDF $name='.$name, LOG_DEBUG);
 		$this->saveToDisk($name);

htdocs/includes/odtphp/Segment.php

Braces +12 added lines, -4 removed lines

@@ -88,10 +88,16 @@ 
 	{
 		// To provide debug information on line number processed
 		global $count;
-		if (empty($count)) $count=1;
-		else $count++;
+		if (empty($count)) {
+			$count=1;
+		} else {
+			$count++;
+		}
 
-		if (empty($this->savxml)) $this->savxml = $this->xml;       // Sav content of line at first line merged, so we will reuse original for next steps
+		if (empty($this->savxml)) {
+			$this->savxml = $this->xml;
+		}
+		// Sav content of line at first line merged, so we will reuse original for next steps
 		$this->xml = $this->savxml;
 		$tmpvars = $this->vars;                                     // Store into $tmpvars so we won't modify this->vars when completing data with empty values
 
@@ -123,7 +129,9 @@ 
 				$reg = '@\[!--\sIF\s' . $key . '\s--\](.*)(\[!--\sELSE\s' . $key . '\s--\](.*))?\[!--\sENDIF\s' . $key . '\s--\]@smU'; // U modifier = all quantifiers are non-greedy
 				preg_match_all($reg, $this->xml, $matches, PREG_SET_ORDER);
 				foreach ($matches as $match) { // For each match, if there is an ELSE clause, we replace the whole block by the value in the ELSE clause
-					if (!empty($match[3])) $this->xml = str_replace($match[0], $match[3], $this->xml);
+					if (!empty($match[3])) {
+						$this->xml = str_replace($match[0], $match[3], $this->xml);
+					}
 				}
 				// Cleanup the other conditional blocks (all the others where there were no ELSE clause, we can just remove them altogether)
 				$this->xml = preg_replace($reg, '', $this->xml);

htdocs/includes/sabre/sabre/event/lib/WildcardEmitterTrait.php

Braces +6 added lines, -2 removed lines

@@ -113,7 +113,9 @@ 
                 }
 
                 if ($counter > 0) {
-                    if (!$continueCallBack()) break;
+                    if (!$continueCallBack()) {
+                    	break;
+                    }
                 }
 
             }
@@ -140,9 +142,11 @@ 
             // Create a new index.
             $listeners = [];
             $listenersPriority = [];
-            if (isset($this->listeners[$eventName])) foreach ($this->listeners[$eventName] as $listener) {
+            if (isset($this->listeners[$eventName])) {
+            	foreach ($this->listeners[$eventName] as $listener) {
 
                 $listenersPriority[] = $listener[0];
+            }
                 $listeners[] = $listener[1];
 
             }

htdocs/includes/sabre/sabre/event/lib/EmitterTrait.php

Braces +3 added lines, -1 removed lines

@@ -105,7 +105,9 @@
                 }
 
                 if ($counter > 0) {
-                    if (!$continueCallBack()) break;
+                    if (!$continueCallBack()) {
+                    	break;
+                    }
                 }
 
             }

htdocs/includes/restler/framework/Luracast/Restler/Data/Text.php

Braces +6 added lines, -3 removed lines

@@ -25,8 +25,9 @@ 
 	 */
 	public static function contains($haystack, $needle, $caseSensitive = true)
 	{
-		if (empty($needle))
-			return true;
+		if (empty($needle)) {
+					return true;
+		}
 			return $caseSensitive
 			? strpos($haystack, $needle) !== false
 			: stripos($haystack, $needle) !== false;
@@ -62,7 +63,9 @@ 
 		}
 		
 		// @CHANGE LDR
-		if (!is_string($haystack)) return false;
+		if (!is_string($haystack)) {
+			return false;
+		}
 		
 		return (substr($haystack, -$length) === $needle);
 	}

htdocs/includes/restler/framework/Luracast/Restler/Data/Arr.php

Braces +7 added lines, -3 removed lines

@@ -25,9 +25,13 @@
     {
         $copy = array();
         foreach ($arr as $key => $value) {
-            if (is_array($value)) $copy[$key] = static::copy($value);
-            else if (is_object($value)) $copy[$key] = clone $value;
-            else $copy[$key] = $value;
+            if (is_array($value)) {
+            	$copy[$key] = static::copy($value);
+            } else if (is_object($value)) {
+            	$copy[$key] = clone $value;
+            } else {
+            	$copy[$key] = $value;
+            }
         }
         return $copy;
     }

htdocs/includes/restler/framework/Luracast/Restler/Data/Validator.php

Braces +3 added lines, -1 removed lines

@@ -549,7 +549,9 @@
                 case 'string' :
                 case 'password' : //password fields with string
                 case 'search' : //search field with string
-                    if (is_bool($input)) $input = $input ? 'true' : 'false';
+                    if (is_bool($input)) {
+                    	$input = $input ? 'true' : 'false';
+                    }
                     if (!is_string($input)) {
                         $error .= '. Expecting alpha numeric value';
                         break;

htdocs/includes/restler/framework/Luracast/Restler/Resources.php

Braces +33 added lines, -20 removed lines

@@ -231,8 +231,9 @@ 
                 }
                 $fLen = strlen($fullPath);
                 if ($tSlash) {
-                    if ($fLen != $tLen && !Text::beginsWith($fullPath, $target . '/'))
-                        continue;
+                    if ($fLen != $tLen && !Text::beginsWith($fullPath, $target . '/')) {
+                                            continue;
+                    }
                 } elseif ($fLen > $tLen + 1 && $fullPath[$tLen + 1] != '{' && !Text::beginsWith($fullPath, '{')) {
                     //when mapped to root exclude paths that have static parts
                     //they are listed else where under that static part name
@@ -244,8 +245,9 @@ 
                 }
                 foreach (static::$excludedPaths as $exclude) {
                     if (empty($exclude)) {
-                        if ($fullPath == $exclude)
-                            continue 2;
+                        if ($fullPath == $exclude) {
+                                                    continue 2;
+                        }
                     } elseif (Text::beginsWith($fullPath, $exclude)) {
                         continue 2;
                     }
@@ -283,8 +285,9 @@ 
                 }
                 $nickname = $this->_nickname($route);
                 $index = static::$placeFormatExtensionBeforeDynamicParts && $pos > 0 ? $pos : 0;
-                if (!empty($parts[$index]))
-                    $parts[$index] .= $this->formatString;
+                if (!empty($parts[$index])) {
+                                    $parts[$index] .= $this->formatString;
+                }
 
                 $fullPath = implode('/', $parts);
                 $description = isset(
@@ -388,8 +391,9 @@ 
         if (!$count) {
             throw new RestException(404);
         }
-        if (!is_null($r))
-            $r->models = $this->_models;
+        if (!is_null($r)) {
+                    $r->models = $this->_models;
+        }
         usort(
             $r->apis,
             function ($a, $b) {
@@ -681,8 +685,9 @@ 
             // Create default object that includes parameters to be submitted
             $defaultObject = new \StdClass();
             foreach ($this->_bodyParam['names'] as $name => $values) {
-                if (!$values->required)
-                    continue;
+                if (!$values->required) {
+                                    continue;
+                }
                 if (class_exists($values->dataType)) {
                     $myClassName = $values->dataType;
                     $defaultObject->$name = new $myClassName();
@@ -706,8 +711,9 @@ 
         }
         $properties = array();
         if (!$instance) {
-            if (!class_exists($className))
-                return;
+            if (!class_exists($className)) {
+                            return;
+            }
             $instance = new $className();
         }
         $data = get_object_vars($instance);
@@ -828,8 +834,12 @@ 
             }
             return 'array';
         }
-        if (is_bool($o)) return 'boolean';
-        if (is_numeric($o)) return is_float($o) ? 'float' : 'int';
+        if (is_bool($o)) {
+        	return 'boolean';
+        }
+        if (is_numeric($o)) {
+        	return is_float($o) ? 'float' : 'int';
+        }
         return 'string';
     }
 
@@ -879,8 +889,9 @@ 
      */
     public function index()
     {
-        if (!static::$accessControlFunction && Defaults::$accessControlFunction)
-            static::$accessControlFunction = Defaults::$accessControlFunction;
+        if (!static::$accessControlFunction && Defaults::$accessControlFunction) {
+                    static::$accessControlFunction = Defaults::$accessControlFunction;
+        }
         $version = $this->restler->getRequestedApiVersion();
         $allRoutes = Util::nestedValue(Routes::toArray(), "v$version");
         $r = $this->_resourceListing();
@@ -934,8 +945,9 @@ 
         foreach ($allRoutes as $fullPath => $routes) {
             $path = explode('/', $fullPath);
             $resource = isset($path[0]) ? $path[0] : '';
-            if ($resource == 'resources' || Text::endsWith($resource, 'index'))
-                continue;
+            if ($resource == 'resources' || Text::endsWith($resource, 'index')) {
+                            continue;
+            }
             foreach ($routes as $httpMethod => $route) {
                 if (in_array($httpMethod, static::$excludedHttpMethods)) {
                     continue;
@@ -946,8 +958,9 @@ 
 
                 foreach (static::$excludedPaths as $exclude) {
                     if (empty($exclude)) {
-                        if ($fullPath == $exclude)
-                            continue 2;
+                        if ($fullPath == $exclude) {
+                                                    continue 2;
+                        }
                     } elseif (Text::beginsWith($fullPath, $exclude)) {
                         continue 2;
                     }