EGroupware / egroupware

api/src/Mail.php

Braces +1775 added lines, -524 removed lines

@@ -198,7 +198,10 @@ 
 	public static function getInstance($_restoreSession=true, &$_profileID=0, $_validate=true, $_oldImapServerObject=false, $_reuseCache=null)
 	{
 		//$_restoreSession=false;
-		if (is_null($_reuseCache)) $_reuseCache = $_restoreSession;
+		if (is_null($_reuseCache))
+		{
+			$_reuseCache = $_restoreSession;
+		}
 		//error_log(__METHOD__.' ('.__LINE__.') '.' RestoreSession:'.$_restoreSession.' ProfileId:'.$_profileID.'/'.Mail\Account::get_default_acc_id().' for user:'.$GLOBALS['egw_info']['user']['account_lid'].' called from:'.function_backtrace());
 		//error_log(__METHOD__.' ('.__LINE__.') '.array2string($_oldImapServerObject));
 		if ($_oldImapServerObject instanceof Mail\Imap)
@@ -224,9 +227,15 @@ 
 			{
 				$profileID = Mail\Account::get_default_acc_id();
 			}
-			if ($profileID!=$_profileID) $_restoreSession==false;
+			if ($profileID!=$_profileID)
+			{
+				$_restoreSession==false;
+			}
 			$_profileID=$profileID;
-			if (self::$debug) error_log(__METHOD__.' ('.__LINE__.') '.' called with profileID==0 using '.$profileID.' instead->'.function_backtrace());
+			if (self::$debug)
+			{
+				error_log(__METHOD__.' ('.__LINE__.') '.' called with profileID==0 using '.$profileID.' instead->'.function_backtrace());
+			}
 		}
 		// no validation or restoreSession for old ImapServer Object, just fetch it and return it
 		if ($_oldImapServerObject===true)
@@ -265,7 +274,8 @@ 
 				// TODO: merge mailprefs into userprefs, for easy treatment
 				self::$instances[$_profileID]->mailPreferences = $GLOBALS['egw_info']['user']['preferences']['mail'];
 				self::$instances[$_profileID]->htmlOptions  = self::$instances[$_profileID]->mailPreferences['htmlOptions'];
-			} catch (\Exception $e)
+			}
+			catch (\Exception $e)
 			{
 				$newprofileID = Mail\Account::get_default_acc_id();
 				// try loading the default profile for the user
@@ -283,9 +293,15 @@ 
 			self::storeActiveProfileIDToPref(self::$instances[$_profileID]->icServer, $_profileID, $_validate );
 		}
 		self::$instances[$_profileID]->profileID = $_profileID;
-		if (!isset(self::$instances[$_profileID]->idna2)) self::$instances[$_profileID]->idna2 = new Horde_Idna;
+		if (!isset(self::$instances[$_profileID]->idna2))
+		{
+			self::$instances[$_profileID]->idna2 = new Horde_Idna;
+		}
 		//if ($_profileID==0); error_log(__METHOD__.' ('.__LINE__.') '.' RestoreSession:'.$_restoreSession.' ProfileId:'.$_profileID);
-		if (is_null(self::$mailConfig)) self::$mailConfig = Config::read('mail');
+		if (is_null(self::$mailConfig))
+		{
+			self::$mailConfig = Config::read('mail');
+		}
 		return self::$instances[$_profileID];
 	}
 
@@ -310,14 +326,20 @@ 
 			}
 			catch (\Exception $e)
 			{
-				if ($_profileID != Mail\Account::get_default_acc_id()) $_profileID = Mail\Account::get_default_acc_id();
+				if ($_profileID != Mail\Account::get_default_acc_id())
+				{
+					$_profileID = Mail\Account::get_default_acc_id();
+				}
 				error_log(__METHOD__.__LINE__.' '.$e->getMessage());
 				return false;
 			}
 		}
 		if ($oldProfileID != $_profileID)
 		{
-			if ($oldProfileID && $_profileID==0) $_profileID = $oldProfileID;
+			if ($oldProfileID && $_profileID==0)
+			{
+				$_profileID = $oldProfileID;
+			}
 			$GLOBALS['egw']->preferences->add('mail','ActiveProfileID',$_profileID,'user');
 			// save prefs
 			$GLOBALS['egw']->preferences->save_repository(true);
@@ -347,11 +369,17 @@ 
 				{
 					return $_acc_id;
 				}
-				if (self::$debug) error_log(__METHOD__."($_acc_id) account NOT valid, no imap-host!");
+				if (self::$debug)
+				{
+					error_log(__METHOD__."($_acc_id) account NOT valid, no imap-host!");
+				}
 			}
 			catch (\Exception $e) {
 				unset($e);
-				if (self::$debug) error_log(__METHOD__."($_acc_id) account NOT found!");
+				if (self::$debug)
+				{
+					error_log(__METHOD__."($_acc_id) account NOT found!");
+				}
 			}
 		}
 		// no account specified or specified account not found or not valid
@@ -360,11 +388,17 @@ 
 		{
 			if (!empty($imap_host) && ($account = Mail\Account::read($acc_id)) && $account->is_imap())
 			{
-				if (self::$debug && $_acc_id) error_log(__METHOD__."($_acc_id) using $acc_id instead");
+				if (self::$debug && $_acc_id)
+				{
+					error_log(__METHOD__."($_acc_id) using $acc_id instead");
+				}
 				return $acc_id;
 			}
 		}
-		if (self::$debug) error_log(__METHOD__."($_acc_id) NO valid account found!");
+		if (self::$debug)
+		{
+			error_log(__METHOD__."($_acc_id) NO valid account found!");
+		}
 		return 0;
 	}
 
