EGroupware / egroupware

api/src/Contacts/Merge.php

Spacing +34 added lines, -34 removed lines

@@ -16,7 +16,7 @@ 
 use EGroupware\Api;
 
 // explicit import not namespaced classes
-use calendar_boupdate;	// we detect if it is not available
+use calendar_boupdate; // we detect if it is not available
 
 /**
  * Contacts document merge
@@ -55,22 +55,22 @@ 
 	 * @param boolean $ignore_acl =false true: no acl check
 	 * @return array|boolean
 	 */
-	protected function get_replacements($id,&$content=null,$ignore_acl=false)
+	protected function get_replacements($id, &$content = null, $ignore_acl = false)
 	{
-		if (!($replacements = $this->contact_replacements($id,'',$ignore_acl, $content)))
+		if (!($replacements = $this->contact_replacements($id, '', $ignore_acl, $content)))
 		{
 			return false;
 		}
-		if($content && strpos($content, '$$#') !== false)
+		if ($content && strpos($content, '$$#') !== false)
 		{
-			$this->cf_link_to_expand($this->contacts->read($id, $ignore_acl), $content, $replacements,'addressbook');
+			$this->cf_link_to_expand($this->contacts->read($id, $ignore_acl), $content, $replacements, 'addressbook');
 		}
 
 		// Links
 		$replacements += $this->get_all_links('addressbook', $id, '', $content);
-		if (!(strpos($content,'$$calendar/') === false))
+		if (!(strpos($content, '$$calendar/') === false))
 		{
-			$replacements += $this->calendar_replacements($id,!(strpos($content,'$$calendar/-1/') === false));
+			$replacements += $this->calendar_replacements($id, !(strpos($content, '$$calendar/-1/') === false));
 		}
 		return $replacements;
 	}
@@ -82,7 +82,7 @@ 
 	 * @param boolean $last_event_too =false also include information about the last event
 	 * @return array
 	 */
-	protected function calendar_replacements($id,$last_event_too=false)
+	protected function calendar_replacements($id, $last_event_too = false)
 	{
 		if (!class_exists('calendar_boupdate')) return array();
 
@@ -101,7 +101,7 @@ 
 		{
 			$events = array();
 		}
-		if ($last_event_too==true)
+		if ($last_event_too == true)
 		{
 			$last = $calendar->search(array(
 				'end' => $calendar->now_su,
@@ -111,41 +111,41 @@ 
 				'order' => 'cal_start DESC',
 				'enum_recurring' => true
 			));
-			$events['-1'] = $last ? array_shift($last) : array();	// returned events are indexed by cal_id!
+			$events['-1'] = $last ? array_shift($last) : array(); // returned events are indexed by cal_id!
 		}
 		$replacements = array();
-		$n = 1;  // Returned events are indexed by cal_id, need to index sequentially
-		foreach($events as $key => $event)
+		$n = 1; // Returned events are indexed by cal_id, need to index sequentially
+		foreach ($events as $key => $event)
 		{
 			// Use -1 for previous key
-			if($key < 0) $n = $key;
+			if ($key < 0) $n = $key;
 
-			foreach($calendar->event2array($event) as $name => $data)
+			foreach ($calendar->event2array($event) as $name => $data)
 			{
-				if (substr($name,-4) == 'date') $name = substr($name,0,-4);
-				$replacements['$$calendar/'.$n.'/'.$name.'$$'] = is_array($data['data']) ? implode(', ',$data['data']) : $data['data'];
+				if (substr($name, -4) == 'date') $name = substr($name, 0, -4);
+				$replacements['$$calendar/'.$n.'/'.$name.'$$'] = is_array($data['data']) ? implode(', ', $data['data']) : $data['data'];
 			}
-			foreach(array('start','end') as $what)
+			foreach (array('start', 'end') as $what)
 			{
-				foreach(array(
+				foreach (array(
 					'date' => $GLOBALS['egw_info']['user']['preferences']['common']['dateformat'],
 					'day'  => 'l',
 					'time' => $GLOBALS['egw_info']['user']['preferences']['common']['timeformat'] == 12 ? 'h:i a' : 'H:i',
 				) as $name => $format)
 				{
-					$value = $event[$what] ? date($format,$event[$what]) : '';
+					$value = $event[$what] ? date($format, $event[$what]) : '';
 					if ($format == 'l') $value = lang($value);
 					$replacements['$$calendar/'.$n.'/'.$what.$name.'$$'] = $value;
 				}
 			}
-			$duration = ($event['end'] - $event['start'])/60;
-			$replacements['$$calendar/'.$n.'/duration$$'] = floor($duration/60).lang('h').($duration%60 ? $duration%60 : '');
+			$duration = ($event['end'] - $event['start']) / 60;
+			$replacements['$$calendar/'.$n.'/duration$$'] = floor($duration / 60).lang('h').($duration % 60 ? $duration % 60 : '');
 
 			++$n;
 		}
 
 		// Need to set some keys if there is no previous event
-		if($last_event_too && count($events['-1']) == 0) {
+		if ($last_event_too && count($events['-1']) == 0) {
 			$replacements['$$calendar/-1/start$$'] = '';
 			$replacements['$$calendar/-1/end$$'] = '';
 			$replacements['$$calendar/-1/owner$$'] = '';
@@ -168,18 +168,18 @@ 
 		echo '<tr><td colspan="4"><h3>'.lang('Contact fields:')."</h3></td></tr>";
 
 		$n = 0;
-		foreach($this->contacts->contact_fields as $name => $label)
+		foreach ($this->contacts->contact_fields as $name => $label)
 		{
-			if (in_array($name,array('tid','label','geo'))) continue;	// dont show them, as they are not used in the UI atm.
+			if (in_array($name, array('tid', 'label', 'geo'))) continue; // dont show them, as they are not used in the UI atm.
 
-			if (in_array($name,array('email','org_name','tel_work','url')) && $n&1)		// main values, which should be in the first column
+			if (in_array($name, array('email', 'org_name', 'tel_work', 'url')) && $n&1)		// main values, which should be in the first column
 			{
 				echo "</tr>\n";
 				$n++;
 			}
 			if (!($n&1)) echo '<tr>';
 			echo '<td>{{'.$name.'}}</td><td>'.$label.'</td>';
-			if($name == 'cat_id')
+			if ($name == 'cat_id')
 			{
 				if ($n&1) echo "</tr>\n";
 				echo '<td>{{categories}}</td><td>'.lang('Category path').'</td>';
@@ -190,13 +190,13 @@ 
 		}
 
 		echo '<tr><td colspan="4"><h3>'.lang('Custom fields').":</h3></td></tr>";
-		foreach($this->contacts->customfields as $name => $field)
+		foreach ($this->contacts->customfields as $name => $field)
 		{
 			echo '<tr><td>{{#'.$name.'}}</td><td colspan="3">'.$field['label']."</td></tr>\n";
 		}
 
 		echo '<tr><td colspan="4"><h3>'.lang('General fields:')."</h3></td></tr>";
-		foreach(array(
+		foreach (array(
 			'link' => lang('HTML link to the current record'),
 			'links' => lang('Titles of any entries linked to the current record, excluding attached files'),
  			'attachments' => lang('List of files linked to the current record'),
@@ -223,7 +223,7 @@ 
 
 		Api\Translation::add_app('calendar');
 		echo '<tr><td colspan="4"><h3>'.lang('Calendar fields:')." # = 1, 2, ..., 20, -1</h3></td></tr>";
-		foreach(array(
+		foreach (array(
 			'title' => lang('Title'),
 			'description' => lang('Description'),
 			'participants' => lang('Participants'),
@@ -245,7 +245,7 @@ 
 			'owner'    => lang('Owner'),
 		) as $name => $label)
 		{
-			if (in_array($name,array('start','end')) && $n&1)		// main values, which should be in the first column
+			if (in_array($name, array('start', 'end')) && $n&1)		// main values, which should be in the first column
 			{
 				echo "</tr>\n";
 				$n++;
@@ -274,8 +274,8 @@ 
 	 * @param int|string $export_limit =null export-limit, default $GLOBALS['egw_info']['server']['export_limit']
 	 * @return array see nextmatch_widget::egw_actions
 	 */
-	public static function document_action($dirs, $group=0, $caption='Insert in document', $prefix='document_', $default_doc='',
-		$export_limit=null)
+	public static function document_action($dirs, $group = 0, $caption = 'Insert in document', $prefix = 'document_', $default_doc = '',
+		$export_limit = null)
 	{
 		$actions = parent::document_action($dirs, $group, $caption, $prefix, $default_doc, $export_limit);
 
@@ -287,14 +287,14 @@ 
 
 	protected static function customise_mail_actions(&$action)
 	{
-		if(strpos($action['egw_open'], 'edit-mail') === 0)
+		if (strpos($action['egw_open'], 'edit-mail') === 0)
 		{
 			unset($action['confirm_multiple']);
 			$action['onExecute'] = 'javaScript:app.addressbook.merge_mail';
 		}
 		else if ($action['children'])
 		{
-			foreach($action['children'] as &$child)
+			foreach ($action['children'] as &$child)
 			{
 				static::customise_mail_actions($child);
 			}

api/src/Link/Sharing.php

Spacing +4 added lines, -4 removed lines

@@ -36,11 +36,11 @@ 
 	 * @param boolean $keep_session =null null: create a new session, true: try mounting it into existing (already verified) session
 	 * @return string with sessionid
 	 */
-	public static function create_session($keep_session=null)
+	public static function create_session($keep_session = null)
 	{
 		$share = array();
 		$success = static::check_token($keep_session, $share);
-		if($success)
+		if ($success)
 		{
 			static::setup_entry($share);
 			return static::login($keep_session, $share);
@@ -93,9 +93,9 @@ 
 				// Just email already filtered out
 				$documents['children'] = $documents['children']['message/rfc822']['children'];
 			}
-			foreach($documents['children'] as $key => &$document)
+			foreach ($documents['children'] as $key => &$document)
 			{
-				if(strpos($document['target'],'compose_') === FALSE)
+				if (strpos($document['target'], 'compose_') === FALSE)
 				{
 					unset($documents['children'][$key]);
 					continue;

admin/inc/class.admin_cmd.inc.php

Spacing +124 added lines, -125 removed lines

@@ -60,14 +60,14 @@ 
 	const successful = 2;
 	const failed     = 3;
 	const pending    = 4;
-	const queued     = 5;	// command waits to be fetched from remote
+	const queued     = 5; // command waits to be fetched from remote
 
 	/**
 	 * Status which stil need passwords available
 	 *
 	 * @var array
 	 */
-	static $require_pw_stati = array(self::scheduled,self::pending,self::queued);
+	static $require_pw_stati = array(self::scheduled, self::pending, self::queued);
 
 	/**
 	 * The status of the command, one of either scheduled, successful, failed or deleted
@@ -153,7 +153,7 @@ 
 	 * @return string success message
 	 * @throws Exception()
 	 */
-	protected abstract function exec($check_only=false);
+	protected abstract function exec($check_only = false);
 
 	/**
 	 * Return a title / string representation for a given command, eg. to display it
@@ -190,7 +190,7 @@ 
 
 		$this->type = get_class($this);
 
-		foreach($data as $name => $value)
+		foreach ($data as $name => $value)
 		{
 			$this->$name = $name == 'data' && !is_array($value) ? json_php_unserialize($value) : $value;
 		}
@@ -209,13 +209,13 @@ 
 	 * @return mixed return value of the command
 	 * @throws Exceptions on error
 	 */
-	function run($time=null,$set_modifier=true,$skip_checks=false,$dry_run=false)
+	function run($time = null, $set_modifier = true, $skip_checks = false, $dry_run = false)
 	{
 		if (!is_null($time))
 		{
 			$this->scheduled = $time;
 			$this->status = admin_cmd::scheduled;
-			$ret = lang('Command scheduled to run at %1',date('Y-m-d H:i',$time));
+			$ret = lang('Command scheduled to run at %1', date('Y-m-d H:i', $time));
 			// running the checks of the arguments for local commands, if not explicitly requested to not run them
 			if (!$this->remote_id && !$skip_checks)
 			{
@@ -281,21 +281,21 @@ 
 	{
 		if (!($remote = $this->read_remote($this->remote_id)))
 		{
-			throw new Api\Exception\WrongUserinput(lang('Invalid remote id or name "%1"!',$this->remote_id),997);
+			throw new Api\Exception\WrongUserinput(lang('Invalid remote id or name "%1"!', $this->remote_id), 997);
 		}
 		if (!$this->uid)
 		{
-			$this->save();	// to get the uid
+			$this->save(); // to get the uid
 		}
 		$secret = md5($this->uid.$remote['remote_hash']);
 
 		$postdata = $this->as_array();
 		if (is_object($GLOBALS['egw']->translation))
 		{
-			$postdata = Api\Translation::convert($postdata,Api\Translation::charset(),'utf-8');
+			$postdata = Api\Translation::convert($postdata, Api\Translation::charset(), 'utf-8');
 		}
 		// dont send the id's which have no meaning on the remote install
-		foreach(array('id','creator','modifier','requested','remote_id') as $name)
+		foreach (array('id', 'creator', 'modifier', 'requested', 'remote_id') as $name)
 		{
 			unset($postdata[$name]);
 		}
@@ -321,12 +321,12 @@ 
 		}
 		if (is_object($GLOBALS['egw']->translation))
 		{
-			$message = Api\Translation::convert($message,'utf-8');
+			$message = Api\Translation::convert($message, 'utf-8');
 		}
 		$matches = null;
-		if (is_string($message) && preg_match('/^([0-9]+) (.*)$/',$message,$matches))
+		if (is_string($message) && preg_match('/^([0-9]+) (.*)$/', $message, $matches))
 		{
-			throw new Api\Exception($matches[2],(int)$matches[1]);
+			throw new Api\Exception($matches[2], (int)$matches[1]);
 		}
 		return $message;
 	}
@@ -352,7 +352,7 @@ 
 	 * @param boolean $set_modifier =true set the current user as modifier or 0 (= run by the system)
 	 * @return boolean true on success, false otherwise
 	 */
-	function save($set_modifier=true)
+	function save($set_modifier = true)
 	{
 		admin_cmd::_instanciate_sql();
 
@@ -367,7 +367,7 @@ 
 			$this->modifier = $set_modifier ? $GLOBALS['egw_info']['user']['account_id'] : 0;
 			if ($set_modifier) $this->modifier_email = admin_cmd::user_email();
 		}
-		$vars = get_object_vars($this);	// does not work in php5.1.2 due a bug
+		$vars = get_object_vars($this); // does not work in php5.1.2 due a bug
 
 		// data is stored serialized
 		// paswords are masked / removed, if we dont need them anymore
@@ -406,7 +406,7 @@ 
 			$this->set_periodic_job();
 		}
 		// existing object with no rrule, cancle evtl. running periodic job
-		elseif($vars['id'])
+		elseif ($vars['id'])
 		{
 			$this->cancel_periodic_job();
 		}
@@ -420,13 +420,13 @@ 
 	 * @param boolean $return_serialized =true true: return json serialized string, false: return array
 	 * @return string|array see $return_serialized
 	 */
-	static function mask_passwords($data, $return_serialized=true)
+	static function mask_passwords($data, $return_serialized = true)
 	{
 		if (!is_array($data))
 		{
 			$data = json_php_unserialize($data);
 		}
-		foreach($data as $key => &$value)
+		foreach ($data as $key => &$value)
 		{
 			if (is_array($value))
 			{
@@ -474,10 +474,10 @@ 
 		{
 			$data['data'] = json_php_unserialize($data['data']);
 		}
-		if (!(class_exists($class = 'EGroupware\\'.$data['type']) ||	// namespaced class
+		if (!(class_exists($class = 'EGroupware\\'.$data['type']) || // namespaced class
 			class_exists($class = $data['type'])) || $data['type'] == 'admin_cmd')
 		{
-			throw new Api\Exception\WrongParameter(lang('Unknown command %1!',$class), 10);
+			throw new Api\Exception\WrongParameter(lang('Unknown command %1!', $class), 10);
 		}
 		$cmd = new $class($data);
 
@@ -485,7 +485,7 @@ 
 		{
 			return $cmd;
 		}
-		throw new Api\Exception\WrongParameter(lang('%1 is no command!',$class), 10);
+		throw new Api\Exception\WrongParameter(lang('%1 is no command!', $class), 10);
 	}
 
 	/**
@@ -497,7 +497,7 @@ 
 	 * @param array $readonlys
 	 * @return int
 	 */
-	static function get_rows($query,&$rows,$readonlys)
+	static function get_rows($query, &$rows, $readonlys)
 	{
 		admin_cmd::_instanciate_sql();
 
@@ -514,21 +514,21 @@ 
 			$query['col_filter'][] = 'cmd_rrule IS NOT NULL';
 		}
 		unset($query['col_filter']['periodic']);
-		if($query['col_filter']['parent'])
+		if ($query['col_filter']['parent'])
 		{
 			$query['col_filter']['parent'] = (int)$query['col_filter']['parent'];
 		}
 
-		$total = admin_cmd::$sql->get_rows($query,$rows,$readonlys);
+		$total = admin_cmd::$sql->get_rows($query, $rows, $readonlys);
 
 		if (!$rows) return 0;
 
 		$async = new Api\Asyncservice();
-		foreach($rows as &$row)
+		foreach ($rows as &$row)
 		{
 			try {
 				$cmd = admin_cmd::instanciate($row);
-				$row['title'] = $cmd->__tostring();	// we call __tostring explicit, as a cast to string requires php5.2+
+				$row['title'] = $cmd->__tostring(); // we call __tostring explicit, as a cast to string requires php5.2+
 			}
 			catch (Exception $e) {
 				$row['title'] = $e->getMessage();
@@ -536,25 +536,24 @@ 
 
 			$row['value'] = $cmd->value;
 
-			if(method_exists($cmd, 'summary'))
+			if (method_exists($cmd, 'summary'))
 			{
 				$row['data'] = $cmd->summary();
 			}
 			else
 			{
-				$row['data'] = !($data = json_php_unserialize($row['data'])) ? '' :
-					json_encode($data+(empty($row['rrule'])?array():array('rrule' => $row['rrule'])),
+				$row['data'] = !($data = json_php_unserialize($row['data'])) ? '' : json_encode($data + (empty($row['rrule']) ? array() : array('rrule' => $row['rrule'])),
 						JSON_PRETTY_PRINT|JSON_UNESCAPED_UNICODE|JSON_UNESCAPED_SLASHES);
 			}
-			if($row['rrule'])
+			if ($row['rrule'])
 			{
-				$rrule = calendar_rrule::event2rrule(calendar_rrule::parseRrule($row['rrule'],true)+array(
+				$rrule = calendar_rrule::event2rrule(calendar_rrule::parseRrule($row['rrule'], true) + array(
 					'start' => time(),
 					'tzid'=> Api\DateTime::$server_timezone->getName()
 				));
 				$row['rrule'] = ''.$rrule;
 			}
-			if(!$row['scheduled'] && $cmd && $cmd->async_job_id)
+			if (!$row['scheduled'] && $cmd && $cmd->async_job_id)
 			{
 				$job = $async->read($cmd->async_job_id);
 
@@ -580,11 +579,11 @@ 
 			admin_cmd::_instanciate_sql();
 
 			// Need a new one to avoid column name modification
-			$sql = new Api\Storage\Base('admin','egw_admin_queue',null);
+			$sql = new Api\Storage\Base('admin', 'egw_admin_queue', null);
 			$labels = $sql->query_list('cmd_type');
 
 			// for admin app we also add all available cmd objects
-			foreach(scandir(__DIR__) as $file)
+			foreach (scandir(__DIR__) as $file)
 			{
 				$matches = null;
 				if (preg_match('/^class\.(admin_cmd_.*)\.inc\.php$/', $file, $matches))
@@ -595,15 +594,15 @@ 
 					}
 				}
 			}
-			foreach($labels as $class => &$label)
+			foreach ($labels as $class => &$label)
 			{
-				if(class_exists($class))
+				if (class_exists($class))
 				{
 					$label = $class::name();
 				}
-				elseif (class_exists('EGroupware\\' . $class))
+				elseif (class_exists('EGroupware\\'.$class))
 				{
-					$class = 'EGroupware\\' . $class;
+					$class = 'EGroupware\\'.$class;
 					$label = $class::name();
 				}
 			}
@@ -634,11 +633,11 @@ 
 	 * @param array $filter =null if set (!=null) col-data pairs, to be and-ed (!) into the query without wildcards
 	 * @return array
 	 */
-	static function &search($criteria,$only_keys=True,$order_by='',$extra_cols='',$wildcard='',$empty=False,$op='AND',$start=false,$filter=null)
+	static function &search($criteria, $only_keys = True, $order_by = '', $extra_cols = '', $wildcard = '', $empty = False, $op = 'AND', $start = false, $filter = null)
 	{
 		admin_cmd::_instanciate_sql();
 
-		return admin_cmd::$sql->search($criteria,$only_keys,$order_by,$extra_cols,$wildcard,$empty,$op,$start,$filter);
+		return admin_cmd::$sql->search($criteria, $only_keys, $order_by, $extra_cols, $wildcard, $empty, $op, $start, $filter);
 	}
 
 	/**
@@ -650,7 +649,7 @@ 
 	{
 		if (is_null(admin_cmd::$sql))
 		{
-			admin_cmd::$sql = new Api\Storage\Base('admin','egw_admin_queue',null,'cmd_');
+			admin_cmd::$sql = new Api\Storage\Base('admin', 'egw_admin_queue', null, 'cmd_');
 		}
 	}
 
@@ -663,7 +662,7 @@ 
 	{
 		if (is_null(admin_cmd::$remote))
 		{
-			admin_cmd::$remote = new Api\Storage\Base('admin','egw_admin_remote');
+			admin_cmd::$remote = new Api\Storage\Base('admin', 'egw_admin_remote');
 		}
 	}
 
@@ -675,11 +674,11 @@ 
 	 */
 	function __get($property)
 	{
-		if (property_exists('admin_cmd',$property))
+		if (property_exists('admin_cmd', $property))
 		{
-			return $this->$property;	// making all (non static) class vars readonly available
+			return $this->$property; // making all (non static) class vars readonly available
 		}
-		switch($property)
+		switch ($property)
 		{
 			case 'accounts':
 				self::_instanciate_accounts();
@@ -698,9 +697,9 @@ 
 	 */
 	function __isset($property)
 	{
-		if (property_exists('admin_cmd',$property))
+		if (property_exists('admin_cmd', $property))
 		{
-			return isset($this->$property);	// making all (non static) class vars readonly available
+			return isset($this->$property); // making all (non static) class vars readonly available
 		}
 		return isset($this->data[$property]);
 	}
@@ -712,7 +711,7 @@ 
 	 * @param mixed $value
 	 * @return mixed
 	 */
-	function __set($property,$value)
+	function __set($property, $value)
 	{
 		$this->data[$property] = $value;
 	}
@@ -734,19 +733,19 @@ 
 	 */
 	function as_array()
 	{
-		if (version_compare(PHP_VERSION,'5.1.2','>'))
+		if (version_compare(PHP_VERSION, '5.1.2', '>'))
 		{
-			$vars = get_object_vars($this);	// does not work in php5.1.2 due a bug
+			$vars = get_object_vars($this); // does not work in php5.1.2 due a bug
 		}
 		else
 		{
-			foreach(array_keys(get_class_vars(__CLASS__)) as $name)
+			foreach (array_keys(get_class_vars(__CLASS__)) as $name)
 			{
 				$vars[$name] = $this->$name;
 			}
 		}
 		unset($vars['data']);
-		if ($this->data) $vars = array_merge($this->data,$vars);
+		if ($this->data) $vars = array_merge($this->data, $vars);
 
 		return $vars;
 	}
@@ -758,14 +757,14 @@ 
 	 * @param int $extra_deny =null further admin rights to check, eg. 16 = deny edit Api\Accounts
 	 * @throws Api\Exception\NoPermission\Admin
 	 */
-	protected function _check_admin($extra_acl=null,$extra_deny=null)
+	protected function _check_admin($extra_acl = null, $extra_deny = null)
 	{
 		if ($this->creator)
 		{
 			admin_cmd::_instanciate_acl($this->creator);
 			// todo: check only if and with $this->creator
-			if (!admin_cmd::$acl->check('run',1,'admin') &&		// creator is no longer admin
-				$extra_acl && $extra_deny && admin_cmd::$acl->check($extra_acl,$extra_deny,'admin'))	// creator is explicitly forbidden to do something
+			if (!admin_cmd::$acl->check('run', 1, 'admin') && // creator is no longer admin
+				$extra_acl && $extra_deny && admin_cmd::$acl->check($extra_acl, $extra_deny, 'admin'))	// creator is explicitly forbidden to do something
 			{
 				throw new Api\Exception\NoPermission\Admin();
 			}
@@ -781,13 +780,13 @@ 
 	 */
 	static function parse_apps(array $apps)
 	{
-		foreach($apps as $key => $name)
+		foreach ($apps as $key => $name)
 		{
 			if (!isset($GLOBALS['egw_info']['apps'][$name]))
 			{
-				foreach($GLOBALS['egw_info']['apps'] as $app => $data)	// check against title and localised name
+				foreach ($GLOBALS['egw_info']['apps'] as $app => $data)	// check against title and localised name
 				{
-					if (!strcasecmp($name,$data['title']) || !strcasecmp($name,lang($app)))
+					if (!strcasecmp($name, $data['title']) || !strcasecmp($name, lang($app)))
 					{
 						$apps[$key] = $name = $app;
 						break;
@@ -796,7 +795,7 @@ 
 			}
 			if (!isset($GLOBALS['egw_info']['apps'][$name]))
 			{
-				throw new Api\Exception\WrongUserinput(lang("Application '%1' not found (maybe not installed or misspelled)!",$name),8);
+				throw new Api\Exception\WrongUserinput(lang("Application '%1' not found (maybe not installed or misspelled)!", $name), 8);
 			}
 		}
 		return $apps;
@@ -811,20 +810,20 @@ 
 	 * @throws Api\Exception\WrongUserinput(lang("Unknown account: %1 !!!",$account), 15);
 	 * @throws Api\Exception\WrongUserinput(lang("Wrong account type: %1 is NO %2 !!!",$account,$allow_only_user?lang('user'):lang('group')), 16);
 	 */
-	static function parse_account($account,$allow_only_user=null)
+	static function parse_account($account, $allow_only_user = null)
 	{
 		admin_cmd::_instanciate_accounts();
 
 		if (!($type = admin_cmd::$accounts->exists($account)) ||
-			!is_numeric($id=$account) && !($id = admin_cmd::$accounts->name2id($account)))
+			!is_numeric($id = $account) && !($id = admin_cmd::$accounts->name2id($account)))
 		{
-			throw new Api\Exception\WrongUserinput(lang("Unknown account: %1 !!!",$account), 15);
+			throw new Api\Exception\WrongUserinput(lang("Unknown account: %1 !!!", $account), 15);
 		}
 		if (!is_null($allow_only_user) && $allow_only_user !== ($type == 1))
 		{
-			throw new Api\Exception\WrongUserinput(lang("Wrong account type: %1 is NO %2 !!!",$account,$allow_only_user?lang('user'):lang('group')), 16);
+			throw new Api\Exception\WrongUserinput(lang("Wrong account type: %1 is NO %2 !!!", $account, $allow_only_user ?lang('user') : lang('group')), 16);
 		}
-		if ($type == 2 && $id > 0) $id = -$id;	// groups use negative id's internally, fix it, if user given the wrong sign
+		if ($type == 2 && $id > 0) $id = -$id; // groups use negative id's internally, fix it, if user given the wrong sign
 
 		return $id;
 	}
@@ -838,14 +837,14 @@ 
 	 * @throws Api\Exception\WrongUserinput(lang("Unknown account: %1 !!!",$account), 15);
 	 * @throws Api\Exception\WrongUserinput(lang("Wrong account type: %1 is NO %2 !!!",$account,$allow_only?lang('user'):lang('group')), 16);
 	 */
-	static function parse_accounts($accounts,$allow_only_user=null)
+	static function parse_accounts($accounts, $allow_only_user = null)
 	{
 		if (!$accounts) return null;
 
 		$ids = array();
-		foreach(is_array($accounts) ? $accounts : explode(',',$accounts) as $account)
+		foreach (is_array($accounts) ? $accounts : explode(',', $accounts) as $account)
 		{
-			$ids[] = admin_cmd::parse_account($account,$allow_only_user);
+			$ids[] = admin_cmd::parse_account($account, $allow_only_user);
 		}
 		return $ids;
 	}
@@ -863,11 +862,11 @@ 
 		{
 			$datein = $date;
 			// convert german DD.MM.YYYY format into ISO YYYY-MM-DD format
-			$date = preg_replace('/^([0-9]{1,2})\.([0-9]{1,2})\.([0-9]{4})$/','\3-\2-\1',$date);
+			$date = preg_replace('/^([0-9]{1,2})\.([0-9]{1,2})\.([0-9]{4})$/', '\3-\2-\1', $date);
 
-			if (($date = strtotime($date))  === false)
+			if (($date = strtotime($date)) === false)
 			{
-				throw new Api\Exception\WrongUserinput(lang('Invalid formated date "%1"!',$datein),6);
+				throw new Api\Exception\WrongUserinput(lang('Invalid formated date "%1"!', $datein), 6);
 			}
 		}
 		return (int)$date;
@@ -881,7 +880,7 @@ 
 	 * @return boolean
 	 * @throws Api\Exception\WrongUserinput(lang('Invalid value "%1" use yes or no!',$value),998);
 	 */
-	static function parse_boolean($value,$default=null)
+	static function parse_boolean($value, $default = null)
 	{
 		if (is_bool($value) || is_int($value))
 		{
@@ -891,15 +890,15 @@ 
 		{
 			return $default;
 		}
-		if (in_array($value,array('1','yes','true',lang('yes'),lang('true'))))
+		if (in_array($value, array('1', 'yes', 'true', lang('yes'), lang('true'))))
 		{
 			return true;
 		}
-		if (in_array($value,array('0','no','false',lang('no'),lang('false'))))
+		if (in_array($value, array('0', 'no', 'false', lang('no'), lang('false'))))
 		{
 			return false;
 		}
-		throw new Api\Exception\WrongUserinput(lang('Invalid value "%1" use yes or no!',$value),998);
+		throw new Api\Exception\WrongUserinput(lang('Invalid value "%1" use yes or no!', $value), 998);
 	}
 
 	/**
@@ -917,9 +916,9 @@ 
 			'remote_id' => $id_or_name,
 			'remote_name' => $id_or_name,
 			'remote_domain' => $id_or_name,
-		),true,'','','',false,'OR')) || count($remotes) != 1)
+		), true, '', '', '', false, 'OR')) || count($remotes) != 1)
 		{
-			throw new Api\Exception\WrongUserinput(lang('Invalid remote id or name "%1"!',$id_or_name),997);
+			throw new Api\Exception\WrongUserinput(lang('Invalid remote id or name "%1"!', $id_or_name), 997);
 		}
 		return $remotes[0]['remote_id'];
 	}
@@ -936,7 +935,7 @@ 
 		{
 			if (!is_object($GLOBALS['egw']->accounts))
 			{
-				throw new Api\Exception\AssertionFailed(lang('%1 class not instanciated','accounts'),999);
+				throw new Api\Exception\AssertionFailed(lang('%1 class not instanciated', 'accounts'), 999);
 			}
 			admin_cmd::$accounts = $GLOBALS['egw']->accounts;
 		}
@@ -949,13 +948,13 @@ 
 	 * @param int $account =null account_id the class needs to be instanciated for, default need only account-independent methods
 	 * @throws Api\Exception\AssertionFailed(lang('%1 class not instanciated','acl'),999);
 	 */
-	protected function _instanciate_acl($account=null)
+	protected function _instanciate_acl($account = null)
 	{
 		if (!is_object(admin_cmd::$acl) || $account && admin_cmd::$acl->account_id != $account)
 		{
 			if (!is_object($GLOBALS['egw']->acl))
 			{
-				throw new Api\Exception\AssertionFailed(lang('%1 class not instanciated','acl'),999);
+				throw new Api\Exception\AssertionFailed(lang('%1 class not instanciated', 'acl'), 999);
 			}
 			if ($account && $GLOBALS['egw']->acl->account_id != $account)
 			{
@@ -975,20 +974,20 @@ 
 	 * @param $account_id =null account_id, default current user
 	 * @return string
 	 */
-	static function user_email($account_id=null)
+	static function user_email($account_id = null)
 	{
 		if ($account_id)
 		{
 			admin_cmd::_instanciate_accounts();
-			$fullname = admin_cmd::$accounts->id2name($account_id,'account_fullname');
-			$email = admin_cmd::$accounts->id2name($account_id,'account_email');
+			$fullname = admin_cmd::$accounts->id2name($account_id, 'account_fullname');
+			$email = admin_cmd::$accounts->id2name($account_id, 'account_email');
 		}
 		else
 		{
 			$fullname = $GLOBALS['egw_info']['user']['account_fullname'];
 			$email = $GLOBALS['egw_info']['user']['account_email'];
 		}
-		return $fullname . ($email ? ' <'.$email.'>' : '');
+		return $fullname.($email ? ' <'.$email.'>' : '');
 	}
 
 	/**
@@ -1012,11 +1011,11 @@ 
 		{
 			return false;
 		}
-		if (!($jobs = admin_cmd::search(array(),false,'cmd_scheduled','','',false,'AND',array(0,1),array(
+		if (!($jobs = admin_cmd::search(array(), false, 'cmd_scheduled', '', '', false, 'AND', array(0, 1), array(
 			'cmd_status' => admin_cmd::scheduled,
 		))))
 		{
-			return false;		// no schduled command, no need to setup the job
+			return false; // no schduled command, no need to setup the job
 		}
 		$next = $jobs[0];
 		if (($time = $next['scheduled']) < time())	// should run immediatly
@@ -1026,11 +1025,11 @@ 
 		$async = new Api\Asyncservice();
 
 		// we cant use this class as callback, as it's abstract and ExecMethod used by the async service instanciated the class!
-		list($app) = explode('_',$class=$next['type']);
+		list($app) = explode('_', $class = $next['type']);
 		$callback = $app.'.'.$class.'.run_queued_jobs';
 
-		$async->cancel_timer(admin_cmd::async_job_id);	// we delete it in case a job already exists
-		return $async->set_timer($time,admin_cmd::async_job_id,$callback,null,$next['creator']);
+		$async->cancel_timer(admin_cmd::async_job_id); // we delete it in case a job already exists
+		return $async->set_timer($time, admin_cmd::async_job_id, $callback, null, $next['creator']);
 	}
 
 	/**
@@ -1040,20 +1039,20 @@ 
 	 */
 	static function run_queued_jobs()
 	{
-		if (!($jobs = admin_cmd::search(array(),false,'cmd_scheduled','','',false,'AND',false,array(
+		if (!($jobs = admin_cmd::search(array(), false, 'cmd_scheduled', '', '', false, 'AND', false, array(
 			'cmd_status' => admin_cmd::scheduled,
 			'cmd_scheduled <= '.time(),
 		))))
 		{
-			return false;		// no schduled commands, no need to setup the job
+			return false; // no schduled commands, no need to setup the job
 		}
-		admin_cmd::$running_queued_jobs = true;	// stop admin_cmd::run() which calls admin_cmd::save() to install a new job
+		admin_cmd::$running_queued_jobs = true; // stop admin_cmd::run() which calls admin_cmd::save() to install a new job
 
-		foreach($jobs as $job)
+		foreach ($jobs as $job)
 		{
 			try {
 				$cmd = admin_cmd::instanciate($job);
-				$cmd->run(null,false);	// false = dont set current user as modifier, as job is run by the queue/system itself
+				$cmd->run(null, false); // false = dont set current user as modifier, as job is run by the queue/system itself
 			}
 			catch (Exception $e) {	// we need to mark that command as failed, to prevent further execution
 				_egw_log_exception($e);
@@ -1083,13 +1082,13 @@ 
 		if (empty($this->rrule)) return false;
 
 		// parse rrule and calculate next execution time
-		$event = calendar_rrule::parseRrule($this->rrule, true);	// true: allow HOURLY or MINUTELY
+		$event = calendar_rrule::parseRrule($this->rrule, true); // true: allow HOURLY or MINUTELY
 		// rrule can depend on start-time, use policy creation time by default, if rrule_start is not set
 		$event['start'] = empty($this->rrule_start) ? $this->created : $this->rrule_start;
 		$event['tzid'] = Api\DateTime::$server_timezone->getName();
-		$rrule = calendar_rrule::event2rrule($event, false);	// false = server timezone
+		$rrule = calendar_rrule::event2rrule($event, false); // false = server timezone
 		$rrule->rewind();
-		while((($time = $rrule->current()->format('ts'))) <= time())
+		while ((($time = $rrule->current()->format('ts'))) <= time())
 		{
 			$rrule->next();
 		}
@@ -1097,7 +1096,7 @@ 
 		// schedule run_periodic_job to run at that time
 		$async = new Api\Asyncservice();
 		$job_id = empty($this->async_job_id) ? self::PERIOD_ASYNC_ID_PREFIX.$this->id : $this->async_job_id;
-		$async->cancel_timer($job_id);	// we delete it in case a job already exists
+		$async->cancel_timer($job_id); // we delete it in case a job already exists
 		return $async->set_timer($time, $job_id, __CLASS__.'::run_periodic_job', $this->as_array(), $this->creator);
 	}
 
@@ -1110,7 +1109,7 @@ 
 	{
 		$async = new Api\Asyncservice();
 		$job_id = empty($this->async_job_id) ? self::PERIOD_ASYNC_ID_PREFIX.$this->id : $this->async_job_id;
-		$async->cancel_timer($job_id);	// we delete it in case a job already exists
+		$async->cancel_timer($job_id); // we delete it in case a job already exists
 	}
 
 	/**
@@ -1127,9 +1126,9 @@ 
 		$single = $cmd->as_array();
 		$single['parent'] = $single['id'];
 		$single = array_diff_key($single, array(
-			'id','uid',
-			'created','modified','modifier',
-			'async_job_id','rrule','scheduled',
+			'id', 'uid',
+			'created', 'modified', 'modifier',
+			'async_job_id', 'rrule', 'scheduled',
 			'status', 'set', 'old'
 		));
 
@@ -1156,9 +1155,9 @@ 
 		admin_cmd::_instanciate_remote();
 
 		$sites = array(lang('local'));
-		if (($remote = admin_cmd::$remote->query_list('remote_name','remote_id')))
+		if (($remote = admin_cmd::$remote->query_list('remote_name', 'remote_id')))
 		{
-			$sites = array_merge($sites,$remote);
+			$sites = array_merge($sites, $remote);
 		}
 		return $sites;
 	}
@@ -1171,11 +1170,11 @@ 
 	 * @param array &$readonlys
 	 * @return int
 	 */
-	static function get_remotes($query,&$rows,&$readonlys)
+	static function get_remotes($query, &$rows, &$readonlys)
 	{
 		admin_cmd::_instanciate_remote();
 
-		return admin_cmd::$remote->get_rows($query,$rows,$readonlys);
+		return admin_cmd::$remote->get_rows($query, $rows, $readonlys);
 	}
 
 	/**
@@ -1203,7 +1202,7 @@ 
 
 		if ($data['install_id'] && $data['config_passwd'])	// calculate hash
 		{
-			$data['remote_hash'] = self::remote_hash($data['install_id'],$data['config_passwd']);
+			$data['remote_hash'] = self::remote_hash($data['install_id'], $data['config_passwd']);
 		}
 		elseif (!$data['remote_hash'] && !($data['install_id'] && $data['config_passwd']))
 		{
@@ -1215,12 +1214,12 @@ 
 		// check if a unique key constrain would be violated by saving the entry
 		if (($num = admin_cmd::$remote->not_unique()))
 		{
-			$col = admin_cmd::$remote->table_def['uc'][$num-1];	// $num is 1 based!
-			throw new egw_exception_db_not_unique(lang('Value for column %1 is not unique!',$this->table_name.'.'.$col),$num);
+			$col = admin_cmd::$remote->table_def['uc'][$num - 1]; // $num is 1 based!
+			throw new egw_exception_db_not_unique(lang('Value for column %1 is not unique!', $this->table_name.'.'.$col), $num);
 		}
 		if (admin_cmd::$remote->save() != 0)
 		{
-			throw new Api\Db\Exception(lang('Error saving to db:').' '.$this->sql->db->Error.' ('.$this->sql->db->Errno.')',$this->sql->db->Errno);
+			throw new Api\Db\Exception(lang('Error saving to db:').' '.$this->sql->db->Error.' ('.$this->sql->db->Errno.')', $this->sql->db->Errno);
 		}
 		return admin_cmd::$remote->data['remote_id'];
 	}
@@ -1232,11 +1231,11 @@ 
 	 * @param string $config_passwd
 	 * @return string 32char md5 hash
 	 */
-	static function remote_hash($install_id,$config_passwd)
+	static function remote_hash($install_id, $config_passwd)
 	{
 		if (empty($config_passwd) || !self::is_md5($install_id))
 		{
-			throw new Api\Exception\WrongParameter(empty($config_passwd)?'Empty Api\Config password':'install_id no md5 hash');
+			throw new Api\Exception\WrongParameter(empty($config_passwd) ? 'Empty Api\Config password' : 'install_id no md5 hash');
 		}
 		if (!self::is_md5($config_passwd)) $config_passwd = md5($config_passwd);
 
@@ -1266,7 +1265,7 @@ 
 	 */
 	static function is_md5($str)
 	{
-		return preg_match('/^[0-9a-f]{32}$/',$str);
+		return preg_match('/^[0-9a-f]{32}$/', $str);
 	}
 
 	/**
@@ -1283,26 +1282,26 @@ 
 	 * @param string $config_passwd of the current domain
 	 * @throws Api\Exception\NoPermission
 	 */
-	function check_remote_access($secret,$config_passwd)
+	function check_remote_access($secret, $config_passwd)
 	{
 		// as a security measure remote administration need to be enabled under Admin > Site configuration
-		list(,$remote_admin_install_id) = explode('-',$this->uid);
-		$allowed_remote_admin_ids = $GLOBALS['egw_info']['server']['allow_remote_admin'] ? explode(',',$GLOBALS['egw_info']['server']['allow_remote_admin']) : array();
+		list(,$remote_admin_install_id) = explode('-', $this->uid);
+		$allowed_remote_admin_ids = $GLOBALS['egw_info']['server']['allow_remote_admin'] ? explode(',', $GLOBALS['egw_info']['server']['allow_remote_admin']) : array();
 
 		// to authenticate with the installation we use a secret, which is a md5 hash build from the uid
 		// of the command (to not allow to send new commands with an earsdroped secret) and the md5 hash
 		// of the md5 hash of the Api\Config password and the install_id (egw_admin_remote.remote_hash)
-		if (is_null($config_passwd) || is_numeric($this->uid) || !in_array($remote_admin_install_id,$allowed_remote_admin_ids) ||
-			$secret != ($md5=md5($this->uid.$this->remote_hash($GLOBALS['egw_info']['server']['install_id'],$config_passwd))))
+		if (is_null($config_passwd) || is_numeric($this->uid) || !in_array($remote_admin_install_id, $allowed_remote_admin_ids) ||
+			$secret != ($md5 = md5($this->uid.$this->remote_hash($GLOBALS['egw_info']['server']['install_id'], $config_passwd))))
 		{
 			//die("secret='$secret' != '$md5', is_null($config_passwd)=".is_null($config_passwd).", uid=$this->uid, remote_install_id=$remote_admin_install_id, allowed: ".implode(', ',$allowed_remote_admin_ids));
 			unset($md5);
 			$msg = lang('Permission denied!');
-			if (!in_array($remote_admin_install_id,$allowed_remote_admin_ids))
+			if (!in_array($remote_admin_install_id, $allowed_remote_admin_ids))
 			{
 				$msg .= "\n".lang('Remote administration need to be enabled in the remote instance under Admin > Site configuration!');
 			}
-			throw new Api\Exception\NoPermission($msg,0);
+			throw new Api\Exception\NoPermission($msg, 0);
 		}
 	}
 
@@ -1312,7 +1311,7 @@ 
 	 * @param int $len =16
 	 * @return string
 	 */
-	static function randomstring($len=16)
+	static function randomstring($len = 16)
 	{
 		return Api\Auth::randomstring($len);
 	}
@@ -1334,7 +1333,7 @@ 
 	 */
 	protected function get_etemplate()
 	{
-		static $tpl = null;	// some caching to not instanciate it twice
+		static $tpl = null; // some caching to not instanciate it twice
 
 		if (!isset($tpl))
 		{
@@ -1364,7 +1363,7 @@ 
 		{
 			$tpl->run(function($cname, $expand, $widget) use (&$labels, &$label)
 			{
-				switch($widget->type)
+				switch ($widget->type)
 				{
 					// remember label from last description widget
 					case 'description':
@@ -1407,7 +1406,7 @@ 
 		{
 			$tpl->run(function($cname, $expand, $widget) use (&$widgets, &$last_select, $selectboxes)
 			{
-				switch($widget->type)
+				switch ($widget->type)
 				{
 					// ignore non input-widgets
 					case 'hbox': case 'vbox': case 'box': case 'groupbox':
@@ -1459,12 +1458,12 @@ 
 	 */
 	public function get_result()
 	{
-		if($this->result)
+		if ($this->result)
 		{
 			return is_array($this->result) ? implode("\n", $this->result) : $this->result;
 		}
 		return lang("Command was run %1 on %2",
-				static::$stati[ $this->status ],
+				static::$stati[$this->status],
 				Api\DateTime::to($this->created));
 	}
 }