EGroupware / egroupware

preferences/inc/class.preferences_settings.inc.php

Doc Comments +1 added lines, -1 removed lines

@@ -292,7 +292,7 @@
 	 * @param array &$sel_options
 	 * @param array &$readonlys
 	 * @param array &$types on return setting-name => setting-type
-	 * @param etemplate $tpl
+	 * @param Etemplate $tpl
 	 * @throws Api\Exception\WrongParameter
 	 * @return array content
 	 */

Spacing +55 added lines, -56 removed lines

@@ -45,7 +45,7 @@ 
 	 * @param array $content =null
 	 * @param string $msg =''
 	 */
-	function index(array $content=null, $msg='')
+	function index(array $content = null, $msg = '')
 	{
 		$tpl = new Etemplate('preferences.settings');
 		if (!is_array($content))
@@ -70,7 +70,7 @@ 
 			{
 				list($button) = each($content['button']);
 				$appname = $content['old_appname'] ? $content['old_appname'] : 'common';
-				switch($button)
+				switch ($button)
 				{
 					case 'save':
 					case 'apply':
@@ -79,10 +79,10 @@ 
 						{
 							throw new Api\Exception\NoPermission\Admin;
 						}
-						list($type,$account_id) = explode(':', $content['old_type']);
+						list($type, $account_id) = explode(':', $content['old_type']);
 						// merge prefs of all tabs together again
 						$prefs = array();
-						foreach($content as $name => $val)
+						foreach ($content as $name => $val)
 						{
 							if (is_array($val) && strpos($name, 'tab') === 0)
 							{
@@ -100,7 +100,7 @@ 
 						$old_values = array_intersect_key($GLOBALS['egw_info']['user']['preferences']['common'], array_flip($require_reload));
 
 						$attribute = $type == 'group' ? 'user' : $type;
-						if (!($msg=$this->process_array($GLOBALS['egw']->preferences->$attribute, $prefs, $content['types'], $appname, $attribute)))
+						if (!($msg = $this->process_array($GLOBALS['egw']->preferences->$attribute, $prefs, $content['types'], $appname, $attribute)))
 						{
 							$msg_type = 'success';
 							$msg = lang('Preferences saved.');
@@ -124,14 +124,14 @@ 
 						// update client-side Api\Preferences in response
 						Framework::ajax_get_preference($appname);
 				}
-				if (in_array($button, array('save','cancel')))
+				if (in_array($button, array('save', 'cancel')))
 				{
 					Api\Json\Response::get()->call('egw.message', $msg, $msg_type);
 					Framework::window_close();
 				}
 			}
 			$appname = $content['appname'] ? $content['appname'] : 'common';
-			list($type,$account_id) = explode(':', $content['type']);
+			list($type, $account_id) = explode(':', $content['type']);
 			//_debug_array($prefs);
 		}
 		if ($account_id && $account_id != $GLOBALS['egw']->preferences->get_account_id())
@@ -150,7 +150,7 @@ 
 		$preserve['current_app'] = $content['current_app'];
 		$GLOBALS['egw_info']['flags']['currentapp'] = $content['current_app'] == 'common' ?
 			'preferences' : $content['current_app'];
-		Framework::includeCSS('preferences','app');
+		Framework::includeCSS('preferences', 'app');
 
 		// if not just saved, call validation before, to be able to show failed validation of current prefs
 		if (!isset($button))
@@ -182,17 +182,17 @@ 
 	 * @param boolean $only_verify =false
 	 * @return string with verification error or null on success
 	 */
-	function process_array(array &$repository, array $values, array $types, $appname, $type, $only_verify=false)
+	function process_array(array &$repository, array $values, array $types, $appname, $type, $only_verify = false)
 	{
 		//_debug_array($repository);
 		$prefs = &$repository[$appname];
 
 		unset($prefs['']);
 		//_debug_array($values);exit;
-		foreach($values as $var => $value)
+		foreach ($values as $var => $value)
 		{
 			// type specific validation
-			switch((string)$types[$var])
+			switch ((string)$types[$var])
 			{
 				case 'password':	// dont write empty password-fields
 					if (empty($value)) continue 2;
@@ -205,7 +205,7 @@ 
 						// empty is always allowed
 
 						// If forced, empty == not set
-						if($type == 'forced')
+						if ($type == 'forced')
 						{
 							unset($prefs[$var]);
 							// need to call preferences::delete, to also set affective prefs!
@@ -217,19 +217,19 @@ 
 					{
 						if ($value[0] != '/' || !Vfs::stat($value) || Vfs::is_dir($value))
 						{
-							$error = lang('%1 is no existing vfs file!',htmlspecialchars($value));
+							$error = lang('%1 is no existing vfs file!', htmlspecialchars($value));
 						}
 					}
 					else
 					{
 						// split multiple comma or whitespace separated directories
 						// to still allow space or comma in dirnames, we also use the trailing slash of all pathes to split
-						foreach($types[$var] == 'vfs_dir' ? array($value) : preg_split('/[,\s]+\//', $value) as $n => $dir)
+						foreach ($types[$var] == 'vfs_dir' ? array($value) : preg_split('/[,\s]+\//', $value) as $n => $dir)
 						{
-							if ($n) $dir = '/'.$dir;	// re-adding trailing slash removed by split
+							if ($n) $dir = '/'.$dir; // re-adding trailing slash removed by split
 							if ($dir[0] != '/' || !Vfs::stat($dir) || !Vfs::is_dir($dir))
 							{
-								$error .= ($error ? ' ' : '').lang('%1 is no existing vfs directory!',$dir);
+								$error .= ($error ? ' ' : '').lang('%1 is no existing vfs directory!', $dir);
 							}
 						}
 					}
@@ -241,7 +241,7 @@ 
 
 			if (isset($value) && $value !== '' && $value !== '**NULL**' && $value !== array())
 			{
-				if (is_array($value)) $value = implode(',',$value);	// multiselect
+				if (is_array($value)) $value = implode(',', $value); // multiselect
 
 				$prefs[$var] = $value;
 
@@ -261,7 +261,7 @@ 
 		// if you return something else than False, it is treated as an error-msg and
 		// displayed to the user (the prefs are not saved)
 		//
-		if(($error .= Api\Hooks::single(array(
+		if (($error .= Api\Hooks::single(array(
 				'location' => 'verify_settings',
 				'prefs'    => &$repository[$appname],
 				'type'     => $type,
@@ -273,7 +273,7 @@ 
 			return $error;
 		}
 
-		if (!$only_verify) $GLOBALS['egw']->preferences->save_repository(True,$type);
+		if (!$only_verify) $GLOBALS['egw']->preferences->save_repository(True, $type);
 
 		// certain common prefs (language, template, ...) require the session to be re-created
 		if ($appname == 'common' && !$only_verify)
@@ -309,19 +309,19 @@ 
 		$sel_options = $readonlys = $content = $tabs = array();
 		// disable all but first tab and name current tab "tab1", for apps not using sections
 		$tab = 'tab1';
-		foreach($this->settings as $setting)
+		foreach ($this->settings as $setting)
 		{
 			if (!is_array($setting)) continue;
 			if ($type != 'forced' && (string)$GLOBALS['egw']->preferences->forced[$appname][$setting['name']] !== '')
 			{
-				continue;	// forced preferences are not displayed, unless we edit them
+				continue; // forced preferences are not displayed, unless we edit them
 			}
 			$types[$setting['name']] = $old_type = $setting['type'];
 
-			switch($old_type)
+			switch ($old_type)
 			{
 				case 'section':
-					$tab = 'tab'.(1+count($tabs));
+					$tab = 'tab'.(1 + count($tabs));
 					$tabs[] = array(
 						'id' => $tab,
 						'template' => 'preferences.settings.tab1',
@@ -336,24 +336,24 @@ 
 					if (is_array($setting['values'])) $vars += $setting['values'];
 					$GLOBALS['egw']->preferences->{$attribute}[$appname][$setting['name']] =
 						$GLOBALS['egw']->preferences->lang_notify($GLOBALS['egw']->preferences->{$attribute}[$appname][$setting['name']], $vars);
-					$types[$setting['name']] = $vars;	// store vars for re-translation, instead type "notify"
+					$types[$setting['name']] = $vars; // store vars for re-translation, instead type "notify"
 					if ($setting['help'] && ($setting['run_lang'] || !isset($setting['run_lang'])))
 					{
 						$setting['help'] = lang($setting['help']);
 					}
 					$setting['help'] .= '<p><b>'.lang('Substitutions and their meanings:').'</b>';
-					foreach($vars as $var => $var_help)
+					foreach ($vars as $var => $var_help)
 					{
 						$lname = ($lname = lang($var)) == $var.'*' ? $var : $lname;
 						$setting['help'] .= "<br>\n".'<b>$$'.$lname.'$$</b>: '.$var_help;
 					}
 					$setting['help'] .= "</p>\n";
-					$setting['run_lang'] = false;	// already done now
+					$setting['run_lang'] = false; // already done now
 					// handle as textarea
 				case 'textarea':
 					$setting['type'] = is_a($tpl, 'etemplate') ? 'textarea' : 'textbox';
 					$tpl->setElementAttribute($tab.'['.$setting['name'].']', 'multiline', 'true');
-					$tpl->setElementAttribute($tab. '[' . $setting['name'] . ']','width', '99%' );
+					$tpl->setElementAttribute($tab.'['.$setting['name'].']', 'width', '99%');
 					// anyway setting via css: width: 99%, height: 5em
 					// for old eT use size attribute
 					if (is_a($tpl, 'etemplate') && (!empty($setting['cols']) || !empty($setting['rows'])))
@@ -375,7 +375,7 @@ 
 				case 'multiselect':
 					$setting['type'] = 'select';
 					$tpl->setElementAttribute($tab.'['.$setting['name'].']', 'rows', 5);
-					if (!isset($setting['size'])) $setting['size'] = '5';	// old eT
+					if (!isset($setting['size'])) $setting['size'] = '5'; // old eT
 					break;
 				case 'color':
 					$setting['type'] = 'colorpicker';
@@ -383,7 +383,7 @@ 
 				case 'date-duration':
 					if (!isset($setting['size'])) $setting['size'] = 'm,dhm,24,1';
 					$attrs = explode(',', $setting['size']);
-					foreach(array("data_format","display_format", "hours_per_day", "empty_not_0", "short_labels") as $n => $name)
+					foreach (array("data_format", "display_format", "hours_per_day", "empty_not_0", "short_labels") as $n => $name)
 					{
 						if ((string)$attrs[$n] !== '') $tpl->setElementAttribute($tab.'['.$setting['name'].']', $name, $attrs[$n]);
 					}
@@ -394,17 +394,17 @@ 
 			{
 				if ($old_type != 'multiselect' && $old_type != 'notify')
 				{
-					switch($type)
+					switch ($type)
 					{
 						case 'user':
-							$setting['values'] = array('' => lang('Use default'))+$setting['values'];
+							$setting['values'] = array('' => lang('Use default')) + $setting['values'];
 							break;
 						case 'default':
 						case 'group':
-							$setting['values'] = array('' => lang('No default'))+$setting['values'];
+							$setting['values'] = array('' => lang('No default')) + $setting['values'];
 							break;
 						case 'forced';
-							$setting['values'] = array('**NULL**' => lang('Users choice'))+$setting['values'];
+							$setting['values'] = array('**NULL**' => lang('Users choice')) + $setting['values'];
 							break;
 					}
 				}
@@ -413,8 +413,7 @@ 
 			if ($type == 'user')
 			{
 				$default = $GLOBALS['egw']->preferences->group[$appname][$setting['name']] ?
-					$GLOBALS['egw']->preferences->group[$appname][$setting['name']] :
-					$GLOBALS['egw']->preferences->default[$appname][$setting['name']];
+					$GLOBALS['egw']->preferences->group[$appname][$setting['name']] : $GLOBALS['egw']->preferences->default[$appname][$setting['name']];
 
 				// replace default value(s) for selectboxes with selectbox labels
 				if (isset($setting['values']) && is_array($setting['values']))
@@ -434,7 +433,7 @@ 
 				'name' => $setting['name'],
 				'type' => $setting['type'],
 				'label' => preg_replace('|<br[ /]*>|i', "\n", $setting['label']),
-				'help' => lang($setting['help']),	// is html
+				'help' => lang($setting['help']), // is html
 				//'size' => $setting['size'],	// old eT
 				'default' => !empty($default) ? lang('Default').': '.$default : null,
 				'onchange' => $setting['onchange'],
@@ -460,7 +459,7 @@ 
 
 		$content['appname'] = $appname;
 		$sel_options['appname'] = array();
-		foreach(Api\Hooks::implemented('settings') as $app)
+		foreach (Api\Hooks::implemented('settings') as $app)
 		{
 			if ($app != 'preferences' && $GLOBALS['egw_info']['user']['apps'][$app])
 			{
@@ -486,7 +485,7 @@ 
 				$user_apps = $GLOBALS['egw']->acl->get_user_applications($id);
 				$sel_options['appname'] = array_intersect_key($sel_options['appname'], $user_apps);
 			}
-			foreach($GLOBALS['egw']->accounts->search(array('type' => 'groups', 'order' => 'account_lid')) as $account_id => $group)
+			foreach ($GLOBALS['egw']->accounts->search(array('type' => 'groups', 'order' => 'account_lid')) as $account_id => $group)
 			{
 				$sel_options['type']['group:'.$account_id] = lang('Preferences').' '.Api\Accounts::format_username($group['account_lid'], '', '', $account_id);
 			}
@@ -509,7 +508,7 @@ 
 	 * @param boolean $lang =true
 	 * @return string comma-separated and translated labels
 	 */
-	protected static function get_default_label($default, array $values, $lang=true)
+	protected static function get_default_label($default, array $values, $lang = true)
 	{
 		// explode comma-separated multiple default values
 		if (!is_array($default) && !isset($values[$default]) && strpos($default, ',') !== false)
@@ -520,7 +519,7 @@ 
 		{
 			$labels = (array)$default;
 		}
-		foreach($labels as &$def)
+		foreach ($labels as &$def)
 		{
 			if (isset($values[$def]))
 			{
@@ -528,7 +527,7 @@ 
 			}
 			else	// value could be in an optgroup
 			{
-				foreach($values as $value)
+				foreach ($values as $value)
 				{
 					if (is_array($value) && !isset($value['label']) && isset($value[$def]))
 					{
@@ -554,16 +553,16 @@ 
 	 * @param int|string $account_id =null account_id for user or group prefs, or "forced" or "default"
 	 * @return boolean
 	 */
-	protected function call_hook($appname, $type='user', $account_id=null)
+	protected function call_hook($appname, $type = 'user', $account_id = null)
 	{
 		$this->appname = $appname == 'common' ? 'preferences' : $appname;
 
 		// Set framework here to make sure we get the right settings for user's [newly] selected template
 		$GLOBALS['egw_info']['server']['template_set'] = $GLOBALS['egw']->preferences->data['common']['template_set'];
 		Api\Translation::add_app($this->appname);
-		if($this->appname != 'preferences')
+		if ($this->appname != 'preferences')
 		{
-			Api\Translation::add_app('preferences');	// we need the prefs translations too
+			Api\Translation::add_app('preferences'); // we need the prefs translations too
 		}
 
 		// make type available, to hooks from Egw\Applications can use it, eg. activesync
@@ -572,7 +571,7 @@ 
 			'type' => $type,
 			'account_id' => $account_id,
 		);
-		$GLOBALS['type'] = $type;	// old global variable
+		$GLOBALS['type'] = $type; // old global variable
 
 		// calling app specific settings hook
 		$settings = Api\Hooks::single($hook_data, $this->appname);
@@ -581,30 +580,30 @@ 
 		{
 			$this->settings = array_merge($this->settings, $settings);
 		}
-		elseif(isset($GLOBALS['settings']) && is_array($GLOBALS['settings']) && $GLOBALS['settings'])
+		elseif (isset($GLOBALS['settings']) && is_array($GLOBALS['settings']) && $GLOBALS['settings'])
 		{
 			$this->settings = array_merge($this->settings, $GLOBALS['settings']);
 		}
 		else
 		{
-			return False;	// no settings returned
+			return False; // no settings returned
 		}
 
 		// calling settings hook all apps can answer (for a specific app)
 		$hook_data['location'] = 'settings_'.$this->appname;
-		foreach(Api\Hooks::process($hook_data, $this->appname,true) as $settings)
+		foreach (Api\Hooks::process($hook_data, $this->appname, true) as $settings)
 		{
 			if (isset($settings) && is_array($settings) && $settings)
 			{
-				$this->settings = array_merge($this->settings,$settings);
+				$this->settings = array_merge($this->settings, $settings);
 			}
 		}
 		/* Remove ui-only settings */
-		if($this->xmlrpc)
+		if ($this->xmlrpc)
 		{
-			foreach($this->settings as $key => $valarray)
+			foreach ($this->settings as $key => $valarray)
 			{
-				if(!$valarray['xmlrpc'])
+				if (!$valarray['xmlrpc'])
 				{
 					unset($this->settings[$key]);
 				}
@@ -616,10 +615,10 @@ 
 			 This is not handled by the hooks class and is only valid if not using xml-rpc.
 			*/
 			$tmpl_settings = EGW_SERVER_ROOT.$GLOBALS['egw']->framework->template_dir.'/hook_settings.inc.php';
-			if($this->appname == 'preferences' && file_exists($tmpl_settings))
+			if ($this->appname == 'preferences' && file_exists($tmpl_settings))
 			{
 				include($tmpl_settings);
-				$this->settings = array_merge($this->settings,$GLOBALS['settings']);
+				$this->settings = array_merge($this->settings, $GLOBALS['settings']);
 			}
 		}
 		// check if we have a default/forced value from the settings hook,
@@ -640,9 +639,9 @@ 
 		}
 		if ($need_update)
 		{
-			$GLOBALS['egw']->preferences->save_repository(false,'default',true);
+			$GLOBALS['egw']->preferences->save_repository(false, 'default', true);
 		}
-		if($this->debug)
+		if ($this->debug)
 		{
 			_debug_array($this->settings);
 		}

Braces +67 added lines, -16 removed lines

@@ -164,9 +164,15 @@ 
 		$data = $this->get_content($appname, $type, $sel_options, $readonlys, $preserve['types'], $tpl);
 		$preserve['appname'] = $preserve['old_appname'] = $data['appname'];
 		$preserve['type'] = $preserve['old_type'] = $data['type'];
-		if (isset($old_tab)) $data['tabs'] = $old_tab;
+		if (isset($old_tab))
+		{
+			$data['tabs'] = $old_tab;
+		}
 
-		if ($msg) Framework::message($msg, $msg_type ? $msg_type : 'error');
+		if ($msg)
+		{
+			Framework::message($msg, $msg_type ? $msg_type : 'error');
+		}
 
 		$tpl->exec('preferences.preferences_settings.index', $data, $sel_options, $readonlys, $preserve, 2);
 	}
@@ -195,7 +201,10 @@ 
 			switch((string)$types[$var])
 			{
 				case 'password':	// dont write empty password-fields
-					if (empty($value)) continue 2;
+					if (empty($value))
+					{
+						continue 2;
+					}
 					break;
 				case 'vfs_file':
 				case 'vfs_dir':
@@ -209,7 +218,10 @@ 
 						{
 							unset($prefs[$var]);
 							// need to call preferences::delete, to also set affective prefs!
-							if (!$only_verify) $GLOBALS['egw']->preferences->delete($appname, $var, $type);
+							if (!$only_verify)
+							{
+								$GLOBALS['egw']->preferences->delete($appname, $var, $type);
+							}
 							continue 2;
 						}
 					}
@@ -226,7 +238,11 @@ 
 						// to still allow space or comma in dirnames, we also use the trailing slash of all pathes to split
 						foreach($types[$var] == 'vfs_dir' ? array($value) : preg_split('/[,\s]+\//', $value) as $n => $dir)
 						{
-							if ($n) $dir = '/'.$dir;	// re-adding trailing slash removed by split
+							if ($n)
+							{
+								$dir = '/'.$dir;
+							}
+							// re-adding trailing slash removed by split
 							if ($dir[0] != '/' || !Vfs::stat($dir) || !Vfs::is_dir($dir))
 							{
 								$error .= ($error ? ' ' : '').lang('%1 is no existing vfs directory!',$dir);
@@ -241,19 +257,29 @@ 
 
 			if (isset($value) && $value !== '' && $value !== '**NULL**' && $value !== array())
 			{
-				if (is_array($value)) $value = implode(',',$value);	// multiselect
+				if (is_array($value))
+				{
+					$value = implode(',',$value);
+				}
+				// multiselect
 
 				$prefs[$var] = $value;
 
 				// need to call preferences::add, to also set affective prefs!
-				if (!$only_verify) $GLOBALS['egw']->preferences->add($appname, $var, $prefs[$var], $type);
+				if (!$only_verify)
+				{
+					$GLOBALS['egw']->preferences->add($appname, $var, $prefs[$var], $type);
+				}
 			}
 			else
 			{
 				unset($prefs[$var]);
 
 				// need to call preferences::delete, to also set affective prefs!
-				if (!$only_verify) $GLOBALS['egw']->preferences->delete($appname, $var, $type);
+				if (!$only_verify)
+				{
+					$GLOBALS['egw']->preferences->delete($appname, $var, $type);
+				}
 			}
 		}
 
@@ -273,7 +299,10 @@ 
 			return $error;
 		}
 
-		if (!$only_verify) $GLOBALS['egw']->preferences->save_repository(True,$type);
+		if (!$only_verify)
+		{
+			$GLOBALS['egw']->preferences->save_repository(True,$type);
+		}
 
 		// certain common prefs (language, template, ...) require the session to be re-created
 		if ($appname == 'common' && !$only_verify)
@@ -311,7 +340,10 @@ 
 		$tab = 'tab1';
 		foreach($this->settings as $setting)
 		{
-			if (!is_array($setting)) continue;
+			if (!is_array($setting))
+			{
+				continue;
+			}
 			if ($type != 'forced' && (string)$GLOBALS['egw']->preferences->forced[$appname][$setting['name']] !== '')
 			{
 				continue;	// forced preferences are not displayed, unless we edit them
@@ -333,7 +365,10 @@ 
 
 				case 'notify':
 					$vars = $GLOBALS['egw']->preferences->vars;
-					if (is_array($setting['values'])) $vars += $setting['values'];
+					if (is_array($setting['values']))
+					{
+						$vars += $setting['values'];
+					}
 					$GLOBALS['egw']->preferences->{$attribute}[$appname][$setting['name']] =
 						$GLOBALS['egw']->preferences->lang_notify($GLOBALS['egw']->preferences->{$attribute}[$appname][$setting['name']], $vars);
 					$types[$setting['name']] = $vars;	// store vars for re-translation, instead type "notify"
@@ -375,17 +410,27 @@ 
 				case 'multiselect':
 					$setting['type'] = 'select';
 					$tpl->setElementAttribute($tab.'['.$setting['name'].']', 'rows', 5);
-					if (!isset($setting['size'])) $setting['size'] = '5';	// old eT
+					if (!isset($setting['size']))
+					{
+						$setting['size'] = '5';
+					}
+					// old eT
 					break;
 				case 'color':
 					$setting['type'] = 'colorpicker';
 					break;
 				case 'date-duration':
-					if (!isset($setting['size'])) $setting['size'] = 'm,dhm,24,1';
+					if (!isset($setting['size']))
+					{
+						$setting['size'] = 'm,dhm,24,1';
+					}
 					$attrs = explode(',', $setting['size']);
 					foreach(array("data_format","display_format", "hours_per_day", "empty_not_0", "short_labels") as $n => $name)
 					{
-						if ((string)$attrs[$n] !== '') $tpl->setElementAttribute($tab.'['.$setting['name'].']', $name, $attrs[$n]);
+						if ((string)$attrs[$n] !== '')
+						{
+							$tpl->setElementAttribute($tab.'['.$setting['name'].']', $name, $attrs[$n]);
+						}
 					}
 					break;
 			}
@@ -421,10 +466,13 @@ 
 				{
 					$default = self::get_default_label($default, $setting['values']);
 				}
-				if (is_array($types[$setting['name']]))	// translate the substitution names
+				if (is_array($types[$setting['name']]))
+				{
+					// translate the substitution names
 				{
 					$default = $GLOBALS['egw']->preferences->lang_notify($default, $types[$setting['name']]);
 				}
+				}
 			}
 			if ($setting['help'] && ($setting['run_lang'] || !isset($setting['run_lang'])))
 			{
@@ -537,7 +585,10 @@ 
 					}
 				}
 			}
-			if ($lang) $def = lang($def);
+			if ($lang)
+			{
+				$def = lang($def);
+			}
 		}
 		$label = implode(', ', $labels);
 		//error_log(__METHOD__."(".array2string($default).', '.array2string($values).") returning $label");

redirect.php

Doc Comments +4 added lines

@@ -24,6 +24,10 @@
 
 	if(!function_exists('html_entity_decode'))
 	{
+
+		/**
+		 * @param integer $quote_style
+		 */
 		function html_entity_decode($given_html, $quote_style = ENT_QUOTES)
 		{
 			$trans_table = array_flip(get_html_translation_table( HTML_SPECIALCHARS, $quote_style));

Spacing +9 added lines, -9 removed lines

@@ -22,11 +22,11 @@ 
 	  "<a href=\"$webserverURL/redirect.php?go=".htmlentities(urlencode('http://www.egroupware.org')).'">'
 	*/
 
-	if(!function_exists('html_entity_decode'))
+	if (!function_exists('html_entity_decode'))
 	{
 		function html_entity_decode($given_html, $quote_style = ENT_QUOTES)
 		{
-			$trans_table = array_flip(get_html_translation_table( HTML_SPECIALCHARS, $quote_style));
+			$trans_table = array_flip(get_html_translation_table(HTML_SPECIALCHARS, $quote_style));
 			$trans_table['&#39;'] = "'";
 			return(strtr($given_html, $trans_table));
 		}
@@ -45,17 +45,17 @@ 
 
 	/* Only allow redirects from inside this eGroupware installation. */
 	$valid_referer = array();
-	$path = preg_replace('/\/[^\/]*$/','',$_SERVER['PHP_SELF']) . '/';
+	$path = preg_replace('/\/[^\/]*$/', '', $_SERVER['PHP_SELF']).'/';
 	array_push($valid_referer, $path);
-	array_push($valid_referer, ($_SERVER['HTTPS'] ? 'https://' : 'http://') . $_SERVER['SERVER_ADDR'] . $path);
-	array_push($valid_referer, ($_SERVER['HTTPS'] ? 'https://' : 'http://') . $_SERVER['SERVER_NAME'] . $path);
+	array_push($valid_referer, ($_SERVER['HTTPS'] ? 'https://' : 'http://').$_SERVER['SERVER_ADDR'].$path);
+	array_push($valid_referer, ($_SERVER['HTTPS'] ? 'https://' : 'http://').$_SERVER['SERVER_NAME'].$path);
 
 	$referrer = trim($_SERVER['HTTP_REFERER']);
 	if ((!isset($_SERVER['HTTP_REFERER'])) || (empty($referrer)))
 	{
 		echo "Only usable from within eGroupware.\n";
 	}
-	else if($_GET['go'])
+	else if ($_GET['go'])
 	{
 		$allow = false;
 		foreach ($valid_referer as $urlRoot)
@@ -69,15 +69,15 @@ 
 		}
 		if ($allow)
 		{
-			$url= html_entity_decode(urldecode($_GET['go']));
+			$url = html_entity_decode(urldecode($_GET['go']));
 			unset($_GET['go']);
 			/* Only add "&" if there is something to append. */
 			if (!empty($_GET))
 			{
-				$url=$url."&".http_build_query($_GET);
+				$url = $url."&".http_build_query($_GET);
 			}
 
-			Header('Location: ' . html_entity_decode(urldecode($url)));
+			Header('Location: '.html_entity_decode(urldecode($url)));
 			exit;
 		}
 		else

resources/inc/class.resources_acl_bo.inc.php

Doc Comments +1 added lines, -2 removed lines

@@ -43,7 +43,6 @@ 
 	/**
 	 * Constructor
 	 *
-	 * @param int $user=null account_id of user whos rights to return, or null for current user
 	 * @param boolean $session
 	 */
 	function __construct($session=False, $user=null)
@@ -77,7 +76,7 @@ 
 	*
 	* @author Cornelius Weiss <egw@von-und-zu-weiss.de>
 	* @param int $perm_type one of Acl::READ, Acl::ADD, Acl::EDIT, Acl::DELETE, self::DIRECT_BOOKING
-	* @param int $parent_id=0 cat_id of parent to return only children of that category
+	* @param int $parent_id cat_id of parent to return only children of that category
 	* @return array cat_id => cat_name
 	* TODO mark subcats and so on!
 	*/

Indentation +25 added lines, -25 removed lines

@@ -73,14 +73,14 @@ 
 	}
 
 	/**
-	* get list of cats where current user has given rights
-	*
-	* @author Cornelius Weiss <egw@von-und-zu-weiss.de>
-	* @param int $perm_type one of Acl::READ, Acl::ADD, Acl::EDIT, Acl::DELETE, self::DIRECT_BOOKING
-	* @param int $parent_id=0 cat_id of parent to return only children of that category
-	* @return array cat_id => cat_name
-	* TODO mark subcats and so on!
-	*/
+	 * get list of cats where current user has given rights
+	 *
+	 * @author Cornelius Weiss <egw@von-und-zu-weiss.de>
+	 * @param int $perm_type one of Acl::READ, Acl::ADD, Acl::EDIT, Acl::DELETE, self::DIRECT_BOOKING
+	 * @param int $parent_id=0 cat_id of parent to return only children of that category
+	 * @return array cat_id => cat_name
+	 * TODO mark subcats and so on!
+	 */
 	function get_cats($perm_type,$parent_id=0)
 	{
 		$cats = $this->egw_cats->return_sorted_array(0,false,'','','',true,$parent_id);
@@ -105,24 +105,24 @@ 
 
 
 	/**
-	* gets name of category
-	*
-	* @author Lukas Weiss <wnz.gh05t@users.sourceforge.net>
-	* @param int $cat_id
-	* @return mixed name of category
-	*/
+	 * gets name of category
+	 *
+	 * @author Lukas Weiss <wnz.gh05t@users.sourceforge.net>
+	 * @param int $cat_id
+	 * @return mixed name of category
+	 */
 	static public function get_cat_name($cat_id)
 	{
 		return $GLOBALS['egw']->categories->id2name($cat_id);
 	}
 
 	/**
-	* gets userid of admin for given category
-	*
-	* @author Cornelius Weiss <egw@von-und-zu-weiss.de>
-	* @param int $cat_id
-	* @return int userid of cat admin
-	*/
+	 * gets userid of admin for given category
+	 *
+	 * @author Cornelius Weiss <egw@von-und-zu-weiss.de>
+	 * @param int $cat_id
+	 * @return int userid of cat admin
+	 */
 	static public function get_cat_admin($cat_id)
 	{
 		$cat_rights = self::get_rights($cat_id);
@@ -194,11 +194,11 @@ 
 	}
 
 	/**
-	* gets all rights from all user for given cat
-	*
-	* @param int $cat_id
-	* @return array userid => right
-	*/
+	 * gets all rights from all user for given cat
+	 *
+	 * @param int $cat_id
+	 * @return array userid => right
+	 */
 	static public function get_rights($cat_id)
 	{
 		return $GLOBALS['egw']->acl->get_all_rights('L'.$cat_id,'resources');

Braces +12 added lines, -4 removed lines

@@ -85,9 +85,13 @@ 
 	{
 		$cats = $this->egw_cats->return_sorted_array(0,false,'','','',true,$parent_id);
 		#_debug_array($cats);
-		if (!is_array($cats)) $cats = array();
+		if (!is_array($cats))
+		{
+			$cats = array();
+		}
 		$perm_cats = array();
-		foreach($cats as $key=>$cat) {
+		foreach($cats as $key=>$cat)
+		{
 			#echo "key:$key"._debug_array($value)."<br>";
 			#_debug_array($cat)."hier<br>";
 			if($this->is_permitted($cat['id'],$perm_type))
@@ -215,14 +219,18 @@ 
 			'order' => $this->order,
 			'limit' => $this->limit,
 		);
-		if($this->debug) { echo '<br>Read:'; _debug_array($data); }
+		if($this->debug)
+		{
+echo '<br>Read:'; _debug_array($data); }
 		Api\Cache::setSession('resources_acl', 'session_data', $data);
 	}
 
 	function read_sessiondata()
 	{
 		$data = Api\Cache::getSession('resources_acl', 'session_data');
-		if($this->debug) { echo '<br>Read:'; _debug_array($data); }
+		if($this->debug)
+		{
+echo '<br>Read:'; _debug_array($data); }
 
 		$this->start  = $data['start'];
 		$this->query  = $data['query'];

Spacing +28 added lines, -28 removed lines

@@ -46,17 +46,17 @@ 
 	 * @param int $user=null account_id of user whos rights to return, or null for current user
 	 * @param boolean $session
 	 */
-	function __construct($session=False, $user=null)
+	function __construct($session = False, $user = null)
 	{
 		$this->egw_cats = new Api\Categories($user, 'resources');
 		$this->debug = False;
 
 		//all this is only needed when called from uiacl.
-		if($session)
+		if ($session)
 		{
 			$this->read_sessiondata();
 			$this->use_session = True;
-			foreach(array('start','query','sort','order') as $var)
+			foreach (array('start', 'query', 'sort', 'order') as $var)
 			{
 				if (isset($_POST[$var]))
 				{
@@ -68,7 +68,7 @@ 
 				}
 			}
 			$this->save_sessiondata();
-			$this->cats = $this->egw_cats->return_sorted_array(0,false,'','','',true);
+			$this->cats = $this->egw_cats->return_sorted_array(0, false, '', '', '', true);
 		}
 	}
 
@@ -81,18 +81,18 @@ 
 	* @return array cat_id => cat_name
 	* TODO mark subcats and so on!
 	*/
-	function get_cats($perm_type,$parent_id=0)
+	function get_cats($perm_type, $parent_id = 0)
 	{
-		$cats = $this->egw_cats->return_sorted_array(0,false,'','','',true,$parent_id);
+		$cats = $this->egw_cats->return_sorted_array(0, false, '', '', '', true, $parent_id);
 		#_debug_array($cats);
 		if (!is_array($cats)) $cats = array();
 		$perm_cats = array();
-		foreach($cats as $key=>$cat) {
+		foreach ($cats as $key=>$cat) {
 			#echo "key:$key"._debug_array($value)."<br>";
 			#_debug_array($cat)."hier<br>";
-			if($this->is_permitted($cat['id'],$perm_type))
+			if ($this->is_permitted($cat['id'], $perm_type))
 			{
-				$s = str_repeat('&nbsp; ',$cat['level']) . stripslashes($cat['name']);
+				$s = str_repeat('&nbsp; ', $cat['level']).stripslashes($cat['name']);
 				if ($cat['app_name'] == 'phpgw' || $cat['owner'] == '-1')
 				{
 					$s .= ' &#9830;';
@@ -100,7 +100,7 @@ 
 				$perm_cats[$cat['id']] = $s;
 			}
 		}
-		return isset($perm_cats)?$perm_cats:array();
+		return isset($perm_cats) ? $perm_cats : array();
 	}
 
 
@@ -128,13 +128,13 @@ 
 		$cat_rights = self::get_rights($cat_id);
 		foreach ($cat_rights as $userid => $right)
 		{
-			if ($right & self::CAT_ADMIN)
+			if ($right&self::CAT_ADMIN)
 			{
 				return $userid;
 			}
 		}
 		// check for an inherited cat admin
-		if (($parent = $GLOBALS['egw']->categories->id2name($cat_id,'parent')))
+		if (($parent = $GLOBALS['egw']->categories->id2name($cat_id, 'parent')))
 		{
 			return self::get_cat_admin($parent);
 		}
@@ -161,10 +161,10 @@ 
 		{
 			if (is_null(self::$resource_acl))
 			{
-				self::$resource_acl = $GLOBALS['egw']->acl->get_all_location_rights($GLOBALS['egw_info']['user']['account_id'],'resources',true);
+				self::$resource_acl = $GLOBALS['egw']->acl->get_all_location_rights($GLOBALS['egw_info']['user']['account_id'], 'resources', true);
 			}
 			self::$permissions[$cat_id] = (int)self::$resource_acl['L'.$cat_id];
-			if (($parent = $GLOBALS['egw']->categories->id2name($cat_id,'parent')))
+			if (($parent = $GLOBALS['egw']->categories->id2name($cat_id, 'parent')))
 			{
 				self::$permissions[$cat_id] |= self::get_permissions($parent);
 			}
@@ -182,7 +182,7 @@ 
 	 * @param int $right
 	 * @return boolean user is permitted or not for right
 	 */
-	static public function is_permitted($cat_id,$right)
+	static public function is_permitted($cat_id, $right)
 	{
 		if (!isset(self::$permissions[$cat_id]))
 		{
@@ -190,7 +190,7 @@ 
 		}
 		//echo "<p>".__METHOD__."($cat_id,$right) = ".self::$permissions[$cat_id]." & $right = ".(self::$permissions[$cat_id] & $right)."</p>\n";
 
-		return (boolean) (self::$permissions[$cat_id] & $right);
+		return (boolean)(self::$permissions[$cat_id]&$right);
 	}
 
 	/**
@@ -201,7 +201,7 @@ 
 	*/
 	static public function get_rights($cat_id)
 	{
-		return $GLOBALS['egw']->acl->get_all_rights('L'.$cat_id,'resources');
+		return $GLOBALS['egw']->acl->get_all_rights('L'.$cat_id, 'resources');
 	}
 
 
@@ -215,14 +215,14 @@ 
 			'order' => $this->order,
 			'limit' => $this->limit,
 		);
-		if($this->debug) { echo '<br>Read:'; _debug_array($data); }
+		if ($this->debug) { echo '<br>Read:'; _debug_array($data); }
 		Api\Cache::setSession('resources_acl', 'session_data', $data);
 	}
 
 	function read_sessiondata()
 	{
 		$data = Api\Cache::getSession('resources_acl', 'session_data');
-		if($this->debug) { echo '<br>Read:'; _debug_array($data); }
+		if ($this->debug) { echo '<br>Read:'; _debug_array($data); }
 
 		$this->start  = $data['start'];
 		$this->query  = $data['query'];
@@ -231,7 +231,7 @@ 
 		$this->limit = $data['limit'];
 	}
 
-	public static function set_rights($cat_id,$read,$write,$calread,$calbook,$admin)
+	public static function set_rights($cat_id, $read, $write, $calread, $calbook, $admin)
 	{
 		// Clear cache
 		unset(self::$permissions[$cat_id]);
@@ -242,19 +242,19 @@ 
 		$calbookcat = $calbook ? $calbook : array();
 		$admincat = $admin ? $admin : array();
 
-		$GLOBALS['egw']->acl->delete_repository('resources','L' . $cat_id,false);
+		$GLOBALS['egw']->acl->delete_repository('resources', 'L'.$cat_id, false);
 
-		foreach(array_unique(array_merge($readcat, $writecat, $calreadcat, $calbookcat, $admincat)) as $account_id)
+		foreach (array_unique(array_merge($readcat, $writecat, $calreadcat, $calbookcat, $admincat)) as $account_id)
 		{
 			$rights = false;
-			$rights = in_array($account_id,$readcat) ? ($rights | Acl::READ) : false;
-			$rights = in_array($account_id,$writecat) ? ($rights | Acl::READ | Acl::ADD | Acl::EDIT | Acl::DELETE): $rights;
-			$rights = in_array($account_id,$calreadcat) ? ($rights | self::CAL_READ) : $rights;
-			$rights = in_array($account_id,$calbookcat) ? ($rights | self::DIRECT_BOOKING | self::CAL_READ) : $rights;
-			$rights = in_array($account_id,$admincat) ? ($rights = 511) : $rights;
+			$rights = in_array($account_id, $readcat) ? ($rights|Acl::READ) : false;
+			$rights = in_array($account_id, $writecat) ? ($rights|Acl::READ|Acl::ADD|Acl::EDIT|Acl::DELETE) : $rights;
+			$rights = in_array($account_id, $calreadcat) ? ($rights|self::CAL_READ) : $rights;
+			$rights = in_array($account_id, $calbookcat) ? ($rights|self::DIRECT_BOOKING|self::CAL_READ) : $rights;
+			$rights = in_array($account_id, $admincat) ? ($rights = 511) : $rights;
 			if ($rights)
 			{
-				$GLOBALS['egw']->acl->add_repository('resources','L'.$cat_id,$account_id,$rights);
+				$GLOBALS['egw']->acl->add_repository('resources', 'L'.$cat_id, $account_id, $rights);
 			}
 		}
 	}

resources/inc/class.resources_bo.inc.php

Doc Comments +3 added lines, -3 removed lines

@@ -79,7 +79,7 @@ 
 	/**
 	 * Constructor
 	 *
-	 * @param int $user=null account_id of user to use for Acl, default current user
+	 * @param integer $user
 	 */
 	function __construct($user=null)
 	{
@@ -774,8 +774,8 @@ 
 	 *
 	 * Cornelius Weiss <egw@von-und-zu-weiss.de>
 	 * @param array $file array with key => value
-	 * @param int $resource_id
-	 * @return mixed string with msg if somthing went wrong; nothing if all right
+	 * @param int $resouce_id
+	 * @return string|null string with msg if somthing went wrong; nothing if all right
 	 */
 	function save_picture($file,$resouce_id)
 	{

Indentation +5 added lines, -5 removed lines

@@ -368,10 +368,10 @@ 
 		}
 
 		// Check for restore of deleted, restore held links
-                if($old && $old['deleted'] && !$resource['deleted'])
-                {
-                        Link::restore('resources', $resource['res_id']);
-                }
+				if($old && $old['deleted'] && !$resource['deleted'])
+				{
+						Link::restore('resources', $resource['res_id']);
+				}
 
 		// delete old pictures
 		if($resource['picture_src'] != 'own_src')
@@ -695,7 +695,7 @@ 
 						$list[$id] = ' ('.lang('conflict').') '.$resource['name']. ($resource['short_description'] ? ', ['.$resource['short_description'].']':'');
 					}
 				} else {
-				        $list[$id] = $resource['name']. ($resource['short_description'] ? ', ['.$resource['short_description'].']':'');
+						$list[$id] = $resource['name']. ($resource['short_description'] ? ', ['.$resource['short_description'].']':'');
 				}
 			}
 		} else {

Braces +84 added lines, -31 removed lines

@@ -105,9 +105,12 @@ 
 		{
 			Api\Cache::setSession('resources', 'index_nm', $query);
 		}
-		if ($query['store_state'])	// request to store state in session and filter in prefs?
+		if ($query['store_state'])
+		{
+			// request to store state in session and filter in prefs?
 		{
 			Api\Cache::setSession('resources',$query['store_state'],$query);
+		}
 			//echo "<p>".__METHOD__."() query[filter]=$query[filter], prefs[resources][filter]={$GLOBALS['egw_info']['user']['preferences']['resources']['filter']}</p>\n";
 			if ($query['filter'] != $GLOBALS['egw_info']['user']['preferences']['resources']['filter'])
 			{
@@ -115,7 +118,10 @@ 
 				$GLOBALS['egw']->preferences->save_repository();
 			}
 		}
-		if ($this->debug) _debug_array($query);
+		if ($this->debug)
+		{
+			_debug_array($query);
+		}
 		$read_onlys = 'res_id,name,short_description,quantity,useable,bookable,buyable,cat_id,location,storage_info';
 
 		$filter = array();
@@ -172,7 +178,8 @@ 
 			$filter['cat_id'] = array_keys($readcats);
 		}
 		// if there is no catfilter -> this means you have no rights, so set the cat filter to null
-		if (!isset($filter['cat_id']) || empty($filter['cat_id'])) {
+		if (!isset($filter['cat_id']) || empty($filter['cat_id']))
+		{
 			$filter['cat_id'] = NUll;
 		}
 
@@ -183,7 +190,10 @@ 
 		$order_by = $query['order'] ? $query['order'].' '. $query['sort'] : '';
 		$start = (int)$query['start'];
 
-		foreach ($filter as $k => $v) $query['col_filter'][$k] = $v;
+		foreach ($filter as $k => $v)
+		{
+			$query['col_filter'][$k] = $v;
+		}
 		$this->so->get_rows($query, $rows, $readonlys, $join, false, false, $extra_cols);
 		$nr = $this->so->total;
 
@@ -212,8 +222,7 @@ 
 			}
 			if (!$this->acl->is_permitted($resource['cat_id'],Acl::DELETE) ||
 				($resource['deleted'] && !$GLOBALS['egw_info']['user']['apps']['admin'] && $config['history'] == 'history')
-			)
-			{
+			) {
 				$readonlys["delete[$resource[res_id]]"] = true;
 				$resource['class'] .= 'no_delete ';
 			}
@@ -251,7 +260,9 @@ 
 				{
 					$resource['accessories'][] = array('acc_id' => $acc_id, 'name' => $this->link_title($acc_id));
 				}
-			} elseif ($resource['accessory_of'] > 0) {
+			}
+			elseif ($resource['accessory_of'] > 0)
+			{
 				$resource['accessory_of_label'] = $this->link_title($resource['accessory_of']);
 			}
 
@@ -500,13 +511,19 @@ 
 	 */
 	function get_acc_list($res_id,$deleted=false)
 	{
-		if($res_id < 1){return;}
+		if($res_id < 1)
+		{
+return;}
 		$data = $this->so->search('','res_id,name,deleted','','','','','',$start,array('accessory_of' => $res_id),'',$need_full_no_count=true);
 		$acc_list = array();
-		if($data) {
+		if($data)
+		{
 			foreach($data as $num => $resource)
 			{
-				if($resource['deleted'] && !$deleted) continue;
+				if($resource['deleted'] && !$deleted)
+				{
+					continue;
+				}
 				$acc_list[$resource['res_id']] = $resource['name'];
 			}
 		}
@@ -522,7 +539,10 @@ 
 	function get_calendar_info($res_id)
 	{
 		//echo "<p>resources_bo::get_calendar_info(".print_r($res_id,true).")</p>\n";
-		if(!is_array($res_id) && $res_id < 1) return;
+		if(!is_array($res_id) && $res_id < 1)
+		{
+			return;
+		}
 
 		$data = $this->so->search(array('res_id' => $res_id),self::TITLE_COLS.',useable');
 		if (!is_array($data))
@@ -598,7 +618,8 @@ 
 			'deleted' => null
 		);
 		$limit = false;
-		if($options['start'] || $options['num_rows']) {
+		if($options['start'] || $options['num_rows'])
+		{
 			$limit = array($options['start'], $options['num_rows']);
 		}
 		if($options['accessory_of'])
@@ -622,11 +643,14 @@ 
 				}
 				$start = new Api\DateTime($cal_info['start']);
 				$startarr= getdate($start->format('ts'));
-				if (isset($cal_info['whole_day']) && $cal_info['whole_day']) {
+				if (isset($cal_info['whole_day']) && $cal_info['whole_day'])
+				{
 					$startarr['hour'] = $startarr['minute'] = 0;
 					$start = new Api\DateTime($startarr);
 					$end = $start->format('ts') + 86399;
-				} else {
+				}
+				else
+				{
 					$start = $start->format('ts');
 					$end = $start + ($cal_info['duration']);
 				}
@@ -638,7 +662,8 @@ 
 					// we only need resources id for the search, but with a 'r' prefix
 					// now we take this loop to store a new resource array indexed with resource id
 					// and as we work for calendar we use only bookable resources
-					if ((isset($resource['bookable'])) && ($resource['bookable'])){
+					if ((isset($resource['bookable'])) && ($resource['bookable']))
+					{
 						$res_info_cache[$resource['res_id']]=$resource;
 						$resource_list[]='r'.$resource['res_id'];
 					}
@@ -654,22 +679,34 @@ 
 				// parse theses overlapping events
 				foreach($overlapping_events as $event)
 				{
-					if ($event['non_blocking']) continue; // ignore non_blocking events
-					if (isset($cal_info['event_id']) && $event['id']==$cal_info['event_id']) {
+					if ($event['non_blocking'])
+					{
+						continue;
+					}
+					// ignore non_blocking events
+					if (isset($cal_info['event_id']) && $event['id']==$cal_info['event_id'])
+					{
 						continue; //ignore this event, it's the current edited event, no conflict by def
 					}
 					// now we are interested only on resources booked by theses events
-					if (isset($event['participants']) && is_array($event['participants'])){
-						foreach($event['participants'] as $part_key => $part_detail){
+					if (isset($event['participants']) && is_array($event['participants']))
+					{
+						foreach($event['participants'] as $part_key => $part_detail)
+						{
 							if ($part_key{0}=='r')
-							{ //now we gatta resource here
+							{
+//now we gatta resource here
 								//need to check the quantity of this resource
 								$resource_id=substr($part_key,1);
 								// if we do not find this resource in our indexed array it's certainly
 								// because it was unset, non bookable maybe
-								if (!isset($res_info_cache[$resource_id])) continue;
+								if (!isset($res_info_cache[$resource_id]))
+								{
+									continue;
+								}
 								// to detect ressources with default to 1 quantity
-								if (!isset($res_info_cache[$resource_id]['useable'])) {
+								if (!isset($res_info_cache[$resource_id]['useable']))
+								{
 									$res_info_cache[$resource_id]['useable'] = 1;
 								}
 								// now decrement this quantity useable
@@ -683,30 +720,40 @@ 
 				}
 			}
 		}
-		if (isset($res_info_cache)) {
+		if (isset($res_info_cache))
+		{
 			$show_conflict= $GLOBALS['egw_info']['user']['preferences']['calendar']['defaultresource_sel'] === 'resources_conflict';
 			// if we have this array indexed on resource id it means non-bookable resource are removed and we are working for calendar
 			// so we'll loop on this one and not $data
-			foreach($res_info_cache as $id => $resource) {
+			foreach($res_info_cache as $id => $resource)
+			{
 				//maybe this resource is reserved
 				if ( ($resource['useable'] < 1) )
 				{
-					if($show_conflict) {
+					if($show_conflict)
+					{
 						$list[$id] = ' ('.lang('conflict').') '.$resource['name']. ($resource['short_description'] ? ', ['.$resource['short_description'].']':'');
 					}
-				} else {
+				}
+				else
+				{
 				        $list[$id] = $resource['name']. ($resource['short_description'] ? ', ['.$resource['short_description'].']':'');
 				}
 			}
-		} else {
+		}
+		else
+		{
 			// we are not working for the calendar, we loop on the initial $data
-			if (is_array($data)) {
+			if (is_array($data))
+			{
 				foreach($data as $num => $resource)
 				{
 					$id=$resource['res_id'];
 					$list[$id] = $resource['name']. ($resource['short_description'] ? ', ['.$resource['short_description'].']':'');
 				}
-			} else {
+			}
+			else
+			{
 				error_log(__METHOD__." No Data found for Resource with id ".$resource['res_id']);
 			}
 		}
@@ -725,7 +772,10 @@ 
 	{
 		if (!is_array($resource))
 		{
-			if (!($resource  = $this->read(array('res_id' => $resource)))) return $resource;
+			if (!($resource  = $this->read(array('res_id' => $resource))))
+			{
+				return $resource;
+			}
 		}
 		elseif (!$this->acl->is_permitted($resource['cat_id'],Acl::READ))
 		{
@@ -816,7 +866,10 @@ 
 	 */
 	function get_picture($resource,$fullsize=false)
 	{
-		if ($resource && !is_array($resource)) $resource = $this->read($resource);
+		if ($resource && !is_array($resource))
+		{
+			$resource = $this->read($resource);
+		}
 
 		switch($resource['picture_src'])
 		{

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -572,7 +572,7 @@
 	 * @param array $options Array of options for the search
 	 *
 	 */
-	function link_query( $pattern, Array &$options = array() )
+	function link_query( $pattern, array &$options = array() )
 	{
 		if (is_array($pattern))
 		{

Spacing +136 added lines, -136 removed lines

@@ -81,7 +81,7 @@ 
 	 *
 	 * @param int $user=null account_id of user to use for Acl, default current user
 	 */
-	function __construct($user=null)
+	function __construct($user = null)
 	{
 		$this->so = new resources_so();
 		$this->acl = new resources_acl_bo($user);
@@ -89,8 +89,8 @@ 
 
 		$this->cal_right_transform = array(
 			resources_acl_bo::CAL_READ	=> Acl::READ,
-			resources_acl_bo::DIRECT_BOOKING 	=> Acl::READ | Acl::ADD | Acl::EDIT | Acl::DELETE,
-			resources_acl_bo::CAT_ADMIN 	=> Acl::READ | Acl::ADD | Acl::EDIT | Acl::DELETE,
+			resources_acl_bo::DIRECT_BOOKING 	=> Acl::READ|Acl::ADD|Acl::EDIT|Acl::DELETE,
+			resources_acl_bo::CAT_ADMIN 	=> Acl::READ|Acl::ADD|Acl::EDIT|Acl::DELETE,
 		);
 	}
 
@@ -99,19 +99,19 @@ 
 	 *
 	 * Cornelius Weiss <egw@von-und-zu-weiss.de>
 	 */
-	function get_rows($query,&$rows,&$readonlys)
+	function get_rows($query, &$rows, &$readonlys)
 	{
-		if(!$query['csv_export'])
+		if (!$query['csv_export'])
 		{
 			Api\Cache::setSession('resources', 'index_nm', $query);
 		}
 		if ($query['store_state'])	// request to store state in session and filter in prefs?
 		{
-			Api\Cache::setSession('resources',$query['store_state'],$query);
+			Api\Cache::setSession('resources', $query['store_state'], $query);
 			//echo "<p>".__METHOD__."() query[filter]=$query[filter], prefs[resources][filter]={$GLOBALS['egw_info']['user']['preferences']['resources']['filter']}</p>\n";
 			if ($query['filter'] != $GLOBALS['egw_info']['user']['preferences']['resources']['filter'])
 			{
-				$GLOBALS['egw']->preferences->add('resources','filter',$query['filter'],'user');
+				$GLOBALS['egw']->preferences->add('resources', 'filter', $query['filter'], 'user');
 				$GLOBALS['egw']->preferences->save_repository();
 			}
 		}
@@ -125,7 +125,7 @@ 
 		// Sub-query to get the count of accessories
 		$acc_join = "LEFT JOIN (SELECT accessory_of AS accessory_id, count(res_id) as acc_count FROM {$this->so->table_name} GROUP BY accessory_of) AS acc ON acc.accessory_id = {$this->so->table_name}.res_id ";
 
-		switch($query['filter2'])
+		switch ($query['filter2'])
 		{
 			case -1:
 				// Resources only
@@ -150,14 +150,14 @@ 
 			default:
 				$filter['accessory_of'] = $query['filter2'];
 		}
-		if($query['filter2'] != self::DELETED)
+		if ($query['filter2'] != self::DELETED)
 		{
 			$filter['deleted'] = null;
 		}
 
 		if ($query['filter'])
 		{
-			if (($children = $this->acl->get_cats(Acl::READ,$query['filter'])))
+			if (($children = $this->acl->get_cats(Acl::READ, $query['filter'])))
 			{
 				$filter['cat_id'] = array_keys($children);
 				$filter['cat_id'][] = $query['filter'];
@@ -180,7 +180,7 @@ 
 		{
 			$filter['bookable'] = true;
 		}
-		$order_by = $query['order'] ? $query['order'].' '. $query['sort'] : '';
+		$order_by = $query['order'] ? $query['order'].' '.$query['sort'] : '';
 		$start = (int)$query['start'];
 
 		foreach ($filter as $k => $v) $query['col_filter'][$k] = $v;
@@ -188,10 +188,10 @@ 
 		$nr = $this->so->total;
 
 		// we are called to serve bookable resources (e.g. calendar-dialog)
-		if($query['show_bookable'])
+		if ($query['show_bookable'])
 		{
 			// This is somehow ugly, i know...
-			foreach((array)$rows as $num => $resource)
+			foreach ((array)$rows as $num => $resource)
 			{
 				$rows[$num]['default_qty'] = 1;
 			}
@@ -200,24 +200,24 @@ 
 		}
 
 		$config = Api\Config::read('resources');
-		foreach($rows as $num => &$resource)
+		foreach ($rows as $num => &$resource)
 		{
-			if (!$this->acl->is_permitted($resource['cat_id'],Acl::EDIT))
+			if (!$this->acl->is_permitted($resource['cat_id'], Acl::EDIT))
 			{
 				$readonlys["edit[$resource[res_id]]"] = true;
 			}
-			elseif($resource['deleted'])
+			elseif ($resource['deleted'])
 			{
 				$resource['class'] .= 'deleted ';
 			}
-			if (!$this->acl->is_permitted($resource['cat_id'],Acl::DELETE) ||
+			if (!$this->acl->is_permitted($resource['cat_id'], Acl::DELETE) ||
 				($resource['deleted'] && !$GLOBALS['egw_info']['user']['apps']['admin'] && $config['history'] == 'history')
 			)
 			{
 				$readonlys["delete[$resource[res_id]]"] = true;
 				$resource['class'] .= 'no_delete ';
 			}
-			if ((!$this->acl->is_permitted($resource['cat_id'],Acl::ADD)) ||
+			if ((!$this->acl->is_permitted($resource['cat_id'], Acl::ADD)) ||
 				// Allow new accessory action when viewing accessories of a certain resource
 				$query['filter2'] <= 0 && $resource['accessory_of'] != -1)
 			{
@@ -231,7 +231,7 @@ 
 				$resource['class'] .= 'no_book ';
 				$resource['class'] .= 'no_view_calendar ';
 			}
-			if(!$this->acl->is_permitted($resource['cat_id'],resources_acl_bo::CAL_READ))
+			if (!$this->acl->is_permitted($resource['cat_id'], resources_acl_bo::CAL_READ))
 			{
 				$readonlys["calendar[$resource[res_id]]"] = true;
 				$resource['class'] .= 'no_view_calendar ';
@@ -242,12 +242,12 @@ 
 				$resource['class'] .= 'no_buy ';
 			}
 			$readonlys["view_acc[{$resource['res_id']}]"] = ($resource['acc_count'] == 0);
-			$resource['class'] .= ($resource['accessory_of']==-1 ? 'resource ' : 'accessory ');
-			if($resource['acc_count'])
+			$resource['class'] .= ($resource['accessory_of'] == -1 ? 'resource ' : 'accessory ');
+			if ($resource['acc_count'])
 			{
 				$resource['class'] .= 'hasAccessories ';
-				$accessories = $this->get_acc_list($resource['res_id'],$query['filter2']==self::DELETED);
-				foreach($accessories as $acc_id => $acc_name)
+				$accessories = $this->get_acc_list($resource['res_id'], $query['filter2'] == self::DELETED);
+				foreach ($accessories as $acc_id => $acc_name)
 				{
 					$resource['accessories'][] = array('acc_id' => $acc_id, 'name' => $this->link_title($acc_id));
 				}
@@ -255,7 +255,7 @@ 
 				$resource['accessory_of_label'] = $this->link_title($resource['accessory_of']);
 			}
 
-			if($resource['deleted'])
+			if ($resource['deleted'])
 			{
 				$rows[$num]['picture_thumb'] = 'deleted';
 			}
@@ -276,7 +276,7 @@ 
 			$rows[$num]['admin'] = $this->acl->get_cat_admin($resource['cat_id']);
 		}
 
-		if(!Api\Storage\Customfields::get('resources'))
+		if (!Api\Storage\Customfields::get('resources'))
 		{
 			$rows['no_customfields'] = true;
 		}
@@ -294,11 +294,11 @@ 
 	{
 		if (!($data = $this->so->read(array('res_id' => $res_id))))
 		{
-			return null;	// not found
+			return null; // not found
 		}
-		if (!$this->acl->is_permitted($data['cat_id'],Acl::READ))
+		if (!$this->acl->is_permitted($data['cat_id'], Acl::READ))
 		{
-			return false;	// permission denied
+			return false; // permission denied
 		}
 		return $data;
 	}
@@ -312,35 +312,35 @@ 
 	 */
 	function save($resource)
 	{
-		if(!$this->acl->is_permitted($resource['cat_id'],Acl::EDIT))
+		if (!$this->acl->is_permitted($resource['cat_id'], Acl::EDIT))
 		{
 			return lang('You are not permitted to edit this resource!');
 		}
 		$old = array();
 		// we need an id to save pictures and make links...
-		if(!$resource['res_id'])
+		if (!$resource['res_id'])
 		{
 			$resource['res_owner'] = $GLOBALS['egw_info']['user']['account_id'];
-			$resource['res_created'] = Api\DateTime::server2user(time(),'ts');
+			$resource['res_created'] = Api\DateTime::server2user(time(), 'ts');
 			$resource['res_id'] = $this->so->save($resource);
 		}
 		else
 		{
 			$resource['res_modifier'] = $GLOBALS['egw_info']['user']['account_id'];
-			$resource['res_modified'] = Api\DateTime::server2user(time(),'ts');
+			$resource['res_modified'] = Api\DateTime::server2user(time(), 'ts');
 			$old = $this->read($resource['res_id']);
 		}
 
 		switch ($resource['picture_src'])
 		{
 			case 'own_src':
-				if($resource['own_file']['size'] > 0)
+				if ($resource['own_file']['size'] > 0)
 				{
-					$msg = $this->save_picture($resource['own_file'],$resource['res_id']);
+					$msg = $this->save_picture($resource['own_file'], $resource['res_id']);
 					unset($resource['own_file']);
 					break;
 				}
-				elseif(@egw_vfs::stat('/apps/resources/'.$resource['res_id'].'/'.self::PICTURE_NAME))
+				elseif (@egw_vfs::stat('/apps/resources/'.$resource['res_id'].'/'.self::PICTURE_NAME))
 				{
 					break;
 				}
@@ -351,10 +351,10 @@ 
 				$resource['picture_src'] = 'gen_src';
 				break;
 			default:
-				if($resource['own_file']['size'] > 0)
+				if ($resource['own_file']['size'] > 0)
 				{
 					$resource['picture_src'] = 'own_src';
-					$msg = $this->save_picture($resource['own_file'],$resource['res_id']);
+					$msg = $this->save_picture($resource['own_file'], $resource['res_id']);
 				}
 				else
 				{
@@ -362,40 +362,40 @@ 
 				}
 		}
 		// somthing went wrong on saving own picture
-		if($msg)
+		if ($msg)
 		{
 			return $msg;
 		}
 
 		// Check for restore of deleted, restore held links
-                if($old && $old['deleted'] && !$resource['deleted'])
+                if ($old && $old['deleted'] && !$resource['deleted'])
                 {
                         Link::restore('resources', $resource['res_id']);
                 }
 
 		// delete old pictures
-		if($resource['picture_src'] != 'own_src')
+		if ($resource['picture_src'] != 'own_src')
 		{
 			$this->remove_picture($resource['res_id']);
 		}
 
 		// Update link title
-		Link::notify_update('resources',$resource['res_id'], $resource);
+		Link::notify_update('resources', $resource['res_id'], $resource);
 		// save links
-		if(is_array($resource['link_to']['to_id']))
+		if (is_array($resource['link_to']['to_id']))
 		{
-			Link::link('resources',$resource['res_id'],$resource['link_to']['to_id']);
+			Link::link('resources', $resource['res_id'], $resource['link_to']['to_id']);
 		}
-		if($resource['accessory_of'] != $old['accessory_of'])
+		if ($resource['accessory_of'] != $old['accessory_of'])
 		{
-			Link::unlink(0,'resources',$resource['res_id'],'','resources',$old['accessory_of']);
+			Link::unlink(0, 'resources', $resource['res_id'], '', 'resources', $old['accessory_of']);
 
 			// Check for resource changing to accessory - move its accessories to resource
-			if($old['accessory_of'] == -1 && ($accessories = $this->get_acc_list($resource['res_id'])))
+			if ($old['accessory_of'] == -1 && ($accessories = $this->get_acc_list($resource['res_id'])))
 			{
-				foreach($accessories as $accessory => $name)
+				foreach ($accessories as $accessory => $name)
 				{
-					Link::unlink(0,'resources',$accessory,'','resources',$resource['res_id']);
+					Link::unlink(0, 'resources', $accessory, '', 'resources', $resource['res_id']);
 					if (($acc = $this->read($accessory)))
 					{
 						$acc['accessory_of'] = -1;
@@ -404,15 +404,15 @@ 
 				}
 			}
 		}
-		if($resource['accessory_of'] != -1)
+		if ($resource['accessory_of'] != -1)
 		{
-			Link::link('resources',$resource['res_id'],'resources',$resource['accessory_of']);
+			Link::link('resources', $resource['res_id'], 'resources', $resource['accessory_of']);
 		}
 
-		if(!empty($resource['res_id']) && $this->so->get_value("cat_id",$resource['res_id']) != $resource['cat_id'] && $resource['accessory_of'] == -1)
+		if (!empty($resource['res_id']) && $this->so->get_value("cat_id", $resource['res_id']) != $resource['cat_id'] && $resource['accessory_of'] == -1)
 		{
 			$accessories = $this->get_acc_list($resource['res_id']);
-			foreach($accessories as $accessory => $name)
+			foreach ($accessories as $accessory => $name)
 			{
 				if (($acc = $this->so->read($accessory)))
 				{
@@ -430,9 +430,9 @@ 
 		{
 			$this->tracking = new resources_tracking();
 		}
-		if ($this->tracking->track($resource,$old,$this->user) === false)
+		if ($this->tracking->track($resource, $old, $this->user) === false)
 		{
-			return implode(', ',$this->tracking->errors);
+			return implode(', ', $this->tracking->errors);
 		}
 
 		return $res_id ? $res_id : lang('Something went wrong by saving resource');
@@ -447,7 +447,7 @@ 
 	 */
 	function delete($res_id)
 	{
-		if(!$this->acl->is_permitted($this->so->get_value('cat_id',$res_id),Acl::DELETE))
+		if (!$this->acl->is_permitted($this->so->get_value('cat_id', $res_id), Acl::DELETE))
 		{
 			return lang('You are not permitted to delete this resource!');
 		}
@@ -462,16 +462,16 @@ 
 		{
 			$old['deleted'] = time();
 			$this->save($old);
-			Link::unlink(0,'resources',$res_id,'','','',true);
+			Link::unlink(0, 'resources', $res_id, '', '', '', true);
 			$accessories = $this->get_acc_list($res_id);
-			foreach($accessories as $acc_id => $name)
+			foreach ($accessories as $acc_id => $name)
 			{
 				// Don't purge already deleted accessories
 				if (($acc = $this->read($acc_id)) && !$acc['deleted'])
 				{
 					$acc['deleted'] = time();
 					$this->save($acc);
-					Link::unlink(0,'resources',$acc_id,'','','',true);
+					Link::unlink(0, 'resources', $acc_id, '', '', '', true);
 				}
 			}
 			return false;
@@ -479,18 +479,18 @@ 
 		elseif ($this->so->delete(array('res_id'=>$res_id)))
 		{
 			$accessories = $this->get_acc_list($res_id, true);
-			foreach($accessories as $acc_id => $name)
+			foreach ($accessories as $acc_id => $name)
 			{
-				if($this->delete($acc_id) && ($acc = $this->read($acc_id)))
+				if ($this->delete($acc_id) && ($acc = $this->read($acc_id)))
 				{
 					$acc['accessory_of'] = -1;
 					$this->save($acc);
 				}
 			};
 			$this->remove_picture($res_id);
-	 		Link::unlink(0,'resources',$res_id);
+	 		Link::unlink(0, 'resources', $res_id);
 	 		// delete the resource from the calendar
-	 		ExecMethod('calendar.calendar_so.deleteaccount','r'.$res_id);
+	 		ExecMethod('calendar.calendar_so.deleteaccount', 'r'.$res_id);
 	 		return false;
 		}
 		return lang('Something went wrong by deleting resource');
@@ -504,15 +504,15 @@ 
 	 * @param boolean $deleted Include deleted accessories
 	 * @return array
 	 */
-	function get_acc_list($res_id,$deleted=false)
+	function get_acc_list($res_id, $deleted = false)
 	{
-		if($res_id < 1){return;}
-		$data = $this->so->search('','res_id,name,deleted','','','','','',$start,array('accessory_of' => $res_id),'',$need_full_no_count=true);
+		if ($res_id < 1) {return; }
+		$data = $this->so->search('', 'res_id,name,deleted', '', '', '', '', '', $start, array('accessory_of' => $res_id), '', $need_full_no_count = true);
 		$acc_list = array();
-		if($data) {
-			foreach($data as $num => $resource)
+		if ($data) {
+			foreach ($data as $num => $resource)
 			{
-				if($resource['deleted'] && !$deleted) continue;
+				if ($resource['deleted'] && !$deleted) continue;
 				$acc_list[$resource['res_id']] = $resource['name'];
 			}
 		}
@@ -528,20 +528,20 @@ 
 	function get_calendar_info($res_id)
 	{
 		//echo "<p>resources_bo::get_calendar_info(".print_r($res_id,true).")</p>\n";
-		if(!is_array($res_id) && $res_id < 1) return;
+		if (!is_array($res_id) && $res_id < 1) return;
 
-		$data = $this->so->search(array('res_id' => $res_id),self::TITLE_COLS.',useable');
+		$data = $this->so->search(array('res_id' => $res_id), self::TITLE_COLS.',useable');
 		if (!is_array($data))
 		{
 			error_log(__METHOD__." No Calendar Data found for Resource with id $res_id");
 			return array();
 		}
-		foreach($data as $num => &$resource)
+		foreach ($data as $num => &$resource)
 		{
 			$resource['rights'] = false;
-			foreach($this->cal_right_transform as $res_right => $cal_right)
+			foreach ($this->cal_right_transform as $res_right => $cal_right)
 			{
-				if($this->acl->is_permitted($resource['cat_id'],$res_right))
+				if ($this->acl->is_permitted($resource['cat_id'], $res_right))
 				{
 					$resource['rights'] = $cal_right;
 				}
@@ -549,7 +549,7 @@ 
 			$resource['responsible'] = $this->acl->get_cat_admin($resource['cat_id']);
 
 			// preseed the cache
-			Link::set_cache('resources',$resource['res_id'],$t=$this->link_title($resource));
+			Link::set_cache('resources', $resource['res_id'], $t = $this->link_title($resource));
 		}
 		return $data;
 	}
@@ -566,7 +566,7 @@ 
 		{
 			return false;
 		}
-		return $this->acl->is_permitted($data['cat_id'],resources_acl_bo::DIRECT_BOOKING) ? A : U;
+		return $this->acl->is_permitted($data['cat_id'], resources_acl_bo::DIRECT_BOOKING) ? A : U;
 	}
 
 	/**
@@ -578,11 +578,11 @@ 
 	 * @param array $options Array of options for the search
 	 *
 	 */
-	function link_query( $pattern, Array &$options = array() )
+	function link_query($pattern, Array &$options = array())
 	{
 		if (is_array($pattern))
 		{
-			$criteria =array('name' => $pattern['search']
+			$criteria = array('name' => $pattern['search']
 					,'short_description' => $pattern['search']);
 		}
 		else
@@ -593,7 +593,7 @@ 
 		$only_keys = 'res_id,name,short_description,bookable,useable';
 
 		// If no read access to any category, just stop
-		if(!$this->acl->get_cats(Acl::READ))
+		if (!$this->acl->get_cats(Acl::READ))
 		{
 			$options['total'] = 0;
 			return array();
@@ -604,30 +604,30 @@ 
 			'deleted' => null
 		);
 		$limit = false;
-		if($options['start'] || $options['num_rows']) {
+		if ($options['start'] || $options['num_rows']) {
 			$limit = array($options['start'], $options['num_rows']);
 		}
-		if($options['accessory_of'])
+		if ($options['accessory_of'])
 		{
 			$filter['accessory_of'] = $options['accessory_of'];
 		}
-		$data = $this->so->search($criteria,$only_keys,$order_by='name',$extra_cols='',$wildcard='%',$empty,$op='OR',$limit,$filter);
+		$data = $this->so->search($criteria, $only_keys, $order_by = 'name', $extra_cols = '', $wildcard = '%', $empty, $op = 'OR', $limit, $filter);
 		// maybe we need to check disponibility of the searched resources in the calendar if $pattern ['exec'] contains some extra args
-		$show_conflict=False;
+		$show_conflict = False;
 		if ($options['exec'] && $GLOBALS['egw_info']['preferences']['calendar']['defaultresource_sel'] !== 'resources')
 		{
 			// we'll use a cache for resources info taken from database
 			static $res_info_cache = array();
-			$cal_info=$options['exec'];
-			if ( isset($cal_info['start']) && isset($cal_info['duration']))
+			$cal_info = $options['exec'];
+			if (isset($cal_info['start']) && isset($cal_info['duration']))
 			{
 				//get a calendar objet for reservations
-				if ( (!isset($this->bocal)) || !(is_object($this->bocal)))
+				if ((!isset($this->bocal)) || !(is_object($this->bocal)))
 				{
 					$this->bocal = new calendar_bo();
 				}
 				$start = new Api\DateTime($cal_info['start']);
-				$startarr= getdate($start->format('ts'));
+				$startarr = getdate($start->format('ts'));
 				if (isset($cal_info['whole_day']) && $cal_info['whole_day']) {
 					$startarr['hour'] = $startarr['minute'] = 0;
 					$start = new Api\DateTime($startarr);
@@ -638,39 +638,39 @@ 
 				}
 
 				// search events matching our timestamps
- 				$resource_list=array();
-				foreach($data as $num => $resource)
+ 				$resource_list = array();
+				foreach ($data as $num => $resource)
 				{
 					// we only need resources id for the search, but with a 'r' prefix
 					// now we take this loop to store a new resource array indexed with resource id
 					// and as we work for calendar we use only bookable resources
-					if ((isset($resource['bookable'])) && ($resource['bookable'])){
-						$res_info_cache[$resource['res_id']]=$resource;
-						$resource_list[]='r'.$resource['res_id'];
+					if ((isset($resource['bookable'])) && ($resource['bookable'])) {
+						$res_info_cache[$resource['res_id']] = $resource;
+						$resource_list[] = 'r'.$resource['res_id'];
 					}
 				}
-				$overlapping_events =& $this->bocal->search(array(
+				$overlapping_events = & $this->bocal->search(array(
 					'start' => $start,
 					'end'   => $end,
 					'users' => $resource_list,
-					'ignore_acl' => true,   // otherwise we get only events readable by the user
-					'enum_groups' => false,  // otherwise group-events would not block time
+					'ignore_acl' => true, // otherwise we get only events readable by the user
+					'enum_groups' => false, // otherwise group-events would not block time
 				));
 
 				// parse theses overlapping events
-				foreach($overlapping_events as $event)
+				foreach ($overlapping_events as $event)
 				{
 					if ($event['non_blocking']) continue; // ignore non_blocking events
-					if (isset($cal_info['event_id']) && $event['id']==$cal_info['event_id']) {
+					if (isset($cal_info['event_id']) && $event['id'] == $cal_info['event_id']) {
 						continue; //ignore this event, it's the current edited event, no conflict by def
 					}
 					// now we are interested only on resources booked by theses events
-					if (isset($event['participants']) && is_array($event['participants'])){
-						foreach($event['participants'] as $part_key => $part_detail){
-							if ($part_key{0}=='r')
+					if (isset($event['participants']) && is_array($event['participants'])) {
+						foreach ($event['participants'] as $part_key => $part_detail) {
+							if ($part_key{0} == 'r')
 							{ //now we gatta resource here
 								//need to check the quantity of this resource
-								$resource_id=substr($part_key,1);
+								$resource_id = substr($part_key, 1);
 								// if we do not find this resource in our indexed array it's certainly
 								// because it was unset, non bookable maybe
 								if (!isset($res_info_cache[$resource_id])) continue;
@@ -680,9 +680,9 @@ 
 								}
 								// now decrement this quantity useable
 								$quantity = 1;
-								$this->bocal->so->split_status($part_detail,$quantity);
+								$this->bocal->so->split_status($part_detail, $quantity);
 
-								$res_info_cache[$resource_id]['useable']-=$quantity;
+								$res_info_cache[$resource_id]['useable'] -= $quantity;
 							}
 						}
 					}
@@ -690,27 +690,27 @@ 
 			}
 		}
 		if (isset($res_info_cache)) {
-			$show_conflict= $GLOBALS['egw_info']['user']['preferences']['calendar']['defaultresource_sel'] === 'resources_conflict';
+			$show_conflict = $GLOBALS['egw_info']['user']['preferences']['calendar']['defaultresource_sel'] === 'resources_conflict';
 			// if we have this array indexed on resource id it means non-bookable resource are removed and we are working for calendar
 			// so we'll loop on this one and not $data
-			foreach($res_info_cache as $id => $resource) {
+			foreach ($res_info_cache as $id => $resource) {
 				//maybe this resource is reserved
-				if ( ($resource['useable'] < 1) )
+				if (($resource['useable'] < 1))
 				{
-					if($show_conflict) {
-						$list[$id] = ' ('.lang('conflict').') '.$resource['name']. ($resource['short_description'] ? ', ['.$resource['short_description'].']':'');
+					if ($show_conflict) {
+						$list[$id] = ' ('.lang('conflict').') '.$resource['name'].($resource['short_description'] ? ', ['.$resource['short_description'].']' : '');
 					}
 				} else {
-				        $list[$id] = $resource['name']. ($resource['short_description'] ? ', ['.$resource['short_description'].']':'');
+				        $list[$id] = $resource['name'].($resource['short_description'] ? ', ['.$resource['short_description'].']' : '');
 				}
 			}
 		} else {
 			// we are not working for the calendar, we loop on the initial $data
 			if (is_array($data)) {
-				foreach($data as $num => $resource)
+				foreach ($data as $num => $resource)
 				{
-					$id=$resource['res_id'];
-					$list[$id] = $resource['name']. ($resource['short_description'] ? ', ['.$resource['short_description'].']':'');
+					$id = $resource['res_id'];
+					$list[$id] = $resource['name'].($resource['short_description'] ? ', ['.$resource['short_description'].']' : '');
 				}
 			} else {
 				error_log(__METHOD__." No Data found for Resource with id ".$resource['res_id']);
@@ -727,17 +727,17 @@ 
 	 * @param int|array $resource
 	 * @return string|boolean string with title, null if resource does not exist or false if no perms to view it
 	 */
-	function link_title( $resource )
+	function link_title($resource)
 	{
 		if (!is_array($resource))
 		{
-			if (!($resource  = $this->read(array('res_id' => $resource)))) return $resource;
+			if (!($resource = $this->read(array('res_id' => $resource)))) return $resource;
 		}
-		elseif (!$this->acl->is_permitted($resource['cat_id'],Acl::READ))
+		elseif (!$this->acl->is_permitted($resource['cat_id'], Acl::READ))
 		{
 			return false;
 		}
-		return $resource['name']. ($resource['short_description'] ? ', ['.$resource['short_description'].']':'');
+		return $resource['name'].($resource['short_description'] ? ', ['.$resource['short_description'].']' : '');
 	}
 
 	/**
@@ -757,15 +757,15 @@ 
 	function link_titles(array $ids)
 	{
 		$titles = array();
-		if (($resources =& $this->so->search(array('res_id' => $ids),self::TITLE_COLS)))
+		if (($resources = & $this->so->search(array('res_id' => $ids), self::TITLE_COLS)))
 		{
-			foreach($resources as $resource)
+			foreach ($resources as $resource)
 			{
 				$titles[$resource['res_id']] = $this->link_title($resource);
 			}
 		}
 		// we assume all not returned contacts are not readable for the user (as we report all deleted contacts to Link)
-		foreach($ids as $id)
+		foreach ($ids as $id)
 		{
 			if (!isset($titles[$id]))
 			{
@@ -783,9 +783,9 @@ 
 	 * @param int $resource_id
 	 * @return mixed string with msg if somthing went wrong; nothing if all right
 	 */
-	function save_picture($file,$resouce_id)
+	function save_picture($file, $resouce_id)
 	{
-		switch($file['type'])
+		switch ($file['type'])
 		{
 			case 'image/gif':
 				$src_img = imagecreatefromgif($file['tmp_name']);
@@ -802,11 +802,11 @@ 
 				return lang('Picture type is not supported, sorry!');
 		}
 
-		$tmp_name = tempnam($GLOBALS['egw_info']['server']['temp_dir'],'resources-picture');
-		imagejpeg($src_img,$tmp_name);
+		$tmp_name = tempnam($GLOBALS['egw_info']['server']['temp_dir'], 'resources-picture');
+		imagejpeg($src_img, $tmp_name);
 		imagedestroy($src_img);
 
-		Link::attach_file('resources',$resouce_id,array(
+		Link::attach_file('resources', $resouce_id, array(
 			'tmp_name' => $tmp_name,
 			'name'     => self::PICTURE_NAME,
 			'type'     => 'image/jpeg',
@@ -820,14 +820,14 @@ 
 	 * @param bool $fullsize false = thumb, true = full pic
 	 * @return string url of picture
 	 */
-	function get_picture($resource,$fullsize=false)
+	function get_picture($resource, $fullsize = false)
 	{
 		if ($resource && !is_array($resource)) $resource = $this->read($resource);
 
-		switch($resource['picture_src'])
+		switch ($resource['picture_src'])
 		{
 			case 'own_src':
-				$picture = Link::vfs_path('resources',$resource['res_id'],self::PICTURE_NAME,true);	// vfs path
+				$picture = Link::vfs_path('resources', $resource['res_id'], self::PICTURE_NAME, true); // vfs path
 				if ($fullsize)
 				{
 					$picture = Egw::link(Vfs::download_url($picture));
@@ -836,23 +836,23 @@ 
 				{
 					$picture = Egw::link('/api/thumbnail.php', array(
 						'path' => $picture
-					),false);
+					), false);
 				}
 				break;
 
 			case 'cat_src':
 				$picture = Api\Categories::id2name($resource['cat_id'], 'data');
-				if($picture['icon'])
+				if ($picture['icon'])
 				{
-					$picture = !$fullsize?$GLOBALS['egw_info']['server']['webserver_url'].self::ICON_PATH.'/'.$picture['icon']:self::ICON_PATH.'/'.$picture['icon'];
+					$picture = !$fullsize ? $GLOBALS['egw_info']['server']['webserver_url'].self::ICON_PATH.'/'.$picture['icon'] : self::ICON_PATH.'/'.$picture['icon'];
 					break;
 				}
 				// fall through
 			case 'gen_src':
 			default :
 				$src = $resource['picture_src'];
-				$picture = !$fullsize?$GLOBALS['egw_info']['server']['webserver_url'].$this->resource_icons:$this->resource_icons;
-				$picture .= strpos($src,'.') !== false ? $src : 'generic.png';
+				$picture = !$fullsize ? $GLOBALS['egw_info']['server']['webserver_url'].$this->resource_icons : $this->resource_icons;
+				$picture .= strpos($src, '.') !== false ? $src : 'generic.png';
 		}
 		return $picture;
 	}
@@ -866,9 +866,9 @@ 
 	 */
 	function remove_picture($res_id)
 	{
-		if (($arr = Link::delete_attached('resources',$res_id,self::PICTURE_NAME)) && is_array($arr))
+		if (($arr = Link::delete_attached('resources', $res_id, self::PICTURE_NAME)) && is_array($arr))
 		{
-			return array_shift($arr);	// $arr = array($path => (bool)$ok);
+			return array_shift($arr); // $arr = array($path => (bool)$ok);
 		}
 		return false;
 	}
@@ -884,11 +884,11 @@ 
 	{
 		$icons['generic.png'] = lang('gernal resource');
 		$dir = dir(EGW_SERVER_ROOT.$this->resource_icons);
-		while($file = $dir->read())
+		while ($file = $dir->read())
 		{
-			if (preg_match('/\\.(png|gif|jpe?g)$/i',$file) && $file != 'generic.png')
+			if (preg_match('/\\.(png|gif|jpe?g)$/i', $file) && $file != 'generic.png')
 			{
-				$icons[$file] = substr($file,0,strpos($file,'.'));
+				$icons[$file] = substr($file, 0, strpos($file, '.'));
 			}
 		}
 		$dir->close();

resources/inc/class.resources_export_csv.inc.php

Doc Comments +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
 	/**
 	 * Exports records as defined in $_definition
 	 *
-	 * @param egw_record $_definition
+	 * @param importexport_definition $_definition
 	 */
 	public function export( $_stream, importexport_definition $_definition) {
 		$options = $_definition->plugin_options;

Indentation +2 added lines, -2 removed lines

@@ -170,9 +170,9 @@
 	 * Get selectbox values
 	 */
 	protected function get_selects()
-        {
+		{
 		$this->selects = array();
-        }
+		}
 
 	/**
 	 * Customize automatically generated filter fields

Spacing +27 added lines, -27 removed lines

@@ -31,54 +31,54 @@ 
 	 *
 	 * @param egw_record $_definition
 	 */
-	public function export( $_stream, importexport_definition $_definition) {
+	public function export($_stream, importexport_definition $_definition) {
 		$options = $_definition->plugin_options;
 
 		$selection = array();
 		if ($options['selection'] == 'search') {
 			// ui selection with checkbox 'selected'
 			$query = Api\Cache::getSession('resources', 'get_rows');
-			$query['num_rows'] = -1;	// all
+			$query['num_rows'] = -1; // all
 			unset($query['store_state']);
-			$query['csv_export'] = true;	// so get_rows method _can_ produce different content or not store state in the session
-			$this->bo->get_rows($query,$selection,$readonlys);
+			$query['csv_export'] = true; // so get_rows method _can_ produce different content or not store state in the session
+			$this->bo->get_rows($query, $selection, $readonlys);
 		}
-		elseif ( $options['selection'] == 'all' || $options['selection'] == 'filter') {
+		elseif ($options['selection'] == 'all' || $options['selection'] == 'filter') {
 			$query = array(
 				'num_rows'	=> -1,
-				'filter2'	=> -3,  // Accessories & resources
-				'csv_export' => true,	// so get_rows method _can_ produce different content or not store state in the session
-			);	// all
+				'filter2'	=> -3, // Accessories & resources
+				'csv_export' => true, // so get_rows method _can_ produce different content or not store state in the session
+			); // all
 			
-			if($options['selection'] == 'filter')
+			if ($options['selection'] == 'filter')
 			{
 				$filter = $_definition->filter;
 				// Handle ranges
-				foreach($filter as $field => $value)
+				foreach ($filter as $field => $value)
 				{
-					if($field == 'cat_id')
+					if ($field == 'cat_id')
 					{
 						$query['filter'] = $value;
 						continue;
 					}
-					if($field == 'filter2')
+					if ($field == 'filter2')
 					{
 						$query['filter2'] = $value;
 						continue;
 					}
 					$query['col_filter'][$field] = $value;
-					if(!is_array($value) || (!$value['from'] && !$value['to'])) continue;
+					if (!is_array($value) || (!$value['from'] && !$value['to'])) continue;
 
 					// Ranges are inclusive, so should be provided that way (from 2 to 10 includes 2 and 10)
-					if($value['from']) $query['col_filter'][] = "$field >= " . (int)$value['from'];
-					if($value['to']) $query['col_filter'][] = "$field <= " . (int)$value['to'];
+					if ($value['from']) $query['col_filter'][] = "$field >= ".(int)$value['from'];
+					if ($value['to']) $query['col_filter'][] = "$field <= ".(int)$value['to'];
 					unset($query['col_filter'][$field]);
 				}
 			}
 			
-			$this->bo->get_rows($query,$selection,$readonlys);
+			$this->bo->get_rows($query, $selection, $readonlys);
 		} else {
-			$selection = explode(',',$options['selection']);
+			$selection = explode(',', $options['selection']);
 		}
 
 		$export_object = new importexport_export_csv($_stream, (array)$options);
@@ -86,28 +86,28 @@ 
 
 		// Check if we need to load the custom fields
 		$need_custom = false;
-		foreach(Api\Storage\Customfields::get('resources') as $field => $settings) {
-			if($options['mapping']['#'.$field]) {
+		foreach (Api\Storage\Customfields::get('resources') as $field => $settings) {
+			if ($options['mapping']['#'.$field]) {
 				$need_custom = true;
 				break;
 			}
 		}
 
 		foreach ($selection as $record) {
-			if(!is_array($record) || !$record['res_id']) continue;
+			if (!is_array($record) || !$record['res_id']) continue;
 
-			if($need_custom) {
+			if ($need_custom) {
 				$record = $this->bo->read($record['res_id']);
 			}
 			$resource = new resources_egw_record();
 			$resource->set_record($record);
 			$resource->long_description = strip_tags($resource->long_description);
-			if($options['convert']) {
+			if ($options['convert']) {
 				importexport_export_csv::convert($resource, resources_egw_record::$types, 'resources', $this->selects);
 			} else {
 				// Implode arrays, so they don't say 'Array'
-				foreach($resource->get_record_array() as $key => $value) {
-					if(is_array($value)) $resource->$key = implode(',', $value);
+				foreach ($resource->get_record_array() as $key => $value) {
+					if (is_array($value)) $resource->$key = implode(',', $value);
 				}
  			}
 
@@ -182,7 +182,7 @@ 
 		// In resources, not all categories are used
 		$filters['cat_id']['type'] = 'select';
 		$filters['cat_id']['name'] = 'filter';
-		$filters['cat_id']['values']= (array)$this->bo->acl->get_cats(Acl::READ);
+		$filters['cat_id']['values'] = (array)$this->bo->acl->get_cats(Acl::READ);
 
 		// Add in resources / accessories
 		$filters['filter2'] = array(
@@ -192,9 +192,9 @@ 
 			'rows' => 5,
 			'values' => resources_bo::$filter_options
 		);
-		foreach($filters as $field_name => &$settings)
+		foreach ($filters as $field_name => &$settings)
 		{
-			if($this->selects[$field_name]) $settings['values'] = $this->selects[$field_name];
+			if ($this->selects[$field_name]) $settings['values'] = $this->selects[$field_name];
 		}
 	}
 

Braces +63 added lines, -25 removed lines

@@ -17,7 +17,8 @@ 
 /**
  * export resources to CSV
  */
-class resources_export_csv implements importexport_iface_export_plugin {
+class resources_export_csv implements importexport_iface_export_plugin
+{
 
 	public function __construct()
 	{
@@ -31,11 +32,13 @@ 
 	 *
 	 * @param egw_record $_definition
 	 */
-	public function export( $_stream, importexport_definition $_definition) {
+	public function export( $_stream, importexport_definition $_definition)
+	{
 		$options = $_definition->plugin_options;
 
 		$selection = array();
-		if ($options['selection'] == 'search') {
+		if ($options['selection'] == 'search')
+		{
 			// ui selection with checkbox 'selected'
 			$query = Api\Cache::getSession('resources', 'get_rows');
 			$query['num_rows'] = -1;	// all
@@ -43,7 +46,8 @@ 
 			$query['csv_export'] = true;	// so get_rows method _can_ produce different content or not store state in the session
 			$this->bo->get_rows($query,$selection,$readonlys);
 		}
-		elseif ( $options['selection'] == 'all' || $options['selection'] == 'filter') {
+		elseif ( $options['selection'] == 'all' || $options['selection'] == 'filter')
+		{
 			$query = array(
 				'num_rows'	=> -1,
 				'filter2'	=> -3,  // Accessories & resources
@@ -67,17 +71,28 @@ 
 						continue;
 					}
 					$query['col_filter'][$field] = $value;
-					if(!is_array($value) || (!$value['from'] && !$value['to'])) continue;
+					if(!is_array($value) || (!$value['from'] && !$value['to']))
+					{
+						continue;
+					}
 
 					// Ranges are inclusive, so should be provided that way (from 2 to 10 includes 2 and 10)
-					if($value['from']) $query['col_filter'][] = "$field >= " . (int)$value['from'];
-					if($value['to']) $query['col_filter'][] = "$field <= " . (int)$value['to'];
+					if($value['from'])
+					{
+						$query['col_filter'][] = "$field >= " . (int)$value['from'];
+					}
+					if($value['to'])
+					{
+						$query['col_filter'][] = "$field <= " . (int)$value['to'];
+					}
 					unset($query['col_filter'][$field]);
 				}
 			}
 			
 			$this->bo->get_rows($query,$selection,$readonlys);
-		} else {
+		}
+		else
+		{
 			$selection = explode(',',$options['selection']);
 		}
 
@@ -86,28 +101,42 @@ 
 
 		// Check if we need to load the custom fields
 		$need_custom = false;
-		foreach(Api\Storage\Customfields::get('resources') as $field => $settings) {
-			if($options['mapping']['#'.$field]) {
+		foreach(Api\Storage\Customfields::get('resources') as $field => $settings)
+		{
+			if($options['mapping']['#'.$field])
+			{
 				$need_custom = true;
 				break;
 			}
 		}
 
-		foreach ($selection as $record) {
-			if(!is_array($record) || !$record['res_id']) continue;
+		foreach ($selection as $record)
+		{
+			if(!is_array($record) || !$record['res_id'])
+			{
+				continue;
+			}
 
-			if($need_custom) {
+			if($need_custom)
+			{
 				$record = $this->bo->read($record['res_id']);
 			}
 			$resource = new resources_egw_record();
 			$resource->set_record($record);
 			$resource->long_description = strip_tags($resource->long_description);
-			if($options['convert']) {
+			if($options['convert'])
+			{
 				importexport_export_csv::convert($resource, resources_egw_record::$types, 'resources', $this->selects);
-			} else {
+			}
+			else
+			{
 				// Implode arrays, so they don't say 'Array'
-				foreach($resource->get_record_array() as $key => $value) {
-					if(is_array($value)) $resource->$key = implode(',', $value);
+				foreach($resource->get_record_array() as $key => $value)
+				{
+					if(is_array($value))
+					{
+						$resource->$key = implode(',', $value);
+					}
 				}
  			}
 
@@ -122,7 +151,8 @@ 
 	 *
 	 * @return string name
 	 */
-	public static function get_name() {
+	public static function get_name()
+	{
 		return lang('Resources CSV export');
 	}
 
@@ -131,7 +161,8 @@ 
 	 *
 	 * @return string descriprion
 	 */
-	public static function get_description() {
+	public static function get_description()
+	{
 		return lang("Exports a list of resources to a CSV File.");
 	}
 
@@ -140,11 +171,13 @@ 
 	 *
 	 * @return string suffix
 	 */
-	public static function get_filesuffix() {
+	public static function get_filesuffix()
+	{
 		return 'csv';
 	}
 
-	public static function get_mimetype() {
+	public static function get_mimetype()
+	{
 		return 'text/csv';
 	}
 
@@ -153,14 +186,16 @@ 
 	 * this way the plugin has all opportunities for options tab
 	 *
 	 */
-	public function get_options_etpl() {
+	public function get_options_etpl()
+	{
 	}
 
 	/**
 	 * returns selectors information
 	 *
 	 */
-	public function get_selectors_etpl() {
+	public function get_selectors_etpl()
+	{
 		return array(
 			'name'	=> 'importexport.export_csv_selectors',
 		);
@@ -170,7 +205,7 @@ 
 	 * Get selectbox values
 	 */
 	protected function get_selects()
-        {
+	{
 		$this->selects = array();
         }
 
@@ -194,7 +229,10 @@ 
 		);
 		foreach($filters as $field_name => &$settings)
 		{
-			if($this->selects[$field_name]) $settings['values'] = $this->selects[$field_name];
+			if($this->selects[$field_name])
+			{
+				$settings['values'] = $this->selects[$field_name];
+			}
 		}
 	}
 

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -247,7 +247,7 @@
 		$this->selects['info_status'] = $this->bo->get_status();
 	}
 
-	public function get_filter_fields(Array &$filters)
+	public function get_filter_fields(array &$filters)
 	{
 		foreach($filters as $field_name => &$settings)
 		{

resources/inc/class.resources_import_csv.inc.php

Doc Comments +2 added lines, -4 removed lines

@@ -28,9 +28,7 @@ 
 
 	/**
 	 * imports entries according to given definition object.
-	 * @param resource $_stream
-	 * @param string $_charset
-	 * @param definition $_definition
+	 * @param importexport_definition $_definition
 	 */
 	public function init(importexport_definition $_definition ) {
 
@@ -52,7 +50,7 @@ 
 	*
 	* Updates the count of actions taken
 	*
-	* @return boolean success
+	* @return null|boolean success
 	*/
 	protected function import_record(importexport_iface_egw_record &$record, &$import_csv)
 	{

Indentation +17 added lines, -17 removed lines

@@ -45,15 +45,15 @@ 
 	}
 
 	/**
-	* Import a single record
-	*
-	* You don't need to worry about mappings or translations, they've been done already.
-	* You do need to handle the conditions and the actions taken.
-	*
-	* Updates the count of actions taken
-	*
-	* @return boolean success
-	*/
+	 * Import a single record
+	 *
+	 * You don't need to worry about mappings or translations, they've been done already.
+	 * You do need to handle the conditions and the actions taken.
+	 *
+	 * Updates the count of actions taken
+	 *
+	 * @return boolean success
+	 */
 	protected function import_record(importexport_iface_egw_record &$record, &$import_csv)
 	{
 		// Check for an un-matched accessory of, try again on just name
@@ -250,14 +250,14 @@ 
 	}
 
 	/**
-        * Returns warnings that were encountered during importing
-        * Maximum of one warning message per record, but you can append if you need to
-        *
-        * @return Array (
-        *       record_# => warning message
-        *       )
-        */
-        public function get_warnings() {
+	 * Returns warnings that were encountered during importing
+	 * Maximum of one warning message per record, but you can append if you need to
+	 *
+	 * @return Array (
+	 *       record_# => warning message
+	 *       )
+	 */
+		public function get_warnings() {
 		return $this->warnings;
 	}
 }

Spacing +32 added lines, -32 removed lines

@@ -16,7 +16,7 @@ 
 /**
  * class to import resources from CSV
  */
-class resources_import_csv extends importexport_basic_import_csv  {
+class resources_import_csv extends importexport_basic_import_csv {
 
 
 	/**
@@ -24,7 +24,7 @@ 
 	 *
 	 * @var array
 	 */
-	protected static $conditions = array( 'exists' );
+	protected static $conditions = array('exists');
 
 	/**
 	 * imports entries according to given definition object.
@@ -32,7 +32,7 @@ 
 	 * @param string $_charset
 	 * @param definition $_definition
 	 */
-	public function init(importexport_definition $_definition ) {
+	public function init(importexport_definition $_definition) {
 
 		// fetch the resource bo
 		$this->bo = new resources_bo();
@@ -54,33 +54,33 @@ 
 	*
 	* @return boolean success
 	*/
-	protected function import_record(importexport_iface_egw_record &$record, &$import_csv)
+	protected function import_record(importexport_iface_egw_record&$record, &$import_csv)
 	{
 		// Check for an un-matched accessory of, try again on just name
-		if(!is_numeric($record->accessory_of))
+		if (!is_numeric($record->accessory_of))
 		{
 			$accessory_of = $record->accessory_of;
 
 			// Look for exact match in just name
-			$results = $this->bo->so->search(array('name' => $record->accessory_of),array('res_id','name'));
-			if(count($results) >= 1)
+			$results = $this->bo->so->search(array('name' => $record->accessory_of), array('res_id', 'name'));
+			if (count($results) >= 1)
 			{
 				// More than 1 result?  Bad names.  Pick one.
-				foreach($results as $result)
+				foreach ($results as $result)
 				{
-					if($result['name'] == $record->accessory_of)
+					if ($result['name'] == $record->accessory_of)
 					{
 						$record->accessory_of = $result['res_id'];
 						break;
 					}
 				}
-				if(is_numeric($record->accessory_of))
+				if (is_numeric($record->accessory_of))
 				{
 					// Import/Export conversion gave a warning, so cancel it
-					$pattern = lang('Unable to link to %1 "%2"',lang('resources'),$accessory_of) . ' - ('.lang('too many matches') . '|'.lang('no matches') . ')';
+					$pattern = lang('Unable to link to %1 "%2"', lang('resources'), $accessory_of).' - ('.lang('too many matches').'|'.lang('no matches').')';
 					$this->warnings[$import_csv->get_current_position()] = preg_replace($pattern, '', $this->warnings[$import_csv->get_current_position()], 1);
 					// If that was the only warning, clear it for this row
-					if(trim($this->warnings[$import_csv->get_current_position()]) == '')
+					if (trim($this->warnings[$import_csv->get_current_position()]) == '')
 					{
 						unset($this->warnings[$import_csv->get_current_position()]);
 					}
@@ -92,7 +92,7 @@ 
 		// Check for a new category, it needs permissions set
 		$category = $GLOBALS['egw']->categories->read($record->cat_id);
 
-		if($category['last_mod'] >= $this->start_time) {
+		if ($category['last_mod'] >= $this->start_time) {
 			// New category.  Give read & write permissions to the current user's default group
 			$this->acl_bo->set_rights($record['cat_id'],
 				array($GLOBALS['egw_info']['user']['account_primary_group']),
@@ -104,37 +104,37 @@ 
 			// Refresh ACL
 			//$GLOBALS['egw']->acl->read_repository();
 		}
-		if(!$record->accessory_of) $record->accessory_of = -1;
+		if (!$record->accessory_of) $record->accessory_of = -1;
 		//error_log(__METHOD__.__LINE__.array2string($_definition->plugin_options['conditions']));
 		if ($this->definition->plugin_options['conditions']) {
 
-			foreach ( $this->definition->plugin_options['conditions'] as $condition ) {
+			foreach ($this->definition->plugin_options['conditions'] as $condition) {
 				$results = array();
-				switch ( $condition['type'] ) {
+				switch ($condition['type']) {
 					// exists
 					case 'exists' :
-						if($record->{$condition['string']}) {
+						if ($record->{$condition['string']}) {
 							$results = $this->bo->so->search(
-								array( $condition['string'] => $record->{$condition['string']}),
+								array($condition['string'] => $record->{$condition['string']}),
 								False
 							);
 						}
 
-						if ( is_array( $results ) && count( array_keys( $results )) >= 1) {
+						if (is_array($results) && count(array_keys($results)) >= 1) {
 							// apply action to all contacts matching this exists condition
 							$action = $condition['true'];
-							foreach ( (array)$results as $resource ) {
+							foreach ((array)$results as $resource) {
 								$record->res_id = $resource['res_id'];
-								if ( $_definition->plugin_options['update_cats'] == 'add' ) {
-									if ( !is_array( $resource['cat_id'] ) ) $resource['cat_id'] = explode( ',', $resource['cat_id'] );
-									if ( !is_array( $record->cat_id ) ) $record->cat_id = explode( ',', $record->cat_id );
-									$record->cat_id = implode( ',', array_unique( array_merge( $record->cat_id, $resource['cat_id'] ) ) );
+								if ($_definition->plugin_options['update_cats'] == 'add') {
+									if (!is_array($resource['cat_id'])) $resource['cat_id'] = explode(',', $resource['cat_id']);
+									if (!is_array($record->cat_id)) $record->cat_id = explode(',', $record->cat_id);
+									$record->cat_id = implode(',', array_unique(array_merge($record->cat_id, $resource['cat_id'])));
 								}
-								$success = $this->action(  $action['action'], $record, $import_csv->get_current_position() );
+								$success = $this->action($action['action'], $record, $import_csv->get_current_position());
 							}
 						} else {
 							$action = $condition['false'];
-							$success = ($this->action(  $action['action'], $record, $import_csv->get_current_position() ));
+							$success = ($this->action($action['action'], $record, $import_csv->get_current_position()));
 						}
 						break;
 
@@ -147,7 +147,7 @@ 
 			}
 		} else {
 			// unconditional insert
-			$success = $this->action( 'insert', $record, $import_csv->get_current_position() );
+			$success = $this->action('insert', $record, $import_csv->get_current_position());
 		}
 		return $success;
 	}
@@ -159,7 +159,7 @@ 
 	 * @param importexport_iface_egw_record $record Entry record
 	 * @return bool success or not
 	 */
-	protected function action ( $_action, importexport_iface_egw_record &$record, $record_num = 0 ) {
+	protected function action($_action, importexport_iface_egw_record&$record, $record_num = 0) {
 		$_data = $record->get_record_array();
 		switch ($_action) {
 			case 'none' :
@@ -173,17 +173,17 @@ 
 
 				// Fall through
 			case 'insert' :
-				if($_action == 'insert') {
+				if ($_action == 'insert') {
 					// Backend doesn't like inserting with ID specified, it can overwrite
 					unset($_data['res_id']);
 				}
-				if ( $this->dry_run ) {
+				if ($this->dry_run) {
 					//print_r($_data);
 					$this->results[$_action]++;
 					return true;
 				} else {
-					$result = $this->bo->save( $_data );
-					if($result && !is_numeric($result)) {
+					$result = $this->bo->save($_data);
+					if ($result && !is_numeric($result)) {
 						$this->errors[$record_num] = $result;
 						return false;
 					} else {

Braces +70 added lines, -29 removed lines

@@ -16,7 +16,8 @@ 
 /**
  * class to import resources from CSV
  */
-class resources_import_csv extends importexport_basic_import_csv  {
+class resources_import_csv extends importexport_basic_import_csv
+{
 
 
 	/**
@@ -32,7 +33,8 @@ 
 	 * @param string $_charset
 	 * @param definition $_definition
 	 */
-	public function init(importexport_definition $_definition ) {
+	public function init(importexport_definition $_definition )
+	{
 
 		// fetch the resource bo
 		$this->bo = new resources_bo();
@@ -92,7 +94,8 @@ 
 		// Check for a new category, it needs permissions set
 		$category = $GLOBALS['egw']->categories->read($record->cat_id);
 
-		if($category['last_mod'] >= $this->start_time) {
+		if($category['last_mod'] >= $this->start_time)
+		{
 			// New category.  Give read & write permissions to the current user's default group
 			$this->acl_bo->set_rights($record['cat_id'],
 				array($GLOBALS['egw_info']['user']['account_primary_group']),
@@ -104,35 +107,53 @@ 
 			// Refresh ACL
 			//$GLOBALS['egw']->acl->read_repository();
 		}
-		if(!$record->accessory_of) $record->accessory_of = -1;
+		if(!$record->accessory_of)
+		{
+			$record->accessory_of = -1;
+		}
 		//error_log(__METHOD__.__LINE__.array2string($_definition->plugin_options['conditions']));
-		if ($this->definition->plugin_options['conditions']) {
+		if ($this->definition->plugin_options['conditions'])
+		{
 
-			foreach ( $this->definition->plugin_options['conditions'] as $condition ) {
+			foreach ( $this->definition->plugin_options['conditions'] as $condition )
+			{
 				$results = array();
-				switch ( $condition['type'] ) {
+				switch ( $condition['type'] )
+				{
 					// exists
 					case 'exists' :
-						if($record->{$condition['string']}) {
+						if($record->{$condition['string']})
+						{
 							$results = $this->bo->so->search(
 								array( $condition['string'] => $record->{$condition['string']}),
 								False
 							);
 						}
 
-						if ( is_array( $results ) && count( array_keys( $results )) >= 1) {
+						if ( is_array( $results ) && count( array_keys( $results )) >= 1)
+						{
 							// apply action to all contacts matching this exists condition
 							$action = $condition['true'];
-							foreach ( (array)$results as $resource ) {
+							foreach ( (array)$results as $resource )
+							{
 								$record->res_id = $resource['res_id'];
-								if ( $_definition->plugin_options['update_cats'] == 'add' ) {
-									if ( !is_array( $resource['cat_id'] ) ) $resource['cat_id'] = explode( ',', $resource['cat_id'] );
-									if ( !is_array( $record->cat_id ) ) $record->cat_id = explode( ',', $record->cat_id );
+								if ( $_definition->plugin_options['update_cats'] == 'add' )
+								{
+									if ( !is_array( $resource['cat_id'] ) )
+									{
+										$resource['cat_id'] = explode( ',', $resource['cat_id'] );
+									}
+									if ( !is_array( $record->cat_id ) )
+									{
+										$record->cat_id = explode( ',', $record->cat_id );
+									}
 									$record->cat_id = implode( ',', array_unique( array_merge( $record->cat_id, $resource['cat_id'] ) ) );
 								}
 								$success = $this->action(  $action['action'], $record, $import_csv->get_current_position() );
 							}
-						} else {
+						}
+						else
+						{
 							$action = $condition['false'];
 							$success = ($this->action(  $action['action'], $record, $import_csv->get_current_position() ));
 						}
@@ -143,9 +164,14 @@ 
 						die('condition / action not supported!!!');
 						break;
 				}
-				if ($action['last']) break;
+				if ($action['last'])
+				{
+					break;
+				}
 			}
-		} else {
+		}
+		else
+		{
 			// unconditional insert
 			$success = $this->action( 'insert', $record, $import_csv->get_current_position() );
 		}
@@ -159,9 +185,11 @@ 
 	 * @param importexport_iface_egw_record $record Entry record
 	 * @return bool success or not
 	 */
-	protected function action ( $_action, importexport_iface_egw_record &$record, $record_num = 0 ) {
+	protected function action ( $_action, importexport_iface_egw_record &$record, $record_num = 0 )
+	{
 		$_data = $record->get_record_array();
-		switch ($_action) {
+		switch ($_action)
+		{
 			case 'none' :
 				return true;
 			case 'update' :
@@ -173,20 +201,27 @@ 
 
 				// Fall through
 			case 'insert' :
-				if($_action == 'insert') {
+				if($_action == 'insert')
+				{
 					// Backend doesn't like inserting with ID specified, it can overwrite
 					unset($_data['res_id']);
 				}
-				if ( $this->dry_run ) {
+				if ( $this->dry_run )
+				{
 					//print_r($_data);
 					$this->results[$_action]++;
 					return true;
-				} else {
+				}
+				else
+				{
 					$result = $this->bo->save( $_data );
-					if($result && !is_numeric($result)) {
+					if($result && !is_numeric($result))
+					{
 						$this->errors[$record_num] = $result;
 						return false;
-					} else {
+					}
+					else
+					{
 						$this->results[$_action]++;
 						return true;
 					}
@@ -202,7 +237,8 @@ 
 	 *
 	 * @return string name
 	 */
-	public static function get_name() {
+	public static function get_name()
+	{
 		return lang('Resources CSV import');
 	}
 
@@ -211,7 +247,8 @@ 
 	 *
 	 * @return string descriprion
 	 */
-	public static function get_description() {
+	public static function get_description()
+	{
 		return lang("Imports a list of resources from a CSV file.");
 	}
 
@@ -220,7 +257,8 @@ 
 	 *
 	 * @return string suffix (comma seperated)
 	 */
-	public static function get_filesuffix() {
+	public static function get_filesuffix()
+	{
 		return 'csv';
 	}
 
@@ -236,7 +274,8 @@ 
 	 * 		preserv		=> array,
 	 * )
 	 */
-	public function get_options_etpl() {
+	public function get_options_etpl()
+	{
 		// lets do it!
 	}
 
@@ -245,7 +284,8 @@ 
 	 *
 	 * @return string etemplate name
 	 */
-	public function get_selectors_etpl() {
+	public function get_selectors_etpl()
+	{
 		// lets do it!
 	}
 
@@ -257,7 +297,8 @@ 
         *       record_# => warning message
         *       )
         */
-        public function get_warnings() {
+        public function get_warnings()
+        {
 		return $this->warnings;
 	}
 }

resources/inc/class.resources_so.inc.php

Doc Comments +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
 	 *
 	 * @param string $key key of value to get
 	 * @param int $res_id resource id
-	 * @return mixed value of key and resource, false if key or id not found.
+	 * @return integer value of key and resource, false if key or id not found.
 	 */
 	function get_value($key,$res_id)
 	{

Spacing +6 added lines, -7 removed lines

@@ -22,10 +22,10 @@ 
 {
 	function __construct()
 	{
-		parent::__construct('resources','egw_resources', 'egw_resources_extra', '',
-			'extra_name', 'extra_value', 'extra_id' );
+		parent::__construct('resources', 'egw_resources', 'egw_resources_extra', '',
+			'extra_name', 'extra_value', 'extra_id');
 
-		$this->columns_to_search = array('name','short_description','inventory_number','long_description','location');
+		$this->columns_to_search = array('name', 'short_description', 'inventory_number', 'long_description', 'location');
 	}
 
 	/**
@@ -35,10 +35,9 @@ 
 	 * @param int $res_id resource id
 	 * @return mixed value of key and resource, false if key or id not found.
 	 */
-	function get_value($key,$res_id)
+	function get_value($key, $res_id)
 	{
-		return $res_id == $this->data['res_id'] ? $this->data[$key] :
-			$this->db->select($this->table_name,$key,array('res_id' => $res_id),__LINE__,__FILE__)->fetchColumn();
+		return $res_id == $this->data['res_id'] ? $this->data[$key] : $this->db->select($this->table_name, $key, array('res_id' => $res_id), __LINE__, __FILE__)->fetchColumn();
 	}
 
 	/**
@@ -90,7 +89,7 @@ 
 	function save($resource)
 	{
 		$this->data = $resource;
-		if(parent::save() != 0) return false;
+		if (parent::save() != 0) return false;
 		$res_id = $this->data['res_id'];
 
 		return $res_id;

Braces +8 added lines, -2 removed lines

@@ -51,7 +51,10 @@ 
 	 */
 	function read($res_id)
 	{
-		if (is_array($res_id) && count($res_id) == 1 && isset($res_id['res_id'])) $res_id = $res_id['res_id'];
+		if (is_array($res_id) && count($res_id) == 1 && isset($res_id['res_id']))
+		{
+			$res_id = $res_id['res_id'];
+		}
 
 		/*if (!is_array($res_id) && $res_id == $this->data['res_id'])
 		{
@@ -90,7 +93,10 @@ 
 	function save($resource)
 	{
 		$this->data = $resource;
-		if(parent::save() != 0) return false;
+		if(parent::save() != 0)
+		{
+			return false;
+		}
 		$res_id = $this->data['res_id'];
 
 		return $res_id;

setup/check_install.php

Doc Comments +1 added lines

@@ -316,6 +316,7 @@
  * Check if given package is installed via composer in EGroupware's vendor directory
  *
  * @param string $package package-name in composer notation eg. "pear-pear.horde.org/Horde_Imap_Client" or "pear-pear.php.net/Net_Sieve"
+ * @return string|null
  */
 function composer_check($package)
 {

Spacing +119 added lines, -123 removed lines

@@ -13,7 +13,7 @@ 
 use EGroupware\Api;
 
 $run_by_webserver = !!$_SERVER['PHP_SELF'];
-$is_windows = strtoupper(substr(PHP_OS,0,3)) == 'WIN';
+$is_windows = strtoupper(substr(PHP_OS, 0, 3)) == 'WIN';
 
 if ($run_by_webserver)
 {
@@ -41,9 +41,9 @@ 
 	$error_icon = '*** Error: ';
 	$warning_icon = '!!! Warning: ';
 
-	function lang($msg,$arg1=NULL,$arg2=NULL,$arg3=NULL,$arg4=NULL)
+	function lang($msg, $arg1 = NULL, $arg2 = NULL, $arg3 = NULL, $arg4 = NULL)
 	{
-		return is_null($arg1) ? $msg : str_replace(array('%1','%2','%3','%4'),array($arg1,$arg2,$arg3,$arg4),$msg);
+		return is_null($arg1) ? $msg : str_replace(array('%1', '%2', '%3', '%4'), array($arg1, $arg2, $arg3, $arg4), $msg);
 	}
 }
 
@@ -79,13 +79,13 @@ 
 		'func' => 'php_ini_check',
 		'value' => 0,
 		'verbose_value' => 'Off',
-		'warning' => lang('%1 is set to %2. This is NOT recommeded for a production system, as displayed error messages can contain passwords or other sensitive information!','display_errors',ini_get('display_errors')),
+		'warning' => lang('%1 is set to %2. This is NOT recommeded for a production system, as displayed error messages can contain passwords or other sensitive information!', 'display_errors', ini_get('display_errors')),
 	),
 	'memory_limit' => array(
 		'func' => 'php_ini_check',
 		'value' => '128M',
 		'check' => '>=',
-		'error' => lang('memory_limit is set to less than %1: some applications of eGroupWare need more than the recommend 8M, expect occasional failures','24M'),
+		'error' => lang('memory_limit is set to less than %1: some applications of eGroupWare need more than the recommend 8M, expect occasional failures', '24M'),
 		'change' => 'memory_limit = 24M'
 	),
 	'max_execution_time' => array(
@@ -105,14 +105,14 @@ 
 		'func' => 'php_ini_check',
 		'value' => '8M',
 		'check' => '>=',
-		'error' => lang('%1 is set to %2, you will NOT be able to upload or attach files bigger then that!','upload_max_filesize',ini_get('upload_max_filesize')),
+		'error' => lang('%1 is set to %2, you will NOT be able to upload or attach files bigger then that!', 'upload_max_filesize', ini_get('upload_max_filesize')),
 		'change' => 'upload_max_filesize = 8M'
 	),
 	'post_max_size' => array(
 		'func' => 'php_ini_check',
 		'value' => '8M',
 		'check' => '>=',
-		'error' => lang('%1 is set to %2, you will NOT be able to upload or attach files bigger then that!','post_max_size',ini_get('max_post_size')),
+		'error' => lang('%1 is set to %2, you will NOT be able to upload or attach files bigger then that!', 'post_max_size', ini_get('max_post_size')),
 		'change' => 'post_max_size = 8M'
 	),
 	'allow_url_fopen' => array(
@@ -138,7 +138,7 @@ 
 		'verbose_value' => '"System/Localtime"',
 		'check' => '!=',
 		'error' => lang('No VALID timezone set! ("%1" is NOT sufficient, you have to use a timezone identifer like "%2", see %3full list of valid identifers%4)',
-			'System/Localtime','Europe/Berlin','<a href="http://www.php.net/manual/en/timezones.php" target="_blank">','</a>'),
+			'System/Localtime', 'Europe/Berlin', '<a href="http://www.php.net/manual/en/timezones.php" target="_blank">', '</a>'),
 	),
 	'pdo' => array(
 		'func' => 'extension_check',
@@ -146,19 +146,19 @@ 
 	),
 	'mysqli' => array(
 		'func' => 'extension_check',
-		'warning' => lang('The %1 extension is needed, if you plan to use a %2 database.','mysql','MySQL')
+		'warning' => lang('The %1 extension is needed, if you plan to use a %2 database.', 'mysql', 'MySQL')
 	),
 	'pdo_mysql' => array(
 		'func' => 'extension_check',
-		'warning' => lang('The %1 extension is needed, if you plan to use a %2 database.','pdo_mysql','MySQL')
+		'warning' => lang('The %1 extension is needed, if you plan to use a %2 database.', 'pdo_mysql', 'MySQL')
 	),
 	'pgsql' => array(
 		'func' => 'extension_check',
-		'warning' => lang('The %1 extension is needed, if you plan to use a %2 database.','pgsql','pgSQL')
+		'warning' => lang('The %1 extension is needed, if you plan to use a %2 database.', 'pgsql', 'pgSQL')
 	),
 	'pdo_pgsql' => array(
 		'func' => 'extension_check',
-		'warning' => lang('The %1 extension is needed, if you plan to use a %2 database.','pdo_pgsql','pgSQL')
+		'warning' => lang('The %1 extension is needed, if you plan to use a %2 database.', 'pdo_pgsql', 'pgSQL')
 	),
 	/* disable checks for other database extensions, as we are not really supporting them anymore
 	'mssql' => array(
@@ -202,7 +202,7 @@ 
 	realpath('..') => array(
 		'func' => 'permission_check',
 		'is_world_writable' => False,
-		'only_if_exists' => true,	// quitens "file does not exist" for doc symlinks in Debian to files outside open_basedir
+		'only_if_exists' => true, // quitens "file does not exist" for doc symlinks in Debian to files outside open_basedir
 		'recursiv' => True
 	),
 	realpath('../header.inc.php') => array(
@@ -236,10 +236,10 @@ 
 	$sp_visible = true;
 	if (($open_basedir = ini_get('open_basedir')) && $open_basedir != 'none')
 	{
-		foreach(explode(PATH_SEPARATOR,$open_basedir) as $dir)
+		foreach (explode(PATH_SEPARATOR, $open_basedir) as $dir)
 		{
 			$dir = realpath($dir);
-			if (($sp_visible = substr($session_path,0,strlen($dir)) == $dir)) break;
+			if (($sp_visible = substr($session_path, 0, strlen($dir)) == $dir)) break;
 		}
 	}
 	if ($sp_visible)	// only check if session_save_path is visible by webserver
@@ -247,13 +247,13 @@ 
 		$checks[$session_path] = array(
 			'func' => 'permission_check',
 			'is_writable' => true,
-			'msg' => lang("Checking if php.ini setting session.save_path='%1' is writable by the webserver",session_save_path()),
+			'msg' => lang("Checking if php.ini setting session.save_path='%1' is writable by the webserver", session_save_path()),
 			'error' => lang('You will NOT be able to log into eGroupWare using PHP sessions: "session could not be verified" !!!'),
 		);
 	}
 }
 $setup_info = $GLOBALS['egw_setup']->detection->get_versions();
-foreach($setup_info as $app => $app_data)
+foreach ($setup_info as $app => $app_data)
 {
 	if (!isset($app_data['check_install'])) continue;
 
@@ -261,9 +261,9 @@ 
 	{
 		if (isset($checks[$name]))
 		{
-			if ($checks[$name] == $data) continue;	// identical check --> ignore it
+			if ($checks[$name] == $data) continue; // identical check --> ignore it
 
-			if ($data['func'] == 'pear_check' || in_array($data['func'],array('extension_check','php_ini_check')) && !isset($data['warning']))
+			if ($data['func'] == 'pear_check' || in_array($data['func'], array('extension_check', 'php_ini_check')) && !isset($data['warning']))
 			{
 				if (isset($checks[$name]['from']) && $checks[$name]['from'] && !is_array($checks[$name]['from']))
 				{
@@ -271,7 +271,7 @@ 
 				}
 				if (!isset($data['from'])) $data['from'] = $app;
 				if (!isset($checks[$name]['from']) || !is_array($checks[$name]['from'])) $checks[$name]['from'] = array();
-				if (!in_array($data['from'],$checks[$name]['from'])) $checks[$name]['from'][] = $data['from'];
+				if (!in_array($data['from'], $checks[$name]['from'])) $checks[$name]['from'][] = $data['from'];
 			}
 			else
 			{
@@ -287,9 +287,9 @@ 
 	}
 }
 $sorted_checks = array();
-foreach(array('php_version','php_ini_check','extension_check','pear_check','gd_check','permission_check') as $func)
+foreach (array('php_version', 'php_ini_check', 'extension_check', 'pear_check', 'gd_check', 'permission_check') as $func)
 {
-	foreach($checks as $name => $data)
+	foreach ($checks as $name => $data)
 	{
 		if ($data['func'] == $func)
 		{
@@ -300,15 +300,15 @@ 
 }
 if ($checks) $sorted_checks += $checks;
 
-function php_version($name,$args)
+function php_version($name, $args)
 {
 	global $passed_icon, $error_icon;
-	unset($name);	// not used, but required by function signature
+	unset($name); // not used, but required by function signature
 
-	$version_ok = version_compare(phpversion(),$args['value']) >= 0;
+	$version_ok = version_compare(phpversion(), $args['value']) >= 0;
 
 	echo '<div>'.($version_ok ? $passed_icon : $error_icon).' <span'.($version_ok ? '' : ' class="setup_error"').'>'.
-		lang('Checking required PHP version %1 (recommended %2)',$args['verbose_value'],$args['recommended']).': '.
+		lang('Checking required PHP version %1 (recommended %2)', $args['verbose_value'], $args['recommended']).': '.
 		phpversion().' ==> '.($version_ok ? lang('True') : lang('False'))."</span></div>\n";
 }
 
@@ -319,13 +319,13 @@ 
  */
 function composer_check($package)
 {
-	static $installed=null;
+	static $installed = null;
 	if (!isset($installed))
 	{
 		$installed = array();
-		if (file_exists(EGW_SERVER_ROOT.'/vendor') && file_exists($json=EGW_SERVER_ROOT.'/vendor/composer/installed.json'))
+		if (file_exists(EGW_SERVER_ROOT.'/vendor') && file_exists($json = EGW_SERVER_ROOT.'/vendor/composer/installed.json'))
 		{
-			foreach(json_decode(file_get_contents($json), true) as $package_data)
+			foreach (json_decode(file_get_contents($json), true) as $package_data)
 			{
 				$installed[strtolower($package_data['name'])] = $package_data['version'];
 			}
@@ -341,9 +341,9 @@ 
  * @param $channel =null use default or given channel
  * @return array with package-name => version pairs, eg. array('Log' => '1.9.8','PEAR' => '1.4.11')
  */
-function get_installed_pear_packages($channel=null)
+function get_installed_pear_packages($channel = null)
 {
-	$pear_config = '';	// use the system default
+	$pear_config = ''; // use the system default
 	// fix for SuSE having the pear.conf only for cli, will fail with open_basedir - no idea what to do then
 	if (@is_dir('/etc/php5/apache2') && !file_exists('/etc/php5/apache2/pear.conf') && @file_exists('/etc/php5/cli/pear.conf'))
 	{
@@ -353,13 +353,13 @@ 
 
 	if (!class_exists('PEAR_Config')) return false;
 
-	$config = new PEAR_Config('',$pear_config);
+	$config = new PEAR_Config('', $pear_config);
 	//echo "<pre>config = ".print_r($config,true)."</pre>\n";
 
 	if (empty($channel)) $channel = $config->get('default_channel');
 	//echo "<pre>channel = ".print_r($channel,true)."</pre>\n";
 
-	if (!method_exists($config,'getRegistry')) return false;	// PEAR version to old
+	if (!method_exists($config, 'getRegistry')) return false; // PEAR version to old
 
 	$reg = &$config->getRegistry();
 	//echo "<pre>reg = ".print_r($reg,true)."</pre>\n";
@@ -368,11 +368,11 @@ 
 	// bug reported: http://pear.php.net/bugs/bug.php?id=11317
 	if (!file_exists($reg->install_dir)) return false;
 
-	$installed = $reg->packageInfo(null,null,$channel);
+	$installed = $reg->packageInfo(null, null, $channel);
 
 	//echo "<pre>installed =".print_r($installed,true)."</pre>\n";
 	$packages = array();
-	foreach($installed as $package)
+	foreach ($installed as $package)
 	{
 		$name = isset($package['package']) ? $package['package'] : $package['name'];
 		$version = $package['version'];
@@ -386,7 +386,7 @@ 
 	return $packages;
 }
 
-function pear_check($package,$args)
+function pear_check($package, $args)
 {
 	global $passed_icon, $warning_icon;
 	static $pear_available = null;
@@ -430,7 +430,7 @@ 
 			$found = $pear_available;
 			if ($pear_available && $package)
 			{
-				$file = str_replace('_','/',$package == 'Mail_Mime' ? 'Mail_mime' : $package).'.php';
+				$file = str_replace('_', '/', $package == 'Mail_Mime' ? 'Mail_mime' : $package).'.php';
 
 				$found = @include_once($file);
 
@@ -439,16 +439,15 @@ 
 		}
 	}
 	// is the right version availible
-	$available = $found && (!$min_version || version_compare($min_version,$version_available) <= 0);
+	$available = $found && (!$min_version || version_compare($min_version, $version_available) <= 0);
 	echo '<div>'.($available ? $passed_icon : $warning_icon).' <span'.($available ? '' : ' class="setup_warning"').'>'.
-		lang('Checking PEAR%1 is installed',($package?($channel?' '.$channel.'/':'::').$package:'').($min_version?" ($min_version)":'')).': '.
-		($available ? ($version_available ? $version_available : lang('True')) :
-		($found ? lang('Found, but unknown version') : lang('False')))."</span></div>\n";
+		lang('Checking PEAR%1 is installed', ($package ? ($channel ? ' '.$channel.'/' : '::').$package : '').($min_version ? " ($min_version)" : '')).': '.
+		($available ? ($version_available ? $version_available : lang('True')) : ($found ? lang('Found, but unknown version') : lang('False')))."</span></div>\n";
 
 	if (!$available)	// give further info only if not availible
 	{
-		echo '<div class="setup_info">' . lang('PEAR%1 is needed by: %2.',$package ? '::'.$package : '',
-			is_array($args['from']) ? implode(', ',$args['from']) : $args['from']);
+		echo '<div class="setup_info">'.lang('PEAR%1 is needed by: %2.', $package ? '::'.$package : '',
+			is_array($args['from']) ? implode(', ', $args['from']) : $args['from']);
 
 		// if using Composer, dont confuse user with PEAR ;-)
 		if (file_exists(EGW_SERVER_ROOT.'/vendor'))
@@ -461,7 +460,7 @@ 
 			if (!$pear_available)
 			{
 				echo '<br/>'.lang('PEAR (%1) is a PHP repository and is usually in a package called %2.',
-					'<a href="http://pear.php.net" target="_blank">pear.php.net</a>','php-pear');
+					'<a href="http://pear.php.net" target="_blank">pear.php.net</a>', 'php-pear');
 			}
 			elseif ($package && !$found)
 			{
@@ -471,12 +470,12 @@ 
 			}
 			elseif ($min_version && !$version_available)
 			{
-				echo '<br/>'.lang('We could not determine the version of %1, please make sure it is at least %2',$package,$min_version);
+				echo '<br/>'.lang('We could not determine the version of %1, please make sure it is at least %2', $package, $min_version);
 			}
-			elseif ($min_version && version_compare($min_version,$version_available) > 0)
+			elseif ($min_version && version_compare($min_version, $version_available) > 0)
 			{
 				echo '<br/>'.lang('Your installed version of %1 is %2, required is at least %3, please run: ',
-					$package,$version_available,$min_version).' pear upgrade '.$package;
+					$package, $version_available, $min_version).' pear upgrade '.$package;
 			}
 			echo '<br/>'.lang('Alternatively you can use %1Composer%2 to install all requirements at once. Downloading it and run:',
 				'<a href="https://getcomposer.org/" target="_blank">', '</a>').
@@ -489,26 +488,26 @@ 
 	return $available;
 }
 
-function extension_check($name,$args)
+function extension_check($name, $args)
 {
 	//echo "<p>extension_check($name,".print_r($args,true).")</p>\n";
 	global $passed_icon, $warning_icon, $is_windows;
 
 	if (isset($args['win_only']) && $args['win_only'] && !$is_windows)
 	{
-		return True;	// check only under windows
+		return True; // check only under windows
 	}
 	// we check for the existens of 'dl', as multithreaded webservers dont have it !!!
 	$available = check_load_extension($name);
 
-	echo '<div>'.($available ? $passed_icon : $warning_icon).' <span'.($available ? '' : ' class="setup_warning"').'>'.lang('Checking extension %1 is loaded or loadable',$name).': '.($available ? lang('True') : lang('False'))."</span></div>\n";
+	echo '<div>'.($available ? $passed_icon : $warning_icon).' <span'.($available ? '' : ' class="setup_warning"').'>'.lang('Checking extension %1 is loaded or loadable', $name).': '.($available ? lang('True') : lang('False'))."</span></div>\n";
 
 	if (!$available)
 	{
 		if (!isset($args['warning']))
 		{
-			$args['warning'] = lang('The %1 extension is needed from: %2.',$name,
-				is_array($args['from']) ? implode(', ',$args['from']) : $args['from']);
+			$args['warning'] = lang('The %1 extension is needed from: %2.', $name,
+				is_array($args['from']) ? implode(', ', $args['from']) : $args['from']);
 		}
 		echo "<div class='setup_info'>".$args['warning'].'</div>';
 	}
@@ -517,20 +516,20 @@ 
 	return $available;
 }
 
-function function_check($name,$args)
+function function_check($name, $args)
 {
 	global $passed_icon, $warning_icon;
 
 	$available = function_exists($name);
 
-	echo '<div>'.($available ? $passed_icon : $warning_icon).' <span'.($available ? '' : ' class="setup_warning"').'>'.lang('Checking function %1 exists',$name).': '.($available ? lang('True') : lang('False'))."</span></div>\n";
+	echo '<div>'.($available ? $passed_icon : $warning_icon).' <span'.($available ? '' : ' class="setup_warning"').'>'.lang('Checking function %1 exists', $name).': '.($available ? lang('True') : lang('False'))."</span></div>\n";
 
 	if (!$available)
 	{
 		if (!isset($args['warning']))
 		{
-			$args['warning'] = lang('The function %1 is needed from: %2.',$name,
-				is_array($args['from'] ? implode(', ',$args['from']) : $args['from']));
+			$args['warning'] = lang('The function %1 is needed from: %2.', $name,
+				is_array($args['from'] ? implode(', ', $args['from']) : $args['from']));
 		}
 		echo "<div class='setup_info'>".$args['warning'].'</div>';
 	}
@@ -539,33 +538,33 @@ 
 	return $available;
 }
 
-function verbosePerms( $in_Perms )
+function verbosePerms($in_Perms)
 {
-	if($in_Perms & 0x1000)     // FIFO pipe
+	if ($in_Perms&0x1000)     // FIFO pipe
 	{
 		$sP = 'p';
 	}
-	elseif($in_Perms & 0x2000) // Character special
+	elseif ($in_Perms&0x2000) // Character special
 	{
 		$sP = 'c';
 	}
-	elseif($in_Perms & 0x4000) // Directory
+	elseif ($in_Perms&0x4000) // Directory
 	{
 		$sP = 'd';
 	}
-	elseif($in_Perms & 0x6000) // Block special
+	elseif ($in_Perms&0x6000) // Block special
 	{
 		$sP = 'b';
 	}
-	elseif($in_Perms & 0x8000) // Regular
+	elseif ($in_Perms&0x8000) // Regular
 	{
 		$sP = '-';
 	}
-	elseif($in_Perms & 0xA000) // Symbolic Link
+	elseif ($in_Perms&0xA000) // Symbolic Link
 	{
 		$sP = 'l';
 	}
-	elseif($in_Perms & 0xC000) // Socket
+	elseif ($in_Perms&0xC000) // Socket
 	{
 		$sP = 's';
 	}
@@ -575,33 +574,30 @@ 
 	}
 
 	// owner
-	$sP .= (($in_Perms & 0x0100) ? 'r' : '-') .
-	(($in_Perms & 0x0080) ? 'w' : '-') .
-	(($in_Perms & 0x0040) ? (($in_Perms & 0x0800) ? 's' : 'x' ) :
-	(($in_Perms & 0x0800) ? 'S' : '-'));
+	$sP .= (($in_Perms&0x0100) ? 'r' : '-').
+	(($in_Perms&0x0080) ? 'w' : '-').
+	(($in_Perms&0x0040) ? (($in_Perms&0x0800) ? 's' : 'x') : (($in_Perms&0x0800) ? 'S' : '-'));
 
 	// group
-	$sP .= (($in_Perms & 0x0020) ? 'r' : '-') .
-	(($in_Perms & 0x0010) ? 'w' : '-') .
-	(($in_Perms & 0x0008) ? (($in_Perms & 0x0400) ? 's' : 'x' ) :
-	(($in_Perms & 0x0400) ? 'S' : '-'));
+	$sP .= (($in_Perms&0x0020) ? 'r' : '-').
+	(($in_Perms&0x0010) ? 'w' : '-').
+	(($in_Perms&0x0008) ? (($in_Perms&0x0400) ? 's' : 'x') : (($in_Perms&0x0400) ? 'S' : '-'));
 
 	// world
-	$sP .= (($in_Perms & 0x0004) ? 'r' : '-') .
-	(($in_Perms & 0x0002) ? 'w' : '-') .
-	(($in_Perms & 0x0001) ? (($in_Perms & 0x0200) ? 't' : 'x' ) :
-	(($in_Perms & 0x0200) ? 'T' : '-'));
+	$sP .= (($in_Perms&0x0004) ? 'r' : '-').
+	(($in_Perms&0x0002) ? 'w' : '-').
+	(($in_Perms&0x0001) ? (($in_Perms&0x0200) ? 't' : 'x') : (($in_Perms&0x0200) ? 'T' : '-'));
 	return $sP;
 }
 
-function permission_check($name,$args,$verbose=True)
+function permission_check($name, $args, $verbose = True)
 {
-	global $passed_icon, $error_icon, $warning_icon,$is_windows;
+	global $passed_icon, $error_icon, $warning_icon, $is_windows;
 	//echo "<p>permision_check('$name',".print_r($args,True).",'$verbose')</p>\n";
 
 	// add a ../ for non-absolute pathes
 	$rel_name = $name;
-	if ($name && substr($name,0,3) != '../' && $name[0] != '/' && $name[0] != '\\' && strpos($name,':') === false)
+	if ($name && substr($name, 0, 3) != '../' && $name[0] != '/' && $name[0] != '\\' && strpos($name, ':') === false)
 	{
 		$name = '../'.$name;
 	}
@@ -623,22 +619,22 @@ 
 	if (isset($args['is_readable']))
 	{
 		$checks[] = lang('readable by the webserver');
-		$check_not = (!$args['is_readable']?lang('not'):'');
+		$check_not = (!$args['is_readable'] ? lang('not') : '');
 	}
 	if (isset($args['is_writable']))
 	{
 		$checks[] = lang('writable by the webserver');
-		$check_not = (!$args['is_writable']?lang('not'):'');
+		$check_not = (!$args['is_writable'] ? lang('not') : '');
 	}
 	if (isset($args['is_world_readable']))
 	{
 		$checks[] = lang('world readable');
-		$check_not = (!$args['is_world_readable']?lang('not'):'');
+		$check_not = (!$args['is_world_readable'] ? lang('not') : '');
 	}
 	if (isset($args['is_world_writable']))
 	{
 		$checks[] = lang('world writable');
-		$check_not = (!$args['is_world_writable']?lang('not'):'');
+		$check_not = (!$args['is_world_writable'] ? lang('not') : '');
 	}
 
 	if (isset($args['msg']) && ($msg = $args['msg']))
@@ -647,7 +643,7 @@ 
 	}
 	else
 	{
-		$msg = lang('Checking file-permissions of %1 for %2 %3: %4',$rel_name,$check_not,implode(', ',$checks),$perms)."<br />\n";
+		$msg = lang('Checking file-permissions of %1 for %2 %3: %4', $rel_name, $check_not, implode(', ', $checks), $perms)."<br />\n";
 	}
 	$extra_error_msg = '';
 	if (isset($args['error']) && $args['error'])
@@ -656,13 +652,13 @@ 
 	}
 	if (!file_exists($name))
 	{
-		echo '<div>'. $error_icon . '<span class="setup_error">' . $msg . lang('%1 does not exist !!!',$rel_name).$extra_error_msg."</span></div>\n";
+		echo '<div>'.$error_icon.'<span class="setup_error">'.$msg.lang('%1 does not exist !!!', $rel_name).$extra_error_msg."</span></div>\n";
 		return False;
 	}
 	$warning = False;
 	if (!$GLOBALS['run_by_webserver'] && (@$args['is_readable'] || @$args['is_writable']))
 	{
-		echo $warning_icon.' '.$msg. lang('Check can only be performed, if called via a webserver, as the user-id/-name of the webserver is not known.')."\n";
+		echo $warning_icon.' '.$msg.lang('Check can only be performed, if called via a webserver, as the user-id/-name of the webserver is not known.')."\n";
 		unset($args['is_readable']);
 		unset($args['is_writable']);
 		$warning = True;
@@ -670,22 +666,22 @@ 
 	$Ok = True;
 	if (isset($args['is_writable']) && is_writable($name) != $args['is_writable'])
 	{
-		echo '<div>'.$error_icon.' <span class="setup_error">'.$msg.' '.lang('%1 is %2%3 !!!',$rel_name,$args['is_writable']?lang('not').' ':'',lang('writable by the webserver')).$extra_error_msg."</span></div>\n";
+		echo '<div>'.$error_icon.' <span class="setup_error">'.$msg.' '.lang('%1 is %2%3 !!!', $rel_name, $args['is_writable'] ? lang('not').' ' : '', lang('writable by the webserver')).$extra_error_msg."</span></div>\n";
 		$Ok = False;
 	}
 	if (isset($args['is_readable']) && is_readable($name) != $args['is_readable'])
 	{
-		echo '<div>'.$error_icon.' <span class="setup_error">'.$msg.' '.lang('%1 is %2%3 !!!',$rel_name,$args['is_readable']?lang('not').' ':'',lang('readable by the webserver')).$extra_error_msg."</span></div>\n";
+		echo '<div>'.$error_icon.' <span class="setup_error">'.$msg.' '.lang('%1 is %2%3 !!!', $rel_name, $args['is_readable'] ? lang('not').' ' : '', lang('readable by the webserver')).$extra_error_msg."</span></div>\n";
 		$Ok = False;
 	}
-	if (!$is_windows && isset($args['is_world_readable']) && !(fileperms($name) & 04) == $args['is_world_readable'])
+	if (!$is_windows && isset($args['is_world_readable']) && !(fileperms($name)&04) == $args['is_world_readable'])
 	{
-		echo '<div>'.$error_icon.' <span class="setup_error">'.$msg.' '.lang('%1 is %2%3 !!!',$rel_name,$args['is_world_readable']?lang('not').' ':'',lang('world readable')).$extra_error_msg."</span></div>\n";
+		echo '<div>'.$error_icon.' <span class="setup_error">'.$msg.' '.lang('%1 is %2%3 !!!', $rel_name, $args['is_world_readable'] ? lang('not').' ' : '', lang('world readable')).$extra_error_msg."</span></div>\n";
 		$Ok = False;
 	}
-	if (!$is_windows && isset($args['is_world_writable']) && !(fileperms($name) & 02) == $args['is_world_writable'])
+	if (!$is_windows && isset($args['is_world_writable']) && !(fileperms($name)&02) == $args['is_world_writable'])
 	{
-		echo '<div>'.$error_icon.' <span class="setup_error">'.$msg.' '.lang('%1 is %2%3 !!!',$rel_name,$args['is_world_writable']?lang('not').' ':'',lang('world writable')).$extra_error_msg."</span></div>\n";
+		echo '<div>'.$error_icon.' <span class="setup_error">'.$msg.' '.lang('%1 is %2%3 !!!', $rel_name, $args['is_world_writable'] ? lang('not').' ' : '', lang('world writable')).$extra_error_msg."</span></div>\n";
 		$Ok = False;
 	}
 	if ($Ok && !$warning && $verbose)
@@ -697,16 +693,16 @@ 
 		if ($verbose)
 		{
 			@set_time_limit(0);
-			echo "<div class='setup_info'>" . lang('This might take a while, please wait ...')."</div>\n";
+			echo "<div class='setup_info'>".lang('This might take a while, please wait ...')."</div>\n";
 			flush();
 		}
 		@set_time_limit(0);
 		$handle = @opendir($name);
-		while($handle && ($file = readdir($handle)))
+		while ($handle && ($file = readdir($handle)))
 		{
 			if ($file != '.' && $file != '..')
 			{
-				$Ok = $Ok && permission_check(($name!='.'?$name.'/':'').$file,$args,False);
+				$Ok = $Ok && permission_check(($name != '.' ? $name.'/' : '').$file, $args, False);
 			}
 		}
 		if ($handle) closedir($handle);
@@ -719,12 +715,12 @@ 
 function mk_value($value)
 {
 	$matches = null;
-	if (!preg_match('/^([0-9]+)([mk]+)$/i',$value,$matches)) return $value;
+	if (!preg_match('/^([0-9]+)([mk]+)$/i', $value, $matches)) return $value;
 
-	return (strtolower($matches[2]) == 'm' ? 1024*1024 : 1024) * (int) $matches[1];
+	return (strtolower($matches[2]) == 'm' ? 1024 * 1024 : 1024) * (int)$matches[1];
 }
 
-function php_ini_check($name,$args)
+function php_ini_check($name, $args)
 {
 	global $passed_icon, $error_icon, $warning_icon, $is_windows;
 
@@ -742,20 +738,20 @@ 
 	{
 		case 'not set':
 			$check = lang('not set');
-			$result = !($ini_value & $args['value']);
+			$result = !($ini_value&$args['value']);
 			break;
 		case 'set':
 			$check = lang('set');
-			$result = !!($ini_value & $args['value']);
+			$result = !!($ini_value&$args['value']);
 			break;
 		case '>=':
-			$result = !$ini_value ||	// value not used, eg. no memory limit
-			(int) mk_value($ini_value) >= (int) mk_value($args['value']);
+			$result = !$ini_value || // value not used, eg. no memory limit
+			(int)mk_value($ini_value) >= (int)mk_value($args['value']);
 			break;
 		case 'contain':
 			$check = lang('contain');
 			$sep = $is_windows ? '/[; ]+/' : '/[: ]+/';
-			$result = in_array($args['value'],preg_split($sep,$ini_value));
+			$result = in_array($args['value'], preg_split($sep, $ini_value));
 			break;
 		case '!=':
 			$check = lang('set and not');
@@ -772,9 +768,9 @@ 
 			$tz = new DateTimeZone($ini_value);
 			unset($tz);
 		}
-		catch(Exception $e) {
+		catch (Exception $e) {
 			unset($e);
-			$result = false;	// no valid timezone
+			$result = false; // no valid timezone
 		}
 	}
 	$msg = ' '.lang('Checking php.ini').": $name $check $verbose_value: <span class='setup_info'>ini_get('$name')='$ini_value'$ini_value_verbose</span>";
@@ -796,7 +792,7 @@ 
 		elseif (!isset($args['safe_mode']))
 		{
 			echo "<div>".$warning_icon.' <span class="setup_warning">'.$msg.'</span><div class="setup_info">'.
-				lang('%1 is needed by: %2.',$name,is_array($args['from']) ? implode(', ',$args['from']) : $args['from'])
+				lang('%1 is needed by: %2.', $name, is_array($args['from']) ? implode(', ', $args['from']) : $args['from'])
 				."</div></div>\n";
 		}
 		if (isset($args['safe_mode']) && $safe_mode || @$args['change'])
@@ -806,7 +802,7 @@ 
 				echo '<div>'.$error_icon.' <span class="setup_error">'.$msg.'</span></div>';
 			}
 			echo "<div class='setup_error'>\n";
-			echo '*** '.lang('Please make the following change in your php.ini').' ('.get_php_ini().'): '.(@$args['safe_mode']?$args['safe_mode']:$args['change'])."<br />\n";
+			echo '*** '.lang('Please make the following change in your php.ini').' ('.get_php_ini().'): '.(@$args['safe_mode'] ? $args['safe_mode'] : $args['change'])."<br />\n";
 			echo '*** '.lang('AND reload your webserver, so the above changes take effect !!!')."</div>\n";
 		}
 	}
@@ -821,16 +817,16 @@ 
 	ob_end_clean();
 
 	$found = null;
-	return preg_match('/\(php.ini\).*<\/td><td[^>]*>([^ <]+)/',$phpinfo,$found) ? $found[1] : False;
+	return preg_match('/\(php.ini\).*<\/td><td[^>]*>([^ <]+)/', $phpinfo, $found) ? $found[1] : False;
 }
 
 function gd_check()
 {
 	global $passed_icon, $warning_icon;
 
-	$available = (function_exists('imagecopyresampled')  || function_exists('imagecopyresized'));
+	$available = (function_exists('imagecopyresampled') || function_exists('imagecopyresized'));
 
-	echo "<div>".($available ? $passed_icon : $warning_icon).' <span'.($available?'':' class="setup_warning"').'>'.lang('Checking for GD support...').': '.($available ? lang('True') : lang('False'))."</span></div>\n";
+	echo "<div>".($available ? $passed_icon : $warning_icon).' <span'.($available ? '' : ' class="setup_warning"').'>'.lang('Checking for GD support...').': '.($available ? lang('True') : lang('False'))."</span></div>\n";
 
 	if (!$available)
 	{
@@ -849,24 +845,24 @@ 
 	));
 	$ConfigDomain = $_REQUEST['ConfigDomain'];
 	if (@$_GET['intro']) {
-		if(($ConfigLang = setup::get_lang()))
+		if (($ConfigLang = setup::get_lang()))
 		{
-			$GLOBALS['egw_setup']->set_cookie('ConfigLang',$ConfigLang,(int) (time()+(1200*9)),'/');
+			$GLOBALS['egw_setup']->set_cookie('ConfigLang', $ConfigLang, (int)(time() + (1200 * 9)), '/');
 		}
-		$GLOBALS['egw_setup']->html->show_header(lang('Welcome to the eGroupWare Installation'),False,'config');
+		$GLOBALS['egw_setup']->html->show_header(lang('Welcome to the eGroupWare Installation'), False, 'config');
 		echo '<h1>'.lang('Welcome to the eGroupWare Installation')."</h1>\n";
-		if(!$ConfigLang)
+		if (!$ConfigLang)
 		{
-			echo '<p><form action="check_install.php?intro=1" method="Post">Please Select your language '.setup_html::lang_select(True,'en')."</form></p>\n";
+			echo '<p><form action="check_install.php?intro=1" method="Post">Please Select your language '.setup_html::lang_select(True, 'en')."</form></p>\n";
 		}
 		echo '<p>'.lang('The first step in installing eGroupWare is to ensure your environment has the necessary settings to correctly run the application.').'</p>';
 		echo '<p>'.lang('We will now run a series of tests, which may take a few minutes.  Click the link below to proceed.').'</p>';
 		echo '<h3><a href="check_install.php">'.lang('Run installation tests').'</a></h3>';
 		echo '<p><a href="manageheader.php">'.lang('Skip the installation tests (not recommended)')."</a></p>\n";
-		$setup_tpl->pparse('out','T_footer');
+		$setup_tpl->pparse('out', 'T_footer');
 		exit;
 	} else {
-		$GLOBALS['egw_setup']->html->show_header(lang('Checking the eGroupWare Installation'),False,'config',$ConfigDomain ? $ConfigDomain . '(' . @$GLOBALS['egw_domain'][$ConfigDomain]['db_type'] . ')' : '');
+		$GLOBALS['egw_setup']->html->show_header(lang('Checking the eGroupWare Installation'), False, 'config', $ConfigDomain ? $ConfigDomain.'('.@$GLOBALS['egw_domain'][$ConfigDomain]['db_type'].')' : '');
 		echo '<h1>'.lang('Checking the eGroupWare Installation')."</h1>\n";
 		# echo "<pre style=\"text-align: left;\">\n";;
 	}
@@ -880,7 +876,7 @@ 
 $Ok = True;
 foreach ($sorted_checks as $name => $args)
 {
-	$check_ok = $args['func']($name,$args);
+	$check_ok = $args['func']($name, $args);
 	$Ok = $Ok && $check_ok;
 }
 
@@ -892,9 +888,9 @@ 
 	{
 		if (!$Ok)
 		{
-			echo '<h3>'.lang('Please fix the above errors (%1) and warnings(%2)',$error_icon,$warning_icon)."</h3>\n";
+			echo '<h3>'.lang('Please fix the above errors (%1) and warnings(%2)', $error_icon, $warning_icon)."</h3>\n";
 			echo '<h3><a href="check_install.php">'.lang('Click here to re-run the installation tests')."</a></h3>\n";
-			echo '<h3>'.lang('or %1Continue to the Header Admin%2','<a href="manageheader.php">','</a>')."</h3>\n";
+			echo '<h3>'.lang('or %1Continue to the Header Admin%2', '<a href="manageheader.php">', '</a>')."</h3>\n";
 		}
 		else
 		{
@@ -906,10 +902,10 @@ 
 		echo '<h3>';
 		if (!$Ok)
 		{
-			echo lang('Please fix the above errors (%1) and warnings(%2)',$error_icon,$warning_icon).'. ';
+			echo lang('Please fix the above errors (%1) and warnings(%2)', $error_icon, $warning_icon).'. ';
 		}
-		echo '<br /><a href="'.str_replace('check_install.php','',@$_SERVER['HTTP_REFERER']).'">'.lang('Return to Setup')."</a></h3>\n";
+		echo '<br /><a href="'.str_replace('check_install.php', '', @$_SERVER['HTTP_REFERER']).'">'.lang('Return to Setup')."</a></h3>\n";
 	}
-	$setup_tpl->pparse('out','T_footer');
+	$setup_tpl->pparse('out', 'T_footer');
 	//echo "</body>\n</html>\n";
 }

Braces +119 added lines, -30 removed lines

@@ -239,10 +239,15 @@ 
 		foreach(explode(PATH_SEPARATOR,$open_basedir) as $dir)
 		{
 			$dir = realpath($dir);
-			if (($sp_visible = substr($session_path,0,strlen($dir)) == $dir)) break;
+			if (($sp_visible = substr($session_path,0,strlen($dir)) == $dir))
+			{
+				break;
+			}
 		}
 	}
-	if ($sp_visible)	// only check if session_save_path is visible by webserver
+	if ($sp_visible)
+	{
+		// only check if session_save_path is visible by webserver
 	{
 		$checks[$session_path] = array(
 			'func' => 'permission_check',
@@ -251,17 +256,25 @@ 
 			'error' => lang('You will NOT be able to log into eGroupWare using PHP sessions: "session could not be verified" !!!'),
 		);
 	}
+	}
 }
 $setup_info = $GLOBALS['egw_setup']->detection->get_versions();
 foreach($setup_info as $app => $app_data)
 {
-	if (!isset($app_data['check_install'])) continue;
+	if (!isset($app_data['check_install']))
+	{
+		continue;
+	}
 
 	foreach ($app_data['check_install'] as $name => $data)
 	{
 		if (isset($checks[$name]))
 		{
-			if ($checks[$name] == $data) continue;	// identical check --> ignore it
+			if ($checks[$name] == $data)
+			{
+				continue;
+			}
+			// identical check --> ignore it
 
 			if ($data['func'] == 'pear_check' || in_array($data['func'],array('extension_check','php_ini_check')) && !isset($data['warning']))
 			{
@@ -269,9 +282,18 @@ 
 				{
 					$checks[$name]['from'] = array($checks[$name]['from']);
 				}
-				if (!isset($data['from'])) $data['from'] = $app;
-				if (!isset($checks[$name]['from']) || !is_array($checks[$name]['from'])) $checks[$name]['from'] = array();
-				if (!in_array($data['from'],$checks[$name]['from'])) $checks[$name]['from'][] = $data['from'];
+				if (!isset($data['from']))
+				{
+					$data['from'] = $app;
+				}
+				if (!isset($checks[$name]['from']) || !is_array($checks[$name]['from']))
+				{
+					$checks[$name]['from'] = array();
+				}
+				if (!in_array($data['from'],$checks[$name]['from']))
+				{
+					$checks[$name]['from'][] = $data['from'];
+				}
 			}
 			else
 			{
@@ -280,7 +302,10 @@ 
 		}
 		else
 		{
-			if (!isset($data['from'])) $data['from'] = $app;
+			if (!isset($data['from']))
+			{
+				$data['from'] = $app;
+			}
 			$checks[$name] = $data;
 		}
 		//echo "added check $data[func]($name) for $app"; _debug_array($data);
@@ -298,7 +323,10 @@ 
 		}
 	}
 }
-if ($checks) $sorted_checks += $checks;
+if ($checks)
+{
+	$sorted_checks += $checks;
+}
 
 function php_version($name,$args)
 {
@@ -351,22 +379,35 @@ 
 	}
 	@include_once 'PEAR/Config.php';
 
-	if (!class_exists('PEAR_Config')) return false;
+	if (!class_exists('PEAR_Config'))
+	{
+		return false;
+	}
 
 	$config = new PEAR_Config('',$pear_config);
 	//echo "<pre>config = ".print_r($config,true)."</pre>\n";
 
-	if (empty($channel)) $channel = $config->get('default_channel');
+	if (empty($channel))
+	{
+		$channel = $config->get('default_channel');
+	}
 	//echo "<pre>channel = ".print_r($channel,true)."</pre>\n";
 
-	if (!method_exists($config,'getRegistry')) return false;	// PEAR version to old
+	if (!method_exists($config,'getRegistry'))
+	{
+		return false;
+	}
+	// PEAR version to old
 
 	$reg = &$config->getRegistry();
 	//echo "<pre>reg = ".print_r($reg,true)."</pre>\n";
 
 	// a bug in pear causes an endless loop if the install-dir does not exist
 	// bug reported: http://pear.php.net/bugs/bug.php?id=11317
-	if (!file_exists($reg->install_dir)) return false;
+	if (!file_exists($reg->install_dir))
+	{
+		return false;
+	}
 
 	$installed = $reg->packageInfo(null,null,$channel);
 
@@ -376,7 +417,10 @@ 
 	{
 		$name = isset($package['package']) ? $package['package'] : $package['name'];
 		$version = $package['version'];
-		if (is_array($version)) $version = $version['release'];
+		if (is_array($version))
+		{
+			$version = $version['release'];
+		}
 
 		$packages[$name] = $version;
 		//echo "<p>$name: ".print_r($package['version'],true)."</p>\n";
@@ -415,7 +459,10 @@ 
 		{
 			$pear_available = $found = true;
 			// check if package is installed
-			if ($package && isset($pear_packages[$package])) $available = true;
+			if ($package && isset($pear_packages[$package]))
+			{
+				$available = true;
+			}
 			// check if it's the right version
 			$version_available = $pear_packages[$package ? $package : 'PEAR'];
 		}
@@ -425,7 +472,10 @@ 
 			{
 				$pear_available = @include_once('PEAR.php');
 
-				if (!class_exists('PEAR')) $pear_available = false;
+				if (!class_exists('PEAR'))
+				{
+					$pear_available = false;
+				}
 			}
 			$found = $pear_available;
 			if ($pear_available && $package)
@@ -434,7 +484,10 @@ 
 
 				$found = @include_once($file);
 
-				if (!class_exists($package)) $found = false;
+				if (!class_exists($package))
+				{
+					$found = false;
+				}
 			}
 		}
 	}
@@ -445,10 +498,13 @@ 
 		($available ? ($version_available ? $version_available : lang('True')) :
 		($found ? lang('Found, but unknown version') : lang('False')))."</span></div>\n";
 
-	if (!$available)	// give further info only if not availible
+	if (!$available)
+	{
+		// give further info only if not availible
 	{
 		echo '<div class="setup_info">' . lang('PEAR%1 is needed by: %2.',$package ? '::'.$package : '',
 			is_array($args['from']) ? implode(', ',$args['from']) : $args['from']);
+	}
 
 		// if using Composer, dont confuse user with PEAR ;-)
 		if (file_exists(EGW_SERVER_ROOT.'/vendor'))
@@ -541,34 +597,55 @@ 
 
 function verbosePerms( $in_Perms )
 {
-	if($in_Perms & 0x1000)     // FIFO pipe
+	if($in_Perms & 0x1000)
+	{
+		// FIFO pipe
 	{
 		$sP = 'p';
 	}
-	elseif($in_Perms & 0x2000) // Character special
+	}
+	elseif($in_Perms & 0x2000)
+	{
+		// Character special
 	{
 		$sP = 'c';
 	}
-	elseif($in_Perms & 0x4000) // Directory
+	}
+	elseif($in_Perms & 0x4000)
+	{
+		// Directory
 	{
 		$sP = 'd';
 	}
-	elseif($in_Perms & 0x6000) // Block special
+	}
+	elseif($in_Perms & 0x6000)
+	{
+		// Block special
 	{
 		$sP = 'b';
 	}
-	elseif($in_Perms & 0x8000) // Regular
+	}
+	elseif($in_Perms & 0x8000)
+	{
+		// Regular
 	{
 		$sP = '-';
 	}
-	elseif($in_Perms & 0xA000) // Symbolic Link
+	}
+	elseif($in_Perms & 0xA000)
+	{
+		// Symbolic Link
 	{
 		$sP = 'l';
 	}
-	elseif($in_Perms & 0xC000) // Socket
+	}
+	elseif($in_Perms & 0xC000)
+	{
+		// Socket
 	{
 		$sP = 's';
 	}
+	}
 	else                         // UNKNOWN
 	{
 		$sP = 'u';
@@ -709,9 +786,15 @@ 
 				$Ok = $Ok && permission_check(($name!='.'?$name.'/':'').$file,$args,False);
 			}
 		}
-		if ($handle) closedir($handle);
+		if ($handle)
+		{
+			closedir($handle);
+		}
+	}
+	if ($verbose)
+	{
+		echo "\n";
 	}
-	if ($verbose) echo "\n";
 
 	return $Ok;
 }
@@ -719,7 +802,10 @@ 
 function mk_value($value)
 {
 	$matches = null;
-	if (!preg_match('/^([0-9]+)([mk]+)$/i',$value,$matches)) return $value;
+	if (!preg_match('/^([0-9]+)([mk]+)$/i',$value,$matches))
+	{
+		return $value;
+	}
 
 	return (strtolower($matches[2]) == 'm' ? 1024*1024 : 1024) * (int) $matches[1];
 }
@@ -848,7 +934,8 @@ 
 		'T_footer' => 'footer.tpl',
 	));
 	$ConfigDomain = $_REQUEST['ConfigDomain'];
-	if (@$_GET['intro']) {
+	if (@$_GET['intro'])
+	{
 		if(($ConfigLang = setup::get_lang()))
 		{
 			$GLOBALS['egw_setup']->set_cookie('ConfigLang',$ConfigLang,(int) (time()+(1200*9)),'/');
@@ -865,7 +952,9 @@ 
 		echo '<p><a href="manageheader.php">'.lang('Skip the installation tests (not recommended)')."</a></p>\n";
 		$setup_tpl->pparse('out','T_footer');
 		exit;
-	} else {
+	}
+	else
+	{
 		$GLOBALS['egw_setup']->html->show_header(lang('Checking the eGroupWare Installation'),False,'config',$ConfigDomain ? $ConfigDomain . '(' . @$GLOBALS['egw_domain'][$ConfigDomain]['db_type'] . ')' : '');
 		echo '<h1>'.lang('Checking the eGroupWare Installation')."</h1>\n";
 		# echo "<pre style=\"text-align: left;\">\n";;

setup/inc/class.setup.inc.php

Doc Comments +5 added lines, -5 removed lines

@@ -741,8 +741,8 @@ 
 	/**
 	 * egw version checking, is param 1 < param 2 in phpgw versionspeak?
 	 * @param	$a	phpgw version number to check if less than $b
-	 * @param	$b	phpgw version number to check $a against
-	 * @return	True if $a < $b
+	 * @param	string $b	phpgw version number to check $a against
+	 * @return	boolean if $a < $b
 	 */
 	function alessthanb($a,$b,$DEBUG=False)
 	{
@@ -829,7 +829,7 @@ 
 	 *
 	 * @param	$a	phpgw version number to check if more than $b
 	 * @param	$b	phpgw version number to check $a against
-	 * @return	True if $a < $b
+	 * @return	boolean if $a < $b
 	 */
 	function amorethanb($a,$b,$DEBUG=False)
 	{
@@ -1101,7 +1101,7 @@ 
 	 *
 	 * Dont use it to set group-membership, use set_memberships instead!
 	 *
-	 * @param string|array $apps app-names
+	 * @param string $apps app-names
 	 * @param string $location eg. "run"
 	 * @param int|string $account accountid or account_lid
 	 * @param int $rights rights to set, default 1
@@ -1147,7 +1147,7 @@ 
 	/**
 	 * checks if one of the given tables exist, returns the first match
 	 *
-	 * @param array $tables array with possible table-names
+	 * @param string[] $tables array with possible table-names
 	 * @return string/boolean tablename or false
 	 */
 	function table_exist($tables,$force_refresh=False)

Indentation +27 added lines, -27 removed lines

@@ -151,10 +151,10 @@ 
 	}
 
 	/**
-	* Set the domain used for cookies
-	*
-	* @return string domain
-	*/
+	 * Set the domain used for cookies
+	 *
+	 * @return string domain
+	 */
 	static function cookiedomain()
 	{
 		// Use HTTP_X_FORWARDED_HOST if set, which is the case behind a none-transparent proxy
@@ -175,12 +175,12 @@ 
 	}
 
 	/**
-	* Set a cookie
-	*
-	* @param string $cookiename name of cookie to be set
-	* @param string $cookievalue value to be used, if unset cookie is cleared (optional)
-	* @param int $cookietime when cookie should expire, 0 for session only (optional)
-	*/
+	 * Set a cookie
+	 *
+	 * @param string $cookiename name of cookie to be set
+	 * @param string $cookievalue value to be used, if unset cookie is cleared (optional)
+	 * @param int $cookietime when cookie should expire, 0 for session only (optional)
+	 */
 	function set_cookie($cookiename,$cookievalue='',$cookietime=0)
 	{
 		if(!isset($this->cookie_domain))
@@ -344,19 +344,19 @@ 
 		return true;
 	}
 
-    /**
-    * check if username and password is valid
-    *
-    * this function compares the supplied and stored username and password
-	* as any of the passwords can be clear text or md5 we convert them to md5
-	* internal and compare always the md5 hashs
-    *
-    * @param string $user the user supplied username
-    * @param string $pw the user supplied password
-    * @param string $conf_user the configured username
-    * @param string $hash hash to check password agains (no {prefix} for plain and md5!)
-    * @returns bool true on success
-    */
+	/**
+	 * check if username and password is valid
+	 *
+	 * this function compares the supplied and stored username and password
+	 * as any of the passwords can be clear text or md5 we convert them to md5
+	 * internal and compare always the md5 hashs
+	 *
+	 * @param string $user the user supplied username
+	 * @param string $pw the user supplied password
+	 * @param string $conf_user the configured username
+	 * @param string $hash hash to check password agains (no {prefix} for plain and md5!)
+	 * @returns bool true on success
+	 */
 	static function check_auth($user, $pw, $conf_user, $hash)
 	{
 		if ($user !== $conf_user)
@@ -728,10 +728,10 @@ 
 	}
 
 	/**
-	  * call the hooks for a single application
-	  *
-	  * @param $location hook location - required
-	  * @param $appname application name - optional
+	 * call the hooks for a single application
+	 *
+	 * @param $location hook location - required
+	 * @param $appname application name - optional
 	 */
 	static function hook($location, $appname='')
 	{

Braces +128 added lines, -37 removed lines

@@ -85,11 +85,20 @@ 
 		$this->detection = new setup_detection();
 		$this->process   = new setup_process();
 
-		if (preg_match('/^[a-z0-9-]+$/i', $_REQUEST['system_charset'])) $this->system_charset = $_REQUEST['system_charset'];
+		if (preg_match('/^[a-z0-9-]+$/i', $_REQUEST['system_charset']))
+		{
+			$this->system_charset = $_REQUEST['system_charset'];
+		}
 
 		/* The setup application needs these */
-		if ($html) $this->html = new setup_html();
-		if ($translation) $this->translation = new setup_translation();
+		if ($html)
+		{
+			$this->html = new setup_html();
+		}
+		if ($translation)
+		{
+			$this->translation = new setup_translation();
+		}
 	}
 
 	/**
@@ -132,12 +141,15 @@ 
 					$this->db_charset_was = $this->db->Link_ID->GetCharSet();	// needed for the update
 
 					// we can NOT set the DB charset for mysql, if the api version < 1.0.1.019, as it would mess up the DB content!!!
-					if (substr($this->db->Type,0,5) == 'mysql')	// we need to check the api version
+					if (substr($this->db->Type,0,5) == 'mysql')
+					{
+						// we need to check the api version
 					{
 						$api_version = $this->db->select($this->applications_table,'app_version',array(
 								'app_name'  => 'phpgwapi',
 							),__LINE__,__FILE__)->fetchColumn();
 					}
+					}
 					if (!$api_version || !$this->alessthanb($api_version,'1.0.1.019'))
 					{
 						$this->db->Link_ID->SetCharSet($this->system_charset);
@@ -205,7 +217,10 @@ 
 		}
 		else
 		{
-			if (!isset($_SESSION)) self::session_start();
+			if (!isset($_SESSION))
+			{
+				self::session_start();
+			}
 			$ConfigLang = $_SESSION['ConfigLang'];
 		}
 		if (!preg_match('/^[a-z]{2}(-[a-z]{2})?$/',$ConfigLang))
@@ -240,7 +255,10 @@ 
 			// if called via HTTPS, only send cookie for https and only allow cookie access via HTTP (true)
 			!empty($_SERVER['HTTPS']) && $_SERVER['HTTPS'] !== 'off', true);
 
-		if (isset($_COOKIE[self::SESSIONID])) session_id($_COOKIE[self::SESSIONID]);
+		if (isset($_COOKIE[self::SESSIONID]))
+		{
+			session_id($_COOKIE[self::SESSIONID]);
+		}
 
 		$ok = @session_start();	// suppress notice if session already started or warning in CLI
 		// need to decrypt session, in case session encryption is switched on in header.inc.php
@@ -453,7 +471,10 @@ 
 	 */
 	function register_app($appname, $_enable=99, array $setup_info=null)
 	{
-		if (!isset($setup_info)) $setup_info = $GLOBALS['setup_info'];
+		if (!isset($setup_info))
+		{
+			$setup_info = $GLOBALS['setup_info'];
+		}
 
 		if(!$appname)
 		{
@@ -547,7 +568,10 @@ 
 	 */
 	function update_app($appname, array $setup_info=null)
 	{
-		if (!isset($setup_info)) $setup_info = $GLOBALS['setup_info'];
+		if (!isset($setup_info))
+		{
+			$setup_info = $GLOBALS['setup_info'];
+		}
 
 		if(!$appname)
 		{
@@ -692,7 +716,10 @@ 
 		{
 			$GLOBALS['settings'] = array();
 			include_once($file);
-			if ($GLOBALS['settings']) $settings = array_merge($settings,$GLOBALS['settings']);
+			if ($GLOBALS['settings'])
+			{
+				$settings = array_merge($settings,$GLOBALS['settings']);
+			}
 		}
 		$default = $forced = array();
 		foreach($settings as $name => $setting)
@@ -774,52 +801,72 @@ 
 
 		for($i=0;$i<count($testa);$i++)
 		{
-			if($DEBUG) { echo'<br>Checking if '. (int)$testa[$i] . ' is less than ' . (int)$testb[$i] . ' ...'; }
+			if($DEBUG)
+			{
+echo'<br>Checking if '. (int)$testa[$i] . ' is less than ' . (int)$testb[$i] . ' ...'; }
 			if((int)$testa[$i] < (int)$testb[$i])
 			{
-				if ($DEBUG) { echo ' yes.'; }
+				if ($DEBUG)
+				{
+echo ' yes.'; }
 				$less++;
 				if($i<3)
 				{
 					/* Ensure that this is definitely smaller */
-					if($DEBUG) { echo"  This is the $num[$i] octet, so A is definitely less than B."; }
+					if($DEBUG)
+					{
+echo"  This is the $num[$i] octet, so A is definitely less than B."; }
 					$less = 5;
 					break;
 				}
 			}
 			elseif((int)$testa[$i] > (int)$testb[$i])
 			{
-				if($DEBUG) { echo ' no.'; }
+				if($DEBUG)
+				{
+echo ' no.'; }
 				$less--;
 				if($i<2)
 				{
 					/* Ensure that this is definitely greater */
-					if($DEBUG) { echo"  This is the $num[$i] octet, so A is definitely greater than B."; }
+					if($DEBUG)
+					{
+echo"  This is the $num[$i] octet, so A is definitely greater than B."; }
 					$less = -5;
 					break;
 				}
 			}
 			else
 			{
-				if($DEBUG) { echo ' no, they are equal or of different length.'; }
+				if($DEBUG)
+				{
+echo ' no, they are equal or of different length.'; }
 				// makes sure eg. '1.0.0' is counted less the '1.0.0.xxx' !
 				$less = count($testa) < count($testb) ? 1 : 0;
 			}
 		}
-		if($DEBUG) { echo '<br>Check value is: "'.$less.'"'; }
+		if($DEBUG)
+		{
+echo '<br>Check value is: "'.$less.'"'; }
 		if($less>0)
 		{
-			if($DEBUG) { echo '<br>A is less than B'; }
+			if($DEBUG)
+			{
+echo '<br>A is less than B'; }
 			return True;
 		}
 		elseif($less<0)
 		{
-			if($DEBUG) { echo '<br>A is greater than B'; }
+			if($DEBUG)
+			{
+echo '<br>A is greater than B'; }
 			return False;
 		}
 		else
 		{
-			if($DEBUG) { echo '<br>A is equal to B'; }
+			if($DEBUG)
+			{
+echo '<br>A is equal to B'; }
 			return False;
 		}
 	}
@@ -856,51 +903,71 @@ 
 
 		for($i=0;$i<count($testa);$i++)
 		{
-			if($DEBUG) { echo'<br>Checking if '. (int)$testa[$i] . ' is more than ' . (int)$testb[$i] . ' ...'; }
+			if($DEBUG)
+			{
+echo'<br>Checking if '. (int)$testa[$i] . ' is more than ' . (int)$testb[$i] . ' ...'; }
 			if((int)$testa[$i] > (int)$testb[$i])
 			{
-				if($DEBUG) { echo ' yes.'; }
+				if($DEBUG)
+				{
+echo ' yes.'; }
 				$less++;
 				if($i<3)
 				{
 					/* Ensure that this is definitely greater */
-					if($DEBUG) { echo"  This is the $num[$i] octet, so A is definitely greater than B."; }
+					if($DEBUG)
+					{
+echo"  This is the $num[$i] octet, so A is definitely greater than B."; }
 					$less = 5;
 					break;
 				}
 			}
 			elseif((int)$testa[$i] < (int)$testb[$i])
 			{
-				if($DEBUG) { echo ' no.'; }
+				if($DEBUG)
+				{
+echo ' no.'; }
 				$less--;
 				if($i<2)
 				{
 					/* Ensure that this is definitely smaller */
-					if($DEBUG) { echo"  This is the $num[$i] octet, so A is definitely less than B."; }
+					if($DEBUG)
+					{
+echo"  This is the $num[$i] octet, so A is definitely less than B."; }
 					$less = -5;
 					break;
 				}
 			}
 			else
 			{
-				if($DEBUG) { echo ' no, they are equal.'; }
+				if($DEBUG)
+				{
+echo ' no, they are equal.'; }
 				$less = 0;
 			}
 		}
-		if($DEBUG) { echo '<br>Check value is: "'.$less.'"'; }
+		if($DEBUG)
+		{
+echo '<br>Check value is: "'.$less.'"'; }
 		if($less>0)
 		{
-			if($DEBUG) { echo '<br>A is greater than B'; }
+			if($DEBUG)
+			{
+echo '<br>A is greater than B'; }
 			return True;
 		}
 		elseif($less<0)
 		{
-			if($DEBUG) { echo '<br>A is less than B'; }
+			if($DEBUG)
+			{
+echo '<br>A is less than B'; }
 			return False;
 		}
 		else
 		{
-			if($DEBUG) { echo '<br>A is equal to B'; }
+			if($DEBUG)
+			{
+echo '<br>A is equal to B'; }
 			return False;
 		}
 	}
@@ -937,7 +1004,10 @@ 
 				echo "<p><b>".$e->getMessage()."</b></p>\n";
 				return false;
 			}
-			if (!isset($GLOBALS['egw']->accounts)) $GLOBALS['egw']->accounts = $this->accounts;
+			if (!isset($GLOBALS['egw']->accounts))
+			{
+				$GLOBALS['egw']->accounts = $this->accounts;
+			}
 			Api\Accounts::cache_invalidate();	// the cache is shared for all instances of the class
 		}
 		return true;
@@ -972,7 +1042,10 @@ 
 
 		if ($username == 'anonymous')
 		{
-			if (!isset($this->anonpw)) $this->anonpw = Api\Auth::randomstring(16);
+			if (!isset($this->anonpw))
+			{
+				$this->anonpw = Api\Auth::randomstring(16);
+			}
 			$passwd = $anonpw = $this->anonpw;
 		}
 
@@ -1031,11 +1104,17 @@ 
 				'account_lid' => $username,
 			));
 		}
-		if ($primary_group)	// only for users, NOT groups
+		if ($primary_group)
+		{
+			// only for users, NOT groups
 		{
 			$this->set_memberships(array($primary_group_id), $accountid);
+		}
 
-			if (!$changepw) $this->add_acl('preferences','nopasswordchange',$accountid);
+			if (!$changepw)
+			{
+				$this->add_acl('preferences','nopasswordchange',$accountid);
+			}
 		}
 		//error_log("setup::add_account('$username','$first','$last',\$passwd,'$primary_group',$changepw,'$email') successfull created accountid=$accountid");
 		return $accountid;
@@ -1071,7 +1150,10 @@ 
 	 */
 	function accounts_exist()
 	{
-		if (!$this->setup_account_object()) return false;
+		if (!$this->setup_account_object())
+		{
+			return false;
+		}
 
 		$this->accounts->search(array(
 			'type'   => 'accounts',
@@ -1154,9 +1236,15 @@ 
 	{
 		static $table_names = False;
 
-		if (!$table_names || $force_refresh) $table_names = $this->db->table_names();
+		if (!$table_names || $force_refresh)
+		{
+			$table_names = $this->db->table_names();
+		}
 
-		if (!$table_names) return false;
+		if (!$table_names)
+		{
+			return false;
+		}
 
 		foreach($table_names as $data)
 		{
@@ -1186,10 +1274,13 @@ 
 		{
 			$table = $this->table_exist($tables,$force_refresh);
 
-			if ($table && $table != $this->$name)	// only overwrite the default name, if we realy got one (important for new installs)
+			if ($table && $table != $this->$name)
+			{
+				// only overwrite the default name, if we realy got one (important for new installs)
 			{
 				$this->$name = $table;
 			}
+			}
 			//echo "<p>setup::set_table_names: $name = '{$this->$name}'</p>\n";
 		}
 	}

Spacing +177 added lines, -177 removed lines

@@ -73,10 +73,10 @@ 
 	 */
 	var $recommended_php_version = '5.6';
 
-	function __construct($html=False, $translation=False)
+	function __construct($html = False, $translation = False)
 	{
 		// setup us as $GLOBALS['egw_setup'], as this gets used in our sub-objects
-		$GLOBALS['egw_setup'] =& $this;
+		$GLOBALS['egw_setup'] = & $this;
 
 		if (!is_object($GLOBALS['egw']))
 		{
@@ -95,9 +95,9 @@ 
 	/**
 	 * include api db class for the ConfigDomain and connect to the db
 	 */
-	function loaddb($connect_and_setcharset=true)
+	function loaddb($connect_and_setcharset = true)
 	{
-		if(!isset($this->ConfigDomain) || empty($this->ConfigDomain))
+		if (!isset($this->ConfigDomain) || empty($this->ConfigDomain))
 		{
 			$this->ConfigDomain = isset($_REQUEST['ConfigDomain']) ? $_REQUEST['ConfigDomain'] : $_POST['FormDomain'];
 		}
@@ -121,24 +121,24 @@ 
 		if ($connect_and_setcharset)
 		{
 			try {
-				$this->set_table_names();		// sets/checks config- and applications-table-name
+				$this->set_table_names(); // sets/checks config- and applications-table-name
 
 				// Set the DB's client charset if a system-charset is set
-				if (($this->system_charset = $this->db->select($this->config_table,'config_value',array(
+				if (($this->system_charset = $this->db->select($this->config_table, 'config_value', array(
 						'config_app'  => 'phpgwapi',
 						'config_name' => 'system_charset',
-					),__LINE__,__FILE__)->fetchColumn()))
+					), __LINE__, __FILE__)->fetchColumn()))
 				{
-					$this->db_charset_was = $this->db->Link_ID->GetCharSet();	// needed for the update
+					$this->db_charset_was = $this->db->Link_ID->GetCharSet(); // needed for the update
 
 					// we can NOT set the DB charset for mysql, if the api version < 1.0.1.019, as it would mess up the DB content!!!
-					if (substr($this->db->Type,0,5) == 'mysql')	// we need to check the api version
+					if (substr($this->db->Type, 0, 5) == 'mysql')	// we need to check the api version
 					{
-						$api_version = $this->db->select($this->applications_table,'app_version',array(
+						$api_version = $this->db->select($this->applications_table, 'app_version', array(
 								'app_name'  => 'phpgwapi',
-							),__LINE__,__FILE__)->fetchColumn();
+							), __LINE__, __FILE__)->fetchColumn();
 					}
-					if (!$api_version || !$this->alessthanb($api_version,'1.0.1.019'))
+					if (!$api_version || !$this->alessthanb($api_version, '1.0.1.019'))
 					{
 						$this->db->Link_ID->SetCharSet($this->system_charset);
 					}
@@ -158,15 +158,15 @@ 
 	static function cookiedomain()
 	{
 		// Use HTTP_X_FORWARDED_HOST if set, which is the case behind a none-transparent proxy
-		$cookie_domain = isset($_SERVER['HTTP_X_FORWARDED_HOST']) ?  $_SERVER['HTTP_X_FORWARDED_HOST'] : $_SERVER['HTTP_HOST'];
+		$cookie_domain = isset($_SERVER['HTTP_X_FORWARDED_HOST']) ? $_SERVER['HTTP_X_FORWARDED_HOST'] : $_SERVER['HTTP_HOST'];
 
 		// remove port from HTTP_HOST
 		$arr = null;
-		if (preg_match("/^(.*):(.*)$/",$cookie_domain,$arr))
+		if (preg_match("/^(.*):(.*)$/", $cookie_domain, $arr))
 		{
 			$cookie_domain = $arr[1];
 		}
-		if (count(explode('.',$cookie_domain)) <= 1)
+		if (count(explode('.', $cookie_domain)) <= 1)
 		{
 			// setcookie dont likes domains without dots, leaving it empty, gets setcookie to fill the domain in
 			$cookie_domain = '';
@@ -181,9 +181,9 @@ 
 	* @param string $cookievalue value to be used, if unset cookie is cleared (optional)
 	* @param int $cookietime when cookie should expire, 0 for session only (optional)
 	*/
-	function set_cookie($cookiename,$cookievalue='',$cookietime=0)
+	function set_cookie($cookiename, $cookievalue = '', $cookietime = 0)
 	{
-		if(!isset($this->cookie_domain))
+		if (!isset($this->cookie_domain))
 		{
 			$this->cookie_domain = self::cookiedomain();
 		}
@@ -208,9 +208,9 @@ 
 			if (!isset($_SESSION)) self::session_start();
 			$ConfigLang = $_SESSION['ConfigLang'];
 		}
-		if (!preg_match('/^[a-z]{2}(-[a-z]{2})?$/',$ConfigLang))
+		if (!preg_match('/^[a-z]{2}(-[a-z]{2})?$/', $ConfigLang))
 		{
-			$ConfigLang = null;	// not returning 'en', as it suppresses the language selection in check_install and manageheader
+			$ConfigLang = null; // not returning 'en', as it suppresses the language selection in check_install and manageheader
 		}
 		//error_log(__METHOD__."() \$_POST['ConfigLang']=".array2string($_POST['ConfigLang']).", \$_SESSION['ConfigLang']=".array2string($_SESSION['ConfigLang'])." returning ".array2string($ConfigLang));
 		return $ConfigLang;
@@ -242,7 +242,7 @@ 
 
 		if (isset($_COOKIE[self::SESSIONID])) session_id($_COOKIE[self::SESSIONID]);
 
-		$ok = @session_start();	// suppress notice if session already started or warning in CLI
+		$ok = @session_start(); // suppress notice if session already started or warning in CLI
 		// need to decrypt session, in case session encryption is switched on in header.inc.php
 		Api\Session::decrypt();
 		//error_log(__METHOD__."() returning ".array2string($ok).' _SESSION='.array2string($_SESSION));
@@ -254,12 +254,12 @@ 
 	 *
 	 * @param string $_auth_type ='config' 'config' or 'header' (caseinsensitiv)
 	 */
-	function auth($_auth_type='config')
+	function auth($_auth_type = 'config')
 	{
 		$auth_type = strtolower($_auth_type);
 		$GLOBALS['egw_info']['setup']['HeaderLoginMSG'] = $GLOBALS['egw_info']['setup']['ConfigLoginMSG'] = '';
 
-		if(!$this->checkip($_SERVER['REMOTE_ADDR']))
+		if (!$this->checkip($_SERVER['REMOTE_ADDR']))
 		{
 			//error_log(__METHOD__."('$auth_type') invalid IP");
 			return false;
@@ -271,7 +271,7 @@ 
 			//error_log(__METHOD__."('$auth_type') \$_COOKIE['".self::SESSIONID."'] = ".array2string($_COOKIE[self::SESSIONID]).", \$_SESSION=".array2string($_SESSION).", \$_POST=".array2string($_POST));
 			if (isset($_REQUEST['FormLogout']))
 			{
-				$this->set_cookie(self::SESSIONID, '', time()-86400);
+				$this->set_cookie(self::SESSIONID, '', time() - 86400);
 				session_destroy();
 				if ($_REQUEST['FormLogout'] == 'config')
 				{
@@ -287,7 +287,7 @@ 
 			{
 				return false;
 			}
-			switch($auth_type)
+			switch ($auth_type)
 			{
 				case 'config':
 					if (!isset($GLOBALS['egw_domain'][$_POST['FormDomain']]) ||
@@ -327,7 +327,7 @@ 
 		//error_log(__METHOD__."('$auth_type') \$_COOKIE['".self::SESSIONID."'] = ".array2string($_COOKIE[self::SESSIONID]).", \$_SESSION=".array2string($_SESSION));
 		if ($_SESSION['egw_last_action_time'] < time() - self::TIMEOUT)
 		{
-			$this->set_cookie(self::SESSIONID, '', time()-86400);
+			$this->set_cookie(self::SESSIONID, '', time() - 86400);
 			session_destroy();
 			$GLOBALS['egw_info']['setup'][$_SESSION['egw_setup_auth_type'] == 'config' ? 'ConfigLoginMSG' : 'HeaderLoginMSG'] =
 				lang('Session expired');
@@ -382,37 +382,37 @@ 
 	 * @param string $remoteip
 	 * @return boolean
 	 */
-	function checkip($remoteip='')
+	function checkip($remoteip = '')
 	{
 		//echo "<p>setup::checkip($remoteip) against setup_acl='".$GLOBALS['egw_info']['server']['setup_acl']."'</p>\n";
-		$allowed_ips = explode(',',@$GLOBALS['egw_info']['server']['setup_acl']);
-		if(empty($GLOBALS['egw_info']['server']['setup_acl']) || !is_array($allowed_ips))
+		$allowed_ips = explode(',', @$GLOBALS['egw_info']['server']['setup_acl']);
+		if (empty($GLOBALS['egw_info']['server']['setup_acl']) || !is_array($allowed_ips))
 		{
-			return True;	// no test
+			return True; // no test
 		}
-		$remotes = explode('.',$remoteip);
-		foreach($allowed_ips as $value)
+		$remotes = explode('.', $remoteip);
+		foreach ($allowed_ips as $value)
 		{
-			if (!preg_match('/^[0-9.]+$/',$value))
+			if (!preg_match('/^[0-9.]+$/', $value))
 			{
-				$value = gethostbyname($was=$value);		// resolve domain-name, eg. a dyndns account
+				$value = gethostbyname($was = $value); // resolve domain-name, eg. a dyndns account
 				//echo "resolving '$was' to '$value'<br>\n";
 			}
-			$values = explode('.',$value);
-			for($i = 0; $i < count($values); ++$i)
+			$values = explode('.', $value);
+			for ($i = 0; $i < count($values); ++$i)
 			{
-				if ((int) $values[$i] != (int) $remotes[$i])
+				if ((int)$values[$i] != (int)$remotes[$i])
 				{
 					break;
 				}
 			}
 			if ($i == count($values))
 			{
-				return True;	// match
+				return True; // match
 			}
 		}
 		$GLOBALS['egw_info']['setup']['ConfigLoginMSG'] = lang('Invalid IP address').' '.$remoteip;
-		error_log(__METHOD__.'-> checking IP failed:'.print_r($remoteip,true));
+		error_log(__METHOD__.'-> checking IP failed:'.print_r($remoteip, true));
 		return False;
 	}
 
@@ -423,14 +423,14 @@ 
 	 */
 	function get_major($versionstring)
 	{
-		if(!$versionstring)
+		if (!$versionstring)
 		{
 			return False;
 		}
 
-		$version = str_replace('pre','.',$versionstring);
-		$varray  = explode('.',$version);
-		$major   = implode('.',array($varray[0],$varray[1],$varray[2]));
+		$version = str_replace('pre', '.', $versionstring);
+		$varray  = explode('.', $version);
+		$major   = implode('.', array($varray[0], $varray[1], $varray[2]));
 
 		return $major;
 	}
@@ -451,55 +451,55 @@ 
 	 * @param $_enable =99 set to True/False to override setup.inc.php setting
 	 * @param array $setup_info =null default use $GLOBALS['setup_info']
 	 */
-	function register_app($appname, $_enable=99, array $setup_info=null)
+	function register_app($appname, $_enable = 99, array $setup_info = null)
 	{
 		if (!isset($setup_info)) $setup_info = $GLOBALS['setup_info'];
 
-		if(!$appname)
+		if (!$appname)
 		{
 			return False;
 		}
 
-		if($_enable == 99)
+		if ($_enable == 99)
 		{
 			$_enable = $setup_info[$appname]['enable'];
 		}
 		$enable = (int)$_enable;
 
-		if($GLOBALS['DEBUG'])
+		if ($GLOBALS['DEBUG'])
 		{
-			echo '<br>register_app(): ' . $appname . ', version: ' . $setup_info[$appname]['version'] . ', tables: ' . implode(', ',$setup_info[$appname]['tables']) . '<br>';
+			echo '<br>register_app(): '.$appname.', version: '.$setup_info[$appname]['version'].', tables: '.implode(', ', $setup_info[$appname]['tables']).'<br>';
 			// _debug_array($setup_info[$appname]);
 		}
 
-		if($setup_info[$appname]['version'])
+		if ($setup_info[$appname]['version'])
 		{
-			if($setup_info[$appname]['tables'])
+			if ($setup_info[$appname]['tables'])
 			{
-				$tables = implode(',',$setup_info[$appname]['tables']);
+				$tables = implode(',', $setup_info[$appname]['tables']);
 			}
 			if ($setup_info[$appname]['tables_use_prefix'] == True)
 			{
-				if($GLOBALS['DEBUG'])
+				if ($GLOBALS['DEBUG'])
 				{
-					echo "<br>$appname uses tables_use_prefix, storing ". $setup_info[$appname]['tables_prefix']." as prefix for tables\n";
+					echo "<br>$appname uses tables_use_prefix, storing ".$setup_info[$appname]['tables_prefix']." as prefix for tables\n";
 				}
-				$this->db->insert($this->config_table,array(
+				$this->db->insert($this->config_table, array(
 						'config_app'	=> $appname,
 						'config_name'	=> $appname.'_tables_prefix',
 						'config_value'	=> $setup_info[$appname]['tables_prefix'],
-					),False,__LINE__,__FILE__);
+					), False, __LINE__, __FILE__);
 			}
-			$this->db->insert($this->applications_table,array(
+			$this->db->insert($this->applications_table, array(
 					'app_name'		=> $appname,
 					'app_enabled'	=> $enable,
 					'app_order'		=> $setup_info[$appname]['app_order'],
-					'app_tables'	=> (string)$tables,	// app_tables is NOT NULL
+					'app_tables'	=> (string)$tables, // app_tables is NOT NULL
 					'app_version'	=> $setup_info[$appname]['version'],
 					'app_index'     => $setup_info[$appname]['index'],
 					'app_icon'      => $setup_info[$appname]['icon'],
 					'app_icon_app'  => $setup_info[$appname]['icon_app'],
-				),False,__LINE__,__FILE__);
+				), False, __LINE__, __FILE__);
 
 			$this->clear_session_cache();
 		}
@@ -513,26 +513,26 @@ 
 	 */
 	function app_registered($appname)
 	{
-		if(!$appname)
+		if (!$appname)
 		{
 			return False;
 		}
 
-		if(@$GLOBALS['DEBUG'])
+		if (@$GLOBALS['DEBUG'])
 		{
-			echo '<br>app_registered(): checking ' . $appname . ', table: ' . $this->applications_table;
+			echo '<br>app_registered(): checking '.$appname.', table: '.$this->applications_table;
 			// _debug_array($setup_info[$appname]);
 		}
 
-		if ($this->db->select($this->applications_table,'COUNT(*)',array('app_name' => $appname),__LINE__,__FILE__)->fetchColumn())
+		if ($this->db->select($this->applications_table, 'COUNT(*)', array('app_name' => $appname), __LINE__, __FILE__)->fetchColumn())
 		{
-			if(@$GLOBALS['DEBUG'])
+			if (@$GLOBALS['DEBUG'])
 			{
 				echo '... app previously registered.';
 			}
 			return True;
 		}
-		if(@$GLOBALS['DEBUG'])
+		if (@$GLOBALS['DEBUG'])
 		{
 			echo '... app not registered';
 		}
@@ -545,42 +545,42 @@ 
 	 * @param string $appname	Application 'name' with a matching $setup_info[$appname] array slice
 	 * @param array $setup_info =null default use $GLOBALS['setup_info']
 	 */
-	function update_app($appname, array $setup_info=null)
+	function update_app($appname, array $setup_info = null)
 	{
 		if (!isset($setup_info)) $setup_info = $GLOBALS['setup_info'];
 
-		if(!$appname)
+		if (!$appname)
 		{
 			return False;
 		}
 
-		if($GLOBALS['DEBUG'])
+		if ($GLOBALS['DEBUG'])
 		{
-			echo '<br>update_app(): ' . $appname . ', version: ' . $setup_info[$appname]['currentver'] . ', table: ' . $this->applications_table . '<br>';
+			echo '<br>update_app(): '.$appname.', version: '.$setup_info[$appname]['currentver'].', table: '.$this->applications_table.'<br>';
 			// _debug_array($setup_info[$appname]);
 		}
 
-		if(!$this->app_registered($appname))
+		if (!$this->app_registered($appname))
 		{
 			return False;
 		}
 
-		if($setup_info[$appname]['version'])
+		if ($setup_info[$appname]['version'])
 		{
 			//echo '<br>' . $setup_info[$appname]['version'];
-			if($setup_info[$appname]['tables'])
+			if ($setup_info[$appname]['tables'])
 			{
-				$tables = implode(',',$setup_info[$appname]['tables']);
+				$tables = implode(',', $setup_info[$appname]['tables']);
 			}
-			$this->db->update($this->applications_table,array(
+			$this->db->update($this->applications_table, array(
 					'app_enabled'	=> $setup_info[$appname]['enable'],
 					'app_order'		=> $setup_info[$appname]['app_order'],
-					'app_tables'	=> (string)$tables,	// app_tables is NOT NULL
+					'app_tables'	=> (string)$tables, // app_tables is NOT NULL
 					'app_version'	=> $setup_info[$appname]['version'],
 					'app_index'     => $setup_info[$appname]['index'],
 					'app_icon'      => $setup_info[$appname]['icon'],
 					'app_icon_app'  => $setup_info[$appname]['icon_app'],
-				),array('app_name'=>$appname),__LINE__,__FILE__);
+				), array('app_name'=>$appname), __LINE__, __FILE__);
 		}
 	}
 
@@ -593,20 +593,20 @@ 
 	 */
 	function update_app_version($setup_info, $appname, $tableschanged = True)
 	{
-		if(!$appname)
+		if (!$appname)
 		{
 			return False;
 		}
 
-		if($tableschanged == True)
+		if ($tableschanged == True)
 		{
 			$GLOBALS['egw_info']['setup']['tableschanged'] = True;
 		}
-		if($setup_info[$appname]['currentver'])
+		if ($setup_info[$appname]['currentver'])
 		{
-			$this->db->update($this->applications_table,array(
+			$this->db->update($this->applications_table, array(
 					'app_version'	=> $setup_info[$appname]['currentver'],
-				),array('app_name'=>$appname),__LINE__,__FILE__);
+				), array('app_name'=>$appname), __LINE__, __FILE__);
 		}
 		return $setup_info;
 	}
@@ -618,22 +618,22 @@ 
 	 */
 	function deregister_app($appname)
 	{
-		if(!$appname)
+		if (!$appname)
 		{
 			return False;
 		}
 
 		// Remove categories
-		$this->db->delete(Api\Categories::TABLE, array('cat_appname'=>$appname),__LINE__,__FILE__);
+		$this->db->delete(Api\Categories::TABLE, array('cat_appname'=>$appname), __LINE__, __FILE__);
 		Api\Categories::invalidate_cache($appname);
 
 		// Remove config, if we are not deinstalling old phpgwapi (as that's global api config!)
 		if ($appname != 'phpgwapi')
 		{
-			$this->db->delete(Api\Config::TABLE, array('config_app'=>$appname),__LINE__,__FILE__);
+			$this->db->delete(Api\Config::TABLE, array('config_app'=>$appname), __LINE__, __FILE__);
 		}
 		//echo 'DELETING application: ' . $appname;
-		$this->db->delete($this->applications_table,array('app_name'=>$appname),__LINE__,__FILE__);
+		$this->db->delete($this->applications_table, array('app_name'=>$appname), __LINE__, __FILE__);
 
 		// Remove links to the app
 		Link::unlink(0, $appname);
@@ -648,7 +648,7 @@ 
 	 */
 	function register_hooks($appname)
 	{
-		if(!$appname)
+		if (!$appname)
 		{
 			return False;
 		}
@@ -668,21 +668,21 @@ 
 
 		if (!isset($setup_info) || !isset($setup_info['hooks']))
 		{
-			return false;	// app not found or no hook
+			return false; // app not found or no hook
 		}
 		$GLOBALS['settings'] = array();
-		$hook_data = array('location' => 'settings','setup' => true);
+		$hook_data = array('location' => 'settings', 'setup' => true);
 		if (isset($setup_info['hooks']['settings']))
 		{
-			$settings = ExecMethod($setup_info['hooks']['settings'],$hook_data);
+			$settings = ExecMethod($setup_info['hooks']['settings'], $hook_data);
 		}
-		elseif(in_array('settings',$setup_info['hooks']) && file_exists($file = EGW_INCLUDE_ROOT.'/'.$appname.'/inc/hook_settings.inc.php'))
+		elseif (in_array('settings', $setup_info['hooks']) && file_exists($file = EGW_INCLUDE_ROOT.'/'.$appname.'/inc/hook_settings.inc.php'))
 		{
 			include_once($file);
 		}
 		if (!isset($settings) || !is_array($settings))
 		{
-			$settings = $GLOBALS['settings'];	// old file hook or not updated new hook
+			$settings = $GLOBALS['settings']; // old file hook or not updated new hook
 		}
 		if (!is_array($settings) || !count($settings))
 		{
@@ -695,10 +695,10 @@ 
 		{
 			$GLOBALS['settings'] = array();
 			include_once($file);
-			if ($GLOBALS['settings']) $settings = array_merge($settings,$GLOBALS['settings']);
+			if ($GLOBALS['settings']) $settings = array_merge($settings, $GLOBALS['settings']);
 		}
 		$default = $forced = array();
-		foreach($settings as $name => $setting)
+		foreach ($settings as $name => $setting)
 		{
 			if (isset($setting['default']))
 			{
@@ -712,14 +712,14 @@ 
 		// store default/forced preferences, if any found
 		$preferences = new Api\Preferences();
 		$preferences->read_repository(false);
-		foreach(array(
+		foreach (array(
 			'default' => $default,
 			'forced'  => $forced,
 		) as $type => $prefs)
 		{
 			if ($prefs)
 			{
-				foreach($prefs as $name => $value)
+				foreach ($prefs as $name => $value)
 				{
 					$preferences->add($appname == 'preferences' ? 'common' : $appname, $name, $value, $type);
 				}
@@ -736,9 +736,9 @@ 
 	  * @param $location hook location - required
 	  * @param $appname application name - optional
 	 */
-	static function hook($location, $appname='')
+	static function hook($location, $appname = '')
 	{
-		return Api\Hooks::single($location,$appname,True,True);
+		return Api\Hooks::single($location, $appname, True, True);
 	}
 
 	/**
@@ -747,82 +747,82 @@ 
 	 * @param	$b	phpgw version number to check $a against
 	 * @return	True if $a < $b
 	 */
-	function alessthanb($a,$b,$DEBUG=False)
+	function alessthanb($a, $b, $DEBUG = False)
 	{
-		$num = array('1st','2nd','3rd','4th');
+		$num = array('1st', '2nd', '3rd', '4th');
 
-		if($DEBUG)
+		if ($DEBUG)
 		{
 			echo'<br>Input values: '
 				. 'A="'.$a.'", B="'.$b.'"';
 		}
-		$newa = str_replace('pre','.',$a);
-		$newb = str_replace('pre','.',$b);
-		$testa = explode('.',$newa);
-		if(@$testa[1] == '')
+		$newa = str_replace('pre', '.', $a);
+		$newb = str_replace('pre', '.', $b);
+		$testa = explode('.', $newa);
+		if (@$testa[1] == '')
 		{
 			$testa[1] = 0;
 		}
 
-		$testb = explode('.',$newb);
-		if(@$testb[1] == '')
+		$testb = explode('.', $newb);
+		if (@$testb[1] == '')
 		{
 			$testb[1] = 0;
 		}
-		if(@$testb[3] == '')
+		if (@$testb[3] == '')
 		{
 			$testb[3] = 0;
 		}
 		$less = 0;
 
-		for($i=0;$i<count($testa);$i++)
+		for ($i = 0; $i < count($testa); $i++)
 		{
-			if($DEBUG) { echo'<br>Checking if '. (int)$testa[$i] . ' is less than ' . (int)$testb[$i] . ' ...'; }
-			if((int)$testa[$i] < (int)$testb[$i])
+			if ($DEBUG) { echo'<br>Checking if '.(int)$testa[$i].' is less than '.(int)$testb[$i].' ...'; }
+			if ((int)$testa[$i] < (int)$testb[$i])
 			{
 				if ($DEBUG) { echo ' yes.'; }
 				$less++;
-				if($i<3)
+				if ($i < 3)
 				{
 					/* Ensure that this is definitely smaller */
-					if($DEBUG) { echo"  This is the $num[$i] octet, so A is definitely less than B."; }
+					if ($DEBUG) { echo"  This is the $num[$i] octet, so A is definitely less than B."; }
 					$less = 5;
 					break;
 				}
 			}
-			elseif((int)$testa[$i] > (int)$testb[$i])
+			elseif ((int)$testa[$i] > (int)$testb[$i])
 			{
-				if($DEBUG) { echo ' no.'; }
+				if ($DEBUG) { echo ' no.'; }
 				$less--;
-				if($i<2)
+				if ($i < 2)
 				{
 					/* Ensure that this is definitely greater */
-					if($DEBUG) { echo"  This is the $num[$i] octet, so A is definitely greater than B."; }
+					if ($DEBUG) { echo"  This is the $num[$i] octet, so A is definitely greater than B."; }
 					$less = -5;
 					break;
 				}
 			}
 			else
 			{
-				if($DEBUG) { echo ' no, they are equal or of different length.'; }
+				if ($DEBUG) { echo ' no, they are equal or of different length.'; }
 				// makes sure eg. '1.0.0' is counted less the '1.0.0.xxx' !
 				$less = count($testa) < count($testb) ? 1 : 0;
 			}
 		}
-		if($DEBUG) { echo '<br>Check value is: "'.$less.'"'; }
-		if($less>0)
+		if ($DEBUG) { echo '<br>Check value is: "'.$less.'"'; }
+		if ($less > 0)
 		{
-			if($DEBUG) { echo '<br>A is less than B'; }
+			if ($DEBUG) { echo '<br>A is less than B'; }
 			return True;
 		}
-		elseif($less<0)
+		elseif ($less < 0)
 		{
-			if($DEBUG) { echo '<br>A is greater than B'; }
+			if ($DEBUG) { echo '<br>A is greater than B'; }
 			return False;
 		}
 		else
 		{
-			if($DEBUG) { echo '<br>A is equal to B'; }
+			if ($DEBUG) { echo '<br>A is equal to B'; }
 			return False;
 		}
 	}
@@ -834,76 +834,76 @@ 
 	 * @param	$b	phpgw version number to check $a against
 	 * @return	True if $a < $b
 	 */
-	function amorethanb($a,$b,$DEBUG=False)
+	function amorethanb($a, $b, $DEBUG = False)
 	{
-		$num = array('1st','2nd','3rd','4th');
+		$num = array('1st', '2nd', '3rd', '4th');
 
-		if($DEBUG)
+		if ($DEBUG)
 		{
 			echo'<br>Input values: '
 				. 'A="'.$a.'", B="'.$b.'"';
 		}
-		$newa = str_replace('pre','.',$a);
-		$newb = str_replace('pre','.',$b);
-		$testa = explode('.',$newa);
-		if($testa[3] == '')
+		$newa = str_replace('pre', '.', $a);
+		$newb = str_replace('pre', '.', $b);
+		$testa = explode('.', $newa);
+		if ($testa[3] == '')
 		{
 			$testa[3] = 0;
 		}
-		$testb = explode('.',$newb);
-		if($testb[3] == '')
+		$testb = explode('.', $newb);
+		if ($testb[3] == '')
 		{
 			$testb[3] = 0;
 		}
 		$less = 0;
 
-		for($i=0;$i<count($testa);$i++)
+		for ($i = 0; $i < count($testa); $i++)
 		{
-			if($DEBUG) { echo'<br>Checking if '. (int)$testa[$i] . ' is more than ' . (int)$testb[$i] . ' ...'; }
-			if((int)$testa[$i] > (int)$testb[$i])
+			if ($DEBUG) { echo'<br>Checking if '.(int)$testa[$i].' is more than '.(int)$testb[$i].' ...'; }
+			if ((int)$testa[$i] > (int)$testb[$i])
 			{
-				if($DEBUG) { echo ' yes.'; }
+				if ($DEBUG) { echo ' yes.'; }
 				$less++;
-				if($i<3)
+				if ($i < 3)
 				{
 					/* Ensure that this is definitely greater */
-					if($DEBUG) { echo"  This is the $num[$i] octet, so A is definitely greater than B."; }
+					if ($DEBUG) { echo"  This is the $num[$i] octet, so A is definitely greater than B."; }
 					$less = 5;
 					break;
 				}
 			}
-			elseif((int)$testa[$i] < (int)$testb[$i])
+			elseif ((int)$testa[$i] < (int)$testb[$i])
 			{
-				if($DEBUG) { echo ' no.'; }
+				if ($DEBUG) { echo ' no.'; }
 				$less--;
-				if($i<2)
+				if ($i < 2)
 				{
 					/* Ensure that this is definitely smaller */
-					if($DEBUG) { echo"  This is the $num[$i] octet, so A is definitely less than B."; }
+					if ($DEBUG) { echo"  This is the $num[$i] octet, so A is definitely less than B."; }
 					$less = -5;
 					break;
 				}
 			}
 			else
 			{
-				if($DEBUG) { echo ' no, they are equal.'; }
+				if ($DEBUG) { echo ' no, they are equal.'; }
 				$less = 0;
 			}
 		}
-		if($DEBUG) { echo '<br>Check value is: "'.$less.'"'; }
-		if($less>0)
+		if ($DEBUG) { echo '<br>Check value is: "'.$less.'"'; }
+		if ($less > 0)
 		{
-			if($DEBUG) { echo '<br>A is greater than B'; }
+			if ($DEBUG) { echo '<br>A is greater than B'; }
 			return True;
 		}
-		elseif($less<0)
+		elseif ($less < 0)
 		{
-			if($DEBUG) { echo '<br>A is less than B'; }
+			if ($DEBUG) { echo '<br>A is less than B'; }
 			return False;
 		}
 		else
 		{
-			if($DEBUG) { echo '<br>A is equal to B'; }
+			if ($DEBUG) { echo '<br>A is equal to B'; }
 			return False;
 		}
 	}
@@ -915,7 +915,7 @@ 
 	 */
 	var $accounts;
 
-	function setup_account_object(array $config=array())
+	function setup_account_object(array $config = array())
 	{
 		if (!isset($this->accounts) || $this->accounts->config || $config)
 		{
@@ -926,9 +926,9 @@ 
 			if (!$config)
 			{
 				// load the configuration from the database
-				foreach($this->db->select($this->config_table,'config_name,config_value',
+				foreach ($this->db->select($this->config_table, 'config_name,config_value',
 					"config_name LIKE 'ads%' OR config_name LIKE 'ldap%' OR config_name LIKE 'account_%' OR config_name LIKE '%encryption%' OR config_name='auth_type'",
-					__LINE__,__FILE__) as $row)
+					__LINE__, __FILE__) as $row)
 				{
 					$GLOBALS['egw_info']['server'][$row['config_name']] = $config[$row['config_name']] = $row['config_value'];
 				}
@@ -941,7 +941,7 @@ 
 				return false;
 			}
 			if (!isset($GLOBALS['egw']->accounts)) $GLOBALS['egw']->accounts = $this->accounts;
-			Api\Accounts::cache_invalidate();	// the cache is shared for all instances of the class
+			Api\Accounts::cache_invalidate(); // the cache is shared for all instances of the class
 		}
 		return true;
 	}
@@ -967,7 +967,7 @@ 
 	 * @param string &$anonpw=null on return password for anonymous user
 	 * @return int the numerical user-id
 	 */
-	function add_account($username,$first,$last,$passwd,$primary_group='Default',$changepw=False,$email='',&$anonpw=null)
+	function add_account($username, $first, $last, $passwd, $primary_group = 'Default', $changepw = False, $email = '', &$anonpw = null)
 	{
 		$this->setup_account_object();
 
@@ -979,7 +979,7 @@ 
 			$passwd = $anonpw = $this->anonpw;
 		}
 
-		if(!($accountid = $this->accounts->name2id($username, 'account_lid', $primary_group ? 'u' : 'g')))
+		if (!($accountid = $this->accounts->name2id($username, 'account_lid', $primary_group ? 'u' : 'g')))
 		{
 			$account = array(
 				'account_type'      => $primary_group ? 'u' : 'g',
@@ -1000,7 +1000,7 @@ 
 			}
 		}
 		// set password for existing account, if given and not '*unchanged*'
-		elseif($passwd && $passwd != '*unchanged*')
+		elseif ($passwd && $passwd != '*unchanged*')
 		{
 			try {
 				$auth = new Api\Auth;
@@ -1038,7 +1038,7 @@ 
 		{
 			$this->set_memberships(array($primary_group_id), $accountid);
 
-			if (!$changepw) $this->add_acl('preferences','nopasswordchange',$accountid);
+			if (!$changepw) $this->add_acl('preferences', 'nopasswordchange', $accountid);
 		}
 		//error_log("setup::add_account('$username','$first','$last',\$passwd,'$primary_group',$changepw,'$email') successfull created accountid=$accountid");
 		return $accountid;
@@ -1109,7 +1109,7 @@ 
 	 * @param int|string $account accountid or account_lid
 	 * @param int $rights rights to set, default 1
 	 */
-	function add_acl($apps,$location,$account,$rights=1)
+	function add_acl($apps, $location, $account, $rights = 1)
 	{
 		//error_log("setup::add_acl(".(is_array($apps) ? "array('".implode("','",$apps)."')" : "'$apps'").",'$location',$account,$rights)");
 		if (!is_numeric($account))
@@ -1117,32 +1117,32 @@ 
 			$this->setup_account_object();
 			$account = $this->accounts->name2id($account);
 		}
-		if(!is_object($this->db))
+		if (!is_object($this->db))
 		{
 			$this->loaddb();
 		}
 
-		if(!is_array($apps))
+		if (!is_array($apps))
 		{
 			$apps = array($apps);
 		}
-		foreach($apps as $app)
+		foreach ($apps as $app)
 		{
-			$this->db->delete($this->acl_table,array(
+			$this->db->delete($this->acl_table, array(
 				'acl_appname'  => $app,
 				'acl_location' => $location,
 				'acl_account'  => $account
-			),__LINE__,__FILE__);
+			), __LINE__, __FILE__);
 
-			if ((int) $rights)
+			if ((int)$rights)
 			{
-				$this->db->insert($this->acl_table,array(
+				$this->db->insert($this->acl_table, array(
 					'acl_rights' => $rights
-				),array(
+				), array(
 					'acl_appname'  => $app,
 					'acl_location' => $location,
 					'acl_account'  => $account,
-				),__LINE__,__FILE__);
+				), __LINE__, __FILE__);
 			}
 		}
 	}
@@ -1153,7 +1153,7 @@ 
 	 * @param array $tables array with possible table-names
 	 * @return string/boolean tablename or false
 	 */
-	function table_exist($tables,$force_refresh=False)
+	function table_exist($tables, $force_refresh = False)
 	{
 		static $table_names = False;
 
@@ -1161,9 +1161,9 @@ 
 
 		if (!$table_names) return false;
 
-		foreach($table_names as $data)
+		foreach ($table_names as $data)
 		{
-			if (($key = array_search($data['table_name'],$tables)) !== false)
+			if (($key = array_search($data['table_name'], $tables)) !== false)
 			{
 				return $tables[$key];
 			}
@@ -1176,18 +1176,18 @@ 
 	 *
 	 * Other tables can always use the most up to date name
 	 */
-	function set_table_names($force_refresh=False)
+	function set_table_names($force_refresh = False)
 	{
-		foreach(array(
-			'config_table'       => array('egw_config','phpgw_config','config'),
-			'applications_table' => array('egw_applications','phpgw_applications','applications'),
-			'accounts_table'     => array('egw_accounts','phpgw_accounts'),
-			'acl_table'          => array('egw_acl','phpgw_acl'),
-			'lang_table'         => array('egw_lang','phpgw_lang','lang'),
-			'languages_table'    => array('egw_languages','phpgw_languages','languages'),
+		foreach (array(
+			'config_table'       => array('egw_config', 'phpgw_config', 'config'),
+			'applications_table' => array('egw_applications', 'phpgw_applications', 'applications'),
+			'accounts_table'     => array('egw_accounts', 'phpgw_accounts'),
+			'acl_table'          => array('egw_acl', 'phpgw_acl'),
+			'lang_table'         => array('egw_lang', 'phpgw_lang', 'lang'),
+			'languages_table'    => array('egw_languages', 'phpgw_languages', 'languages'),
 		) as $name => $tables)
 		{
-			$table = $this->table_exist($tables,$force_refresh);
+			$table = $this->table_exist($tables, $force_refresh);
 
 			if ($table && $table != $this->$name)	// only overwrite the default name, if we realy got one (important for new installs)
 			{