EGroupware / egroupware

api/src/Etemplate/Widget/HtmlArea.php

Braces +8 added lines, -2 removed lines

@@ -46,7 +46,10 @@ 
 		$font_span = '<span style="width: 100%; display: inline; '.
 			($font?'font-family:'.$font.'; ':'').($font_size?'font-size:'.$font_size.'; ':'').
 			'">&#8203;</span>';
-		if (empty($font) && empty($font_size)) $font_span = '';
+		if (empty($font) && empty($font_size))
+		{
+			$font_span = '';
+		}
 		if($font_span)
 		{
 			$config['preference_style'] = $font_span;
@@ -78,7 +81,10 @@ 
 				$value = Api\Html\HtmLawed::purify($value, $this->attrs['validation_rules']);
 			}
 			$valid =& self::get_array($validated, $form_name, true);
-			if (true) $valid = $value;
+			if (true)
+			{
+				$valid = $value;
+			}
 		}
 	}
 }

api/src/Etemplate/Widget/ItemPicker.php

Braces +2 added lines, -1 removed lines

@@ -31,7 +31,8 @@
 	 */
 	public function __construct($xml = '')
 	{
-		if($xml) {
+		if($xml)
+		{
 			parent::__construct($xml);
 		}
 	}

api/src/Etemplate/Widget/HistoryLog.php

Braces +8 added lines, -2 removed lines

@@ -60,7 +60,10 @@ 
 				{
 					// need to use self::form_name($form_name, $key) as index into sel_options see comment in header
 					$options =& self::get_array(self::$request->sel_options, self::form_name($form_name, $key), true);
-					if (!is_array($options)) $options = array();
+					if (!is_array($options))
+					{
+						$options = array();
+					}
 					$options += $type;
 				}
 
@@ -87,7 +90,10 @@ 
 		$form_name = self::form_name($cname, $this->id, $expand);
 		$value = self::get_array($content, $form_name);
 		$valid =& self::get_array($validated, $form_name, true);
-		if (true) $valid = $value;
+		if (true)
+		{
+			$valid = $value;
+		}
 		return true;
 	}
 }

api/src/Etemplate/Widget/Toolbar.php

Braces +4 added lines, -1 removed lines

@@ -37,7 +37,10 @@
 		{
 			$value = self::get_array($content, $form_name);
 			$valid =& self::get_array($validated, $form_name, true);
-			if (true) $valid = $value;
+			if (true)
+			{
+				$valid = $value;
+			}
 		}
 	}
 }

api/src/Etemplate/Widget/Textbox.php

Braces +14 added lines, -4 removed lines

@@ -41,7 +41,10 @@ 
 		// normalize types
 		if ($this->type !== 'textbox')
 		{
-			if ($this->type == 'int') $this->type = 'integer';
+			if ($this->type == 'int')
+			{
+				$this->type = 'integer';
+			}
 
 			$this->attrs['type'] = $this->type;
 			$this->type = 'textbox';
@@ -90,7 +93,10 @@ 
 			if (!empty($value))
 			{
 				$preserv =& self::get_array(self::$request->preserv, $form_name, true);
-				if (true) $preserv = (string)$value;
+				if (true)
+				{
+					$preserv = (string)$value;
+				}
 				$value = str_repeat('*', strlen($preserv));
 			}
 		}
@@ -171,11 +177,15 @@ 
 						break;
 				}
 			}
