EGroupware / egroupware

api/src/Mail.php

Braces +1825 added lines, -545 removed lines

@@ -200,7 +200,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));
 		self::$profileDefunct = Cache::getCache(Cache::INSTANCE,'email','profileDefunct'.trim($GLOBALS['egw_info']['user']['account_id']),null,array(),5*1);
@@ -231,9 +234,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)
@@ -272,7 +281,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
@@ -290,9 +300,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];
 	}
 
@@ -317,14 +333,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);
@@ -354,11 +376,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
@@ -367,11 +395,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;
 	}
 
@@ -387,10 +421,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());
@@ -405,7 +448,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);
@@ -429,7 +475,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');
+		}
 	}
 
 	/**
@@ -458,7 +507,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);
+		}
 	}
 
 	/**
@@ -468,7 +520,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];
+		}
 	}
 
 	/**
@@ -477,7 +532,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);
@@ -498,8 +556,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)))
 		{
@@ -532,7 +596,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')
 		{
@@ -561,7 +628,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
@@ -600,7 +670,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))
@@ -674,10 +747,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;
@@ -695,13 +772,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));
@@ -721,12 +805,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'],
@@ -769,15 +857,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]));
 			}
@@ -837,7 +935,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
@@ -847,12 +948,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__.') ');
+		}
 	}
 
 
@@ -867,13 +972,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);
@@ -898,7 +1012,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__.') ');
+		}
 	}
 
 	/**
@@ -910,7 +1027,10 @@ 
 	function getQuotaRoot()
 	{
 		static $quota;
-		if (isset($quota)) return $quota;
+		if (isset($quota))
+		{
+			return $quota;
+		}
 		if (isset(self::$profileDefunct[$this->profileID]) && strlen(self::$profileDefunct[$this->profileID]))
 		{
 			// something is wrong. Do not proceed. either no folder or profile is marked as defunct for this request
@@ -919,7 +1039,8 @@ 
 		try
 		{
 			$this->icServer->getCurrentMailbox();
-			if(!$this->icServer->hasCapability('QUOTA')) {
+			if(!$this->icServer->hasCapability('QUOTA'))
+			{
 				$quota = false;
 				return false;
 			}
@@ -937,12 +1058,15 @@ 
 			}
 		}
 		//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;
@@ -958,7 +1082,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;
 	}
 
@@ -976,9 +1104,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)
@@ -995,7 +1127,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'];
 					}
@@ -1022,7 +1156,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 "";
 	}
@@ -1036,8 +1173,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];
@@ -1071,14 +1214,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';
@@ -1106,7 +1254,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;
@@ -1159,7 +1308,10 @@ 
 	 */
 	function getFolderStatus($_folderName,$ignoreStatusCache=false,$basicInfoOnly=false,$fetchSubscribedInfo=true)
 	{
-		if (self::$debug) error_log(__METHOD__.' ('.__LINE__.') '." called with:$_folderName,$ignoreStatusCache,$basicInfoOnly");
+		if (self::$debug)
+		{
+			error_log(__METHOD__.' ('.__LINE__.') '." called with:$_folderName,$ignoreStatusCache,$basicInfoOnly");
+		}
 		if (!is_string($_folderName) || empty($_folderName)||(isset(self::$profileDefunct[$this->profileID]) && strlen(self::$profileDefunct[$this->profileID])))
 		{
 			// something is wrong. Do not proceed. either no folder or profile is marked as defunct for this request
@@ -1171,7 +1323,10 @@ 
 		{
 			$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;
 /*
@@ -1200,7 +1355,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
 			{
@@ -1209,7 +1367,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);
@@ -1244,7 +1403,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');
 		}
@@ -1253,7 +1413,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))
 		{
@@ -1265,24 +1428,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;
 		}
 
@@ -1301,8 +1474,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
@@ -1314,13 +1486,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;
@@ -1345,8 +1523,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);
@@ -1367,21 +1551,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;
@@ -1393,7 +1590,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;
@@ -1405,7 +1603,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;
@@ -1419,19 +1620,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
 		{
@@ -1440,7 +1658,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();
@@ -1464,7 +1685,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,
 			));
@@ -1487,14 +1711,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();
@@ -1507,7 +1737,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)
@@ -1518,12 +1751,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));
 				}
@@ -1533,11 +1770,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));
@@ -1569,7 +1811,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);
 						}
 					}
@@ -1585,7 +1831,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)
@@ -1598,7 +1847,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));
 						}
@@ -1614,7 +1864,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();
@@ -1624,7 +1877,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))));
@@ -1646,10 +1900,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)) ||
@@ -1660,7 +1923,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')
 						{
@@ -1674,18 +1940,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);
@@ -1696,25 +1971,35 @@ 
 				$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['REPLY-TO']) && $headerObject['REPLY-TO'][0]) {
+				if(is_array($headerObject['REPLY-TO']) && $headerObject['REPLY-TO'][0])
+				{
 					$retValue['header'][$sortOrder[$uid]]['reply_to_address'] = self::decode_header($headerObject['REPLY-TO'][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]));
@@ -1722,7 +2007,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)
 					{
@@ -1736,27 +2022,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;
@@ -1773,7 +2073,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);
@@ -1824,26 +2127,45 @@ 
 		//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);
+			}
 			$five=true;
 			$dReverse=1;
 			$deletedMessages = $this->getSortedList($_folderName, 0, $dReverse, array('status'=>array('DELETED')),$five,false);
-			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::$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']);
 		}
 		$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]);
@@ -1854,15 +2176,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(
@@ -1874,7 +2207,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(
@@ -1886,9 +2222,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
@@ -1902,7 +2246,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
 				{
@@ -1913,11 +2260,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)
 		{
@@ -1946,7 +2297,8 @@ 
 		$_reverse=false;
 		if (is_numeric($_sort))
 		{
-			switch($_sort) {
+			switch($_sort)
+			{
 				case 2:
 					$retValue = array(Horde_Imap_Client::SORT_FROM);
 					break;
@@ -1968,7 +2320,8 @@ 
 		}
 		else
 		{
-			switch(strtoupper($_sort)) {
+			switch(strtoupper($_sort))
+			{
 				case 'FROMADDRESS':
 					$retValue = array(Horde_Imap_Client::SORT_FROM);
 					break;
@@ -1994,7 +2347,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;
 	}
@@ -2013,7 +2369,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']) ||
@@ -2028,11 +2387,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':
@@ -2115,9 +2476,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':
@@ -2125,9 +2488,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)
@@ -2158,7 +2524,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;
 					}
@@ -2195,8 +2564,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']))
 		{
@@ -2211,7 +2586,8 @@ 
 				unset($_criterias['before']);
 				$criteria=$_criterias['range']='ON';
 			}
-			switch ($criteria) {
+			switch ($criteria)
+			{
 				case 'BETWEEN':
 					//try to be smart about missing
 					//enddate
@@ -2247,8 +2623,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)
 		{
@@ -2256,10 +2638,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;
 		}
 	}
@@ -2316,7 +2701,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')
 			{
@@ -2341,7 +2729,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)
@@ -2379,7 +2770,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');
 	}
@@ -2395,21 +2787,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
@@ -2457,13 +2864,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);
@@ -2537,8 +2950,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;
@@ -2546,23 +2965,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();
@@ -2575,7 +3006,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
@@ -2594,12 +3026,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
@@ -2650,8 +3084,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;
 						}
 
@@ -2681,7 +3119,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(
@@ -2704,20 +3143,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']);
+					}
 				}
 			}
 		}
@@ -2726,24 +3178,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);
 
@@ -2751,17 +3229,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';
@@ -2769,24 +3250,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];
@@ -2794,7 +3285,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
@@ -2803,7 +3295,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);
@@ -2815,7 +3310,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];
 	}
 
@@ -2843,31 +3341,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);
@@ -2943,7 +3454,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
@@ -2953,12 +3467,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)
 			{
@@ -2966,6 +3485,7 @@ 
 				case 0:
 				case 2:
 					$path = $_nodePath.''.$delimiter;
+		}
 					break;
 				// Node itself
 				// shouldn't contain next level delimiter
@@ -2984,11 +3504,14 @@ 
 
 			uasort($folders,array($this,'sortByMailbox'));//ksort($folders);
 		}
-		elseif(!$_nodePath) // all
+		elseif(!$_nodePath)
+		{
+			// all
 		{
 			if ($_subscribedOnly)
 			{
 				$folders = $this->icServer->listSubscribedMailboxes('', 0, true);
+		}
 			}
 			else
 			{
@@ -3000,7 +3523,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);
@@ -3010,73 +3536,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;
@@ -3092,7 +3648,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]);
@@ -3119,9 +3676,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);
@@ -3138,7 +3698,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
@@ -3188,7 +3751,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;
 	}
@@ -3215,7 +3784,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);
@@ -3244,7 +3816,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;
 	}
 
@@ -3275,7 +3850,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;
 	}
 
@@ -3296,11 +3874,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'];
@@ -3327,7 +3912,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();
 		}
@@ -3341,7 +3927,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>";
 
@@ -3349,7 +3936,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>";
@@ -3358,9 +3946,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);
 		}
 	}
@@ -3384,14 +3977,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
@@ -3401,25 +4001,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)
 			{
@@ -3428,9 +4042,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]))
 		{
@@ -3456,7 +4077,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)
 				{
@@ -3559,25 +4183,35 @@ 
 	function isSentFolder($_folderName, $_checkexistance=TRUE, $_exactMatch=false)
 	{
 		$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 ($_exactMatch)
 		{
-			if(false !== stripos($_folderName, $sentFolder)&& strlen($_folderName)==strlen($sentFolder)) {
+			if(false !== stripos($_folderName, $sentFolder)&& strlen($_folderName)==strlen($sentFolder))
+			{
 				return true;
-			} else {
+			}
+			else
+			{
 				return false;
 			}
-		} else {
-			if(false !== stripos($_folderName, $sentFolder)) {
+		}
+		else
+		{
+			if(false !== stripos($_folderName, $sentFolder))
+			{
 				return true;
-			} else {
+			}
+			else
+			{
 				return false;
 			}
 		}
@@ -3592,23 +4226,33 @@ 
 	 */
 	function isOutbox($_folderName, $_checkexistance=TRUE, $_exactMatch=false)
 	{
-		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 ($_exactMatch)
 			{
-				if(false !== stripos($_folderName, $outboxFolder)&& strlen($_folderName)==strlen($outboxFolder)) {
+				if(false !== stripos($_folderName, $outboxFolder)&& strlen($_folderName)==strlen($outboxFolder))
+				{
 					return true;
-				} else {
+				}
+				else
+				{
 					return false;
 				}
-			} else {
-				if(false !== stripos($_folderName, $outboxFolder)) {
+			}
+			else
+			{
+				if(false !== stripos($_folderName, $outboxFolder))
+				{
 					return true;
-				} else {
+				}
+				else
+				{
 					return false;
 				}
 			}
@@ -3626,25 +4270,38 @@ 
 	function isDraftFolder($_folderName, $_checkexistance=TRUE, $_exactMatch=false)
 	{
 		$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 (is_a($_folderName,"Horde_Imap_Client_Mailbox"))
+		{
+			$_folderName = $_folderName->utf8;
+		}
 		if ($_exactMatch)
 		{
-			if(false !== stripos($_folderName, $draftFolder)&& strlen($_folderName)==strlen($draftFolder)) {
+			if(false !== stripos($_folderName, $draftFolder)&& strlen($_folderName)==strlen($draftFolder))
+			{
+				return true;
+			}
+			else
+			{
+				return false;
+			}
+		}
+		else
+		{
+			if(false !== stripos($_folderName, $draftFolder))
+			{
 				return true;
-			} else {
-				return false;
 			}
-		} else {
-			if(false !== stripos($_folderName, $draftFolder)) {
-				return true;
-			} else {
+			else
+			{
 				return false;
 			}
 		}
@@ -3660,25 +4317,35 @@ 
 	function isTrashFolder($_folderName, $_checkexistance=TRUE, $_exactMatch=false)
 	{
 		$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 ($_exactMatch)
 		{
-			if(false !== stripos($_folderName, $trashFolder)&& strlen($_folderName)==strlen($trashFolder)) {
+			if(false !== stripos($_folderName, $trashFolder)&& strlen($_folderName)==strlen($trashFolder))
+			{
 				return true;
-			} else {
+			}
+			else
+			{
 				return false;
 			}
-		} else {
-			if(false !== stripos($_folderName, $trashFolder)) {
+		}
+		else
+		{
+			if(false !== stripos($_folderName, $trashFolder))
+			{
 				return true;
-			} else {
+			}
+			else
+			{
 				return false;
 			}
 		}
@@ -3694,24 +4361,34 @@ 
 	function isTemplateFolder($_folderName, $_checkexistance=TRUE, $_exactMatch=false)
 	{
 		$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 ($_exactMatch)
 		{
-			if(false !== stripos($_folderName, $templateFolder)&& strlen($_folderName)==strlen($templateFolder)) {
+			if(false !== stripos($_folderName, $templateFolder)&& strlen($_folderName)==strlen($templateFolder))
+			{
 				return true;
-			} else {
+			}
+			else
+			{
 				return false;
 			}
-		} else {
-			if(false !== stripos($_folderName, $templateFolder)) {
+		}
+		else
+		{
+			if(false !== stripos($_folderName, $templateFolder))
+			{
 				return true;
-			} else {
+			}
+			else
+			{
 				return false;
 			}
 		}
@@ -3730,15 +4407,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)
 		{
@@ -3756,7 +4445,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
@@ -3797,9 +4487,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);
 		}
 	}
@@ -3818,10 +4511,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')
@@ -3831,32 +4530,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
 					{
@@ -3872,7 +4587,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
@@ -3880,7 +4598,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)
@@ -3892,7 +4613,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);
@@ -3909,7 +4633,8 @@ 
 				$this->icServer->expunge($_folder);
 				break;
 		}
-		if($oldMailbox != '') {
+		if($oldMailbox != '')
+		{
 			$this->icServer->openMailbox($oldMailbox);
 		}
 
@@ -3923,11 +4648,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();
@@ -3935,8 +4664,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();
 				}
@@ -3961,10 +4692,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)
 		{
@@ -3972,10 +4709,14 @@ 
 		}
 
 		if ( stripos( array2string($flags),'MDNSent')!==false)
-			return true;
+		{
+					return true;
+		}
 
 		if ( stripos( array2string($flags),'MDNnotSent')!==false)
-			return false;
+		{
+					return false;
+		}
 
 		return null;
 	}
@@ -3996,7 +4737,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']));
@@ -4004,7 +4748,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')
@@ -4013,7 +4760,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
@@ -4029,7 +4779,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;
@@ -4115,7 +4866,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;
 
@@ -4144,7 +4898,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')
@@ -4156,7 +4913,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']);
@@ -4178,12 +4938,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();
@@ -4267,7 +5029,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)
@@ -4299,9 +5065,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;
 	}
 
@@ -4321,18 +5093,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])
@@ -4371,8 +5166,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;
 			}
@@ -4393,7 +5190,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':
@@ -4434,13 +5234,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)
 			{
@@ -4456,15 +5262,24 @@ 
 					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;
 
 						case 'calendar':
-							if ($mimePart->getBytes() > 0)  $partCalendar = $mimePart;
+							if ($mimePart->getBytes() > 0)
+							{
+								$partCalendar = $mimePart;
+							}
 							break;
 					}
 					break;
@@ -4477,7 +5292,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
 							}
@@ -4558,9 +5376,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();
@@ -4568,7 +5392,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;
@@ -4586,7 +5413,10 @@ 
 			switch($part->getPrimaryType())
 			{
 				case 'multipart':
-					if ($part->getDisposition() == 'attachment') continue;
+					if ($part->getDisposition() == 'attachment')
+					{
+						continue;
+					}
 					switch($part->getSubType())
 					{
 						case 'alternative':
@@ -4651,7 +5481,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;
 					}
@@ -4694,7 +5526,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))
 		{
@@ -4703,7 +5538,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();
@@ -4711,9 +5549,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,
 			);