@@ -380,10 +414,19 @@ 
 	 */
 	private function __construct($_displayCharset='utf-8',$_restoreSession=true, $_profileID=0, $_oldImapServerObject=false, $_reuseCache=null)
 	{
-		if (is_null($_reuseCache)) $_reuseCache = $_restoreSession;
-		if (!empty($_displayCharset)) self::$displayCharset = $_displayCharset;
+		if (is_null($_reuseCache))
+		{
+			$_reuseCache = $_restoreSession;
+		}
+		if (!empty($_displayCharset))
+		{
+			self::$displayCharset = $_displayCharset;
+		}
 		// not nummeric, we assume we only want an empty class object
-		if (!is_numeric($_profileID)) return true;
+		if (!is_numeric($_profileID))
+		{
+			return true;
+		}
 		if ($_restoreSession)
 		{
 			//error_log(__METHOD__." Session restore ".function_backtrace());
@@ -398,7 +441,10 @@ 
 			$firstMessage = $this->sessionData['previewMessage'];
 			$this->sessionData = array();
 		}
-		if (!$_reuseCache) $this->forcePrefReload($_profileID,!$_reuseCache);
+		if (!$_reuseCache)
+		{
+			$this->forcePrefReload($_profileID,!$_reuseCache);
+		}
 		try
 		{
 			$this->profileID = self::validateProfileID($_profileID);
@@ -422,7 +468,10 @@ 
 			$_profileID = $this->profileID = $GLOBALS['egw_info']['user']['preferences']['mail']['ActiveProfileID'] = $this->icServer->ImapServerId;
 		}
 
-		if (is_null(self::$mailConfig)) self::$mailConfig = Config::read('mail');
+		if (is_null(self::$mailConfig))
+		{
+			self::$mailConfig = Config::read('mail');
+		}
 	}
 
 	/**
@@ -451,7 +500,10 @@ 
 		// unset the mail_preferences session object, to force the reload/rebuild
 		Cache::setSession('mail','mail_preferences',serialize(array()));
 		Cache::setSession('emailadmin','session_data',serialize(array()));
-		if ($_resetFolderObjects) self::resetFolderObjectCache($_profile_id);
+		if ($_resetFolderObjects)
+		{
+			self::resetFolderObjectCache($_profile_id);
+		}
 	}
 
 	/**
@@ -461,7 +513,10 @@ 
 	{
 		$this->sessionData = array();//Cache::getCache(Cache::SESSION,'mail','session_data',$callback=null,$callback_params=array(),$expiration=60*60*1);
 		self::$activeFolderCache = Cache::getCache(Cache::INSTANCE,'email','activeMailbox'.trim($GLOBALS['egw_info']['user']['account_id']),null,array(),60*60*10);
-		if (!empty(self::$activeFolderCache[$this->profileID])) $this->sessionData['mailbox'] = self::$activeFolderCache[$this->profileID];
+		if (!empty(self::$activeFolderCache[$this->profileID]))
+		{
+			$this->sessionData['mailbox'] = self::$activeFolderCache[$this->profileID];
+		}
 	}
 
 	/**
@@ -470,7 +525,10 @@ 
 	function saveSessionData()
 	{
 		//error_log(__METHOD__.' ('.__LINE__.') '.array2string(array_keys($this->sessionData)));
-		if (!empty($this->sessionData['mailbox'])) self::$activeFolderCache[$this->profileID]=$this->sessionData['mailbox'];
+		if (!empty($this->sessionData['mailbox']))
+		{
+			self::$activeFolderCache[$this->profileID]=$this->sessionData['mailbox'];
+		}
 		if (isset(self::$activeFolderCache) && is_array(self::$activeFolderCache))
 		{
 			Cache::setCache(Cache::INSTANCE,'email','activeMailbox'.trim($GLOBALS['egw_info']['user']['account_id']),self::$activeFolderCache, 60*60*10);
@@ -491,8 +549,14 @@ 
 	 */
 	static function unsetCachedObjects($_profileID=null)
 	{
-		if (is_null($_profileID)) $_profileID = Mail\Account::get_default_acc_id();
-		if (is_array($_profileID) && $_profileID['account_id']) $account_id = $_profileID['account_id'];
+		if (is_null($_profileID))
+		{
+			$_profileID = Mail\Account::get_default_acc_id();
+		}
+		if (is_array($_profileID) && $_profileID['account_id'])
+		{
+			$account_id = $_profileID['account_id'];
+		}
 		//error_log(__METHOD__.__LINE__.' called with ProfileID:'.array2string($_profileID).' from '.function_backtrace());
 		if (!is_array($_profileID) && (is_numeric($_profileID) || !(stripos($_profileID,'tracker_')===false)))
 		{
@@ -525,7 +589,10 @@ 
 				Cache::setCache(Cache::INSTANCE,'email','vacationNotice'.trim($account_id),$vacationCached, $expiration=60*60*24*1);
 			}
 
-			if (isset(self::$instances[$_profileID])) unset(self::$instances[$_profileID]);
+			if (isset(self::$instances[$_profileID]))
+			{
+				unset(self::$instances[$_profileID]);
+			}
 		}
 		if (is_array($_profileID) && $_profileID['location'] == 'clear_cache')
 		{
@@ -554,7 +621,10 @@ 
 	static function resetConnectionErrorCache($_ImapServerId=null,$account_id=null)
 	{
 		//error_log(__METHOD__.' ('.__LINE__.') '.' for Profile:'.array2string($_ImapServerId) .' for user:'.trim($account_id));
-		if (is_null($account_id)) $account_id = $GLOBALS['egw_info']['user']['account_id'];
+		if (is_null($account_id))
+		{
+			$account_id = $GLOBALS['egw_info']['user']['account_id'];
+		}
 		if (is_array($_ImapServerId))
 		{
 			// called via hook
@@ -593,7 +663,10 @@ 
 	static function resetFolderObjectCache($_ImapServerId=null,$account_id=null)
 	{
 		//error_log(__METHOD__.' ('.__LINE__.') '.' called for Profile:'.array2string($_ImapServerId).'->'.function_backtrace());
-		if (is_null($account_id)) $account_id = $GLOBALS['egw_info']['user']['account_id'];
+		if (is_null($account_id))
+		{
+			$account_id = $GLOBALS['egw_info']['user']['account_id'];
+		}
 		// on [location] => verify_settings we coud either use [prefs] => Array([ActiveProfileID] => 9, .. as $_ImapServerId
 		// or treat it as not given. we try that path
 		if (is_null($_ImapServerId)||is_array($_ImapServerId))
@@ -667,10 +740,14 @@ 
 
 		$userEMailAdresses = array($acc['ident_email']=>$acc['ident_realname']);
 
-		foreach($identities as $ik => $ident) {
+		foreach($identities as $ik => $ident)
+		{
 			//error_log(__METHOD__.' ('.__LINE__.') '.':'.$ik.'->'.array2string($ident));
 			$identity = Mail\Account::read_identity($ik);
-			if (!empty($identity['ident_email']) && !isset($userEMailAdresses[$identity['ident_email']])) $userEMailAdresses[$identity['ident_email']] = $identity['ident_realname'];
+			if (!empty($identity['ident_email']) && !isset($userEMailAdresses[$identity['ident_email']]))
+			{
+				$userEMailAdresses[$identity['ident_email']] = $identity['ident_realname'];
+			}
 		}
 		//error_log(__METHOD__.' ('.__LINE__.') '.array2string($userEMailAdresses));
 		return $userEMailAdresses;
@@ -688,13 +765,20 @@ 
 		foreach(Mail\Account::search($only_current_user=($_accountToSearch?$_accountToSearch:true), $just_name=true) as $acc_id => $identity_name)
 		{
 			$acc = Mail\Account::read($acc_id,($_accountToSearch?$_accountToSearch:null));
-			if (!$resolve_placeholders) $userEMailAdresses[$acc['ident_id']] = array('acc_id'=>$acc_id,'ident_id'=>$acc['ident_id'],'ident_email'=>$acc['ident_email'],'ident_org'=>$acc['ident_org'],'ident_realname'=>$acc['ident_realname'],'ident_signature'=>$acc['ident_signature'],'ident_name'=>$acc['ident_name']);
+			if (!$resolve_placeholders)
+			{
+				$userEMailAdresses[$acc['ident_id']] = array('acc_id'=>$acc_id,'ident_id'=>$acc['ident_id'],'ident_email'=>$acc['ident_email'],'ident_org'=>$acc['ident_org'],'ident_realname'=>$acc['ident_realname'],'ident_signature'=>$acc['ident_signature'],'ident_name'=>$acc['ident_name']);
+			}
 
-			foreach(Mail\Account::identities($acc) as $ik => $ident) {
+			foreach(Mail\Account::identities($acc) as $ik => $ident)
+			{
 				//error_log(__METHOD__.' ('.__LINE__.') '.':'.$ik.'->'.array2string($ident));
 				$identity = Mail\Account::read_identity($ik,$resolve_placeholders);
 				//error_log(__METHOD__.' ('.__LINE__.') '.':'.$ik.'->'.array2string($identity));
-				if (!isset($userEMailAdresses[$identity['ident_id']])) $userEMailAdresses[$identity['ident_id']] = array('acc_id'=>$acc_id,'ident_id'=>$identity['ident_id'],'ident_email'=>$identity['ident_email'],'ident_org'=>$identity['ident_org'],'ident_realname'=>$identity['ident_realname'],'ident_signature'=>$identity['ident_signature'],'ident_name'=>$identity['ident_name']);
+				if (!isset($userEMailAdresses[$identity['ident_id']]))
+				{
+					$userEMailAdresses[$identity['ident_id']] = array('acc_id'=>$acc_id,'ident_id'=>$identity['ident_id'],'ident_email'=>$identity['ident_email'],'ident_org'=>$identity['ident_org'],'ident_realname'=>$identity['ident_realname'],'ident_signature'=>$identity['ident_signature'],'ident_name'=>$identity['ident_name']);
+				}
 			}
 		}
 		//error_log(__METHOD__.' ('.__LINE__.') '.array2string($userEMailAdresses));
@@ -714,12 +798,16 @@ 
 			$account = Mail\Account::read($account);
 		}
 		$userEMailAdresses = array();
-		foreach(Mail\Account::identities($account, true, 'params') as $ik => $ident) {
+		foreach(Mail\Account::identities($account, true, 'params') as $ik => $ident)
+		{
 			//error_log(__METHOD__.' ('.__LINE__.') '.':'.$ik.'->'.array2string($ident));
 			$identity = Mail\Account::read_identity($ik,true,null,$account);
 			//error_log(__METHOD__.' ('.__LINE__.') '.':'.$ik.'->'.array2string($identity));
 			// standardIdentity has ident_id==acc_id (as it is done within account->identities)
-			if (empty($identity['ident_id'])) $identity['ident_id'] = $identity['acc_id'];
+			if (empty($identity['ident_id']))
+			{
+				$identity['ident_id'] = $identity['acc_id'];
+			}
 			if (!isset($userEMailAdresses[$identity['ident_id']]))
 			{
 				$userEMailAdresses[$identity['ident_id']] = array('acc_id'=>$identity['acc_id'],
@@ -762,15 +850,25 @@ 
 		// account select box
 		$selectedID = $this->profileID;
 		$allAccountData = Mail\Account::search($only_current_user=true, false, null);
-		if ($allAccountData) {
+		if ($allAccountData)
+		{
 			$rememberFirst=$selectedFound=null;
 			foreach ($allAccountData as $tmpkey => $icServers)
 			{
-				if (is_null($rememberFirst)) $rememberFirst = $tmpkey;
-				if ($tmpkey == $selectedID) $selectedFound=true;
+				if (is_null($rememberFirst))
+				{
+					$rememberFirst = $tmpkey;
+				}
+				if ($tmpkey == $selectedID)
+				{
+					$selectedFound=true;
+				}
 				//error_log(__METHOD__.' ('.__LINE__.') '.' Key:'.$tmpkey.'->'.array2string($icServers->acc_imap_host));
 				$host = $icServers->acc_imap_host;
-				if (empty($host)) continue;
+				if (empty($host))
+				{
+					continue;
+				}
 				$identities[$icServers->acc_id] = $icServers['ident_realname'].' '.$icServers['ident_org'].' <'.$icServers['ident_email'].'>';
 				//error_log(__METHOD__.' ('.__LINE__.') '.' Key:'.$tmpkey.'->'.array2string($identities[$icServers->acc_id]));
 			}
@@ -830,7 +928,10 @@ 
 	 */
 	function reopen($_foldername)
 	{
-		if (self::$debugTimes) $starttime = microtime (true);
+		if (self::$debugTimes)
+		{
+			$starttime = microtime (true);
+		}
 
 		//error_log(__METHOD__.' ('.__LINE__.') '."('$_foldername') ".function_backtrace());
 		// TODO: trying to reduce traffic to the IMAP Server here, introduces problems with fetching the bodies of
@@ -840,12 +941,16 @@ 
 		//{
 			//error_log( __METHOD__.' ('.__LINE__.') '." $_foldername ".function_backtrace());
 			//error_log(__METHOD__.' ('.__LINE__.') '.' Connected with icServer for Profile:'.$this->profileID.'?'.print_r($this->icServer->_connected,true));
-			if ($this->folderIsSelectable($_foldername)) {
+			if ($this->folderIsSelectable($_foldername))
+			{
 				$this->icServer->openMailbox($_foldername);
 			}
 			$folderOpened = $_foldername;
 		//}
-		if (self::$debugTimes) self::logRunTimes($starttime,null,'Folder:'.$_foldername,__METHOD__.' ('.__LINE__.') ');
+		if (self::$debugTimes)
+		{
+			self::logRunTimes($starttime,null,'Folder:'.$_foldername,__METHOD__.' ('.__LINE__.') ');
+		}
 	}
 
 
@@ -860,13 +965,22 @@ 
 	{
 		//error_log( "-------------------------->open connection ".function_backtrace());
 		//error_log(__METHOD__.' ('.__LINE__.') '.' ->'.array2string($this->icServer));
-		if (self::$debugTimes) $starttime = microtime (true);
+		if (self::$debugTimes)
+		{
+			$starttime = microtime (true);
+		}
 		$mailbox=null;
 		try
 		{
-			if(isset($this->sessionData['mailbox'])&&$this->folderExists($this->sessionData['mailbox'])) $mailbox = $this->sessionData['mailbox'];
-			if (empty($mailbox)) $mailbox = $this->icServer->getCurrentMailbox();
-/*
+			if(isset($this->sessionData['mailbox'])&&$this->folderExists($this->sessionData['mailbox']))
+			{
+				$mailbox = $this->sessionData['mailbox'];
+			}
+			if (empty($mailbox))
+			{
+				$mailbox = $this->icServer->getCurrentMailbox();
+			}
+			/*
 			if (isset(Mail\Imap::$supports_keywords[$_icServerID]))
 			{
 				$this->icServer->openMailbox($mailbox);
@@ -891,7 +1005,10 @@ 
 			error_log(__METHOD__.' ('.__LINE__.') '."->open connection for Server with profileID:".$_icServerID." trying to examine ($mailbox) failed!".$e->getMessage());
 			throw new Exception(__METHOD__." failed to ".__METHOD__." on Profile to $_icServerID while trying to examine $mailbox:".$e->getMessage());
 		}
-		if (self::$debugTimes) self::logRunTimes($starttime,null,'ProfileID:'.$_icServerID,__METHOD__.' ('.__LINE__.') ');
+		if (self::$debugTimes)
+		{
+			self::logRunTimes($starttime,null,'ProfileID:'.$_icServerID,__METHOD__.' ('.__LINE__.') ');
+		}
 	}
 
 	/**
@@ -903,20 +1020,27 @@ 
 	function getQuotaRoot()
 	{
 		static $quota;
-		if (isset($quota)) return $quota;
+		if (isset($quota))
+		{
+			return $quota;
+		}
 		$this->icServer->getCurrentMailbox();
-		if(!$this->icServer->hasCapability('QUOTA')) {
+		if(!$this->icServer->hasCapability('QUOTA'))
+		{
 			$quota = false;
 			return false;
 		}
 		$quota = $this->icServer->getStorageQuotaRoot('INBOX');
 		//error_log(__METHOD__.' ('.__LINE__.') '.array2string($quota));
-		if(is_array($quota)) {
+		if(is_array($quota))
+		{
 			$quota = array(
 				'usage'	=> $quota['USED'],
 				'limit'	=> $quota['QMAX'],
 			);
-		} else {
+		}
+		else
+		{
 			$quota = false;
 		}
 		return $quota;
@@ -932,7 +1056,11 @@ 
 	static function getTimeOut($_use='IMAP')
 	{
 		$timeout = $GLOBALS['egw_info']['user']['preferences']['mail']['connectionTimeout'];
-		if (empty($timeout)) $timeout = ($_use=='SIEVE'?10:20); // this is the default value
+		if (empty($timeout))
+		{
+			$timeout = ($_use=='SIEVE'?10:20);
+		}
+		// this is the default value
 		return $timeout;
 	}
 
@@ -950,9 +1078,13 @@ 
 		$foldersNameSpace = array();
 		$delimiter = $this->getHierarchyDelimiter();
 		// TODO: cache by $this->icServer->ImapServerId
-		if (is_null($nameSpace)) $nameSpace = $this->icServer->getNameSpaceArray();
+		if (is_null($nameSpace))
+		{
+			$nameSpace = $this->icServer->getNameSpaceArray();
+		}
 		//error_log(__METHOD__.' ('.__LINE__.') '.array2string($nameSpace));
-		if (is_array($nameSpace)) {
+		if (is_array($nameSpace))
+		{
 			foreach($nameSpace as $type => $singleNameSpaceArray)
 			{
 				foreach ($singleNameSpaceArray as $singleNameSpace)
@@ -969,7 +1101,9 @@ 
 						$_foldersNameSpace['prefix_present'] = 'forced';
 						// uw-imap server with mailbox prefix or dovecot maybe
 						$_foldersNameSpace['prefix'] = 'mail';
-					} else {
+					}
+					else
+					{
 						$_foldersNameSpace['prefix_present'] = !empty($singleNameSpace['name']);
 						$_foldersNameSpace['prefix'] = $singleNameSpace['name'];
 					}
@@ -996,7 +1130,10 @@ 
 		foreach($nameSpace as &$singleNameSpace)
 		{
 			//if (substr($singleNameSpace['prefix'],0,strlen($folderName))==$folderName) return $singleNameSpace['prefix'];
-			if (substr($folderName,0,strlen($singleNameSpace['prefix']))==$singleNameSpace['prefix']) return $singleNameSpace['prefix'];
+			if (substr($folderName,0,strlen($singleNameSpace['prefix']))==$singleNameSpace['prefix'])
+			{
+				return $singleNameSpace['prefix'];
+			}
 		}
 		return "";
 	}
@@ -1010,8 +1147,14 @@ 
 	function getHierarchyDelimiter($_useCache=true)
 	{
 		static $HierarchyDelimiter = null;
-		if (is_null($HierarchyDelimiter)) $HierarchyDelimiter = Cache::getCache(Cache::INSTANCE,'email','HierarchyDelimiter'.trim($GLOBALS['egw_info']['user']['account_id']),null,array(),60*60*24*5);
-		if ($_useCache===false) unset($HierarchyDelimiter[$this->icServer->ImapServerId]);
+		if (is_null($HierarchyDelimiter))
+		{
+			$HierarchyDelimiter = Cache::getCache(Cache::INSTANCE,'email','HierarchyDelimiter'.trim($GLOBALS['egw_info']['user']['account_id']),null,array(),60*60*24*5);
+		}
+		if ($_useCache===false)
+		{
+			unset($HierarchyDelimiter[$this->icServer->ImapServerId]);
+		}
 		if (isset($HierarchyDelimiter[$this->icServer->ImapServerId])&&!empty($HierarchyDelimiter[$this->icServer->ImapServerId]))
 		{
 			return $HierarchyDelimiter[$this->icServer->ImapServerId];
@@ -1040,14 +1183,19 @@ 
 	{
 		//error_log(__METHOD__.' ('.__LINE__.') '.':'.$this->icServer->ImapServerId.' Connected:'.$this->icServer->_connected);
 		static $_specialUseFolders = null;
-		if (is_null($_specialUseFolders)||empty($_specialUseFolders)) $_specialUseFolders = Cache::getCache(Cache::INSTANCE,'email','specialUseFolders'.trim($GLOBALS['egw_info']['user']['account_id']),null,array(),60*60*24*5);
+		if (is_null($_specialUseFolders)||empty($_specialUseFolders))
+		{
+			$_specialUseFolders = Cache::getCache(Cache::INSTANCE,'email','specialUseFolders'.trim($GLOBALS['egw_info']['user']['account_id']),null,array(),60*60*24*5);
+		}
 		//error_log(__METHOD__.' ('.__LINE__.') '.array2string($this->icServer->acc_folder_trash));
 		//error_log(__METHOD__.' ('.__LINE__.') '.array2string($this->icServer->acc_folder_sent));
 		//error_log(__METHOD__.' ('.__LINE__.') '.array2string($this->icServer->acc_folder_draft));
 		//error_log(__METHOD__.' ('.__LINE__.') '.array2string($this->icServer->acc_folder_template));
 		self::$specialUseFolders = $_specialUseFolders[$this->icServer->ImapServerId];
 		if (isset($_specialUseFolders[$this->icServer->ImapServerId]) && !empty($_specialUseFolders[$this->icServer->ImapServerId]))
-			return $_specialUseFolders[$this->icServer->ImapServerId];
+		{
+					return $_specialUseFolders[$this->icServer->ImapServerId];
+		}
 		$_specialUseFolders[$this->icServer->ImapServerId]=array();
 		//if (!empty($this->icServer->acc_folder_trash) && !isset($_specialUseFolders[$this->icServer->ImapServerId][$this->icServer->acc_folder_trash]))
 			$_specialUseFolders[$this->icServer->ImapServerId][$this->icServer->acc_folder_trash]='Trash';
@@ -1075,7 +1223,8 @@ 
 	function folderIsSelectable($folderToSelect)
 	{
 		$retval = true;
-		if($folderToSelect && ($folderStatus = $this->getFolderStatus($folderToSelect,false,true))) {
+		if($folderToSelect && ($folderStatus = $this->getFolderStatus($folderToSelect,false,true)))
+		{
 			if (!empty($folderStatus['attributes']) && stripos(array2string($folderStatus['attributes']),'noselect')!==false)
 			{
 				$retval = false;
@@ -1128,18 +1277,27 @@ 
 	 */
 	function getFolderStatus($_folderName,$ignoreStatusCache=false,$basicInfoOnly=false,$fetchSubscribedInfo=true)
 	{
-		if (self::$debug) error_log(__METHOD__.' ('.__LINE__.') '." called with:$_folderName,$ignoreStatusCache,$basicInfoOnly");
-		if (!is_string($_folderName) || empty($_folderName)) // something is wrong. Do not proceed
+		if (self::$debug)
+		{
+			error_log(__METHOD__.' ('.__LINE__.') '." called with:$_folderName,$ignoreStatusCache,$basicInfoOnly");
+		}
+		if (!is_string($_folderName) || empty($_folderName))
+		{
+			// something is wrong. Do not proceed
 		{
 			return false;
 		}
+		}
 		static $folderInfoCache = null; // reduce traffic on single request
 		static $folderBasicInfo = null;
 		if (isset($folderBasicInfo[$this->profileID]))
 		{
 			$folderInfoCache = $folderBasicInfo[$this->profileID];
 		}
-		if (isset($folderInfoCache[$_folderName]) && $ignoreStatusCache==false && $basicInfoOnly) return $folderInfoCache[$_folderName];
+		if (isset($folderInfoCache[$_folderName]) && $ignoreStatusCache==false && $basicInfoOnly)
+		{
+			return $folderInfoCache[$_folderName];
+		}
 		$retValue = array();
 		$retValue['subscribed'] = false;
 /*
@@ -1157,7 +1315,10 @@ 
 			if (is_array($ret))
 			{
 				$retkeys = array_keys($ret);
-				if ($retkeys[0]==$_folderName) $folderInfoCache[$_folderName] = $ret[$retkeys[0]];
+				if ($retkeys[0]==$_folderName)
+				{
+					$folderInfoCache[$_folderName] = $ret[$retkeys[0]];
+				}
 			}
 			else
 			{
@@ -1166,7 +1327,8 @@ 
 		}
 		$folderInfo = $folderInfoCache[$_folderName];
 		//error_log(__METHOD__.' ('.__LINE__.') '.array2string($folderInfo).'->'.function_backtrace());
-		if($ignoreStatusCache||!$folderInfo|| !is_array($folderInfo)) {
+		if($ignoreStatusCache||!$folderInfo|| !is_array($folderInfo))
+		{
 			try
 			{
 				$folderInfo = $this->_getStatus($_folderName,$ignoreStatusCache);
@@ -1196,7 +1358,8 @@ 
 		$retValue['shortName']		= array_pop($shortNameParts);
 		$retValue['displayName']	= $_folderName;
 		$retValue['shortDisplayName']	= $retValue['shortName'];
-		if(strtoupper($retValue['shortName']) == 'INBOX') {
+		if(strtoupper($retValue['shortName']) == 'INBOX')
+		{
 			$retValue['displayName']	= lang('INBOX');
 			$retValue['shortDisplayName']	= lang('INBOX');
 		}
@@ -1205,7 +1368,10 @@ 
 		{
 			$retValue['displayName'] = $retValue['shortDisplayName'] = lang($retValue['shortName']);
 		}
-		if ($folderInfo) $folderBasicInfo[$this->profileID][$_folderName]=$retValue;
+		if ($folderInfo)
+		{
+			$folderBasicInfo[$this->profileID][$_folderName]=$retValue;
+		}
 		//error_log(__METHOD__.' ('.__LINE__.') '.' '.$_folderName.array2string($retValue['attributes']));
 		if ($basicInfoOnly || (isset($retValue['attributes']) && stripos(array2string($retValue['attributes']),'noselect')!==false))
 		{
@@ -1217,24 +1383,34 @@ 
 		static $subscribedFolders = null;
 		static $nameSpace = null;
 		static $prefix = null;
-		if (is_null($nameSpace) || empty($nameSpace[$this->profileID])) $nameSpace[$this->profileID] = $this->_getNameSpaces();
+		if (is_null($nameSpace) || empty($nameSpace[$this->profileID]))
+		{
+			$nameSpace[$this->profileID] = $this->_getNameSpaces();
+		}
 		if (!empty($nameSpace[$this->profileID]))
 		{
 			$nsNoPersonal=array();
 			foreach($nameSpace[$this->profileID] as &$ns)
 			{
-				if ($ns['type']!='personal') $nsNoPersonal[]=$ns;
+				if ($ns['type']!='personal')
+				{
+					$nsNoPersonal[]=$ns;
+				}
 			}
 			$nameSpace[$this->profileID]=$nsNoPersonal;
 		}
-		if (is_null($prefix) || empty($prefix[$this->profileID]) || empty($prefix[$this->profileID][$_folderName])) $prefix[$this->profileID][$_folderName] = $this->getFolderPrefixFromNamespace($nameSpace[$this->profileID], $_folderName);
+		if (is_null($prefix) || empty($prefix[$this->profileID]) || empty($prefix[$this->profileID][$_folderName]))
+		{
+			$prefix[$this->profileID][$_folderName] = $this->getFolderPrefixFromNamespace($nameSpace[$this->profileID], $_folderName);
+		}
 
 		if ($fetchSubscribedInfo && is_null($subscribedFolders) || empty($subscribedFolders[$this->profileID]))
 		{
 			$subscribedFolders[$this->profileID] = $this->icServer->listSubscribedMailboxes();
 		}
 
-		if($fetchSubscribedInfo && is_array($subscribedFolders[$this->profileID]) && in_array($_folderName,$subscribedFolders[$this->profileID])) {
+		if($fetchSubscribedInfo && is_array($subscribedFolders[$this->profileID]) && in_array($_folderName,$subscribedFolders[$this->profileID]))
+		{
 			$retValue['subscribed'] = true;
 		}
 
@@ -1253,8 +1429,7 @@ 
 				(isset($this->mailPreferences['trustServersUnseenInfo']) &&
 				$this->mailPreferences['trustServersUnseenInfo']==2 &&
 				$prefix[$this->profileID][$_folderName] != '' && stripos($_folderName,$prefix[$this->profileID][$_folderName]) !== false)
-			)
-			{
+			) {
 				//error_log(__METHOD__." returned folderStatus for Folder $_folderName:".print_r($prefix,true).' TS:'.$this->mailPreferences['trustServersUnseenInfo']);
 				// we filter for the combined status of unseen and undeleted, as this is what we show in list
 				try
@@ -1264,13 +1439,19 @@ 
 				}
 				catch (\Exception $ee)
 				{
-					if (self::$debug) error_log(__METHOD__." could not fetch/calculate unseen counter for $_folderName Reason:'".$ee->getMessage()."' but requested.");
+					if (self::$debug)
+					{
+						error_log(__METHOD__." could not fetch/calculate unseen counter for $_folderName Reason:'".$ee->getMessage()."' but requested.");
+					}
 				}
 			}
 		}
 		catch (\Exception $e)
 		{
-			if (self::$debug) error_log(__METHOD__." returned folderStatus for Folder $_folderName:".print_r($e->getMessage(),true));
+			if (self::$debug)
+			{
+				error_log(__METHOD__." returned folderStatus for Folder $_folderName:".print_r($e->getMessage(),true));
+			}
 		}
 
 		return $retValue;
@@ -1295,8 +1476,14 @@ 
 	function getHeaders($_folderName, $_startMessage, $_numberOfMessages, $_sort, $_reverse, $_filter, $_thisUIDOnly=null, $_cacheResult=true, $_fetchPreviews=false)
 	{
 		//self::$debug=true;
-		if (self::$debug) error_log(__METHOD__.' ('.__LINE__.') '.function_backtrace());
-		if (self::$debug) error_log(__METHOD__.' ('.__LINE__.') '."$_folderName,$_startMessage, $_numberOfMessages, $_sort, $_reverse, ".array2string($_filter).", $_thisUIDOnly");
+		if (self::$debug)
+		{
+			error_log(__METHOD__.' ('.__LINE__.') '.function_backtrace());
+		}
+		if (self::$debug)
+		{
+			error_log(__METHOD__.' ('.__LINE__.') '."$_folderName,$_startMessage, $_numberOfMessages, $_sort, $_reverse, ".array2string($_filter).", $_thisUIDOnly");
+		}
 		$reverse = (bool)$_reverse;
 		// get the list of messages to fetch
 		$this->reopen($_folderName);
@@ -1317,21 +1504,34 @@ 
 				//$_filter['range'] ="$start:$end";
 				//$_filter['range'] ="$_startMessage:*";
 			}
-			if (self::$debug) error_log(__METHOD__.' ('.__LINE__.') '."$_folderName, $_sort, $reverse, ".array2string($_filter).", $rByUid");
-			if (self::$debug||self::$debugTimes) $starttime = microtime (true);
+			if (self::$debug)
+			{
+				error_log(__METHOD__.' ('.__LINE__.') '."$_folderName, $_sort, $reverse, ".array2string($_filter).", $rByUid");
+			}
+			if (self::$debug||self::$debugTimes)
+			{
+				$starttime = microtime (true);
+			}
 			//see this example below for a 12 week datefilter (since)
 			//$_filter = array('status'=>array('UNDELETED'),'type'=>"SINCE",'string'=> date("d-M-Y", $starttime-(3600*24*7*12)));
 			$_sortResult = $this->getSortedList($_folderName, $_sort, $reverse, $_filter, $rByUid, $_cacheResult);
 			$sortResult = $_sortResult['match']->ids;
 			//$modseq = $_sortResult['modseq'];
 			//error_log(__METHOD__.' ('.__LINE__.') '.'Modsequence:'.$modseq);
-			if (self::$debug||self::$debugTimes) self::logRunTimes($starttime,null,' call getSortedList for Folder:'.$_folderName.' Filter:'.array2string($_filter).' Ids:'.array2string($_thisUIDOnly),__METHOD__.' ('.__LINE__.') ');
+			if (self::$debug||self::$debugTimes)
+			{
+				self::logRunTimes($starttime,null,' call getSortedList for Folder:'.$_folderName.' Filter:'.array2string($_filter).' Ids:'.array2string($_thisUIDOnly),__METHOD__.' ('.__LINE__.') ');
+			}
 
-			if (self::$debug) error_log(__METHOD__.' ('.__LINE__.') '.array2string($sortResult));
+			if (self::$debug)
+			{
+				error_log(__METHOD__.' ('.__LINE__.') '.array2string($sortResult));
+			}
 			#$this->icServer->setDebug(false);
 			#print "</pre>";
 			// nothing found
-			if(!is_array($sortResult) || empty($sortResult)) {
+			if(!is_array($sortResult) || empty($sortResult))
+			{
 				$retValue = array();
 				$retValue['info']['total']	= 0;
 				$retValue['info']['first']	= 0;
@@ -1343,7 +1543,8 @@ 
 			#_debug_array($sortResult);
 			#_debug_array(array_slice($sortResult, -5, -2));
 			//error_log("REVERSE: $reverse");
-			if($reverse === true) {
+			if($reverse === true)
+			{
 				if  ($_startMessage<=$total)
 				{
 					$startMessage = $_startMessage-1;
@@ -1355,7 +1556,10 @@ 
 					{
 						$numberOfMessages = $total%$_numberOfMessages;
 						//$numberOfMessages = abs($_startMessage-$total-1);
-						if ($numberOfMessages>0 && $numberOfMessages<=$_numberOfMessages) $_numberOfMessages = $numberOfMessages;
+						if ($numberOfMessages>0 && $numberOfMessages<=$_numberOfMessages)
+						{
+							$_numberOfMessages = $numberOfMessages;
+						}
 						//error_log(__METHOD__.' ('.__LINE__.') '.' Start:'.$_startMessage.' NumberOfMessages:'.$_numberOfMessages.' Total:'.$total);
 					}
 					$startMessage=($total-$_numberOfMessages)-1;
@@ -1369,19 +1573,36 @@ 
 					//$retValue['info']['first'] = $startMessage;
 					//$retValue['info']['last'] = $total;
 				}
-				if($startMessage > 0) {
-					if (self::$debug) error_log(__METHOD__.' ('.__LINE__.') '.' StartMessage:'.(-($_numberOfMessages+$startMessage)).', '.-$startMessage.' Number of Messages:'.count($sortResult));
+				if($startMessage > 0)
+				{
+					if (self::$debug)
+					{
+						error_log(__METHOD__.' ('.__LINE__.') '.' StartMessage:'.(-($_numberOfMessages+$startMessage)).', '.-$startMessage.' Number of Messages:'.count($sortResult));
+					}
 					$sortResult = array_slice($sortResult, -($_numberOfMessages+$startMessage), -$startMessage);
-				} else {
-					if (self::$debug) error_log(__METHOD__.' ('.__LINE__.') '.' StartMessage:'.(-($_numberOfMessages+($_startMessage-1))).', AllTheRest, Number of Messages:'.count($sortResult));
+				}
+				else
+				{
+					if (self::$debug)
+					{
+						error_log(__METHOD__.' ('.__LINE__.') '.' StartMessage:'.(-($_numberOfMessages+($_startMessage-1))).', AllTheRest, Number of Messages:'.count($sortResult));
+					}
 					$sortResult = array_slice($sortResult, -($_numberOfMessages+($_startMessage-1)));
 				}
 				$sortResult = array_reverse($sortResult);
-			} else {
-				if (self::$debug) error_log(__METHOD__.' ('.__LINE__.') '.' StartMessage:'.($_startMessage-1).', '.$_numberOfMessages.' Number of Messages:'.count($sortResult));
+			}
+			else
+			{
+				if (self::$debug)
+				{
+					error_log(__METHOD__.' ('.__LINE__.') '.' StartMessage:'.($_startMessage-1).', '.$_numberOfMessages.' Number of Messages:'.count($sortResult));
+				}
 				$sortResult = array_slice($sortResult, $_startMessage-1, $_numberOfMessages);
 			}
-			if (self::$debug) error_log(__METHOD__.' ('.__LINE__.') '.array2string($sortResult));
+			if (self::$debug)
+			{
+				error_log(__METHOD__.' ('.__LINE__.') '.array2string($sortResult));
+			}
 		}
 		else
 		{
@@ -1390,7 +1611,10 @@ 
 
 
 		// fetch the data for the selected messages
-		if (self::$debug||self::$debugTimes) $starttime = microtime(true);
+		if (self::$debug||self::$debugTimes)
+		{
+			$starttime = microtime(true);
+		}
 		try
 		{
 			$uidsToFetch = new Horde_Imap_Client_Ids();
@@ -1414,7 +1638,10 @@ 
 			$fquery->flags();
 			$fquery->imapDate();// needed to ensure getImapDate fetches the internaldate, not the current time
 			// if $_fetchPreviews is activated fetch part of the messages too
-			if ($_fetchPreviews) $fquery->fullText(array('peek'=>true,'length'=>((int)$_fetchPreviews<5000?5000:$_fetchPreviews),'start'=>0));
+			if ($_fetchPreviews)
+			{
+				$fquery->fullText(array('peek'=>true,'length'=>((int)$_fetchPreviews<5000?5000:$_fetchPreviews),'start'=>0));
+			}
 			$headersNew = $this->icServer->fetch($_folderName, $fquery, array(
 				'ids' => $uidsToFetch,
 			));
@@ -1437,14 +1664,20 @@ 
 
 		$cnt = 0;
 
-		foreach((array)$sortResult as $uid) {
+		foreach((array)$sortResult as $uid)
+		{
 			$sortOrder[$uid] = $cnt++;
 		}
 
 		$count = 0;
-		if (is_object($headersNew)) {
-			if (self::$debug||self::$debugTimes) $starttime = microtime(true);
-			foreach($headersNew->ids() as $id) {
+		if (is_object($headersNew))
+		{
+			if (self::$debug||self::$debugTimes)
+			{
+				$starttime = microtime(true);
+			}
+			foreach($headersNew->ids() as $id)
+			{
 				$_headerObject = $headersNew->get($id);
 				//error_log(__METHOD__.' ('.__LINE__.') '.array2string($_headerObject));
 				$headerObject = array();
@@ -1457,7 +1690,10 @@ 
 				// Get already cached headers, 'fetchHeaders' is a label matchimg above
 				$headerForPrio = $_headerObject->getHeaders('fetchHeaders',Horde_Imap_Client_Data_Fetch::HEADER_PARSE)->toArray();
 				// Try to fetch header with key='' as some servers might have no fetchHeaders index. e.g. yandex.com
-				if (empty($headerForPrio)) $headerForPrio = $_headerObject->getHeaders('',Horde_Imap_Client_Data_Fetch::HEADER_PARSE)->toArray();
+				if (empty($headerForPrio))
+				{
+					$headerForPrio = $_headerObject->getHeaders('',Horde_Imap_Client_Data_Fetch::HEADER_PARSE)->toArray();
+				}
 				//fetch the fullMsg part if all conditions match to be available in case $_headerObject->getHeaders returns
 				//nothing worthwhile (as it does for googlemail accounts, when preview is switched on
 				if ($_fetchPreviews)
@@ -1468,12 +1704,16 @@ 
 					if (empty($headerForPrio)||(is_array($headerForPrio)&&count($headerForPrio)===1&&$headerForPrio['']))
 					{
 						$length = strpos($bodyPreview, Horde_Mime_Part::RFC_EOL.Horde_Mime_Part::RFC_EOL);
-						if ($length===false) $length = strlen($bodyPreview);
+						if ($length===false)
+						{
+							$length = strlen($bodyPreview);
+						}
 						$headerForPrio =  Horde_Mime_Headers::parseHeaders(substr($bodyPreview, 0,$length))->toArray();
 					}
 				}
 				$headerForPrio = array_change_key_case($headerForPrio, CASE_UPPER);
-				if (self::$debug) {
+				if (self::$debug)
+				{
 					error_log(__METHOD__.' ('.__LINE__.') '.array2string($_headerObject).'UID:'.$_headerObject->getUid().' Size:'.$_headerObject->getSize().' Date:'.$_headerObject->getImapDate().'/'.DateTime::to($_headerObject->getImapDate(),'Y-m-d H:i:s'));
 					error_log(__METHOD__.' ('.__LINE__.') '.array2string($headerForPrio));
 				}
@@ -1483,11 +1723,16 @@ 
 					$total--;
 					continue;
 				}
-				if ( isset($headerForPrio['DISPOSITION-NOTIFICATION-TO']) ) {
+				if ( isset($headerForPrio['DISPOSITION-NOTIFICATION-TO']) )
+				{
 					$headerObject['DISPOSITION-NOTIFICATION-TO'] = self::decode_header(trim($headerForPrio['DISPOSITION-NOTIFICATION-TO']));
-				} else if ( isset($headerForPrio['RETURN-RECEIPT-TO']) ) {
+				}
+				else if ( isset($headerForPrio['RETURN-RECEIPT-TO']) )
+				{
 					$headerObject['DISPOSITION-NOTIFICATION-TO'] = self::decode_header(trim($headerForPrio['RETURN-RECEIPT-TO']));
-				} else if ( isset($headerForPrio['X-CONFIRM-READING-TO']) ) {
+				}
+				else if ( isset($headerForPrio['X-CONFIRM-READING-TO']) )
+				{
 					$headerObject['DISPOSITION-NOTIFICATION-TO'] = self::decode_header(trim($headerForPrio['X-CONFIRM-READING-TO']));
 				} /*else $sent_not = "";*/
 				//error_log(__METHOD__.' ('.__LINE__.') '.array2string($headerObject));
@@ -1518,7 +1763,11 @@ 
 						$rfcAddr = self::parseAddressList($ad); // does some fixing of known problems too
 						foreach ($rfcAddr as $_rfcAddr)
 						{
-							if (!$_rfcAddr->valid)	continue; // skip. not a valid address
+							if (!$_rfcAddr->valid)
+							{
+								continue;
+							}
+							// skip. not a valid address
 							$address[] = imap_rfc822_write_address($_rfcAddr->mailbox,$_rfcAddr->host,$_rfcAddr->personal);
 						}
 					}
@@ -1534,7 +1783,10 @@ 
 				{
 					// $bodyPreview is populated at the beginning of the loop, as it may be
 					// needed to parse the Headers of the Message
-					if (empty($bodyPreview)) $bodyPreview = $_headerObject->getFullMsg();
+					if (empty($bodyPreview))
+					{
+						$bodyPreview = $_headerObject->getFullMsg();
+					}
 					//error_log(__METHOD__.' ('.__LINE__.') '.array2string($bodyPreview));
 					$base = Horde_Mime_Part::parseMessage($bodyPreview);
 					foreach($base->partIterator() as $part)
@@ -1547,7 +1799,8 @@ 
 												'encode' => Horde_Mime_Part::ENCODE_BINARY,	// otherwise we cant recode charset
 											)), $charset, 'utf-8');
 							$headerObject['BODYPREVIEW']=trim(str_replace(array("\r\n","\r","\n"),' ',mb_substr(Translation::convert_jsonsafe($buffer),0,((int)$_fetchPreviews<300?300:$_fetchPreviews))));
-						} elseif (empty($headerObject['BODYPREVIEW'])&&$part->getPrimaryType()== 'multipart')
+						}
+						elseif (empty($headerObject['BODYPREVIEW'])&&$part->getPrimaryType()== 'multipart')
 						{
 							//error_log(__METHOD__.' ('.__LINE__.') '.array2string($part));
 						}
@@ -1563,7 +1816,10 @@ 
 				$messageMimeType='';
 				foreach ($mailStructureObject->contentTypeMap() as $mime_id => $mime_type)
 				{
-					if ($mime_id==0 || $messageMimeType==='') $messageMimeType = $mime_type;
+					if ($mime_id==0 || $messageMimeType==='')
+					{
+						$messageMimeType = $mime_type;
+					}
 					$part = $mailStructureObject->getPart($mime_id);
 					$partdisposition = $part->getDisposition();
 					$partPrimaryType = $part->getPrimaryType();
@@ -1573,7 +1829,8 @@ 
 					// bodypreview could be found (multipart/....)
 					if ($_fetchPreviews && empty($headerObject['BODYPREVIEW'])&&($partPrimaryType == 'text') &&
 						((intval($mime_id) === 1) || !$mime_id) &&
-						($partdisposition !== 'attachment')) {
+						($partdisposition !== 'attachment'))
+					{
 							$_structure=$part;
 							$this->fetchPartContents($uid, $_structure, false,true);
 							$headerObject['BODYPREVIEW']=trim(str_replace(array("\r\n","\r","\n"),' ',mb_substr(Mail\Html::convertHTMLToText($_structure->getContents()),0,((int)$_fetchPreviews<300?300:$_fetchPreviews))));
@@ -1595,10 +1852,19 @@ 
 					if ($mime_type=='message/rfc822')
 					{
 						//error_log(__METHOD__.' ('.__LINE__.') '.' Uid:'.$uid.'->'.$mime_id.':'.array2string($part->contentTypeMap()));
-						foreach($part->contentTypeMap() as $sub_id => $sub_type) { if ($sub_id != $mime_id) $skipParts[$sub_id] = $sub_type;}
+						foreach($part->contentTypeMap() as $sub_id => $sub_type)
+						{
+if ($sub_id != $mime_id)
+						{
+							$skipParts[$sub_id] = $sub_type;
+						}
+						}
 					}
 					//error_log(__METHOD__.' ('.__LINE__.') '.' Uid:'.$uid.'->'.$mime_id.' Disp:'.$partdisposition.' Type:'.$partPrimaryType.' Skip:'.array2string($skipParts));
-					if (array_key_exists($mime_id,$skipParts)) continue;
+					if (array_key_exists($mime_id,$skipParts))
+					{
+						continue;
+					}
 					if ($partdisposition=='attachment' ||
 						($partdisposition=='inline'&&$partPrimaryType == 'image'&&$mime_type=='image/tiff') || // as we are not able to display tiffs
 						($partdisposition=='inline'&&$partPrimaryType == 'image'&&empty($cid)) ||
@@ -1609,7 +1875,10 @@ 
 						$headerObject['ATTACHMENTS'][$mime_id]['uid']=$uid;
 						$headerObject['ATTACHMENTS'][$mime_id]['cid'] = $cid;
 						$headerObject['ATTACHMENTS'][$mime_id]['partID']=$mime_id;
-						if (!isset($headerObject['ATTACHMENTS'][$mime_id]['name']))$headerObject['ATTACHMENTS'][$mime_id]['name']=$part->getName();
+						if (!isset($headerObject['ATTACHMENTS'][$mime_id]['name']))
+						{
+							$headerObject['ATTACHMENTS'][$mime_id]['name']=$part->getName();
+						}
 						if (!strcasecmp($headerObject['ATTACHMENTS'][$mime_id]['name'],'winmail.dat') ||
 							$headerObject['ATTACHMENTS'][$mime_id]['mimeType']=='application/ms-tnef')
 						{
@@ -1623,18 +1892,27 @@ 
 				//error_log(__METHOD__.' ('.__LINE__.') '.' FindBody (plain):'.array2string($mailStructureObject->findBody('plain')));
 				//error_log(__METHOD__.' ('.__LINE__.') '.' FindBody (html):'.array2string($mailStructureObject->findBody('html')));
 				//if($count == 0) error_log(__METHOD__.array2string($headerObject));
-				if (empty($headerObject['UID'])) continue;
+				if (empty($headerObject['UID']))
+				{
+					continue;
+				}
 				//$uid = ($rByUid ? $headerObject['UID'] : $headerObject['MSG_NUM']);
 				// make dates like "Mon, 23 Apr 2007 10:11:06 UT" working with strtotime
-				if(substr($headerObject['DATE'],-2) === 'UT') {
+				if(substr($headerObject['DATE'],-2) === 'UT')
+				{
 					$headerObject['DATE'] .= 'C';
 				}
-				if(substr($headerObject['INTERNALDATE'],-2) === 'UT') {
+				if(substr($headerObject['INTERNALDATE'],-2) === 'UT')
+				{
 					$headerObject['INTERNALDATE'] .= 'C';
 				}
 				//error_log(__METHOD__.' ('.__LINE__.') '.' '.$headerObject['SUBJECT'].'->'.$headerObject['DATE'].'<->'.$headerObject['INTERNALDATE'] .'#');
 				//error_log(__METHOD__.' ('.__LINE__.') '.' '.$this->decode_subject($headerObject['SUBJECT']).'->'.$headerObject['DATE']);
-				if (isset($headerObject['ATTACHMENTS']) && count($headerObject['ATTACHMENTS'])) foreach ($headerObject['ATTACHMENTS'] as &$a) { $retValue['header'][$sortOrder[$uid]]['attachments'][]=$a;}
+				if (isset($headerObject['ATTACHMENTS']) && count($headerObject['ATTACHMENTS']))
+				{
+					foreach ($headerObject['ATTACHMENTS'] as &$a) { $retValue['header'][$sortOrder[$uid]]['attachments'][]=$a;
+				}
+				}
 				$retValue['header'][$sortOrder[$uid]]['subject']	= $this->decode_subject($headerObject['SUBJECT']);
 				$retValue['header'][$sortOrder[$uid]]['size'] 		= $headerObject['SIZE'];
 				$retValue['header'][$sortOrder[$uid]]['date']		= self::_strtotime(($headerObject['DATE']&&!($headerObject['DATE']=='NIL')?$headerObject['DATE']:$headerObject['INTERNALDATE']),'ts',true);
@@ -1645,22 +1923,31 @@ 
 				$retValue['header'][$sortOrder[$uid]]['bodypreview']		= $headerObject['BODYPREVIEW'];
 				$retValue['header'][$sortOrder[$uid]]['priority']		= ($headerObject['PRIORITY']?$headerObject['PRIORITY']:3);
 				//error_log(__METHOD__.' ('.__LINE__.') '.' '.array2string($retValue['header'][$sortOrder[$uid]]));
-				if (isset($headerObject['DISPOSITION-NOTIFICATION-TO'])) $retValue['header'][$sortOrder[$uid]]['disposition-notification-to'] = $headerObject['DISPOSITION-NOTIFICATION-TO'];
-				if (is_array($headerObject['FLAGS'])) {
+				if (isset($headerObject['DISPOSITION-NOTIFICATION-TO']))
+				{
+					$retValue['header'][$sortOrder[$uid]]['disposition-notification-to'] = $headerObject['DISPOSITION-NOTIFICATION-TO'];
+				}
+				if (is_array($headerObject['FLAGS']))
+				{
 					$retValue['header'][$sortOrder[$uid]] = array_merge($retValue['header'][$sortOrder[$uid]],self::prepareFlagsArray($headerObject));
 				}
 				//error_log(__METHOD__.' ('.__LINE__.') '.$headerObject['SUBJECT'].'->'.array2string($_headerObject->getEnvelope()->__get('from')));
-				if(is_array($headerObject['FROM']) && $headerObject['FROM'][0]) {
+				if(is_array($headerObject['FROM']) && $headerObject['FROM'][0])
+				{
 					$retValue['header'][$sortOrder[$uid]]['sender_address'] = self::decode_header($headerObject['FROM'][0],true);
 				}
-				if(is_array($headerObject['TO']) && $headerObject['TO'][0]) {
+				if(is_array($headerObject['TO']) && $headerObject['TO'][0])
+				{
 					$retValue['header'][$sortOrder[$uid]]['to_address'] = self::decode_header($headerObject['TO'][0],true);
 					if (count($headerObject['TO'])>1)
 					{
 						$ki=0;
 						foreach($headerObject['TO'] as $k => $add)
 						{
-							if ($k==0) continue;
+							if ($k==0)
+							{
+								continue;
+							}
 							//error_log(__METHOD__.' ('.__LINE__.') '."-> $k:".array2string($add));
 							$retValue['header'][$sortOrder[$uid]]['additional_to_addresses'][$ki] = self::decode_header($add,true);
 							//error_log(__METHOD__.' ('.__LINE__.') '.array2string($retValue['header'][$sortOrder[$uid]]['additional_to_addresses'][$ki]));
@@ -1668,7 +1955,8 @@ 
 						}
 					}
 				}
-				if(is_array($headerObject['CC']) && count($headerObject['CC'])>0) {
+				if(is_array($headerObject['CC']) && count($headerObject['CC'])>0)
+				{
 					$ki=0;
 					foreach($headerObject['CC'] as $k => $add)
 					{
@@ -1682,27 +1970,41 @@ 
 
 				$count++;
 			}
-			if (self::$debug||self::$debugTimes) self::logRunTimes($starttime,null,' fetching Headers and stuff for Folder:'.$_folderName,__METHOD__.' ('.__LINE__.') ');
+			if (self::$debug||self::$debugTimes)
+			{
+				self::logRunTimes($starttime,null,' fetching Headers and stuff for Folder:'.$_folderName,__METHOD__.' ('.__LINE__.') ');
+			}
 			//self::$debug=false;
 			// sort the messages to the requested displayorder
-			if(is_array($retValue['header'])) {
+			if(is_array($retValue['header']))
+			{
 				$countMessages = $total;
-				if (isset($_filter['range'])) $countMessages = self::$folderStatusCache[$this->profileID][$_folderName]['messages'];
+				if (isset($_filter['range']))
+				{
+					$countMessages = self::$folderStatusCache[$this->profileID][$_folderName]['messages'];
+				}
 				ksort($retValue['header']);
 				$retValue['info']['total']	= $total;
 				//if ($_startMessage>$total) $_startMessage = $total-($count-1);
 				$retValue['info']['first']	= $_startMessage;
 				$retValue['info']['last']	= $_startMessage + $count - 1 ;
 				return $retValue;
-			} else {
+			}
+			else
+			{
 				$retValue = array();
 				$retValue['info']['total']	= 0;
 				$retValue['info']['first']	= 0;
 				$retValue['info']['last']	= 0;
 				return $retValue;
 			}
-		} else {
-			if ($headersNew == null && empty($_thisUIDOnly)) error_log(__METHOD__." -> retrieval of Message Details to Query $queryString failed: ".print_r($headersNew,TRUE));
+		}
+		else
+		{
+			if ($headersNew == null && empty($_thisUIDOnly))
+			{
+				error_log(__METHOD__." -> retrieval of Message Details to Query $queryString failed: ".print_r($headersNew,TRUE));
+			}
 			$retValue = array();
 			$retValue['info']['total']  = 0;
 			$retValue['info']['first']  = 0;
@@ -1719,7 +2021,10 @@ 
 	 */
 	static function prepareFlagsArray($headerObject)
 	{
-		if (is_array($headerObject['FLAGS'])) $headerFlags = array_map('strtolower',$headerObject['FLAGS']);
+		if (is_array($headerObject['FLAGS']))
+		{
+			$headerFlags = array_map('strtolower',$headerObject['FLAGS']);
+		}
 		$retValue = array();
 		$retValue['recent']		= in_array('\\recent', $headerFlags);
 		$retValue['flagged']	= in_array('\\flagged', $headerFlags);
@@ -1770,24 +2075,43 @@ 
 		//error_log(__METHOD__.' ('.__LINE__.') '.' Filter:'.array2string($_filter));
 		$try2useCache = true;
 		static $eMailListContainsDeletedMessages = null;
-		if (is_null($eMailListContainsDeletedMessages)) $eMailListContainsDeletedMessages = Cache::getCache(Cache::INSTANCE,'email','eMailListContainsDeletedMessages'.trim($GLOBALS['egw_info']['user']['account_id']),null,array(),60*60*1);
+		if (is_null($eMailListContainsDeletedMessages))
+		{
+			$eMailListContainsDeletedMessages = Cache::getCache(Cache::INSTANCE,'email','eMailListContainsDeletedMessages'.trim($GLOBALS['egw_info']['user']['account_id']),null,array(),60*60*1);
+		}
 		// this indicates, that there is no Filter set, and the returned set/subset should not contain DELETED Messages, nor filtered for UNDELETED
 		if ($setSession==true && ((strpos(array2string($_filter), 'UNDELETED') === false && strpos(array2string($_filter), 'DELETED') === false)))
 		{
-			if (self::$debugTimes) $starttime = microtime(true);
-			if (is_null($eMailListContainsDeletedMessages) || empty($eMailListContainsDeletedMessages[$this->profileID]) || empty($eMailListContainsDeletedMessages[$this->profileID][$_folderName])) $eMailListContainsDeletedMessages = Cache::getCache(Cache::INSTANCE,'email','eMailListContainsDeletedMessages'.trim($GLOBALS['egw_info']['user']['account_id']),null,array(),60*60*1);
+			if (self::$debugTimes)
+			{
+				$starttime = microtime(true);
+			}
+			if (is_null($eMailListContainsDeletedMessages) || empty($eMailListContainsDeletedMessages[$this->profileID]) || empty($eMailListContainsDeletedMessages[$this->profileID][$_folderName]))
+			{
+				$eMailListContainsDeletedMessages = Cache::getCache(Cache::INSTANCE,'email','eMailListContainsDeletedMessages'.trim($GLOBALS['egw_info']['user']['account_id']),null,array(),60*60*1);
+			}
 			$deletedMessages = $this->getSortedList($_folderName, 0, $three=1, array('status'=>array('DELETED')),$five=true,false);
-			if (self::$debug) error_log(__METHOD__.' ('.__LINE__.') Found DeletedMessages:'.array2string($eMailListContainsDeletedMessages));
+			if (self::$debug)
+			{
+				error_log(__METHOD__.' ('.__LINE__.') Found DeletedMessages:'.array2string($eMailListContainsDeletedMessages));
+			}
 			$eMailListContainsDeletedMessages[$this->profileID][$_folderName] =$deletedMessages['count'];
 			Cache::setCache(Cache::INSTANCE,'email','eMailListContainsDeletedMessages'.trim($GLOBALS['egw_info']['user']['account_id']),$eMailListContainsDeletedMessages, 60*60*1);
-			if (self::$debugTimes) self::logRunTimes($starttime,null,'setting eMailListContainsDeletedMessages for Profile:'.$this->profileID.' Folder:'.$_folderName.' to '.$eMailListContainsDeletedMessages[$this->profileID][$_folderName],__METHOD__.' ('.__LINE__.') ');			//error_log(__METHOD__.' ('.__LINE__.') '.' Profile:'.$this->profileID.' Folder:'.$_folderName.' -> EXISTS/SessStat:'.array2string($folderStatus['MESSAGES']).'/'.self::$folderStatusCache[$this->profileID][$_folderName]['messages'].' ListContDelMsg/SessDeleted:'.$eMailListContainsDeletedMessages[$this->profileID][$_folderName].'/'.self::$folderStatusCache[$this->profileID][$_folderName]['deleted']);
+			if (self::$debugTimes)
+			{
+				self::logRunTimes($starttime,null,'setting eMailListContainsDeletedMessages for Profile:'.$this->profileID.' Folder:'.$_folderName.' to '.$eMailListContainsDeletedMessages[$this->profileID][$_folderName],__METHOD__.' ('.__LINE__.') ');
+			}
+			//error_log(__METHOD__.' ('.__LINE__.') '.' Profile:'.$this->profileID.' Folder:'.$_folderName.' -> EXISTS/SessStat:'.array2string($folderStatus['MESSAGES']).'/'.self::$folderStatusCache[$this->profileID][$_folderName]['messages'].' ListContDelMsg/SessDeleted:'.$eMailListContainsDeletedMessages[$this->profileID][$_folderName].'/'.self::$folderStatusCache[$this->profileID][$_folderName]['deleted']);
 		}
 		$try2useCache = false;
 		//self::$supportsORinQuery[$this->profileID]=true;
 		if (is_null(self::$supportsORinQuery) || !isset(self::$supportsORinQuery[$this->profileID]))
 		{
 			self::$supportsORinQuery = Cache::getCache(Cache::INSTANCE,'email','supportsORinQuery'.trim($GLOBALS['egw_info']['user']['account_id']),null,array(),60*60*10);
-			if (!isset(self::$supportsORinQuery[$this->profileID])) self::$supportsORinQuery[$this->profileID]=true;
+			if (!isset(self::$supportsORinQuery[$this->profileID]))
+			{
+				self::$supportsORinQuery[$this->profileID]=true;
+			}
 		}
 		//error_log(__METHOD__.' ('.__LINE__.') '.array2string($_filter).' SupportsOrInQuery:'.self::$supportsORinQuery[$this->profileID]);
 		$filter = $this->createIMAPFilter($_folderName, $_filter,self::$supportsORinQuery[$this->profileID]);
@@ -1798,15 +2122,26 @@ 
 		}
 		//_debug_array($filter);
 		//error_log(__METHOD__.' ('.__LINE__.') '.array2string($filter).'#'.array2string($this->icServer->capability()));
-		if($this->icServer->hasCapability('SORT')) {
+		if($this->icServer->hasCapability('SORT'))
+		{
 			// when using an orQuery and we sort by date. sort seems to fail on certain servers => ZIMBRA with Horde_Imap_Client
 			// thus we translate the search request from date to Horde_Imap_Client::SORT_SEQUENCE (which should be the same, if
 			// there is no messing with the dates)
 			//if (self::$supportsORinQuery[$this->profileID]&&$_sort=='date'&&$_filter['type']=='quick'&&!empty($_filter['string']))$_sort='INTERNALDATE';
-			if (self::$debug) error_log(__METHOD__." Mailserver has SORT Capability, SortBy: ".array2string($_sort)." Reverse: $_reverse");
+			if (self::$debug)
+			{
+				error_log(__METHOD__." Mailserver has SORT Capability, SortBy: ".array2string($_sort)." Reverse: $_reverse");
+			}
 			$sortOrder = $this->_getSortString($_sort, $_reverse);
-			if ($_reverse && in_array(Horde_Imap_Client::SORT_REVERSE,$sortOrder)) $_reverse=false; // as we reversed the result already
-			if (self::$debug) error_log(__METHOD__." Mailserver runs SORT: SortBy:".array2string($_sort)."->".array2string($sortOrder)." Filter: ".array2string($filter));
+			if ($_reverse && in_array(Horde_Imap_Client::SORT_REVERSE,$sortOrder))
+			{
+				$_reverse=false;
+			}
+			// as we reversed the result already
+			if (self::$debug)
+			{
+				error_log(__METHOD__." Mailserver runs SORT: SortBy:".array2string($_sort)."->".array2string($sortOrder)." Filter: ".array2string($filter));
+			}
 			try
 			{
 				$sortResult = $this->icServer->search($_folderName, $filter, array(
@@ -1818,7 +2153,10 @@ 
 				//error_log(__METHOD__.'('.__LINE__.'):'.$e->getMessage());
 				$resultByUid = false;
 				$sortOrder = array(Horde_Imap_Client::SORT_SEQUENCE);
-				if ($_reverse) array_unshift($sortOrder,Horde_Imap_Client::SORT_REVERSE);
+				if ($_reverse)
+				{
+					array_unshift($sortOrder,Horde_Imap_Client::SORT_REVERSE);
+				}
 				try
 				{
 					$sortResult = $this->icServer->search($_folderName, $filter, array(
@@ -1830,9 +2168,17 @@ 
 					$sortResult = self::$folderStatusCache[$this->profileID][$_folderName]['sortResult'];
 				}
 			}
-			if (self::$debug) error_log(__METHOD__.print_r($sortResult,true));
-		} else {
-			if (self::$debug) error_log(__METHOD__." Mailserver has NO SORT Capability");
+			if (self::$debug)
+			{
+				error_log(__METHOD__.print_r($sortResult,true));
+			}
+		}
+		else
+		{
+			if (self::$debug)
+			{
+				error_log(__METHOD__." Mailserver has NO SORT Capability");
+			}
 			//$sortOrder = array(Horde_Imap_Client::SORT_SEQUENCE);
 			//if ($_reverse) array_unshift($sortOrder,Horde_Imap_Client::SORT_REVERSE);
 			try
@@ -1846,7 +2192,10 @@ 
 				// possible error OR Query. But Horde gives no detailed Info :-(
 				self::$supportsORinQuery[$this->profileID]=false;
 				Cache::setCache(Cache::INSTANCE,'email','supportsORinQuery'.trim($GLOBALS['egw_info']['user']['account_id']),self::$supportsORinQuery,60*60*10);
-				if (self::$debug) error_log(__METHOD__.__LINE__." Mailserver seems to have NO OR Capability for Search:".$sortResult->message);
+				if (self::$debug)
+				{
+					error_log(__METHOD__.__LINE__." Mailserver seems to have NO OR Capability for Search:".$sortResult->message);
+				}
 				$filter = $this->createIMAPFilter($_folderName, $_filter, self::$supportsORinQuery[$this->profileID]);
 				try
 				{
@@ -1857,11 +2206,15 @@ 
 				{
 				}
 			}
-			if(is_array($sortResult['match'])) {
+			if(is_array($sortResult['match']))
+			{
 					// not sure that this is going so succeed as $sortResult['match'] is a hordeObject
 					sort($sortResult['match'], SORT_NUMERIC);
 			}
-			if (self::$debug) error_log(__METHOD__." using Filter:".print_r($filter,true)." ->".print_r($sortResult,true));
+			if (self::$debug)
+			{
+				error_log(__METHOD__." using Filter:".print_r($filter,true)." ->".print_r($sortResult,true));
+			}
 		}
 		if ($setSession)
 		{
@@ -1890,7 +2243,8 @@ 
 		$_reverse=false;
 		if (is_numeric($_sort))
 		{
-			switch($_sort) {
+			switch($_sort)
+			{
 				case 2:
 					$retValue = array(Horde_Imap_Client::SORT_FROM);
 					break;
@@ -1912,7 +2266,8 @@ 
 		}
 		else
 		{
-			switch(strtoupper($_sort)) {
+			switch(strtoupper($_sort))
+			{
 				case 'FROMADDRESS':
 					$retValue = array(Horde_Imap_Client::SORT_FROM);
 					break;
@@ -1938,7 +2293,10 @@ 
 					break;
 			}
 		}
-		if ($_reverse) array_unshift($retValue,Horde_Imap_Client::SORT_REVERSE);
+		if ($_reverse)
+		{
+			array_unshift($retValue,Horde_Imap_Client::SORT_REVERSE);
+		}
 		//error_log(__METHOD__.' ('.__LINE__.') '.' '.($_reverse?'REVERSE ':'').$_sort.'->'.$retValue);
 		return $retValue;
 	}
@@ -1957,7 +2315,10 @@ 
 		$imapFilter->charset('UTF-8');
 
 		//_debug_array($_criterias);
-		if (self::$debug) error_log(__METHOD__.' ('.__LINE__.') '.' Criterias:'.(!is_array($_criterias)?" none -> returning":array2string($_criterias)));
+		if (self::$debug)
+		{
+			error_log(__METHOD__.' ('.__LINE__.') '.' Criterias:'.(!is_array($_criterias)?" none -> returning":array2string($_criterias)));
+		}
 		if((!is_array($_criterias) || $_criterias['status']=='any') &&
 			(!isset($_criterias['string']) || empty($_criterias['string'])) &&
 			(!isset($_criterias['range'])|| empty($_criterias['range']) ||
@@ -1972,11 +2333,13 @@ 
 		// statusQuery MUST be placed first, as search for subject/mailbody and such is
 		// depending on charset. flagSearch is not BUT messes the charset if called afterwards
 		$statusQueryValid = false;
-		foreach((array)$_criterias['status'] as $k => $criteria) {
+		foreach((array)$_criterias['status'] as $k => $criteria)
+		{
 			$imapStatusFilter = new Horde_Imap_Client_Search_Query();
 			$imapStatusFilter->charset('UTF-8');
 			$criteria = strtoupper($criteria);
-			switch ($criteria) {
+			switch ($criteria)
+			{
 				case 'ANSWERED':
 				case 'DELETED':
 				case 'FLAGGED':
@@ -2059,9 +2422,11 @@ 
 		$imapSearchFilter = new Horde_Imap_Client_Search_Query();
 		$imapSearchFilter->charset('UTF-8');
 
-		if(!empty($_criterias['string'])) {
+		if(!empty($_criterias['string']))
+		{
 			$criteria = strtoupper($_criterias['type']);
-			switch ($criteria) {
+			switch ($criteria)
+			{
 				case 'BYDATE':
 				case 'QUICK':
 				case 'QUICKWITHCC':
@@ -2069,9 +2434,12 @@ 
 					//$imapSearchFilter->charset('UTF-8');
 					$imapFilter2 = new Horde_Imap_Client_Search_Query();
 					$imapFilter2->charset('UTF-8');
-					if($this->isSentFolder($_folder)) {
+					if($this->isSentFolder($_folder))
+					{
 						$imapFilter2->headerText('TO', $_criterias['string'], $not=false);
-					} else {
+					}
+					else
+					{
 						$imapFilter2->headerText('FROM', $_criterias['string'], $not=false);
 					}
 					if ($_supportsOrInQuery)
@@ -2102,7 +2470,10 @@ 
 											'GB'=>1024*1000*1000,'G'=>1024*1000*1000,
 											'TB'=>1024*1000*1000*1000,'T'=>1024*1000*1000*1000);
 						$numberinBytes=(float)$_criterias['string'];
-						if (isset($multipleBy[$unit])) $numberinBytes=(float)$_criterias['string']*$multipleBy[$unit];
+						if (isset($multipleBy[$unit]))
+						{
+							$numberinBytes=(float)$_criterias['string']*$multipleBy[$unit];
+						}
 						//error_log(__METHOD__.__LINE__.'#'.$_criterias['string'].'->'.(float)$_criterias['string'].'#'.$unit.' ='.$numberinBytes);
 						$_criterias['string']=$numberinBytes;
 					}
@@ -2139,8 +2510,14 @@ 
 					break;
 			}
 		}
-		if ($statusQueryValid && !$queryValid) $queryValid=true;
-		if ($queryValid) $imapFilter->andSearch($imapSearchFilter);
+		if ($statusQueryValid && !$queryValid)
+		{
+			$queryValid=true;
+		}
+		if ($queryValid)
+		{
+			$imapFilter->andSearch($imapSearchFilter);
+		}
 
 		if (isset($_criterias['range']) && !empty($_criterias['range']))
 		{
@@ -2155,7 +2532,8 @@ 
 				unset($_criterias['before']);
 				$criteria=$_criterias['range']='ON';
 			}
-			switch ($criteria) {
+			switch ($criteria)
+			{
 				case 'BETWEEN':
 					//try to be smart about missing
 					//enddate
@@ -2191,8 +2569,14 @@ 
 					$rangeValid = true;
 					break;
 			}
-			if ($rangeValid && !$queryValid) $queryValid=true;
-			if ($rangeValid) $imapFilter->andSearch($imapRangeFilter);
+			if ($rangeValid && !$queryValid)
+			{
+				$queryValid=true;
+			}
+			if ($rangeValid)
+			{
+				$imapFilter->andSearch($imapRangeFilter);
+			}
 		}
 		if (self::$debug)
 		{
@@ -2200,10 +2584,13 @@ 
 			$query_str = $imapFilter->build();
 			//error_log(__METHOD__.' ('.__LINE__.') '.' '.$query_str['query'].' created by Criterias:'.(!is_array($_criterias)?" none -> returning":array2string($_criterias)));
 		}
-		if($queryValid==false) {
+		if($queryValid==false)
+		{
 			$imapFilter->flag('DELETED', $set=false);
 			return $imapFilter;
-		} else {
+		}
+		else
+		{
 			return $imapFilter;
 		}
 	}
@@ -2260,7 +2647,10 @@ 
 					}
 					$stringA[] = imap_rfc822_write_address($_rfcAddr->mailbox,Horde_Idna::decode($_rfcAddr->host),$_rfcAddr->personal);
 				}
-				if (!empty($stringA)) $_string = implode(',',$stringA);
+				if (!empty($stringA))
+				{
+					$_string = implode(',',$stringA);
+				}
 			}
 			if ($_tryIDNConversion==='FORCE')
 			{
@@ -2285,7 +2675,10 @@ 
 		{
 			return 'No Subject';
 		}
-		if ($decode) $_string = self::decode_header($_string);
+		if ($decode)
+		{
+			$_string = self::decode_header($_string);
+		}
 		// make sure its utf-8
 		$test = @json_encode($_string);
 		if (($test=="null" || $test === false || is_null($test)) && strlen($_string)>0)
@@ -2323,7 +2716,8 @@ 
 	 * @param string _parent the parent foldername
 	 * @return ISO-8859-1 / UTF7-IMAP encoded string
 	 */
-	function _encodeFolderName($_folderName) {
+	function _encodeFolderName($_folderName)
+	{
 		return Translation::convert($_folderName, self::$displayCharset, 'ISO-8859-1');
 		#return Translation::convert($_folderName, self::$displayCharset, 'UTF7-IMAP');
 	}
@@ -2339,21 +2733,36 @@ 
 	 */
 	function createFolder($_parent, $_folderName, &$_error)
 	{
-		if (self::$debug) error_log(__METHOD__.' ('.__LINE__.') '."->"."$_parent, $_folderName called from:".function_backtrace());
+		if (self::$debug)
+		{
+			error_log(__METHOD__.' ('.__LINE__.') '."->"."$_parent, $_folderName called from:".function_backtrace());
+		}
 		$parent		= $_parent;//$this->_encodeFolderName($_parent);
 		$folderName	= $_folderName;//$this->_encodeFolderName($_folderName);
 
-		if(empty($parent)) {
+		if(empty($parent))
+		{
 			$newFolderName = $folderName;
-		} else {
+		}
+		else
+		{
 			$HierarchyDelimiter = $this->getHierarchyDelimiter();
 			$newFolderName = $parent . $HierarchyDelimiter . $folderName;
 		}
-		if (empty($newFolderName)) return false;
-		if (self::$debug) error_log(__METHOD__.' ('.__LINE__.') '.'->'.$newFolderName);
+		if (empty($newFolderName))
+		{
+			return false;
+		}
+		if (self::$debug)
+		{
+			error_log(__METHOD__.' ('.__LINE__.') '.'->'.$newFolderName);
+		}
 		if ($this->folderExists($newFolderName,true))
 		{
-			if (self::$debug) error_log(__METHOD__.' ('.__LINE__.') '." Folder $newFolderName already exists.");
+			if (self::$debug)
+			{
+				error_log(__METHOD__.' ('.__LINE__.') '." Folder $newFolderName already exists.");
+			}
 			return $newFolderName;
 		}
 		try
@@ -2401,13 +2810,19 @@ 
 		$parent		= $_parent;//$this->_encodeFolderName($_parent);
 		$folderName	= $_folderName;//$this->_encodeFolderName($_folderName);
 
-		if(empty($parent)) {
+		if(empty($parent))
+		{
 			$newFolderName = $folderName;
-		} else {
+		}
+		else
+		{
 			$HierarchyDelimiter = $this->getHierarchyDelimiter();
 			$newFolderName = $parent . $HierarchyDelimiter . $folderName;
 		}
-		if (self::$debug) error_log("create folder: $newFolderName");
+		if (self::$debug)
+		{
+			error_log("create folder: $newFolderName");
+		}
 		try
 		{
 			$this->icServer->renameMailbox($oldFolderName, $newFolderName);
@@ -2481,8 +2896,14 @@ 
 	 */
 	function getFolderObjects($_subscribedOnly=false, $_getCounters=false, $_alwaysGetDefaultFolders=false,$_useCacheIfPossible=true)
 	{
-		if (self::$debug) error_log(__METHOD__.' ('.__LINE__.') '.' ServerID:'.$this->icServer->ImapServerId.", subscribedOnly:$_subscribedOnly, getCounters:$_getCounters, alwaysGetDefaultFolders:$_alwaysGetDefaultFolders, _useCacheIfPossible:$_useCacheIfPossible");
-		if (self::$debugTimes) $starttime = microtime (true);
+		if (self::$debug)
+		{
+			error_log(__METHOD__.' ('.__LINE__.') '.' ServerID:'.$this->icServer->ImapServerId.", subscribedOnly:$_subscribedOnly, getCounters:$_getCounters, alwaysGetDefaultFolders:$_alwaysGetDefaultFolders, _useCacheIfPossible:$_useCacheIfPossible");
+		}
+		if (self::$debugTimes)
+		{
+			$starttime = microtime (true);
+		}
 		static $folders2return;
 		//$_subscribedOnly=false;
 		// always use static on single request if info is available;
@@ -2490,23 +2911,35 @@ 
 		// set $_useCacheIfPossible to false !
 		if ($_useCacheIfPossible && isset($folders2return[$this->icServer->ImapServerId]) && !empty($folders2return[$this->icServer->ImapServerId]))
 		{
-			if (self::$debugTimes) self::logRunTimes($starttime,null,'using static',__METHOD__.' ('.__LINE__.') ');
+			if (self::$debugTimes)
+			{
+				self::logRunTimes($starttime,null,'using static',__METHOD__.' ('.__LINE__.') ');
+			}
 			return $folders2return[$this->icServer->ImapServerId];
 		}
 
 		if ($_subscribedOnly && $_getCounters===false)
 		{
-			if (is_null($folders2return)) $folders2return = Cache::getCache(Cache::INSTANCE,'email','folderObjects'.trim($GLOBALS['egw_info']['user']['account_id']),null,array(),60*60*1);
+			if (is_null($folders2return))
+			{
+				$folders2return = Cache::getCache(Cache::INSTANCE,'email','folderObjects'.trim($GLOBALS['egw_info']['user']['account_id']),null,array(),60*60*1);
+			}
 			if ($_useCacheIfPossible && isset($folders2return[$this->icServer->ImapServerId]) && !empty($folders2return[$this->icServer->ImapServerId]))
 			{
 				//error_log(__METHOD__.' ('.__LINE__.') '.' using Cached folderObjects'.array2string($folders2return[$this->icServer->ImapServerId]));
-				if (self::$debugTimes) self::logRunTimes($starttime,null,'from Cache',__METHOD__.' ('.__LINE__.') ');
+				if (self::$debugTimes)
+				{
+					self::logRunTimes($starttime,null,'from Cache',__METHOD__.' ('.__LINE__.') ');
+				}
 				return $folders2return[$this->icServer->ImapServerId];
 			}
 		}
 		// use $folderBasicInfo for holding attributes and other basic folderinfo $folderBasicInfo[$this->icServer->ImapServerId]
 		static $folderBasicInfo;
-		if (is_null($folderBasicInfo)||!isset($folderBasicInfo[$this->icServer->ImapServerId])) $folderBasicInfo = Cache::getCache(Cache::INSTANCE,'email','folderBasicInfo'.trim($GLOBALS['egw_info']['user']['account_id']),null,array(),60*60*1);
+		if (is_null($folderBasicInfo)||!isset($folderBasicInfo[$this->icServer->ImapServerId]))
+		{
+			$folderBasicInfo = Cache::getCache(Cache::INSTANCE,'email','folderBasicInfo'.trim($GLOBALS['egw_info']['user']['account_id']),null,array(),60*60*1);
+		}
 		//error_log(__METHOD__.' ('.__LINE__.') '.array2string(array_keys($folderBasicInfo[$this->icServer->ImapServerId])));
 
 		$delimiter = $this->getHierarchyDelimiter();
@@ -2519,7 +2952,8 @@ 
 		$inboxData->shortFolderName	= 'INBOX';
 		$inboxData->shortDisplayName	= lang('INBOX');
 		$inboxData->subscribed = true;
-		if($_getCounters == true) {
+		if($_getCounters == true)
+		{
 			$inboxData->counter = $this->getMailBoxCounters('INBOX');
 		}
 		// force unsubscribed by preference showAllFoldersInFolderPane
@@ -2538,12 +2972,14 @@ 
 		//error_log(__METHOD__.__LINE__.array2string($nameSpace));
 		if (is_array($nameSpace))
 		{
-			foreach($nameSpace as $k => $singleNameSpace) {
+			foreach($nameSpace as $k => $singleNameSpace)
+			{
 				$type = $singleNameSpace['type'];
 				// the following line (assumption that for the same namespace the delimiter should be equal) may be wrong
 				$foldersNameSpace[$type]['delimiter']  = $singleNameSpace['delimiter'];
 
-				if(is_array($singleNameSpace)&&$fetchedAllInOneGo==false) {
+				if(is_array($singleNameSpace)&&$fetchedAllInOneGo==false)
+				{
 					// fetch and sort the subscribed folders
 					// we alway fetch the subscribed, as this provides the only way to tell
 					// if a folder is subscribed or not
@@ -2594,8 +3030,12 @@ 
 							}
 						}
 						//error_log(__METHOD__.' ('.__LINE__.') '.' '.$type.'->'.array2string($foldersNameSpace[$type]['subscribed']));
-						if (!is_array($foldersNameSpace[$type]['all'])) $foldersNameSpace[$type]['all'] = array();
-						if ($_subscribedOnly == true && !empty($foldersNameSpace[$type]['subscribed'])) {
+						if (!is_array($foldersNameSpace[$type]['all']))
+						{
+							$foldersNameSpace[$type]['all'] = array();
+						}
+						if ($_subscribedOnly == true && !empty($foldersNameSpace[$type]['subscribed']))
+						{
 							continue;
 						}
 
@@ -2625,7 +3065,8 @@ 
 					}
 
 					//error_log(__METHOD__.' ('.__LINE__.') '.' '.$type.'->'.array2string($allMailboxesExt));
-					foreach ($allMailboxesExt as $mbx) {
+					foreach ($allMailboxesExt as $mbx)
+					{
 						if (!isset($folderBasicInfo[$this->icServer->ImapServerId][$mbx['MAILBOX']]))
 						{
 							$folderBasicInfo[$this->icServer->ImapServerId][$mbx['MAILBOX']]=array(
@@ -2648,20 +3089,33 @@ 
 						if (isset($allMailBoxesExtSorted[$mbx['MAILBOX']])||
 							isset($allMailBoxesExtSorted[$mbx['MAILBOX'].$foldersNameSpace[$type]['delimiter']])||
 							(substr($mbx['MAILBOX'],-1)==$foldersNameSpace[$type]['delimiter'] && isset($allMailBoxesExtSorted[substr($mbx['MAILBOX'],0,-1)]))
-						) continue;
+						)
+						{
+							continue;
+						}
 
 						//echo '#'.$mbx['MAILBOX'].':'.array2string($mbx)."#<br>";
 						$allMailBoxesExtSorted[$mbx['MAILBOX']] = $mbx;
 					}
-					if (is_array($allMailBoxesExtSorted)) ksort($allMailBoxesExtSorted);
+					if (is_array($allMailBoxesExtSorted))
+					{
+						ksort($allMailBoxesExtSorted);
+					}
 					//_debug_array(array_keys($allMailBoxesExtSorted));
 					$allMailboxes = array();
-					foreach ((array)$allMailBoxesExtSorted as $mbx) {
-						if (!in_array($mbx['MAILBOX'],$allMailboxes)) $allMailboxes[] = $mbx['MAILBOX'];
+					foreach ((array)$allMailBoxesExtSorted as $mbx)
+					{
+						if (!in_array($mbx['MAILBOX'],$allMailboxes))
+						{
+							$allMailboxes[] = $mbx['MAILBOX'];
+						}
 						//echo "Result:";_debug_array($allMailboxes);
 					}
 					$foldersNameSpace[$type]['all'] = $allMailboxes;
-					if (is_array($foldersNameSpace[$type]['all'])) sort($foldersNameSpace[$type]['all']);
+					if (is_array($foldersNameSpace[$type]['all']))
+					{
+						sort($foldersNameSpace[$type]['all']);
+					}
 				}
 			}
 		}
@@ -2670,24 +3124,50 @@ 
 		//echo "<br>FolderNameSpace To Process:";_debug_array($foldersNameSpace);
 		$autoFolderObjects = $folders = array();
 		$autofolder_exists = array();
-		foreach( array('personal', 'others', 'shared') as $type) {
-			if(isset($foldersNameSpace[$type])) {
-				if($_subscribedOnly) {
-					if( !empty($foldersNameSpace[$type]['subscribed']) ) $listOfFolders = $foldersNameSpace[$type]['subscribed'];
-				} else {
-					if( !empty($foldersNameSpace[$type]['all'])) $listOfFolders = $foldersNameSpace[$type]['all'];
-				}
-				foreach((array)$listOfFolders as $folderName) {
+		foreach( array('personal', 'others', 'shared') as $type)
+		{
+			if(isset($foldersNameSpace[$type]))
+			{
+				if($_subscribedOnly)
+				{
+					if( !empty($foldersNameSpace[$type]['subscribed']) )
+					{
+						$listOfFolders = $foldersNameSpace[$type]['subscribed'];
+					}
+				}
+				else
+				{
+					if( !empty($foldersNameSpace[$type]['all']))
+					{
+						$listOfFolders = $foldersNameSpace[$type]['all'];
+					}
+				}
+				foreach((array)$listOfFolders as $folderName)
+				{
 					//echo "<br>FolderToCheck:$folderName<br>";
 					//error_log(__METHOD__.__LINE__.'#Delimiter:'.$delimiter.':#'.$folderName);
-					if ($_subscribedOnly && empty($foldersNameSpace[$type]['all'])) continue;//when subscribedonly, we fetch all folders in one go.
-					if($_subscribedOnly && !(in_array($folderName, $foldersNameSpace[$type]['all'])||in_array($folderName.$foldersNameSpace[$type]['delimiter'], $foldersNameSpace[$type]['all']))) {
+					if ($_subscribedOnly && empty($foldersNameSpace[$type]['all']))
+					{
+						continue;
+					}
+					//when subscribedonly, we fetch all folders in one go.
+					if($_subscribedOnly && !(in_array($folderName, $foldersNameSpace[$type]['all'])||in_array($folderName.$foldersNameSpace[$type]['delimiter'], $foldersNameSpace[$type]['all'])))
+					{
 						#echo "$folderName failed to be here <br>";
 						continue;
 					}
-					if (isset($folders[$folderName])) continue;
-					if (isset($autoFolderObjects[$folderName])) continue;
-					if (empty($delimiter)||$delimiter != $foldersNameSpace[$type]['delimiter']) $delimiter = $foldersNameSpace[$type]['delimiter'];
+					if (isset($folders[$folderName]))
+					{
+						continue;
+					}
+					if (isset($autoFolderObjects[$folderName]))
+					{
+						continue;
+					}
+					if (empty($delimiter)||$delimiter != $foldersNameSpace[$type]['delimiter'])
+					{
+						$delimiter = $foldersNameSpace[$type]['delimiter'];
+					}
 					$folderParts = explode($delimiter, $folderName);
 					$shortName = array_pop($folderParts);
 
@@ -2695,17 +3175,20 @@ 
 					$folderObject->delimiter	= $delimiter;
 					$folderObject->folderName	= $folderName;
 					$folderObject->shortFolderName	= $shortName;
-					if(!$_subscribedOnly) {
+					if(!$_subscribedOnly)
+					{
 						#echo $folderName."->".$type."<br>";
 						#_debug_array($foldersNameSpace[$type]['subscribed']);
 						$folderObject->subscribed = in_array($folderName, (array)$foldersNameSpace[$type]['subscribed']);
 					}
 
-					if($_getCounters == true) {
+					if($_getCounters == true)
+					{
 						//error_log(__METHOD__.' ('.__LINE__.') '.' getCounter forFolder:'.$folderName);
 						$folderObject->counter = $this->getMailBoxCounters($folderName);
 					}
-					if(strtoupper($folderName) == 'INBOX') {
+					if(strtoupper($folderName) == 'INBOX')
+					{
 						$folderName = 'INBOX';
 						$folderObject->folderName	= 'INBOX';
 						$folderObject->shortFolderName	= 'INBOX';
@@ -2713,24 +3196,34 @@ 
 						$folderObject->shortDisplayName = lang('INBOX');
 						$folderObject->subscribed	= true;
 					// translate the automatic Folders (Sent, Drafts, ...) like the INBOX
-					} elseif (in_array($shortName,self::$autoFolders)) {
+					}
+					elseif (in_array($shortName,self::$autoFolders))
+					{
 						$tmpfolderparts = explode($delimiter,$folderObject->folderName);
 						array_pop($tmpfolderparts);
 						$folderObject->displayName = implode($delimiter,$tmpfolderparts).$delimiter.lang($shortName);
 						$folderObject->shortDisplayName = lang($shortName);
 						unset($tmpfolderparts);
-					} else {
+					}
+					else
+					{
 						$folderObject->displayName = $folderObject->folderName;
 						$folderObject->shortDisplayName = $shortName;
 					}
 					//$folderName = $folderName;
-					if (in_array($shortName,self::$autoFolders)&&self::searchValueInFolderObjects($shortName,$autoFolderObjects)===false) {
+					if (in_array($shortName,self::$autoFolders)&&self::searchValueInFolderObjects($shortName,$autoFolderObjects)===false)
+					{
 						$autoFolderObjects[$folderName] = $folderObject;
-					} else {
+					}
+					else
+					{
 						$folders[$folderName] = $folderObject;
 					}
 					//error_log(__METHOD__.' ('.__LINE__.') '.':'.$folderObject->folderName);
-					if (!isset(self::$specialUseFolders)) $this->getSpecialUseFolders ();
+					if (!isset(self::$specialUseFolders))
+					{
+						$this->getSpecialUseFolders ();
+					}
 					if (isset(self::$specialUseFolders[$folderName]))
 					{
 						$autofolder_exists[$folderName] = self::$specialUseFolders[$folderName];
@@ -2738,7 +3231,8 @@ 
 				}
 			}
 		}
-		if (is_array($autoFolderObjects) && !empty($autoFolderObjects)) {
+		if (is_array($autoFolderObjects) && !empty($autoFolderObjects))
+		{
 			uasort($autoFolderObjects,array($this,"sortByAutoFolderPos"));
 		}
 		// check if some standard folders are missing and need to be created
@@ -2747,7 +3241,10 @@ 
 			// if new folders have been created, re-read folders ignoring the cache
 			return $this->getFolderObjects($_subscribedOnly, $_getCounters, $_alwaysGetDefaultFolders, false);	// false = do NOT use cache
 		}
-		if (is_array($folders)) uasort($folders,array($this,"sortByDisplayName"));
+		if (is_array($folders))
+		{
+			uasort($folders,array($this,"sortByDisplayName"));
+		}
 		//$folders2return = array_merge($autoFolderObjects,$folders);
 		//_debug_array($folders2return); #exit;
 		$folders2return[$this->icServer->ImapServerId] = array_merge((array)$inboxFolderObject,(array)$autoFolderObjects,(array)$folders);
@@ -2759,7 +3256,10 @@ 
 			Cache::setCache(Cache::INSTANCE,'email','folderObjects'.trim($GLOBALS['egw_info']['user']['account_id']),$folders2return,$expiration=60*60*1);
 		}
 		Cache::setCache(Cache::INSTANCE,'email','folderBasicInfo'.trim($GLOBALS['egw_info']['user']['account_id']),$folderBasicInfo,$expiration=60*60*1);
-		if (self::$debugTimes) self::logRunTimes($starttime,null,function_backtrace(),__METHOD__.' ('.__LINE__.') ');
+		if (self::$debugTimes)
+		{
+			self::logRunTimes($starttime,null,function_backtrace(),__METHOD__.' ('.__LINE__.') ');
+		}
 		return $folders2return[$this->icServer->ImapServerId];
 	}
 
@@ -2787,31 +3287,44 @@ 
 
 		$folders = $nameSpace =  array();
 		$nameSpaceTmp = $this->_getNameSpaces();
-		foreach($nameSpaceTmp as $k => $singleNameSpace) {
+		foreach($nameSpaceTmp as $k => $singleNameSpace)
+		{
 			$nameSpace[$singleNameSpace['type']]=$singleNameSpace;
 		}
 		unset($nameSpaceTmp);
 
 		//error_log(__METHOD__.__LINE__.array2string($nameSpace));
 		// Get special use folders
-		if (!isset(self::$specialUseFolders)) $this->getSpecialUseFolders (); // Set self::$specialUseFolders
+		if (!isset(self::$specialUseFolders))
+		{
+			$this->getSpecialUseFolders ();
+		}
+		// Set self::$specialUseFolders
 		// topLevelQueries generally ignore the $_search param. Except for Config::examineNamespace
-		if ($_onlyTopLevel) // top level leaves
+		if ($_onlyTopLevel)
+		{
+			// top level leaves
 		{
 			// Get top mailboxes of icServer
 			$topFolders = $this->icServer->getMailboxes("", 2, true);
+		}
 			// Trigger examination of namespace to retrieve
 			// folders located in other and shared; needed only for some servers
-			if (is_null(self::$mailConfig)) self::$mailConfig = Config::read('mail');
+			if (is_null(self::$mailConfig))
+			{
+				self::$mailConfig = Config::read('mail');
+			}
 			if (self::$mailConfig['examineNamespace'])
 			{
 				$prefixes=array();
 				if (is_array($nameSpace))
 				{
-					foreach($nameSpace as $k => $singleNameSpace) {
+					foreach($nameSpace as $k => $singleNameSpace)
+					{
 						$type = $singleNameSpace['type'];
 
-						if(is_array($singleNameSpace) && $singleNameSpace['prefix']){
+						if(is_array($singleNameSpace) && $singleNameSpace['prefix'])
+						{
 							$prefixes[$type] = $singleNameSpace['prefix'];
 							//regard extra care for nameSpacequeries when configured AND respect $_search
 							$result = $this->icServer->getMailboxes($singleNameSpace['prefix'], $_search==0?0:2, true);
@@ -2887,7 +3400,10 @@ 
 							$nFolders [$path] = $folder;
 						}
 					}
-					if (is_array($aFolders)) uasort ($aFolders, array($this,'sortByAutofolder'));
+					if (is_array($aFolders))
+					{
+						uasort ($aFolders, array($this,'sortByAutofolder'));
+					}
 					//ksort($aFolders);
 
 					// Sort none auto folders base on mailbox name
@@ -2897,12 +3413,17 @@ 
 				}
 				else
 				{
-					if (is_array($subFolders)) ksort($subFolders);
+					if (is_array($subFolders))
+					{
+						ksort($subFolders);
+					}
 				}
 				$folders = array_merge($folders,(array)$mainFolder, (array)$subFolders);
 			}
 		}
-		elseif ($_nodePath) // single node
+		elseif ($_nodePath)
+		{
+			// single node
 		{
 			switch ($_search)
 			{
@@ -2910,6 +3431,7 @@ 
 				case 0:
 				case 2:
 					$path = $_nodePath.''.$delimiter;
+		}
 					break;
 				// Node itself
 				// shouldn't contain next level delimiter
@@ -2928,11 +3450,14 @@ 
 
 			uasort($folders,array($this,'sortByMailbox'));//ksort($folders);
 		}
-		elseif(!$_nodePath) // all
+		elseif(!$_nodePath)
+		{
+			// all
 		{
 			if ($_subscribedOnly)
 			{
 				$folders = $this->icServer->listSubscribedMailboxes('', 0, true);
+		}
 			}
 			else
 			{
@@ -2944,7 +3469,10 @@ 
 		{
 			// SORTING FOLDERS
 			//self::$debugTimes=true;
-			if (self::$debugTimes) $starttime = microtime (true);
+			if (self::$debugTimes)
+			{
+				$starttime = microtime (true);
+			}
 			// Merge of all auto folders and specialusefolders
 			$autoFoldersTmp = array_unique((array_merge(self::$autoFolders, array_values(self::$specialUseFolders))));
 			uasort($folders,array($this,'sortByMailbox'));//ksort($folders);
@@ -2954,73 +3482,103 @@ 
 			$isGoogleMail=false;
 			foreach($autoFoldersTmp as $afk=>$aF)
 			{
-				if (!isset($mySpecialUseFolders[$aF]) && $aF) $mySpecialUseFolders[$aF]=$this->getFolderByType($aF,false);
+				if (!isset($mySpecialUseFolders[$aF]) && $aF)
+				{
+					$mySpecialUseFolders[$aF]=$this->getFolderByType($aF,false);
+				}
 				//error_log($afk.':'.$aF.'->'.$mySpecialUseFolders[$aF]);
 			}
 			//error_log(array2string($mySpecialUseFolders));
-			foreach ($tmpFolders as $k => $f) {
+			foreach ($tmpFolders as $k => $f)
+			{
 				$sorted=false;
-				if (strtoupper(substr($k,0,5))=='INBOX') {
-					if (strtoupper($k)=='INBOX') {
+				if (strtoupper(substr($k,0,5))=='INBOX')
+				{
+					if (strtoupper($k)=='INBOX')
+					{
 						//error_log(__METHOD__.__LINE__.':'.strtoupper(substr($k,0,5)).':'.$k);
 						$inboxFolderObject[$k]=$f;
 						unset($folders[$k]);
 						$sorted=true;
-					} else {
+					}
+					else
+					{
 						$isAutoFolder=false;
 						foreach($autoFoldersTmp as $afk=>$aF)
 						{
 							//error_log(__METHOD__.__LINE__.$k.':'.$aF.'->'.$mySpecialUseFolders[$aF]);
 							if($aF && strlen($mySpecialUseFolders[$aF])&&/*strlen($k)>=strlen($mySpecialUseFolders[$aF])&&*/
 								($mySpecialUseFolders[$aF]==$k || substr($k,0,strlen($mySpecialUseFolders[$aF].$delimiter))==$mySpecialUseFolders[$aF].$delimiter || //k may be child of an autofolder
-								stristr($mySpecialUseFolders[$aF],$k.$delimiter)!==false)) // k is parent of an autofolder
+								stristr($mySpecialUseFolders[$aF],$k.$delimiter)!==false))
+							{
+								// k is parent of an autofolder
 							{
 								//error_log(__METHOD__.__LINE__.$k.'->'.$mySpecialUseFolders[$aF]);
 								$isAutoFolder=true;
+							}
 								$autoFolderObjects[$k]=$f;
 								break;
 							}
 						}
-						if ($isAutoFolder==false) $inboxSubFolderObjects[$k]=$f;
+						if ($isAutoFolder==false)
+						{
+							$inboxSubFolderObjects[$k]=$f;
+						}
 						unset($folders[$k]);
 						$sorted=true;
 					}
-				} elseif (strtoupper(substr($k,0,13))=='[GOOGLE MAIL]') {
+				}
+				elseif (strtoupper(substr($k,0,13))=='[GOOGLE MAIL]')
+				{
 					$isGoogleMail=true;
-					if (strtoupper($k)=='[GOOGLE MAIL]') {
+					if (strtoupper($k)=='[GOOGLE MAIL]')
+					{
 						$googleMailFolderObject[$k]=$f;
 						unset($folders[$k]);
 						$sorted=true;
-					} else {
+					}
+					else
+					{
 						$isAutoFolder=false;
 						foreach($autoFoldersTmp as $afk=>$aF)
 						{
 							//error_log($k.':'.$aF.'->'.$mySpecialUseFolders[$aF]);
 							if($aF && strlen($mySpecialUseFolders[$aF])&&/*strlen($k)>=strlen($mySpecialUseFolders[$aF])&&*/
 								($mySpecialUseFolders[$aF]==$k || substr($k,0,strlen($mySpecialUseFolders[$aF].$delimiter))==$mySpecialUseFolders[$aF].$delimiter|| //k may be child of an autofolder
-								stristr($mySpecialUseFolders[$aF],$k.$delimiter)!==false)) // k is parent of an autofolder
+								stristr($mySpecialUseFolders[$aF],$k.$delimiter)!==false))
+							{
+								// k is parent of an autofolder
 							{
 								//error_log($k.'->'.$mySpecialUseFolders[$aF]);
 								$isAutoFolder=true;
+							}
 								$googleAutoFolderObjects[$k]=$f;
 								break;
 							}
 						}
-						if ($isAutoFolder==false) $googleSubFolderObjects[$k]=$f;
+						if ($isAutoFolder==false)
+						{
+							$googleSubFolderObjects[$k]=$f;
+						}
 						unset($folders[$k]);
 						$sorted=true;
 					}
-				} else {
+				}
+				else
+				{
 					$isAutoFolder=false;
 					foreach($autoFoldersTmp as $afk=>$aF)
 					{
 						//error_log($k.':'.$aF.'->'.$mySpecialUseFolders[$aF]);
 						if($aF && strlen($mySpecialUseFolders[$aF])&&/*strlen($k)>=strlen($mySpecialUseFolders[$aF])&&*/
 								($mySpecialUseFolders[$aF]==$k || substr($k,0,strlen($mySpecialUseFolders[$aF].$delimiter))==$mySpecialUseFolders[$aF].$delimiter|| //k may be child of an autofolder
-								stristr($mySpecialUseFolders[$aF],$k.$delimiter)!==false)) // k is parent of an autofolder
+								stristr($mySpecialUseFolders[$aF],$k.$delimiter)!==false))
+						{
+							// k is parent of an autofolder
 						{
 							//error_log($k.'->'.$mySpecialUseFolders[$aF]);
 							$isAutoFolder=true;
+						}
 							$autoFolderObjects[$k]=$f;
 							unset($folders[$k]);
 							$sorted=true;
@@ -3036,7 +3594,8 @@ 
 						if ($nameSpace[$type]['prefix_present']&&$nameSpace[$type]['prefix'])
 						{
 							if (substr($k,0,strlen($nameSpace[$type]['prefix']))==$nameSpace[$type]['prefix']||
-								substr($k,0,strlen($nameSpace[$type]['prefix'])-strlen($nameSpace[$type]['delimiter']))==substr($nameSpace[$type]['prefix'],0,strlen($nameSpace[$type]['delimiter'])*-1)) {
+								substr($k,0,strlen($nameSpace[$type]['prefix'])-strlen($nameSpace[$type]['delimiter']))==substr($nameSpace[$type]['prefix'],0,strlen($nameSpace[$type]['delimiter'])*-1))
+							{
 								//error_log(__METHOD__.__LINE__.':'.substr($k,0,strlen($nameSpace[$type]['prefix'])).':'.$k);
 								$typeFolderObject[$type][$k]=$f;
 								unset($folders[$k]);
@@ -3063,9 +3622,12 @@ 
 				}
 			}
 			//error_log(__METHOD__.__LINE__.array2string($autoFolderObjects));
-			if (!$isGoogleMail) {
+			if (!$isGoogleMail)
+			{
 				$folders = array_merge($inboxFolderObject,$autoFolderObjects,(array)$inboxSubFolderObjects,(array)$folders,(array)$typeFolderObject['others'],(array)$typeFolderObject['shared']);
-			} else {
+			}
+			else
+			{
 				// avoid calling sortByAutoFolder as it is not regarding subfolders
 				$gAutoFolderObjectsTmp = $googleAutoFolderObjects;
 				unset($googleAutoFolderObjects);
@@ -3082,7 +3644,10 @@ 
 				}
 				$folders = array_merge($inboxFolderObject,$autoFolderObjects,(array)$folders,(array)$googleMailFolderObject,$googleAutoFolderObjects,$googleSubFolderObjects,(array)$typeFolderObject['others'],(array)$typeFolderObject['shared']);
 			}
-			if (self::$debugTimes) self::logRunTimes($starttime,null,function_backtrace(),__METHOD__.' ('.__LINE__.') Sorting:');
+			if (self::$debugTimes)
+			{
+				self::logRunTimes($starttime,null,function_backtrace(),__METHOD__.' ('.__LINE__.') Sorting:');
+			}
 			//self::$debugTimes=false;
 		}
 		// Get counter information and add them to each fetched folders array
@@ -3132,7 +3697,13 @@ 
 		$rv = false;
 		foreach ($haystack as $k => $v)
 		{
-			foreach($v as &$sv) {if (trim($sv)==trim($needle)) return $k;}
+			foreach($v as &$sv)
+			{
+if (trim($sv)==trim($needle))
+			{
+				return $k;
+			}
+			}
 		}
 		return $rv;
 	}
@@ -3159,7 +3730,10 @@ 
 	 */
 	static function pathToFolderData ($_path, $_hDelimiter)
 	{
-		if (!strpos($_path, self::DELIMITER)) $_path = self::DELIMITER.$_path;
+		if (!strpos($_path, self::DELIMITER))
+		{
+			$_path = self::DELIMITER.$_path;
+		}
 		list(,$path) = explode(self::DELIMITER, $_path);
 		$path_chain = $parts = explode($_hDelimiter, $path);
 		$name = array_pop($parts);
@@ -3188,7 +3762,10 @@ 
 		$b = self::pathToFolderData($_b['MAILBOX'], $_b['delimiter']);
 		$pos1 = array_search(trim($a['name']),self::$autoFolders);
 		$pos2 = array_search(trim($b['name']),self::$autoFolders);
-		if ($pos1 == $pos2) return 0;
+		if ($pos1 == $pos2)
+		{
+			return 0;
+		}
 		return ($pos1 < $pos2) ? -1 : 1;
 	}
 
@@ -3219,7 +3796,10 @@ 
 		// 0, 1 und -1
 		$pos1 = array_search(trim($a->shortFolderName),self::$autoFolders);
 		$pos2 = array_search(trim($b->shortFolderName),self::$autoFolders);
-		if ($pos1 == $pos2) return 0;
+		if ($pos1 == $pos2)
+		{
+			return 0;
+		}
 		return ($pos1 < $pos2) ? -1 : 1;
 	}
 
@@ -3240,11 +3820,18 @@ 
 		}
 		catch (\Exception $e)
 		{
-			if (self::$debug) error_log(__METHOD__." returned FolderStatus for Folder $folderName:".$e->getMessage());
+			if (self::$debug)
+			{
+				error_log(__METHOD__." returned FolderStatus for Folder $folderName:".$e->getMessage());
+			}
 			return false;
 		}
-		if(is_array($folderStatus)) {
-			if ($_returnObject===false) return $folderStatus;
+		if(is_array($folderStatus))
+		{
+			if ($_returnObject===false)
+			{
+				return $folderStatus;
+			}
 			$status =  new \stdClass;
 			$status->messages   = $folderStatus['MESSAGES'];
 			$status->unseen     = $folderStatus['UNSEEN'];
@@ -3271,7 +3858,8 @@ 
 	{
 		#echo __METHOD__." retrieve SubFolders for $_mailbox$delimiter <br>";
 		$maxreclevel=25;
-		if ($reclevel > $maxreclevel) {
+		if ($reclevel > $maxreclevel)
+		{
 			error_log( __METHOD__." Recursion Level Exeeded ($reclevel) while looking up $_mailbox$delimiter ");
 			return array();
 		}
@@ -3285,7 +3873,8 @@ 
 //error_log(__METHOD__.' ('.__LINE__.') '.' Delimiter:'.array2string($delimiter));
 //error_log(__METHOD__.' ('.__LINE__.') '.array2string($mbx));
 		// Example: Array([INBOX/GaGa] => Array([MAILBOX] => INBOX/GaGa[ATTRIBUTES] => Array([0] => \\unmarked)[delimiter] => /))
-		if (is_array($mbx[$mbxkeys[0]]["ATTRIBUTES"]) && (in_array('\HasChildren',$mbx[$mbxkeys[0]]["ATTRIBUTES"]) || in_array('\Haschildren',$mbx[$mbxkeys[0]]["ATTRIBUTES"]) || in_array('\haschildren',$mbx[$mbxkeys[0]]["ATTRIBUTES"]))) {
+		if (is_array($mbx[$mbxkeys[0]]["ATTRIBUTES"]) && (in_array('\HasChildren',$mbx[$mbxkeys[0]]["ATTRIBUTES"]) || in_array('\Haschildren',$mbx[$mbxkeys[0]]["ATTRIBUTES"]) || in_array('\haschildren',$mbx[$mbxkeys[0]]["ATTRIBUTES"])))
+		{
 			// if there are children fetch them
 			//echo $mbx[$mbxkeys[0]]['MAILBOX']."<br>";
 
@@ -3293,7 +3882,8 @@ 
 			//$buff = $this->icServer->getMailboxes($mbx[$mbxkeys[0]]['MAILBOX'],2,false);
 			//_debug_array($buff);
 			$allMailboxes = array();
-			foreach ($buff as $mbxname) {
+			foreach ($buff as $mbxname)
+			{
 //error_log(__METHOD__.' ('.__LINE__.') '.array2string($mbxname));
 				$mbxname = preg_replace('~'.($delimiter == '.' ? "\\".$delimiter:$delimiter).'+~s',$delimiter,$mbxname['MAILBOX']);
 				#echo "About to recur in level $reclevel:".$mbxname."<br>";
@@ -3302,9 +3892,14 @@ 
 					$allMailboxes = array_merge($allMailboxes, self::getMailBoxesRecursive($mbxname, $delimiter, $prefix, $reclevel));
 				}
 			}
-			if (!(in_array('\NoSelect',$mbx[$mbxkeys[0]]["ATTRIBUTES"]) || in_array('\Noselect',$mbx[$mbxkeys[0]]["ATTRIBUTES"]) || in_array('\noselect',$mbx[$mbxkeys[0]]["ATTRIBUTES"]))) $allMailboxes[] = $mbx[$mbxkeys[0]]['MAILBOX'];
+			if (!(in_array('\NoSelect',$mbx[$mbxkeys[0]]["ATTRIBUTES"]) || in_array('\Noselect',$mbx[$mbxkeys[0]]["ATTRIBUTES"]) || in_array('\noselect',$mbx[$mbxkeys[0]]["ATTRIBUTES"])))
+			{
+				$allMailboxes[] = $mbx[$mbxkeys[0]]['MAILBOX'];
+			}
 			return $allMailboxes;
-		} else {
+		}
+		else
+		{
 			return array($_mailbox);
 		}
 	}
@@ -3328,14 +3923,21 @@ 
 			'Outbox'   => array('profileKey'=>'acc_folder_outbox','autoFolderName'=>'Outbox'),
 			'Archive'   => array('profileKey'=>'acc_folder_archive','autoFolderName'=>'Archive'),
 		);
-		if ($_type == 'Templates') $_type = 'Template';	// for some reason self::$autofolders uses 'Templates'!
+		if ($_type == 'Templates')
+		{
+			$_type = 'Template';
+		}
+		// for some reason self::$autofolders uses 'Templates'!
 		$created = false;
 		if (!isset($types[$_type]))
 		{
 			error_log(__METHOD__.' ('.__LINE__.') '.' '.$_type.' not supported for '.__METHOD__);
 			return false;
 		}
-		if (is_null(self::$specialUseFolders) || empty(self::$specialUseFolders)) self::$specialUseFolders = $this->getSpecialUseFolders();
+		if (is_null(self::$specialUseFolders) || empty(self::$specialUseFolders))
+		{
+			self::$specialUseFolders = $this->getSpecialUseFolders();
+		}
 
 		//highest precedence
 		try
@@ -3345,25 +3947,39 @@ 
 		catch (\Exception $e)
 		{
 			// we know that outbox is not supported, but we use this here, as we autocreate expected SpecialUseFolders in this function
-			if ($_type != 'Outbox') error_log(__METHOD__.' ('.__LINE__.') '.' Failed to retrieve Folder'.$_folderName." for ".array2string($types[$_type]).":".$e->getMessage());
+			if ($_type != 'Outbox')
+			{
+				error_log(__METHOD__.' ('.__LINE__.') '.' Failed to retrieve Folder'.$_folderName." for ".array2string($types[$_type]).":".$e->getMessage());
+			}
 			$_folderName = false;
 		}
 		// do not try to autocreate configured Archive-Folder. Return false if configured folder does not exist
-		if ($_type == 'Archive') {
-			if ($_folderName && $_checkexistance && strtolower($_folderName) !='none' && !$this->folderExists($_folderName,true)) {
+		if ($_type == 'Archive')
+		{
+			if ($_folderName && $_checkexistance && strtolower($_folderName) !='none' && !$this->folderExists($_folderName,true))
+			{
 				return false;
-			} else {
+			}
+			else
+			{
 				return $_folderName;
 			}
 
 		}
 		// does the folder exist??? (is configured/preset, but non-existent)
-		if ($_folderName && $_checkexistance && strtolower($_folderName) !='none' && !$this->folderExists($_folderName,true)) {
+		if ($_folderName && $_checkexistance && strtolower($_folderName) !='none' && !$this->folderExists($_folderName,true))
+		{
 			try
 			{
 				$error = null;
-				if (($_folderName = $this->createFolder('', $_folderName, $error))) $created = true;
-				if ($error) error_log(__METHOD__.' ('.__LINE__.') '.' Failed to create Folder '.$_folderName." for $_type:".$error);
+				if (($_folderName = $this->createFolder('', $_folderName, $error)))
+				{
+					$created = true;
+				}
+				if ($error)
+				{
+					error_log(__METHOD__.' ('.__LINE__.') '.' Failed to create Folder '.$_folderName." for $_type:".$error);
+				}
 			}
 			catch(Exception $e)
 			{
@@ -3372,9 +3988,16 @@ 
 			}
 		}
 		// not sure yet if false is the correct behavior on none
-		if ($_folderName =='none') return 'none' ; //false;
+		if ($_folderName =='none')
+		{
+			return 'none' ;
+		}
+		//false;
 		//no (valid) folder found yet; try specialUseFolders
-		if (empty($_folderName) && is_array(self::$specialUseFolders) && ($f = array_search($_type,self::$specialUseFolders))) $_folderName = $f;
+		if (empty($_folderName) && is_array(self::$specialUseFolders) && ($f = array_search($_type,self::$specialUseFolders)))
+		{
+			$_folderName = $f;
+		}
 		//no specialUseFolder; try some Defaults
 		if (empty($_folderName) && isset($types[$_type]))
 		{
@@ -3400,7 +4023,10 @@ 
 					$error = null;
 					$this->createFolder('', $prefix.$types[$_type]['autoFolderName'],$error);
 					$_folderName = $prefix.$types[$_type]['autoFolderName'];
-					if ($error) error_log(__METHOD__.' ('.__LINE__.') '.' Failed to create Folder '.$_folderName." for $_type:".$error);
+					if ($error)
+					{
+						error_log(__METHOD__.' ('.__LINE__.') '.' Failed to create Folder '.$_folderName." for $_type:".$error);
+					}
 				}
 				catch(Exception $e)
 				{
@@ -3502,17 +4128,22 @@ 
 	function isSentFolder($_folderName, $_checkexistance=TRUE)
 	{
 		$sentFolder = $this->getSentFolder($_checkexistance);
-		if(empty($sentFolder)) {
+		if(empty($sentFolder))
+		{
 			return false;
 		}
 		// does the folder exist???
-		if ($_checkexistance && !$this->folderExists($_folderName)) {
+		if ($_checkexistance && !$this->folderExists($_folderName))
+		{
 			return false;
 		}
 
-		if(false !== stripos($_folderName, $sentFolder)) {
+		if(false !== stripos($_folderName, $sentFolder))
+		{
 			return true;
-		} else {
+		}
+		else
+		{
 			return false;
 		}
 	}
@@ -3525,15 +4156,20 @@ 
 	 */
 	function isOutbox($_folderName, $_checkexistance=TRUE)
 	{
-		if (stripos($_folderName, 'Outbox')===false) {
+		if (stripos($_folderName, 'Outbox')===false)
+		{
 			return false;
 		}
 		// does the folder exist???
-		if ($_checkexistance && $GLOBALS['egw_info']['user']['apps']['activesync'] && !$this->folderExists($_folderName)) {
+		if ($_checkexistance && $GLOBALS['egw_info']['user']['apps']['activesync'] && !$this->folderExists($_folderName))
+		{
 			$outboxFolder = $this->getOutboxFolder($_checkexistance);
-			if(false !== stripos($_folderName, $outboxFolder)) {
+			if(false !== stripos($_folderName, $outboxFolder))
+			{
 				return true;
-			} else {
+			}
+			else
+			{
 				return false;
 			}
 		}
@@ -3549,17 +4185,25 @@ 
 	function isDraftFolder($_folderName, $_checkexistance=TRUE)
 	{
 		$draftFolder = $this->getDraftFolder($_checkexistance);
-		if(empty($draftFolder)) {
+		if(empty($draftFolder))
+		{
 			return false;
 		}
 		// does the folder exist???
-		if ($_checkexistance && !$this->folderExists($_folderName)) {
+		if ($_checkexistance && !$this->folderExists($_folderName))
+		{
 			return false;
 		}
-		if (is_a($_folderName,"Horde_Imap_Client_Mailbox")) $_folderName = $_folderName->utf8;
-		if(false !== stripos($_folderName, $draftFolder)) {
+		if (is_a($_folderName,"Horde_Imap_Client_Mailbox"))
+		{
+			$_folderName = $_folderName->utf8;
+		}
+		if(false !== stripos($_folderName, $draftFolder))
+		{
 			return true;
-		} else {
+		}
+		else
+		{
 			return false;
 		}
 	}
@@ -3573,17 +4217,22 @@ 
 	function isTrashFolder($_folderName, $_checkexistance=TRUE)
 	{
 		$trashFolder = $this->getTrashFolder($_checkexistance);
-		if(empty($trashFolder)) {
+		if(empty($trashFolder))
+		{
 			return false;
 		}
 		// does the folder exist???
-		if ($_checkexistance && !$this->folderExists($_folderName)) {
+		if ($_checkexistance && !$this->folderExists($_folderName))
+		{
 			return false;
 		}
 
-		if(false !== stripos($_folderName, $trashFolder)) {
+		if(false !== stripos($_folderName, $trashFolder))
+		{
 			return true;
-		} else {
+		}
+		else
+		{
 			return false;
 		}
 	}
@@ -3597,17 +4246,22 @@ 
 	function isTemplateFolder($_folderName, $_checkexistance=TRUE)
 	{
 		$templateFolder = $this->getTemplateFolder($_checkexistance);
-		if(empty($templateFolder)) {
+		if(empty($templateFolder))
+		{
 			return false;
 		}
 		// does the folder exist???
-		if ($_checkexistance && !$this->folderExists($_folderName)) {
+		if ($_checkexistance && !$this->folderExists($_folderName))
+		{
 			return false;
 		}
 
-		if(false !== stripos($_folderName, $templateFolder)) {
+		if(false !== stripos($_folderName, $templateFolder))
+		{
 			return true;
-		} else {
+		}
+		else
+		{
 			return false;
 		}
 	}
@@ -3625,15 +4279,27 @@ 
 		if (empty($_folder))
 		{
 			// this error is more or less without significance, unless we force the check
-			if ($_forceCheck===true) error_log(__METHOD__.' ('.__LINE__.') '.' Called with empty Folder:'.$_folder.function_backtrace());
+			if ($_forceCheck===true)
+			{
+				error_log(__METHOD__.' ('.__LINE__.') '.' Called with empty Folder:'.$_folder.function_backtrace());
+			}
 			return false;
 		}
 		// when check is not enforced , we assume a folder represented as Horde_Imap_Client_Mailbox as existing folder
-		if (is_a($_folder,"Horde_Imap_Client_Mailbox")&&$_forceCheck===false) return true;
-		if (is_a($_folder,"Horde_Imap_Client_Mailbox")) $_folder =  $_folder->utf8;
+		if (is_a($_folder,"Horde_Imap_Client_Mailbox")&&$_forceCheck===false)
+		{
+			return true;
+		}
+		if (is_a($_folder,"Horde_Imap_Client_Mailbox"))
+		{
+			$_folder =  $_folder->utf8;
+		}
 		// reduce traffic within the Instance per User; Expire every 5 hours
 		//error_log(__METHOD__.' ('.__LINE__.') '.' Called with Folder:'.$_folder.function_backtrace());
-		if (is_null($folderInfo)) $folderInfo = Cache::getCache(Cache::INSTANCE,'email','icServerFolderExistsInfo'.trim($GLOBALS['egw_info']['user']['account_id']),null,array(),$expiration=60*60*5);
+		if (is_null($folderInfo))
+		{
+			$folderInfo = Cache::getCache(Cache::INSTANCE,'email','icServerFolderExistsInfo'.trim($GLOBALS['egw_info']['user']['account_id']),null,array(),$expiration=60*60*5);
+		}
 		//error_log(__METHOD__.' ('.__LINE__.') '.'Cached Info on Folder:'.$_folder.' for Profile:'.$this->profileID.($forceCheck?'(forcedCheck)':'').':'.array2string($folderInfo));
 		if (!empty($folderInfo) && isset($folderInfo[$this->profileID]) && isset($folderInfo[$this->profileID][$_folder]) && $forceCheck===false)
 		{
@@ -3651,7 +4317,8 @@ 
 
 		// does the folder exist???
 		//error_log(__METHOD__."->Connected?".$this->icServer->_connected.", ".$_folder.", ".($forceCheck?' forceCheck activated':'dont check on server'));
-		if ( $forceCheck || empty($folderInfo) || !isset($folderInfo[$this->profileID]) || !isset($folderInfo[$this->profileID][$_folder])) {
+		if ( $forceCheck || empty($folderInfo) || !isset($folderInfo[$this->profileID]) || !isset($folderInfo[$this->profileID][$_folder]))
+		{
 			//error_log(__METHOD__."->NotConnected and forceCheck with profile:".$this->profileID);
 			//return false;
 			//try to connect
@@ -3691,9 +4358,12 @@ 
 
 		$this->icServer->openMailbox($folderName);
 
-		if(strtolower($folderName) == strtolower($trashFolder) && $deleteOptions == "move_to_trash") {
+		if(strtolower($folderName) == strtolower($trashFolder) && $deleteOptions == "move_to_trash")
+		{
 			$this->deleteMessages('all',$folderName,'remove_immediately');
-		} else {
+		}
+		else
+		{
 			$this->icServer->expunge($folderName);
 		}
 	}
@@ -3712,10 +4382,16 @@ 
 	{
 		//error_log(__METHOD__.' ('.__LINE__.') '.'->'.array2string($_messageUID).','.array2string($_folder).', '.$_forceDeleteMethod);
 		$oldMailbox = '';
-		if (is_null($_folder) || empty($_folder)) $_folder = $this->sessionData['mailbox'];
+		if (is_null($_folder) || empty($_folder))
+		{
+			$_folder = $this->sessionData['mailbox'];
+		}
 		if (empty($_messageUID))
 		{
-			if (self::$debug) error_log(__METHOD__." no messages Message(s): ".implode(',',$_messageUID));
+			if (self::$debug)
+			{
+				error_log(__METHOD__." no messages Message(s): ".implode(',',$_messageUID));
+			}
 			return false;
 		}
 		elseif ($_messageUID==='all')
@@ -3725,32 +4401,48 @@ 
 		else
 		{
 			$uidsToDelete = new Horde_Imap_Client_Ids();
-			if (!(is_object($_messageUID) || is_array($_messageUID))) $_messageUID = (array)$_messageUID;
+			if (!(is_object($_messageUID) || is_array($_messageUID)))
+			{
+				$_messageUID = (array)$_messageUID;
+			}
 			$uidsToDelete->add($_messageUID);
 		}
 		$deleteOptions = $_forceDeleteMethod; // use forceDeleteMethod if not "no", or unknown method
-		if ($_forceDeleteMethod === 'no' || !in_array($_forceDeleteMethod,array('move_to_trash',"mark_as_deleted","remove_immediately"))) $deleteOptions  = ($this->mailPreferences['deleteOptions']?$this->mailPreferences['deleteOptions']:"mark_as_deleted");
+		if ($_forceDeleteMethod === 'no' || !in_array($_forceDeleteMethod,array('move_to_trash',"mark_as_deleted","remove_immediately")))
+		{
+			$deleteOptions  = ($this->mailPreferences['deleteOptions']?$this->mailPreferences['deleteOptions']:"mark_as_deleted");
+		}
 		//error_log(__METHOD__.' ('.__LINE__.') '.'->'.array2string($_messageUID).','.$_folder.'/'.$this->sessionData['mailbox'].' Option:'.$deleteOptions);
 		$trashFolder    = $this->getTrashFolder();
 		$draftFolder	= $this->getDraftFolder(); //$GLOBALS['egw_info']['user']['preferences']['mail']['draftFolder'];
 		$templateFolder = $this->getTemplateFolder(); //$GLOBALS['egw_info']['user']['preferences']['mail']['templateFolder'];
 		if((strtolower($_folder) == strtolower($trashFolder) && $deleteOptions == "move_to_trash") ||
-		   (strtolower($_folder) == strtolower($draftFolder))) {
+		   (strtolower($_folder) == strtolower($draftFolder)))
+		{
 			$deleteOptions = "remove_immediately";
 		}
-		if($this->icServer->getCurrentMailbox() != $_folder) {
+		if($this->icServer->getCurrentMailbox() != $_folder)
+		{
 			$oldMailbox = $this->icServer->getCurrentMailbox();
 			$this->icServer->openMailbox($_folder);
 		}
 		//error_log(__METHOD__.' ('.__LINE__.') '.'->'.array2string($_messageUID).','.$_folder.'/'.$this->sessionData['mailbox'].' Option:'.$deleteOptions);
 		$updateCache = false;
-		switch($deleteOptions) {
+		switch($deleteOptions)
+		{
 			case "move_to_trash":
 				//error_log(__METHOD__.' ('.__LINE__.') ');
 				$updateCache = true;
-				if(!empty($trashFolder)) {
-					if (self::$debug) error_log(__METHOD__.' ('.__LINE__.') '.implode(' : ', $_messageUID));
-					if (self::$debug) error_log(__METHOD__.' ('.__LINE__.') '."$trashFolder <= $_folder / ". $this->sessionData['mailbox']);
+				if(!empty($trashFolder))
+				{
+					if (self::$debug)
+					{
+						error_log(__METHOD__.' ('.__LINE__.') '.implode(' : ', $_messageUID));
+					}
+					if (self::$debug)
+					{
+						error_log(__METHOD__.' ('.__LINE__.') '."$trashFolder <= $_folder / ". $this->sessionData['mailbox']);
+					}
 					// copy messages
 					try
 					{
@@ -3766,7 +4458,10 @@ 
 			case "mark_as_deleted":
 				//error_log(__METHOD__.' ('.__LINE__.') ');
 				// mark messages as deleted
-				if (is_null($_messageUID)) $_messageUID='all';
+				if (is_null($_messageUID))
+				{
+					$_messageUID='all';
+				}
 				foreach((array)$_messageUID as $key =>$uid)
 				{
 					//flag messages, that are flagged for deletion as seen too
@@ -3774,7 +4469,10 @@ 
 					$flags = $this->getFlags($uid);
 					$this->flagMessages('delete', $uid, $_folder);
 					//error_log(__METHOD__.' ('.__LINE__.') '.array2string($flags));
-					if (strpos( array2string($flags),'Deleted')!==false) $undelete[] = $uid;
+					if (strpos( array2string($flags),'Deleted')!==false)
+					{
+						$undelete[] = $uid;
+					}
 					unset($flags);
 				}
 				foreach((array)$undelete as $key =>$uid)
@@ -3786,7 +4484,10 @@ 
 			case "remove_immediately":
 				//error_log(__METHOD__.' ('.__LINE__.') ');
 				$updateCache = true;
-				if (is_null($_messageUID)) $_messageUID='all';
+				if (is_null($_messageUID))
+				{
+					$_messageUID='all';
+				}
 				if (is_object($_messageUID))
 				{
 					$this->flagMessages('delete', $_messageUID, $_folder);
@@ -3803,7 +4504,8 @@ 
 				$this->icServer->expunge($_folder);
 				break;
 		}
-		if($oldMailbox != '') {
+		if($oldMailbox != '')
+		{
 			$this->icServer->openMailbox($oldMailbox);
 		}
 
@@ -3817,11 +4519,15 @@ 
 	 *
 	 * @return null/array flags
 	 */
-	function getFlags ($_messageUID) {
+	function getFlags ($_messageUID)
+	{
 		try
 		{
 			$uidsToFetch = new Horde_Imap_Client_Ids();
-			if (!(is_object($_messageUID) || is_array($_messageUID))) $_messageUID = (array)$_messageUID;
+			if (!(is_object($_messageUID) || is_array($_messageUID)))
+			{
+				$_messageUID = (array)$_messageUID;
+			}
 			$uidsToFetch->add($_messageUID);
 			$_folderName = $this->icServer->getCurrentMailbox();
 			$fquery = new Horde_Imap_Client_Fetch_Query();
@@ -3829,8 +4535,10 @@ 
 			$headersNew = $this->icServer->fetch($_folderName, $fquery, array(
 				'ids' => $uidsToFetch,
 			));
-			if (is_object($headersNew)) {
-				foreach($headersNew->ids() as $id) {
+			if (is_object($headersNew))
+			{
+				foreach($headersNew->ids() as $id)
+				{
 					$_headerObject = $headersNew->get($id);
 					$flags = $_headerObject->getFlags();
 				}
@@ -3855,10 +4563,16 @@ 
 	 */
 	function getNotifyFlags ($_messageUID, $flags=null)
 	{
-		if (self::$debug) error_log(__METHOD__.$_messageUID.' Flags:'.array2string($flags));
+		if (self::$debug)
+		{
+			error_log(__METHOD__.$_messageUID.' Flags:'.array2string($flags));
+		}
 		try
 		{
-			if($flags===null) $flags =  $this->getFlags($_messageUID);
+			if($flags===null)
+			{
+				$flags =  $this->getFlags($_messageUID);
+			}
 		}
 		catch (\Exception $e)
 		{
@@ -3866,10 +4580,14 @@ 
 		}
 
 		if ( stripos( array2string($flags),'MDNSent')!==false)
-			return true;
+		{
+					return true;
+		}
 
 		if ( stripos( array2string($flags),'MDNnotSent')!==false)
-			return false;
+		{
+					return false;
+		}
 
 		return null;
 	}
@@ -3890,7 +4608,10 @@ 
 		//error_log(__METHOD__.' ('.__LINE__.') '.'->' .$_flag." ".array2string($_messageUID).",$_folder /".$this->sessionData['mailbox']);
 		if (empty($_messageUID))
 		{
-			if (self::$debug) error_log(__METHOD__." no messages Message(s): ".implode(',',$_messageUID));
+			if (self::$debug)
+			{
+				error_log(__METHOD__." no messages Message(s): ".implode(',',$_messageUID));
+			}
 			return false;
 		}
 		$this->icServer->openMailbox(($_folder?$_folder:$this->sessionData['mailbox']));
@@ -3898,7 +4619,10 @@ 
 		if (is_array($_messageUID)&& count($_messageUID)>50)
 		{
 			$count = $this->getMailBoxCounters($folder,true);
-			if ($count->messages == count($_messageUID)) $_messageUID='all';
+			if ($count->messages == count($_messageUID))
+			{
+				$_messageUID='all';
+			}
 		}
 
 		if ($_messageUID==='all')
@@ -3907,7 +4631,10 @@ 
 		}
 		else
 		{
-			if (!(is_object($_messageUID) || is_array($_messageUID))) $_messageUID = (array)$_messageUID;
+			if (!(is_object($_messageUID) || is_array($_messageUID)))
+			{
+				$_messageUID = (array)$_messageUID;
+			}
 			$messageUIDs = array_chunk($_messageUID,50,true);
 		}
 		try
@@ -3923,7 +4650,8 @@ 
 					$uidsToModify = new Horde_Imap_Client_Ids();
 					$uidsToModify->add($uids);
 				}
-				switch($_flag) {
+				switch($_flag)
+				{
 					case "delete":
 						$ret = $this->icServer->store($folder, array('add'=>array('\\Deleted'), 'ids'=> $uidsToModify));
 						break;
@@ -4009,7 +4737,10 @@ 
 		{
 			error_log(__METHOD__.__LINE__.' Error, could not flag messages in folder '.$folder.' Reason:'.$e->getMessage());
 		}
-		if ($folder instanceof Horde_Imap_Client_Mailbox) $_folder = $folder->utf8;
+		if ($folder instanceof Horde_Imap_Client_Mailbox)
+		{
+			$_folder = $folder->utf8;
+		}
 		//error_log(__METHOD__.__LINE__.'#'.$this->icServer->ImapServerId.'#'.array2string($_folder).'#');
 		self::$folderStatusCache[$this->icServer->ImapServerId][(!empty($_folder)?$_folder: $this->sessionData['mailbox'])]['uidValidity'] = 0;
 
@@ -4038,7 +4769,10 @@ 
 		//$deleteOptions  = $GLOBALS['egw_info']["user"]["preferences"]["mail"]["deleteOptions"];
 		if (empty($_messageUID))
 		{
-			if (self::$debug) error_log(__METHOD__." no Message(s): ".implode(',',$_messageUID));
+			if (self::$debug)
+			{
+				error_log(__METHOD__." no Message(s): ".implode(',',$_messageUID));
+			}
 			return false;
 		}
 		elseif ($_messageUID==='all')
@@ -4050,7 +4784,10 @@ 
 		{
 			//error_log(__METHOD__." Message(s): ".implode(',',$_messageUID));
 			$uidsToMove = new Horde_Imap_Client_Ids();
-			if (!(is_object($_messageUID) || is_array($_messageUID))) $_messageUID = (array)$_messageUID;
+			if (!(is_object($_messageUID) || is_array($_messageUID)))
+			{
+				$_messageUID = (array)$_messageUID;
+			}
 			$uidsToMove->add($_messageUID);
 		}
 		$sourceFolder = (!empty($currentFolder)?$currentFolder: $this->sessionData['mailbox']);
@@ -4072,12 +4809,14 @@ 
 
 			//error_log(__METHOD__.' ('.__LINE__.') '.' Sourceserver:'.$source->ImapServerId.' mailheaders:'.array2string($headersNew));
 
-			if (is_object($headersNew)) {
+			if (is_object($headersNew))
+			{
 				$c=0;
 				$retUid = new Horde_Imap_Client_Ids();
 				// we copy chunks of 5 to avoid too much memory and/or server stress
 				// some servers seem not to allow/support the appendig of multiple messages. so we are down to one
-				foreach($headersNew as &$_headerObject) {
+				foreach($headersNew as &$_headerObject)
+				{
 					$c++;
 					$flags = $_headerObject->getFlags(); //unseen status seems to be lost when retrieving the full message
 					$date = $_headerObject->getImapDate();
@@ -4161,7 +4900,11 @@ 
 	{
 		try {
 			$date = new DateTime($_date);	// parse date & time including timezone (throws exception, if not parsable)
-			if ($convert2usertime) $date->setUser();	// convert to user-time
+			if ($convert2usertime)
+			{
+				$date->setUser();
+			}
+			// convert to user-time
 			$date2return = $date->format($format);
 		}
 		catch(\Exception $e)
@@ -4193,9 +4936,15 @@ 
 	static function htmlentities($_string, $_charset=false)
 	{
 		//setting the charset (if not given)
-		if ($_charset===false) $_charset = self::$displayCharset;
+		if ($_charset===false)
+		{
+			$_charset = self::$displayCharset;
+		}
 		$string = @htmlentities($_string, ENT_QUOTES, $_charset, false);
-		if (empty($string) && !empty($_string)) $string = @htmlentities(Translation::convert($_string,Translation::detect_encoding($_string),$_charset),ENT_QUOTES | ENT_IGNORE,$_charset, false);
+		if (empty($string) && !empty($_string))
+		{
+			$string = @htmlentities(Translation::convert($_string,Translation::detect_encoding($_string),$_charset),ENT_QUOTES | ENT_IGNORE,$_charset, false);
+		}
 		return $string;
 	}
 
@@ -4215,18 +4964,41 @@ 
 		$_html = str_replace(array('&amp;amp;','<DIV><BR></DIV>',"<DIV>&nbsp;</DIV>",'<div>&nbsp;</div>','</td></font>','<br><td>','<tr></tr>','<o:p></o:p>','<o:p>','</o:p>'),
 							 array('&amp;',    '<BR>',           '<BR>',             '<BR>',             '</font></td>','<td>',    '',         '',           '',  ''),$_html);
 		//$_html = str_replace(array('&amp;amp;'),array('&amp;'),$_html);
-		if (stripos($_html,'style')!==false) Mail\Html::replaceTagsCompletley($_html,'style'); // clean out empty or pagewide style definitions / left over tags
-		if (stripos($_html,'head')!==false) Mail\Html::replaceTagsCompletley($_html,'head'); // Strip out stuff in head
+		if (stripos($_html,'style')!==false)
+		{
+			Mail\Html::replaceTagsCompletley($_html,'style');
+		}
+		// clean out empty or pagewide style definitions / left over tags
+		if (stripos($_html,'head')!==false)
+		{
+			Mail\Html::replaceTagsCompletley($_html,'head');
+		}
+		// Strip out stuff in head
 		//if (stripos($_html,'![if')!==false && stripos($_html,'<![endif]>')!==false) Mail\Html::replaceTagsCompletley($_html,'!\[if','<!\[endif\]>',false); // Strip out stuff in ifs
 		//if (stripos($_html,'!--[if')!==false && stripos($_html,'<![endif]-->')!==false) Mail\Html::replaceTagsCompletley($_html,'!--\[if','<!\[endif\]-->',false); // Strip out stuff in ifs
 		//error_log(__METHOD__.' ('.__LINE__.') '.$_html);
 
-		if (get_magic_quotes_gpc() === 1) $_html = stripslashes($_html);
+		if (get_magic_quotes_gpc() === 1)
+		{
+			$_html = stripslashes($_html);
+		}
 		// Strip out doctype in head, as htmlLawed cannot handle it TODO: Consider extracting it and adding it afterwards
-		if (stripos($_html,'!doctype')!==false) Mail\Html::replaceTagsCompletley($_html,'!doctype');
-		if (stripos($_html,'?xml:namespace')!==false) Mail\Html::replaceTagsCompletley($_html,'\?xml:namespace','/>',false);
-		if (stripos($_html,'?xml version')!==false) Mail\Html::replaceTagsCompletley($_html,'\?xml version','\?>',false);
-		if (strpos($_html,'!CURSOR')!==false) Mail\Html::replaceTagsCompletley($_html,'!CURSOR');
+		if (stripos($_html,'!doctype')!==false)
+		{
+			Mail\Html::replaceTagsCompletley($_html,'!doctype');
+		}
+		if (stripos($_html,'?xml:namespace')!==false)
+		{
+			Mail\Html::replaceTagsCompletley($_html,'\?xml:namespace','/>',false);
+		}
+		if (stripos($_html,'?xml version')!==false)
+		{
+			Mail\Html::replaceTagsCompletley($_html,'\?xml version','\?>',false);
+		}
+		if (strpos($_html,'!CURSOR')!==false)
+		{
+			Mail\Html::replaceTagsCompletley($_html,'!CURSOR');
+		}
 		// htmLawed filter only the 'body'
 		//preg_match('`(<htm.+?<body[^>]*>)(.+?)(</body>.*?</html>)`ims', $_html, $matches);
 		//if ($matches[2])
@@ -4265,8 +5037,10 @@ 
 		//$charSet = 'iso-8859-1';//self::$displayCharset; //'iso-8859-1'; // self::displayCharset seems to be asmarter fallback than iso-8859-1
 		$CharsetFound=false;
 		//echo "#".$_mimePartObject->encoding.'#<br>';
-		if(is_array($_mimePartObject->parameters)) {
-			if(isset($_mimePartObject->parameters['CHARSET'])) {
+		if(is_array($_mimePartObject->parameters))
+		{
+			if(isset($_mimePartObject->parameters['CHARSET']))
+			{
 				$charSet = $_mimePartObject->parameters['CHARSET'];
 				$CharsetFound=true;
 			}
@@ -4287,7 +5061,10 @@ 
 	function decodeMimePart($_mimeMessage, $_encoding, $_charset = '')
 	{
 		// decode the part
-		if (self::$debug) error_log(__METHOD__."() with $_encoding and $_charset:".print_r($_mimeMessage,true));
+		if (self::$debug)
+		{
+			error_log(__METHOD__."() with $_encoding and $_charset:".print_r($_mimeMessage,true));
+		}
 		switch (strtoupper($_encoding))
 		{
 			case 'BASE64':
@@ -4327,13 +5104,19 @@ 
 		// sometimes there are 3 parts, when there is an ics/ical attached/included-> we want to show that
 		// as attachment AND as abstracted ical information (we use our notification style here).
 		$partText = $partCalendar = $partHTML = null;
-		if (self::$debug) _debug_array(array("METHOD"=>__METHOD__,"LINE"=>__LINE__,"STRUCTURE"=>$_structure));
+		if (self::$debug)
+		{
+			_debug_array(array("METHOD"=>__METHOD__,"LINE"=>__LINE__,"STRUCTURE"=>$_structure));
+		}
 		//error_log(__METHOD__.' ('.__LINE__.') ');
 		$ignore_first_part = true;
 		foreach($_structure->contentTypeMap() as $mime_id => $mime_type)
 		{
 			//error_log(__METHOD__."($_uid, ".$_structure->getMimeId().") $mime_id: $mime_type"." ignoreFirstPart:".$ignore_first_part);
-			if (self::$debug) echo __METHOD__."($_uid, partID=".$_structure->getMimeId().") $mime_id: $mime_type<br>";
+			if (self::$debug)
+			{
+				echo __METHOD__."($_uid, partID=".$_structure->getMimeId().") $mime_id: $mime_type<br>";
+			}
 
 			if ($ignore_first_part)
 			{
@@ -4349,11 +5132,17 @@ 
 					switch($mimePart->getSubType())
 					{
 						case 'plain':
-							if ($mimePart->getBytes() > 0) $partText = $mimePart;
+							if ($mimePart->getBytes() > 0)
+							{
+								$partText = $mimePart;
+							}
 							break;
 
 						case 'html':
-							if ($mimePart->getBytes() > 0)  $partHTML = $mimePart;
+							if ($mimePart->getBytes() > 0)
+							{
+								$partHTML = $mimePart;
+							}
 							break;
 					}
 					break;
@@ -4366,7 +5155,10 @@ 
 							if (count($mimePart->getParts()) > 1)
 							{
 								// in a multipart alternative we treat the multipart/related as html part
-								if (self::$debug) error_log(__METHOD__." process MULTIPART/".$mimePart->getSubType()." with array as subparts");
+								if (self::$debug)
+								{
+									error_log(__METHOD__." process MULTIPART/".$mimePart->getSubType()." with array as subparts");
+								}
 								$partHTML = $mimePart;
 								break 3; // GET OUT OF LOOP, will be processed according to type
 							}
@@ -4447,9 +5239,15 @@ 
 	 */
 	function getMultipartMixed($_uid, Horde_Mime_Part $_structure, $_htmlMode, $_preserveSeen = false, &$skipParts=array())
 	{
-		if (self::$debug) echo __METHOD__."$_uid, $_htmlMode<br>";
+		if (self::$debug)
+		{
+			echo __METHOD__."$_uid, $_htmlMode<br>";
+		}
 		$bodyPart = array();
-		if (self::$debug) _debug_array($_structure);
+		if (self::$debug)
+		{
+			_debug_array($_structure);
+		}
 
 		$ignore_first_part = true;
 		//$skipParts = array();
@@ -4457,7 +5255,10 @@ 
 		foreach($_structure->contentTypeMap() as $mime_id => $mime_type)
 		{
 			//error_log(__METHOD__."($_uid, ".$_structure->getMimeId().") $mime_id: $mime_type");
-			if (self::$debug) echo __METHOD__."($_uid, partID=".$_structure->getMimeId().") $mime_id: $mime_type<br>";
+			if (self::$debug)
+			{
+				echo __METHOD__."($_uid, partID=".$_structure->getMimeId().") $mime_id: $mime_type<br>";
+			}
 			if ($ignore_first_part)
 			{
 				$ignore_first_part = false;
@@ -4475,7 +5276,10 @@ 
 			switch($part->getPrimaryType())
 			{
 				case 'multipart':
-					if ($part->getDisposition() == 'attachment') continue;
+					if ($part->getDisposition() == 'attachment')
+					{
+						continue;
+					}
 					switch($part->getSubType())
 					{
 						case 'alternative':
@@ -4540,7 +5344,9 @@ 
 					if($part->getSubType() == 'rfc822' || $part->getDisposition() == 'attachment')
 					{
 						$skipParts[$mime_id.'.0'] = $mime_type;
-						foreach($part->contentTypeMap() as $sub_id => $sub_type){ $skipParts[$sub_id] = $sub_type;}
+						foreach($part->contentTypeMap() as $sub_id => $sub_type)
+						{
+$skipParts[$sub_id] = $sub_type;}
 						//error_log(__METHOD__.' ('.__LINE__.') '.' Uid:'.$_uid.' Part:'.$mime_id.':'.array2string($skipParts));
 						//break 2;
 					}
@@ -4583,7 +5389,10 @@ 
 	 */
 	function getBodyPart($_uid, $_partID=null, $_folder=null, $_preserveSeen=false, $_stream=false, &$_encoding=null, $_tryDecodingServerside=true)
 	{
-		if (self::$debug) error_log( __METHOD__.__LINE__."(".array2string($_uid).", $_partID, $_folder, $_preserveSeen, $_stream, $_encoding, $_tryDecodingServerside)");
+		if (self::$debug)
+		{
+			error_log( __METHOD__.__LINE__."(".array2string($_uid).", $_partID, $_folder, $_preserveSeen, $_stream, $_encoding, $_tryDecodingServerside)");
+		}
 
 		if (empty($_folder))
 		{
@@ -4592,7 +5401,10 @@ 
 		//error_log(__METHOD__.' ('.__LINE__.') '.array2string($_folder).'/'.$this->icServer->getCurrentMailbox().'/'. $this->sessionData['mailbox']);
 		// querying contents of body part
 		$uidsToFetch = new Horde_Imap_Client_Ids();
-		if (!(is_object($_uid) || is_array($_uid))) $_uid = (array)$_uid;
+		if (!(is_object($_uid) || is_array($_uid)))
+		{
+			$_uid = (array)$_uid;
+		}
 		$uidsToFetch->add($_uid);
 
 		$fquery = new Horde_Imap_Client_Fetch_Query();
@@ -4600,9 +5412,12 @@ 
 			'peek' => $_preserveSeen,
 			'decode' => true,	// try decode on server, does NOT neccessary work
 		);
-		if ($_tryDecodingServerside===false)// || ($_tryDecodingServerside&&$this->isDraftFolder($_folder)))
+		if ($_tryDecodingServerside===false)
+		{
+			// || ($_tryDecodingServerside&&$this->isDraftFolder($_folder)))
 		{
 			$_tryDecodingServerside=false;
+		}
 			$fetchParams = array(
 				'peek' => $_preserveSeen,
 			);
@@ -4644,7 +5459,10 @@ 
 	{
 		//error_log(__METHOD__.' ('.__LINE__.') '.'->'.$_uid.':'.array2string($_structure).' '.function_backtrace());
 		$bodyPart = array();
-		if (self::$debug) _debug_array(array($_structure,function_backtrace()));
+		if (self::$debug)
+		{
+			_debug_array(array($_structure,function_backtrace()));
+		}
 
 		if($_structure->getSubType() == 'html' && !in_array($_htmlMode, array('html_only', 'always_display', 'only_if_no_text')))
 		{
@@ -4694,8 +5512,12 @@ 
 	 */
 	function getMessageBody($_uid, $_htmlOptions='', $_partID=null, Horde_Mime_Part $_structure=null, $_preserveSeen = false, $_folder = '')
 	{
-		if (self::$debug) echo __METHOD__."$_uid, $_htmlOptions, $_partID<br>";
-		if($_htmlOptions != '') {
+		if (self::$debug)
+		{
+			echo __METHOD__."$_uid, $_htmlOptions, $_partID<br>";
+		}
+		if($_htmlOptions != '')
+		{
 			$this->htmlOptions = $_htmlOptions;
 		}
 		if (empty($_folder))
@@ -4784,7 +5606,9 @@ 
 						default:
 							$bodyPart = array($this->getTextPart($_uid, $_structure, $this->htmlOptions, $_preserveSeen));
 					}
-				} else {
+				}
+				else
+				{
 					// what if the structure->disposition is attachment ,...
 				}
 				return self::normalizeBodyParts($bodyPart);
@@ -4795,13 +5619,18 @@ 
 				{
 					case 'rfc822':
 						$newStructure = $_structure->getParts();
-						if (self::$debug) {echo __METHOD__." Message -> RFC -> NewStructure:"; _debug_array($newStructure[0]);}
+						if (self::$debug)
+						{
+echo __METHOD__." Message -> RFC -> NewStructure:"; _debug_array($newStructure[0]);}
 						return self::normalizeBodyParts($this->getMessageBody($_uid, $_htmlOptions, $newStructure[0]->getMimeId(), $newStructure[0], $_preserveSeen, $_folder));
 				}
 				break;
 
 			default:
-				if (self::$debug) _debug_array($_structure);
+				if (self::$debug)
+				{
+					_debug_array($_structure);
+				}
 				return array(
 					array(
 						'body'		=> lang('The mimeparser can not parse this message.').$_structure->getType(),
@@ -4825,9 +5654,12 @@ 
 		{
 			foreach($_bodyParts as $singleBodyPart)
 			{
-				if (!isset($singleBodyPart['body'])) {
+				if (!isset($singleBodyPart['body']))
+				{
 					$buff = self::normalizeBodyParts($singleBodyPart);
-					foreach ((array)$buff as $val) { $body2return[] = $val;}
+					foreach ((array)$buff as $val)
+					{
+$body2return[] = $val;}
 					continue;
 				}
 				$body2return[] = $singleBodyPart;
@@ -4853,13 +5685,20 @@ 
 		$message='';
 		for($i=0; $i<count($bodyParts); $i++)
 		{
-			if (!isset($bodyParts[$i]['body'])) {
+			if (!isset($bodyParts[$i]['body']))
+			{
 				$bodyParts[$i]['body'] = self::getdisplayableBody($mailClass, $bodyParts[$i], $preserveHTML, $useTidy);
 				$message .= empty($bodyParts[$i]['body'])?'':$bodyParts[$i]['body'];
 				continue;
 			}
-			if (isset($bodyParts[$i]['error'])) continue;
-			if (empty($bodyParts[$i]['body'])) continue;
+			if (isset($bodyParts[$i]['error']))
+			{
+				continue;
+			}
+			if (empty($bodyParts[$i]['body']))
+			{
+				continue;
+			}
 			// some characterreplacements, as they fail to translate
 			$sar = array(
 				'@(\x84|\x93|\x94)@',
@@ -4886,13 +5725,17 @@ 
 				$bodyParts[$i]['body'] = preg_replace($sar,$rar,$bodyParts[$i]['body']);
 			}
 
-			if ($bodyParts[$i]['charSet']===false) $bodyParts[$i]['charSet'] = Translation::detect_encoding($bodyParts[$i]['body']);
+			if ($bodyParts[$i]['charSet']===false)
+			{
+				$bodyParts[$i]['charSet'] = Translation::detect_encoding($bodyParts[$i]['body']);
+			}
 			// add line breaks to $bodyParts
 			//error_log(__METHOD__.' ('.__LINE__.') '.' Charset:'.$bodyParts[$i]['charSet'].'->'.$bodyParts[$i]['body']);
 			$newBody  = Translation::convert_jsonsafe($bodyParts[$i]['body'], $bodyParts[$i]['charSet']);
 			//error_log(__METHOD__.' ('.__LINE__.') '.' MimeType:'.$bodyParts[$i]['mimeType'].'->'.$newBody);
 			$mailClass->activeMimeType = 'text/plain';
-			if ($bodyParts[$i]['mimeType'] == 'text/html') {
+			if ($bodyParts[$i]['mimeType'] == 'text/html')
+			{
 				$mailClass->activeMimeType = $bodyParts[$i]['mimeType'];
 				if (!$preserveHTML)
 				{
@@ -4938,15 +5781,28 @@ 
 						// as we switched off HTMLaweds tidy functionality
 						$newBody = str_replace(array('&amp;amp;','<DIV><BR></DIV>',"<DIV>&nbsp;</DIV>",'<div>&nbsp;</div>'),array('&amp;','<BR>','<BR>','<BR>'),$newBody);
 						$newBody = $htmLawed->run($newBody,self::$htmLawed_config);
-						if ($hasOther && $preserveHTML) $newBody = $matches[1]. $newBody. $matches[3];
+						if ($hasOther && $preserveHTML)
+						{
+							$newBody = $matches[1]. $newBody. $matches[3];
+						}
 						$alreadyHtmlLawed=true;
 					}
 					//error_log(__METHOD__.' ('.__LINE__.') '.' after purify:'.$newBody);
-					if ($preserveHTML==false) $newBody = Mail\Html::convertHTMLToText($newBody,self::$displayCharset,true,true);
+					if ($preserveHTML==false)
+					{
+						$newBody = Mail\Html::convertHTMLToText($newBody,self::$displayCharset,true,true);
+					}
 					//error_log(__METHOD__.' ('.__LINE__.') '.' after convertHTMLToText:'.$newBody);
-					if ($preserveHTML==false) $newBody = nl2br($newBody); // we need this, as htmLawed removes \r\n
+					if ($preserveHTML==false)
+					{
+						$newBody = nl2br($newBody);
+					}
+					// we need this, as htmLawed removes \r\n
 					/*if (!$alreadyHtmlLawed) */ $mailClass->getCleanHTML($newBody); // remove stuff we regard as unwanted
-					if ($preserveHTML==false) $newBody = str_replace("<br />","\r\n",$newBody);
+					if ($preserveHTML==false)
+					{
+						$newBody = str_replace("<br />","\r\n",$newBody);
+					}
 					//error_log(__METHOD__.' ('.__LINE__.') '.' after getClean:'.$newBody);
 				}
 				$message .= $newBody;
@@ -4983,12 +5839,12 @@ 
 				  substr($line,0,strlen($dontbreaklinesstartingwith)) != $dontbreaklinesstartingwith
 				 )
 				)
-			   )
-			{
+			   ) {
 				$s=explode(" ", $line);
 				$line = "";
 				$linecnt = 0;
-				foreach ($s as &$v) {
+				foreach ($s as &$v)
+				{
 					$cnt = strlen($v);
 					// only break long words within the wordboundaries,
 					// but it may destroy links, so we check for href and dont do it if we find one
@@ -4998,14 +5854,20 @@ 
 						$v=wordwrap($v, $allowedLength, $cut, true);
 					}
 					// the rest should be broken at the start of the new word that exceeds the limit
-					if ($linecnt+$cnt > $allowedLength) {
+					if ($linecnt+$cnt > $allowedLength)
+					{
 						$v=$cut.$v;
 						#$linecnt = 0;
 						$linecnt =strlen($v)-strlen($cut);
-					} else {
+					}
+					else
+					{
 						$linecnt += $cnt;
 					}
-					if (strlen($v)) $line .= (strlen($line) ? " " : "").$v;
+					if (strlen($v))
+					{
+						$line .= (strlen($line) ? " " : "").$v;
+					}
 				}
 			}
 			$newStr .= $line . "\n";
@@ -5026,11 +5888,18 @@ 
 	function getMessageEnvelope($_uid, $_partID = '',$decode=false, $_folder='', $_useHeaderInsteadOfEnvelope=false)
 	{
 		//error_log(__METHOD__.' ('.__LINE__.') '.":$_uid,$_partID,$decode,$_folder".function_backtrace());
-		if (empty($_folder)) $_folder = ($this->sessionData['mailbox']? $this->sessionData['mailbox'] : $this->icServer->getCurrentMailbox());
+		if (empty($_folder))
+		{
+			$_folder = ($this->sessionData['mailbox']? $this->sessionData['mailbox'] : $this->icServer->getCurrentMailbox());
+		}
 		//error_log(__METHOD__.' ('.__LINE__.') '.":$_uid,$_partID,$decode,$_folder");
-		if((empty($_partID)||$_partID=='null')&&$_useHeaderInsteadOfEnvelope===false) {
+		if((empty($_partID)||$_partID=='null')&&$_useHeaderInsteadOfEnvelope===false)
+		{
 			$uidsToFetch = new Horde_Imap_Client_Ids();
-			if (!(is_object($_uid) || is_array($_uid))) $_uid = (array)$_uid;
+			if (!(is_object($_uid) || is_array($_uid)))
+			{
+				$_uid = (array)$_uid;
+			}
 			$uidsToFetch->add($_uid);
 
 			$fquery = new Horde_Imap_Client_Fetch_Query();
@@ -5040,8 +5909,10 @@ 
 			$headersNew = $this->icServer->fetch($_folder, $fquery, array(
 				'ids' => $uidsToFetch,
 			));
-			if (is_object($headersNew)) {
-				foreach($headersNew as &$_headerObject) {
+			if (is_object($headersNew))
+			{
+				foreach($headersNew as &$_headerObject)
+				{
 					$env = $_headerObject->getEnvelope();
 					//_debug_array($envFields->singleFields());
 					$singleFields = $envFields->singleFields();
@@ -5058,7 +5929,10 @@ 
 								//error_log(__METHOD__.' ('.__LINE__.') '.$v.'->'.array2string($env->$v->addresses));
 								$envelope[$v]=$env->$v->addresses;
 								$address = array();
-								if (!is_array($envelope[$v])) break;
+								if (!is_array($envelope[$v]))
+								{
+									break;
+								}
 								foreach ($envelope[$v] as $k => $ad)
 								{
 									if (stripos($ad,'@')===false)
@@ -5095,7 +5969,9 @@ 
 				}
 			}
 			return $envelope;
-		} else {
+		}
+		else
+		{
 
 			$headers = $this->getMessageHeader($_uid, $_partID, true,true,$_folder);
 
@@ -5106,19 +5982,43 @@ 
 				'SUBJECT'	=> ($decode ? self::decode_header($headers['SUBJECT']):$headers['SUBJECT']),
 				'MESSAGE_ID'	=> $headers['MESSAGE-ID']
 			);
-			if (isset($headers['IN-REPLY-TO'])) $newData['IN-REPLY-TO'] = $headers['IN-REPLY-TO'];
-			if (isset($headers['REFERENCES'])) $newData['REFERENCES'] = $headers['REFERENCES'];
-			if (isset($headers['THREAD-TOPIC'])) $newData['THREAD-TOPIC'] = $headers['THREAD-TOPIC'];
-			if (isset($headers['THREAD-INDEX'])) $newData['THREAD-INDEX'] = $headers['THREAD-INDEX'];
-			if (isset($headers['LIST-ID'])) $newData['LIST-ID'] = $headers['LIST-ID'];
-			if (isset($headers['SIZE'])) $newData['SIZE'] = $headers['SIZE'];
+			if (isset($headers['IN-REPLY-TO']))
+			{
+				$newData['IN-REPLY-TO'] = $headers['IN-REPLY-TO'];
+			}
+			if (isset($headers['REFERENCES']))
+			{
+				$newData['REFERENCES'] = $headers['REFERENCES'];
+			}
+			if (isset($headers['THREAD-TOPIC']))
+			{
+				$newData['THREAD-TOPIC'] = $headers['THREAD-TOPIC'];
+			}
+			if (isset($headers['THREAD-INDEX']))
+			{
+				$newData['THREAD-INDEX'] = $headers['THREAD-INDEX'];
+			}
+			if (isset($headers['LIST-ID']))
+			{
+				$newData['LIST-ID'] = $headers['LIST-ID'];
+			}
+			if (isset($headers['SIZE']))
+			{
+				$newData['SIZE'] = $headers['SIZE'];
+			}
 			//_debug_array($newData);
 			$recepientList = array('FROM', 'TO', 'CC', 'BCC', 'SENDER', 'REPLY-TO');
-			foreach($recepientList as $recepientType) {
-				if(isset($headers[$recepientType])) {
-					if ($decode) $headers[$recepientType] =  self::decode_header($headers[$recepientType],true);
+			foreach($recepientList as $recepientType)
+			{
+				if(isset($headers[$recepientType]))
+				{
+					if ($decode)
+					{
+						$headers[$recepientType] =  self::decode_header($headers[$recepientType],true);
+					}
 					//error_log(__METHOD__.__LINE__." ".$recepientType."->".array2string($headers[$recepientType]));
-					foreach(self::parseAddressList($headers[$recepientType]) as $singleAddress) {
+					foreach(self::parseAddressList($headers[$recepientType]) as $singleAddress)
+					{
 						$addressData = array(
 							'PERSONAL_NAME'		=> $singleAddress->personal ? $singleAddress->personal : 'NIL',
 							'AT_DOMAIN_LIST'	=> $singleAddress->adl ? $singleAddress->adl : 'NIL',
@@ -5126,17 +6026,25 @@ 
 							'HOST_NAME'		=> $singleAddress->host ? $singleAddress->host : 'NIL',
 							'EMAIL'			=> $singleAddress->host ? $singleAddress->mailbox.'@'.$singleAddress->host : $singleAddress->mailbox,
 						);
-						if($addressData['PERSONAL_NAME'] != 'NIL') {
+						if($addressData['PERSONAL_NAME'] != 'NIL')
+						{
 							$addressData['RFC822_EMAIL'] = imap_rfc822_write_address($singleAddress->mailbox, $singleAddress->host, $singleAddress->personal);
-						} else {
+						}
+						else
+						{
 							$addressData['RFC822_EMAIL'] = 'NIL';
 						}
 						$newData[$recepientType][] = ($addressData['RFC822_EMAIL']!='NIL'?$addressData['RFC822_EMAIL']:$addressData['EMAIL']);//$addressData;
 					}
-				} else {
-					if($recepientType == 'SENDER' || $recepientType == 'REPLY-TO') {
+				}
+				else
+				{
+					if($recepientType == 'SENDER' || $recepientType == 'REPLY-TO')
+					{
 						$newData[$recepientType] = $newData['FROM'];
-					} else {
+					}
+					else
+					{
 						$newData[$recepientType] = array();
 					}
 				}
@@ -5159,9 +6067,15 @@ 
 	function getMessageHeader($_uid, $_partID = '',$decode=false, $preserveUnSeen=false, $_folder='')
 	{
 		//error_log(__METHOD__.' ('.__LINE__.') '.':'.$_uid.', '.$_partID.', '.$decode.', '.$preserveUnSeen.', '.$_folder);
-		if (empty($_folder)) $_folder = ($this->sessionData['mailbox']? $this->sessionData['mailbox'] : $this->icServer->getCurrentMailbox());
+		if (empty($_folder))
+		{
+			$_folder = ($this->sessionData['mailbox']? $this->sessionData['mailbox'] : $this->icServer->getCurrentMailbox());
+		}
 		$uidsToFetch = new Horde_Imap_Client_Ids();
-		if (!(is_object($_uid) || is_array($_uid))) $_uid = (array)$_uid;
+		if (!(is_object($_uid) || is_array($_uid)))
+		{
+			$_uid = (array)$_uid;
+		}
 		$uidsToFetch->add($_uid);
 
 		$fquery = new Horde_Imap_Client_Fetch_Query();
@@ -5179,7 +6093,8 @@ 
 		$headersNew = $this->icServer->fetch($_folder, $fquery, array(
 			'ids' => $uidsToFetch,
 		));
-		if (is_object($headersNew)) {
+		if (is_object($headersNew))
+		{
 			foreach($headersNew as $_fetchObject)
 			{
 				$headers = $_fetchObject->getHeaderText(0,Horde_Imap_Client_Data_Fetch::HEADER_PARSE);
@@ -5201,11 +6116,17 @@ 
 			}
 			if ($decode === 'object')
 			{
-				if (is_object($headers)) $headers->setUserAgent('EGroupware API '.$GLOBALS['egw_info']['server']['versions']['phpgwapi']);
+				if (is_object($headers))
+				{
+					$headers->setUserAgent('EGroupware API '.$GLOBALS['egw_info']['server']['versions']['phpgwapi']);
+				}
 				return $headers;
 			}
 			$retValue = is_object($headers) ? $headers->toArray():array();
-			if ($size) $retValue['size'] = $size;
+			if ($size)
+			{
+				$retValue['size'] = $size;
+			}
 		}
 		$retValue = array_change_key_case($retValue,CASE_UPPER);
 		//error_log(__METHOD__.' ('.__LINE__.') '.array2string($retValue));
@@ -5237,10 +6158,16 @@ 
 	function getMessageRawHeader($_uid, $_partID = '', $_folder = '')
 	{
 		static $rawHeaders;
-		if (empty($_folder)) $_folder = ($this->sessionData['mailbox']? $this->sessionData['mailbox'] : $this->icServer->getCurrentMailbox());
+		if (empty($_folder))
+		{
+			$_folder = ($this->sessionData['mailbox']? $this->sessionData['mailbox'] : $this->icServer->getCurrentMailbox());
+		}
 		//error_log(__METHOD__.' ('.__LINE__.') '." Try Using Cache for raw Header $_uid, $_partID in Folder $_folder");
 
-		if (is_null($rawHeaders)||!is_array($rawHeaders)) $rawHeaders = Cache::getCache(Cache::INSTANCE,'email','rawHeadersCache'.trim($GLOBALS['egw_info']['user']['account_id']),null,array(),60*60*1);
+		if (is_null($rawHeaders)||!is_array($rawHeaders))
+		{
+			$rawHeaders = Cache::getCache(Cache::INSTANCE,'email','rawHeadersCache'.trim($GLOBALS['egw_info']['user']['account_id']),null,array(),60*60*1);
+		}
 		if (isset($rawHeaders[$this->icServer->ImapServerId][(string)$_folder][$_uid][(empty($_partID)?'NIL':$_partID)]))
 		{
 			//error_log(__METHOD__.' ('.__LINE__.') '." Using Cache for raw Header $_uid, $_partID in Folder $_folder");
@@ -5248,7 +6175,10 @@ 
 		}
 		$uidsToFetch = new Horde_Imap_Client_Ids();
 		$uid = $_uid;
-		if (!(is_object($_uid) || is_array($_uid))) $uid = (array)$_uid;
+		if (!(is_object($_uid) || is_array($_uid)))
+		{
+			$uid = (array)$_uid;
+		}
 		$uidsToFetch->add($uid);
 
 		$fquery = new Horde_Imap_Client_Fetch_Query();
@@ -5264,8 +6194,10 @@ 
 		$headersNew = $this->icServer->fetch($_folder, $fquery, array(
 			'ids' => $uidsToFetch,
 		));
-		if (is_object($headersNew)) {
-			foreach($headersNew as &$_headerObject) {
+		if (is_object($headersNew))
+		{
+			foreach($headersNew as &$_headerObject)
+			{
 				$retValue = $_headerObject->getHeaderText();
 				if ($_partID != '')
 				{
@@ -5293,22 +6225,33 @@ 
 	static function &getStyles($_bodyParts)
 	{
 		$style = '';
-		if (empty($_bodyParts)) return "";
-		foreach((array)$_bodyParts as $singleBodyPart) {
-			if (!isset($singleBodyPart['body'])) {
+		if (empty($_bodyParts))
+		{
+			return "";
+		}
+		foreach((array)$_bodyParts as $singleBodyPart)
+		{
+			if (!isset($singleBodyPart['body']))
+			{
 				$singleBodyPart['body'] = self::getStyles($singleBodyPart);
 				$style .= $singleBodyPart['body'];
 				continue;
 			}
 
-			if ($singleBodyPart['charSet']===false) $singleBodyPart['charSet'] = Translation::detect_encoding($singleBodyPart['body']);
+			if ($singleBodyPart['charSet']===false)
+			{
+				$singleBodyPart['charSet'] = Translation::detect_encoding($singleBodyPart['body']);
+			}
 			$singleBodyPart['body'] = Translation::convert(
 				$singleBodyPart['body'],
 				strtolower($singleBodyPart['charSet'])
 			);
 			$ct = 0;
 			$newStyle=array();
-			if (stripos($singleBodyPart['body'],'<style')!==false)  $ct = preg_match_all('#<style(?:\s.*)?>(.+)</style>#isU', $singleBodyPart['body'], $newStyle);
+			if (stripos($singleBodyPart['body'],'<style')!==false)
+			{
+				$ct = preg_match_all('#<style(?:\s.*)?>(.+)</style>#isU', $singleBodyPart['body'], $newStyle);
+			}
 			if ($ct>0)
 			{
 				//error_log(__METHOD__.' ('.__LINE__.') '.'#'.$ct.'#'.array2string($newStyle));
@@ -5340,7 +6283,11 @@ 
 		// CSS Security
 		// http://code.google.com/p/browsersec/wiki/Part1#Cascading_stylesheets
 		$css = preg_replace('/(javascript|expression|-moz-binding)/i','',$style);
-		if (stripos($css,'script')!==false) Mail\Html::replaceTagsCompletley($css,'script'); // Strip out script that may be included
+		if (stripos($css,'script')!==false)
+		{
+			Mail\Html::replaceTagsCompletley($css,'script');
+		}
+		// Strip out script that may be included
 		// we need this, as styledefinitions are enclosed with curly brackets; and template stuff tries to replace everything between curly brackets that is having no horizontal whitespace
 		// as the comments as <!-- styledefinition --> in stylesheet are outdated, and ck-editor does not understand it, we remove it
 		$css = str_replace(array(':','<!--','-->'),array(': ','',''),$css);
@@ -5361,8 +6308,14 @@ 
 	{
 		//TODO: caching einbauen static!
 		static $rawBody;
-		if (is_null($rawBody)) $rawBody = array();
-		if (empty($_folder)) $_folder = ($this->sessionData['mailbox']? $this->sessionData['mailbox'] : $this->icServer->getCurrentMailbox());
+		if (is_null($rawBody))
+		{
+			$rawBody = array();
+		}
+		if (empty($_folder))
+		{
+			$_folder = ($this->sessionData['mailbox']? $this->sessionData['mailbox'] : $this->icServer->getCurrentMailbox());
+		}
 		if (isset($rawBody[$this->icServer->ImapServerId][$_folder][$_uid][(empty($_partID)?'NIL':$_partID)]))
 		{
 			//error_log(__METHOD__.' ('.__LINE__.') '." Using Cache for raw Body $_uid, $_partID in Folder $_folder");
@@ -5371,7 +6324,10 @@ 
 
 		$uidsToFetch = new Horde_Imap_Client_Ids();
 		$uid = $_uid;
-		if (!(is_object($_uid) || is_array($_uid))) $uid = (array)$_uid;
+		if (!(is_object($_uid) || is_array($_uid)))
+		{
+			$uid = (array)$_uid;
+		}
 		$uidsToFetch->add($uid);
 
 		$fquery = new Horde_Imap_Client_Fetch_Query();
@@ -5384,8 +6340,10 @@ 
 		$headersNew = $this->icServer->fetch($_folder, $fquery, array(
 			'ids' => $uidsToFetch,
 		));
-		if (is_object($headersNew)) {
-			foreach($headersNew as &$_headerObject) {
+		if (is_object($headersNew))
+		{
+			foreach($headersNew as &$_headerObject)
+			{
 				$body = $_headerObject->getFullMsg();
 				if ($_partID != '')
 				{
@@ -5418,14 +6376,20 @@ 
 	 */
 	function getStructure($_uid, $_partID=null, $_folder=null, $_preserveSeen=false)
 	{
-		if (self::$debug) error_log( __METHOD__.' ('.__LINE__.') '.":$_uid, $_partID");
+		if (self::$debug)
+		{
+			error_log( __METHOD__.' ('.__LINE__.') '.":$_uid, $_partID");
+		}
 
 		if (empty($_folder))
 		{
 			$_folder = ($this->sessionData['mailbox']? $this->sessionData['mailbox'] : $this->icServer->getCurrentMailbox());
 		}
 		$uidsToFetch = new Horde_Imap_Client_Ids();
-		if (!(is_object($_uid) || is_array($_uid))) $_uid = (array)$_uid;
+		if (!(is_object($_uid) || is_array($_uid)))
+		{
+			$_uid = (array)$_uid;
+		}
 		$uidsToFetch->add($_uid);
 		try
 		{
@@ -5434,7 +6398,10 @@ 
 	//		$fquery->envelope();
 	//		$fquery->size();
 			$_fquery->structure();
-			if ($_partID) $_fquery->bodyPart($_partID, array('peek' => $_preserveSeen));
+			if ($_partID)
+			{
+				$_fquery->bodyPart($_partID, array('peek' => $_preserveSeen));
+			}
 
 			$mail = $this->icServer->fetch($_folder, $_fquery, array(
 				'ids' => $uidsToFetch,
@@ -5465,16 +6432,28 @@ 
 	 */
 	function getMessageAttachments($_uid, $_partID=null, Horde_Mime_Part $_structure=null, $fetchEmbeddedImages=true, $fetchTextCalendar=false, $resolveTNEF=true, $_folder='')
 	{
-		if (self::$debug) error_log( __METHOD__.":$_uid, $_partID");
-		if (empty($_folder)) $_folder = ($this->sessionData['mailbox']? $this->sessionData['mailbox'] : $this->icServer->getCurrentMailbox());
+		if (self::$debug)
+		{
+			error_log( __METHOD__.":$_uid, $_partID");
+		}
+		if (empty($_folder))
+		{
+			$_folder = ($this->sessionData['mailbox']? $this->sessionData['mailbox'] : $this->icServer->getCurrentMailbox());
+		}
 		$attachments = array();
 		if (!isset($_structure))
 		{
 			$_structure = $this->getStructure($_uid, $_partID,$_folder,true);
 			//error_log(__METHOD__.' ('.__LINE__.') '.':'.print_r($_structure->contentTypeMap(),true));
 		}
-		if (!$_structure || !$_structure->contentTypeMap()) return array();
-		if (!empty($_partID)) $_structure = $_structure->getPart($_partID);
+		if (!$_structure || !$_structure->contentTypeMap())
+		{
+			return array();
+		}
+		if (!empty($_partID))
+		{
+			$_structure = $_structure->getPart($_partID);
+		}
 		$skipParts = array();
 		$tnefParts = array();
 		$skip = 0;
@@ -5503,7 +6482,13 @@ 
 			if ($mime_type=='message/rfc822' && $_partID!=$mime_id)
 			{
 				//error_log(__METHOD__.' ('.__LINE__.') '.' Uid:'.$uid.'->'.$mime_id.':'.array2string($part->contentTypeMap()));
-				foreach($part->contentTypeMap() as $sub_id => $sub_type) {if ($sub_id != $mime_id) $skipParts[$sub_id] = $sub_type;}
+				foreach($part->contentTypeMap() as $sub_id => $sub_type)
+				{
+if ($sub_id != $mime_id)
+				{
+					$skipParts[$sub_id] = $sub_type;
+				}
+				}
 			}
 			if (empty($partDisposition) && $partPrimaryType != 'multipart' && $partPrimaryType != 'text')
 			{
@@ -5514,7 +6499,10 @@ 
 				$partDisposition='attachment';
 			}
 			//error_log(__METHOD__.' ('.__LINE__.') '.' Uid:'.$uid.' Part:'.$_partID.'->'.$mime_id.':'.array2string($skipParts));
-			if (array_key_exists($mime_id,$skipParts)) continue;
+			if (array_key_exists($mime_id,$skipParts))
+			{
+				continue;
+			}
 
 			if ($partDisposition == 'attachment' ||
 				(($partDisposition == 'inline' || empty($partDisposition)) && $partPrimaryType == 'image' && $part->getContentId()=='') ||
@@ -5525,23 +6513,41 @@ 
 			{
 				// if type is message/rfc822 and _partID is given, and MimeID equals partID
 				// we attempt to fetch "ourselves"
-				if ($_partID==$part->getMimeId() && $part->getPrimaryType()=='message') continue;
+				if ($_partID==$part->getMimeId() && $part->getPrimaryType()=='message')
+				{
+					continue;
+				}
 				$attachment = $part->getAllDispositionParameters();
 				$attachment['disposition'] = $part->getDisposition();
 				$attachment['mimeType'] = $mime_type;
 				$attachment['uid'] = $_uid;
 				$attachment['partID'] = $mime_id;
-				if (!isset($attachment['name'])||empty($attachment['name'])) $attachment['name'] = $part->getName();
+				if (!isset($attachment['name'])||empty($attachment['name']))
+				{
+					$attachment['name'] = $part->getName();
+				}
 				if ($fetchTextCalendar)
 				{
 					//error_log(__METHOD__.' ('.__LINE__.') '.array2string($part->getAllContentTypeParameters()));
 					$method = $part->getContentTypeParameter('method');
-					if ($method) $attachment['method'] = $method;
-					if (!isset($attachment['name'])) $attachment['name'] = 'event.ics';
+					if ($method)
+					{
+						$attachment['method'] = $method;
+					}
+					if (!isset($attachment['name']))
+					{
+						$attachment['name'] = 'event.ics';
+					}
 				}
 				$attachment['size'] = $part->getBytes();
-				if (($cid = $part->getContentId())) $attachment['cid'] = $cid;
-				if (empty($attachment['name'])) $attachment['name'] = (isset($attachment['cid'])&&!empty($attachment['cid'])?$attachment['cid']:lang("unknown").'_Uid'.$_uid.'_Part'.$mime_id).'.'.MimeMagic::mime2ext($mime_type);
+				if (($cid = $part->getContentId()))
+				{
+					$attachment['cid'] = $cid;
+				}
+				if (empty($attachment['name']))
+				{
+					$attachment['name'] = (isset($attachment['cid'])&&!empty($attachment['cid'])?$attachment['cid']:lang("unknown").'_Uid'.$_uid.'_Part'.$mime_id).'.'.MimeMagic::mime2ext($mime_type);
+				}
 				//error_log(__METHOD__.' ('.__LINE__.') '.' Uid:'.$uid.' Part:'.$_partID.'->'.$mime_id.':'.array2string($attachment));
 				//typical winmail.dat attachment is
 				//Array([size] => 1462762[filename] => winmail.dat[mimeType] => application/ms-tnef[uid] => 100[partID] => 2[name] => winmail.dat)
@@ -5577,14 +6583,26 @@ 
 						$attachment['uid'] = $tnp['uid'];
 						$attachment['partID'] = $tnp['partID'];
 						$attachment['is_winmail'] = $tnp['uid'].'@'.$tnp['partID'].'@'.$mime_id;
-						if (!isset($attachment['name'])||empty($attachment['name'])) $attachment['name'] = $part->getName();
+						if (!isset($attachment['name'])||empty($attachment['name']))
+						{
+							$attachment['name'] = $part->getName();
+						}
 						$attachment['size'] = $part->getBytes();
-						if (($cid = $part->getContentId())) $attachment['cid'] = $cid;
-						if (empty($attachment['name'])) $attachment['name'] = (isset($attachment['cid'])&&!empty($attachment['cid'])?$attachment['cid']:lang("unknown").'_Uid'.$_uid.'_Part'.$mime_id).'.'.MimeMagic::mime2ext($attachment['mimeType']);
+						if (($cid = $part->getContentId()))
+						{
+							$attachment['cid'] = $cid;
+						}
+						if (empty($attachment['name']))
+						{
+							$attachment['name'] = (isset($attachment['cid'])&&!empty($attachment['cid'])?$attachment['cid']:lang("unknown").'_Uid'.$_uid.'_Part'.$mime_id).'.'.MimeMagic::mime2ext($attachment['mimeType']);
+						}
 						$attachments[] = $attachment;
 					}
 				}
-				if ($tnefResolved===false) $attachments[]=$tnp;
+				if ($tnefResolved===false)
+				{
+					$attachments[]=$tnp;
+				}
 			}
 		}
 		//error_log(__METHOD__.__LINE__.array2string($attachments));
@@ -5688,8 +6706,14 @@ 
 
 				$attachment = $part->getAllDispositionParameters();
 				$attachment['mimeType'] = $part->getType();
-				if (!isset($attachment['filename'])||empty($attachment['filename'])) $attachment['filename'] = $part->getName();
-				if (($cid = $part->getContentId())) $attachment['cid'] = $cid;
+				if (!isset($attachment['filename'])||empty($attachment['filename']))
+				{
+					$attachment['filename'] = $part->getName();
+				}
+				if (($cid = $part->getContentId()))
+				{
+					$attachment['cid'] = $cid;
+				}
 				if (empty($attachment['filename']))
 				{
 					$attachment['filename'] = (isset($attachment['cid'])&&!empty($attachment['cid'])?
@@ -5701,7 +6725,10 @@ 
 				$attachments[$_uid.'@'.$_partID.'@'.$mime_id] = $attachment;
 			}
 		}
-		if (!is_array($attachments)) return false;
+		if (!is_array($attachments))
+		{
+			return false;
+		}
 		return $attachments;
 	}
 
@@ -5720,10 +6747,16 @@ 
 	function getAttachment($_uid, $_partID, $_winmail_nr=0, $_returnPart=true, $_stream=false, $_folder=null)
 	{
 		//error_log(__METHOD__.__LINE__."Uid:$_uid, PartId:$_partID, WinMailNr:$_winmail_nr, ReturnPart:$_returnPart, Stream:$_stream, Folder:$_folder".function_backtrace());
-		if (!isset($_folder)) $_folder = ($this->sessionData['mailbox']? $this->sessionData['mailbox'] : $this->icServer->getCurrentMailbox());
+		if (!isset($_folder))
+		{
+			$_folder = ($this->sessionData['mailbox']? $this->sessionData['mailbox'] : $this->icServer->getCurrentMailbox());
+		}
 
 		$uidsToFetch = new Horde_Imap_Client_Ids();
-		if (!(is_object($_uid) || is_array($_uid))) $_uid = (array)$_uid;
+		if (!(is_object($_uid) || is_array($_uid)))
+		{
+			$_uid = (array)$_uid;
+		}
 		$uidsToFetch->add($_uid);
 
 		$fquery = new Horde_Imap_Client_Fetch_Query();
@@ -5732,8 +6765,10 @@ 
 		$headersNew = $this->icServer->fetch($_folder, $fquery, array(
 			'ids' => $uidsToFetch,
 		));
-		if (is_object($headersNew)) {
-			foreach($headersNew as $id=>$_headerObject) {
+		if (is_object($headersNew))
+		{
+			foreach($headersNew as $id=>$_headerObject)
+			{
 				$body = $_headerObject->getFullMsg();
 				if ($_partID != '')
 				{
@@ -5747,7 +6782,10 @@ 
 					}
 					// if $partDisposition is empty, we assume attachment, and hope that the function
 					// itself is only triggered to fetch attachments
-					if (empty($partDisposition)) $partDisposition='attachment';
+					if (empty($partDisposition))
+					{
+						$partDisposition='attachment';
+					}
 					if ($part && ($partDisposition=='attachment' || $partDisposition=='inline' || ($part->getPrimaryType() == 'text' && $part->getSubType() == 'calendar')))
 					{
 						//$headerObject=$part->getAllDispositionParameters();//not used anywhere around here
@@ -5756,13 +6794,19 @@ 
 						$charset = $part->getContentTypeParameter('charset');
 						//$structure_bytes = $part->getBytes(); $structure_partID=$part->getMimeId(); error_log(__METHOD__.__LINE__." fetchPartContents(".array2string($_uid).", $structure_partID, $_stream, $_preserveSeen,$structure_mime)" );
 						$this->fetchPartContents($_uid, $part, $_stream, $_preserveSeen=true,$structure_mime);
-						if ($_returnPart) return $part;
+						if ($_returnPart)
+						{
+							return $part;
+						}
 					}
 				}
 			}
 		}
 		$ext = MimeMagic::mime2ext($structure_mime);
-		if ($ext && stripos($filename,'.')===false && stripos($filename,$ext)===false) $filename = trim($filename).'.'.$ext;
+		if ($ext && stripos($filename,'.')===false && stripos($filename,$ext)===false)
+		{
+			$filename = trim($filename).'.'.$ext;
+		}
 		if (!$part)
 		{
 			throw new Exception\WrongParameter("Error: Could not fetch attachment for Uid=".array2string($_uid).", PartId=$_partID, WinMailNr=$_winmail_nr, folder=$_folder");
@@ -5778,7 +6822,10 @@ 
 		);
 
 		// try guessing the mimetype, if we get the application/octet-stream
-		if (strtolower($attachmentData['type']) == 'application/octet-stream') $attachmentData['type'] = MimeMagic::filename2mime($attachmentData['filename']);
+		if (strtolower($attachmentData['type']) == 'application/octet-stream')
+		{
+			$attachmentData['type'] = MimeMagic::filename2mime($attachmentData['filename']);
+		}
 		# if the attachment holds a winmail number and is a winmail.dat then we have to handle that.
 		if ( $filename == 'winmail.dat' && $_winmail_nr)
 		{
@@ -5801,9 +6848,18 @@ 
 					if ($_winmail_nr == $wantedPart.'@'.$mime_id)
 					{
 						//error_log(__METHOD__.__LINE__.'#'.$structure_mime.'#'.$filename.'#'.array2string($attachment));
-						if (!isset($attachment['filename'])||empty($attachment['filename'])) $attachment['filename'] = $part->getName();
-						if (($cid = $part->getContentId())) $attachment['cid'] = $cid;
-						if (empty($attachment['filename'])) $attachment['filename'] = (isset($attachment['cid'])&&!empty($attachment['cid'])?$attachment['cid']:lang("unknown").'_Uid'.$_uid.'_Part'.$mime_id).'.'.MimeMagic::mime2ext($attachment['mimeType']);
+						if (!isset($attachment['filename'])||empty($attachment['filename']))
+						{
+							$attachment['filename'] = $part->getName();
+						}
+						if (($cid = $part->getContentId()))
+						{
+							$attachment['cid'] = $cid;
+						}
+						if (empty($attachment['filename']))
+						{
+							$attachment['filename'] = (isset($attachment['cid'])&&!empty($attachment['cid'])?$attachment['cid']:lang("unknown").'_Uid'.$_uid.'_Part'.$mime_id).'.'.MimeMagic::mime2ext($attachment['mimeType']);
+						}
 						$wmattach = $attachment;
 						$wmattach['attachment'] = $part->getContents(array('stream'=>$_stream));
 
@@ -5813,7 +6869,10 @@ 
 			if ($tnefResolved)
 			{
 				$ext = MimeMagic::mime2ext($wmattach['mimeType']);
-				if ($ext && stripos($wmattach['filename'],'.')===false && stripos($wmattach['filename'],$ext)===false) $wmattach['filename'] = trim($wmattach['filename']).'.'.$ext;
+				if ($ext && stripos($wmattach['filename'],'.')===false && stripos($wmattach['filename'],$ext)===false)
+				{
+					$wmattach['filename'] = trim($wmattach['filename']).'.'.$ext;
+				}
 				$attachmentData = array(
 					'type'       => $wmattach['mimeType'],
 					'filename'   => $wmattach['filename'],
@@ -5843,7 +6902,10 @@ 
 		static $uid=null, $part=null, $structure=null;
 		//error_log(__METHOD__.' ('.__LINE__.') '.":$_uid, $_cid, $_part");
 
-		if(empty($_cid)) return false;
+		if(empty($_cid))
+		{
+			return false;
+		}
 
 		if ($_uid != $uid || $_part != $part)
 		{
@@ -5868,13 +6930,19 @@ 
 					$attachment = $part;
 				}
 				// everything else we only consider after we checked all
-				if (!isset($attachment)) $attachment = $part;
+				if (!isset($attachment))
+				{
+					$attachment = $part;
+				}
 				// do we want content fetched, can be done later, if not needed
 				if (isset($_stream))
 				{
 					$this->fetchPartContents($_uid, $attachment, $_stream);
 				}
-				if (isset($attachment)) break;
+				if (isset($attachment))
+				{
+					break;
+				}
 			}
 		}
 		// set name as filename, if not set
@@ -5905,10 +6973,17 @@ 
 	 */
 	public function fetchPartContents($_uid, Horde_Mime_Part $part=null, $_stream=false, $_preserveSeen=false, $_mimetype=null)
 	{
-		if (is_null($part)) return null;//new Horde_Mime_Part;
+		if (is_null($part))
+		{
+			return null;
+		}
+		//new Horde_Mime_Part;
 		$encoding = null;
 		$fetchAsBinary = true;
-		if ($_mimetype && strtolower($_mimetype)=='message/rfc822') $fetchAsBinary = false;
+		if ($_mimetype && strtolower($_mimetype)=='message/rfc822')
+		{
+			$fetchAsBinary = false;
+		}
 		// we need to set content on structure to decode transfer encoding
 		$part->setContents(
 			$this->getBodyPart($_uid, $part->getMimeId(), null, $_preserveSeen, $_stream, $encoding, $fetchAsBinary),
@@ -5945,7 +7020,10 @@ 
 		// the recent flag is the default enforced here ; as we assume the _flags is always set,
 		// we default it to hordes default (Recent) (, other wise we should not pass the parameter
 		// for flags at all)
-		if (empty($_flags)) $_flags = '\\Recent';
+		if (empty($_flags))
+		{
+			$_flags = '\\Recent';
+		}
 		//if (!is_array($_flags) && stripos($_flags,',')!==false) $_flags=explode(',',$_flags);
 		//if (!is_array($_flags)) $_flags = (array) $_flags;
 		try
@@ -5962,18 +7040,27 @@ 
 		}
 		catch (\Exception $e)
 		{
-			if (self::$debug) error_log("Could not append Message: ".$e->getMessage());
+			if (self::$debug)
+			{
+				error_log("Could not append Message: ".$e->getMessage());
+			}
 			throw new Exception\WrongUserinput(lang("Could not append Message:").' '.$e->getMessage().': '.$e->details);
 			//return false;
 		}
 		//error_log(__METHOD__.' ('.__LINE__.') '.' appended UID:'.$messageid);
 		//$messageid = true; // for debug reasons only
-		if ($messageid === true || empty($messageid)) // try to figure out the message uid
+		if ($messageid === true || empty($messageid))
+		{
+			// try to figure out the message uid
 		{
 			$list = $this->getHeaders($_folderName, $_startMessage=1, 1, 'INTERNALDATE', true, array(),null, false);
+		}
 			if ($list)
 			{
-				if (self::$debug) error_log(__METHOD__.' ('.__LINE__.') '.' MessageUid:'.$messageid.' but found:'.array2string($list));
+				if (self::$debug)
+				{
+					error_log(__METHOD__.' ('.__LINE__.') '.' MessageUid:'.$messageid.' but found:'.array2string($list));
+				}
 				$messageid = $list['header'][0]['uid'];
 			}
 		}
@@ -6014,7 +7101,10 @@ 
 	{
 			//echo __METHOD__." called for $uid,$partid <br>";
 			$headers = $mailClass->getMessageHeader($uid,$partid,true,false,$mailbox);
-			if (empty($headers)) return false;
+			if (empty($headers))
+			{
+				return false;
+			}
 			// dont force retrieval of the textpart, let mailClass preferences decide
 			$bodyParts = $mailClass->getMessageBody($uid,($preserveHTML?'always_display':'only_if_no_text'),$partid,null,false,$mailbox);
 			// if we do not want HTML but there is no TextRepresentation with the message itself, try converting
@@ -6032,15 +7122,30 @@ 
 			//error_log(array2string($bodyParts));
 			$attachments = $includeAttachments?$mailClass->getMessageAttachments($uid,$partid,null,true,false,true,$mailbox):array();
 
-			if ($mailClass->isSentFolder($mailbox)) $mailaddress = $headers['TO'];
-			elseif (isset($headers['FROM'])) $mailaddress = $headers['FROM'];
-			elseif (isset($headers['SENDER'])) $mailaddress = $headers['SENDER'];
-			if (isset($headers['CC'])) $mailaddress .= ','.$headers['CC'];
+			if ($mailClass->isSentFolder($mailbox))
+			{
+				$mailaddress = $headers['TO'];
+			}
+			elseif (isset($headers['FROM']))
+			{
+				$mailaddress = $headers['FROM'];
+			}
+			elseif (isset($headers['SENDER']))
+			{
+				$mailaddress = $headers['SENDER'];
+			}
+			if (isset($headers['CC']))
+			{
+				$mailaddress .= ','.$headers['CC'];
+			}
 			//_debug_array(array($headers,$mailaddress));
 			$subject = $headers['SUBJECT'];
 
 			$message = self::getdisplayableBody($mailClass, $bodyParts, $preserveHTML);
-			if ($preserveHTML && $mailClass->activeMimeType == 'text/plain') $message = '<pre>'.$message.'</pre>';
+			if ($preserveHTML && $mailClass->activeMimeType == 'text/plain')
+			{
+				$message = '<pre>'.$message.'</pre>';
+			}
 			$headdata = ($addHeaderSection ? self::createHeaderInfoSection($headers, '',$preserveHTML) : '');
 			$message = $headdata.$message;
 			//echo __METHOD__.'<br>';
@@ -6091,7 +7196,8 @@ 
 							$attachments[$num]['attachment'] = $c->getContents();
 						}
 						// no attempt to convert, if we dont know about the charset
-						if (isset($attachments[$num]['charset'])&&!empty($attachments[$num]['charset'])) {
+						if (isset($attachments[$num]['charset'])&&!empty($attachments[$num]['charset']))
+						{
 							// we do not try guessing the charset, if it is not set
 							//if ($attachments[$num]['charset']===false) $attachments[$num]['charset'] = Translation::detect_encoding($attachments[$num]['attachment']);
 							Translation::convert($attachments[$num]['attachment'],$attachments[$num]['charset']);
@@ -6112,7 +7218,10 @@ 
 						unset($attachments[$num]['attachment']);
 					}
 				}
-				if (is_array($attachedMessages)) $attachments = array_merge($attachments,$attachedMessages);
+				if (is_array($attachedMessages))
+				{
+					$attachments = array_merge($attachments,$attachedMessages);
+				}
 			}
 			return array(
 					'mailaddress'=>$mailaddress,
@@ -6134,10 +7243,17 @@ 
 	{
 		$c = 0;
 		// use the standardIdentity
-		foreach($_identities as $key => $acc) {
-			if ($c==0) $identity = $acc;
+		foreach($_identities as $key => $acc)
+		{
+			if ($c==0)
+			{
+				$identity = $acc;
+			}
 			//error_log(__METHOD__.__LINE__." $key == $_profile_id ");
-			if ($key==$_profile_id) $identity = $acc;
+			if ($key==$_profile_id)
+			{
+				$identity = $acc;
+			}
 			$c++;
 		}
 		return $identity;
@@ -6153,20 +7269,53 @@ 
 	{
 		$headdata = null;
 		//error_log(__METHOD__.' ('.__LINE__.') '.array2string($header).function_backtrace());
-		if ($header['SUBJECT']) $headdata = lang('subject').': '.$header['SUBJECT'].($createHTML?"<br />":"\n");
-		if ($header['FROM']) $headdata .= lang('from').': '.self::convertAddressArrayToString($header['FROM'], $createHTML).($createHTML?"<br />":"\n");
-		if ($header['SENDER']) $headdata .= lang('sender').': '.self::convertAddressArrayToString($header['SENDER'], $createHTML).($createHTML?"<br />":"\n");
-		if ($header['TO']) $headdata .= lang('to').': '.self::convertAddressArrayToString($header['TO'], $createHTML).($createHTML?"<br />":"\n");
-		if ($header['CC']) $headdata .= lang('cc').': '.self::convertAddressArrayToString($header['CC'], $createHTML).($createHTML?"<br />":"\n");
-		if ($header['BCC']) $headdata .= lang('bcc').': '.self::convertAddressArrayToString($header['BCC'], $createHTML).($createHTML?"<br />":"\n");
-		if ($header['DATE']) $headdata .= lang('date').': '.$header['DATE'].($createHTML?"<br />":"\n");
-		if ($header['PRIORITY'] && $header['PRIORITY'] != 'normal') $headdata .= lang('priority').': '.$header['PRIORITY'].($createHTML?"<br />":"\n");
-		if ($header['IMPORTANCE'] && $header['IMPORTANCE'] !='normal') $headdata .= lang('importance').': '.$header['IMPORTANCE'].($createHTML?"<br />":"\n");
+		if ($header['SUBJECT'])
+		{
+			$headdata = lang('subject').': '.$header['SUBJECT'].($createHTML?"<br />":"\n");
+		}
+		if ($header['FROM'])
+		{
+			$headdata .= lang('from').': '.self::convertAddressArrayToString($header['FROM'], $createHTML).($createHTML?"<br />":"\n");
+		}
+		if ($header['SENDER'])
+		{
+			$headdata .= lang('sender').': '.self::convertAddressArrayToString($header['SENDER'], $createHTML).($createHTML?"<br />":"\n");
+		}
+		if ($header['TO'])
+		{
+			$headdata .= lang('to').': '.self::convertAddressArrayToString($header['TO'], $createHTML).($createHTML?"<br />":"\n");
+		}
+		if ($header['CC'])
+		{
+			$headdata .= lang('cc').': '.self::convertAddressArrayToString($header['CC'], $createHTML).($createHTML?"<br />":"\n");
+		}
+		if ($header['BCC'])
+		{
+			$headdata .= lang('bcc').': '.self::convertAddressArrayToString($header['BCC'], $createHTML).($createHTML?"<br />":"\n");
+		}
+		if ($header['DATE'])
+		{
+			$headdata .= lang('date').': '.$header['DATE'].($createHTML?"<br />":"\n");
+		}
+		if ($header['PRIORITY'] && $header['PRIORITY'] != 'normal')
+		{
+			$headdata .= lang('priority').': '.$header['PRIORITY'].($createHTML?"<br />":"\n");
+		}
+		if ($header['IMPORTANCE'] && $header['IMPORTANCE'] !='normal')
+		{
+			$headdata .= lang('importance').': '.$header['IMPORTANCE'].($createHTML?"<br />":"\n");
+		}
 		//if ($mailcontent['headers']['ORGANIZATION']) $headdata .= lang('organization').': '.$mailcontent['headers']['ORGANIZATION']."\
 		if (!empty($headdata))
 		{
-			if (!empty($headline) && $headline != 'SUPPRESS') $headdata = "---------------------------- $headline ----------------------------".($createHTML?"<br />":"\n").$headdata;
-			if (empty($headline)) $headdata = ($headline != 'SUPPRESS'?"--------------------------------------------------------".($createHTML?"<br />":"\n"):'').$headdata;
+			if (!empty($headline) && $headline != 'SUPPRESS')
+			{
+				$headdata = "---------------------------- $headline ----------------------------".($createHTML?"<br />":"\n").$headdata;
+			}
+			if (empty($headline))
+			{
+				$headdata = ($headline != 'SUPPRESS'?"--------------------------------------------------------".($createHTML?"<br />":"\n"):'').$headdata;
+			}
 			$headdata .= ($headline != 'SUPPRESS'?"--------------------------------------------------------".($createHTML?"<br />":"\n"):'');
 		}
 		else
@@ -6200,12 +7349,15 @@ 
 		$returnAddr ='';
 		if (is_array($rfcAddressArray))
 		{
-			foreach((array)$rfcAddressArray as $addressData) {
+			foreach((array)$rfcAddressArray as $addressData)
+			{
 				//error_log(__METHOD__.' ('.__LINE__.') '.array2string($addressData));
-				if($addressData['MAILBOX_NAME'] == 'NIL') {
+				if($addressData['MAILBOX_NAME'] == 'NIL')
+				{
 					continue;
 				}
-				if(strtolower($addressData['MAILBOX_NAME']) == 'undisclosed-recipients') {
+				if(strtolower($addressData['MAILBOX_NAME']) == 'undisclosed-recipients')
+				{
 					continue;
 				}
 				if ($addressData['RFC822_EMAIL'])
@@ -6219,7 +7371,10 @@ 
 				}
 				$addressObject = $addressObjectA[0];
 				//error_log(__METHOD__.' ('.__LINE__.') '.array2string($addressObject));
-				if (!$addressObject->valid) continue;
+				if (!$addressObject->valid)
+				{
+					continue;
+				}
 				//$mb =(string)$addressObject->mailbox;
 				//$h = (string)$addressObject->host;
 				//$p = (string)$addressObject->personal;
@@ -6236,7 +7391,10 @@ 
 			// do not mess with strings, return them untouched /* ToDo: validate string as Address */
 			$rfcAddressArray = self::decode_header($rfcAddressArray,true);
 			$rfcAddressArray = str_replace(array('<','>','"\'','\'"'),array('[',']','"','"'),$rfcAddressArray);
-			if (is_string($rfcAddressArray)) return $rfcAddressArray;
+			if (is_string($rfcAddressArray))
+			{
+				return $rfcAddressArray;
+			}
 		}
 		return $returnAddr;
 	}
@@ -6253,10 +7411,19 @@ 
 	{
 		$mergeobj = new Contacts\Merge();
 
-		if (empty($mimetype)) $mimetype = (strlen(strip_tags($content)) == strlen($content) ?'text/plain':'text/html');
+		if (empty($mimetype))
+		{
+			$mimetype = (strlen(strip_tags($content)) == strlen($content) ?'text/plain':'text/html');
+		}
 		$rv = $mergeobj->merge_string($content,$ids,$err='',$mimetype, array(), self::$displayCharset);
-		if (empty($rv) && !empty($content) && !empty($err)) $rv = $content;
-		if (!empty($err) && !empty($content) && !empty($ids)) error_log(__METHOD__.' ('.__LINE__.') '.' Merge failed for Ids:'.array2string($ids).' ContentType:'.$mimetype.' Content:'.$content.' Reason:'.array2string($err));
+		if (empty($rv) && !empty($content) && !empty($err))
+		{
+			$rv = $content;
+		}
+		if (!empty($err) && !empty($content) && !empty($ids))
+		{
+			error_log(__METHOD__.' ('.__LINE__.') '.' Merge failed for Ids:'.array2string($ids).' ContentType:'.$mimetype.' Content:'.$content.' Reason:'.array2string($err));
+		}
 		return $rv;
 	}
 
@@ -6293,13 +7460,15 @@ 
 			settype($bytes, 'integer');
 			$bytes /= 10;
 		}
-		else
-			settype($bytes, 'integer');
+		else {
+					settype($bytes, 'integer');
+		}
 
 		return $bytes . ' ' . $type ;
 	}
 
-	static function detect_qp(&$sting) {
+	static function detect_qp(&$sting)
+	{
 		$needle = '/(=[0-9][A-F])|(=[A-F][0-9])|(=[A-F][A-F])|(=[0-9][0-9])/';
 		return preg_match("$needle",$string);
 	}
@@ -6316,9 +7485,15 @@ 
 	 */
 	static function logRunTimes($_starttime,$_endtime=null,$_message='',$_methodNline='')
 	{
-		if (is_null($_endtime)) $_endtime = microtime(true);
+		if (is_null($_endtime))
+		{
+			$_endtime = microtime(true);
+		}
 		$usagetime = microtime(true) - $_starttime;
-		if (self::$debugTimes) error_log($_methodNline.' took:'.number_format($usagetime,5).'(s) '.($_message?'Details:'.$_message:''));
+		if (self::$debugTimes)
+		{
+			error_log($_methodNline.' took:'.number_format($usagetime,5).'(s) '.($_message?'Details:'.$_message:''));
+		}
 	}
 
 	/**
@@ -6333,7 +7508,10 @@ 
 	 */
 	static function checkFileBasics(&$_formData, $IDtoAddToFileName='', $reqMimeType='message/rfc822')
 	{
-		if (parse_url($_formData['file'],PHP_URL_SCHEME) == 'egw-data') return $_formData['file'];
+		if (parse_url($_formData['file'],PHP_URL_SCHEME) == 'egw-data')
+		{
+			return $_formData['file'];
+		}
 
 		//error_log(__METHOD__.__FILE__.array2string($_formData).' Id:'.$IDtoAddToFileName.' ReqMimeType:'.$reqMimeType);
 		$importfailed = $tmpFileName = false;
@@ -6382,8 +7560,15 @@ 
 			{
 				$buff = explode('.',$_formData['name']);
 				$suffix = '';
-				if (is_array($buff)) $suffix = array_pop($buff); // take the last extension to check with ext2mime
-				if (!empty($suffix)) $sfxMimeType = MimeMagic::ext2mime($suffix);
+				if (is_array($buff))
+				{
+					$suffix = array_pop($buff);
+				}
+				// take the last extension to check with ext2mime
+				if (!empty($suffix))
+				{
+					$sfxMimeType = MimeMagic::ext2mime($suffix);
+				}
 				if (!empty($suffix) && !empty($sfxMimeType) &&
 					(strlen(trim($_formData['type']))==0 || (strtolower(trim($_formData['type'])) != $sfxMimeType)))
 				{
@@ -6431,7 +7616,9 @@ 
 			{
 				rename($_formData['file'], $GLOBALS['egw_info']['server']['temp_dir'].'/'.$tmpFileName);
 			}
-		} else {
+		}
+		else
+		{
 			//error_log("Import of message ".$_formData['file']." failes to meet basic restrictions");
 			$importfailed = true;
 			$alert_msg .= lang("Processing of file %1 failed. Failed to meet basic restrictions.",$_formData['name']);
@@ -6477,21 +7664,32 @@ 
 				if (substr($url, 0, 5) !== 'data:')
 				{
 					$filename = basename($url);
-					if (($directory = dirname($url)) == '.') $directory = '';
+					if (($directory = dirname($url)) == '.')
+					{
+						$directory = '';
+					}
 					$ext = pathinfo($filename, PATHINFO_EXTENSION);
 					$mimeType  = MimeMagic::ext2mime($ext);
-					if ( strlen($directory) > 1 && substr($directory,-1) != '/') { $directory .= '/'; }
+					if ( strlen($directory) > 1 && substr($directory,-1) != '/')
+					{
+$directory .= '/'; }
 					$myUrl = $directory.$filename;
-					if ($myUrl[0]=='/') // local path -> we only allow path's that are available via http/https (or vfs)
+					if ($myUrl[0]=='/')
+					{
+						// local path -> we only allow path's that are available via http/https (or vfs)
 					{
 						$basedir = ($_SERVER['HTTPS']?'https://':'http://'.$_SERVER['HTTP_HOST']);
 					}
+					}
 					// use vfs instead of url containing webdav.php
 					// ToDo: we should test if the webdav url is of our own scope, as we cannot handle foreign
 					// webdav.php urls as vfs
-					if (strpos($myUrl,'/webdav.php') !== false) // we have a webdav link, so we build a vfs/sqlfs link of it.
+					if (strpos($myUrl,'/webdav.php') !== false)
+					{
+						// we have a webdav link, so we build a vfs/sqlfs link of it.
 					{
 						Vfs::load_wrapper('vfs');
+					}
 						list(,$myUrl) = explode('/webdav.php',$myUrl,2);
 						$basedir = 'vfs://default';
 						$needTempFile = false;
@@ -6520,8 +7718,13 @@ 
 						}
 					}
 
-					if ( strlen($basedir) > 1 && substr($basedir,-1) != '/' && $myUrl[0]!='/') { $basedir .= '/'; }
-					if ($needTempFile && !$attachment && substr($myUrl,0,4) !== "http") $data = file_get_contents($basedir.urldecode($myUrl));
+					if ( strlen($basedir) > 1 && substr($basedir,-1) != '/' && $myUrl[0]!='/')
+					{
+$basedir .= '/'; }
+					if ($needTempFile && !$attachment && substr($myUrl,0,4) !== "http")
+					{
+						$data = file_get_contents($basedir.urldecode($myUrl));
+					}
 				}
 				if (substr($url,0,strlen('data:'))=='data:')
 				{
@@ -6622,7 +7825,10 @@ 
 				$_folder = $this->getSentFolder();
 			}
 			$delimiter = $this->getHierarchyDelimiter();
-			if($_folder=='INBOX'.$delimiter) $_folder='INBOX';
+			if($_folder=='INBOX'.$delimiter)
+			{
+				$_folder='INBOX';
+			}
 			if ($importfailed === false)
 			{
 				$Subject = $mailObject->getHeader('Subject');
@@ -6651,7 +7857,9 @@ 
 					//error_log(__METHOD__.' ('.__LINE__.') '.' Id To Merge:'.$val);
 					if (/*$GLOBALS['egw_info']['flags']['currentapp'] == 'addressbook' &&*/
 						count($SendAndMergeTocontacts) > 1 && $val &&
-						(is_numeric($val) || $GLOBALS['egw']->accounts->name2id($val))) // do the merge
+						(is_numeric($val) || $GLOBALS['egw']->accounts->name2id($val)))
+					{
+						// do the merge
 					{
 						//error_log(__METHOD__.' ('.__LINE__.') '.array2string($mailObject));
 
@@ -6660,6 +7868,7 @@ 
 						{
 							//error_log('ID ' . $val . ' ' .$type . ': ' . $mailObject->getHeader(Mailer::$type2header[$type]) . ' -> ' .$bo_merge->merge_string($mailObject->getHeader(Mailer::$type2header[$type]),$val,$e,'text/plain',array(),self::$displayCharset));
 							$merged = $bo_merge->merge_string($mailObject->getHeader(Mailer::$type2header[$type]),$val,$e,'text/plain',array(),self::$displayCharset);
+					}
 							$mailObject->addAddress($merged,'',$type);
 							if($type == 'to')
 							{
@@ -6691,9 +7900,15 @@ 
 						$mailObject->clearCustomHeaders();
 						$mailObject->addHeader('Subject', $bo_merge->merge_string($Subject, $val, $e, 'text/plain', array(), self::$displayCharset));
 						//error_log(__METHOD__.' ('.__LINE__.') '.' ContentType:'.$mailObject->BodyContentType);
-						if($text_body) $text_body->setContents($bo_merge->merge_string($Body, $val, $e, 'text/plain', array(), self::$displayCharset),array('encoding'=>Horde_Mime_Part::DEFAULT_ENCODING));
+						if($text_body)
+						{
+							$text_body->setContents($bo_merge->merge_string($Body, $val, $e, 'text/plain', array(), self::$displayCharset),array('encoding'=>Horde_Mime_Part::DEFAULT_ENCODING));
+						}
 						//error_log(__METHOD__.' ('.__LINE__.') '.' Result:'.$mailObject->Body.' error:'.array2string($e));
-						if($html_body) $html_body->setContents($bo_merge->merge_string($AltBody, $val, $e, 'text/html', array(), self::$displayCharset),array('encoding'=>Horde_Mime_Part::DEFAULT_ENCODING));
+						if($html_body)
+						{
+							$html_body->setContents($bo_merge->merge_string($AltBody, $val, $e, 'text/html', array(), self::$displayCharset),array('encoding'=>Horde_Mime_Part::DEFAULT_ENCODING));
+						}
 
 						//error_log(__METHOD__.' ('.__LINE__.') '.array2string($mailObject));
 						// set a higher timeout for big messages
@@ -6708,9 +7923,12 @@ 
 							//error_log(__METHOD__.' ('.__LINE__.') '.array2string($errorInfo));
 						}
 					}
-					elseif (!$k)	// 1. entry, further entries will fail for apps other then addressbook
+					elseif (!$k)
+					{
+						// 1. entry, further entries will fail for apps other then addressbook
 					{
 						$openAsDraft = true;
+					}
 						$mailObject->removeHeader('Message-ID');
 						$mailObject->removeHeader('Date');
 						$mailObject->clearCustomHeaders();
@@ -6725,9 +7943,12 @@ 
 
 						// No addresses from placeholders?  Treat it as just a contact ID
 						if (count($mailObject->getAddresses('to',true)) == 0 &&
-							is_numeric($val) || $GLOBALS['egw']->accounts->name2id($val)) // do the merge
+							is_numeric($val) || $GLOBALS['egw']->accounts->name2id($val))
+						{
+							// do the merge
 						{
 							$contact = $bo_merge->contacts->read($val);
+						}
 							//error_log(__METHOD__.' ('.__LINE__.') '.array2string($contact));
 							$email = ($contact['email'] ? $contact['email'] : $contact['email_home']);
 							$nfn = ($contact['n_fn'] ? $contact['n_fn'] : $contact['n_given'].' '.$contact['n_family']);
@@ -6738,9 +7959,15 @@ 
 						}
 						$mailObject->addHeader('Subject', $bo_merge->merge_string($Subject, $val, $e, 'text/plain', array(), self::$displayCharset));
 						//error_log(__METHOD__.' ('.__LINE__.') '.' ContentType:'.$mailObject->BodyContentType);
-						if (!empty($Body)) $text_body->setContents($bo_merge->merge_string($Body, $val, $e, 'text/plain', array(), self::$displayCharset),array('encoding'=>Horde_Mime_Part::DEFAULT_ENCODING));
+						if (!empty($Body))
+						{
+							$text_body->setContents($bo_merge->merge_string($Body, $val, $e, 'text/plain', array(), self::$displayCharset),array('encoding'=>Horde_Mime_Part::DEFAULT_ENCODING));
+						}
 						//error_log(__METHOD__.' ('.__LINE__.') '.' Result:'.$mailObject->Body.' error:'.array2string($e));
-						if (!empty($AltBody)) $html_body->setContents($bo_merge->merge_string($AltBody, $val, $e, 'text/html', array(), self::$displayCharset),array('encoding'=>Horde_Mime_Part::DEFAULT_ENCODING));
+						if (!empty($AltBody))
+						{
+							$html_body->setContents($bo_merge->merge_string($AltBody, $val, $e, 'text/html', array(), self::$displayCharset),array('encoding'=>Horde_Mime_Part::DEFAULT_ENCODING));
+						}
 						$_folder = $this->getDraftFolder();
 					}
 					if ($sendOK || $openAsDraft)
@@ -6748,11 +7975,15 @@ 
 						if ($this->folderExists($_folder,true))
 						{
 						    if($this->isSentFolder($_folder))
-							{
+						    {
 						        $flags = '\\Seen';
-						    } elseif($this->isDraftFolder($_folder)) {
+						    }
+						    elseif($this->isDraftFolder($_folder))
+						    {
 						        $flags = '\\Draft';
-						    } else {
+						    }
+						    else
+						    {
 						        $flags = '';
 						    }
 							$savefailed = false;
@@ -6786,7 +8017,10 @@ 
 						}
 						else
 						{
-							if (!$openComposeWindow) $processStats['failed'][$val] = $errorInfo?$errorInfo:'Send failed to '.$nfn.'<'.$email.'> See error_log for details';
+							if (!$openComposeWindow)
+							{
+								$processStats['failed'][$val] = $errorInfo?$errorInfo:'Send failed to '.$nfn.'<'.$email.'> See error_log for details';
+							}
 						}
 					}
 					if (!is_null($sendOK) && $sendOK===false && is_null($openComposeWindow))
@@ -6834,7 +8068,10 @@ 
 				$tmpFileName = $GLOBALS['egw_info']['server']['temp_dir'].'/'.basename($tmpFileName);
 				break;
 		}
-		if (!isset($message)) $message = fopen($tmpFileName, 'r');
+		if (!isset($message))
+		{
+			$message = fopen($tmpFileName, 'r');
+		}
 
 		if (!$message)
 		{
@@ -6865,7 +8102,10 @@ 
 				(fseek($message, 0, SEEK_SET) == -1 ? '' : fread($message, 8192));
 
 			$length = strpos($start, Horde_Mime_Part::RFC_EOL.Horde_Mime_Part::RFC_EOL);
-			if ($length===false) $length = strlen($start);
+			if ($length===false)
+			{
+				$length = strlen($start);
+			}
 			$headers = Horde_Mime_Headers::parseHeaders(substr($start, 0,$length));
 
 			foreach($headers->toArray(array('nowrap' => true)) as $header => $value)
@@ -6894,7 +8134,10 @@ 
 		}
 		else
 		{
-			if (($type = gettype($message)) == 'object') $type = get_class ($message);
+			if (($type = gettype($message)) == 'object')
+			{
+				$type = get_class ($message);
+			}
 			throw new Exception\WrongParameter('Wrong parameter type for message: '.$type);
 		}
 	}
@@ -6918,7 +8161,9 @@ 
 			$matches = array();
 			preg_match_all("/[\w\.,-.,_.,0-9.]+@[\w\.,-.,_.,0-9.]+/",$addresses,$matches);
 			//error_log(__METHOD__.__LINE__.array2string($matches));
-			foreach ($matches[0] as &$match) {$match = trim($match,', ');}
+			foreach ($matches[0] as &$match)
+			{
+$match = trim($match,', ');}
 			$addresses = implode(',',$matches[0]);
 			//error_log(__METHOD__.__LINE__.array2string($addresses));
 			$ret = $rfc822->parseAddressList($addresses, $default_domain ? array('default_domain' => $default_domain) : array());
@@ -6949,7 +8194,10 @@ 
 			}
 			else
 			{
-				if ($previousFailed && $remember) $adr->personal = $remember. ' ' . $adr->personal;
+				if ($previousFailed && $remember)
+				{
+					$adr->personal = $remember. ' ' . $adr->personal;
+				}
 				$remember = '';
 				$previousFailed=false;
 				//error_log(__METHOD__.__LINE__."('$addresses', $default_domain) parsed $i: mailbox=$adr->mailbox, host=$adr->host, personal=$adr->personal");
@@ -6971,7 +8219,10 @@ 
 	{
 		$acc = Mail\Account::read($this->profileID);
 		$identity = Mail\Account::read_identity($acc['ident_id'], true, null, $acc);
-		if (self::$debug) error_log(__METHOD__.__LINE__.array2string($identity));
+		if (self::$debug)
+		{
+			error_log(__METHOD__.__LINE__.array2string($identity));
+		}
 		$headers = $this->getMessageHeader($uid, '', 'object', true, $_folder);
 
 		$mdn = new Horde_Mime_Mdn($headers);