-			elseif ($this->type == 'integer' || $this->type == 'float')	// cast int and float and check range
+			elseif ($this->type == 'integer' || $this->type == 'float')
+			{
+				// cast int and float and check range
 			{
 				if ((string)$value !== '' || $this->attrs['needed'])	// empty values are Ok if needed is not set
 				{
-					$value = $this->type == 'integer' ? (int) $value : (float) str_replace(',','.',$value);	// allow for german (and maybe other) format
+					$value = $this->type == 'integer' ? (int) $value : (float) str_replace(',','.',$value);
+			}
+			// allow for german (and maybe other) format
 
 					if (!empty($this->attrs['min']) && $value < $this->attrs['min'])
 					{

api/src/Etemplate/Widget/Image.php

Braces +4 added lines, -1 removed lines

@@ -36,7 +36,10 @@
 
 		$image = $value != '' ? $value : $this->attrs['src'];
 
-		if (is_string($image)) list($app,$img) = explode('/',$image,2);
+		if (is_string($image))
+		{
+			list($app,$img) = explode('/',$image,2);
+		}
 		if (!$app || !$img || !is_dir(EGW_SERVER_ROOT.'/'.$app) || strpos($img,'/')!==false)
 		{
 			$img = $image;

api/src/Etemplate/Widget/Checkbox.php

Braces +13 added lines, -3 removed lines

@@ -56,7 +56,11 @@ 
 		if (!$this->is_readonly($cname, $form_name))
 		{
 			$value = self::get_array($content, $form_name);
-			if (!isset($value))	return;	// value not transmitted --> nothing to validate
+			if (!isset($value))
+			{
+				return;
+			}
+			// value not transmitted --> nothing to validate
 			$valid =& self::get_array($validated, $form_name, true);
 
 			if (!$value && $this->attrs['needed'])
@@ -101,7 +105,10 @@ 
 			{
 				if ($multiple)
 				{
-					if (!isset($valid)) $valid = array();
+					if (!isset($valid))
+					{
+						$valid = array();
+					}
 					$valid[] = $selected_value;
 				}
 				else
@@ -113,7 +120,10 @@ 
 			{
 				if ($multiple)
 				{
-					if (!isset($valid)) $valid = array();
+					if (!isset($valid))
+					{
+						$valid = array();
+					}
 				}
 				elseif ($value === 'true')
 				{

api/src/Etemplate/Widget.php

Braces +68 added lines, -18 removed lines

@@ -100,7 +100,9 @@ 
 
 		// Update content?
 		if(self::$cont == null)
-			self::$cont = is_array(self::$request->content) ? self::$request->content : array();
+		{
+					self::$cont = is_array(self::$request->content) ? self::$request->content : array();
+		}
 		if($this->id && is_array(self::$cont[$this->id]))
 		{
 			$old_cont = self::$cont;
@@ -119,7 +121,8 @@ 
 		}
 
 		// Reset content as we leave
-		if($old_cont) {
+		if($old_cont)
+		{
 			self::$cont = $old_cont;
 		}
 	}
@@ -200,7 +203,10 @@ 
 					$legacy_options = explode(',', $legacy_options);
 					foreach(self::csv_split($value, count($legacy_options)) as $n => $val)
 					{
-						if ($legacy_options[$n] && (string)$val !== '') $template->attrs[$legacy_options[$n]] = $val;
+						if ($legacy_options[$n] && (string)$val !== '')
+						{
+							$template->attrs[$legacy_options[$n]] = $val;
+						}
 					}
 				}
 			}
@@ -305,7 +311,9 @@ 
 
 		$widget_registry = Api\Cache::getInstance('etemplate', 'widget_registry');
 
-		if (!$widget_registry)	// not in instance cache --> rescan from filesystem
+		if (!$widget_registry)
+		{
+			// not in instance cache --> rescan from filesystem
 		{
 			foreach(new \RecursiveIteratorIterator(new \RecursiveDirectoryIterator(__DIR__.'/Widget')) as $path)
 			{
@@ -314,6 +322,7 @@ 
 					try
 					{
 						include_once($path);
+		}
 					}
 					catch(Exception $e)
 					{
@@ -410,7 +419,10 @@ 
 
 		if(!$xml)
 		{
-			if (empty($type)) $type = 'widget';
+			if (empty($type))
+			{
+				$type = 'widget';
+			}
 			$xml = "<$type id='$id'/>";
 		}
 		//error_log(__METHOD__."('$type', ..., '$id') using $class_name");
@@ -509,7 +521,10 @@ 
 					$params[$index] = (array)$params[$index];
 				}
 			}
-			if($call) call_user_func_array(array($this, $method_name), $params);
+			if($call)
+			{
+				call_user_func_array(array($this, $method_name), $params);
+			}
 		}
 		foreach($this->children as $child)
 		{
@@ -565,7 +580,10 @@ 
 		$val = self::expand_name($value, $expand['c'], $expand['row'], $expand['c_'], $expand['row_'], $expand['cont']);
 		$check_val = self::expand_name($check, $expand['c'], $expand['row'], $expand['c_'], $expand['row_'], $expand['cont']);
 		$result = count($vals) == 1 ? $val != '' : ($check_val[0] == '/' ? preg_match($check_val,$val) : $val == $check_val);
-		if ($not) $result = !$result;
+		if ($not)
+		{
+			$result = !$result;
+		}
 
 		//error_log(__METHOD__."('".($not?'!':'')."$disabled' = '$val' ".(count($vals) == 1 ? '' : ($not?'!':'=')."= '$check_val'")." = ".($result?'True':'False'));
 		return $result;
@@ -615,7 +633,10 @@ 
 			{
 				$cont = array();
 			}
-			if (!is_numeric($c)) $c = self::chrs2num($c);
+			if (!is_numeric($c))
+			{
+				$c = self::chrs2num($c);
+			}
 			$col = self::num2chrs($c-1);	// $c-1 to get: 0:'@', 1:'A', ...
 			$col_ = self::num2chrs($c_-1);
 			$row_cont = $cont[$row];
@@ -698,7 +719,8 @@ 
 	 */
 	public function __clone()
 	{
-		foreach($this->children as $child_num => $child) {
+		foreach($this->children as $child_num => $child)
+		{
 			$this->children[$child_num] = clone $child;
 		}
 	}
@@ -712,7 +734,10 @@ 
 	public function toXml($indent='')
 	{
 		echo "$indent<$this->type";
-		if ($this->id) echo ' id="'.htmlspecialchars($this->id).'"';
+		if ($this->id)
+		{
+			echo ' id="'.htmlspecialchars($this->id).'"';
+		}
 		foreach($this->attrs as $name => $value)
 		{
 			if ($name == 'options' && $this->legacy_options && (!is_array($this->legacy_options) ||
@@ -792,7 +817,11 @@ 
 		{
 			throw new Api\Exception\AssertionFailed(__METHOD__."(\$arr,'$_idx',$reference_into,$skip_empty) \$arr is no array!");
 		}
-		if (is_object($_idx)) return false;	// given an error in php5.2
+		if (is_object($_idx))
+		{
+			return false;
+		}
+		// given an error in php5.2
 
 		// Make sure none of these are left
 		$idx = str_replace(array('&#x5B;','&#x5D;'), array('[',']'), $_idx);
@@ -815,7 +844,10 @@ 
 				//if ($reference_into) error_log(__METHOD__."(".(strlen($s=array2string($arr))>512?substr($s,0,512).'...':$s).", '$idx', ".array2string($reference_into).", ".array2string($skip_empty).") ".function_backtrace());
 				return null;
 			}
-			if($skip_empty && (!is_array($pos) || !isset($pos[$idx]))) return null;
+			if($skip_empty && (!is_array($pos) || !isset($pos[$idx])))
+			{
+				return null;
+			}
 			$pos = &$pos[$idx];
 		}
 		return $pos;
@@ -834,7 +866,10 @@ 
 	static function set_array(&$_arr, $_idx, $_value)
 	{
 		$ref =& self::get_array($_arr, $_idx, true);
-		if (true) $ref = $_value;
+		if (true)
+		{
+			$ref = $_value;
+		}
 	}
 
 	/**
@@ -886,7 +921,10 @@ 
 		// not yet used: if (is_null($cname)) $cname = self::$name_vars;
 		error_log(__METHOD__."('$name','$error','$cname') ".function_backtrace());
 
-		if ($cname) $name = self::form_name($cname,$name);
+		if ($cname)
+		{
+			$name = self::form_name($cname,$name);
+		}
 
 		if ($error === false)
 		{
@@ -913,11 +951,17 @@ 
 	{
 		// not yet used: if (is_null($cname)) $cname = self::$name_vars;
 		//echo "<p>uietemplate::validation_errors('$ignore_validation','$cname') validation_error="; _debug_array(self::$validation_errors);
-		if (!$ignore_validation) return count(self::$validation_errors) > 0;
+		if (!$ignore_validation)
+		{
+			return count(self::$validation_errors) > 0;
+		}
 
 		foreach(array_values(self::$validation_errors) as $name)
 		{
-			if ($cname) $name = preg_replace('/^'.$cname.'\[([^\]]+)\](.*)$/','\\1\\2',$name);
+			if ($cname)
+			{
+				$name = preg_replace('/^'.$cname.'\[([^\]]+)\](.*)$/','\\1\\2',$name);
+			}
 
 			// treat $ignoare_validation only as regular expression, if it starts with a slash
 			if ($ignore_validation[0] == '/' && !preg_match($ignore_validation,$name) ||
@@ -974,8 +1018,14 @@ 
 			self::$request->unset_to_process('');
 			//error_log(__METHOD__."('$name', '$attr', ...) ".function_backtrace());
 		}
-		if (isset($this)) $this->attrs[$attr] = $val;
-		if (!is_null($val)) $ref = $val;
+		if (isset($this))
+		{
+			$this->attrs[$attr] = $val;
+		}
+		if (!is_null($val))
+		{
+			$ref = $val;
+		}
 
 		//error_log(__METHOD__."('$name', '$attr', ".array2string($val).')');
 		return $ref;

api/src/Etemplate/KeyManager.php

Braces +2 added lines, -1 removed lines

@@ -101,7 +101,8 @@
 		}
 
 		// Special keys
-		switch ($keyCode) {
+		switch ($keyCode)
+		{
 			case self::BACKSPACE:
 				return lang("Back");
 			case self::TAB: