EGroupware / egroupware

filemanager/inc/class.filemanager_ui.inc.php

Braces +142 added lines, -36 removed lines

@@ -88,7 +88,10 @@ 
 			// search for plugins with additional filemanager views
 			foreach(Api\Hooks::process('filemanager_views') as $views)
 			{
-				if (is_array($views)) static::$views += $views;
+				if (is_array($views))
+				{
+					static::$views += $views;
+				}
 			}
 			static::$views_init = true;
 		}
@@ -405,7 +408,10 @@ 
 			$content['nm']['home_dir'] = static::get_home_dir();
 			$content['nm']['view'] = $GLOBALS['egw_info']['user']['preferences']['filemanager']['nm_view'];
 
-			if (isset($_GET['msg'])) $msg = $_GET['msg'];
+			if (isset($_GET['msg']))
+			{
+				$msg = $_GET['msg'];
+			}
 
 			// Blank favorite set via GET needs special handling for path
 			if (isset($_GET['favorite']) && $_GET['favorite'] == 'blank')
@@ -439,7 +445,10 @@ 
 				// reset lettersearch as it confuses users (they think the dir is empty)
 				$content['nm']['searchletter'] = false;
 				// switch recusive display off
-				if (!$content['nm']['filter']) $content['nm']['filter'] = '';
+				if (!$content['nm']['filter'])
+				{
+					$content['nm']['filter'] = '';
+				}
 			}
 		}
 		$view = static::get_view();
@@ -488,37 +497,70 @@ 
 	{
 		$tpl = new Etemplate('filemanager.index');
 
-		if($msg) Framework::message($msg);
+		if($msg)
+		{
+			Framework::message($msg);
+		}
 
 		if (($content['nm']['action'] || $content['nm']['rows']) && (empty($content['button']) || !isset($content['button'])))
 		{
 			if ($content['nm']['action'])
 			{
 				$msg = static::action($content['nm']['action'],$content['nm']['selected'],$content['nm']['path']);
-				if($msg) Framework::message($msg);
+				if($msg)
+				{
+					Framework::message($msg);
+				}
 
 				// clean up after action
 				unset($content['nm']['selected']);
 				// reset any occasion where action may be stored, as it may be ressurected out of the helpers by etemplate, which is quite unconvenient in case of action delete
-				if (isset($content['nm']['action'])) unset($content['nm']['action']);
-				if (isset($content['nm']['nm_action'])) unset($content['nm']['nm_action']);
-				if (isset($content['nm_action'])) unset($content['nm_action']);
+				if (isset($content['nm']['action']))
+				{
+					unset($content['nm']['action']);
+				}
+				if (isset($content['nm']['nm_action']))
+				{
+					unset($content['nm']['nm_action']);
+				}
+				if (isset($content['nm_action']))
+				{
+					unset($content['nm_action']);
+				}
 				// we dont use ['nm']['rows']['delete'], so unset it, if it is present
-				if (isset($content['nm']['rows']['delete'])) unset($content['nm']['rows']['delete']);
+				if (isset($content['nm']['rows']['delete']))
+				{
+					unset($content['nm']['rows']['delete']);
+				}
 			}
 			elseif($content['nm']['rows']['delete'])
 			{
 				$msg = static::action('delete',array_keys($content['nm']['rows']['delete']),$content['nm']['path']);
-				if($msg) Framework::message($msg);
+				if($msg)
+				{
+					Framework::message($msg);
+				}
 
 				// clean up after action
 				unset($content['nm']['rows']['delete']);
 				// reset any occasion where action may be stored, as we use ['nm']['rows']['delete'] anyhow
 				// we clean this up, as it may be ressurected out of the helpers by etemplate, which is quite unconvenient in case of action delete
-				if (isset($content['nm']['action'])) unset($content['nm']['action']);
-				if (isset($content['nm']['nm_action'])) unset($content['nm']['nm_action']);
-				if (isset($content['nm_action'])) unset($content['nm_action']);
-				if (isset($content['nm']['selected'])) unset($content['nm']['selected']);
+				if (isset($content['nm']['action']))
+				{
+					unset($content['nm']['action']);
+				}
+				if (isset($content['nm']['nm_action']))
+				{
+					unset($content['nm']['nm_action']);
+				}
+				if (isset($content['nm_action']))
+				{
+					unset($content['nm_action']);
+				}
+				if (isset($content['nm']['selected']))
+				{
+					unset($content['nm']['selected']);
+				}
 			}
 			unset($content['nm']['rows']);
 			Api\Cache::setSession('filemanager', 'index',$content['nm']);
@@ -685,9 +727,12 @@ 
 						foreach(Vfs::find($path) as $p)
 						{
 							$to = $dir.substr($p,$len);
-							if ($to == $p)	// cant copy into itself!
+							if ($to == $p)
+							{
+								// cant copy into itself!
 							{
 								++$errs;
+							}
 								continue;
 							}
 							if (($is_dir = Vfs::is_dir($p)) && Vfs::mkdir($to,null,STREAM_MKDIR_RECURSIVE))
@@ -742,7 +787,10 @@ 
 							continue;
 						}
 					}
-					if ($target[0] != '/') $target = Vfs::concat($dir, $target);
+					if ($target[0] != '/')
+					{
+						$target = Vfs::concat($dir, $target);
+					}
 					if (!Vfs::stat($target))
 					{
 						return lang('Link target %1 not found!', Vfs::decodePath($target));
@@ -785,10 +833,16 @@ 
 				switch($action)
 				{
 					case 'document':
-						if (!$settings) $settings = $GLOBALS['egw_info']['user']['preferences']['filemanager']['default_document'];
+						if (!$settings)
+						{
+							$settings = $GLOBALS['egw_info']['user']['preferences']['filemanager']['default_document'];
+						}
 						$document_merge = new filemanager_merge(Vfs::decodePath($dir));
 						$msg = $document_merge->download($settings, $selected, '', $GLOBALS['egw_info']['user']['preferences']['filemanager']['document_dir']);
-						if($msg) return $msg;
+						if($msg)
+						{
+							return $msg;
+						}
 						$errs = count($selected);
 						return false;
 				}
@@ -825,7 +879,9 @@ 
 				unset($selected[$key]);
 			}
 		}
-		if ($selected)	// somethings left to delete
+		if ($selected)
+		{
+			// somethings left to delete
 		{
 			// some precaution to never allow to (recursivly) remove /, /apps or /home
 			foreach((array)$selected as $path)
@@ -833,6 +889,7 @@ 
 				if (Vfs::isProtectedDir($path))
 				{
 					$errs++;
+		}
 					return lang("Cautiously rejecting to remove folder '%1'!",Vfs::decodePath($path));
 				}
 			}
@@ -881,7 +938,10 @@ 
 			Api\Cache::setSession('filemanager', 'index',
 				array_diff_key ($query, array_flip(array('rows','actions','action_links','placeholder_actions'))));
 		}
-		if(!$query['path']) $query['path'] = static::get_home_dir();
+		if(!$query['path'])
+		{
+			$query['path'] = static::get_home_dir();
+		}
 
 		// Change template to match selected view
 		if($query['view'])
@@ -939,7 +999,10 @@ 
 		$filter = $query['filter'] === '' ? 1 : $query['filter'];
 
 		$maxdepth = $filter && $filter != 4 ? (int)(boolean)$filter : null;
-		if($filter == 5) $maxdepth = 2;
+		if($filter == 5)
+		{
+			$maxdepth = 2;
+		}
 		$n = 0;
 		$vfs_options = array(
 			'mindepth' => 1,
@@ -1002,10 +1065,16 @@ 
 			{
 				unset($row);	// fixes a weird problem with php5.1, does NOT happen with php5.2
 				$row =& $rows[$path2n[$path]];
-				if ( !is_array($props) ) continue;
+				if ( !is_array($props) )
+				{
+					continue;
+				}
 				foreach($props as $prop)
 				{
-					if (!$all_cfs && $prop['name'][0] == '#' && !in_array($prop['name'],$cols_to_show)) continue;
+					if (!$all_cfs && $prop['name'][0] == '#' && !in_array($prop['name'],$cols_to_show))
+					{
+						continue;
+					}
 					$row[$prop['name']] = strlen($prop['val']) < 64 ? $prop['val'] : substr($prop['val'],0,64).' ...';
 				}
 			}
@@ -1130,7 +1199,10 @@ 
 							if (!($dir = Vfs::dirname($path)))
 							{
 								$msg .= lang('File or directory not found!')." Vfs::dirname('$path')===false";
-								if ($button == 'save') $button = 'apply';
+								if ($button == 'save')
+								{
+									$button = 'apply';
+								}
 								continue;
 							}
 							$to = Vfs::concat($dir, $content['name']);
@@ -1139,7 +1211,10 @@ 
 								$tpl->set_validation_error('name',lang("There's already a file with that name!").'<br />'.
 									lang('To overwrite the existing file store again.',lang($button)));
 								$content['confirm_overwrite'] = $to;
-								if ($button == 'save') $button = 'apply';
+								if ($button == 'save')
+								{
+									$button = 'apply';
+								}
 								continue;
 							}
 							if (Vfs::rename($path,$to))
@@ -1212,13 +1287,19 @@ 
 								}
 								if ($ok && !$failed)
 								{
-									if(!$perm_changed++) $msg .= lang('Permissions of %1 changed.',$path.' '.lang('and all it\'s childeren'));
+									if(!$perm_changed++)
+									{
+										$msg .= lang('Permissions of %1 changed.',$path.' '.lang('and all it\'s childeren'));
+									}
 									$content['old'][$name] = $content[$name];
 								}
 								elseif($failed)
 								{
-									if(!$perm_failed++) $msg .= lang('Failed to change permissions of %1!',$path.lang('and all it\'s childeren').
+									if(!$perm_failed++)
+									{
+										$msg .= lang('Failed to change permissions of %1!',$path.lang('and all it\'s childeren').
 										($ok ? ' ('.lang('%1 failed, %2 succeded',$failed,$ok).')' : ''));
+									}
 								}
 							}
 							elseif (call_user_func_array($cmd,array($path,$value)))
@@ -1271,7 +1352,10 @@ 
 				}
 			}
 			Framework::refresh_opener($msg, 'filemanager', $refresh_path ? $refresh_path : $path, 'edit', null, '&path=[^&]*');