@@ -4755,7 +5596,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')))
 		{
@@ -4806,8 +5650,12 @@ 
 	 */
 	function getMessageBody($_uid, $_htmlOptions='', $_partID=null, Horde_Mime_Part $_structure=null, $_preserveSeen = false, $_folder = '', &$calendar_part=null)
 	{
-		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))
@@ -4896,7 +5744,9 @@ 
 						default:
 							$bodyPart = array($this->getTextPart($_uid, $_structure, $this->htmlOptions, $_preserveSeen));
 					}
-				} else {
+				}
+				else
+				{
 					// what if the structure->disposition is attachment ,...
 				}
 				return self::normalizeBodyParts($bodyPart);
@@ -4907,13 +5757,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(),
@@ -4937,9 +5792,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;
@@ -4965,13 +5823,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)@',
@@ -4998,13 +5863,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)
 				{
@@ -5050,15 +5919,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;
@@ -5095,12 +5977,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
@@ -5110,14 +5992,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";
@@ -5138,11 +6026,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();
@@ -5152,8 +6047,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();
@@ -5170,7 +6067,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)
@@ -5207,7 +6107,9 @@ 
 				}
 			}
 			return $envelope;
-		} else {
+		}
+		else
+		{
 
 			$headers = $this->getMessageHeader($_uid, $_partID, true,true,$_folder);
 
@@ -5218,19 +6120,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',
@@ -5238,17 +6164,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();
 					}
 				}
@@ -5271,9 +6205,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();
@@ -5291,7 +6231,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);
@@ -5313,11 +6254,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));
@@ -5349,10 +6296,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");
@@ -5360,7 +6313,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();
@@ -5376,8 +6332,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 != '')
 				{
@@ -5405,22 +6363,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));
@@ -5452,7 +6421,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);
@@ -5474,8 +6447,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 (!$_stream && 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");
@@ -5484,7 +6463,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();
@@ -5497,8 +6479,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($_stream);
 				if ($_partID != '')
 				{
@@ -5534,14 +6518,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
 		{
@@ -5550,7 +6540,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,
@@ -5581,16 +6574,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;
@@ -5619,7 +6624,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')
 			{
@@ -5630,7 +6641,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()=='') ||
@@ -5641,23 +6655,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)
@@ -5693,14 +6725,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));
@@ -5804,8 +6848,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'])?
@@ -5817,7 +6867,10 @@ 
 				$attachments[$_uid.'@'.$_partID.'@'.$mime_id] = $attachment;
 			}
 		}
-		if (!is_array($attachments)) return false;
+		if (!is_array($attachments))
+		{
+			return false;
+		}
 		return $attachments;
 	}
 
@@ -5836,10 +6889,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();
@@ -5848,8 +6907,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 != '')
 				{
@@ -5863,7 +6924,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
@@ -5872,13 +6936,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");
@@ -5894,7 +6964,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)
 		{
@@ -5917,9 +6990,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));
 
@@ -5929,7 +7011,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'],
@@ -5959,7 +7044,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)
 		{
@@ -5984,13 +7072,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
@@ -6021,10 +7115,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),
@@ -6061,7 +7162,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
@@ -6078,18 +7182,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'];
 			}
 		}
@@ -6130,7 +7243,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
@@ -6148,15 +7264,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>';
@@ -6207,7 +7338,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']);
@@ -6228,7 +7360,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,
@@ -6250,10 +7385,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;
@@ -6269,20 +7411,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
@@ -6316,12 +7491,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'])
@@ -6335,7 +7513,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;
@@ -6352,7 +7533,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;
 	}
@@ -6369,10 +7553,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;
 	}
 
@@ -6409,13 +7602,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);
 	}
@@ -6432,9 +7627,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:''));
+		}
 	}
 
 	/**
@@ -6449,7 +7650,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;
@@ -6498,8 +7702,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)))
 				{
@@ -6547,7 +7758,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']);
@@ -6593,21 +7806,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;
@@ -6636,8 +7860,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:')
 				{
@@ -6745,7 +7974,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');
@@ -6774,7 +8006,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));
 
@@ -6783,6 +8017,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')
 							{
@@ -6814,9 +8049,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
@@ -6831,9 +8072,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();
@@ -6848,9 +8092,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']);
@@ -6861,9 +8108,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)
@@ -6871,11 +8124,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;
@@ -6909,7 +8166,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))
@@ -6957,7 +8217,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)
 		{
@@ -6996,7 +8259,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)
@@ -7014,7 +8280,10 @@ 
 							break;
 						default:
 							//error_log(__METHOD__.__LINE__.':'.$header.'->'.$val);
-							if (strtolower($header)=='bcc') $bccAddresses .= $val;
+							if (strtolower($header)=='bcc')
+							{
+								$bccAddresses .= $val;
+							}
 							$mailer->addHeader($header, $val, $overwrite);
 							//error_log(__METHOD__.__LINE__.':'.'getHeader('.$header.')'.array2string($mailer->getHeader($header)).(strtolower($header)=='bcc'?$bccAddresses:''));
 					}
@@ -7027,7 +8296,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);
 		}
 	}
@@ -7051,7 +8323,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());
@@ -7082,7 +8356,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");
@@ -7104,7 +8381,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);

mail/inc/class.mail_zpush.inc.php

Braces +605 added lines, -177 removed lines

@@ -84,29 +84,47 @@ 
 	 */
 	public function __construct(activesync_backend $backend)
 	{
-		if ($GLOBALS['egw_setup']) return;
+		if ($GLOBALS['egw_setup'])
+		{
+			return;
+		}
 
 		//$this->debugLevel=2;
 		$this->backend = $backend;
 		if (!isset($GLOBALS['egw_info']['user']['preferences']['activesync']['mail-ActiveSyncProfileID']))
 		{
-			if ($this->debugLevel>1) error_log(__METHOD__.__LINE__.' Noprefs set: using 0 as default');
+			if ($this->debugLevel>1)
+			{
+				error_log(__METHOD__.__LINE__.' Noprefs set: using 0 as default');
+			}
 			// globals preferences add appname varname value
 			$GLOBALS['egw']->preferences->add('activesync','mail-ActiveSyncProfileID',0,'user');
 			// save prefs
 			$GLOBALS['egw']->preferences->save_repository(true);
 		}
-		if ($this->debugLevel>1) error_log(__METHOD__.__LINE__.' ActiveProfileID:'.array2string(self::$profileID));
+		if ($this->debugLevel>1)
+		{
+			error_log(__METHOD__.__LINE__.' ActiveProfileID:'.array2string(self::$profileID));
+		}
 
 		if (is_null(self::$profileID))
 		{
-			if ($this->debugLevel>1) error_log(__METHOD__.__LINE__.' self::ProfileID isNUll:'.array2string(self::$profileID));
+			if ($this->debugLevel>1)
+			{
+				error_log(__METHOD__.__LINE__.' self::ProfileID isNUll:'.array2string(self::$profileID));
+			}
 			self::$profileID =& Api\Cache::getSession('mail','activeSyncProfileID');
-			if ($this->debugLevel>1) error_log(__METHOD__.__LINE__.' ActiveProfileID (after reading Cache):'.array2string(self::$profileID));
+			if ($this->debugLevel>1)
+			{
+				error_log(__METHOD__.__LINE__.' ActiveProfileID (after reading Cache):'.array2string(self::$profileID));
+			}
 		}
 		if (isset($GLOBALS['egw_info']['user']['preferences']['activesync']['mail-ActiveSyncProfileID']))
 		{
-			if ($this->debugLevel>1) error_log(__METHOD__.__LINE__.' Pref for ProfileID:'.array2string($GLOBALS['egw_info']['user']['preferences']['activesync']['mail-ActiveSyncProfileID']));
+			if ($this->debugLevel>1)
+			{
+				error_log(__METHOD__.__LINE__.' Pref for ProfileID:'.array2string($GLOBALS['egw_info']['user']['preferences']['activesync']['mail-ActiveSyncProfileID']));
+			}
 			if ($GLOBALS['egw_info']['user']['preferences']['activesync']['mail-ActiveSyncProfileID'] == 'G')
 			{
 				self::$profileID = 'G'; // this should trigger the fetch of the first negative profile (or if no negative profile is available the firstb there is)
@@ -116,7 +134,10 @@ 
 				self::$profileID = (int)$GLOBALS['egw_info']['user']['preferences']['activesync']['mail-ActiveSyncProfileID'];
 			}
 		}
-		if ($this->debugLevel>1) error_log(__METHOD__.__LINE__.' Profile Selected (after reading Prefs):'.array2string(self::$profileID));
+		if ($this->debugLevel>1)
+		{
+			error_log(__METHOD__.__LINE__.' Profile Selected (after reading Prefs):'.array2string(self::$profileID));
+		}
 
 		// verify we are on an existing profile, if not running in setup (settings can not be static according to interface!)
 		if (!isset($GLOBALS['egw_setup']))
@@ -129,7 +150,10 @@ 
 				self::$profileID = Mail\Account::get_default_acc_id();
 			}
 		}
-		if ($this->debugLevel>0) error_log(__METHOD__.'::'.__LINE__.' ProfileSelected:'.self::$profileID);
+		if ($this->debugLevel>0)
+		{
+			error_log(__METHOD__.'::'.__LINE__.' ProfileSelected:'.self::$profileID);
+		}
 		//$this->debugLevel=0;
 	}
 
@@ -245,8 +269,14 @@ 
 				$this->_connect(0,true);
 				$this->_disconnect();
 
-				if (!$this->_wasteID) $errors[] = lang('No valid %1 folder configured!', '<b>'.lang('trash').'</b>');
-				if (!$this->_sentID) $errors[] = lang('No valid %1 folder configured!', '<b>'.lang('send').'</b>');
+				if (!$this->_wasteID)
+				{
+					$errors[] = lang('No valid %1 folder configured!', '<b>'.lang('trash').'</b>');
+				}
+				if (!$this->_sentID)
+				{
+					$errors[] = lang('No valid %1 folder configured!', '<b>'.lang('send').'</b>');
+				}
 			}
 			catch(Exception $e) {
 				$errors[] = lang('Can not open IMAP connection').': '.$e->getMessage();
@@ -268,8 +298,14 @@ 
 	 */
 	private function _connect($account=0)
 	{
-		if (!$account) $account = self::$profileID ? self::$profileID : 0;
-		if ($this->mail && $this->account != $account) $this->_disconnect();
+		if (!$account)
+		{
+			$account = self::$profileID ? self::$profileID : 0;
+		}
+		if ($this->mail && $this->account != $account)
+		{
+			$this->_disconnect();
+		}
 
 		$this->_wasteID = false;
 		$this->_sentID = false;
@@ -280,12 +316,18 @@ 
 			// todo: tell mail which account to use
 			//error_log(__METHOD__.__LINE__.' create object with ProfileID:'.array2string(self::$profileID));
 			$this->mail = Mail::getInstance(false,self::$profileID,true,false,true);
-			if (self::$profileID == 0 && isset($this->mail->icServer->ImapServerId) && !empty($this->mail->icServer->ImapServerId)) self::$profileID = $this->mail->icServer->ImapServerId;
+			if (self::$profileID == 0 && isset($this->mail->icServer->ImapServerId) && !empty($this->mail->icServer->ImapServerId))
+			{
+				self::$profileID = $this->mail->icServer->ImapServerId;
+			}
 		}
 		else
 		{
 			//error_log(__METHOD__.__LINE__." connect with profileID: ".self::$profileID);
-			if (self::$profileID == 0 && isset($this->mail->icServer->ImapServerId) && !empty($this->mail->icServer->ImapServerId)) self::$profileID = $this->mail->icServer->ImapServerId;
+			if (self::$profileID == 0 && isset($this->mail->icServer->ImapServerId) && !empty($this->mail->icServer->ImapServerId))
+			{
+				self::$profileID = $this->mail->icServer->ImapServerId;
+			}
 		}
 		$this->mail->openConnection(self::$profileID,false);
 
@@ -302,8 +344,14 @@ 
 	 */
 	private function _disconnect()
 	{
-		if ($this->debugLevel>0) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__);
-		if ($this->mail) $this->mail->closeConnection();
+		if ($this->debugLevel>0)
+		{
+			ZLog::Write(LOGLEVEL_DEBUG,__METHOD__);
+		}
+		if ($this->mail)
+		{
+			$this->mail->closeConnection();
+		}
 
 		unset($this->mail);
 		unset($this->account);
@@ -318,24 +366,43 @@ 
 	public function GetFolderList()
 	{
 		$folderlist = array();
-		if ($this->debugLevel>0) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__);
+		if ($this->debugLevel>0)
+		{
+			ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__);
+		}
 		/*foreach($available_accounts as $account)*/ $account = 0;
 		{
 			$this->_connect($account);
-			if (!isset($this->folders)) $this->folders = $this->mail->getFolderObjects(true,false,$_alwaysGetDefaultFolders=true);
-			if ($this->debugLevel>1) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.array2string($this->folders));
+			if (!isset($this->folders))
+			{
+				$this->folders = $this->mail->getFolderObjects(true,false,$_alwaysGetDefaultFolders=true);
+			}
+			if ($this->debugLevel>1)
+			{
+				ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.array2string($this->folders));
+			}
 
-			foreach ($this->folders as $folder => $folderObj) {
-				if ($this->debugLevel>1) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' folder='.$folder);
+			foreach ($this->folders as $folder => $folderObj)
+			{
+				if ($this->debugLevel>1)
+				{
+					ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' folder='.$folder);
+				}
 				$folderlist[] = $f = array(
 					'id'     => $this->createID($account,$folder),
 					'mod'    => $folderObj->shortDisplayName,
 					'parent' => $this->getParentID($account,$folder),
 				);
-				if ($this->debugLevel>1) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."() returning ".array2string($f));
+				if ($this->debugLevel>1)
+				{
+					ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."() returning ".array2string($f));
+				}
 			}
 		}
-		if ($this->debugLevel>0) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."() returning ".array2string($folderlist));
+		if ($this->debugLevel>0)
+		{
+			ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."() returning ".array2string($folderlist));
+		}
 
 		return $folderlist;
 	}
@@ -396,11 +463,17 @@ 
 		$smartdata_task = ($smartdata->replyflag?'reply':($smartdata->forwardflag?'forward':'new'));
 
    		ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__ . (isset($smartdata->mime) ? $smartdata->mime : ""). "task: ".(isset($smartdata_task) ? $smartdata_task : "")." itemid: ".(isset($smartdata->source->itemid) ? $smartdata->source->itemid : "")." folder: ".(isset($smartdata->source->folderid) ? $smartdata->source->folderid : ""));
-		if ($this->debugLevel>0) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."(".__LINE__."): Smartdata = ".array2string($smartdata));
+		if ($this->debugLevel>0)
+		{
+			ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."(".__LINE__."): Smartdata = ".array2string($smartdata));
+		}
 		//error_log("IMAP-Sendmail: Smartdata = ".array2string($smartdata));
 
 		// initialize our Mail
-		if (!isset($this->mail)) $this->mail = Mail::getInstance(false,self::$profileID,true,false,true);
+		if (!isset($this->mail))
+		{
+			$this->mail = Mail::getInstance(false,self::$profileID,true,false,true);
+		}
 		$activeMailProfiles = $this->mail->getAccountIdentities(self::$profileID);
 		// use the standardIdentity
 		$activeMailProfile = Mail::getStandardIdentityForProfile($activeMailProfiles,self::$profileID);
@@ -428,12 +501,18 @@ 
 		$beforeHtml = ($disableRuler ?'&nbsp;<br>':'&nbsp;<br><hr style="border:dotted 1px silver; width:90%; border:dotted 1px silver;">');
 		$beforePlain = ($disableRuler ?"\r\n\r\n":"\r\n\r\n-- \r\n");
 		$sigText = Mail::merge($signature,array($GLOBALS['egw']->accounts->id2name($GLOBALS['egw_info']['user']['account_id'],'person_id')));
-		if ($this->debugLevel>0) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' Signature to use:'.$sigText);
+		if ($this->debugLevel>0)
+		{
+			ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' Signature to use:'.$sigText);
+		}
 		$sigTextHtml = $beforeHtml.$sigText;
 		$sigTextPlain = $beforePlain.Api\Mail\Html::convertHTMLToText($sigText);
 
 		$force8bit=false;
-		if (Api\Translation::detect_encoding($sigTextPlain)!='ascii') $force8bit=true;
+		if (Api\Translation::detect_encoding($sigTextPlain)!='ascii')
+		{
+			$force8bit=true;
+		}
 		// initialize the new Api\Mailer object for sending
 		$mailObject = new Api\Mailer(self::$profileID);
 		$bccAddresses='';
@@ -445,22 +524,34 @@ 
 
 		// prepare addressee list; moved the adding of addresses to the mailobject down
 		// to
-		foreach(Mail::parseAddressList($mailObject->getHeader("To")) as $addressObject) {
-			if (!$addressObject->valid) continue;
+		foreach(Mail::parseAddressList($mailObject->getHeader("To")) as $addressObject)
+		{
+			if (!$addressObject->valid)
+			{
+				continue;
+			}
 			ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."(".__LINE__.") Header Sentmail To: ".array2string($addressObject) );
 			//$mailObject->AddAddress($addressObject->mailbox. ($addressObject->host ? '@'.$addressObject->host : ''),$addressObject->personal);
 			$toMailAddr[] = imap_rfc822_write_address($addressObject->mailbox, $addressObject->host, $addressObject->personal);
 		}
 		// CC
-		foreach(Mail::parseAddressList($mailObject->getHeader("Cc")) as $addressObject) {
-			if (!$addressObject->valid) continue;
+		foreach(Mail::parseAddressList($mailObject->getHeader("Cc")) as $addressObject)
+		{
+			if (!$addressObject->valid)
+			{
+				continue;
+			}
 			ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."(".__LINE__.") Header Sentmail CC: ".array2string($addressObject) );
 			//$mailObject->AddCC($addressObject->mailbox. ($addressObject->host ? '@'.$addressObject->host : ''),$addressObject->personal);
 			$ccMailAddr[] = imap_rfc822_write_address($addressObject->mailbox, $addressObject->host, $addressObject->personal);
 		}
 		// BCC
-		foreach(Mail::parseAddressList(($bccAddresses?$bccAddresses:$mailObject->getHeader("Bcc"))) as $addressObject) {
-			if (!$addressObject->valid) continue;
+		foreach(Mail::parseAddressList(($bccAddresses?$bccAddresses:$mailObject->getHeader("Bcc"))) as $addressObject)
+		{
+			if (!$addressObject->valid)
+			{
+				continue;
+			}
 			ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."(".__LINE__.") Header Sentmail BCC: ".array2string($addressObject) );
 			//$mailObject->AddBCC($addressObject->mailbox. ($addressObject->host ? '@'.$addressObject->host : ''),$addressObject->personal);
 			$bccMailAddr[] = imap_rfc822_write_address($addressObject->mailbox, $addressObject->host, $addressObject->personal);
@@ -473,12 +564,14 @@ 
 		$ContentType =$mailObject->getHeader('Content-Type');
 		//error_log(__METHOD__.__LINE__." Header Sentmail original Header (filtered): " . $k.  " = ".trim($ContentType));
 		// if the message is a multipart message, then we should use the sent body
-		if (preg_match("/multipart/i", $ContentType)) {
+		if (preg_match("/multipart/i", $ContentType))
+		{
 			$use_orgbody = true;
 		}
 
 		// save the original content-type header for the body part when forwarding
-		if ($smartdata_task == 'forward' && $smartdata->source->itemid && !$use_orgbody) {
+		if ($smartdata_task == 'forward' && $smartdata->source->itemid && !$use_orgbody)
+		{
 			//continue; // ignore
 		}
 		// horde/egw_ mailer does everything as utf-8, the following should not be needed
@@ -487,7 +580,8 @@ 
 		// if the message is a multipart message, then we should use the sent body
 		if (($smartdata_task == 'new' || $smartdata_task == 'reply' || $smartdata_task == 'forward') &&
 			((isset($smartdata->replacemime) && $smartdata->replacemime == true) ||
-			$k == "Content-Type" && preg_match("/multipart/i", $ContentType))) {
+			$k == "Content-Type" && preg_match("/multipart/i", $ContentType)))
+		{
 			$use_orgbody = true;
 		}
 		$Body =  $AltBody = "";
@@ -520,12 +614,19 @@ 
 			$AltBody = preg_replace("/(<|&lt;)*(([\w\.,-.,_.,0-9.]+)@([\w\.,-.,_.,0-9.]+))(>|&gt;)*/i","[$2]",
 				($html_body = $mailObject->findBody('html')) ? $html_body->getContents() : null);
 		}
-		if ($this->debugLevel>1 && $Body) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."(".__LINE__.") fetched Body as with MessageContentType:". $ContentType.'=>'.$Body);
-		if ($this->debugLevel>1 && $AltBody) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."(".__LINE__.") fetched AltBody as with MessageContentType:". $ContentType.'=>'.$AltBody);
+		if ($this->debugLevel>1 && $Body)
+		{
+			ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."(".__LINE__.") fetched Body as with MessageContentType:". $ContentType.'=>'.$Body);
+		}
+		if ($this->debugLevel>1 && $AltBody)
+		{
+			ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."(".__LINE__.") fetched AltBody as with MessageContentType:". $ContentType.'=>'.$AltBody);
+		}
 		//error_log(__METHOD__.__LINE__.array2string($mailObject));
 		// if this is a multipart message with a boundary, we must use the original body
 		//if ($this->debugLevel>2) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' mailObject after Inital Parse:'.array2string($mailObject));
-        if ($use_orgbody) {
+        if ($use_orgbody)
+        {
     	    ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."(".__LINE__.") use_orgbody = true ContentType:".$ContentType);
  			// if it is a ClientSideMeetingRequest, we report it as send at all times
 			if (($cal_body = $mailObject->findBody('calendar')) &&
@@ -535,15 +636,20 @@ 
 				{
 					$organizer = calendar_ical::getIcalOrganizer($cal_body->getContents());
 				}
-				if ($this->debugLevel) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."(".__LINE__.") we have a Client Side Meeting Request from organizer=$organizer");
+				if ($this->debugLevel)
+				{
+					ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."(".__LINE__.") we have a Client Side Meeting Request from organizer=$organizer");
+				}
 				$ClientSideMeetingRequest = true;
 			}
         }
 		// now handle the addressee list
 		$toCount = 0;
 		//error_log(__METHOD__.__LINE__.array2string($toMailAddr));
-		foreach((array)$toMailAddr as $address) {
-			foreach(Mail::parseAddressList((get_magic_quotes_gpc()?stripslashes($address):$address)) as $addressObject) {
+		foreach((array)$toMailAddr as $address)
+		{
+			foreach(Mail::parseAddressList((get_magic_quotes_gpc()?stripslashes($address):$address)) as $addressObject)
+			{
 				$emailAddress = $addressObject->mailbox. ($addressObject->host ? '@'.$addressObject->host : '');
 				if ($ClientSideMeetingRequest === true && $allowSendingInvitations == 'sendifnocalnotif' &&
 					calendar_boupdate::email_update_requested($emailAddress, isset($cSMRMethod) ? $cSMRMethod : 'REQUEST',
@@ -557,19 +663,29 @@ 
 			}
 		}
 		$ccCount = 0;
-		foreach((array)$ccMailAddr as $address) {
-			foreach(Mail::parseAddressList((get_magic_quotes_gpc()?stripslashes($address):$address)) as $addressObject) {
+		foreach((array)$ccMailAddr as $address)
+		{
+			foreach(Mail::parseAddressList((get_magic_quotes_gpc()?stripslashes($address):$address)) as $addressObject)
+			{
 				$emailAddress = $addressObject->mailbox. ($addressObject->host ? '@'.$addressObject->host : '');
-				if ($ClientSideMeetingRequest === true && $allowSendingInvitations == 'sendifnocalnotif' && calendar_boupdate::email_update_requested($emailAddress)) continue;
+				if ($ClientSideMeetingRequest === true && $allowSendingInvitations == 'sendifnocalnotif' && calendar_boupdate::email_update_requested($emailAddress))
+				{
+					continue;
+				}
 				$mailObject->AddCC($emailAddress, $addressObject->personal);
 				$ccCount++;
 			}
 		}
 		$bccCount = 0;
-		foreach((array)$bccMailAddr as $address) {
-			foreach(Mail::parseAddressList((get_magic_quotes_gpc()?stripslashes($address):$address)) as $addressObject) {
+		foreach((array)$bccMailAddr as $address)
+		{
+			foreach(Mail::parseAddressList((get_magic_quotes_gpc()?stripslashes($address):$address)) as $addressObject)
+			{
 				$emailAddress = $addressObject->mailbox. ($addressObject->host ? '@'.$addressObject->host : '');
-				if ($ClientSideMeetingRequest === true && $allowSendingInvitations == 'sendifnocalnotif' && calendar_boupdate::email_update_requested($emailAddress)) continue;
+				if ($ClientSideMeetingRequest === true && $allowSendingInvitations == 'sendifnocalnotif' && calendar_boupdate::email_update_requested($emailAddress))
+				{
+					continue;
+				}
 				$mailObject->AddBCC($emailAddress, $addressObject->personal);
 				$bccCount++;
 			}
@@ -579,7 +695,10 @@ 
 		{
 			return $ClientSideMeetingRequest && $allowSendingInvitations === 'sendifnocalnotif' && $organizer ? true : 0; // noone to send mail to
 		}
-		if ($ClientSideMeetingRequest === true && $allowSendingInvitations===false) return true;
+		if ($ClientSideMeetingRequest === true && $allowSendingInvitations===false)
+		{
+			return true;
+		}
 		// as we use our mailer (horde mailer) it is detecting / setting the mimetype by itself while creating the mail
 /*
 		if ($this->debugLevel>2) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' retrieved Body:'.$body);
@@ -602,10 +721,17 @@ 
 			$this->mail->reopen($folder);
 			$bodyStruct = $this->mail->getMessageBody($uid, 'html_only');
 			$bodyBUFFHtml = $this->mail->getdisplayableBody($this->mail,$bodyStruct,true,false);
-			if ($this->debugLevel>3) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' html_only:'.$bodyBUFFHtml);
-		    if ($bodyBUFFHtml != "" && (is_array($bodyStruct) && $bodyStruct[0]['mimeType']=='text/html')) {
+			if ($this->debugLevel>3)
+			{
+				ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' html_only:'.$bodyBUFFHtml);
+			}
+		    if ($bodyBUFFHtml != "" && (is_array($bodyStruct) && $bodyStruct[0]['mimeType']=='text/html'))
+		    {
 				// may be html
-				if ($this->debugLevel>0) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."(".__LINE__.") MIME Body".' Type:html (fetched with html_only):'.$bodyBUFFHtml);
+				if ($this->debugLevel>0)
+				{
+					ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."(".__LINE__.") MIME Body".' Type:html (fetched with html_only):'.$bodyBUFFHtml);
+				}
 				$AltBody = $AltBody."</br>".$bodyBUFFHtml.$sigTextHtml;
 				$isreply = true;
 			}
@@ -614,8 +740,12 @@ 
 			// if the new part of the message is html, we must preserve it, and handle that the original mail is text/plain
 			$bodyStruct = $this->mail->getMessageBody($uid,'never_display');//'never_display');
 			$bodyBUFF = $this->mail->getdisplayableBody($this->mail,$bodyStruct,false,false);
-			if ($bodyBUFF != "" && (is_array($bodyStruct) && $bodyStruct[0]['mimeType']=='text/plain')) {
-				if ($this->debugLevel>0) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."(".__LINE__.") MIME Body".' Type:plain (fetched with never_display):'.$bodyBUFF);
+			if ($bodyBUFF != "" && (is_array($bodyStruct) && $bodyStruct[0]['mimeType']=='text/plain'))
+			{
+				if ($this->debugLevel>0)
+				{
+					ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."(".__LINE__.") MIME Body".' Type:plain (fetched with never_display):'.$bodyBUFF);
+				}
 				$Body = $Body."\r\n".$bodyBUFF.$sigTextPlain;
 				$isreply = true;
 			}
@@ -623,7 +753,10 @@ 
 			{
 				$isreply = true;
 				$AltBody = $AltBody."</br><pre>".nl2br($bodyBUFF).'</pre>'.$sigTextHtml;
-				if ($this->debugLevel>0) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__." no Api\Html Body found use modified plaintext body for txt/html: ".$AltBody);
+				if ($this->debugLevel>0)
+				{
+					ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__." no Api\Html Body found use modified plaintext body for txt/html: ".$AltBody);
+				}
 			}
 		}
 
@@ -637,11 +770,17 @@ 
 			(isset($smartdata->replacemime) && $smartdata->replacemime == false)))
 		{
 			//force the default for the forwarding -> asmail
-			if (is_array($preferencesArray)) {
+			if (is_array($preferencesArray))
+			{
 				if (!array_key_exists('message_forwarding',$preferencesArray)
 					|| !isset($preferencesArray['message_forwarding'])
-					|| empty($preferencesArray['message_forwarding'])) $preferencesArray['message_forwarding'] = 'asmail';
-			} else {
+					|| empty($preferencesArray['message_forwarding']))
+				{
+					$preferencesArray['message_forwarding'] = 'asmail';
+				}
+			}
+			else
+			{
 				$preferencesArray['message_forwarding'] = 'asmail';
 			}
 			// construct the uid of the message out of the itemid - seems to be the uid, no construction needed
@@ -673,27 +812,44 @@ 
 				$this->mail->reopen($folder);
 				$bodyStruct = $this->mail->getMessageBody($uid, 'html_only');
 				$bodyBUFFHtml = $this->mail->getdisplayableBody($this->mail,$bodyStruct,true,false);
-				if ($this->debugLevel>0) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' html_only:'.$bodyBUFFHtml);
-				if ($bodyBUFFHtml != "" && (is_array($bodyStruct) && $bodyStruct[0]['mimeType']=='text/html')) {
+				if ($this->debugLevel>0)
+				{
+					ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' html_only:'.$bodyBUFFHtml);
+				}
+				if ($bodyBUFFHtml != "" && (is_array($bodyStruct) && $bodyStruct[0]['mimeType']=='text/html'))
+				{
 					// may be html
-					if ($this->debugLevel>0) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."(".__LINE__.") MIME Body".' Type:html (fetched with html_only):'.$bodyBUFFHtml);
+					if ($this->debugLevel>0)
+					{
+						ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."(".__LINE__.") MIME Body".' Type:html (fetched with html_only):'.$bodyBUFFHtml);
+					}
 					$AltBody = $AltBody."</br>".$bodyBUFFHtml.$sigTextHtml;
 					$isforward = true;
 				}
 				// plain text Message part
-				if ($this->debugLevel>0) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."(".__LINE__.") MIME Body".' Type:plain, fetch text:');
+				if ($this->debugLevel>0)
+				{
+					ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."(".__LINE__.") MIME Body".' Type:plain, fetch text:');
+				}
 				// if the new part of the message is html, we must preserve it, and handle that the original mail is text/plain
 				$bodyStruct = $this->mail->getMessageBody($uid,'never_display');//'never_display');
 				$bodyBUFF = $this->mail->getdisplayableBody($this->mail,$bodyStruct,false,false);
-				if ($bodyBUFF != "" && (is_array($bodyStruct) && $bodyStruct[0]['mimeType']=='text/plain')) {
-					if ($this->debugLevel>0) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."(".__LINE__.") MIME Body".' Type:plain (fetched with never_display):'.$bodyBUFF);
+				if ($bodyBUFF != "" && (is_array($bodyStruct) && $bodyStruct[0]['mimeType']=='text/plain'))
+				{
+					if ($this->debugLevel>0)
+					{
+						ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."(".__LINE__.") MIME Body".' Type:plain (fetched with never_display):'.$bodyBUFF);
+					}
 					$Body = $Body."\r\n".$bodyBUFF.$sigTextPlain;
 					$isforward = true;
 				}
 				if (!empty($bodyBUFF) && empty($bodyBUFFHtml) && !empty($AltBody))
 				{
 					$AltBody = $AltBody."</br><pre>".nl2br($bodyBUFF).'</pre>'.$sigTextHtml;
-					if ($this->debugLevel>0) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__." no html Body found use modified plaintext body for txt/html: ".$AltBody);
+					if ($this->debugLevel>0)
+					{
+						ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__." no html Body found use modified plaintext body for txt/html: ".$AltBody);
+					}
 					$isforward = true;
 				}
 				// get all the attachments and add them too.
@@ -706,7 +862,10 @@ 
 					ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' gather Attachments for BodyCreation of/for MessageID:'.$uid.' found:'.count($attachments));
 					foreach((array)$attachments as $key => $attachment)
 					{
-						if ($this->debugLevel>0) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' Key:'.$key.'->'.array2string($attachment));
+						if ($this->debugLevel>0)
+						{
+							ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' Key:'.$key.'->'.array2string($attachment));
+						}
 						$attachmentNames .= $attachment['name']."\n";
 						$attachmentData	= $this->mail->getAttachment($uid, $attachment['partID'],0,false,false,$folder);
 						/*$x =*/ $mailObject->AddStringAttachment($attachmentData['attachment'], $attachment['name'], $attachment['mimeType']);
@@ -725,20 +884,29 @@ 
 		// now set the body
 		if ($AltBody && ($html_body = $mailObject->findBody('html')))
 		{
-			if ($this->debugLevel>1) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' -> '.$AltBody);
+			if ($this->debugLevel>1)
+			{
+				ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' -> '.$AltBody);
+			}
 			//error_log(__METHOD__.__LINE__.' html:'.$AltBody);
 			$html_body->setContents($AltBody,array('encoding'=>Horde_Mime_Part::DEFAULT_ENCODING));
 		}
 		if ($Body && ($text_body = $mailObject->findBody('plain')))
 		{
-			if ($this->debugLevel>1) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' -> '.$Body);
+			if ($this->debugLevel>1)
+			{
+				ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' -> '.$Body);
+			}
 			//error_log(__METHOD__.__LINE__.' text:'.$Body);
 			$text_body->setContents($Body,array('encoding'=>Horde_Mime_Part::DEFAULT_ENCODING));
 		}
 		//advanced debugging
 		// Horde SMTP Class uses utf-8 by default.
         //ZLog::Write(LOGLEVEL_DEBUG, "IMAP-SendMail: parsed message: ". print_r($message,1));
-		if ($this->debugLevel>2) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."(".__LINE__."): MailObject:".array2string($mailObject));
+		if ($this->debugLevel>2)
+		{
+			ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."(".__LINE__."): MailObject:".array2string($mailObject));
+		}
 
 		// set a higher timeout for big messages
 		@set_time_limit(120);
@@ -765,7 +933,9 @@ 
 			if ($this->mail->isDraftFolder($folder) && !$this->mail->isTemplateFolder($folder))
 			{
 				$this->mail->deleteMessages(array($uid),$folder);
-			} else {
+			}
+			else
+			{
 				$this->mail->flagMessages("answered", array($uid),$folder);
 				if ($smartdata_task== "forward")
 				{
@@ -779,7 +949,8 @@ 
 		{
 			$asf = false;
 			$sentFolder = $this->mail->getSentFolder();
-			if ($this->_sentID) {
+			if ($this->_sentID)
+			{
 				$folderArray[] = $this->_sentID;
 			}
 			else if(isset($sentFolder) && $sentFolder != 'none')
@@ -793,28 +964,41 @@ 
 				// we dont try guessing
 				$asf = true;
 			}
-			if (count($folderArray) > 0) {
-				foreach((array)$bccMailAddr as $address) {
-					foreach(Mail::parseAddressList((get_magic_quotes_gpc()?stripslashes($address):$address)) as $addressObject) {
+			if (count($folderArray) > 0)
+			{
+				foreach((array)$bccMailAddr as $address)
+				{
+					foreach(Mail::parseAddressList((get_magic_quotes_gpc()?stripslashes($address):$address)) as $addressObject)
+					{
 						$emailAddress = $addressObject->mailbox. ($addressObject->host ? '@'.$addressObject->host : '');
 						$mailAddr[] = array($emailAddress, $addressObject->personal);
 					}
 				}
 				//$BCCmail='';
-				if (count($mailAddr)>0) $mailObject->forceBccHeader();
+				if (count($mailAddr)>0)
+				{
+					$mailObject->forceBccHeader();
+				}
 				//$BCCmail = $mailObject->AddrAppend("Bcc",$mailAddr);
-				foreach($folderArray as $folderName) {
-					if($this->mail->isSentFolder($folderName)) {
+				foreach($folderArray as $folderName)
+				{
+					if($this->mail->isSentFolder($folderName))
+					{
 						$flags = '\\Seen';
-					} elseif($this->mail->isDraftFolder($folderName)) {
+					}
+					elseif($this->mail->isDraftFolder($folderName))
+					{
 						$flags = '\\Draft';
-					} else {
+					}
+					else
+					{
 						$flags = '';
 					}
 					$asf = true;
 					//ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.'->'.array2string($this->mail->icServer));
 					$this->mail->openConnection(self::$profileID,false);
-					if ($this->mail->folderExists($folderName)) {
+					if ($this->mail->folderExists($folderName))
+					{
 						try
 						{
 							$this->mail->appendMessage($folderName,$mailObject->getRaw(), null,
@@ -872,17 +1056,26 @@ 
 		$bodypreference = $contentparameters->GetBodyPreference(); /* fmbiete's contribution r1528, ZP-320 */
 
 		// fix for z-push bug returning additional bodypreference type 4, even if only 1 is requested and mimessupport = 0
-		if (!$mimesupport && ($key = array_search('4', $bodypreference))) unset($bodypreference[$key]);
+		if (!$mimesupport && ($key = array_search('4', $bodypreference)))
+		{
+			unset($bodypreference[$key]);
+		}
 
 		//$this->debugLevel=4;
-		if (!isset($this->mail)) $this->mail = Mail::getInstance(false,self::$profileID,true,false,true);
+		if (!isset($this->mail))
+		{
+			$this->mail = Mail::getInstance(false,self::$profileID,true,false,true);
+		}
 		ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' FolderID:'.$folderid.' ID:'.$id.' TruncSize:'.$truncsize.' Bodypreference: '.array2string($bodypreference));
 		$account = $_folderName = $xid = null;
 		$this->splitID($folderid,$account,$_folderName,$xid);
 		$this->mail->reopen($_folderName);
 		$messages = $this->fetchMessages($folderid, NULL, $id, true);	// true: return all headers
 		$headers = $messages[$id];
-		if ($this->debugLevel>3) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.array2string($headers));
+		if ($this->debugLevel>3)
+		{
+			ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.array2string($headers));
+		}
 		// StatMessage should reopen the folder in question, so we dont need folderids in the following statements.
 		if ($headers)
 		{
@@ -892,16 +1085,25 @@ 
 			//$rawHeaders = $this->mail->getMessageRawHeader($id);
 			// simple style
 			// start AS12 Stuff (bodypreference === false) case = old behaviour
-			if ($this->debugLevel>0) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__. ' for message with ID:'.$id.' with headers:'.array2string($headers));
+			if ($this->debugLevel>0)
+			{
+				ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__. ' for message with ID:'.$id.' with headers:'.array2string($headers));
+			}
 
-			if ($bodypreference === false) {
+			if ($bodypreference === false)
+			{
 				$bodyStruct = $this->mail->getMessageBody($id, 'only_if_no_text', '', null, true,$_folderName);
 				$raw_body = $this->mail->getdisplayableBody($this->mail,$bodyStruct,false,false);
 				//$body = html_entity_decode($body,ENT_QUOTES,$this->mail->detect_encoding($body));
-				if (stripos($raw_body,'<style')!==false) $body = preg_replace("/<style.*?<\/style>/is", "", $raw_body); // in case there is only a html part
+				if (stripos($raw_body,'<style')!==false)
+				{
+					$body = preg_replace("/<style.*?<\/style>/is", "", $raw_body);
+				}
+				// in case there is only a html part
 				// remove all other html
 				$body = strip_tags($raw_body);
-				if(strlen($body) > $truncsize) {
+				if(strlen($body) > $truncsize)
+				{
 					$body = Utils::Utf8_truncate($body, $truncsize);
 					$output->bodytruncated = 1;
 				}
@@ -916,7 +1118,8 @@ 
 			{
 				//Select body type preference
 				$bpReturnType = 1;//SYNC_BODYPREFERENCE_PLAIN;
-				if ($bodypreference !== false) {
+				if ($bodypreference !== false)
+				{
 					// bodypreference can occur multiple times
 					// usually we would use Utils::GetBodyPreferenceBestMatch($bodypreference);
 					$bpReturnType = Utils::GetBodyPreferenceBestMatch($bodypreference);
@@ -950,25 +1153,49 @@ 
 					// fetch the body (try to gather data only once)
 					$css ='';
 					$bodyStruct = $this->mail->getMessageBody($id, 'html_only', '', null, true,$_folderName);
-					if ($this->debugLevel>2) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' html_only Struct:'.array2string($bodyStruct));
+					if ($this->debugLevel>2)
+					{
+						ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' html_only Struct:'.array2string($bodyStruct));
+					}
 					$body = $this->mail->getdisplayableBody($this->mail,$bodyStruct,true,false);
-					if ($this->debugLevel>3) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' html_only:'.$body);
-					if ($body != "" && (is_array($bodyStruct) && $bodyStruct[0]['mimeType']=='text/html')) {
+					if ($this->debugLevel>3)
+					{
+						ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' html_only:'.$body);
+					}
+					if ($body != "" && (is_array($bodyStruct) && $bodyStruct[0]['mimeType']=='text/html'))
+					{
 						// may be html
-						if ($this->debugLevel>0) ZLog::Write(LOGLEVEL_DEBUG, "MIME Body".' Type:html (fetched with html_only)');
+						if ($this->debugLevel>0)
+						{
+							ZLog::Write(LOGLEVEL_DEBUG, "MIME Body".' Type:html (fetched with html_only)');
+						}
 						$css = $this->mail->getStyles($bodyStruct);
 						$output->nativebodytype=2;
-					} else {
+					}
+					else
+					{
 						// plain text Message
-						if ($this->debugLevel>0) ZLog::Write(LOGLEVEL_DEBUG, "MIME Body".' Type:plain, fetch text (HTML, if no text available)');
+						if ($this->debugLevel>0)
+						{
+							ZLog::Write(LOGLEVEL_DEBUG, "MIME Body".' Type:plain, fetch text (HTML, if no text available)');
+						}
 						$output->nativebodytype=1;
 						$bodyStruct = $this->mail->getMessageBody($id,'never_display', '', null, true,$_folderName); //'only_if_no_text');
-						if ($this->debugLevel>3) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' plain text Struct:'.array2string($bodyStruct));
+						if ($this->debugLevel>3)
+						{
+							ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' plain text Struct:'.array2string($bodyStruct));
+						}
 						$body = $this->mail->getdisplayableBody($this->mail,$bodyStruct,false,false);
-						if ($this->debugLevel>3) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' never display html(plain text only):'.$body);
+						if ($this->debugLevel>3)
+						{
+							ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' never display html(plain text only):'.$body);
+						}
 					}
 					// whatever format decode (using the correct encoding)
-					if ($this->debugLevel>3) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__."MIME Body".' Type:'.($output->nativebodytype==2?' html ':' plain ').$body);
+					if ($this->debugLevel>3)
+					{
+						ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__."MIME Body".' Type:'.($output->nativebodytype==2?' html ':' plain ').$body);
+					}
 					//$body = html_entity_decode($body,ENT_QUOTES,$this->mail->detect_encoding($body));
 					// prepare plaintextbody
 					$plainBody='';
@@ -989,12 +1216,18 @@ 
 					// remove all other html
 					$plainBody = preg_replace("/<br.*>/is","\r\n",$plainBody);
 					$plainBody = strip_tags($plainBody);
-					if ($this->debugLevel>3 && $output->nativebodytype==1) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' Plain Text:'.$plainBody);
+					if ($this->debugLevel>3 && $output->nativebodytype==1)
+					{
+						ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' Plain Text:'.$plainBody);
+					}
 					//$body = str_replace("\n","\r\n", str_replace("\r","",$body)); // do we need that?
 
-					if ($bpReturnType==2) //SYNC_BODYPREFERENCE_HTML
+					if ($bpReturnType==2)
+					{
+						//SYNC_BODYPREFERENCE_HTML
 					{
 						if ($this->debugLevel>0) ZLog::Write(LOGLEVEL_DEBUG, "HTML Body with requested pref 2");
+					}
 						// Send HTML if requested and native type was html
 						$output->asbody->type = 2;
 						$htmlbody = '<html>'.
@@ -1006,7 +1239,10 @@ 
 							'<body>';
 						if ($output->nativebodytype==2)
 						{
-							if ($css) Api\Mail\Html::replaceTagsCompletley($body,'style');
+							if ($css)
+							{
+								Api\Mail\Html::replaceTagsCompletley($body,'style');
+							}
 							// as we fetch html, and body is HTML, we may not need to handle this
 							$htmlbody .= $body;
 						}
@@ -1031,7 +1267,10 @@ 
 					else
 					{
 						// Send Plaintext as Fallback or if original body is plainttext
-						if ($this->debugLevel>0) ZLog::Write(LOGLEVEL_DEBUG, "Plaintext Body:".$plainBody);
+						if ($this->debugLevel>0)
+						{
+							ZLog::Write(LOGLEVEL_DEBUG, "Plaintext Body:".$plainBody);
+						}
 						/* we use plainBody (set above) instead
 						$bodyStruct = $this->mail->getMessageBody($id,'only_if_no_text'); //'never_display');
 						$plain = $this->mail->getdisplayableBody($this->mail,$bodyStruct);
@@ -1068,7 +1307,9 @@ 
 			{
 				$output->flag->flagstatus = 2;
 				//$output->flag->flagtype = "Flag for Follow up";
-			} else {
+			}
+			else
+			{
 				$output->flag->flagstatus = 0;
 			}
 			if ($headers['answered'])
@@ -1084,10 +1325,20 @@ 
 				($headers['priority'] < 3 ? 2 : 1) ;
 			$output->datereceived = $this->mail->_strtotime($headers['date'],'ts',true);
 			$output->to = $headers['to_address'];
-			if ($headers['to']) $output->displayto = $headers['to_address']; //$headers['FETCHED_HEADER']['to_name']
+			if ($headers['to'])
+			{
+				$output->displayto = $headers['to_address'];
+			}
+			//$headers['FETCHED_HEADER']['to_name']
 			$output->from = $headers['sender_address'];
-			if (isset($headers['cc_addresses']) && $headers['cc_addresses']) $output->cc = $headers['cc_addresses'];
-			if (isset($headers['reply_to_address']) && $headers['reply_to_address']) $output->reply_to = $headers['reply_to_address'];
+			if (isset($headers['cc_addresses']) && $headers['cc_addresses'])
+			{
+				$output->cc = $headers['cc_addresses'];
+			}
+			if (isset($headers['reply_to_address']) && $headers['reply_to_address'])
+			{
+				$output->reply_to = $headers['reply_to_address'];
+			}
 
 			$output->messageclass = "IPM.Note";
 			if (stripos($headers['mimetype'],'multipart')!== false &&
@@ -1095,7 +1346,8 @@ 
 			{
 				$output->messageclass = "IPM.Note.SMIME.MultipartSigned";
 			}
-			if (Request::GetProtocolVersion() >= 12.0) {
+			if (Request::GetProtocolVersion() >= 12.0)
+			{
 				$output->contentclass = "urn:content-classes:message";
 			}
 
@@ -1108,7 +1360,10 @@ 
 				//error_log(__METHOD__.__LINE__.array2string($attachments));
 				foreach ($attachments as $key => $attach)
 				{
-					if ($this->debugLevel>0) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' Key:'.$key.'->'.array2string($attach));
+					if ($this->debugLevel>0)
+					{
+						ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' Key:'.$key.'->'.array2string($attach));
+					}
 
 					// pass meeting requests to calendar plugin
 					if (strtolower($attach['mimeType']) == 'text/calendar' && strtolower($attach['method']) == 'request' &&
@@ -1125,17 +1380,24 @@ 
 						unset($attachment);
 						continue;	// do NOT add attachment as attachment
 					}
-					if (Request::GetProtocolVersion() >= 12.0) {
+					if (Request::GetProtocolVersion() >= 12.0)
+					{
 						$attachment = new SyncBaseAttachment();
 						if (!isset($output->asattachments) || !is_array($output->asattachments))
-							$output->asattachments = array();
+						{
+													$output->asattachments = array();
+						}
 						$attachment->estimatedDataSize = $attach['size'];
 						$attachment->method = 1;
 						$attachment->filereference = $folderid . ":" . $id . ":" . $attach['partID'];
-					} else {
+					}
+					else
+					{
 						$attachment = new SyncAttachment();
 						if (!isset($output->attachments) || !is_array($output->attachments))
-							$output->attachments = array();
+						{
+													$output->attachments = array();
+						}
 						$attachment->attsize = $attach['size'];
 						$attachment->attmethod = 1;
 						$attachment->attname = $folderid . ":" . $id . ":" . $attach['partID'];//$key;
@@ -1152,10 +1414,13 @@ 
 						{
 							$attachment->isinline = true;
 						}
-						if (Request::GetProtocolVersion() >= 12.0) {
+						if (Request::GetProtocolVersion() >= 12.0)
+						{
 							$attachment->method=1;
 							$attachment->contentid= str_replace(array("<",">"), "",$attach['cid']);
-						} else {
+						}
+						else
+						{
 							$attachment->attmethod=6;
 							$attachment->attoid = str_replace(array("<",">"), "",$attach['cid']);
 						}
@@ -1163,9 +1428,12 @@ 
 						$attachment->contenttype = trim($attach['mimeType']);
 						//	ZLog::Write(LOGLEVEL_DEBUG, "'".$part->headers['content-type']."'  ".$attachment->contentid);
 					}
-					if (Request::GetProtocolVersion() >= 12.0) {
+					if (Request::GetProtocolVersion() >= 12.0)
+					{
 						array_push($output->asattachments, $attachment);
-					} else {
+					}
+					else
+					{
 						array_push($output->attachments, $attachment);
 					}
 					unset($attachment);
@@ -1178,8 +1446,11 @@ 
             // Language Code Page ID: http://msdn.microsoft.com/en-us/library/windows/desktop/dd317756%28v=vs.85%29.aspx
             $output->internetcpid = INTERNET_CPID_UTF8;
 
-			if ($this->debugLevel>3) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.array2string($output));
-//$this->debugLevel=0;
+			if ($this->debugLevel>3)
+			{
+				ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.array2string($output));
+			}
+			//$this->debugLevel=0;
 			return $output;
 		}
 //$this->debugLevel=0;
@@ -1238,7 +1509,8 @@ 
      * @param string $attname - should contain (folder)id
 	 * @return SyncItemOperationsAttachment-object
 	 */
-	function GetAttachmentData($fid,$attname) {
+	function GetAttachmentData($fid,$attname)
+	{
 		ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.": $fid (attname: '$attname')");
 		return $this->_GetAttachmentData($fid,$attname);
 	}
@@ -1253,7 +1525,8 @@ 
      * @param string $attname - should contain (folder)id
 	 * @return SyncItemOperationsAttachment-object
 	 */
-	function ItemOperationsGetAttachmentData($fid,$attname) {
+	function ItemOperationsGetAttachmentData($fid,$attname)
+	{
 		ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.": $fid (attname: '$attname')");
 		return $this->_GetAttachmentData($fid,$attname);
 	}
@@ -1275,7 +1548,10 @@ 
 
 		$this->splitID($folderid, $account, $folder);
 
-		if (!isset($this->mail)) $this->mail = Mail::getInstance(false,self::$profileID,true,false,true);
+		if (!isset($this->mail))
+		{
+			$this->mail = Mail::getInstance(false,self::$profileID,true,false,true);
+		}
 
 		$this->mail->reopen($folder);
 		$attachment = $this->mail->getAttachment($id,$part,0,false,true,$folder);
@@ -1284,7 +1560,9 @@ 
 		$SIOattachment->data = $attachment['attachment'];
 		//ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.": $fid (attname: '$attname') Data:".$attachment['attachment']);
 		if (isset($attachment['type']) )
-			$SIOattachment->contenttype = $attachment['type'];
+		{
+					$SIOattachment->contenttype = $attachment['type'];
+		}
 
 		unset($attachment);
 
@@ -1330,11 +1608,15 @@ 
 		//unset($folderid, $id, $message, $contentParameters);
 		$account = $folder = null;
 		$this->splitID($folderid, $account, $folder);
-		if (isset($message->flag)) {
-			if (isset($message->flag->flagstatus) && $message->flag->flagstatus == 2) {
+		if (isset($message->flag))
+		{
+			if (isset($message->flag->flagstatus) && $message->flag->flagstatus == 2)
+			{
 				$rv = $this->mail->flagMessages((($message->flag->flagstatus == 2) ? "flagged" : "unflagged"), $id,$folder);
 				ZLog::Write(LOGLEVEL_DEBUG,__METHOD__." -> set ".array2string($id).' in Folder '.$folder." as " . (($message->flag->flagstatus == 2) ? "flagged" : "unflagged") . "-->". $rv);
-			} else {
+			}
+			else
+			{
 				$rv = $this->mail->flagMessages("unflagged", $id,$folder);
 				ZLog::Write(LOGLEVEL_DEBUG,__METHOD__." -> set ".array2string($id).' in Folder '.$folder." as " . "unflagged" . "-->". $rv);
 			}
@@ -1364,7 +1646,10 @@ 
 		$this->splitID($folderid, $account, $srcFolder);
 		$this->splitID($newfolderid, $account, $destFolder);
 		ZLog::Write(LOGLEVEL_DEBUG, "IMAP-MoveMessage: (SourceFolder: '$srcFolder'  id: '$id'  DestFolder: '$destFolder' )");
-		if (!isset($this->mail)) $this->mail = Mail::getInstance(false,self::$profileID,true,false,true);
+		if (!isset($this->mail))
+		{
+			$this->mail = Mail::getInstance(false,self::$profileID,true,false,true);
+		}
 		$this->mail->reopen($destFolder);
 		$status = $this->mail->getFolderStatus($destFolder);
 		$uidNext = $status['uidnext'];
@@ -1396,7 +1681,10 @@ 
 				$maximumSyncRangeInDays = $GLOBALS['egw_info']['user']['preferences']['activesync']['mail-maximumSyncRange'];
 			}
 			$cutoffdate = (is_numeric($maximumSyncRangeInDays) ? Api\DateTime::to('now','ts')-(3600*24*$maximumSyncRangeInDays):null);
-			if (is_numeric($maximumSyncRangeInDays)) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.' Client set no truncationdate. Using '.$maximumSyncRangeInDays.' days.'.date("d-M-Y", $cutoffdate));
+			if (is_numeric($maximumSyncRangeInDays))
+			{
+				ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.' Client set no truncationdate. Using '.$maximumSyncRangeInDays.' days.'.date("d-M-Y", $cutoffdate));
+			}
 		}
 		return $this->fetchMessages($folderid, $cutoffdate);
 	}
@@ -1418,7 +1706,10 @@ 
 	{
 		static $headers = array();
 
-		if ($this->debugLevel>1) $gstarttime = microtime (true);
+		if ($this->debugLevel>1)
+		{
+			$gstarttime = microtime (true);
+		}
 		//ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__);
 		$rv_messages = array();
 		// if the message is still available within the class, we use it instead of fetching it again
@@ -1431,7 +1722,10 @@ 
 		{
 			$headers = array();	// clear cache to not use too much memory
 
-			if ($this->debugLevel>1) $starttime = microtime (true);
+			if ($this->debugLevel>1)
+			{
+				$starttime = microtime (true);
+			}
 			$this->_connect($this->account);
 			if ($this->debugLevel>1)
 			{
@@ -1440,8 +1734,14 @@ 
 			}
 			$messagelist = $_filter = array();
 			// if not connected, any further action must fail
-			if (!empty($cutoffdate)) $_filter = array('status'=>array('UNDELETED'),'range'=>"SINCE",'date'=> date("d-M-Y", $cutoffdate));
-			if ($this->debugLevel>1) $starttime = microtime (true);
+			if (!empty($cutoffdate))
+			{
+				$_filter = array('status'=>array('UNDELETED'),'range'=>"SINCE",'date'=> date("d-M-Y", $cutoffdate));
+			}
+			if ($this->debugLevel>1)
+			{
+				$starttime = microtime (true);
+			}
 			$account = $_folderName = $id = null;
 			$this->splitID($folderid,$account,$_folderName,$id);
 			if ($this->debugLevel>1)
@@ -1449,8 +1749,14 @@ 
 				$endtime = microtime(true) - $starttime;
 				ZLog::Write(LOGLEVEL_DEBUG,__METHOD__. " splitID took : ".$endtime.' for FolderID:'.$folderid);
 			}
-			if ($this->debugLevel>1) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.' for Folder:'.$_folderName.' Filter:'.array2string($_filter).' Ids:'.array2string($_id).'/'.$id);
-			if ($this->debugLevel>1) $starttime = microtime (true);
+			if ($this->debugLevel>1)
+			{
+				ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.' for Folder:'.$_folderName.' Filter:'.array2string($_filter).' Ids:'.array2string($_id).'/'.$id);
+			}
+			if ($this->debugLevel>1)
+			{
+				$starttime = microtime (true);
+			}
 			$_numberOfMessages = (empty($cutoffdate)?250:99999);
 			$rv_messages = $this->mail->getHeaders($_folderName, $_startMessage=1, $_numberOfMessages, $_sort=0, $_reverse=false, $_filter, $_id);
 			if ($this->debugLevel>1)
@@ -1459,27 +1765,56 @@ 
 				ZLog::Write(LOGLEVEL_DEBUG,__METHOD__. " getHeaders call took : ".$endtime.' for FolderID:'.$_folderName);
 			}
 		}
-		if ($_id == NULL && $this->debugLevel>1)  ZLog::Write(LOGLEVEL_DEBUG,__METHOD__." found :". count($rv_messages['header']));
+		if ($_id == NULL && $this->debugLevel>1)
+		{
+			ZLog::Write(LOGLEVEL_DEBUG,__METHOD__." found :". count($rv_messages['header']));
+		}
 		//ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' Result:'.array2string($rv_messages));
 		$messagelist = array();
-		if (!isset($rv_messages['header'])||empty($rv_messages['header'])) return $messagelist;
+		if (!isset($rv_messages['header'])||empty($rv_messages['header']))
+		{
+			return $messagelist;
+		}
 		//if ($_returnModHash) $messageFolderHash = array();
 		foreach ((array)$rv_messages['header'] as $k => $vars)
 		{
-			if ($this->debugLevel>3) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' ID to process:'.$vars['uid'].' Subject:'.$vars['subject']);
+			if ($this->debugLevel>3)
+			{
+				ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' ID to process:'.$vars['uid'].' Subject:'.$vars['subject']);
+			}
 			$headers[$vars['uid']] = $vars;
-			if ($this->debugLevel>3) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' MailID:'.$k.'->'.array2string($vars));
-			if (!empty($vars['deleted'])) continue; // cut of deleted messages
-			if ($cutoffdate && $vars['date'] < $cutoffdate) continue; // message is out of range for cutoffdate, ignore it
-			if ($this->debugLevel>0) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' ID to report:'.$vars['uid'].' Subject:'.$vars['subject']);
+			if ($this->debugLevel>3)
+			{
+				ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' MailID:'.$k.'->'.array2string($vars));
+			}
+			if (!empty($vars['deleted']))
+			{
+				continue;
+			}
+			// cut of deleted messages
+			if ($cutoffdate && $vars['date'] < $cutoffdate)
+			{
+				continue;
+			}
+			// message is out of range for cutoffdate, ignore it
+			if ($this->debugLevel>0)
+			{
+				ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.' ID to report:'.$vars['uid'].' Subject:'.$vars['subject']);
+			}
 			$mess = $return_all_headers ? $vars : array();
 			$mess["mod"] = self::doFlagsMod($vars).$vars['date'];
 			$mess["id"] = $vars['uid'];
 			// 'seen' aka 'read' is the only flag we want to know about
 			$mess["flags"] = 0;
 			// outlook supports additional flags, set them to 0
