EGroupware / egroupware

api/src/Etemplate/Widget/Tabbox.php

Spacing +14 added lines, -14 removed lines

@@ -42,30 +42,30 @@ 
 	 * @param array $params =array('') parameter(s) first parameter has to be the cname, second $expand!
 	 * @param boolean $respect_disabled =false false (default): ignore disabled, true: method is NOT run for disabled widgets AND their children
 	 */
-	public function run($method_name, $params=array(''), $respect_disabled=false)
+	public function run($method_name, $params = array(''), $respect_disabled = false)
 	{
 		$form_name = self::form_name($params[0], $this->id, $params[1]);
 
 		// Make sure additional tabs are processed for any method
-		if (!($tabs =& self::getElementAttribute($form_name, 'tabs')))
+		if (!($tabs = & self::getElementAttribute($form_name, 'tabs')))
 		{
 			$tabs = $this->attrs['tabs'];
 		}
-		if($tabs && !$this->tabs_attr_evaluated)
+		if ($tabs && !$this->tabs_attr_evaluated)
 		{
-			$this->tabs_attr_evaluated = true;	// we must not evaluate tabs attribte more then once!
+			$this->tabs_attr_evaluated = true; // we must not evaluate tabs attribte more then once!
 
 			// add_tabs toggles replacing or adding to existing tabs
-			if(!$this->attrs['add_tabs'])
+			if (!$this->attrs['add_tabs'])
 			{
 				$this->children[1]->children = array();
 			}
 
 			//$this->tabs = array();
-			foreach($tabs as &$tab)
+			foreach ($tabs as &$tab)
 			{
-				$template= clone Template::instance($tab['template']);
-				if($tab['id']) $template->attrs['content'] = $tab['id'];
+				$template = clone Template::instance($tab['template']);
+				if ($tab['id']) $template->attrs['content'] = $tab['id'];
 				$this->children[1]->children[] = $template;
 				$tab['url'] = Template::rel2url($template->rel_path);
 				//$this->tabs[] = $tab;
@@ -78,13 +78,13 @@ 
 
 		// Check for disabled tabs set via readonly, and set them as disabled
 		$readonlys = self::get_array(self::$request->readonlys, $form_name);
-		if($respect_disabled && $readonlys)
+		if ($respect_disabled && $readonlys)
 		{
-			foreach($this->children[1]->children as $tab)
+			foreach ($this->children[1]->children as $tab)
 			{
-				$parts = explode('.',$tab->template ? $tab->template : $tab->id);
+				$parts = explode('.', $tab->template ? $tab->template : $tab->id);
 				$ro_id = array_pop($parts);
-				if($readonlys[$ro_id])
+				if ($readonlys[$ro_id])
 				{
 					$tab->attrs['disabled'] = $readonlys[$ro_id];
 				}
@@ -104,14 +104,14 @@ 
 	 * @param array &$validated=array() validated content
 	 * @param array $expand=array values for keys 'c', 'row', 'c_', 'row_', 'cont'
 	 */
-	public function validate($cname, array $expand, array $content, &$validated=array())
+	public function validate($cname, array $expand, array $content, &$validated = array())
 	{
 		$form_name = self::form_name($cname, $this->id, $expand);
 
 		if (!empty($form_name))
 		{
 			$value = self::get_array($content, $form_name);
-			$valid =& self::get_array($validated, $form_name, true);
+			$valid = & self::get_array($validated, $form_name, true);
 			if (true) $valid = $value;
 		}
 	}

Doc Comments -1 removed lines

@@ -52,7 +52,6 @@
 	 * @param array $expand values for keys 'c', 'row', 'c_', 'row_', 'cont'
 	 * @param array $content
 	 * @param array &$validated=array() validated content
-	 * @param array $expand=array values for keys 'c', 'row', 'c_', 'row_', 'cont'
 	 */
 	public function validate($cname, array $expand, array $content, &$validated=array())
 	{

Braces +8 added lines, -2 removed lines

@@ -65,7 +65,10 @@ 
 			foreach($tabs as &$tab)
 			{
 				$template= clone Template::instance($tab['template']);
-				if($tab['id']) $template->attrs['content'] = $tab['id'];
+				if($tab['id'])
+				{
+					$template->attrs['content'] = $tab['id'];
+				}
 				$this->children[1]->children[] = $template;
 				$tab['url'] = Template::rel2url($template->rel_path);
 				//$this->tabs[] = $tab;
@@ -112,7 +115,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/Transformer.php

Spacing +21 added lines, -21 removed lines

@@ -73,7 +73,7 @@ 
 	 *
 	 * @param string $cname
 	 */
-	public function beforeSendToClient($cname, array $expand=array())
+	public function beforeSendToClient($cname, array $expand = array())
 	{
 		$attrs = $this->attrs;
 		$form_name = self::form_name($cname, $this->id);
@@ -82,14 +82,14 @@ 
 			error_log(__METHOD__."() $this has no id!");
 			return;
 		}
-		$attrs['value'] = $value =& self::get_array(self::$request->content, $form_name, false, true);
+		$attrs['value'] = $value = & self::get_array(self::$request->content, $form_name, false, true);
 		$attrs['type'] = $this->type;
 		$attrs['id'] = $this->id;
 
 		$unmodified = $attrs;
 
 		// run the transformation
-		foreach(static::$transformation as $filter => $data)
+		foreach (static::$transformation as $filter => $data)
 		{
 			$this->action($filter, $data, $attrs);
 		}
@@ -97,14 +97,14 @@ 
 		//echo $this; _debug_array($unmodified); _debug_array($attrs); _debug_array(array_diff_assoc($attrs, $unmodified));
 		// compute the difference and send it to the client as modifications
 		$type_changed = false;
-		foreach(array_diff_assoc($attrs, $unmodified) as $attr => $val)
+		foreach (array_diff_assoc($attrs, $unmodified) as $attr => $val)
 		{
-			switch($attr)
+			switch ($attr)
 			{
 				case 'value':
 					if ($val != $value)
 					{
-						$value = $val;	// $value is reference to self::$request->content
+						$value = $val; // $value is reference to self::$request->content
 					}
 					break;
 				case 'sel_options':
@@ -112,15 +112,15 @@ 
 					break;
 				case 'type':	// not an attribute in etemplate2
 					$type_changed = true;
-					if($val == 'template')
+					if ($val == 'template')
 					{
 						// If the widget has been transformed into a template, we
 						// also need to try and instanciate & parse the template too
 						$transformed_template = Template::instance($attrs['template']);
-						if($transformed_template)
+						if ($transformed_template)
 						{
 							$this->expand_widget($transformed_template, $expand);
-							$transformed_template->run('beforeSendToClient',array($cname,$expand));
+							$transformed_template->run('beforeSendToClient', array($cname, $expand));
 						}
 						$type_changed = false;
 					}
@@ -129,14 +129,14 @@ 
 					break;
 			}
 		}
-		if($type_changed)
+		if ($type_changed)
 		{
 			// Run the new widget type's beforeSendToClient
-			$expanded_child = self::factory($attrs['type'], false,$this->id);
+			$expanded_child = self::factory($attrs['type'], false, $this->id);
 			$expanded_child->id = $this->id;
 			$expanded_child->type = $attrs['type'];
 			$expanded_child->attrs = $attrs;
-			$expanded_child->run('beforeSendToClient',array($cname,$expand));
+			$expanded_child->run('beforeSendToClient', array($cname, $expand));
 		}
 	}
 
@@ -158,7 +158,7 @@ 
 			if (strpos($action, '@') !== false)
 			{
 				$replace = array();
-				foreach($attrs as $a => $v)
+				foreach ($attrs as $a => $v)
 				{
 					if (is_scalar($v) || is_null($v)) $replace['@'.$a] = $v;
 				}
@@ -166,14 +166,14 @@ 
 				// now replace with non-scalar value, eg. if values is an array: "@value", "@value[key] or "@value[@key]"
 				if (($a = strstr($action, '@')))
 				{
-					$action = self::get_array($attrs, substr($a,1));
+					$action = self::get_array($attrs, substr($a, 1));
 				}
 			}
 			$attrs[$attr] = $action;
 			if (self::DEBUG) error_log(__METHOD__."('$attr', ".array2string($action).") attrs['$attr'] = ".array2string($action).', attrs='.array2string($attrs));
 		}
 		// action is a serverside callback
-		elseif(is_array($action) && isset($action['__callback__']))
+		elseif (is_array($action) && isset($action['__callback__']))
 		{
 			if (!is_string(($callback = $action['__callback__'])))
 			{
@@ -183,7 +183,7 @@ 
 			{
 				$attrs[$attr] = $this->$callback($attrs[$attr], $attrs);
 			}
-			elseif(count(explode('.', $callback)) == 3)
+			elseif (count(explode('.', $callback)) == 3)
 			{
 				$attrs[$attr] = ExecMethod($callback, $attrs[$attr], $attrs);
 			}
@@ -197,29 +197,29 @@ 
 			}
 		}
 		// action is a clientside callback
-		elseif(is_array($action) && isset($action['__js__']))
+		elseif (is_array($action) && isset($action['__js__']))
 		{
 			// nothing to do here
 		}
 		// TODO: Might be a better way to handle when value to be set is an array
-		elseif(is_array($action) && $attr == 'sel_options')
+		elseif (is_array($action) && $attr == 'sel_options')
 		{
 			$attrs[$attr] = $action;
 		}
 		// action is a switch --> check cases
-		elseif(is_array($action))
+		elseif (is_array($action))
 		{
 			// case matches --> run all actions
 			if (isset($action[$attrs[$attr]]) || !isset($action[$attrs[$attr]]) && isset($action['__default__']))
 			{
 				$actions = isset($action[$attrs[$attr]]) ? $action[$attrs[$attr]] : $action['__default__'];
-				if(!is_array($actions))
+				if (!is_array($actions))
 				{
 					$attrs[$attr] = $actions;
 					$actions = array($attr => $actions);
 				}
 				if (self::DEBUG) error_log(__METHOD__."(attr='$attr', action=".array2string($action).") attrs['$attr']=='{$attrs[$attr]}' --> running actions");
-				foreach($actions as $attr => $action)
+				foreach ($actions as $attr => $action)
 				{
 					$this->action($attr, $action, $attrs);
 				}

Braces +16 added lines, -4 removed lines

@@ -150,7 +150,10 @@ 
 	 */
 	protected function action($attr, $action, array &$attrs)
 	{
-		if (self::DEBUG) error_log(__METHOD__."('$attr', ".array2string($action).')');
+		if (self::DEBUG)
+		{
+			error_log(__METHOD__."('$attr', ".array2string($action).')');
+		}
 		// action is an assignment
 		if (is_scalar($action) || is_null($action))
 		{
@@ -160,7 +163,10 @@ 
 				$replace = array();
 				foreach($attrs as $a => $v)
 				{
-					if (is_scalar($v) || is_null($v)) $replace['@'.$a] = $v;
+					if (is_scalar($v) || is_null($v))
+					{
+						$replace['@'.$a] = $v;
+					}
 				}
 				$action = strtr($action, $replace);
 				// now replace with non-scalar value, eg. if values is an array: "@value", "@value[key] or "@value[@key]"
@@ -170,7 +176,10 @@ 
 				}
 			}
 			$attrs[$attr] = $action;
-			if (self::DEBUG) error_log(__METHOD__."('$attr', ".array2string($action).") attrs['$attr'] = ".array2string($action).', attrs='.array2string($attrs));
+			if (self::DEBUG)
+			{
+				error_log(__METHOD__."('$attr', ".array2string($action).") attrs['$attr'] = ".array2string($action).', attrs='.array2string($attrs));
+			}
 		}
 		// action is a serverside callback
 		elseif(is_array($action) && isset($action['__callback__']))
@@ -218,7 +227,10 @@ 
 					$attrs[$attr] = $actions;
 					$actions = array($attr => $actions);
 				}
-				if (self::DEBUG) error_log(__METHOD__."(attr='$attr', action=".array2string($action).") attrs['$attr']=='{$attrs[$attr]}' --> running actions");
+				if (self::DEBUG)
+				{
+					error_log(__METHOD__."(attr='$attr', action=".array2string($action).") attrs['$attr']=='{$attrs[$attr]}' --> running actions");
+				}
 				foreach($actions as $attr => $action)
 				{
 					$this->action($attr, $action, $attrs);

api/src/Etemplate/Widget/Customfields.php

Spacing +60 added lines, -60 removed lines

@@ -42,12 +42,12 @@ 
 		'date'     => 'Date',
 		'date-time'=> 'Date+Time',
 		'select-account' => 'Select account',
-		'button'   => 'Button',         // button to execute javascript
+		'button'   => 'Button', // button to execute javascript
 		'url'      => 'Url',
 		'url-email'=> 'EMail',
 		'url-phone'=> 'Phone number',
 		'htmlarea' => 'Formatted Text (HTML)',
-		'link-entry' => 'Select entry',         // should be last type, as the individual apps get added behind
+		'link-entry' => 'Select entry', // should be last type, as the individual apps get added behind
 	);
 
 	/**
@@ -86,24 +86,24 @@ 
 	 * @param string $cname
 	 * @param array $expand values for keys 'c', 'row', 'c_', 'row_', 'cont'
 	 */
-	public function beforeSendToClient($cname, array $expand=null)
+	public function beforeSendToClient($cname, array $expand = null)
 	{
 		// No name, no way to get parameters client-side.
-		if(!$this->id) $this->id = self::GLOBAL_ID;
+		if (!$this->id) $this->id = self::GLOBAL_ID;
 
 		$form_name = self::form_name($cname, $this->id, $expand);
 
 		// Store properties at top level, so all customfield widgets can share
-		if($this->attrs['app'])
+		if ($this->attrs['app'])
 		{
 			$app = $this->attrs['app'];
 		}
 		else
 		{
-			$app =& $this->getElementAttribute(self::GLOBAL_VALS, 'app');
-			if($this->getElementAttribute($form_name, 'app'))
+			$app = & $this->getElementAttribute(self::GLOBAL_VALS, 'app');
+			if ($this->getElementAttribute($form_name, 'app'))
 			{
-				$app =& $this->getElementAttribute($form_name, 'app');
+				$app = & $this->getElementAttribute($form_name, 'app');
 			}
 			else
 			{
@@ -112,21 +112,21 @@ 
 			}
 		}
 
-		if($this->getElementAttribute($form_name, 'customfields'))
+		if ($this->getElementAttribute($form_name, 'customfields'))
 		{
-			$customfields =& $this->getElementAttribute($form_name, 'customfields');
+			$customfields = & $this->getElementAttribute($form_name, 'customfields');
 		}
-		elseif($app)
+		elseif ($app)
 		{
 			// Checking creates it even if it wasn't there
 			unset(self::$request->modifications[$form_name]['customfields']);
-			$customfields =& $this->getElementAttribute(self::GLOBAL_VALS, 'customfields');
+			$customfields = & $this->getElementAttribute(self::GLOBAL_VALS, 'customfields');
 		}
 
-		if(!$app)
+		if (!$app)
 		{
-			$app =& $this->setElementAttribute(self::GLOBAL_VALS, 'app', $GLOBALS['egw_info']['flags']['currentapp']);
-			$customfields =& $this->setElementAttribute(self::GLOBAL_VALS, 'customfields', Api\Storage\Customfields::get($app));
+			$app = & $this->setElementAttribute(self::GLOBAL_VALS, 'app', $GLOBALS['egw_info']['flags']['currentapp']);
+			$customfields = & $this->setElementAttribute(self::GLOBAL_VALS, 'customfields', Api\Storage\Customfields::get($app));
 		}
 
 		// if we are in the etemplate editor or the app has no cf's, load the cf's from the app the tpl belongs too
@@ -134,29 +134,29 @@ 
 			($GLOBALS['egw_info']['flags']['currentapp'] == 'etemplate' || !$this->attrs['customfields']) || !isset($customfields))
 		{
 			// app changed
-			$customfields =& Api\Storage\Customfields::get($app);
+			$customfields = & Api\Storage\Customfields::get($app);
 		}
 		// Filter fields
-		if($this->attrs['field-names'])
+		if ($this->attrs['field-names'])
 		{
 			$fields_name = explode(',', $this->attrs['field-names']);
-			foreach($fields_name as &$f)
+			foreach ($fields_name as &$f)
 			{
 				if ($f[0] == "!")
 				{
-					$f= substr($f,1);
-					$negate_fields[]= $f;
+					$f = substr($f, 1);
+					$negate_fields[] = $f;
 				}
-				$field_filters []= $f;
+				$field_filters [] = $f;
 			}
 		}
 
 		$fields = $customfields;
 
-		$use_private = self::expand_name($this->attrs['use-private'],0,0,'','',self::$cont);
-		$this->attrs['sub-type'] = self::expand_name($this->attrs['sub-type'],0,0,'','',self::$cont);
+		$use_private = self::expand_name($this->attrs['use-private'], 0, 0, '', '', self::$cont);
+		$this->attrs['sub-type'] = self::expand_name($this->attrs['sub-type'], 0, 0, '', '', self::$cont);
 
-		foreach((array)$fields as $key => $field)
+		foreach ((array)$fields as $key => $field)
 		{
 			// remove private or non-private cf's, if only one kind should be displayed
 			if ((string)$use_private !== '' && (boolean)$field['private'] != (boolean)$use_private)
@@ -165,36 +165,36 @@ 
 			}
 
 			// Remove filtered fields
-			if($field_filters && in_array($key, $negate_fields) && in_array($key, $field_filters))
+			if ($field_filters && in_array($key, $negate_fields) && in_array($key, $field_filters))
 			{
 				unset($fields[$key]);
 			}
 		}
 		// check if name refers to a single custom field --> show only that
 		$matches = null;
-		if (($pos=strpos($form_name,self::$prefix)) !== false && // allow the prefixed name to be an array index too
-			preg_match($preg = '/'.self::$prefix.'([^\]]+)/',$form_name,$matches) && isset($fields[$name=$matches[1]]))
+		if (($pos = strpos($form_name, self::$prefix)) !== false && // allow the prefixed name to be an array index too
+			preg_match($preg = '/'.self::$prefix.'([^\]]+)/', $form_name, $matches) && isset($fields[$name = $matches[1]]))
 		{
 			$fields = array($name => $fields[$name]);
 			$value = array(self::$prefix.$name => $value);
 			$form_name = self::$prefix.$name;
 		}
 
-		if(!is_array($fields)) $fields = array();
-		switch($type = $this->type)
+		if (!is_array($fields)) $fields = array();
+		switch ($type = $this->type)
 		{
 			case 'customfields-types':
-				foreach(self::$cf_types as $lname => $label)
+				foreach (self::$cf_types as $lname => $label)
 				{
 					$sel_options[$lname] = lang($label);
-					$fields_with_vals[]=$lname;
+					$fields_with_vals[] = $lname;
 				}
 				$link_types = array_intersect_key(Api\Link::app_list('query'), Api\Link::app_list('title'));
 				// Explicitly add in filemanager, which does not support query or title
 				$link_types['filemanager'] = lang('filemanager');
 
 				ksort($link_types);
-				foreach($link_types as $lname => $label)
+				foreach ($link_types as $lname => $label)
 				{
 					$sel_options[$lname] = '- '.$label;
 				}
@@ -202,32 +202,32 @@ 
 				self::$transformation['type'][$type]['no_lang'] = true;
 				return parent::beforeSendToClient($cname, $expand);
 			case 'customfields-list':
-				foreach(array_reverse($fields) as $lname => $field)
+				foreach (array_reverse($fields) as $lname => $field)
 				{
 					if (!empty($this->attrs['sub-type']) && !empty($field['type2']) &&
-						strpos(','.$field['type2'].',',','.$field['type2'].',') === false) continue;    // not for our content type//
+						strpos(','.$field['type2'].',', ','.$field['type2'].',') === false) continue; // not for our content type//
 					if (isset($value[self::$prefix.$lname]) && $value[self::$prefix.$lname] !== '') //break;
 					{
-						$fields_with_vals[]=$lname;
+						$fields_with_vals[] = $lname;
 					}
 					//$stop_at_field = $name;
 				}
 				break;
 			default:
-				foreach(array_reverse($fields) as $lname => $field)
+				foreach (array_reverse($fields) as $lname => $field)
 				{
-					$fields_with_vals[]=$lname;
+					$fields_with_vals[] = $lname;
 				}
 		}
 		// need to encode values/select-options to keep their order
-		foreach($customfields as &$data)
+		foreach ($customfields as &$data)
 		{
 			if (!empty($data['values']))
 			{
 				Select::fix_encoded_options($data['values']);
 			}
 		}
-		if($fields != $customfields)
+		if ($fields != $customfields)
 		{
 			// This widget has different settings from global
 			$this->setElementAttribute($form_name, 'customfields', $fields);
@@ -239,14 +239,14 @@ 
 		parent::beforeSendToClient($cname, $expand);
 
 		// Re-format date custom fields from Y-m-d
-		$field_settings =& self::get_array(self::$request->modifications, "{$this->id}[customfields]",true);
+		$field_settings = & self::get_array(self::$request->modifications, "{$this->id}[customfields]", true);
 		if (true) $field_settings = array();
 		$link_types = Api\Link::app_list();
-		foreach($fields as $fname => $field)
+		foreach ($fields as $fname => $field)
 		{
 			// Run beforeSendToClient for each field
 			$widget = $this->_widget($fname, $field);
-			if(method_exists($widget, 'beforeSendToClient'))
+			if (method_exists($widget, 'beforeSendToClient'))
 			{
 				$widget->beforeSendToClient($this->id == self::GLOBAL_ID ? '' : $this->id, $expand);
 			}
@@ -263,13 +263,13 @@ 
 	protected function _widget($fname, array $field)
 	{
 		static $link_types = null;
-		if (!isset($link_types)) $link_types = Api\Link::app_list ();
+		if (!isset($link_types)) $link_types = Api\Link::app_list();
 
 		$type = $field['type'];
 		// Link-tos needs to change from appname to link-to
-		if($link_types[$field['type']])
+		if ($link_types[$field['type']])
 		{
-			if($type == 'filemanager')
+			if ($type == 'filemanager')
 			{
 				$type = 'vfs-upload';
 			}
@@ -283,18 +283,18 @@ 
 		$widget->attrs['type'] = $type;
 
 		// some type-specific (default) attributes
-		switch($type)
+		switch ($type)
 		{
 			case 'date':
 			case 'date-time':
 				$widget->attrs['dataformat'] = $type == 'date' ? 'Y-m-d' : 'Y-m-d H:i:s';
-				if($field['values']['min']) $widget->attrs['min'] = $field['values']['min'];
-				if($field['values']['max']) $widget->attrs['min'] = $field['values']['max'];
+				if ($field['values']['min']) $widget->attrs['min'] = $field['values']['min'];
+				if ($field['values']['max']) $widget->attrs['min'] = $field['values']['max'];
 				break;
 
 			case 'vfs-upload':
-				$widget->attrs['path'] = $field['app'] . ':' .
-					self::expand_name('$cont['.Api\Link::get_registry($field['app'],'view_id').']',0,0,0,0,self::$request->content).
+				$widget->attrs['path'] = $field['app'].':'.
+					self::expand_name('$cont['.Api\Link::get_registry($field['app'], 'view_id').']', 0, 0, 0, 0, self::$request->content).
 					':'.$field['label'];
 				break;
 
@@ -353,7 +353,7 @@ 
 	 * @param array $content
 	 * @param array &$validated=array() validated content
 	 */
-	public function validate($cname, array $expand, array $content, &$validated=array())
+	public function validate($cname, array $expand, array $content, &$validated = array())
 	{
 		if ($this->id)
 		{
@@ -369,14 +369,14 @@ 
 		// if we have no id / use self::GLOBAL_ID, we have to set $value_in in global namespace for regular widgets validation to find
 		if (!$this->id) $content = array_merge($content, $value_in);
 		//error_log(__METHOD__."($cname, ...) form_name=$form_name, use-private={$this->attrs['use-private']}, value_in=".array2string($value_in));
-		$customfields =& $this->getElementAttribute(self::GLOBAL_VALS, 'customfields');
-		if(is_array($value_in))
+		$customfields = & $this->getElementAttribute(self::GLOBAL_VALS, 'customfields');
+		if (is_array($value_in))
 		{
-			foreach($value_in as $field => $value)
+			foreach ($value_in as $field => $value)
 			{
-				$field_settings = $customfields[$fname=substr($field,1)];
+				$field_settings = $customfields[$fname = substr($field, 1)];
 
-				if ((string)$this->attrs['use-private'] !== '' &&	// are only (non-)private fields requested
+				if ((string)$this->attrs['use-private'] !== '' && // are only (non-)private fields requested
 					(boolean)$field_settings['private'] != ($this->attrs['use-private'] != '0'))
 				{
 					continue;
@@ -396,10 +396,10 @@ 
 				$widget->validate($form_name != self::GLOBAL_ID ? $form_name : $cname, $expand, $content, $validated);
 				if ($field_settings['needed'] && (is_array($value) ? !$value : (string)$value === ''))
 				{
-					self::set_validation_error($field,lang('Field must not be empty !!!'),'');
+					self::set_validation_error($field, lang('Field must not be empty !!!'), '');
 				}
-				$field_name = $this->id[0] == self::$prefix && $customfields[substr($this->id,1)] ? $this->id : self::form_name($form_name != self::GLOBAL_ID ? $form_name : $cname, $field);
-				$valid =& self::get_array($validated, $field_name, true);
+				$field_name = $this->id[0] == self::$prefix && $customfields[substr($this->id, 1)] ? $this->id : self::form_name($form_name != self::GLOBAL_ID ? $form_name : $cname, $field);
+				$valid = & self::get_array($validated, $field_name, true);
 
 				if (is_array($valid)) $valid = implode(',', $valid);
 				// NULL is valid for most fields, but not custom fields due to backend handling
@@ -411,7 +411,7 @@ 
 		elseif ($this->type == 'customfields-types')
 		{
 			// Transformation doesn't handle validation
-			$valid =& self::get_array($validated, $this->id ? $form_name : $field, true);
+			$valid = & self::get_array($validated, $this->id ? $form_name : $field, true);
 			if (true) $valid = $value_in;
 			//error_log(__METHOD__."() $form_name $field: ".array2string($value).' --> '.array2string($value));
 		}

Braces +57 added lines, -14 removed lines

@@ -89,7 +89,10 @@ 
 	public function beforeSendToClient($cname, array $expand=null)
 	{
 		// No name, no way to get parameters client-side.
-		if(!$this->id) $this->id = self::GLOBAL_ID;
+		if(!$this->id)
+		{
+			$this->id = self::GLOBAL_ID;
+		}
 
 		$form_name = self::form_name($cname, $this->id, $expand);
 
@@ -180,7 +183,10 @@ 
 			$form_name = self::$prefix.$name;
 		}
 
-		if(!is_array($fields)) $fields = array();
+		if(!is_array($fields))
+		{
+			$fields = array();
+		}
 		switch($type = $this->type)
 		{
 			case 'customfields-types':
@@ -205,11 +211,18 @@ 
 				foreach(array_reverse($fields) as $lname => $field)
 				{
 					if (!empty($this->attrs['sub-type']) && !empty($field['type2']) &&
-						strpos(','.$field['type2'].',',','.$field['type2'].',') === false) continue;    // not for our content type//
-					if (isset($value[self::$prefix.$lname]) && $value[self::$prefix.$lname] !== '') //break;
+						strpos(','.$field['type2'].',',','.$field['type2'].',') === false)
+					{
+						continue;
+					}
+					// not for our content type//
+					if (isset($value[self::$prefix.$lname]) && $value[self::$prefix.$lname] !== '')
+					{
+						//break;
 					{
 						$fields_with_vals[]=$lname;
 					}
+					}
 					//$stop_at_field = $name;
 				}
 				break;
@@ -240,7 +253,10 @@ 
 
 		// Re-format date custom fields from Y-m-d
 		$field_settings =& self::get_array(self::$request->modifications, "{$this->id}[customfields]",true);
-		if (true) $field_settings = array();
+		if (true)
+		{
+			$field_settings = array();
+		}
 		$link_types = Api\Link::app_list();
 		foreach($fields as $fname => $field)
 		{
@@ -263,7 +279,10 @@ 
 	protected function _widget($fname, array $field)
 	{
 		static $link_types = null;
-		if (!isset($link_types)) $link_types = Api\Link::app_list ();
+		if (!isset($link_types))
+		{
+			$link_types = Api\Link::app_list ();
+		}
 
 		$type = $field['type'];
 		// Link-tos needs to change from appname to link-to
@@ -288,8 +307,14 @@ 
 			case 'date':
 			case 'date-time':
 				$widget->attrs['dataformat'] = $type == 'date' ? 'Y-m-d' : 'Y-m-d H:i:s';
-				if($field['values']['min']) $widget->attrs['min'] = $field['values']['min'];
-				if($field['values']['max']) $widget->attrs['min'] = $field['values']['max'];
+				if($field['values']['min'])
+				{
+					$widget->attrs['min'] = $field['values']['min'];
+				}
+				if($field['values']['max'])
+				{
+					$widget->attrs['min'] = $field['values']['max'];
+				}
 				break;
 
 			case 'vfs-upload':
@@ -306,7 +331,10 @@ 
 				break;
 
 			default:
-				if (substr($type, 0, 7) !== 'select-' && $type != 'ajax_select') break;
+				if (substr($type, 0, 7) !== 'select-' && $type != 'ajax_select')
+				{
+					break;
+				}
 				// fall-through for all select-* widgets
 			case 'select':
 				$this->attrs['multiple'] = $field['rows'] > 1;
@@ -367,7 +395,10 @@ 
 		$all_readonly = $this->is_readonly($cname, $form_name);
 		$value_in = self::get_array($content, $form_name);
 		// if we have no id / use self::GLOBAL_ID, we have to set $value_in in global namespace for regular widgets validation to find
-		if (!$this->id) $content = array_merge($content, $value_in);
+		if (!$this->id)
+		{
+			$content = array_merge($content, $value_in);
+		}
 		//error_log(__METHOD__."($cname, ...) form_name=$form_name, use-private={$this->attrs['use-private']}, value_in=".array2string($value_in));
 		$customfields =& $this->getElementAttribute(self::GLOBAL_VALS, 'customfields');
 		if(is_array($value_in))
@@ -392,7 +423,10 @@ 
 				// run validation method of widget implementing this custom field
 				$widget = $this->_widget($fname, $field_settings);
 				// widget has no validate method, eg. is only displaying stuff --> nothing to validate
-				if (!method_exists($widget, 'validate')) continue;
+				if (!method_exists($widget, 'validate'))
+				{
+					continue;
+				}
 				$widget->validate($form_name != self::GLOBAL_ID ? $form_name : $cname, $expand, $content, $validated);
 				if ($field_settings['needed'] && (is_array($value) ? !$value : (string)$value === ''))
 				{
@@ -401,10 +435,16 @@ 
 				$field_name = $this->id[0] == self::$prefix && $customfields[substr($this->id,1)] ? $this->id : self::form_name($form_name != self::GLOBAL_ID ? $form_name : $cname, $field);
 				$valid =& self::get_array($validated, $field_name, true);
 
-				if (is_array($valid)) $valid = implode(',', $valid);
+				if (is_array($valid))
+				{
+					$valid = implode(',', $valid);
+				}
 				// NULL is valid for most fields, but not custom fields due to backend handling
 				// See so_sql_cf->save()
-				if (is_null($valid)) $valid = false;
+				if (is_null($valid))
+				{
+					$valid = false;
+				}
 				//error_log(__METHOD__."() $field_name: ".array2string($value).' --> '.array2string($valid));
 			}
 		}
@@ -412,7 +452,10 @@ 
 		{
 			// Transformation doesn't handle validation
 			$valid =& self::get_array($validated, $this->id ? $form_name : $field, true);
-			if (true) $valid = $value_in;
+			if (true)
+			{
+				$valid = $value_in;
+			}
 			//error_log(__METHOD__."() $form_name $field: ".array2string($value).' --> '.array2string($value));
 		}
 	}

api/src/Etemplate/Widget/Contact.php

Spacing +7 added lines, -7 removed lines

@@ -128,7 +128,7 @@ 
 
 		$this->contacts->__construct();
 		$options = $this->contacts->contact_fields;
-		foreach($this->contacts->customfields as $name => $data)
+		foreach ($this->contacts->customfields as $name => $data)
 		{
 			$options['#'.$name] = $data['label'];
 		}
@@ -148,28 +148,28 @@ 
 	 */
 	public function get_contact($value, array $attrs)
 	{
-		if (is_array($value) && !(array_key_exists('app',$value) && array_key_exists('id', $value))) return $value;
+		if (is_array($value) && !(array_key_exists('app', $value) && array_key_exists('id', $value))) return $value;
 
-		if(is_array($value) && array_key_exists('app', $value) && array_key_exists('id', $value)) $value = $value['id'];
-		switch($attrs['type'])
+		if (is_array($value) && array_key_exists('app', $value) && array_key_exists('id', $value)) $value = $value['id'];
+		switch ($attrs['type'])
 		{
 			case 'contact-account':
 			case 'contact-template':
-				if (substr($value,0,8) != 'account:')
+				if (substr($value, 0, 8) != 'account:')
 				{
 					$value = 'account:'.($attrs['name'] != 'account:' ? $value : $GLOBALS['egw_info']['user']['account_id']);
 				}
 				// fall-through
 			case 'contact-value':
 			default:
-				if (substr($value,0,12) == 'addressbook:') $value = substr($value,12);	// link-entry syntax
+				if (substr($value, 0, 12) == 'addressbook:') $value = substr($value, 12); // link-entry syntax
 				if (!($contact = $this->contacts->read($value)))
 				{
 					$contact = array();
 				}
 				break;
 		}
-		unset($contact['jpegphoto']);	// makes no sense to return binary image
+		unset($contact['jpegphoto']); // makes no sense to return binary image
 
 		//error_log(__METHOD__."('$value') returning ".array2string($contact));
 		return $contact;

Braces +13 added lines, -3 removed lines

@@ -148,9 +148,15 @@ 
 	 */
 	public function get_contact($value, array $attrs)
 	{
-		if (is_array($value) && !(array_key_exists('app',$value) && array_key_exists('id', $value))) return $value;
+		if (is_array($value) && !(array_key_exists('app',$value) && array_key_exists('id', $value)))
+		{
+			return $value;
+		}
 
-		if(is_array($value) && array_key_exists('app', $value) && array_key_exists('id', $value)) $value = $value['id'];
+		if(is_array($value) && array_key_exists('app', $value) && array_key_exists('id', $value))
+		{
+			$value = $value['id'];
+		}
 		switch($attrs['type'])
 		{
 			case 'contact-account':
@@ -162,7 +168,11 @@ 
 				// fall-through
 			case 'contact-value':
 			default:
-				if (substr($value,0,12) == 'addressbook:') $value = substr($value,12);	// link-entry syntax
+				if (substr($value,0,12) == 'addressbook:')
+				{
+					$value = substr($value,12);
+				}
+				// link-entry syntax
 				if (!($contact = $this->contacts->read($value)))
 				{
 					$contact = array();

api/src/Etemplate/Widget/Select.php

Spacing +87 added lines, -89 removed lines

@@ -81,7 +81,7 @@ 
 	 */
 	public function __construct($xml = '')
 	{
-		if($xml) {
+		if ($xml) {
 			parent::__construct($xml);
 		}
 	}
@@ -96,7 +96,7 @@ 
 	 * @return Template current object or clone, if any attribute was set
 	 * @todo Use legacy_attributes instead of leaving it to typeOptions method to parse them
 	 */
-	public function set_attrs($xml, $cloned=true)
+	public function set_attrs($xml, $cloned = true)
 	{
 		parent::set_attrs($xml, $cloned);
 
@@ -109,7 +109,7 @@ 
 				unset($this->attrs['options']);
 			}
 		}
-		elseif($this->attrs['rows'] > 1)
+		elseif ($this->attrs['rows'] > 1)
 		{
 			$this->attrs['multiple'] = true;
 		}
@@ -125,7 +125,7 @@ 
 	 * @param array $content
 	 * @param array &$validated=array() validated content
 	 */
-	public function validate($cname, array $expand, array $content, &$validated=array())
+	public function validate($cname, array $expand, array $content, &$validated = array())
 	{
 		$form_name = self::form_name($cname, $this->id, $expand);
 		$widget_type = $this->attrs['type'] ? $this->attrs['type'] : $this->type;
@@ -135,21 +135,21 @@ 
 		{
 			$value = $value_in = self::get_array($content, $form_name);
 
-			$allowed2 = self::selOptions($form_name, true);	// true = return array of option-values
+			$allowed2 = self::selOptions($form_name, true); // true = return array of option-values
 			$type_options = self::typeOptions($this,
 				// typeOptions thinks # of rows is the first thing in options
 				($this->attrs['rows'] && strpos($this->attrs['options'], $this->attrs['rows']) !== 0 ? $this->attrs['rows'].','.$this->attrs['options'] : $this->attrs['options']));
-			$allowed = array_merge($allowed2,array_keys($type_options));
+			$allowed = array_merge($allowed2, array_keys($type_options));
 
 			// add option children's values too, "" is not read, therefore we cast to string
-			foreach($this->children as $child)
+			foreach ($this->children as $child)
 			{
 				if ($child->type == 'option') $allowed[] = (string)$child->attrs['value'];
 			}
 
 			if (!$this->attrs['multiple'] || !($this->attrs['options'] > 1)) $allowed[] = '';
 
-			foreach((array) $value as $val)
+			foreach ((array)$value as $val)
 			{
 				// handle empty-label for all widget types
 				if ((string)$val === '' && in_array('', $allowed)) continue;
@@ -158,7 +158,7 @@ 
 				{
 					case 'select-account':
 						// If in allowed options, skip account check to support app-specific options
-						if(count($allowed) > 0 && in_array($val,$allowed)) continue;
+						if (count($allowed) > 0 && in_array($val, $allowed)) continue;
 
 						// validate accounts independent of options know to server
 						$account_type = $this->attrs['account_type'] ? $this->attrs['account_type'] : 'accounts';
@@ -172,7 +172,7 @@ 
 						)
 						{
 							self::set_validation_error($form_name, lang("'%1' is NOT allowed ('%2')!", $val,
-								!$type?'not found' : ($type == 1 ? 'user' : 'group')),'');
+								!$type ? 'not found' : ($type == 1 ? 'user' : 'group')), '');
 							$value = '';
 							break 2;
 						}
@@ -186,9 +186,9 @@ 
 						break;
 
 					default:
-						if(!in_array($val, $allowed))
+						if (!in_array($val, $allowed))
 						{
-							self::set_validation_error($form_name,lang("'%1' is NOT allowed ('%2')!", $val, implode("','",$allowed)),'');
+							self::set_validation_error($form_name, lang("'%1' is NOT allowed ('%2')!", $val, implode("','", $allowed)), '');
 							$value = '';
 							break 2;
 						}
@@ -196,7 +196,7 @@ 
 			}
 			if ($ok && $value === '' && $this->attrs['needed'])
 			{
-				self::set_validation_error($form_name,lang('Field must not be empty !!!',$value),'');
+				self::set_validation_error($form_name, lang('Field must not be empty !!!', $value), '');
 			}
 			// some widgets sub-types need some post-processing
 			// ToDo: move it together with preprocessing to clientside
@@ -204,7 +204,7 @@ 
 			{
 				case 'select-dow':
 					$dow = 0;
-					foreach((array)$value as $val)
+					foreach ((array)$value as $val)
 					{
 						$dow |= $val;
 					}
@@ -230,7 +230,7 @@ 
 						{
 							$value = array_merge($value, (array)self::$request->preserv[$unavailable_name]);
 						}
-						elseif(!$value)	// for single cat, we only restore unavailable one, if no other was selected
+						elseif (!$value)	// for single cat, we only restore unavailable one, if no other was selected
 						{
 							$value = self::$request->preserv[$unavailable_name];
 						}
@@ -254,7 +254,7 @@ 
 	 * @param string $cname
 	 * @param array $expand values for keys 'c', 'row', 'c_', 'row_', 'cont'
 	 */
-	public function beforeSendToClient($cname, array $expand=null)
+	public function beforeSendToClient($cname, array $expand = null)
 	{
 		//error_log(__METHOD__."('$cname') this->id=$this->id, this->type=$this->type, this->attrs=".array2string($this->attrs));
 		$matches = null;
@@ -287,14 +287,14 @@ 
 		{
 			// Check selection preference, we may be able to skip reading some data
 			$select_pref = $GLOBALS['egw_info']['user']['preferences']['common']['account_selection'];
-			if($this->attrs['type'] == 'select-account' && !$GLOBALS['egw_info']['user']['apps']['admin'] && $select_pref == 'none')
+			if ($this->attrs['type'] == 'select-account' && !$GLOBALS['egw_info']['user']['apps']['admin'] && $select_pref == 'none')
 			{
 				// Preserve but do not send the value if preference is 'none'
 				self::$request->preserv[$this->id] = self::$request->content[$this->id];
 				unset(self::$request->content[$this->id]);
 				$this->attrs['readonly'] = true;
 			}
-			if(!in_array($type, self::$cached_types))
+			if (!in_array($type, self::$cached_types))
 			{
 				// adding type specific options here, while keep further options set by app code
 				// we need to make sure to run only once for auto-repeated rows, because
@@ -324,20 +324,20 @@ 
 
 		// Make sure  s, etc.  are properly encoded when sent, and not double-encoded
 		$options = (isset(self::$request->sel_options[$form_name]) ? $form_name : $this->id);
-		if(is_array(self::$request->sel_options[$options]))
+		if (is_array(self::$request->sel_options[$options]))
 		{
-			if(in_array($this->attrs['type'], self::$cached_types) && !isset($form_names_done[$options]))
+			if (in_array($this->attrs['type'], self::$cached_types) && !isset($form_names_done[$options]))
 			{
 				// Fix any custom options from application
-				self::fix_encoded_options(self::$request->sel_options[$options],true);
+				self::fix_encoded_options(self::$request->sel_options[$options], true);
 				$form_names_done[$options] = true;
 			}
 			// Turn on search, if there's a lot of rows (unless explicitly set)
-			if(!array_key_exists('search',$this->attrs) && count(self::$request->sel_options[$options]) >= self::SEARCH_ROW_LIMIT)
+			if (!array_key_exists('search', $this->attrs) && count(self::$request->sel_options[$options]) >= self::SEARCH_ROW_LIMIT)
 			{
 				self::setElementAttribute($form_name, "search", true);
 			}
-			if(!self::$request->sel_options[$options])
+			if (!self::$request->sel_options[$options])
 			{
 				unset(self::$request->sel_options[$options]);
 			}
@@ -352,12 +352,12 @@ 
 	 * @param array $options
 	 * @param boolean $use_array_of_objects Re-indexes options, making everything more complicated
 	 */
-	public static function fix_encoded_options(array &$options, $use_array_of_objects=null)
+	public static function fix_encoded_options(array &$options, $use_array_of_objects = null)
 	{
 		$backup_options = $options;
 
 		$values = array();
-		foreach($options as $value => &$label)
+		foreach ($options as $value => &$label)
 		{
 			// Of course once we re-index the options, we can't detect duplicates
 			// so check here, as we re-index
@@ -377,7 +377,7 @@ 
 				return self::fix_encoded_options($options, true);
 			}
 			// optgroup or values for keys "label" and "title"
-			if(is_array($label))
+			if (is_array($label))
 			{
 				self::fix_encoded_options($label, false);
 				if ($use_array_of_objects && !array_key_exists('value', $label)) $label['value'] = $value;
@@ -408,7 +408,7 @@ 
 	 * @param boolean $return_values =false true: return array with option values, instead of value => label pairs
 	 * @return array
 	 */
-	public static function selOptions($name, $return_values=false)
+	public static function selOptions($name, $return_values = false)
 	{
 		$options = array();
 
@@ -419,12 +419,12 @@ 
 		}
 
 		// Check for base of name in root of sel_options
-		if(!$options)
+		if (!$options)
 		{
-			$name_parts = explode('[',str_replace(array('[',']',']'),array('['),$name));
+			$name_parts = explode('[', str_replace(array('[', ']', ']'), array('['), $name));
 			if (count($name_parts))
 			{
-				$org_name = $name_parts[count($name_parts)-1];
+				$org_name = $name_parts[count($name_parts) - 1];
 				if (isset(self::$request->sel_options[$org_name]) && is_array(self::$request->sel_options[$org_name]))
 				{
 					$options += self::$request->sel_options[$org_name];
@@ -444,7 +444,7 @@ 
 		if ($return_values)
 		{
 			$values = array();
-			foreach($options as $key => $val)
+			foreach ($options as $key => $val)
 			{
 				if (is_array($val))
 				{
@@ -460,7 +460,7 @@ 
 					}
 					else	// optgroup
 					{
-						foreach($val as $k => $v)
+						foreach ($val as $k => $v)
 						{
 							$values[] = is_array($v) && isset($v['value']) ? $v['value'] : $k;
 						}
@@ -489,50 +489,50 @@ 
 	 * @param string $form_name =null
 	 * @return array with value => label pairs
 	 */
-	public static function typeOptions($widget_type, $_legacy_options, &$no_lang=false, $readonly=false, &$value=null, $form_name=null)
+	public static function typeOptions($widget_type, $_legacy_options, &$no_lang = false, $readonly = false, &$value = null, $form_name = null)
 	{
-		if($widget_type && is_object($widget_type))
+		if ($widget_type && is_object($widget_type))
 		{
 			$widget = $widget_type;
 			$widget_type = $widget->attrs['type'] ? $widget->attrs['type'] : $widget->type;
 		}
 		// Legacy / static support
 		// Have to do this explicitly, since legacy options is not defined on class level
-		$legacy_options = explode(',',$_legacy_options);
-		foreach($legacy_options as &$field)
+		$legacy_options = explode(',', $_legacy_options);
+		foreach ($legacy_options as &$field)
 		{
-			$field = self::expand_name($field, 0, 0,'','',self::$cont);
+			$field = self::expand_name($field, 0, 0, '', '', self::$cont);
 		}
 
-		list($rows,$type,$type2,$type3,$type4,$type5) = $legacy_options;
+		list($rows, $type, $type2, $type3, $type4, $type5) = $legacy_options;
 		$no_lang = false;
 		$options = array();
 		switch ($widget_type)
 		{
 			case 'select-percent':	// options: #row,decrement(default=10)
 				$decr = $type > 0 ? $type : 10;
-				for ($i=0; $i <= 100; $i += $decr)
+				for ($i = 0; $i <= 100; $i += $decr)
 				{
 					$options[intval($i)] = intval($i).'%';
 				}
 				$options[100] = '100%';
 				if (!$rows || !empty($value))
 				{
-					$value = intval(($value+($decr/2)) / $decr) * $decr;
+					$value = intval(($value + ($decr / 2)) / $decr) * $decr;
 				}
 				$no_lang = True;
 				break;
 
 			case 'select-priority':
-				$options = array('','low','normal','high');
+				$options = array('', 'low', 'normal', 'high');
 				break;
 
 			case 'select-bool':	// equal to checkbox, can be used with nextmatch-customfilter to filter a boolean column
-				$options = array(0 => 'no',1 => 'yes');
+				$options = array(0 => 'no', 1 => 'yes');
 				break;
 
 			case 'select-country':	// #Row|Extralabel,1=use country name, 0=use 2 letter-code,custom country field name
-				if($type == 0 && $type2)
+				if ($type == 0 && $type2)
 				{
 					$custom_label = is_numeric($type2) ? 'Custom' : $type2;
 					$options = array('-custom-' => lang($custom_label)) + Api\Country::countries();
@@ -546,7 +546,7 @@ 
 					$value = Api\Country::country_code($value);
 					if (!isset($options[$value]))
 					{
-						if($type2)
+						if ($type2)
 						{
 							$options[$value] = $value;
 						}
@@ -562,20 +562,20 @@ 
 
 			case 'select-cat':	// !$type == globals cats too, $type2: extraStyleMultiselect, $type3: application, if not current-app, $type4: parent-id, $type5=owner (-1=global),$type6=show missing
 				if ((!$type3 || $type3 === $GLOBALS['egw']->categories->app_name) &&
-					(!$type5 || $type5 ==  $GLOBALS['egw']->categories->account_id))
+					(!$type5 || $type5 == $GLOBALS['egw']->categories->account_id))
 				{
 					$categories = $GLOBALS['egw']->categories;
 				}
 				else	// we need to instanciate a new cat object for the correct application
 				{
-					$categories = new Api\Categories($type5,$type3);
+					$categories = new Api\Categories($type5, $type3);
 				}
 				// Allow text for global
 				$type = ($type && strlen($type) > 1 ? $type : !$type);
 				// we cast $type4 (parent) to int, to get default of 0 if omitted
-				foreach((array)$categories->return_sorted_array(0,False,'','','',$type,(int)$type4,true) as $cat)
+				foreach ((array)$categories->return_sorted_array(0, False, '', '', '', $type, (int)$type4, true) as $cat)
 				{
-					$s = str_repeat('&nbsp;',$cat['level']) . stripslashes($cat['name']);
+					$s = str_repeat('&nbsp;', $cat['level']).stripslashes($cat['name']);
 
 					if (Api\Categories::is_global($cat))
 					{
@@ -590,13 +590,13 @@ 
 						'children'	=> $cat['children']
 					);
 					// Send data too
-					if(is_array($cat['data']))
+					if (is_array($cat['data']))
 					{
 						$options[$cat['id']] += $cat['data'];
 					}
 				}
 				// preserv unavailible cats (eg. private user-cats)
-				if ($value && ($unavailible = array_diff(is_array($value) ? $value : explode(',',$value),array_keys((array)$options))))
+				if ($value && ($unavailible = array_diff(is_array($value) ? $value : explode(',', $value), array_keys((array)$options))))
 				{
 					// unavailable cats need to be merged in again
 					$unavailable_name = $form_name.self::UNAVAILABLE_CAT_POSTFIX;
@@ -609,11 +609,11 @@ 
 				$options[''] = '';
 				if ($type <= 0)  $type  = 3;
 				if ($type2 <= 0) $type2 = 2;
-				if ($type > 100 && $type2 > 100 && $type > $type) { $y = $type; $type=$type2; $type2=$y; }
-				$y = date('Y')-$type;
-				if ($value && $value-$type < $y || $type > 100) $y = $type > 100 ? $type : $value-$type;
-				$to = date('Y')+$type2;
-				if ($value && $value+$type2 > $to || $type2 > 100) $to = $type2 > 100 ? $type2 : $value+$type2;
+				if ($type > 100 && $type2 > 100 && $type > $type) { $y = $type; $type = $type2; $type2 = $y; }
+				$y = date('Y') - $type;
+				if ($value && $value - $type < $y || $type > 100) $y = $type > 100 ? $type : $value - $type;
+				$to = date('Y') + $type2;
+				if ($value && $value + $type2 > $to || $type2 > 100) $to = $type2 > 100 ? $type2 : $value + $type2;
 				for ($n = 0; $y <= $to && $n < 200; ++$n)
 				{
 					$options[$y] = $y++;
@@ -629,17 +629,17 @@ 
 			case 'select-dow':	// options: rows[,0=summaries befor days, 1=summaries after days, 2=no summaries[,extraStyleMultiselect]]
 				if (!defined('MCAL_M_SUNDAY'))
 				{
-					define('MCAL_M_SUNDAY',1);
-					define('MCAL_M_MONDAY',2);
-					define('MCAL_M_TUESDAY',4);
-					define('MCAL_M_WEDNESDAY',8);
-					define('MCAL_M_THURSDAY',16);
-					define('MCAL_M_FRIDAY',32);
-					define('MCAL_M_SATURDAY',64);
-
-					define('MCAL_M_WEEKDAYS',62);
-					define('MCAL_M_WEEKEND',65);
-					define('MCAL_M_ALLDAYS',127);
+					define('MCAL_M_SUNDAY', 1);
+					define('MCAL_M_MONDAY', 2);
+					define('MCAL_M_TUESDAY', 4);
+					define('MCAL_M_WEDNESDAY', 8);
+					define('MCAL_M_THURSDAY', 16);
+					define('MCAL_M_FRIDAY', 32);
+					define('MCAL_M_SATURDAY', 64);
+
+					define('MCAL_M_WEEKDAYS', 62);
+					define('MCAL_M_WEEKEND', 65);
+					define('MCAL_M_ALLDAYS', 127);
 				}
 				$weekstart = $GLOBALS['egw_info']['user']['preferences']['calendar']['weekdaystarts'];
 				$options = array();
@@ -672,9 +672,9 @@ 
 				}
 				$value_in = $value;
 				$value = array();
-				foreach(array_keys($options) as $val)
+				foreach (array_keys($options) as $val)
 				{
-					if (($value_in & $val) == $val)
+					if (($value_in&$val) == $val)
 					{
 						$value[] = $val;
 
@@ -682,7 +682,7 @@ 
 							$val == MCAL_M_WEEKDAYS && $value_in == MCAL_M_WEEKDAYS ||
 							$val == MCAL_M_WEEKEND && $value_in == MCAL_M_WEEKEND)
 						{
-							break;	// dont set the others
+							break; // dont set the others
 						}
 					}
 				}
@@ -695,22 +695,22 @@ 
 				// fall-through
 
 			case 'select-number':	// options: rows,min,max,decrement,suffix
-				$type = $type === '' ? 1 : intval($type);		// min
-				$type2 = $type2 === '' ? 10 : intval($type2);	// max
+				$type = $type === '' ? 1 : intval($type); // min
+				$type2 = $type2 === '' ? 10 : intval($type2); // max
 				$format = '%d';
 				if (!empty($type3) && $type3[0] == '0')			// leading zero
 				{
 					$format = '%0'.strlen($type3).'d';
 				}
-				$type3 = !$type3 ? 1 : intval($type3);			// decrement
+				$type3 = !$type3 ? 1 : intval($type3); // decrement
 				if (($type <= $type2) != ($type3 > 0))
 				{
-					$type3 = -$type3;	// void infinite loop
+					$type3 = -$type3; // void infinite loop
 				}
 				if (!empty($type4)) $format .= lang($type4);
-				for ($i=0,$n=$type; $n <= $type2 && $i <= 100; $n += $type3,++$i)
+				for ($i = 0, $n = $type; $n <= $type2 && $i <= 100; $n += $type3, ++$i)
 				{
-					$options[$n] = sprintf($format,$n);
+					$options[$n] = sprintf($format, $n);
 				}
 				$no_lang = True;
 				break;
@@ -720,15 +720,14 @@ 
 				for ($h = 0; $h <= 23; ++$h)
 				{
 					$options[$h] = $GLOBALS['egw_info']['user']['preferences']['common']['timeformat'] == 12 ?
-						(($h % 12 ? $h % 12 : 12).$minutes.' '.($h < 12 ? lang('am') : lang('pm'))) :
-						sprintf('%02d',$h).$minutes;
+						(($h % 12 ? $h % 12 : 12).$minutes.' '.($h < 12 ? lang('am') : lang('pm'))) : sprintf('%02d', $h).$minutes;
 				}
 				$no_lang = True;
 				break;
 
 			case 'select-app':	// type2: 'user'=apps of current user, 'enabled', 'installed' (default), 'all' = not installed ones too
 				$apps = self::app_options($type2);
-				$options = is_array($options) ? $options+$apps : $apps;
+				$options = is_array($options) ? $options + $apps : $apps;
 				break;
 
 			case 'select-lang':
@@ -743,7 +742,7 @@ 
 				}
 				if ($readonly)	// for readonly we dont need to fetch all TZ's
 				{
-					$options[$value] = calendar_timezones::tz2id($value,'name');
+					$options[$value] = calendar_timezones::tz2id($value, 'name');
 				}
 				else
 				{
@@ -773,7 +772,7 @@ 
 		{
 			if ($type2 == 'enabled' && (!$data['enabled'] || !$data['status'] || $data['status'] == 3))
 			{
-				continue;	// app not enabled (user can not have run rights for these apps)
+				continue; // app not enabled (user can not have run rights for these apps)
 			}
 			if ($type2 != 'user' || $GLOBALS['egw_info']['user']['apps'][$app])
 			{
@@ -788,7 +787,7 @@ 
 				if (@is_dir(EGW_SERVER_ROOT."/$file/setup") && $file[0] != '.' &&
 				!isset($apps[$app = basename($file)]))
 				{
-					$apps[$app] = $app . ' (*)';
+					$apps[$app] = $app.' (*)';
 				}
 			}
 			closedir($dir);
@@ -806,7 +805,7 @@ 
 	 * @param boolean $show_type =false true: return array with values for keys label and icon, false: only label
 	 * @return string|array
 	 */
-	public static function accountInfo($id,$acc=null,$longnames=0,$show_type=false)
+	public static function accountInfo($id, $acc = null, $longnames = 0, $show_type = false)
 	{
 		if (!$id)
 		{
@@ -817,7 +816,7 @@ 
 		{
 			$data = $GLOBALS['egw']->accounts->get_account_data($id);
 			if (!isset($data[$id])) return '#'.$id;
-			foreach(array('type','lid','firstname','lastname') as $name)
+			foreach (array('type', 'lid', 'firstname', 'lastname') as $name)
 			{
 				$acc['account_'.$name] = $data[$id][$name];
 			}
@@ -833,18 +832,17 @@ 
 				$info .= '&lt;'.$acc['account_lid'].'&gt; ';
 				// fall-through
 			case 1:
-				$info .= $acc['account_type'] == 'g' ? lang('group').' '.$acc['account_lid'] :
-					$acc['account_firstname'].' '.$acc['account_lastname'];
+				$info .= $acc['account_type'] == 'g' ? lang('group').' '.$acc['account_lid'] : $acc['account_firstname'].' '.$acc['account_lastname'];
 				break;
 			case '0':
 				$info .= $acc['account_lid'];
 				break;
 			default:			// use the phpgw default
 				$info = Api\Accounts::format_username($acc['account_lid'],
-					$acc['account_firstname'],$acc['account_lastname']);
+					$acc['account_firstname'], $acc['account_lastname']);
 				break;
 		}
-		if($show_type) {
+		if ($show_type) {
 			$info = array(
 				'label'	=> $info,
 				'icon' => $acc['account_type'] == 'g' ? 'addressbook/group' : 'user'
@@ -864,8 +862,8 @@ 
 	public static function ajax_get_options($type, $attributes, $value = null)
 	{
 		$no_lang = false;
-		$options = self::typeOptions($type, $attributes,$no_lang,false,$value);
-		self::fix_encoded_options($options,true);
+		$options = self::typeOptions($type, $attributes, $no_lang, false, $value);
+		self::fix_encoded_options($options, true);
 		$response = Api\Json\Response::get();
 		$response->data($options);
 	}

Braces +88 added lines, -25 removed lines

@@ -81,7 +81,8 @@ 
 	 */
 	public function __construct($xml = '')
 	{
-		if($xml) {
+		if($xml)
+		{
 			parent::__construct($xml);
 		}
 	}
@@ -144,21 +145,33 @@ 
 			// add option children's values too, "" is not read, therefore we cast to string
 			foreach($this->children as $child)
 			{
-				if ($child->type == 'option') $allowed[] = (string)$child->attrs['value'];
+				if ($child->type == 'option')
+				{
+					$allowed[] = (string)$child->attrs['value'];
+				}
 			}
 
-			if (!$this->attrs['multiple'] || !($this->attrs['options'] > 1)) $allowed[] = '';
+			if (!$this->attrs['multiple'] || !($this->attrs['options'] > 1))
+			{
+				$allowed[] = '';
+			}
 
 			foreach((array) $value as $val)
 			{
 				// handle empty-label for all widget types
-				if ((string)$val === '' && in_array('', $allowed)) continue;
+				if ((string)$val === '' && in_array('', $allowed))
+				{
+					continue;
+				}
 
 				switch ($widget_type)
 				{
 					case 'select-account':
 						// If in allowed options, skip account check to support app-specific options
-						if(count($allowed) > 0 && in_array($val,$allowed)) continue;
+						if(count($allowed) > 0 && in_array($val,$allowed))
+						{
+							continue;
+						}
 
 						// validate accounts independent of options know to server
 						$account_type = $this->attrs['account_type'] ? $this->attrs['account_type'] : 'accounts';
@@ -169,8 +182,7 @@ 
 							in_array($account_type, array('owngroups', 'memberships')) &&
 								!in_array($val, $GLOBALS['egw']->accounts->memberships(
 									$GLOBALS['egw_info']['user']['account_id'], true))
-						)
-						{
+						) {
 							self::set_validation_error($form_name, lang("'%1' is NOT allowed ('%2')!", $val,
 								!$type?'not found' : ($type == 1 ? 'user' : 'group')),'');
 							$value = '';
@@ -230,10 +242,13 @@ 
 						{
 							$value = array_merge($value, (array)self::$request->preserv[$unavailable_name]);
 						}
-						elseif(!$value)	// for single cat, we only restore unavailable one, if no other was selected
+						elseif(!$value)
+						{
+							// for single cat, we only restore unavailable one, if no other was selected
 						{
 							$value = self::$request->preserv[$unavailable_name];
 						}
+						}
 					}
 			}
 			if (isset($value))
@@ -258,10 +273,13 @@ 
 	{
 		//error_log(__METHOD__."('$cname') this->id=$this->id, this->type=$this->type, this->attrs=".array2string($this->attrs));
 		$matches = null;
-		if ($cname == '$row')	// happens eg. with custom-fields: $cname='$row', this->id='#something'
+		if ($cname == '$row')
+		{
+			// happens eg. with custom-fields: $cname='$row', this->id='#something'
 		{
 			$form_name = $this->id;
 		}
+		}
 		// happens with fields in nm-header: $cname='nm', this->id='${row}[something]' or '{$row}[something]'
 		elseif (preg_match('/(\${row}|{\$row})\[([^]]+)\]$/', $this->id, $matches))
 		{
@@ -281,7 +299,10 @@ 
 		{
 			$form_name = self::form_name($cname, $this->id, $expand);
 		}
-		if (!is_array(self::$request->sel_options[$form_name])) self::$request->sel_options[$form_name] = array();
+		if (!is_array(self::$request->sel_options[$form_name]))
+		{
+			self::$request->sel_options[$form_name] = array();
+		}
 		$type = $this->attrs['type'] ? $this->attrs['type'] : $this->type;
 		if ($type != 'select' && $type != 'menupopup')
 		{
@@ -380,7 +401,10 @@ 
 			if(is_array($label))
 			{
 				self::fix_encoded_options($label, false);
-				if ($use_array_of_objects && !array_key_exists('value', $label)) $label['value'] = $value;
+				if ($use_array_of_objects && !array_key_exists('value', $label))
+				{
+					$label['value'] = $value;
+				}
 			}
 			else
 			{
@@ -607,13 +631,27 @@ 
 
 			case 'select-year':	// options: #rows,#before(default=3),#after(default=2)
 				$options[''] = '';
-				if ($type <= 0)  $type  = 3;
-				if ($type2 <= 0) $type2 = 2;
-				if ($type > 100 && $type2 > 100 && $type > $type) { $y = $type; $type=$type2; $type2=$y; }
+				if ($type <= 0)
+				{
+					$type  = 3;
+				}
+				if ($type2 <= 0)
+				{
+					$type2 = 2;
+				}
+				if ($type > 100 && $type2 > 100 && $type > $type)
+				{
+$y = $type; $type=$type2; $type2=$y; }
 				$y = date('Y')-$type;
-				if ($value && $value-$type < $y || $type > 100) $y = $type > 100 ? $type : $value-$type;
+				if ($value && $value-$type < $y || $type > 100)
+				{
+					$y = $type > 100 ? $type : $value-$type;
+				}
 				$to = date('Y')+$type2;
-				if ($value && $value+$type2 > $to || $type2 > 100) $to = $type2 > 100 ? $type2 : $value+$type2;
+				if ($value && $value+$type2 > $to || $type2 > 100)
+				{
+					$to = $type2 > 100 ? $type2 : $value+$type2;
+				}
 				for ($n = 0; $y <= $to && $n < 200; ++$n)
 				{
 					$options[$y] = $y++;
@@ -651,8 +689,14 @@ 
 						MCAL_M_WEEKEND	=> 'weekend',
 					);
 				}
-				if ($weekstart == 'Saturday') $options[MCAL_M_SATURDAY] = 'saturday';
-				if ($weekstart != 'Monday') $options[MCAL_M_SUNDAY] = 'sunday';
+				if ($weekstart == 'Saturday')
+				{
+					$options[MCAL_M_SATURDAY] = 'saturday';
+				}
+				if ($weekstart != 'Monday')
+				{
+					$options[MCAL_M_SUNDAY] = 'sunday';
+				}
 				$options += array(
 					MCAL_M_MONDAY	=> 'monday',
 					MCAL_M_TUESDAY	=> 'tuesday',
@@ -660,8 +704,14 @@ 
 					MCAL_M_THURSDAY	=> 'thursday',
 					MCAL_M_FRIDAY	=> 'friday',
 				);
-				if ($weekstart != 'Saturday') $options[MCAL_M_SATURDAY] = 'saturday';
-				if ($weekstart == 'Monday') $options[MCAL_M_SUNDAY] = 'sunday';
+				if ($weekstart != 'Saturday')
+				{
+					$options[MCAL_M_SATURDAY] = 'saturday';
+				}
+				if ($weekstart == 'Monday')
+				{
+					$options[MCAL_M_SUNDAY] = 'sunday';
+				}
 				if ($rows >= 2 && $type == 1)
 				{
 					$options += array(
@@ -698,16 +748,22 @@ 
 				$type = $type === '' ? 1 : intval($type);		// min
 				$type2 = $type2 === '' ? 10 : intval($type2);	// max
 				$format = '%d';
-				if (!empty($type3) && $type3[0] == '0')			// leading zero
+				if (!empty($type3) && $type3[0] == '0')
+				{
+					// leading zero
 				{
 					$format = '%0'.strlen($type3).'d';
 				}
+				}
 				$type3 = !$type3 ? 1 : intval($type3);			// decrement
 				if (($type <= $type2) != ($type3 > 0))
 				{
 					$type3 = -$type3;	// void infinite loop
 				}
-				if (!empty($type4)) $format .= lang($type4);
+				if (!empty($type4))
+				{
+					$format .= lang($type4);
+				}
 				for ($i=0,$n=$type; $n <= $type2 && $i <= 100; $n += $type3,++$i)
 				{
 					$options[$n] = sprintf($format,$n);
@@ -741,10 +797,13 @@ 
 				{
 					$value = calendar_timezones::id2tz($value);
 				}
-				if ($readonly)	// for readonly we dont need to fetch all TZ's
+				if ($readonly)
+				{
+					// for readonly we dont need to fetch all TZ's
 				{
 					$options[$value] = calendar_timezones::tz2id($value,'name');
 				}
+				}
 				else
 				{
 					$options = $type ? Api\DateTime::getTimezones() : Api\DateTime::getUserTimezones($value);
@@ -816,7 +875,10 @@ 
 		if (!is_array($acc))
 		{
 			$data = $GLOBALS['egw']->accounts->get_account_data($id);
-			if (!isset($data[$id])) return '#'.$id;
+			if (!isset($data[$id]))
+			{
+				return '#'.$id;
+			}
 			foreach(array('type','lid','firstname','lastname') as $name)
 			{
 				$acc['account_'.$name] = $data[$id][$name];
@@ -844,7 +906,8 @@ 
 					$acc['account_firstname'],$acc['account_lastname']);
 				break;
 		}
-		if($show_type) {
+		if($show_type)
+		{
 			$info = array(
 				'label'	=> $info,
 				'icon' => $acc['account_type'] == 'g' ? 'addressbook/group' : 'user'

api/src/Etemplate/Widget/Box.php

Spacing +12 added lines, -12 removed lines

@@ -42,17 +42,17 @@ 
 	 * @param array $params =array('') parameter(s) first parameter has to be cname!
 	 * @param boolean $respect_disabled =false false (default): ignore disabled, true: method is NOT run for disabled widgets AND their children
 	 */
-	public function run($method_name, $params=array(''), $respect_disabled=false)
+	public function run($method_name, $params = array(''), $respect_disabled = false)
 	{
-		$cname =& $params[0];
-		$expand =& $params[1];
+		$cname = & $params[0];
+		$expand = & $params[1];
 		$old_cname = $params[0];
 		$old_expand = $params[1];
 
 		if ($this->id && $this->type != 'groupbox') $cname = self::form_name($cname, $this->id, $params[1]);
 		if ($expand['cname'] !== $cname && $cname)
 		{
-			$expand['cont'] =& self::get_array(self::$request->content, $cname);
+			$expand['cont'] = & self::get_array(self::$request->content, $cname);
 			$expand['cname'] = $cname;
 		}
 		if ($respect_disabled && ($disabled = $this->attrs['disabled'] && self::check_disabled($this->attrs['disabled'], $expand)))
@@ -67,11 +67,11 @@ 
 
 		// Expand children
 		$columns_disabled = null;
-		for($n = 0; ; ++$n)
+		for ($n = 0; ; ++$n)
 		{
 			if (isset($this->children[$n]))
 			{
-				$child =& $this->children[$n];
+				$child = & $this->children[$n];
 				// If type has something that can be expanded, we need to expand it so the correct method is run
 				$this->expand_widget($child, $expand);
 			}
@@ -107,17 +107,17 @@ 
 	 */
 	private function need_autorepeat(Etemplate\Widget $widget, $cname, array $expand)
 	{
-		foreach(array($widget) + $widget->children as $check_widget)
+		foreach (array($widget) + $widget->children as $check_widget)
 		{
 			$pat = $check_widget->id;
-			while(($pattern = strstr($pat, '$')))
+			while (($pattern = strstr($pat, '$')))
 			{
-				$pat = substr($pattern,$pattern[1] == '{' ? 2 : 1);
+				$pat = substr($pattern, $pattern[1] == '{' ? 2 : 1);
 
-				$Ok = $pat[0] == 'r' && !(substr($pat,0,2) == 'r_' ||
-					substr($pat,0,4) == 'row_' && substr($pat,0,8) != 'row_cont');
+				$Ok = $pat[0] == 'r' && !(substr($pat, 0, 2) == 'r_' ||
+					substr($pat, 0, 4) == 'row_' && substr($pat, 0, 8) != 'row_cont');
 
-				if ($Ok && ($fname=self::form_name($cname, $check_widget->id, $expand)) &&
+				if ($Ok && ($fname = self::form_name($cname, $check_widget->id, $expand)) &&
 					// need to break if fname ends in [] as get_array() will ignore it and returns whole array
 					// for an id like "run[$row_cont[appname]]"
 					substr($fname, -2) != '[]' &&

Braces +8 added lines, -2 removed lines

@@ -49,7 +49,10 @@ 
 		$old_cname = $params[0];
 		$old_expand = $params[1];
 
-		if ($this->id && $this->type != 'groupbox') $cname = self::form_name($cname, $this->id, $params[1]);
+		if ($this->id && $this->type != 'groupbox')
+		{
+			$cname = self::form_name($cname, $this->id, $params[1]);
+		}
 		if ($expand['cname'] !== $cname && $cname)
 		{
 			$expand['cont'] =& self::get_array(self::$request->content, $cname);
@@ -121,10 +124,13 @@ 
 					// need to break if fname ends in [] as get_array() will ignore it and returns whole array
 					// for an id like "run[$row_cont[appname]]"
 					substr($fname, -2) != '[]' &&
-					($value = self::get_array(self::$request->content, $fname)) !== null)	// null = not found (can be false!)
+					($value = self::get_array(self::$request->content, $fname)) !== null)
+				{
+					// null = not found (can be false!)
 				{
 					//error_log(__METHOD__."($widget,$cname) $this autorepeating row $expand[row] because of $check_widget->id = '$fname' is ".array2string($value));
 					unset($value);
+				}
 					return true;
 				}
 			}

api/src/Etemplate/Widget/AjaxSelect.php

Spacing +4 added lines, -4 removed lines

@@ -26,7 +26,7 @@ 
 	 * @param string $cname
 	 * @param array $expand values for keys 'c', 'row', 'c_', 'row_', 'cont'
 	 */
-	public function beforeSendToClient($cname, array $expand=null)
+	public function beforeSendToClient($cname, array $expand = null)
 	{
 		$matches = null;
 		if ($cname == '$row')	// happens eg. with custom-fields: $cname='$row', this->id='#something'
@@ -55,13 +55,13 @@ 
 
 		// Make sure  s, etc.  are properly encoded when sent, and not double-encoded
 		$options = (isset(self::$request->sel_options[$form_name]) ? $form_name : $this->id);
-		if(is_array(self::$request->sel_options[$options]))
+		if (is_array(self::$request->sel_options[$options]))
 		{
 
 			// Fix any custom options from application
-			self::fix_encoded_options(self::$request->sel_options[$options],true);
+			self::fix_encoded_options(self::$request->sel_options[$options], true);
 
-			if(!self::$request->sel_options[$options])
+			if (!self::$request->sel_options[$options])
 			{
 				unset(self::$request->sel_options[$options]);
 			}

Braces +4 added lines, -1 removed lines

@@ -29,10 +29,13 @@
 	public function beforeSendToClient($cname, array $expand=null)
 	{
 		$matches = null;
-		if ($cname == '$row')	// happens eg. with custom-fields: $cname='$row', this->id='#something'
+		if ($cname == '$row')
+		{
+			// happens eg. with custom-fields: $cname='$row', this->id='#something'
 		{
 			$form_name = $this->id;
 		}
+		}
 		// happens with fields in nm-header: $cname='nm', this->id='${row}[something]' or '{$row}[something]'
 		elseif (preg_match('/(\${row}|{\$row})\[([^]]+)\]$/', $this->id, $matches))
 		{

api/src/Etemplate/Widget/Nextmatch/Accountfilter.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
 	 * @param string|XMLReader $xml
 	 * @param boolean $cloned =true true: object does NOT need to be cloned, false: to set attribute, set them in cloned object
 	 */
-	public function set_attrs($xml, $cloned=true)
+	public function set_attrs($xml, $cloned = true)
 	{
 		parent::set_attrs($xml, $cloned);
 

api/src/Etemplate/Widget/Nextmatch/Customfilter.php

Spacing +8 added lines, -8 removed lines

@@ -29,18 +29,18 @@ 
 	 * @param string $cname
 	 * @param array $expand values for keys 'c', 'row', 'c_', 'row_', 'cont'
 	 */
-	public function beforeSendToClient($cname, array $expand=null)
+	public function beforeSendToClient($cname, array $expand = null)
 	{
-		switch($this->attrs['type'])
+		switch ($this->attrs['type'])
 		{
 			case "link-entry":
 				self::$transformation['type'] = $this->attrs['type'] = 'nextmatch-entryheader';
 				break;
 			default:
-				list($type) = explode('-',$this->attrs['type']);
-				if($type == 'select')
+				list($type) = explode('-', $this->attrs['type']);
+				if ($type == 'select')
 				{
-					if(in_array($this->attrs['type'], Widget\Select::$cached_types))
+					if (in_array($this->attrs['type'], Widget\Select::$cached_types))
 					{
 						$widget_type = $this->attrs['type'];
 					}
@@ -53,7 +53,7 @@ 
 		$this->setElementAttribute($form_name, 'options', trim($this->attrs['widget_options']) != '' ? $this->attrs['widget_options'] : '');
 
 		$this->setElementAttribute($form_name, 'type', $this->attrs['type']);
-		if($widget_type)
+		if ($widget_type)
 		{
 			$this->setElementAttribute($form_name, 'widget_type', $widget_type);
 		}
@@ -69,11 +69,11 @@ 
 	 * @param array $content
 	 * @param array &$validated=array() validated content
 	 */
-	public function validate($cname, array $expand, array $content, &$validated=array())
+	public function validate($cname, array $expand, array $content, &$validated = array())
 	{
 		$value = $value_in = self::get_array($content, $form_name);
 
-		$valid =& self::get_array($validated, $form_name, true);
+		$valid = & self::get_array($validated, $form_name, true);
 		// returning null instead of array(), as array() will be overwritten by etemplate_new::complete_array_merge()
 		// with preserved old content and therefore user can not empty a taglist
 		$valid = $value ? $value : null;