-			if ($button == 'save') Framework::window_close();
+			if ($button == 'save')
+			{
+				Framework::window_close();
+			}
 		}
 		if ($content['is_link'] && !Vfs::stat($path))
 		{
@@ -1281,13 +1365,19 @@ 
 		$content['icon'] = Vfs::mime_icon($content['mime']);
 		$content['msg'] = $msg;
 
-		if (($readonlys['uid'] = !Vfs::$is_root) && !$content['uid']) $content['ro_uid_root'] = 'root';
+		if (($readonlys['uid'] = !Vfs::$is_root) && !$content['uid'])
+		{
+			$content['ro_uid_root'] = 'root';
+		}
 		// only owner can change group & perms
 		if (($readonlys['gid'] = !$content['is_owner'] ||
 			Vfs::parse_url(Vfs::resolve_url($content['path']),PHP_URL_SCHEME) == 'oldvfs') ||// no uid, gid or perms in oldvfs
 				 !Vfs::is_writable($path))
 		{
-			if (!$content['gid']) $content['ro_gid_root'] = 'root';
+			if (!$content['gid'])
+			{
+				$content['ro_gid_root'] = 'root';
+			}
 			foreach($content['perms'] as $name => $value)
 			{
 				$readonlys['perms['.$name.']'] = true;
@@ -1322,9 +1412,13 @@ 
 				0 => lang('No access'),
 			);
 			if(($content['eacl'] = Vfs::get_eacl($content['path'])) !== false &&	// backend supports eacl
-				$GLOBALS['egw_info']['user']['account_id'] == Vfs::$user)	// leave eACL tab disabled for sharing
+				$GLOBALS['egw_info']['user']['account_id'] == Vfs::$user)
 			{
-				unset($readonlys['tabs']['filemanager.file.eacl']);	// --> switch the tab on again
+				// leave eACL tab disabled for sharing
+			{
+				unset($readonlys['tabs']['filemanager.file.eacl']);
+			}
+			// --> switch the tab on again
 				foreach($content['eacl'] as &$eacl)
 				{
 					$eacl['path'] = rtrim(Vfs::parse_url($eacl['path'],PHP_URL_PATH),'/');
@@ -1381,10 +1475,13 @@ 
 				'comment' => (string)$content['comment'],
 				'mergeapp' => $content['mergeapp']
 			);
-			if ($cfs) foreach($cfs as $name => $data)
+			if ($cfs)
+			{
+				foreach($cfs as $name => $data)
 			{
 				$preserve['old']['#'.$name] = (string)$content['#'.$name];
 			}
+			}
 		}
 		if (Vfs::$is_root)
 		{
@@ -1411,7 +1508,10 @@ 
 			$tpl->setElementAttribute('tabs', 'add_tabs', true);
 
 			$tabs =& $tpl->getElementAttribute('tabs','tabs');
-			if (true) $tabs = array();
+			if (true)
+			{
+				$tabs = array();
+			}
 
 			foreach(isset($extra_tabs[0]) ? $extra_tabs : array($extra_tabs) as $extra_tab)
 			{
@@ -1490,7 +1590,10 @@ 
 			'files' => 0,
 		);
 
-		if (!isset($dir)) $dir = array_pop($selected);
+		if (!isset($dir))
+		{
+			$dir = array_pop($selected);
+		}
 
 		switch($action)
 		{
@@ -1588,7 +1691,10 @@ 
 					$target=Vfs::concat($dir,Vfs::encodePathComponent($file['name']));
 					if (Vfs::file_exists($target) && $app_dir)
 					{
-						if (!Vfs::file_exists($app_dir)) Vfs::mkdir($app_dir);
+						if (!Vfs::file_exists($app_dir))
+						{
+							Vfs::mkdir($app_dir);
+						}
 						error_log("Symlinking $target to $app_dir");
 						Vfs::symlink($target, Vfs::concat($app_dir,Vfs::encodePathComponent($file['name'])));
 					}

Spacing +203 added lines, -205 removed lines

@@ -80,12 +80,12 @@ 
 		if (!static::$views_init)
 		{
 			// translate our labels
-			foreach(static::$views as &$label)
+			foreach (static::$views as &$label)
 			{
 				$label = lang($label);
 			}
 			// search for plugins with additional filemanager views
-			foreach(Api\Hooks::process('filemanager_views') as $views)
+			foreach (Api\Hooks::process('filemanager_views') as $views)
 			{
 				if (is_array($views)) static::$views += $views;
 			}
@@ -101,7 +101,7 @@ 
 	 */
 	public static function get_view()
 	{
-		$view =& Api\Cache::getSession('filemanager', 'view');
+		$view = & Api\Cache::getSession('filemanager', 'view');
 		if (isset($_GET['view']))
 		{
 			$view = $_GET['view'];
@@ -125,7 +125,7 @@ 
 			'open' => array(
 				'caption' => lang('Open'),
 				'icon' => '',
-				'group' => $group=1,
+				'group' => $group = 1,
 				'allowOnMultiple' => false,
 				'onExecute' => 'javaScript:app.filemanager.open',
 				'default' => true
@@ -150,7 +150,7 @@ 
 				'caption' => lang('Edit settings'),
 				'group' => $group,
 				'allowOnMultiple' => false,
-				'onExecute' => Api\Header\UserAgent::mobile()?'javaScript:app.filemanager.viewEntry':'javaScript:app.filemanager.editprefs',
+				'onExecute' => Api\Header\UserAgent::mobile() ? 'javaScript:app.filemanager.viewEntry' : 'javaScript:app.filemanager.editprefs',
 				'mobileViewTemplate' => 'file?'.filemtime(Api\Etemplate\Widget\Template::rel2path('/filemanager/templates/mobile/file.xet'))
 			),
 			'mkdir' => array(
@@ -209,7 +209,7 @@ 
 			),
 			// DRAG and DROP events
 			'file_drag' => array(
-				'dragType' => array('file','link'),
+				'dragType' => array('file', 'link'),
 				'type' => 'drag',
 				'onExecute' => 'javaScript:app.filemanager.drag'
 			),
@@ -248,7 +248,7 @@ 
 		}
 		else
 		{
-			foreach(Vfs\Sharing::$modes as $mode => $data)
+			foreach (Vfs\Sharing::$modes as $mode => $data)
 			{
 				$actions['mail']['children']['mail_'.$mode] = array(
 					'caption' => $data['label'],
@@ -259,12 +259,12 @@ 
 			}
 		}
 		// This would be done automatically, but we're overriding
-		foreach($actions as $action_id => $action)
+		foreach ($actions as $action_id => $action)
 		{
-			if($action['type'] == 'drop' && $action['caption'])
+			if ($action['type'] == 'drop' && $action['caption'])
 			{
 				$action['type'] = 'popup';
-				if($action['acceptedTypes'] == 'file')
+				if ($action['acceptedTypes'] == 'file')
 				{
 					$action['enabled'] = 'javaScript:app.filemanager.paste_enabled';
 				}
@@ -273,7 +273,7 @@ 
 		}
 
 		// Anonymous users have limited actions
-		if(self::is_anonymous($GLOBALS['egw_info']['user']['account_id']))
+		if (self::is_anonymous($GLOBALS['egw_info']['user']['account_id']))
 		{
 			self::restrict_anonymous_actions($actions);
 		}
@@ -290,10 +290,10 @@ 
 	 *
 	 * @return string merge application or NULL if no property found
 	 */
-	private static function get_mergeapp($path, $scope='self')
+	private static function get_mergeapp($path, $scope = 'self')
 	{
 		$app = null;
-		switch($scope)
+		switch ($scope)
 		{
 			case 'self':
 				$props = Vfs::propfind($path, static::$merge_prop_namespace);
@@ -302,10 +302,10 @@ 
 			case 'parents':
 				// search for props in parent directories
 				$currentpath = $path;
-				while($dir = Vfs::dirname($currentpath))
+				while ($dir = Vfs::dirname($currentpath))
 				{
 					$props = Vfs::propfind($dir, static::$merge_prop_namespace);
-					if(!empty($props))
+					if (!empty($props))
 					{
 						// found prop in parent directory
 						return $app = $props[0]['val'];
@@ -324,7 +324,7 @@ 
 	 * @param array $content
 	 * @param string $msg
 	 */
-	function index(array $content=null,$msg=null)
+	function index(array $content = null, $msg = null)
 	{
 		if (!is_array($content))
 		{
@@ -334,16 +334,16 @@ 
 			if (!is_array($content['nm']))
 			{
 				$content['nm'] = array(
-					'get_rows'       =>	'filemanager.filemanager_ui.get_rows',	// I  method/callback to request the data for the rows eg. 'notes.bo.get_rows'
-					'filter'         => '',	// current dir only
-					'no_filter2'     => True,	// I  disable the 2. filter (params are the same as for filter)
-					'no_cat'         => True,	// I  disable the cat-selectbox
-					'lettersearch'   => True,	// I  show a lettersearch
-					'searchletter'   =>	false,	// I0 active letter of the lettersearch or false for [all]
-					'start'          =>	0,		// IO position in list
-					'order'          =>	'name',	// IO name of the column to sort after (optional for the sortheaders)
-					'sort'           =>	'ASC',	// IO direction of the sort: 'ASC' or 'DESC'
-					'default_cols'   => '!comment,ctime',	// I  columns to use if there's no user or default pref (! as first char uses all but the named columns), default all columns
+					'get_rows'       =>	'filemanager.filemanager_ui.get_rows', // I  method/callback to request the data for the rows eg. 'notes.bo.get_rows'
+					'filter'         => '', // current dir only
+					'no_filter2'     => True, // I  disable the 2. filter (params are the same as for filter)
+					'no_cat'         => True, // I  disable the cat-selectbox
+					'lettersearch'   => True, // I  show a lettersearch
+					'searchletter'   =>	false, // I0 active letter of the lettersearch or false for [all]
+					'start'          =>	0, // IO position in list
+					'order'          =>	'name', // IO name of the column to sort after (optional for the sortheaders)
+					'sort'           =>	'ASC', // IO direction of the sort: 'ASC' or 'DESC'
+					'default_cols'   => '!comment,ctime', // I  columns to use if there's no user or default pref (! as first char uses all but the named columns), default all columns
 					'csv_fields'     =>	false, // I  false=disable csv export, true or unset=enable it with auto-detected fieldnames,
 									//or array with name=>label or name=>array('label'=>label,'type'=>type) pairs (type is a eT widget-type)
 					'row_id'         => 'path',
@@ -351,7 +351,7 @@ 
 					'parent_id'      => 'dir',
 					'is_parent'      => 'is_dir',
 					'favorites'      => true,
-					'placeholder_actions' => array('mkdir','paste','file_drop_mail','file_drop_move','file_drop_copy','file_drop_symlink')
+					'placeholder_actions' => array('mkdir', 'paste', 'file_drop_mail', 'file_drop_move', 'file_drop_copy', 'file_drop_symlink')
 				);
 				$content['nm']['path'] = static::get_home_dir();
 			}
@@ -373,7 +373,7 @@ 
 			}
 			if (isset($_GET['path']) && ($path = $_GET['path']))
 			{
-				switch($path)
+				switch ($path)
 				{
 					case '..':
 						$path = Vfs::dirname($content['nm']['path']);
@@ -382,7 +382,7 @@ 
 						$path = static::get_home_dir();
 						break;
 				}
-				if ($path && $path[0] == '/' && Vfs::stat($path,true) && Vfs::is_dir($path) && Vfs::check_access($path,Vfs::READABLE))
+				if ($path && $path[0] == '/' && Vfs::stat($path, true) && Vfs::is_dir($path) && Vfs::check_access($path, Vfs::READABLE))
 				{
 					$content['nm']['path'] = $path;
 				}
@@ -398,7 +398,7 @@ 
 		}
 		$view = static::get_view();
 
-		call_user_func($view,$content,$msg);
+		call_user_func($view, $content, $msg);
 	}
 
 	/**
@@ -411,7 +411,7 @@ 
 	 * @param boolean &$is_setup=null on return true if authenticated user is setup config user, false otherwise
 	 * @return boolean true is root user given, false otherwise (including logout / empty $user)
 	 */
-	protected function sudo($user='',$password=null,&$is_setup=null)
+	protected function sudo($user = '', $password = null, &$is_setup = null)
 	{
 		if (!$user)
 		{
@@ -420,15 +420,15 @@ 
 		else
 		{
 			// config user & password
-			$is_setup = Api\Session::user_pw_hash($user,$password) === $GLOBALS['egw_info']['server']['config_hash'];
+			$is_setup = Api\Session::user_pw_hash($user, $password) === $GLOBALS['egw_info']['server']['config_hash'];
 			// or vfs root user from setup >> configuration
-			$is_root = $is_setup ||	$GLOBALS['egw_info']['server']['vfs_root_user'] &&
-				in_array($user,preg_split('/, */',$GLOBALS['egw_info']['server']['vfs_root_user'])) &&
+			$is_root = $is_setup || $GLOBALS['egw_info']['server']['vfs_root_user'] &&
+				in_array($user, preg_split('/, */', $GLOBALS['egw_info']['server']['vfs_root_user'])) &&
 				$GLOBALS['egw']->auth->authenticate($user, $password, 'text');
 		}
 		//error_log(__METHOD__."('$user','$password',$is_setup) user_pw_hash(...)='".Api\Session::user_pw_hash($user,$password)."', config_hash='{$GLOBALS['egw_info']['server']['config_hash']}' --> returning ".array2string($is_root));
-		Api\Cache::setSession('filemanager', 'is_setup',$is_setup);
-		Api\Cache::setSession('filemanager', 'is_root',Vfs::$is_root = $is_root);
+		Api\Cache::setSession('filemanager', 'is_setup', $is_setup);
+		Api\Cache::setSession('filemanager', 'is_root', Vfs::$is_root = $is_root);
 		return Vfs::$is_root;
 	}
 
@@ -438,18 +438,18 @@ 
 	 * @param array $content
 	 * @param string $msg
 	 */
-	function listview(array $content=null,$msg=null)
+	function listview(array $content = null, $msg = null)
 	{
 		$tpl = new Etemplate('filemanager.index');
 
-		if($msg) Framework::message($msg);
+		if ($msg) Framework::message($msg);
 
 		if (($content['nm']['action'] || $content['nm']['rows']) && (empty($content['button']) || !isset($content['button'])))
 		{
 			if ($content['nm']['action'])
 			{
-				$msg = static::action($content['nm']['action'],$content['nm']['selected'],$content['nm']['path']);
-				if($msg) Framework::message($msg);
+				$msg = static::action($content['nm']['action'], $content['nm']['selected'], $content['nm']['path']);
+				if ($msg) Framework::message($msg);
 
 				// clean up after action
 				unset($content['nm']['selected']);
@@ -460,10 +460,10 @@ 
 				// we dont use ['nm']['rows']['delete'], so unset it, if it is present
 				if (isset($content['nm']['rows']['delete'])) unset($content['nm']['rows']['delete']);
 			}
-			elseif($content['nm']['rows']['delete'])
+			elseif ($content['nm']['rows']['delete'])
 			{
-				$msg = static::action('delete',array_keys($content['nm']['rows']['delete']),$content['nm']['path']);
-				if($msg) Framework::message($msg);
+				$msg = static::action('delete', array_keys($content['nm']['rows']['delete']), $content['nm']['path']);
+				if ($msg) Framework::message($msg);
 
 				// clean up after action
 				unset($content['nm']['rows']['delete']);
@@ -475,11 +475,11 @@ 
 				if (isset($content['nm']['selected'])) unset($content['nm']['selected']);
 			}
 			unset($content['nm']['rows']);
-			Api\Cache::setSession('filemanager', 'index',$content['nm']);
+			Api\Cache::setSession('filemanager', 'index', $content['nm']);
 		}
 
 		// be tolerant with (in previous versions) not correct urlencoded pathes
-		if ($content['nm']['path'][0] == '/' && !Vfs::stat($content['nm']['path'],true) && Vfs::stat(urldecode($content['nm']['path'])))
+		if ($content['nm']['path'][0] == '/' && !Vfs::stat($content['nm']['path'], true) && Vfs::stat(urldecode($content['nm']['path'])))
 		{
 			$content['nm']['path'] = urldecode($content['nm']['path']);
 		}
@@ -490,22 +490,22 @@ 
 				list($button) = each($content['button']);
 				unset($content['button']);
 			}
-			switch($button)
+			switch ($button)
 			{
 				case 'upload':
 					if (!$content['upload'])
 					{
-						Framework::message(lang('You need to select some files first!'),'error');
+						Framework::message(lang('You need to select some files first!'), 'error');
 						break;
 					}
 					$upload_success = $upload_failure = array();
-					foreach(isset($content['upload'][0]) ? $content['upload'] : array($content['upload']) as $upload)
+					foreach (isset($content['upload'][0]) ? $content['upload'] : array($content['upload']) as $upload)
 					{
 						// encode chars which special meaning in url/vfs (some like / get removed!)
-						$to = Vfs::concat($content['nm']['path'],Vfs::encodePathComponent($upload['name']));
+						$to = Vfs::concat($content['nm']['path'], Vfs::encodePathComponent($upload['name']));
 						if ($upload &&
 							(Vfs::is_writable($content['nm']['path']) || Vfs::is_writable($to)) &&
-							copy($upload['tmp_name'],Vfs::PREFIX.$to))
+							copy($upload['tmp_name'], Vfs::PREFIX.$to))
 						{
 							$upload_success[] = $upload['name'];
 						}
@@ -517,12 +517,11 @@ 
 					$content['nm']['msg'] = '';
 					if ($upload_success)
 					{
-						Framework::message( count($upload_success) == 1 && !$upload_failure ? lang('File successful uploaded.') :
-							lang('%1 successful uploaded.',implode(', ',$upload_success)));
+						Framework::message(count($upload_success) == 1 && !$upload_failure ? lang('File successful uploaded.') : lang('%1 successful uploaded.', implode(', ', $upload_success)));
 					}
 					if ($upload_failure)
 					{
-						Framework::message(lang('Error uploading file!')."\n".etemplate::max_upload_size_message(),'error');
+						Framework::message(lang('Error uploading file!')."\n".etemplate::max_upload_size_message(), 'error');
 					}
 					break;
 			}
@@ -543,16 +542,16 @@ 
 			$tpl->setElementAttribute('nm[buttons][upload]', 'drop_target', 'popupMainDiv');
 		}
 		// Set view button to match current settings
-		if($content['nm']['view'] == 'tile')
+		if ($content['nm']['view'] == 'tile')
 		{
-			$tpl->setElementAttribute('nm[button][change_view]','statustext',lang('List view'));
-			$tpl->setElementAttribute('nm[button][change_view]','image','list_row');
+			$tpl->setElementAttribute('nm[button][change_view]', 'statustext', lang('List view'));
+			$tpl->setElementAttribute('nm[button][change_view]', 'image', 'list_row');
 		}
 		// if initial load is done via GET request (idots template or share.php)
 		// get_rows cant call app.filemanager.set_readonly, so we need to do that here
 		$content['initial_path_readonly'] = !Vfs::is_writable($content['nm']['path']);
 
-		$tpl->exec('filemanager.filemanager_ui.index',$content,$sel_options,$readonlys,array('nm' => $content['nm']));
+		$tpl->exec('filemanager.filemanager_ui.index', $content, $sel_options, $readonlys, array('nm' => $content['nm']));
 	}
 
 	/**
@@ -588,7 +587,7 @@ 
 	 * @param int &$files=null on return number of files deleted
 	 * @return string success or failure message displayed to the user
 	 */
-	static public function action($action,$selected,$dir=null,&$errs=null,&$files=null,&$dirs=null)
+	static public function action($action, $selected, $dir = null, &$errs = null, &$files = null, &$dirs = null)
 	{
 		if (!count($selected))
 		{
@@ -596,20 +595,20 @@ 
 		}
 		$errs = $dirs = $files = 0;
 
-		switch($action)
+		switch ($action)
 		{
 
 			case 'delete':
-				return static::do_delete($selected,$errs,$files,$dirs);
+				return static::do_delete($selected, $errs, $files, $dirs);
 
 			case 'mail':
 			case 'copy':
-				foreach($selected as $path)
+				foreach ($selected as $path)
 				{
 					if (strpos($path, 'mail::') === 0 && $path = substr($path, 6))
 					{
 						// Support for dropping mail in filemanager - Pass mail back to mail app
-						if(ExecMethod2('mail.mail_ui.vfsSaveMessage', $path, $dir, false))
+						if (ExecMethod2('mail.mail_ui.vfsSaveMessage', $path, $dir, false))
 						{
 							++$files;
 						}
@@ -620,8 +619,8 @@ 
 					}
 					elseif (!Vfs::is_dir($path))
 					{
-						$to = Vfs::concat($dir,Vfs::basename($path));
-						if ($path != $to && Vfs::copy($path,$to))
+						$to = Vfs::concat($dir, Vfs::basename($path));
+						if ($path != $to && Vfs::copy($path, $to))
 						{
 							++$files;
 						}
@@ -633,19 +632,19 @@ 
 					else
 					{
 						$len = strlen(dirname($path));
-						foreach(Vfs::find($path) as $p)
+						foreach (Vfs::find($path) as $p)
 						{
-							$to = $dir.substr($p,$len);
+							$to = $dir.substr($p, $len);
 							if ($to == $p)	// cant copy into itself!
 							{
 								++$errs;
 								continue;
 							}
-							if (($is_dir = Vfs::is_dir($p)) && Vfs::mkdir($to,null,STREAM_MKDIR_RECURSIVE))
+							if (($is_dir = Vfs::is_dir($p)) && Vfs::mkdir($to, null, STREAM_MKDIR_RECURSIVE))
 							{
 								++$dirs;
 							}
-							elseif(!$is_dir && Vfs::copy($p,$to))
+							elseif (!$is_dir && Vfs::copy($p, $to))
 							{
 								++$files;
 							}
@@ -658,15 +657,15 @@ 
 				}
 				if ($errs)
 				{
-					return lang('%1 errors copying (%2 diretories and %3 files copied)!',$errs,$dirs,$files);
+					return lang('%1 errors copying (%2 diretories and %3 files copied)!', $errs, $dirs, $files);
 				}
-				return $dirs ? lang('%1 directories and %2 files copied.',$dirs,$files) : lang('%1 files copied.',$files);
+				return $dirs ? lang('%1 directories and %2 files copied.', $dirs, $files) : lang('%1 files copied.', $files);
 
 			case 'move':
-				foreach($selected as $path)
+				foreach ($selected as $path)
 				{
-					$to = Vfs::is_dir($dir) || count($selected) > 1 ? Vfs::concat($dir,Vfs::basename($path)) : $dir;
-					if ($path != $to && Vfs::rename($path,$to))
+					$to = Vfs::is_dir($dir) || count($selected) > 1 ? Vfs::concat($dir, Vfs::basename($path)) : $dir;
+					if ($path != $to && Vfs::rename($path, $to))
 					{
 						++$files;
 					}
@@ -677,17 +676,17 @@ 
 				}
 				if ($errs)
 				{
-					return lang('%1 errors moving (%2 files moved)!',$errs,$files);
+					return lang('%1 errors moving (%2 files moved)!', $errs, $files);
 				}
-				return lang('%1 files moved.',$files);
+				return lang('%1 files moved.', $files);
 
 			case 'symlink':	// symlink given files to $dir
-				foreach((array)$selected as $target)
+				foreach ((array)$selected as $target)
 				{
 					$link = Vfs::concat($dir, Vfs::basename($target));
-					if (!Vfs::stat($dir) || ($ok = Vfs::mkdir($dir,0,true)))
+					if (!Vfs::stat($dir) || ($ok = Vfs::mkdir($dir, 0, true)))
 					{
-						if(!$ok)
+						if (!$ok)
 						{
 							$errs++;
 							continue;
@@ -709,15 +708,14 @@ 
 				}
 				if (count((array)$selected) == 1)
 				{
-					return $files ? lang('Symlink to %1 created.', Vfs::decodePath($target)) :
-						lang('Error creating symlink to target %1!', Vfs::decodePath($target));
+					return $files ? lang('Symlink to %1 created.', Vfs::decodePath($target)) : lang('Error creating symlink to target %1!', Vfs::decodePath($target));
 				}
 				$ret = lang('%1 elements linked.', $files);
 				if ($errs)
 				{
-					$ret = lang('%1 errors linking (%2)!',$errs, $ret);
+					$ret = lang('%1 errors linking (%2)!', $errs, $ret);
 				}
-				return $ret;//." Vfs::symlink('$target', '$link')";
+				return $ret; //." Vfs::symlink('$target', '$link')";
 
 			case 'createdir':
 				$dst = Vfs::concat($dir, is_array($selected) ? $selected[0] : $selected);
@@ -733,13 +731,13 @@ 
 
 			default:
 				list($action, $settings) = explode('_', $action, 2);
-				switch($action)
+				switch ($action)
 				{
 					case 'document':
 						if (!$settings) $settings = $GLOBALS['egw_info']['user']['preferences']['filemanager']['default_document'];
 						$document_merge = new filemanager_merge(Vfs::decodePath($dir));
 						$msg = $document_merge->download($settings, $selected, '', $GLOBALS['egw_info']['user']['preferences']['filemanager']['document_dir']);
-						if($msg) return $msg;
+						if ($msg) return $msg;
 						$errs = count($selected);
 						return false;
 				}
@@ -756,12 +754,12 @@ 
 	 * @param int &$files=null on return number of files deleted
 	 * @return string
 	 */
-	public static function do_delete(array $selected, &$errs=null, &$dirs=null, &$files=null)
+	public static function do_delete(array $selected, &$errs = null, &$dirs = null, &$files = null)
 	{
 		$dirs = $files = $errs = 0;
 		// we first delete all selected links (and files)
 		// feeding the links to dirs to Vfs::find() deletes the content of the dirs, not just the link!
-		foreach($selected as $key => $path)
+		foreach ($selected as $key => $path)
 		{
 			if (!Vfs::is_dir($path) || Vfs::is_link($path))
 			{
@@ -779,21 +777,21 @@ 
 		if ($selected)	// somethings left to delete
 		{
 			// some precaution to never allow to (recursivly) remove /, /apps or /home
-			foreach((array)$selected as $path)
+			foreach ((array)$selected as $path)
 			{
 				if (Vfs::isProtectedDir($path))
 				{
 					$errs++;
-					return lang("Cautiously rejecting to remove folder '%1'!",Vfs::decodePath($path));
+					return lang("Cautiously rejecting to remove folder '%1'!", Vfs::decodePath($path));
 				}
 			}
 			// now we use find to loop through all files and dirs: (selected only contains dirs now)
 			// - depth=true to get first the files and then the dir containing it
 			// - hidden=true to also return hidden files (eg. Thumbs.db), as we cant delete non-empty dirs
 			// - show-deleted=false to not (finally) deleted versioned files
-			foreach(Vfs::find($selected,array('depth'=>true,'hidden'=>true,'show-deleted'=>false)) as $path)
+			foreach (Vfs::find($selected, array('depth'=>true, 'hidden'=>true, 'show-deleted'=>false)) as $path)
 			{
-				if (($is_dir = Vfs::is_dir($path) && !Vfs::is_link($path)) && Vfs::rmdir($path,0))
+				if (($is_dir = Vfs::is_dir($path) && !Vfs::is_link($path)) && Vfs::rmdir($path, 0))
 				{
 					++$dirs;
 				}
@@ -809,13 +807,13 @@ 
 		}
 		if ($errs)
 		{
-			return lang('%1 errors deleteting (%2 directories and %3 files deleted)!',$errs,$dirs,$files);
+			return lang('%1 errors deleteting (%2 directories and %3 files deleted)!', $errs, $dirs, $files);
 		}
 		if ($dirs)
 		{
-			return lang('%1 directories and %2 files deleted.',$dirs,$files);
+			return lang('%1 directories and %2 files deleted.', $dirs, $files);
 		}
-		return $files == 1 ? lang('File deleted.') : lang('%1 files deleted.',$files);
+		return $files == 1 ? lang('File deleted.') : lang('%1 files deleted.', $files);
 	}
 
 	/**
@@ -830,37 +828,37 @@ 
 		if (!$query['csv_export'])
 		{
 			Api\Cache::setSession('filemanager', 'index',
-				array_diff_key ($query, array_flip(array('rows','actions','action_links','placeholder_actions'))));
+				array_diff_key($query, array_flip(array('rows', 'actions', 'action_links', 'placeholder_actions'))));
 		}
-		if(!$query['path']) $query['path'] = static::get_home_dir();
+		if (!$query['path']) $query['path'] = static::get_home_dir();
 
 		// Change template to match selected view
-		if($query['view'])
+		if ($query['view'])
 		{
 			$query['template'] = ($query['view'] == 'row' ? 'filemanager.index.rows' : 'filemanager.tile');
 
 			// Store as preference but only for index, not home
-			if($query['get_rows'] == 'filemanager.filemanager_ui.get_rows')
+			if ($query['get_rows'] == 'filemanager.filemanager_ui.get_rows')
 			{
-				$GLOBALS['egw']->preferences->add('filemanager','nm_view',$query['view']);
+				$GLOBALS['egw']->preferences->add('filemanager', 'nm_view', $query['view']);
 				$GLOBALS['egw']->preferences->save_repository();
 			}
 		}
 		// be tolerant with (in previous versions) not correct urlencoded pathes
-		if (!Vfs::stat($query['path'],true) && Vfs::stat(urldecode($query['path'])))
+		if (!Vfs::stat($query['path'], true) && Vfs::stat(urldecode($query['path'])))
 		{
 			$query['path'] = urldecode($query['path']);
 		}
-		if (!Vfs::stat($query['path'],true) || !Vfs::is_dir($query['path']) || !Vfs::check_access($query['path'],Vfs::READABLE))
+		if (!Vfs::stat($query['path'], true) || !Vfs::is_dir($query['path']) || !Vfs::check_access($query['path'], Vfs::READABLE))
 		{
 			// only redirect, if it would be to some other location, gives redirect-loop otherwise
 			if ($query['path'] != ($path = static::get_home_dir()))
 			{
 				// we will leave here, since we are not allowed, or the location does not exist. Index must handle that, and give
 				// an appropriate message
-				Egw::redirect_link('/index.php',array('menuaction'=>'filemanager.filemanager_ui.index',
+				Egw::redirect_link('/index.php', array('menuaction'=>'filemanager.filemanager_ui.index',
 					'path' => $path,
-					'msg' => lang('The requested path %1 is not available.',Vfs::decodePath($query['path'])),
+					'msg' => lang('The requested path %1 is not available.', Vfs::decodePath($query['path'])),
 					'ajax' => 'true'
 				));
 			}
@@ -868,35 +866,35 @@ 
 			return 0;
 		}
 		$rows = $dir_is_writable = array();
-		if($query['searchletter'] && !empty($query['search']))
+		if ($query['searchletter'] && !empty($query['search']))
 		{
-			$namefilter = '/^'.$query['searchletter'].'.*'.str_replace(array('\\?','\\*'),array('.{1}','.*'),preg_quote($query['search'])).'/i';
+			$namefilter = '/^'.$query['searchletter'].'.*'.str_replace(array('\\?', '\\*'), array('.{1}', '.*'), preg_quote($query['search'])).'/i';
 			if ($query['searchletter'] == strtolower($query['search'][0]))
 			{
-				$namefilter = '/^('.$query['searchletter'].'.*'.str_replace(array('\\?','\\*'),array('.{1}','.*'),preg_quote($query['search'])).'|'.
-					str_replace(array('\\?','\\*'),array('.{1}','.*'),preg_quote($query['search'])).')/i';
+				$namefilter = '/^('.$query['searchletter'].'.*'.str_replace(array('\\?', '\\*'), array('.{1}', '.*'), preg_quote($query['search'])).'|'.
+					str_replace(array('\\?', '\\*'), array('.{1}', '.*'), preg_quote($query['search'])).')/i';
 			}
 		}
 		elseif ($query['searchletter'])
 		{
 			$namefilter = '/^'.$query['searchletter'].'/i';
 		}
-		elseif(!empty($query['search']))
+		elseif (!empty($query['search']))
 		{
-			$namefilter = '/'.str_replace(array('\\?','\\*'),array('.{1}','.*'),preg_quote($query['search'])).'/i';
+			$namefilter = '/'.str_replace(array('\\?', '\\*'), array('.{1}', '.*'), preg_quote($query['search'])).'/i';
 		}
 
 		// Re-map so 'No filters' favorite ('') is depth 1
 		$filter = $query['filter'] === '' ? 1 : $query['filter'];
 
 		$maxdepth = $filter && $filter != 4 ? (int)(boolean)$filter : null;
-		if($filter == 5) $maxdepth = 2;
+		if ($filter == 5) $maxdepth = 2;
 		$n = 0;
 		$vfs_options = array(
 			'mindepth' => 1,
 			'maxdepth' => $maxdepth,
 			'dirsontop' => $filter <= 1,
-			'type' => $filter && $filter != 5 ? ($filter == 4 ? 'd' : null) : ($filter == 5 ? 'F':'f'),
+			'type' => $filter && $filter != 5 ? ($filter == 4 ? 'd' : null) : ($filter == 5 ? 'F' : 'f'),
 			'order' => $query['order'], 'sort' => $query['sort'],
 			'limit' => (int)$query['num_rows'].','.(int)$query['start'],
 			'need_mime' => true,
@@ -904,11 +902,11 @@ 
 			'hidden' => $filter == 3,
 			'follow' => $filter == 5,
 		);
-		if($query['col_filter']['mime'])
+		if ($query['col_filter']['mime'])
 		{
 			$vfs_options['mime'] = $query['col_filter']['mime'];
 		}
-		foreach(Vfs::find(!empty($query['col_filter']['dir']) ? $query['col_filter']['dir'] : $query['path'],$vfs_options,true) as $path => $row)
+		foreach (Vfs::find(!empty($query['col_filter']['dir']) ? $query['col_filter']['dir'] : $query['path'], $vfs_options, true) as $path => $row)
 		{
 			//echo $path; _debug_array($row);
 
@@ -927,37 +925,37 @@ 
 				$row['class'] .= 'isDir ';
 				$row['is_dir'] = 1;
 			}
-			if(!$dir_is_writable[$path])
+			if (!$dir_is_writable[$path])
 			{
 				$row['class'] .= 'noEdit ';
 			}
 			$row['download_url'] = Vfs::download_url($path);
-			$row['gid'] = -abs($row['gid']);	// gid are positive, but we use negagive account_id for groups internal
+			$row['gid'] = -abs($row['gid']); // gid are positive, but we use negagive account_id for groups internal
 
 			$rows[++$n] = $row;
 			$path2n[$path] = $n;
 		}
 		// query comments and cf's for the displayed rows
-		$cols_to_show = explode(',',$GLOBALS['egw_info']['user']['preferences']['filemanager']['nextmatch-filemanager.index.rows']);
+		$cols_to_show = explode(',', $GLOBALS['egw_info']['user']['preferences']['filemanager']['nextmatch-filemanager.index.rows']);
 
 		// Always include comment in tiles
-		if($query['view'] == 'tile')
+		if ($query['view'] == 'tile')
 		{
 			$cols_to_show[] = 'comment';
 		}
-		$all_cfs = in_array('customfields',$cols_to_show) && $cols_to_show[count($cols_to_show)-1][0] != '#';
-		if ($path2n && (in_array('comment',$cols_to_show) || in_array('customfields',$cols_to_show)) &&
+		$all_cfs = in_array('customfields', $cols_to_show) && $cols_to_show[count($cols_to_show) - 1][0] != '#';
+		if ($path2n && (in_array('comment', $cols_to_show) || in_array('customfields', $cols_to_show)) &&
 			($path2props = Vfs::propfind(array_keys($path2n))))
 		{
-			foreach($path2props as $path => $props)
+			foreach ($path2props as $path => $props)
 			{
-				unset($row);	// fixes a weird problem with php5.1, does NOT happen with php5.2
-				$row =& $rows[$path2n[$path]];
-				if ( !is_array($props) ) continue;
-				foreach($props as $prop)
+				unset($row); // fixes a weird problem with php5.1, does NOT happen with php5.2
+				$row = & $rows[$path2n[$path]];
+				if (!is_array($props)) continue;
+				foreach ($props as $prop)
 				{
-					if (!$all_cfs && $prop['name'][0] == '#' && !in_array($prop['name'],$cols_to_show)) continue;
-					$row[$prop['name']] = strlen($prop['val']) < 64 ? $prop['val'] : substr($prop['val'],0,64).' ...';
+					if (!$all_cfs && $prop['name'][0] == '#' && !in_array($prop['name'], $cols_to_show)) continue;
+					$row[$prop['name']] = strlen($prop['val']) < 64 ? $prop['val'] : substr($prop['val'], 0, 64).' ...';
 				}
 			}
 		}
@@ -969,7 +967,7 @@ 
 		if ($GLOBALS['egw_info']['flags']['currentapp'] == 'projectmanager')
 		{
 			// we need our app.css file
-			if (!file_exists(EGW_SERVER_ROOT.($css_file='/filemanager/templates/'.$GLOBALS['egw_info']['server']['template_set'].'/app.css')))
+			if (!file_exists(EGW_SERVER_ROOT.($css_file = '/filemanager/templates/'.$GLOBALS['egw_info']['server']['template_set'].'/app.css')))
 			{
 				$css_file = '/filemanager/templates/default/app.css';
 			}
@@ -985,7 +983,7 @@ 
 	 * @param array $content
 	 * @param string $msg
 	 */
-	function file(array $content=null,$msg='')
+	function file(array $content = null, $msg = '')
 	{
 		$tpl = new Etemplate('filemanager.file');
 
@@ -995,9 +993,9 @@ 
 			{
 				$msg .= $_GET['msg'];
 			}
-			if (!($path = str_replace(array('#','?'),array('%23','%3F'),$_GET['path'])) ||	// ?, # need to stay encoded!
+			if (!($path = str_replace(array('#', '?'), array('%23', '%3F'), $_GET['path'])) || // ?, # need to stay encoded!
 				// actions enclose pathes containing comma with "
-				($path[0] == '"' && substr($path,-1) == '"' && !($path = substr(str_replace('""','"',$path),1,-1))) ||
+				($path[0] == '"' && substr($path, -1) == '"' && !($path = substr(str_replace('""', '"', $path), 1, -1))) ||
 				!($stat = Vfs::lstat($path)))
 			{
 				$msg .= lang('File or directory not found!')." path='$path', stat=".array2string($stat);
@@ -1010,10 +1008,10 @@ 
 				$content['path'] = $path;
 				$content['hsize'] = Vfs::hsize($stat['size']);
 				$content['mime'] = Vfs::mime_content_type($path);
-				$content['gid'] *= -1;	// our widgets use negative gid's
+				$content['gid'] *= -1; // our widgets use negative gid's
 				if (($props = Vfs::propfind($path)))
 				{
-					foreach($props as $prop)
+					foreach ($props as $prop)
 					{
 						$content[$prop['name']] = $prop['val'];
 					}
@@ -1026,9 +1024,9 @@ 
 			$content['tabs'] = $_GET['tabs'];
 			if (!($content['is_dir'] = Vfs::is_dir($path) && !Vfs::is_link($path)))
 			{
-				$content['perms']['executable'] = (int)!!($content['mode'] & 0111);
+				$content['perms']['executable'] = (int)!!($content['mode']&0111);
 				$mask = 6;
-				if (preg_match('/^text/',$content['mime']) && $content['size'] < 100000)
+				if (preg_match('/^text/', $content['mime']) && $content['size'] < 100000)
 				{
 					$content['text_content'] = file_get_contents(Vfs::PREFIX.$path);
 				}
@@ -1038,19 +1036,19 @@ 
 				//currently not implemented in backend $content['perms']['sticky'] = (int)!!($content['mode'] & 0x201);
 				$mask = 7;
 			}
-			foreach(array('owner' => 6,'group' => 3,'other' => 0) as $name => $shift)
+			foreach (array('owner' => 6, 'group' => 3, 'other' => 0) as $name => $shift)
 			{
-				$content['perms'][$name] = ($content['mode'] >> $shift) & $mask;
+				$content['perms'][$name] = ($content['mode'] >> $shift)&$mask;
 			}
-			$content['is_owner'] = Vfs::has_owner_rights($path,$content);
+			$content['is_owner'] = Vfs::has_owner_rights($path, $content);
 		}
 		else
 		{
 			//_debug_array($content);
-			$path =& $content['path'];
+			$path = & $content['path'];
 
 			list($button) = @each($content['button']); unset($content['button']);
-			if(!$button && $content['sudo'])
+			if (!$button && $content['sudo'])
 			{
 				// Button to stop sudo is not in button namespace
 				$button = 'sudo';
@@ -1059,21 +1057,21 @@ 
 			// need to check 'setup' button (submit button in sudo popup), as some browsers (eg. chrome) also fill the hidden field
 			if ($button == 'sudo' && Vfs::$is_root || $button == 'setup' && $content['sudo']['user'])
 			{
-				$msg = $this->sudo($button == 'setup' ? $content['sudo']['user'] : '',$content['sudo']['passwd']) ?
+				$msg = $this->sudo($button == 'setup' ? $content['sudo']['user'] : '', $content['sudo']['passwd']) ?
 					lang('Root access granted.') : ($button == 'setup' && $content['sudo']['user'] ?
 					lang('Wrong username or password!') : lang('Root access stopped.'));
 				unset($content['sudo']);
 				$content['is_owner'] = Vfs::has_owner_rights($path);
 			}
-			if (in_array($button,array('save','apply')))
+			if (in_array($button, array('save', 'apply')))
 			{
 				$props = array();
 				$perm_changed = $perm_failed = 0;
-				foreach($content['old'] as $name => $old_value)
+				foreach ($content['old'] as $name => $old_value)
 				{
 					if (isset($content[$name]) && ($old_value != $content[$name] ||
 						// do not check for modification, if modify_subs is checked!
-						$content['modify_subs'] && in_array($name,array('uid','gid','perms'))) &&
+						$content['modify_subs'] && in_array($name, array('uid', 'gid', 'perms'))) &&
 						($name != 'uid' || Vfs::$is_root))
 					{
 						if ($name == 'name')
@@ -1087,23 +1085,23 @@ 
 							$to = Vfs::concat($dir, $content['name']);
 							if (file_exists(Vfs::PREFIX.$to) && $content['confirm_overwrite'] !== $to)
 							{
-								$tpl->set_validation_error('name',lang("There's already a file with that name!").'<br />'.
-									lang('To overwrite the existing file store again.',lang($button)));
+								$tpl->set_validation_error('name', lang("There's already a file with that name!").'<br />'.
+									lang('To overwrite the existing file store again.', lang($button)));
 								$content['confirm_overwrite'] = $to;
 								if ($button == 'save') $button = 'apply';
 								continue;
 							}
-							if (Vfs::rename($path,$to))
+							if (Vfs::rename($path, $to))
 							{
-								$msg .= lang('Renamed %1 to %2.',Vfs::decodePath(basename($path)),Vfs::decodePath(basename($to))).' ';
+								$msg .= lang('Renamed %1 to %2.', Vfs::decodePath(basename($path)), Vfs::decodePath(basename($to))).' ';
 								$content['old']['name'] = $content[$name];
 								$path = $to;
-								$content['mime'] = Api\MimeMagic::filename2mime($path);	// recheck mime type
-								$refresh_path = Vfs::dirname($path);	// for renames, we have to refresh the parent
+								$content['mime'] = Api\MimeMagic::filename2mime($path); // recheck mime type
+								$refresh_path = Vfs::dirname($path); // for renames, we have to refresh the parent
 							}
 							else
 							{
-								$msg .= lang('Rename of %1 to %2 failed!',Vfs::decodePath(basename($path)),Vfs::decodePath(basename($to))).' ';
+								$msg .= lang('Rename of %1 to %2 failed!', Vfs::decodePath(basename($path)), Vfs::decodePath(basename($to))).' ';
 								if (Vfs::deny_script($to))
 								{
 									$msg .= lang('You are NOT allowed to upload a script!').' ';
@@ -1123,7 +1121,7 @@ 
 									'val'	=> (!empty($content[$name]) ? $content[$name] : null),
 								),
 							);
-							if (Vfs::proppatch($path,$mergeprop))
+							if (Vfs::proppatch($path, $mergeprop))
 							{
 								$content['old'][$name] = $content[$name];
 								$msg .= lang('Setting for document merge saved.');
@@ -1135,22 +1133,22 @@ 
 						}
 						else
 						{
-							static $name2cmd = array('uid' => 'chown','gid' => 'chgrp','perms' => 'chmod');
-							$cmd = array('EGroupware\\Api\\Vfs',$name2cmd[$name]);
+							static $name2cmd = array('uid' => 'chown', 'gid' => 'chgrp', 'perms' => 'chmod');
+							$cmd = array('EGroupware\\Api\\Vfs', $name2cmd[$name]);
 							$value = $name == 'perms' ? static::perms2mode($content['perms']) : $content[$name];
 							if ($content['modify_subs'])
 							{
 								if ($name == 'perms')
 								{
-									$changed = Vfs::find($path,array('type'=>'d'),$cmd,array($value));
-									$changed += Vfs::find($path,array('type'=>'f'),$cmd,array($value & 0666));	// no execute for files
+									$changed = Vfs::find($path, array('type'=>'d'), $cmd, array($value));
+									$changed += Vfs::find($path, array('type'=>'f'), $cmd, array($value&0666)); // no execute for files
 								}
 								else
 								{
-									$changed = Vfs::find($path,null,$cmd,array($value));
+									$changed = Vfs::find($path, null, $cmd, array($value));
 								}
 								$ok = $failed = 0;
-								foreach($changed as &$r)
+								foreach ($changed as &$r)
 								{
 									if ($r)
 									{
@@ -1163,32 +1161,32 @@ 
 								}
 								if ($ok && !$failed)
 								{
-									if(!$perm_changed++) $msg .= lang('Permissions of %1 changed.',$path.' '.lang('and all it\'s childeren'));
+									if (!$perm_changed++) $msg .= lang('Permissions of %1 changed.', $path.' '.lang('and all it\'s childeren'));
 									$content['old'][$name] = $content[$name];
 								}
-								elseif($failed)
+								elseif ($failed)
 								{
-									if(!$perm_failed++) $msg .= lang('Failed to change permissions of %1!',$path.lang('and all it\'s childeren').
-										($ok ? ' ('.lang('%1 failed, %2 succeded',$failed,$ok).')' : ''));
+									if (!$perm_failed++) $msg .= lang('Failed to change permissions of %1!', $path.lang('and all it\'s childeren').
+										($ok ? ' ('.lang('%1 failed, %2 succeded', $failed, $ok).')' : ''));
 								}
 							}
-							elseif (call_user_func_array($cmd,array($path,$value)))
+							elseif (call_user_func_array($cmd, array($path, $value)))
 							{
-								$msg .= lang('Permissions of %1 changed.',$path);
+								$msg .= lang('Permissions of %1 changed.', $path);
 								$content['old'][$name] = $content[$name];
 							}
 							else
 							{
-								$msg .= lang('Failed to change permissions of %1!',$path);
+								$msg .= lang('Failed to change permissions of %1!', $path);
 							}
 						}
 					}
 				}
 				if ($props)
 				{
-					if (Vfs::proppatch($path,$props))
+					if (Vfs::proppatch($path, $props))
 					{
-						foreach($props as $prop)
+						foreach ($props as $prop)
 						{
 							$content['old'][$prop['name']] = $prop['val'];
 						}
@@ -1205,8 +1203,8 @@ 
 				if ($content['eacl']['delete'])
 				{
 					list($ino_owner) = each($content['eacl']['delete']);
-					list(, $owner) = explode('-',$ino_owner,2);	// $owner is a group and starts with a minus!
-					$msg .= Vfs::eacl($path,null,$owner) ? lang('ACL deleted.') : lang('Error deleting the ACL entry!');
+					list(, $owner) = explode('-', $ino_owner, 2); // $owner is a group and starts with a minus!
+					$msg .= Vfs::eacl($path, null, $owner) ? lang('ACL deleted.') : lang('Error deleting the ACL entry!');
 				}
 				elseif ($button == 'eacl')
 				{
@@ -1216,7 +1214,7 @@ 
 					}
 					else
 					{
-						$msg .= Vfs::eacl($path,$content['eacl']['rights'],$content['eacl_owner']) ?
+						$msg .= Vfs::eacl($path, $content['eacl']['rights'], $content['eacl_owner']) ?
 							lang('ACL added.') : lang('Error adding the ACL!');
 					}
 				}
@@ -1226,7 +1224,7 @@ 
 		}
 		if ($content['is_link'] && !Vfs::stat($path))
 		{
-			$msg .= ($msg ? "\n" : '').lang('Link target %1 not found!',$content['symlink']);
+			$msg .= ($msg ? "\n" : '').lang('Link target %1 not found!', $content['symlink']);
 		}
 		$content['link'] = Egw::link(Vfs::download_url($path));
 		$content['icon'] = Vfs::mime_icon($content['mime']);
@@ -1235,11 +1233,11 @@ 
 		if (($readonlys['uid'] = !Vfs::$is_root) && !$content['uid']) $content['ro_uid_root'] = 'root';
 		// only owner can change group & perms
 		if (($readonlys['gid'] = !$content['is_owner'] ||
-			Vfs::parse_url(Vfs::resolve_url($content['path']),PHP_URL_SCHEME) == 'oldvfs') ||// no uid, gid or perms in oldvfs
+			Vfs::parse_url(Vfs::resolve_url($content['path']), PHP_URL_SCHEME) == 'oldvfs') || // no uid, gid or perms in oldvfs
 				 !Vfs::is_writable($path))
 		{
 			if (!$content['gid']) $content['ro_gid_root'] = 'root';
-			foreach($content['perms'] as $name => $value)
+			foreach ($content['perms'] as $name => $value)
 			{
 				$readonlys['perms['.$name.']'] = true;
 			}
@@ -1258,38 +1256,38 @@ 
 		}
 		elseif (!Vfs::is_writable($path))
 		{
-			foreach($cfs as $name => $data)
+			foreach ($cfs as $name => $data)
 			{
 				$readonlys['#'.$name] = true;
 			}
 		}
-		$readonlys['tabs']['filemanager.file.eacl'] = true;	// eacl off by default
+		$readonlys['tabs']['filemanager.file.eacl'] = true; // eacl off by default
 		if ($content['is_dir'])
 		{
-			$readonlys['tabs']['filemanager.file.preview'] = true;	// no preview tab for dirs
-			$sel_options['rights']=$sel_options['owner']=$sel_options['group']=$sel_options['other'] = array(
+			$readonlys['tabs']['filemanager.file.preview'] = true; // no preview tab for dirs
+			$sel_options['rights'] = $sel_options['owner'] = $sel_options['group'] = $sel_options['other'] = array(
 				7 => lang('Display and modification of content'),
 				5 => lang('Display of content'),
 				0 => lang('No access'),
 			);
-			if(($content['eacl'] = Vfs::get_eacl($content['path'])) !== false &&	// backend supports eacl
+			if (($content['eacl'] = Vfs::get_eacl($content['path'])) !== false && // backend supports eacl
 				$GLOBALS['egw_info']['user']['account_id'] == Vfs::$user)	// leave eACL tab disabled for sharing
 			{
-				unset($readonlys['tabs']['filemanager.file.eacl']);	// --> switch the tab on again
-				foreach($content['eacl'] as &$eacl)
+				unset($readonlys['tabs']['filemanager.file.eacl']); // --> switch the tab on again
+				foreach ($content['eacl'] as &$eacl)
 				{
-					$eacl['path'] = rtrim(Vfs::parse_url($eacl['path'],PHP_URL_PATH),'/');
+					$eacl['path'] = rtrim(Vfs::parse_url($eacl['path'], PHP_URL_PATH), '/');
 					$readonlys['delete['.$eacl['ino'].'-'.$eacl['owner'].']'] = $eacl['ino'] != $content['ino'] ||
 						$eacl['path'] != $content['path'] || !$content['is_owner'];
 				}
-				array_unshift($content['eacl'],false);	// make the keys start with 1, not 0
+				array_unshift($content['eacl'], false); // make the keys start with 1, not 0
 				$content['eacl']['owner'] = 0;
 				$content['eacl']['rights'] = 5;
 			}
 		}
 		else
 		{
-			$sel_options['owner']=$sel_options['group']=$sel_options['other'] = array(
+			$sel_options['owner'] = $sel_options['group'] = $sel_options['other'] = array(
 				6 => lang('Read & write access'),
 				4 => lang('Read access only'),
 				0 => lang('No access'),
@@ -1299,11 +1297,11 @@ 
 		// mergeapp
 		$content['mergeapp'] = static::get_mergeapp($path, 'self');
 		$content['mergeapp_parent'] = static::get_mergeapp($path, 'parents');
-		if(!empty($content['mergeapp']))
+		if (!empty($content['mergeapp']))
 		{
 			$content['mergeapp_effective'] = $content['mergeapp'];
 		}
-		elseif(!empty($content['mergeapp_parent']))
+		elseif (!empty($content['mergeapp_parent']))
 		{
 			$content['mergeapp_effective'] = $content['mergeapp_parent'];
 		}
@@ -1315,7 +1313,7 @@ 
 		$mergeapp_list = Link::app_list('merge');
 		unset($mergeapp_list[$GLOBALS['egw_info']['flags']['currentapp']]); // exclude filemanager from list
 		$mergeapp_empty = !empty($content['mergeapp_parent'])
-			? $mergeapp_list[$content['mergeapp_parent']] . ' (parent setting)' : '';
+			? $mergeapp_list[$content['mergeapp_parent']].' (parent setting)' : '';
 		$sel_options['mergeapp'] = array(''	=> $mergeapp_empty);
 		$sel_options['mergeapp'] = $sel_options['mergeapp'] + $mergeapp_list;
 		// mergeapp other gui options
@@ -1332,7 +1330,7 @@ 
 				'comment' => (string)$content['comment'],
 				'mergeapp' => $content['mergeapp']
 			);
-			if ($cfs) foreach($cfs as $name => $data)
+			if ($cfs) foreach ($cfs as $name => $data)
 			{
 				$preserve['old']['#'.$name] = (string)$content['#'.$name];
 			}
@@ -1340,31 +1338,31 @@ 
 		if (Vfs::$is_root)
 		{
 			$tpl->setElementAttribute('sudouser', 'label', 'Logout');
-			$tpl->setElementAttribute('sudouser', 'help','Log out as superuser');
+			$tpl->setElementAttribute('sudouser', 'help', 'Log out as superuser');
 			// Need a more complex submit because button type is buttononly, which doesn't submit
-			$tpl->setElementAttribute('sudouser', 'onclick','app.filemanager.set_sudoButton(widget,"login")');
+			$tpl->setElementAttribute('sudouser', 'onclick', 'app.filemanager.set_sudoButton(widget,"login")');
 
 		}
 		elseif ($button == 'sudo')
 		{
 			$tpl->setElementAttribute('sudouser', 'label', 'Superuser');
-			$tpl->setElementAttribute('sudouser', 'help','Enter setup user and password to get root rights');
-			$tpl->setElementAttribute('sudouser', 'onclick','app.filemanager.set_sudoButton(widget,"logout")');
+			$tpl->setElementAttribute('sudouser', 'help', 'Enter setup user and password to get root rights');
+			$tpl->setElementAttribute('sudouser', 'onclick', 'app.filemanager.set_sudoButton(widget,"logout")');
 		}
 		else if (self::is_anonymous($GLOBALS['egw_info']['user']['account_id']))
 		{
 			// Just hide sudo for anonymous users
 			$readonlys['sudouser'] = true;
 		}
-		if (($extra_tabs = Vfs::getExtraInfo($path,$content)))
+		if (($extra_tabs = Vfs::getExtraInfo($path, $content)))
 		{
 			// add to existing tabs in template
 			$tpl->setElementAttribute('tabs', 'add_tabs', true);
 
-			$tabs =& $tpl->getElementAttribute('tabs','tabs');
+			$tabs = & $tpl->getElementAttribute('tabs', 'tabs');
 			if (true) $tabs = array();
 
-			foreach(isset($extra_tabs[0]) ? $extra_tabs : array($extra_tabs) as $extra_tab)
+			foreach (isset($extra_tabs[0]) ? $extra_tabs : array($extra_tabs) as $extra_tab)
 			{
 				$tabs[] = array(
 					'label' =>	$extra_tab['label'],
@@ -1387,7 +1385,7 @@ 
 		Framework::window_focus();
 		$GLOBALS['egw_info']['flags']['app_header'] = lang('Preferences').' '.Vfs::decodePath($path);
 
-		$tpl->exec('filemanager.filemanager_ui.file',$content,$sel_options,$readonlys,$preserve,2);
+		$tpl->exec('filemanager.filemanager_ui.file', $content, $sel_options, $readonlys, $preserve, 2);
 	}
 
 	/**
@@ -1410,7 +1408,7 @@ 
 		$remove = array(
 			'delete'
 		);
-		foreach($remove as $key)
+		foreach ($remove as $key)
 		{
 			unset($actions[$key]);
 		}
@@ -1424,7 +1422,7 @@ 
 	 * @param string $dir=null current directory
 	 * @see static::action()
 	 */
-	public static function ajax_action($action, $selected, $dir=null, $props=null)
+	public static function ajax_action($action, $selected, $dir = null, $props = null)
 	{
 		// do we have root rights, need to run here too, as method is static and therefore does NOT run __construct
 		if (Api\Cache::getSession('filemanager', 'is_root'))
@@ -1443,15 +1441,15 @@ 
 
 		if (!isset($dir)) $dir = array_pop($selected);
 
-		switch($action)
+		switch ($action)
 		{
 			case 'upload':
 				$script_error = 0;
-				foreach($selected as $tmp_name => &$data)
+				foreach ($selected as $tmp_name => &$data)
 				{
 					$path = Vfs::concat($dir, Vfs::encodePathComponent($data['name']));
 
-					if(Vfs::deny_script($path))
+					if (Vfs::deny_script($path))
 					{
 						if (!isset($script_error))
 						{
@@ -1473,7 +1471,7 @@ 
 					{
 						if (is_dir($GLOBALS['egw_info']['server']['temp_dir']) && is_writable($GLOBALS['egw_info']['server']['temp_dir']))
 						{
-							$tmp_path = $GLOBALS['egw_info']['server']['temp_dir'] . '/' . basename($tmp_name);
+							$tmp_path = $GLOBALS['egw_info']['server']['temp_dir'].'/'.basename($tmp_name);
 						}
 						else
 						{
@@ -1505,7 +1503,7 @@ 
 				break;
 
 			case 'sharelink':
-				$share = Vfs\Sharing::create($selected, Vfs\Sharing::READONLY, basename($selected), array() );
+				$share = Vfs\Sharing::create($selected, Vfs\Sharing::READONLY, basename($selected), array());
 				$arr["share_link"] = $link = Vfs\Sharing::share2link($share);
 				$arr["template"] = Api\Etemplate\Widget\Template::rel2url('/filemanager/templates/default/share_dialog.xet');
 				break;
@@ -1514,16 +1512,16 @@ 
 			case 'link':
 				// First upload
 				$arr = static::ajax_action('upload', $selected, $dir, $props);
-				$app_dir = Link::vfs_path($props['entry']['app'],$props['entry']['id'],'',true);
+				$app_dir = Link::vfs_path($props['entry']['app'], $props['entry']['id'], '', true);
 
-				foreach($arr['uploaded'] as $file)
+				foreach ($arr['uploaded'] as $file)
 				{
-					$target=Vfs::concat($dir,Vfs::encodePathComponent($file['name']));
+					$target = Vfs::concat($dir, Vfs::encodePathComponent($file['name']));
 					if (Vfs::file_exists($target) && $app_dir)
 					{
 						if (!Vfs::file_exists($app_dir)) Vfs::mkdir($app_dir);
 						error_log("Symlinking $target to $app_dir");
-						Vfs::symlink($target, Vfs::concat($app_dir,Vfs::encodePathComponent($file['name'])));
+						Vfs::symlink($target, Vfs::concat($app_dir, Vfs::encodePathComponent($file['name'])));
 					}
 				}
 				// Must return to avoid adding to $response again
@@ -1545,7 +1543,7 @@ 
 	 */
 	private function perms2mode(array $perms)
 	{
-		$mode = $perms['owner'] << 6 | $perms['group'] << 3 | $perms['other'];
+		$mode = $perms['owner'] << 6|$perms['group'] << 3|$perms['other'];
 		if ($mode['executable'])
 		{
 			$mode |= 0111;

api/src/Vfs.php

Spacing +289 added lines, -294 removed lines

@@ -145,7 +145,7 @@ 
 	 * @param resource $context =null context to pass to stream-wrapper
 	 * @return resource
 	 */
-	static function fopen($path, $mode, $context=null)
+	static function fopen($path, $mode, $context = null)
 	{
 		if ($path[0] != '/')
 		{
@@ -161,7 +161,7 @@ 
 	 * @param resource $context =null context to pass to stream-wrapper
 	 * @return resource
 	 */
-	static function opendir($path, $context=null)
+	static function opendir($path, $context = null)
 	{
 		if ($path[0] != '/')
 		{
@@ -192,13 +192,13 @@ 
 	 * @param int $sorting_order =0 !$sorting_order (default) alphabetical in ascending order, $sorting_order alphabetical in descending order.
 	 * @return array
 	 */
-	static function scandir($path,$sorting_order=0)
+	static function scandir($path, $sorting_order = 0)
 	{
 		if ($path[0] != '/')
 		{
 			throw new Exception\AssertionFailed("Directory '$path' is not an absolute path!");
 		}
-		return scandir(self::PREFIX.$path,$sorting_order);
+		return scandir(self::PREFIX.$path, $sorting_order);
 	}
 
 	/**
@@ -208,25 +208,25 @@ 
 	 * @param string $to
 	 * @return boolean
 	 */
-	static function copy($from,$to)
+	static function copy($from, $to)
 	{
-		$old_props = self::file_exists($to) ? self::propfind($to,null) : array();
+		$old_props = self::file_exists($to) ? self::propfind($to, null) : array();
 		// copy properties (eg. file comment), if there are any and evtl. existing old properties
-		$props = self::propfind($from,null);
-		if(!$props)
+		$props = self::propfind($from, null);
+		if (!$props)
 		{
 			$props = array();
 		}
-		foreach($old_props as $prop)
+		foreach ($old_props as $prop)
 		{
-			if (!self::find_prop($props,$prop))
+			if (!self::find_prop($props, $prop))
 			{
-				$prop['val'] = null;	// null = delete prop
+				$prop['val'] = null; // null = delete prop
 				$props[] = $prop;
 			}
 		}
 		// using self::copy_uploaded() to treat copying incl. properties as atomar operation in respect of notifications
-		return self::copy_uploaded(self::PREFIX.$from,$to,$props,false);	// false = no is_uploaded_file check!
+		return self::copy_uploaded(self::PREFIX.$from, $to, $props, false); // false = no is_uploaded_file check!
 	}
 
 	/**
@@ -237,14 +237,14 @@ 
 	 * @param string $ns =self::DEFAULT_PROP_NAMESPACE namespace, only if $prop is no array
 	 * @return &array reference to property in $props or null if not found
 	 */
-	static function &find_prop(array &$props,$name,$ns=self::DEFAULT_PROP_NAMESPACE)
+	static function &find_prop(array &$props, $name, $ns = self::DEFAULT_PROP_NAMESPACE)
 	{
 		if (is_array($name))
 		{
 			$ns = $name['ns'];
 			$name = $name['name'];
 		}
-		foreach($props as &$prop)
+		foreach ($props as &$prop)
 		{
 			if ($prop['name'] == $name && $prop['ns'] == $ns) return $prop;
 		}
@@ -258,16 +258,16 @@ 
 	 * @param boolean $try_create_home =false should a non-existing home-directory be automatically created
 	 * @return array
 	 */
-	static function stat($path,$try_create_home=false)
+	static function stat($path, $try_create_home = false)
 	{
 		if ($path[0] != '/' && strpos($path, self::PREFIX.'/') !== 0)
 		{
 			throw new Exception\AssertionFailed("File '$path' is not an absolute path!");
 		}
 		$vfs = new Vfs\StreamWrapper();
-		if (($stat = $vfs->url_stat($path,0,$try_create_home)))
+		if (($stat = $vfs->url_stat($path, 0, $try_create_home)))
 		{
-			$stat = array_slice($stat,13);	// remove numerical indices 0-12
+			$stat = array_slice($stat, 13); // remove numerical indices 0-12
 		}
 		return $stat;
 	}
@@ -279,16 +279,16 @@ 
 	 * @param boolean $try_create_home =false should a non-existing home-directory be automatically created
 	 * @return array
 	 */
-	static function lstat($path,$try_create_home=false)
+	static function lstat($path, $try_create_home = false)
 	{
 		if ($path[0] != '/' && strpos($path, self::PREFIX.'/') !== 0)
 		{
 			throw new Exception\AssertionFailed("File '$path' is not an absolute path!");
 		}
 		$vfs = new Vfs\StreamWrapper();
-		if (($stat = $vfs->url_stat($path,STREAM_URL_STAT_LINK,$try_create_home)))
+		if (($stat = $vfs->url_stat($path, STREAM_URL_STAT_LINK, $try_create_home)))
 		{
-			$stat = array_slice($stat,13);	// remove numerical indices 0-12
+			$stat = array_slice($stat, 13); // remove numerical indices 0-12
 		}
 		return $stat;
 	}
@@ -339,7 +339,7 @@ 
 	 * @param boolean $clear_fstab =false true clear current fstab, false (default) only add given mount
 	 * @return array|boolean array with fstab, if called without parameter or true on successful mount
 	 */
-	static function mount($url=null,$path=null,$check_url=null,$persitent_mount=true,$clear_fstab=false)
+	static function mount($url = null, $path = null, $check_url = null, $persitent_mount = true, $clear_fstab = false)
 	{
 		return Vfs\StreamWrapper::mount($url, $path, $check_url, $persitent_mount, $clear_fstab);
 	}
@@ -372,7 +372,7 @@ 
 	 * @param boolean $allow_versions =false allow .versions or .attic
 	 * @return boolean
 	 */
-	public static function is_hidden($path, $allow_versions=false)
+	public static function is_hidden($path, $allow_versions = false)
 	{
 		$file = self::basename($path);
 
@@ -408,30 +408,30 @@ 
 	 * @param array $exec_params =null further params for exec as array, path is always the first param and stat the last!
 	 * @return array of pathes if no $exec, otherwise path => stat pairs
 	 */
-	static function find($base,$options=null,$exec=null,$exec_params=null)
+	static function find($base, $options = null, $exec = null, $exec_params = null)
 	{
 		//error_log(__METHOD__."(".print_r($base,true).",".print_r($options,true).",".print_r($exec,true).",".print_r($exec_params,true).")\n");
 
-		$type = $options['type'];	// 'd', 'f' or 'F'
-		$dirs_last = $options['depth'];	// put content of dirs before the dir itself
+		$type = $options['type']; // 'd', 'f' or 'F'
+		$dirs_last = $options['depth']; // put content of dirs before the dir itself
 		// show dirs on top by default, if no recursive listing (allways disabled if $type specified, as unnecessary)
-		$dirsontop = !$type && (isset($options['dirsontop']) ? (boolean)$options['dirsontop'] : isset($options['maxdepth'])&&$options['maxdepth']>0);
-		if ($dirsontop) $options['need_mime'] = true;	// otherwise dirsontop can NOT work
+		$dirsontop = !$type && (isset($options['dirsontop']) ? (boolean)$options['dirsontop'] : isset($options['maxdepth']) && $options['maxdepth'] > 0);
+		if ($dirsontop) $options['need_mime'] = true; // otherwise dirsontop can NOT work
 
 		// process some of the options (need to be done only once)
 		if (isset($options['name']) && !isset($options['name_preg']))	// change from simple *,? wildcards to preg regular expression once
 		{
-			$options['name_preg'] = '/^'.str_replace(array('\\?','\\*'),array('.{1}','.*'),preg_quote($options['name'])).'$/i';
+			$options['name_preg'] = '/^'.str_replace(array('\\?', '\\*'), array('.{1}', '.*'), preg_quote($options['name'])).'$/i';
 		}
 		if (isset($options['path']) && !isset($options['preg_path']))	// change from simple *,? wildcards to preg regular expression once
 		{
-			$options['path_preg'] = '/^'.str_replace(array('\\?','\\*'),array('.{1}','.*'),preg_quote($options['path'])).'$/i';
+			$options['path_preg'] = '/^'.str_replace(array('\\?', '\\*'), array('.{1}', '.*'), preg_quote($options['path'])).'$/i';
 		}
 		if (!isset($options['uid']))
 		{
 			if (isset($options['user']))
 			{
-				$options['uid'] = $GLOBALS['egw']->accounts->name2id($options['user'],'account_lid','u');
+				$options['uid'] = $GLOBALS['egw']->accounts->name2id($options['user'], 'account_lid', 'u');
 			}
 			elseif (isset($options['nouser']))
 			{
@@ -442,7 +442,7 @@ 
 		{
 			if (isset($options['group']))
 			{
-				$options['gid'] = abs($GLOBALS['egw']->accounts->name2id($options['group'],'account_lid','g'));
+				$options['gid'] = abs($GLOBALS['egw']->accounts->name2id($options['group'], 'account_lid', 'g'));
 			}
 			elseif (isset($options['nogroup']))
 			{
@@ -451,7 +451,7 @@ 
 		}
 		if ($options['order'] == 'mime')
 		{
-			$options['need_mime'] = true;	// we need to return the mime colum
+			$options['need_mime'] = true; // we need to return the mime colum
 		}
 		// implicit show deleted files, if hidden is enabled (requires versioning!)
 		if (!empty($options['hidden']) && !isset($options['show-deleted']))
@@ -469,37 +469,37 @@ 
 			$base = array($base);
 		}
 		$result = array();
-		foreach($base as $path)
+		foreach ($base as $path)
 		{
 			if (!$url)
 			{
 				if ($path[0] != '/' || !self::stat($path)) continue;
-				$path = self::PREFIX . $path;
+				$path = self::PREFIX.$path;
 			}
 			if (!isset($options['remove']))
 			{
-				$options['remove'] = count($base) == 1 ? count(explode('/',$path))-3+(int)(substr($path,-1)!='/') : 0;
+				$options['remove'] = count($base) == 1 ? count(explode('/', $path)) - 3 + (int)(substr($path, -1) != '/') : 0;
 			}
 			$is_dir = is_dir($path);
 			if ((int)$options['mindepth'] == 0 && (!$dirs_last || !$is_dir))
 			{
-				self::_check_add($options,$path,$result);
+				self::_check_add($options, $path, $result);
 			}
 			if ($is_dir && (!isset($options['maxdepth']) || ($options['maxdepth'] > 0 &&
 				$options['depth'] < $options['maxdepth'])) &&
 				($dir = @opendir($path, $context)))
 			{
-				while(($fname = readdir($dir)) !== false)
+				while (($fname = readdir($dir)) !== false)
 				{
-					if ($fname == '.' || $fname == '..') continue;	// ignore current and parent dir!
+					if ($fname == '.' || $fname == '..') continue; // ignore current and parent dir!
 
-					if (self::is_hidden($fname, $options['show-deleted']) && !$options['hidden']) continue;	// ignore hidden files
+					if (self::is_hidden($fname, $options['show-deleted']) && !$options['hidden']) continue; // ignore hidden files
 
 					$file = self::concat($path, $fname);
 
 					if ((int)$options['mindepth'] <= 1)
 					{
-						self::_check_add($options,$file,$result);
+						self::_check_add($options, $file, $result);
 					}
 					// only descend into subdirs, if it's a real dir (no link to a dir) or we should follow symlinks
 					if (is_dir($file) && ($options['follow'] || !is_link($file)) && (!isset($options['maxdepth']) || $options['maxdepth'] > 1))
@@ -509,7 +509,7 @@ 
 						if ($opts['maxdepth']) $opts['depth']++;
 						unset($opts['order']);
 						unset($opts['limit']);
-						foreach(self::find($options['url']?$file:self::parse_url($file,PHP_URL_PATH),$opts,true) as $p => $s)
+						foreach (self::find($options['url'] ? $file : self::parse_url($file, PHP_URL_PATH), $opts, true) as $p => $s)
 						{
 							unset($result[$p]);
 							$result[$p] = $s;
@@ -520,7 +520,7 @@ 
 			}
 			if ($is_dir && (int)$options['mindepth'] == 0 && $dirs_last)
 			{
-				self::_check_add($options,$path,$result);
+				self::_check_add($options, $path, $result);
 			}
 		}
 		// sort code, to place directories before files, if $dirsontop enabled
@@ -530,7 +530,7 @@ 
 		if (isset($options['order']))
 		{
 			$sort = strtolower($options['sort']) == 'desc' ? '-' : '';
-			switch($options['order'])
+			switch ($options['order'])
 			{
 				// sort numerical
 				case 'size':
@@ -542,7 +542,7 @@ 
 					$code = $dirsfirst.$sort.'($a[\''.$options['order'].'\']-$b[\''.$options['order'].'\']);';
 					// always use name as second sort criteria
 					$code = '$cmp = '.$code.' return $cmp ? $cmp : strcasecmp($a[\'name\'],$b[\'name\']);';
-					$ok = uasort($result,create_function('$a,$b',$code));
+					$ok = uasort($result, create_function('$a,$b', $code));
 					break;
 
 				// sort alphanumerical
@@ -561,7 +561,7 @@ 
 					{
 						$code = 'return '.$code;
 					}
-					$ok = uasort($result,create_function('$a,$b',$code));
+					$ok = uasort($result, create_function('$a,$b', $code));
 					break;
 			}
 			//echo "<p>order='$options[order]', sort='$options[sort]' --> uasort($result,create_function(,'$code'))=".array2string($ok)."</p>>\n";
@@ -570,13 +570,13 @@ 
 		self::$find_total = count($result);
 		if (isset($options['limit']))
 		{
-			list($limit,$start) = explode(',',$options['limit']);
+			list($limit, $start) = explode(',', $options['limit']);
 			if (!$limit && !($limit = $GLOBALS['egw_info']['user']['preferences']['comman']['maxmatches'])) $limit = 15;
 			//echo "total=".self::$find_total.", limit=$options[limit] --> start=$start, limit=$limit<br>\n";
 
 			if ((int)$start || self::$find_total > $limit)
 			{
-				$result = array_slice($result,(int)$start,(int)$limit,true);
+				$result = array_slice($result, (int)$start, (int)$limit, true);
 			}
 		}
 		//echo $path; _debug_array($result);
@@ -586,13 +586,13 @@ 
 			{
 				$exec_params = is_null($exec_params) ? array() : array($exec_params);
 			}
-			foreach($result as $path => &$stat)
+			foreach ($result as $path => &$stat)
 			{
 				$options = $exec_params;
-				array_unshift($options,$path);
-				array_push($options,$stat);
+				array_unshift($options, $path);
+				array_push($options, $stat);
 				//echo "calling ".print_r($exec,true).print_r($options,true)."\n";
-				$stat = call_user_func_array($exec,$options);
+				$stat = call_user_func_array($exec, $options);
 			}
 			return $result;
 		}
@@ -611,15 +611,15 @@ 
 	 * @param string $path name of path to add
 	 * @param array &$result here we add the stat for the key $path, if the checks are successful
 	 */
-	private static function _check_add($options,$path,&$result)
+	private static function _check_add($options, $path, &$result)
 	{
-		$type = $options['type'];	// 'd' or 'f'
+		$type = $options['type']; // 'd' or 'f'
 
 		if ($options['url'])
 		{
 			if (($stat = @lstat($path)))
 			{
-				$stat = array_slice($stat,13);	// remove numerical indices 0-12
+				$stat = array_slice($stat, 13); // remove numerical indices 0-12
 			}
 		}
 		else
@@ -628,30 +628,30 @@ 
 		}
 		if (!$stat)
 		{
-			return;	// not found, should not happen
+			return; // not found, should not happen
 		}
-		if ($type && (($type == 'd') == !($stat['mode'] & Vfs\Sqlfs\StreamWrapper::MODE_DIR) ||	// != is_dir() which can be true for symlinks
+		if ($type && (($type == 'd') == !($stat['mode']&Vfs\Sqlfs\StreamWrapper::MODE_DIR) || // != is_dir() which can be true for symlinks
 		    $type == 'F' && is_dir($path)))	// symlink to a directory
 		{
-			return;	// wrong type
+			return; // wrong type
 		}
-		$stat['path'] = self::parse_url($path,PHP_URL_PATH);
-		$stat['name'] = $options['remove'] > 0 ? implode('/',array_slice(explode('/',$stat['path']),$options['remove'])) : self::basename($path);
+		$stat['path'] = self::parse_url($path, PHP_URL_PATH);
+		$stat['name'] = $options['remove'] > 0 ? implode('/', array_slice(explode('/', $stat['path']), $options['remove'])) : self::basename($path);
 
 		if ($options['mime'] || $options['need_mime'])
 		{
 			$stat['mime'] = self::mime_content_type($path);
 		}
-		if (isset($options['name_preg']) && !preg_match($options['name_preg'],$stat['name']) ||
-			isset($options['path_preg']) && !preg_match($options['path_preg'],$path))
+		if (isset($options['name_preg']) && !preg_match($options['name_preg'], $stat['name']) ||
+			isset($options['path_preg']) && !preg_match($options['path_preg'], $path))
 		{
 			//echo "<p>!preg_match('{$options['name_preg']}','{$stat['name']}')</p>\n";
-			return;	// wrong name or path
+			return; // wrong name or path
 		}
 		if (isset($options['gid']) && $stat['gid'] != $options['gid'] ||
 			isset($options['uid']) && $stat['uid'] != $options['uid'])
 		{
-			return;	// wrong user or group
+			return; // wrong user or group
 		}
 		if (isset($options['mime']) && $options['mime'] != $stat['mime'])
 		{
@@ -659,40 +659,40 @@ 
 			{
 				if (!preg_match($options['mime'], $stat['mime']))
 				{
-					return;	// wrong mime-type
+					return; // wrong mime-type
 				}
 			}
 			else
 			{
-				list($type,$subtype) = explode('/',$options['mime']);
+				list($type, $subtype) = explode('/', $options['mime']);
 				// no subtype (eg. 'image') --> check only the main type
-				if ($subtype || substr($stat['mime'],0,strlen($type)+1) != $type.'/')
+				if ($subtype || substr($stat['mime'], 0, strlen($type) + 1) != $type.'/')
 				{
-					return;	// wrong mime-type
+					return; // wrong mime-type
 				}
 			}
 		}
-		if (isset($options['size']) && !self::_check_num($stat['size'],$options['size']) ||
+		if (isset($options['size']) && !self::_check_num($stat['size'], $options['size']) ||
 			(isset($options['empty']) && !!$options['empty'] !== !$stat['size']))
 		{
-			return;	// wrong size
+			return; // wrong size
 		}
-		if (isset($options['cmin']) && !self::_check_num(round((time()-$stat['ctime'])/60),$options['cmin']) ||
-			isset($options['mmin']) && !self::_check_num(round((time()-$stat['mtime'])/60),$options['mmin']) ||
-			isset($options['ctime']) && !self::_check_num(round((time()-$stat['ctime'])/86400),$options['ctime']) ||
-			isset($options['mtime']) && !self::_check_num(round((time()-$stat['mtime'])/86400),$options['mtime']))
+		if (isset($options['cmin']) && !self::_check_num(round((time() - $stat['ctime']) / 60), $options['cmin']) ||
+			isset($options['mmin']) && !self::_check_num(round((time() - $stat['mtime']) / 60), $options['mmin']) ||
+			isset($options['ctime']) && !self::_check_num(round((time() - $stat['ctime']) / 86400), $options['ctime']) ||
+			isset($options['mtime']) && !self::_check_num(round((time() - $stat['mtime']) / 86400), $options['mtime']))
 		{
-			return;	// not create/modified in the spezified time
+			return; // not create/modified in the spezified time
 		}
 		// do we return url or just vfs pathes
 		if (!$options['url'])
 		{
-			$path = self::parse_url($path,PHP_URL_PATH);
+			$path = self::parse_url($path, PHP_URL_PATH);
 		}
 		$result[$path] = $stat;
 	}
 
-	private static function _check_num($value,$argument)
+	private static function _check_num($value, $argument)
 	{
 		if (is_int($argument) && $argument >= 0 || $argument[0] != '-' && $argument[0] != '+')
 		{
@@ -705,7 +705,7 @@ 
 			return $value < abs($argument);
 		}
 		//echo "_check_num($value,$argument) check > == ".(int)($value > (int)substr($argument,1))."\n";
-		return $value > (int) substr($argument,1);
+		return $value > (int)substr($argument, 1);
 	}
 
 	/**
@@ -735,10 +735,10 @@ 
 	 * @throws Vfs\Exception\ProtectedDirectory if trying to delete a protected directory, see Vfs::isProtected()
 	 * @return array
 	 */
-	static function remove($urls,$allow_urls=false)
+	static function remove($urls, $allow_urls = false)
 	{
 		//error_log(__METHOD__.'('.array2string($urls).')');
-		foreach((array)$urls as $url)
+		foreach ((array)$urls as $url)
 		{
 			// some precaution to never allow to (recursivly) remove /, /apps or /home, see Vfs::isProtected()
 			if (self::isProtectedDir($url))
@@ -746,7 +746,7 @@ 
 				throw new Vfs\Exception\ProtectedDirectory("Deleting protected directory '$url' rejected!");
 			}
 		}
-		return self::find($urls, array('depth'=>true,'url'=>$allow_urls,'hidden'=>true), __CLASS__.'::_rm_rmdir');
+		return self::find($urls, array('depth'=>true, 'url'=>$allow_urls, 'hidden'=>true), __CLASS__.'::_rm_rmdir');
 	}
 
 	/**
@@ -759,12 +759,12 @@ 
 	{
 		if ($url[0] == '/')
 		{
-			$url = self::PREFIX . $url;
+			$url = self::PREFIX.$url;
 		}
 		$vfs = new Vfs\StreamWrapper();
 		if (is_dir($url) && !is_link($url))
 		{
-			return $vfs->rmdir($url,0);
+			return $vfs->rmdir($url, 0);
 		}
 		return $vfs->unlink($url);
 	}
@@ -778,9 +778,9 @@ 
 	 * 	2 = self::WRITABLE, 1 = self::EXECUTABLE
 	 * @return boolean
 	 */
-	static function is_readable($path,$check = self::READABLE)
+	static function is_readable($path, $check = self::READABLE)
 	{
-		return self::check_access($path,$check);
+		return self::check_access($path, $check);
 	}
 
 	/**
@@ -794,7 +794,7 @@ 
 	 * @param int $user =null user used for check, if not current user (self::$user)
 	 * @return boolean
 	 */
-	static function check_access($path, $check, $stat=null, $user=null)
+	static function check_access($path, $check, $stat = null, $user = null)
 	{
 		static $vfs = null;
 
@@ -812,7 +812,7 @@ 
 				if (!isset($vfs)) $vfs = new Vfs\StreamWrapper();
 				$path_user_stat[$path][$user] = $vfs->url_stat($path, 0);
 
-				self::clearstatcache($path);	// we need to clear the stat-cache after the call too, as the next call might be the regular user again!
+				self::clearstatcache($path); // we need to clear the stat-cache after the call too, as the next call might be the regular user again!
 			}
 			if (($stat = $path_user_stat[$path][$user]))
 			{
@@ -828,7 +828,7 @@ 
 			}
 			else
 			{
-				$ret = false;	// no access, if we can not stat the file
+				$ret = false; // no access, if we can not stat the file
 			}
 			self::$user = $backup_user;
 
@@ -853,20 +853,20 @@ 
 		if (is_null($stat))
 		{
 			if (!isset($vfs)) $vfs = new Vfs\StreamWrapper();
-			$stat = $vfs->url_stat($path,0);
+			$stat = $vfs->url_stat($path, 0);
 		}
 		//error_log(__METHOD__."(path=$path||stat[name]={$stat['name']},stat[mode]=".sprintf('%o',$stat['mode']).",$check)");
 
 		if (!$stat)
 		{
 			//error_log(__METHOD__."(path=$path||stat[name]={$stat['name']},stat[mode]=".sprintf('%o',$stat['mode']).",$check) no stat array!");
-			return false;	// file not found
+			return false; // file not found
 		}
 		// check if we use an EGroupwre stream wrapper, or a stock php one
 		// if it's not an EGroupware one, we can NOT use uid, gid and mode!
-		if (($scheme = self::parse_url($stat['url'],PHP_URL_SCHEME)) && !(class_exists(self::scheme2class($scheme))))
+		if (($scheme = self::parse_url($stat['url'], PHP_URL_SCHEME)) && !(class_exists(self::scheme2class($scheme))))
 		{
-			switch($check)
+			switch ($check)
 			{
 				case self::READABLE:
 					return is_readable($stat['url']);
@@ -877,19 +877,19 @@ 
 			}
 		}
 		// check if other rights grant access
-		if (($stat['mode'] & $check) == $check)
+		if (($stat['mode']&$check) == $check)
 		{
 			//error_log(__METHOD__."(path=$path||stat[name]={$stat['name']},stat[mode]=".sprintf('%o',$stat['mode']).",$check) access via other rights!");
 			return true;
 		}
 		// check if there's owner access and we are the owner
-		if (($stat['mode'] & ($check << 6)) == ($check << 6) && $stat['uid'] && $stat['uid'] == self::$user)
+		if (($stat['mode']&($check << 6)) == ($check << 6) && $stat['uid'] && $stat['uid'] == self::$user)
 		{
 			//error_log(__METHOD__."(path=$path||stat[name]={$stat['name']},stat[mode]=".sprintf('%o',$stat['mode']).",$check) access via owner rights!");
 			return true;
 		}
 		// check if there's a group access and we have the right membership
-		if (($stat['mode'] & ($check << 3)) == ($check << 3) && $stat['gid'])
+		if (($stat['mode']&($check << 3)) == ($check << 3) && $stat['gid'])
 		{
 			if (($memberships = $GLOBALS['egw']->accounts->memberships(self::$user, true)) && in_array(-abs($stat['gid']), $memberships))
 			{
@@ -898,7 +898,7 @@ 
 			}
 		}
 		// check backend for extended acls (only if path given)
-		$ret = $path && self::_call_on_backend('check_extended_acl',array(isset($stat['url'])?$stat['url']:$path,$check),true);	// true = fail silent if backend does not support
+		$ret = $path && self::_call_on_backend('check_extended_acl', array(isset($stat['url']) ? $stat['url'] : $path, $check), true); // true = fail silent if backend does not support
 
 		//error_log(__METHOD__."(path=$path||stat[name]={$stat['name']},stat[mode]=".sprintf('%o',$stat['mode']).",$check) ".($ret ? 'backend extended acl granted access.' : 'no access!!!'));
 		return $ret;
@@ -913,7 +913,7 @@ 
 	 */
 	static function is_writable($path)
 	{
-		return self::is_readable($path,self::WRITABLE);
+		return self::is_readable($path, self::WRITABLE);
 	}
 
 	/**
@@ -925,7 +925,7 @@ 
 	 */
 	static function is_executable($path)
 	{
-		return self::is_readable($path,self::EXECUTABLE);
+		return self::is_readable($path, self::EXECUTABLE);
 	}
 
 	/**
@@ -936,7 +936,7 @@ 
 	 */
 	static function deny_script($path)
 	{
-		return self::_call_on_backend('deny_script',array($path),true);
+		return self::_call_on_backend('deny_script', array($path), true);
 	}
 
 	/**
@@ -955,11 +955,11 @@ 
 	 * @param boolean $session_only =false true: set eacl only for this session, does NO further checks currently!
 	 * @return boolean true if acl is set/deleted, false on error
 	 */
-	static function eacl($url,$rights=null,$owner=null,$session_only=false)
+	static function eacl($url, $rights = null, $owner = null, $session_only = false)
 	{
 		if ($session_only)
 		{
-			$session_eacls =& Cache::getSession(__CLASS__, self::SESSION_EACL);
+			$session_eacls = & Cache::getSession(__CLASS__, self::SESSION_EACL);
 			$session_eacls[] = array(
 				'path'   => $url[0] == '/' ? $url : self::parse_url($url, PHP_URL_PATH),
 				'owner'  => $owner ? $owner : self::$user,
@@ -967,7 +967,7 @@ 
 			);
 			return true;
 		}
-		return self::_call_on_backend('eacl',array($url,$rights,$owner));
+		return self::_call_on_backend('eacl', array($url, $rights, $owner));
 	}
 
 	/**
@@ -980,9 +980,9 @@ 
 	 */
 	static function get_eacl($path)
 	{
-		$eacls = self::_call_on_backend('get_eacl',array($path),true);	// true = fail silent (no PHP Warning)
+		$eacls = self::_call_on_backend('get_eacl', array($path), true); // true = fail silent (no PHP Warning)
 
-		$session_eacls =& Cache::getSession(__CLASS__, self::SESSION_EACL);
+		$session_eacls = & Cache::getSession(__CLASS__, self::SESSION_EACL);
 		if ($session_eacls)
 		{
 			// eacl is recursive, therefore we have to match all parent-dirs too
@@ -991,7 +991,7 @@ 
 			{
 				$paths[] = $path = self::dirname($path);
 			}
-			foreach((array)$session_eacls as $eacl)
+			foreach ((array)$session_eacls as $eacl)
 			{
 				if (in_array($eacl['path'], $paths))
 				{
@@ -1014,9 +1014,9 @@ 
 	 * @param array $props array of array with values for keys 'name', 'ns', 'val' (null to delete the prop)
 	 * @return boolean true if props are updated, false otherwise (eg. ressource not found)
 	 */
-	static function proppatch($path,array $props)
+	static function proppatch($path, array $props)
 	{
-		return self::_call_on_backend('proppatch',array($path,$props));
+		return self::_call_on_backend('proppatch', array($path, $props));
 	}
 
 	/**
@@ -1033,9 +1033,9 @@ 
 	 * @return array|boolean array with props (values for keys 'name', 'ns', 'val'), or path => array of props for is_array($path)
 	 * 	false if $path does not exist
 	 */
-	static function propfind($path,$ns=self::DEFAULT_PROP_NAMESPACE)
+	static function propfind($path, $ns = self::DEFAULT_PROP_NAMESPACE)
 	{
-		return self::_call_on_backend('propfind',array($path,$ns),true);	// true = fail silent (no PHP Warning)
+		return self::_call_on_backend('propfind', array($path, $ns), true); // true = fail silent (no PHP Warning)
 	}
 
 	/**
@@ -1053,7 +1053,7 @@ 
 	 * @param int $mode =0 current mode of the file, necessary for +/- operation
 	 * @return int
 	 */
-	static function mode2int($set,$mode=0)
+	static function mode2int($set, $mode = 0)
 	{
 		if (is_int($set))		// already an integer
 		{
@@ -1062,23 +1062,23 @@ 
 		if (is_numeric($set))	// octal string
 		{
 			//error_log(__METHOD__."($set,$mode) returning ".(int)base_convert($set,8,10));
-			return (int)base_convert($set,8,10);	// convert octal to decimal
+			return (int)base_convert($set, 8, 10); // convert octal to decimal
 		}
-		foreach(explode(',',$set) as $s)
+		foreach (explode(',', $set) as $s)
 		{
 			$matches = null;
-			if (!preg_match($use='/^([ugoa]*)([+=-]+)([rwx]+)$/',$s,$matches))
+			if (!preg_match($use = '/^([ugoa]*)([+=-]+)([rwx]+)$/', $s, $matches))
 			{
-				$use = str_replace(array('/','^','$','(',')'),'',$use);
+				$use = str_replace(array('/', '^', '$', '(', ')'), '', $use);
 				throw new Exception\WrongUserinput("$s is not an allowed mode, use $use !");
 			}
-			$base = (strpos($matches[3],'r') !== false ? self::READABLE : 0) |
-				(strpos($matches[3],'w') !== false ? self::WRITABLE : 0) |
-				(strpos($matches[3],'x') !== false ? self::EXECUTABLE : 0);
+			$base = (strpos($matches[3], 'r') !== false ? self::READABLE : 0)|
+				(strpos($matches[3], 'w') !== false ? self::WRITABLE : 0)|
+				(strpos($matches[3], 'x') !== false ? self::EXECUTABLE : 0);
 
-			for($n = $m = 0; $n < strlen($matches[1]); $n++)
+			for ($n = $m = 0; $n < strlen($matches[1]); $n++)
 			{
-				switch($matches[1][$n])
+				switch ($matches[1][$n])
 				{
 					case 'o':
 						$m |= $base;
@@ -1091,10 +1091,10 @@ 
 						break;
 					default:
 					case 'a':
-						$m = $base | ($base << 3) | ($base << 6);
+						$m = $base|($base << 3)|($base << 6);
 				}
 			}
-			switch($matches[2])
+			switch ($matches[2])
 			{
 				case '+':
 					$mode |= $m;
@@ -1116,33 +1116,33 @@ 
 	 * @param int $mode
 	 * @return string
 	 */
-	static function int2mode( $mode )
+	static function int2mode($mode)
 	{
-		if(($mode & self::MODE_LINK) == self::MODE_LINK) // Symbolic Link
+		if (($mode&self::MODE_LINK) == self::MODE_LINK) // Symbolic Link
 		{
 			$sP = 'l';
 		}
-		elseif(($mode & 0xC000) == 0xC000) // Socket
+		elseif (($mode&0xC000) == 0xC000) // Socket
 		{
 			$sP = 's';
 		}
-		elseif($mode & 0x1000)     // FIFO pipe
+		elseif ($mode&0x1000)     // FIFO pipe
 		{
 			$sP = 'p';
 		}
-		elseif($mode & 0x2000) // Character special
+		elseif ($mode&0x2000) // Character special
 		{
 			$sP = 'c';
 		}
-		elseif($mode & 0x4000) // Directory
+		elseif ($mode&0x4000) // Directory
 		{
 			$sP = 'd';
 		}
-		elseif($mode & 0x6000) // Block special
+		elseif ($mode&0x6000) // Block special
 		{
 			$sP = 'b';
 		}
-		elseif($mode & 0x8000) // Regular
+		elseif ($mode&0x8000) // Regular
 		{
 			$sP = '-';
 		}
@@ -1152,22 +1152,19 @@ 
 		}
 
 		// owner
-		$sP .= (($mode & 0x0100) ? 'r' : '-') .
-		(($mode & 0x0080) ? 'w' : '-') .
-		(($mode & 0x0040) ? (($mode & 0x0800) ? 's' : 'x' ) :
-		(($mode & 0x0800) ? 'S' : '-'));
+		$sP .= (($mode&0x0100) ? 'r' : '-').
+		(($mode&0x0080) ? 'w' : '-').
+		(($mode&0x0040) ? (($mode&0x0800) ? 's' : 'x') : (($mode&0x0800) ? 'S' : '-'));
 
 		// group
-		$sP .= (($mode & 0x0020) ? 'r' : '-') .
-		(($mode & 0x0010) ? 'w' : '-') .
-		(($mode & 0x0008) ? (($mode & 0x0400) ? 's' : 'x' ) :
-		(($mode & 0x0400) ? 'S' : '-'));
+		$sP .= (($mode&0x0020) ? 'r' : '-').
+		(($mode&0x0010) ? 'w' : '-').
+		(($mode&0x0008) ? (($mode&0x0400) ? 's' : 'x') : (($mode&0x0400) ? 'S' : '-'));
 
 		// world
-		$sP .= (($mode & 0x0004) ? 'r' : '-') .
-		(($mode & 0x0002) ? 'w' : '-') .
-		(($mode & 0x0001) ? (($mode & 0x0200) ? 't' : 'x' ) :
-		(($mode & 0x0200) ? 'T' : '-'));
+		$sP .= (($mode&0x0004) ? 'r' : '-').
+		(($mode&0x0002) ? 'w' : '-').
+		(($mode&0x0001) ? (($mode&0x0200) ? 't' : 'x') : (($mode&0x0200) ? 'T' : '-'));
 
 		return $sP;
 	}
@@ -1180,26 +1177,26 @@ 
 	 * @param int $size =128
 	 * @return string
 	 */
-	static function mime_icon($mime_type, $et_image=true, $size=128)
+	static function mime_icon($mime_type, $et_image = true, $size = 128)
 	{
 		if ($mime_type == self::DIR_MIME_TYPE)
 		{
 			$mime_type = 'Directory';
 		}
-		if(!$mime_type)
+		if (!$mime_type)
 		{
 			$mime_type = 'unknown';
 		}
-		$mime_full = strtolower(str_replace	('/','_',$mime_type));
-		list($mime_part) = explode('_',$mime_full);
+		$mime_full = strtolower(str_replace('/', '_', $mime_type));
+		list($mime_part) = explode('_', $mime_full);
 
-		if (!($img=Image::find('etemplate',$icon='mime'.$size.'_'.$mime_full)) &&
+		if (!($img = Image::find('etemplate', $icon = 'mime'.$size.'_'.$mime_full)) &&
 			// check mime-alias-map before falling back to more generic icons
 			!(isset(MimeMagic::$mime_alias_map[$mime_type]) &&
-				($img=Image::find('etemplate',$icon='mime'.$size.'_'.str_replace('/','_',MimeMagic::$mime_alias_map[$mime_full])))) &&
-			!($img=Image::find('etemplate',$icon='mime'.$size.'_'.$mime_part)))
+				($img = Image::find('etemplate', $icon = 'mime'.$size.'_'.str_replace('/', '_', MimeMagic::$mime_alias_map[$mime_full])))) &&
+			!($img = Image::find('etemplate', $icon = 'mime'.$size.'_'.$mime_part)))
 		{
-			$img = Image::find('etemplate',$icon='mime'.$size.'_unknown');
+			$img = Image::find('etemplate', $icon = 'mime'.$size.'_unknown');
 		}
 		return $et_image ? 'etemplate/'.$icon : $img;
 	}
@@ -1213,9 +1210,9 @@ 
 	static function hsize($size)
 	{
 		if ($size < 1024) return $size;
-		if ($size < 1024*1024) return sprintf('%3.1lfk',(float)$size/1024);
-		if ($size < 1024*1024*1024) return sprintf('%3.1lfM',(float)$size/(1024*1024));
-		return sprintf('%3.1lfG',(float)$size/(1024*1024*1024));
+		if ($size < 1024 * 1024) return sprintf('%3.1lfk', (float)$size / 1024);
+		if ($size < 1024 * 1024 * 1024) return sprintf('%3.1lfM', (float)$size / (1024 * 1024));
+		return sprintf('%3.1lfG', (float)$size / (1024 * 1024 * 1024));
 	}
 
 	/**
@@ -1225,7 +1222,7 @@ 
 	 */
 	static function int_size($_val)
 	{
-		if(empty($_val))return 0;
+		if (empty($_val))return 0;
 
 		$val = trim($_val);
 
@@ -1233,12 +1230,12 @@ 
 		preg_match('#([0-9]+)[\s]*([a-z]+)#i', $val, $matches);
 
 		$last = '';
-		if(isset($matches[2])){
+		if (isset($matches[2])) {
 			$last = $matches[2];
 		}
 
-		if(isset($matches[1])){
-			$val = (int) $matches[1];
+		if (isset($matches[1])) {
+			$val = (int)$matches[1];
 		}
 
 		switch (strtolower($last))
@@ -1254,7 +1251,7 @@ 
 			$val *= 1024;
 		}
 
-		return (int) $val;
+		return (int)$val;
 	}
 
 	/**
@@ -1265,8 +1262,8 @@ 
 	 */
 	static function basename($_path)
 	{
-		list($path) = explode('?',$_path);	// remove query
-		$parts = explode('/',$path);
+		list($path) = explode('?', $_path); // remove query
+		$parts = explode('/', $path);
 
 		return array_pop($parts);
 	}
@@ -1280,7 +1277,7 @@ 
 	 * @param int $component =-1 PHP_URL_* constants
 	 * @return array|string|boolean on success array or string, if $component given, or false on failure
 	 */
-	static function parse_url($url, $component=-1)
+	static function parse_url($url, $component = -1)
 	{
 		static $component2str = array(
 			PHP_URL_SCHEME => 'scheme',
@@ -1292,9 +1289,9 @@ 
 			PHP_URL_QUERY => 'query',
 			PHP_URL_FRAGMENT => 'fragment',
 		);
-		static $cache = array();	// some caching
+		static $cache = array(); // some caching
 
-		$result =& $cache[$url];
+		$result = & $cache[$url];
 
 		if (!isset($result))
 		{
@@ -1334,22 +1331,22 @@ 
 	 */
 	static function dirname($_url)
 	{
-		list($url,$query) = explode('?',$_url,2);	// strip the query first, as it can contain slashes
+		list($url, $query) = explode('?', $_url, 2); // strip the query first, as it can contain slashes
 
-		if ($url == '/' || $url[0] != '/' && self::parse_url($url,PHP_URL_PATH) == '/')
+		if ($url == '/' || $url[0] != '/' && self::parse_url($url, PHP_URL_PATH) == '/')
 		{
 			//error_log(__METHOD__."($url) returning FALSE: already in root!");
 			return false;
 		}
-		$parts = explode('/',$url);
-		if (substr($url,-1) == '/') array_pop($parts);
+		$parts = explode('/', $url);
+		if (substr($url, -1) == '/') array_pop($parts);
 		array_pop($parts);
 		if ($url[0] != '/' && count($parts) == 3 || count($parts) == 1 && $parts[0] === '')
 		{
-			array_push($parts,'');	// scheme://host is wrong (no path), has to be scheme://host/
+			array_push($parts, ''); // scheme://host is wrong (no path), has to be scheme://host/
 		}
 		//error_log(__METHOD__."($url)=".implode('/',$parts).($query ? '?'.$query : ''));
-		return implode('/',$parts).($query ? '?'.$query : '');
+		return implode('/', $parts).($query ? '?'.$query : '');
 	}
 
 	/**
@@ -1361,18 +1358,18 @@ 
 	 * @param array $stat =null stat for path, default queried by this function
 	 * @return boolean
 	 */
-	static function has_owner_rights($path,array $stat=null)
+	static function has_owner_rights($path, array $stat = null)
 	{
 		if (!$stat)
 		{
 			$vfs = new Vfs\StreamWrapper();
-			$stat = $vfs->url_stat($path,0);
+			$stat = $vfs->url_stat($path, 0);
 		}
-		return $stat['uid'] == self::$user &&	// (current) user is the owner
+		return $stat['uid'] == self::$user && // (current) user is the owner
 				// in sharing current user != self::$user and should NOT have owner rights
 				$GLOBALS['egw_info']['user']['account_id'] == self::$user ||
-			self::$is_root ||					// class runs with root rights
-			!$stat['uid'] && $stat['gid'] && self::$is_admin;	// group directory and user is an eGW admin
+			self::$is_root || // class runs with root rights
+			!$stat['uid'] && $stat['gid'] && self::$is_admin; // group directory and user is an eGW admin
 	}
 
 	/**
@@ -1384,22 +1381,22 @@ 
 	 * @param string $relative relative path to add to $url
 	 * @return string
 	 */
-	static function concat($_url,$relative)
+	static function concat($_url, $relative)
 	{
-		list($url,$query) = explode('?',$_url,2);
-		if (substr($url,-1) == '/') $url = substr($url,0,-1);
+		list($url, $query) = explode('?', $_url, 2);
+		if (substr($url, -1) == '/') $url = substr($url, 0, -1);
 		$ret = ($relative === '' || $relative[0] == '/' ? $url.$relative : $url.'/'.$relative);
 
 		// now normalize the path (remove "/something/..")
-		while (strpos($ret,'/../') !== false)
+		while (strpos($ret, '/../') !== false)
 		{
-			list($a_str,$b_str) = explode('/../',$ret,2);
-			$a = explode('/',$a_str);
+			list($a_str, $b_str) = explode('/../', $ret, 2);
+			$a = explode('/', $a_str);
 			array_pop($a);
-			$b = explode('/',$b_str);
-			$ret = implode('/',array_merge($a,$b));
+			$b = explode('/', $b_str);
+			$ret = implode('/', array_merge($a, $b));
 		}
-		return $ret.($query ? (strpos($url,'?')===false ? '?' : '&').$query : '');
+		return $ret.($query ? (strpos($url, '?') === false ? '?' : '&').$query : '');
 	}
 
 	/**
@@ -1410,11 +1407,11 @@ 
 	 */
 	static function build_url(array $url_parts)
 	{
-		$url = (!isset($url_parts['scheme'])?'':$url_parts['scheme'].'://'.
-			(!isset($url_parts['user'])?'':$url_parts['user'].(!isset($url_parts['pass'])?'':':'.$url_parts['pass']).'@').
+		$url = (!isset($url_parts['scheme']) ? '' : $url_parts['scheme'].'://'.
+			(!isset($url_parts['user']) ? '' : $url_parts['user'].(!isset($url_parts['pass']) ? '' : ':'.$url_parts['pass']).'@').
 			$url_parts['host']).$url_parts['path'].
-			(!isset($url_parts['query'])?'':'?'.$url_parts['query']).
-			(!isset($url_parts['fragment'])?'':'?'.$url_parts['fragment']);
+			(!isset($url_parts['query']) ? '' : '?'.$url_parts['query']).
+			(!isset($url_parts['fragment']) ? '' : '?'.$url_parts['fragment']);
 		//error_log(__METHOD__.'('.array2string($url_parts).") = '".$url."'");
 		return $url;
 	}
@@ -1432,19 +1429,19 @@ 
 	 * @todo get $force_download working through webdav
 	 * @return string
 	 */
-	static function download_url($path,$force_download=false)
+	static function download_url($path, $force_download = false)
 	{
-		if (($url = self::_call_on_backend('download_url',array($path,$force_download),true)))
+		if (($url = self::_call_on_backend('download_url', array($path, $force_download), true)))
 		{
 			return $url;
 		}
 		if ($path[0] != '/')
 		{
-			$path = self::parse_url($path,PHP_URL_PATH);
+			$path = self::parse_url($path, PHP_URL_PATH);
 		}
 		// we do NOT need to encode % itself, as our path are already url encoded, with the exception of ' ' and '+'
 		// we urlencode double quotes '"', as that fixes many problems in html markup
-		return '/webdav.php'.strtr($path,array('+' => '%2B',' ' => '%20','"' => '%22')).($force_download ? '?download' : '');
+		return '/webdav.php'.strtr($path, array('+' => '%2B', ' ' => '%20', '"' => '%22')).($force_download ? '?download' : '');
 	}
 
 	/**
@@ -1472,12 +1469,12 @@ 
 		// eg: User selected /home/nathan/picture.jpg, /home/Pictures/logo.jpg
 		// We want /home
 		$dirs = array();
-		foreach($_files as $file)
+		foreach ($_files as $file)
 		{
 			$dirs[] = self::dirname($file);
 		}
 		$paths = array_unique($dirs);
-		if(count($paths) > 0)
+		if (count($paths) > 0)
 		{
 			// Shortest to longest
 			usort($paths, function($a, $b) {
@@ -1485,13 +1482,13 @@ 
 			});
 
 			// Start with shortest, pop off sub-directories that don't match
-			$parts = explode('/',$paths[0]);
-			foreach($paths as $path)
+			$parts = explode('/', $paths[0]);
+			foreach ($paths as $path)
 			{
-				$dirs = explode('/',$path);
-				foreach($dirs as $dir_index => $dir)
+				$dirs = explode('/', $path);
+				foreach ($dirs as $dir_index => $dir)
 				{
-					if($parts[$dir_index] && $parts[$dir_index] != $dir)
+					if ($parts[$dir_index] && $parts[$dir_index] != $dir)
 					{
 						unset($parts[$dir_index]);
 					}
@@ -1510,24 +1507,23 @@ 
 			// Linux
 			'/',
 			// Windows
-			'\\','?','%','*',':','|',/*'.',*/ '"','<','>'
+			'\\', '?', '%', '*', ':', '|', /*'.',*/ '"', '<', '>'
 		);
 
 		// A nice name for the user,
-		$filename = $GLOBALS['egw_info']['server']['site_title'] . '_' .
-			str_replace($replace,'_',(
+		$filename = $GLOBALS['egw_info']['server']['site_title'].'_'.
+			str_replace($replace, '_', (
 			$name ? $name : (
 			count($_files) == 1 ?
 			// Just one file (hopefully a directory?) selected
-			self::basename($_files[0]) :
-			// Use the lowest common directory (eg: Infolog, Open, nathan)
+			self::basename($_files[0]) : // Use the lowest common directory (eg: Infolog, Open, nathan)
 			self::basename($base_dir))
-		)) . '.zip';
+		)).'.zip';
 
 		// Make sure basename is a dir
-		if(substr($base_dir, -1) != '/')
+		if (substr($base_dir, -1) != '/')
 		{
-			$base_dir .='/';
+			$base_dir .= '/';
 		}
 
 		// Go into directories, find them all
@@ -1538,37 +1534,37 @@ 
 		$tempfiles = array();
 
 		// Give 1 second per file, but try to allow more time for big files when amount of files is low
-		set_time_limit((count($files)<=9?10:count($files)));
+		set_time_limit((count($files) <= 9 ? 10 : count($files)));
 
 		// Add files to archive
-		foreach($files as &$addfile)
+		foreach ($files as &$addfile)
 		{
 			// Use relative paths inside zip
 			$relative = substr($addfile, strlen($base_dir));
 
 			// Use safe names - replace unsafe chars, convert to ASCII (ZIP spec says CP437, but we'll try)
-			$path = explode('/',$relative);
-			$_name = Translation::convert(Translation::to_ascii(implode('/', str_replace($replace,'_',$path))),false,'ASCII');
+			$path = explode('/', $relative);
+			$_name = Translation::convert(Translation::to_ascii(implode('/', str_replace($replace, '_', $path))), false, 'ASCII');
 
 			// Don't go infinite with app entries
-			if(self::is_link($addfile))
+			if (self::is_link($addfile))
 			{
-				if(in_array($addfile, $links)) continue;
+				if (in_array($addfile, $links)) continue;
 				$links[] = $addfile;
 			}
 			// Add directory - if empty, client app might not show it though
-			if(self::is_dir($addfile))
+			if (self::is_dir($addfile))
 			{
 				// Zip directories
 				$zip->addEmptyDir($addfile);
 			}
-			else if(self::is_readable($addfile))
+			else if (self::is_readable($addfile))
 			{
 				// Copy to temp file, as ZipArchive fails to read VFS
 				$temp = tempnam($GLOBALS['egw_info']['server']['temp_dir'], 'zip_');
-				$from = self::fopen($addfile,'r');
-		 		$to = fopen($temp,'w');
-				if(!stream_copy_to_stream($from,$to) || !$zip->addFile($temp, $_name))
+				$from = self::fopen($addfile, 'r');
+		 		$to = fopen($temp, 'w');
+				if (!stream_copy_to_stream($from, $to) || !$zip->addFile($temp, $_name))
 				{
 					unlink($temp);
 					trigger_error("Could not add $addfile to ZIP file", E_USER_ERROR);
@@ -1579,10 +1575,10 @@ 
 
 				// Add comment in
 				$props = self::propfind($addfile);
-				if($props)
+				if ($props)
 				{
-					$comment = self::find_prop($props,'comment');
-					if($comment)
+					$comment = self::find_prop($props, 'comment');
+					if ($comment)
 					{
 						$zip->setCommentName($_name, $comment);
 					}
@@ -1592,34 +1588,34 @@ 
 		}
 
 		// Set a comment to help tell them apart
-		$zip->setArchiveComment(lang('Created by %1', $GLOBALS['egw_info']['user']['account_lid']) . ' ' .DateTime::to());
+		$zip->setArchiveComment(lang('Created by %1', $GLOBALS['egw_info']['user']['account_lid']).' '.DateTime::to());
 
 		// Record total for debug, not available after close()
 		$total_files = $zip->numFiles;
 
 		$result = $zip->close();
-		if(!$result || !filesize($zip_file))
+		if (!$result || !filesize($zip_file))
 		{
 			error_log('close() result: '.array2string($result));
 			return 'Error creating zip file';
 		}
 
-		error_log("Total files: " . $total_files . " Peak memory to zip: " . self::hsize(memory_get_peak_usage(true)));
+		error_log("Total files: ".$total_files." Peak memory to zip: ".self::hsize(memory_get_peak_usage(true)));
 
 		// Stop any buffering
-		while(ob_get_level() > 0)
+		while (ob_get_level() > 0)
 		{
 			ob_end_clean();
 		}
 
 		// Stream the file to the client
 		header("Content-Type: application/zip");
-		header("Content-Length: " . filesize($zip_file));
+		header("Content-Length: ".filesize($zip_file));
 		header("Content-Disposition: attachment; filename=\"$filename\"");
 		readfile($zip_file);
 
 		unlink($zip_file);
-		foreach($tempfiles as $temp_file)
+		foreach ($tempfiles as $temp_file)
 		{
 			unlink($temp_file);
 		}
@@ -1653,11 +1649,11 @@ 
 	 * @param boolean $check_writable =true should we check if the ressource is writable, before granting locks, default yes
 	 * @return boolean true on success
 	 */
-	static function lock($path,&$token,&$timeout,&$owner,&$scope,&$type,$update=false,$check_writable=true)
+	static function lock($path, &$token, &$timeout, &$owner, &$scope, &$type, $update = false, $check_writable = true)
 	{
 		// we require write rights to lock/unlock a resource
 		if (!$path || $update && !$token || $check_writable &&
-			!(self::is_writable($path) || !self::file_exists($path) && ($dir=self::dirname($path)) && self::is_writable($dir)))
+			!(self::is_writable($path) || !self::file_exists($path) && ($dir = self::dirname($path)) && self::is_writable($dir)))
 		{
 			return false;
 		}
@@ -1671,28 +1667,28 @@ 
 
 		if ($update)	// Lock Update
 		{
-			if (($ret = (boolean)($row = self::$db->select(self::LOCK_TABLE,array('lock_owner','lock_exclusive','lock_write'),array(
+			if (($ret = (boolean)($row = self::$db->select(self::LOCK_TABLE, array('lock_owner', 'lock_exclusive', 'lock_write'), array(
 				'lock_path' => $path,
 				'lock_token' => $token,
-			),__LINE__,__FILE__)->fetch())))
+			), __LINE__, __FILE__)->fetch())))
 			{
 				$owner = $row['lock_owner'];
 				$scope = Db::from_bool($row['lock_exclusive']) ? 'exclusive' : 'shared';
 				$type  = Db::from_bool($row['lock_write']) ? 'write' : 'read';
 
-				self::$db->update(self::LOCK_TABLE,array(
+				self::$db->update(self::LOCK_TABLE, array(
 					'lock_expires' => $timeout,
 					'lock_modified' => time(),
-				),array(
+				), array(
 					'lock_path' => $path,
 					'lock_token' => $token,
-				),__LINE__,__FILE__);
+				), __LINE__, __FILE__);
 			}
 		}
 		// HTTP_WebDAV_Server does this check before calling LOCK, but we want to be complete and usable outside WebDAV
-		elseif(($lock = self::checkLock($path)) && ($lock['scope'] == 'exclusive' || $scope == 'exclusive'))
+		elseif (($lock = self::checkLock($path)) && ($lock['scope'] == 'exclusive' || $scope == 'exclusive'))
 		{
-			$ret = false;	// there's alread a lock
+			$ret = false; // there's alread a lock
 		}
 		else
 		{
@@ -1707,7 +1703,7 @@ 
 				$token = HTTP_WebDAV_Server::_new_locktoken();
 			}
 			try {
-				self::$db->insert(self::LOCK_TABLE,array(
+				self::$db->insert(self::LOCK_TABLE, array(
 					'lock_token' => $token,
 					'lock_path'  => $path,
 					'lock_created' => time(),
@@ -1716,12 +1712,12 @@ 
 					'lock_expires' => $timeout,
 					'lock_exclusive' => $scope == 'exclusive',
 					'lock_write' => $type == 'write',
-				),false,__LINE__,__FILE__);
+				), false, __LINE__, __FILE__);
 				$ret = true;
 			}
-			catch(Db\Exception $e) {
+			catch (Db\Exception $e) {
 				unset($e);
-				$ret = false;	// there's already a lock
+				$ret = false; // there's already a lock
 			}
 		}
 		if (self::LOCK_DEBUG) error_log(__METHOD__."($path,$token,$timeout,$owner,$scope,$type,update=$update,check_writable=$check_writable) returns ".($ret ? 'true' : 'false'));
@@ -1736,17 +1732,17 @@ 
 	 * @param boolean $check_writable =true should we check if the ressource is writable, before granting locks, default yes
      * @return boolean true on success
      */
-    static function unlock($path,$token,$check_writable=true)
+    static function unlock($path, $token, $check_writable = true)
     {
 		// we require write rights to lock/unlock a resource
 		if ($check_writable && !self::is_writable($path))
 		{
 			return false;
 		}
-        if (($ret = self::$db->delete(self::LOCK_TABLE,array(
+        if (($ret = self::$db->delete(self::LOCK_TABLE, array(
         	'lock_path' => $path,
         	'lock_token' => $token,
-        ),__LINE__,__FILE__) && self::$db->affected_rows()))
+        ), __LINE__, __FILE__) && self::$db->affected_rows()))
         {
         	// remove the lock from the cache too
         	unset(self::$lock_cache[$path]);
@@ -1765,31 +1761,31 @@ 
 	{
 		if (isset(self::$lock_cache[$path]))
 		{
-			if (self::LOCK_DEBUG) error_log(__METHOD__."($path) returns from CACHE ".str_replace(array("\n",'    '),'',print_r(self::$lock_cache[$path],true)));
+			if (self::LOCK_DEBUG) error_log(__METHOD__."($path) returns from CACHE ".str_replace(array("\n", '    '), '', print_r(self::$lock_cache[$path], true)));
 			return self::$lock_cache[$path];
 		}
 		$where = 'lock_path='.self::$db->quote($path);
 		// ToDo: additional check parent dirs for locks and children of the requested directory
 		//$where .= ' OR '.self::$db->quote($path).' LIKE '.self::$db->concat('lock_path',"'%'").' OR lock_path LIKE '.self::$db->quote($path.'%');
 		// ToDo: shared locks can return multiple rows
-		if (($result = self::$db->select(self::LOCK_TABLE,'*',$where,__LINE__,__FILE__)->fetch()))
+		if (($result = self::$db->select(self::LOCK_TABLE, '*', $where, __LINE__, __FILE__)->fetch()))
 		{
-			$result = Db::strip_array_keys($result,'lock_');
+			$result = Db::strip_array_keys($result, 'lock_');
 			$result['type']  = Db::from_bool($result['write']) ? 'write' : 'read';
 			$result['scope'] = Db::from_bool($result['exclusive']) ? 'exclusive' : 'shared';
 			$result['depth'] = Db::from_bool($result['recursive']) ? 'infinite' : 0;
 		}
 		if ($result && $result['expires'] < time())	// lock is expired --> remove it
 		{
-	        self::$db->delete(self::LOCK_TABLE,array(
+	        self::$db->delete(self::LOCK_TABLE, array(
 	        	'lock_path' => $result['path'],
 	        	'lock_token' => $result['token'],
-	        ),__LINE__,__FILE__);
+	        ), __LINE__, __FILE__);
 
-			if (self::LOCK_DEBUG) error_log(__METHOD__."($path) lock is expired at ".date('Y-m-d H:i:s',$result['expires'])." --> removed");
+			if (self::LOCK_DEBUG) error_log(__METHOD__."($path) lock is expired at ".date('Y-m-d H:i:s', $result['expires'])." --> removed");
 	        $result = false;
 		}
-		if (self::LOCK_DEBUG) error_log(__METHOD__."($path) returns ".($result?array2string($result):'false'));
+		if (self::LOCK_DEBUG) error_log(__METHOD__."($path) returns ".($result ?array2string($result) : 'false'));
 		return self::$lock_cache[$path] = $result;
 	}
 
@@ -1800,10 +1796,10 @@ 
 	 * @param array $content =null
 	 * @return array|boolean array with values for keys 'data','etemplate','name','label','help' or false if not supported by backend
 	 */
-	static function getExtraInfo($path,array $content=null)
+	static function getExtraInfo($path, array $content = null)
 	{
 		$extra = array();
-		if (($extra_info = self::_call_on_backend('extra_info',array($path,$content),true)))	// true = fail silent if backend does NOT support it
+		if (($extra_info = self::_call_on_backend('extra_info', array($path, $content), true)))	// true = fail silent if backend does NOT support it
 		{
 			$extra[] = $extra_info;
 		}
@@ -1814,7 +1810,7 @@ 
 			'content' => $content,
 		))))
 		{
-			foreach($vfs_extra as $data)
+			foreach ($vfs_extra as $data)
 			{
 				$extra = $extra ? array_merge($extra, $data) : $data;
 			}
@@ -1829,7 +1825,7 @@ 
 	 * @param int|string $id
 	 * @return string
 	 */
-	static function app_entry_lock_path($app,$id)
+	static function app_entry_lock_path($app, $id)
 	{
 		return "/apps/$app/entry/$id";
 	}
@@ -1843,7 +1839,7 @@ 
 		//'%' => '%25',	// % should be encoded, but easily leads to double encoding, therefore better NOT encodig it
 		'#' => '%23',
 		'?' => '%3F',
-		'/' => '',	// better remove it completly
+		'/' => '', // better remove it completly
 	);
 
 	/**
@@ -1858,7 +1854,7 @@ 
 	 */
 	static public function encodePathComponent($component)
 	{
-		return str_replace(array_keys(self::$encode),array_values(self::$encode),$component);
+		return str_replace(array_keys(self::$encode), array_values(self::$encode), $component);
 	}
 
 	/**
@@ -1871,7 +1867,7 @@ 
 	 */
 	static public function encodePath($path)
 	{
-		return implode('/',self::encodePathComponent(explode('/',$path)));
+		return implode('/', self::encodePathComponent(explode('/', $path)));
 	}
 
 	/**
@@ -1926,7 +1922,7 @@ 
 		{
 			$image = Image::find($mime_sub, 'navbar');
 		}
-		else if ($file && $mime_main == 'image' && in_array($mime_sub, array('png','jpeg','jpg','gif','bmp')) &&
+		else if ($file && $mime_main == 'image' && in_array($mime_sub, array('png', 'jpeg', 'jpg', 'gif', 'bmp')) &&
 		         (string)$GLOBALS['egw_info']['server']['link_list_thumbnail'] != '0' &&
 		         (string)$GLOBALS['egw_info']['user']['preferences']['common']['link_list_thumbnail'] != '0' &&
 		         ($stat = self::stat($file)) && $stat['size'] < 1500000)
@@ -1981,7 +1977,7 @@ 
 	 * @param array& $copied =null on return files copied
 	 * @return boolean true for no errors, false otherwise
 	 */
-	static public function copy_files(array $src, $dst, &$errs=null, array &$copied=null)
+	static public function copy_files(array $src, $dst, &$errs = null, array &$copied = null)
 	{
 		if (self::is_dir($dst))
 		{
@@ -1999,7 +1995,7 @@ 
 						if ($file !== $target)
 						{
 							// Create the target directory
-							self::mkdir($target,null,STREAM_MKDIR_RECURSIVE);
+							self::mkdir($target, null, STREAM_MKDIR_RECURSIVE);
 
 							$copied[] = $file;
 							$copied[] = $target; // < newly created folder must not be copied again!
@@ -2043,7 +2039,7 @@ 
 		if (self::is_dir($dst))
 		{
 			$vfs = new Vfs\StreamWrapper();
-			foreach($src as $file)
+			foreach ($src as $file)
 			{
 				$target = self::concat($dst, self::basename($file));
 
@@ -2075,7 +2071,7 @@ 
 	 * @param boolean $check_is_uploaded_file =true should method perform an is_uploaded_file check, default yes
 	 * @return boolean|array stat array on success, false on error
 	 */
-	static public function copy_uploaded($src,$target,$props=null,$check_is_uploaded_file=true)
+	static public function copy_uploaded($src, $target, $props = null, $check_is_uploaded_file = true)
 	{
 		$tmp_name = is_array($src) ? $src['tmp_name'] : $src;
 
@@ -2095,12 +2091,12 @@ 
 		}
 		if ($props)
 		{
-			if (!is_array($props)) $props = array(array('name' => 'comment','val' => $props));
+			if (!is_array($props)) $props = array(array('name' => 'comment', 'val' => $props));
 
 			// if $props is name => value pairs, convert it to internal array or array with values for keys 'name', 'val' and optional 'ns'
 			if (!isset($props[0]))
 			{
-				foreach($props as $name => $val)
+				foreach ($props as $name => $val)
 				{
 					if (($name == 'comment' || $name[0] == '#') && $val)	// only copy 'comment' and cfs
 					{
@@ -2118,7 +2114,7 @@ 
 			// set props before copying the file, so notifications already contain them
 			if (!self::stat($target))
 			{
-				self::touch($target);	// create empty file, to be able to attach properties
+				self::touch($target); // create empty file, to be able to attach properties
 				// tell vfs stream-wrapper to treat file in following copy as a new file notification-wises
 				$context = stream_context_create(array(
 					self::SCHEME => array('treat_as_new' => true)
@@ -2136,8 +2132,7 @@ 
 		}
 		else
 		{
-			$ret = ($context ? copy($tmp_name, self::PREFIX.$target, $context) :
-				copy($tmp_name, self::PREFIX.$target)) ?
+			$ret = ($context ? copy($tmp_name, self::PREFIX.$target, $context) : copy($tmp_name, self::PREFIX.$target)) ?
 				self::stat($target) : false;
 		}
 		if (self::LOG_LEVEL > 1 || !$ret && self::LOG_LEVEL) error_log(__METHOD__."($tmp_name, $target, ".array2string($props).") returning ".array2string($ret));
@@ -2183,7 +2178,7 @@ 
 	 * @param array|boolean &$stat=null on return: stat of existing file or false for non-existing files
 	 * @return string|boolean false if the url cant be resolved, should not happen if fstab has a root entry
 	 */
-	static function resolve_url_symlinks($_path,$file_exists=true,$resolve_last_symlink=true,&$stat=null)
+	static function resolve_url_symlinks($_path, $file_exists = true, $resolve_last_symlink = true, &$stat = null)
 	{
 		$vfs = new Vfs\StreamWrapper();
 		return $vfs->resolve_url_symlinks($_path, $file_exists, $resolve_last_symlink, $stat);
@@ -2199,7 +2194,7 @@ 
 	 * @param boolean $fix_url_query =false true append relativ path to url query parameter, default not
 	 * @return string|boolean false if the url cant be resolved, should not happen if fstab has a root entry
 	 */
-	static function resolve_url($_path,$do_symlink=true,$use_symlinkcache=true,$replace_user_pass_host=true,$fix_url_query=false)
+	static function resolve_url($_path, $do_symlink = true, $use_symlinkcache = true, $replace_user_pass_host = true, $fix_url_query = false)
 	{
 		return Vfs\StreamWrapper::resolve_url($_path, $do_symlink, $use_symlinkcache, $replace_user_pass_host, $fix_url_query);
 	}
@@ -2215,7 +2210,7 @@ 
 	 * @param boolean $recursive =false true: create missing parents too
 	 * @return boolean TRUE on success or FALSE on failure
 	 */
-	static function mkdir ($path, $mode=0750, $recursive=false)
+	static function mkdir($path, $mode = 0750, $recursive = false)
 	{
 		return $path[0] == '/' && mkdir(self::PREFIX.$path, $mode, $recursive);
 	}
@@ -2244,7 +2239,7 @@ 
 	 * @param string $path
 	 * @return boolean TRUE on success or FALSE on failure
 	 */
-	static function unlink ( $path )
+	static function unlink($path)
 	{
 		return $path[0] == '/' && unlink(self::PREFIX.$path);
 	}
@@ -2261,60 +2256,60 @@ 
 	 * @param int $path_param_key =0 key in params containing the path, default 0
 	 * @return mixed return value of backend or false if function does not exist on backend
 	 */
-	static protected function _call_on_backend($name,$params,$fail_silent=false,$path_param_key=0)
+	static protected function _call_on_backend($name, $params, $fail_silent = false, $path_param_key = 0)
 	{
 		$pathes = $params[$path_param_key];
 
 		$scheme2urls = array();
-		foreach(is_array($pathes) ? $pathes : array($pathes) as $path)
+		foreach (is_array($pathes) ? $pathes : array($pathes) as $path)
 		{
-			if (!($url = self::resolve_url_symlinks($path,false,false)))
+			if (!($url = self::resolve_url_symlinks($path, false, false)))
 			{
 				return false;
 			}
-			$k=(string)self::parse_url($url,PHP_URL_SCHEME);
+			$k = (string)self::parse_url($url, PHP_URL_SCHEME);
 			if (!(is_array($scheme2urls[$k]))) $scheme2urls[$k] = array();
 			$scheme2urls[$k][$path] = $url;
 		}
 		$ret = array();
-		foreach($scheme2urls as $scheme => $urls)
+		foreach ($scheme2urls as $scheme => $urls)
 		{
 			if ($scheme)
 			{
-				if (!class_exists($class = self::scheme2class($scheme)) || !method_exists($class,$name))
+				if (!class_exists($class = self::scheme2class($scheme)) || !method_exists($class, $name))
 				{
-					if (!$fail_silent) trigger_error("Can't $name for scheme $scheme!\n",E_USER_WARNING);
+					if (!$fail_silent) trigger_error("Can't $name for scheme $scheme!\n", E_USER_WARNING);
 					return false;
 				}
 				if (!is_array($pathes))
 				{
 					$params[$path_param_key] = $url;
 
-					return call_user_func_array(array($class,$name),$params);
+					return call_user_func_array(array($class, $name), $params);
 				}
 				$params[$path_param_key] = $urls;
-				if (!is_array($r = call_user_func_array(array($class,$name),$params)))
+				if (!is_array($r = call_user_func_array(array($class, $name), $params)))
 				{
 					return $r;
 				}
 				// we need to re-translate the urls to pathes, as they can eg. contain symlinks
-				foreach($urls as $path => $url)
+				foreach ($urls as $path => $url)
 				{
-					if (isset($r[$url]) || isset($r[$url=self::parse_url($url,PHP_URL_PATH)]))
+					if (isset($r[$url]) || isset($r[$url = self::parse_url($url, PHP_URL_PATH)]))
 					{
 						$ret[$path] = $r[$url];
 					}
 				}
 			}
 			// call the filesystem specific function (dont allow to use arrays!)
-			elseif(!function_exists($name) || is_array($pathes))
+			elseif (!function_exists($name) || is_array($pathes))
 			{
 				return false;
 			}
 			else
 			{
 				$time = null;
-				return $name($url,$time);
+				return $name($url, $time);
 			}
 		}
 		return $ret;
@@ -2328,7 +2323,7 @@ 
 	 * @param int $atime =null access time (unix timestamp), default null = current time, not implemented in the vfs!
 	 * @return boolean true on success, false otherwise
 	 */
-	static function touch($path,$time=null,$atime=null)
+	static function touch($path, $time = null, $atime = null)
 	{
 		return $path[0] == '/' && touch(self::PREFIX.$path, $time, $atime);
 	}
@@ -2342,7 +2337,7 @@ 
 	 * @param string $mode mode string see Vfs::mode2int
 	 * @return boolean true on success, false otherwise
 	 */
-	static function chmod($path,$mode)
+	static function chmod($path, $mode)
 	{
 		return $path[0] == '/' && chmod(self::PREFIX.$path, $mode);
 	}
@@ -2356,7 +2351,7 @@ 
 	 * @param int|string $owner numeric user id or account-name
 	 * @return boolean true on success, false otherwise
 	 */
-	static function chown($path,$owner)
+	static function chown($path, $owner)
 	{
 		return $path[0] == '/' && chown(self::PREFIX.$path, is_numeric($owner) ? abs($owner) : $owner);
 	}
@@ -2370,7 +2365,7 @@ 
 	 * @param int|string $group numeric group id or group-name
 	 * @return boolean true on success, false otherwise
 	 */
-	static function chgrp($path,$group)
+	static function chgrp($path, $group)
 	{
 		return $path[0] == '/' && chgrp(self::PREFIX.$path, is_numeric($group) ? abs($group) : $group);
 	}
@@ -2385,7 +2380,7 @@ 
 	 */
 	static function readlink($path)
 	{
-		$ret = self::_call_on_backend('readlink',array($path),true);	// true = fail silent, if backend does not support readlink
+		$ret = self::_call_on_backend('readlink', array($path), true); // true = fail silent, if backend does not support readlink
 		//error_log(__METHOD__."('$path') returning ".array2string($ret).' '.function_backtrace());
 		return $ret;
 	}
@@ -2399,9 +2394,9 @@ 
 	 * @param string $link path of the link to create
 	 * @return boolean true on success, false on error
 	 */
-	static function symlink($target,$link)
+	static function symlink($target, $link)
 	{
-		if (($ret = self::_call_on_backend('symlink',array($target,$link),false,1)))	// 1=path is in $link!
+		if (($ret = self::_call_on_backend('symlink', array($target, $link), false, 1)))	// 1=path is in $link!
 		{
 			Vfs\StreamWrapper::symlinkCache_remove($link);
 		}
@@ -2421,13 +2416,13 @@ 
 	 * @param boolean $recheck =false true = do a new check, false = rely on stored mime type (if existing)
 	 * @return string mime-type (self::DIR_MIME_TYPE for directories)
 	 */
-	static function mime_content_type($path,$recheck=false)
+	static function mime_content_type($path, $recheck = false)
 	{
 		if (!($url = self::resolve_url_symlinks($path)))
 		{
 			return false;
 		}
-		if (($scheme = self::parse_url($url,PHP_URL_SCHEME)) && !$recheck)
+		if (($scheme = self::parse_url($url, PHP_URL_SCHEME)) && !$recheck)
 		{
 			// check it it's an eGW stream wrapper returning mime-type via url_stat
 			// we need to first check if the constant is defined, as we get a fatal error in php5.3 otherwise
@@ -2436,7 +2431,7 @@ 
 				($mime_attr = constant($class.'::STAT_RETURN_MIME_TYPE')))
 			{
 				$inst = new $class;
-				$stat = $inst->url_stat(self::parse_url($url,PHP_URL_PATH),0);
+				$stat = $inst->url_stat(self::parse_url($url, PHP_URL_PATH), 0);
 				if ($stat && $stat[$mime_attr])
 				{
 					$mime = $stat[$mime_attr];
@@ -2455,7 +2450,7 @@ 
 		// using EGw's own mime magic (currently only checking the extension!)
 		if (!$mime)
 		{
-			$mime = MimeMagic::filename2mime(self::parse_url($url,PHP_URL_PATH));
+			$mime = MimeMagic::filename2mime(self::parse_url($url, PHP_URL_PATH));
 		}
 		//error_log(__METHOD__."($path,$recheck) mime=$mime");
 		return $mime;
@@ -2485,7 +2480,7 @@ 
 	 *
 	 * @param string $path ='/' path of backend, whos cache to clear
 	 */
-	static function clearstatcache($path='/')
+	static function clearstatcache($path = '/')
 	{
 		//error_log(__METHOD__."('$path')");
 		Vfs\StreamWrapper::clearstatcache($path);
@@ -2505,7 +2500,7 @@ 
 	 * @param string $path_to
 	 * @return boolean TRUE on success or FALSE on failure
 	 */
-	static function rename ( $path_from, $path_to )
+	static function rename($path_from, $path_to)
 	{
 		$vfs = new Vfs\StreamWrapper();
 		return $vfs->rename($path_from, $path_to);