-			if($vars["seen"]) $mess["flags"] = 1;
-			if ($this->debugLevel>3) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.array2string($mess));
+			if($vars["seen"])
+			{
+				$mess["flags"] = 1;
+			}
+			if ($this->debugLevel>3)
+			{
+				ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.array2string($mess));
+			}
 			$messagelist[$vars['uid']] = $mess;
 			unset($mess);
 		}
@@ -1504,9 +1839,18 @@ 
 	static function doFlagsMod($headerFlags)
 	{
 		$flags = 'nnn';
-		if ($headerFlags['flagged']) $flags[0] = 'f';
-		if ($headerFlags['answered']) $flags[1] = 'a';
-		if ($headerFlags['forwarded']) $flags[2] = 'f';
+		if ($headerFlags['flagged'])
+		{
+			$flags[0] = 'f';
+		}
+		if ($headerFlags['answered'])
+		{
+			$flags[1] = 'a';
+		}
+		if ($headerFlags['forwarded'])
+		{
+			$flags[2] = 'f';
+		}
 		//ZLog::Write(LOGLEVEL_DEBUG, __METHOD__.'('.array2string($headerFlags).') returning '.array2string($flags));
 		return $flags;
 	}
@@ -1538,31 +1882,55 @@ 
 	{
 		//$this->debugLevel=1;
 		$searchquery=$_searchquery->GetDataArray();
-		if (!is_array($searchquery)) return array();
-		if ($this->debugLevel>0) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.array2string($searchquery));
+		if (!is_array($searchquery))
+		{
+			return array();
+		}
+		if ($this->debugLevel>0)
+		{
+			ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.array2string($searchquery));
+		}
 
-		if (isset($searchquery['searchrebuildresults'])) {
+		if (isset($searchquery['searchrebuildresults']))
+		{
 			$rebuildresults = $searchquery['searchrebuildresults'];
-		} else {
+		}
+		else
+		{
 			$rebuildresults = false;
 		}
-		if ($this->debugLevel>0) ZLog::Write(LOGLEVEL_DEBUG,  'RebuildResults ['.$rebuildresults.']' );
+		if ($this->debugLevel>0)
+		{
+			ZLog::Write(LOGLEVEL_DEBUG,  'RebuildResults ['.$rebuildresults.']' );
+		}
 
-		if (isset($searchquery['deeptraversal'])) {
+		if (isset($searchquery['deeptraversal']))
+		{
 			$deeptraversal = $searchquery['deeptraversal'];
-		} else {
+		}
+		else
+		{
 			$deeptraversal = false;
 		}
-		if ($this->debugLevel>0) ZLog::Write(LOGLEVEL_DEBUG,  'DeepTraversal ['.$deeptraversal.']' );
+		if ($this->debugLevel>0)
+		{
+			ZLog::Write(LOGLEVEL_DEBUG,  'DeepTraversal ['.$deeptraversal.']' );
+		}
 
-		if (isset($searchquery['searchrange'])) {
+		if (isset($searchquery['searchrange']))
+		{
 			$range = explode("-",$_searchquery->GetSearchRange());
 			$start =$range[0] + 1;
 			$limit = $range[1] - $range[0] + 1;
-		} else {
+		}
+		else
+		{
 			$range = false;
 		}
-		if ($this->debugLevel>0) ZLog::Write(LOGLEVEL_DEBUG,  'Range ['.print_r($range, true).']' );
+		if ($this->debugLevel>0)
+		{
+			ZLog::Write(LOGLEVEL_DEBUG,  'Range ['.print_r($range, true).']' );
+		}
 
 		//foreach($searchquery['query'] as $k => $value) {
 		//	$query = $value;
@@ -1614,7 +1982,10 @@ 
 			$_filter['since'] = date("d-M-Y", Api\DateTime::to($sincedate,'ts'));
 		}
 		//$_filter[] = array('type'=>"SINCE",'string'=> date("d-M-Y", $cutoffdate));
-		if ($this->debugLevel>1) ZLog::Write(LOGLEVEL_DEBUG, __METHOD__.' for Folder:'.$_folderName.' Filter:'.array2string($_filter));
+		if ($this->debugLevel>1)
+		{
+			ZLog::Write(LOGLEVEL_DEBUG, __METHOD__.' for Folder:'.$_folderName.' Filter:'.array2string($_filter));
+		}
 		$rv_messages = $this->mail->getHeaders($_folderName, $_startMessage=($range?$start:1), $_numberOfMessages=($limit?$limit:9999999), $_sort=0, $_reverse=false, $_filter, $_id=NULL);
 		//ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__.array2string($rv_messages));
 		$list=array();
@@ -1646,17 +2017,26 @@ 
 	private function getParentID($account,$folder)
 	{
 		$this->_connect($account);
-		if (!isset($this->folders)) $this->folders = $this->mail->getFolderObjects(true,false);
+		if (!isset($this->folders))
+		{
+			$this->folders = $this->mail->getFolderObjects(true,false);
+		}
 
 		$mailFolder = $this->folders[$folder];
-		if (!isset($mailFolder)) return false;
+		if (!isset($mailFolder))
+		{
+			return false;
+		}
 		$delimiter = (isset($mailFolder->delimiter)?$mailFolder->delimiter:$this->mail->getHierarchyDelimiter());
 		$parent = explode($delimiter,$folder);
 		array_pop($parent);
 		$parent = implode($delimiter,$parent);
 
 		$id = $parent && $this->folders[$parent] ? $this->createID($account, $parent) : '0';
-		if ($this->debugLevel>1) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."('$folder') --> parent=$parent --> $id");
+		if ($this->debugLevel>1)
+		{
+			ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."('$folder') --> parent=$parent --> $id");
+		}
 		return $id;
 	}
 
@@ -1670,7 +2050,10 @@ 
 	{
 		static $last_id = null;
 		static $folderObj = null;
-		if (isset($last_id) && $last_id === $id) return $folderObj;
+		if (isset($last_id) && $last_id === $id)
+		{
+			return $folderObj;
+		}
 
 		try {
 			$account = $folder = null;
@@ -1681,18 +2064,30 @@ 
 			return $folderObj=false;
 		}
 		$this->_connect($account);
-		if (!isset($this->folders)) $this->folders = $this->mail->getFolderObjects(true,false);
+		if (!isset($this->folders))
+		{
+			$this->folders = $this->mail->getFolderObjects(true,false);
+		}
 
 		$mailFolder = $this->folders[$folder];
-		if (!isset($mailFolder)) return $folderObj=false;
+		if (!isset($mailFolder))
+		{
+			return $folderObj=false;
+		}
 
 		$folderObj = new SyncFolder();
 		$folderObj->serverid = $id;
 		$folderObj->parentid = $this->getParentID($account,$folder);
 		$folderObj->displayname = $mailFolder->shortDisplayName;
-		if ($this->debugLevel>1) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__." ID: $id, Account:$account, Folder:$folder");
+		if ($this->debugLevel>1)
+		{
+			ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__." ID: $id, Account:$account, Folder:$folder");
+		}
 		// get folder-type
-		foreach($this->folders as $inbox => $mailFolder) break;
+		foreach($this->folders as $inbox => $mailFolder)
+		{
+			break;
+		}
 		if ($folder == $inbox)
 		{
 			$folderObj->type = SYNC_FOLDER_TYPE_INBOX;
@@ -1729,7 +2124,10 @@ 
 			$folderObj->type = SYNC_FOLDER_TYPE_USER_MAIL;
 		}
 
-		if ($this->debugLevel>1) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."($id) --> $folder --> type=$folderObj->type, parentID=$folderObj->parentid, displayname=$folderObj->displayname");
+		if ($this->debugLevel>1)
+		{
+			ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."($id) --> $folder --> type=$folderObj->type, parentID=$folderObj->parentid, displayname=$folderObj->displayname");
+		}
 		return $folderObj;
 	}
 
@@ -1773,11 +2171,20 @@ 
 	{
 		$account = $folder = null;
 		$this->splitID($folderid, $account, $folder);
-		if (is_numeric($account)) $type = 'mail';
+		if (is_numeric($account))
+		{
+			$type = 'mail';
+		}
 
-		if ($type != 'mail') return false;
+		if ($type != 'mail')
+		{
+			return false;
+		}
 
-		if (!isset($this->mail)) $this->mail = Mail::getInstance(false,self::$profileID,true,false,true);
+		if (!isset($this->mail))
+		{
+			$this->mail = Mail::getInstance(false,self::$profileID,true,false,true);
+		}
 
         $this->mail->reopen($folder);
 
@@ -1822,7 +2229,7 @@ 
      * @throws StatusException              could throw specific SYNC_STATUS_* exceptions
      */
     public function DeleteMessage($folderid, $id, $contentParameters)
-	{
+    {
 		unset($contentParameters);	// not used, but required by function signature
 		ZLog::Write(LOGLEVEL_DEBUG, "IMAP-DeleteMessage: (fid: '$folderid'  id: '$id' )");
 		/*
@@ -1848,7 +2255,10 @@ 
 			$error = $e->getMessage();
 			ZLog::Write(LOGLEVEL_DEBUG,__METHOD__.__LINE__." $_messageUID, $folder ->".$error);
 			// if the server thinks the message does not exist report deletion as success
-			if (stripos($error,'[NONEXISTENT]')!==false) return true;
+			if (stripos($error,'[NONEXISTENT]')!==false)
+			{
+				return true;
+			}
 			return false;
 		}
 
@@ -1881,7 +2291,7 @@ 
      * @throws StatusException              could throw specific SYNC_STATUS_* exceptions
      */
     public function SetReadFlag($folderid, $id, $flags, $contentParameters)
-	{
+    {
 		unset($contentParameters);	// not used, but required by function signature
 		// ZLog::Write(LOGLEVEL_DEBUG, "IMAP-SetReadFlag: (fid: '$folderid'  id: '$id'  flags: '$flags' )");
 		$account = $folder = null;
@@ -1965,10 +2375,16 @@ 
 		$account = $parent_id = $app = null;
 		$this->splitID($id, $account, $folder, $app);
 		$old_hash = $this->folder2hash($account, $folder);
-		if ($parentid) $this->splitID($parentid, $account, $parentfolder, $app);
+		if ($parentid)
+		{
+			$this->splitID($parentid, $account, $parentfolder, $app);
+		}
 		ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."( '$id (-> $folder)','$parentid ".($parentid?'(->'.$parentfolder.')':'')."') called!");
 		$ret = $this->mail->deleteFolder($folder);
-		if ($ret) $newHash = $this->rename_folder_hash($account, $old_hash, "##Dele#edFolder#$folder##");
+		if ($ret)
+		{
+			$newHash = $this->rename_folder_hash($account, $old_hash, "##Dele#edFolder#$folder##");
+		}
 		return $ret;
 	}
 
@@ -2013,7 +2429,10 @@ 
 
 		$str = $this->backend->createID($account, $folder, $id);
 
-		if ($this->debugLevel>1) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."($account,'$f',$id) type=$account, folder=$folder --> '$str'");
+		if ($this->debugLevel>1)
+		{
+			ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."($account,'$f',$id) type=$account, folder=$folder --> '$str'");
+		}
 
 		return $str;
 	}
@@ -2034,7 +2453,10 @@ 
 		// convert numeric folder-id back to folder name
 		$folder = $this->hash2folder($account,$f=$folder);
 
-		if ($this->debugLevel>1) ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."('$str','$account','$folder',$appid)");
+		if ($this->debugLevel>1)
+		{
+			ZLog::Write(LOGLEVEL_DEBUG,__METHOD__."('$str','$account','$folder',$appid)");
+		}
 	}
 
 	/**
@@ -2053,7 +2475,10 @@ 
 	 */
 	private function folder2hash($account,$folder)
 	{
-		if(!isset($this->folderHashes)) $this->readFolderHashes();
+		if(!isset($this->folderHashes))
+		{
+			$this->readFolderHashes();
+		}
 
 		if (($index = array_search($folder, (array)$this->folderHashes[$account])) === false)
 		{
@@ -2076,7 +2501,10 @@ 
 	 */
 	private function hash2folder($account,$index)
 	{
-		if(!isset($this->folderHashes)) $this->readFolderHashes();
+		if(!isset($this->folderHashes))
+		{
+			$this->readFolderHashes();
+		}
 
 		return isset($this->folderHashes[$account]) ? $this->folderHashes[$account][$index] : null;
 	}