EGroupware / egroupware

api/src/Cache/Files.php

Braces +24 added lines, -6 removed lines

@@ -50,7 +50,10 @@ 
 		else
 		{
 			$this->base_path = Api\Cache::get_system_config('temp_dir', false);
-			if (isset($this->base_path)) $this->base_path .= '/egw_cache';
+			if (isset($this->base_path))
+			{
+				$this->base_path .= '/egw_cache';
+			}
 		}
 		if (!isset($this->base_path) || !file_exists($this->base_path) && !mkdir($this->base_path,0700,true))
 		{
@@ -86,7 +89,10 @@ 
 		{
 			flock($ret, LOCK_EX);
 			$ok = fwrite($ret, serialize($data));
-			if ((int)$expiration > 0) file_put_contents($fname.self::EXPIRATION_EXTENSION,(string)$expiration);
+			if ((int)$expiration > 0)
+			{
+				file_put_contents($fname.self::EXPIRATION_EXTENSION,(string)$expiration);
+			}
 			flock($ret, LOCK_UN);
 			fclose($ret);
 			$ret = $ok !== false;
@@ -106,7 +112,10 @@ 
 	{
 		if (($ret = @file_put_contents($fname=$this->filename($keys,true),serialize($data),LOCK_EX) > 0))
 		{
-			if ((int)$expiration > 0) file_put_contents($fname.self::EXPIRATION_EXTENSION,(string)$expiration);
+			if ((int)$expiration > 0)
+			{
+				file_put_contents($fname.self::EXPIRATION_EXTENSION,(string)$expiration);
+			}
 		}
 		return $ret;
 	}
@@ -182,17 +191,26 @@ 
 		}
 		foreach(scandir($path) as $file)
 		{
-			if ($file == '.' || $file == '..') continue;
+			if ($file == '.' || $file == '..')
+			{
+				continue;
+			}
 
 			$file = $path.'/'.$file;
 
 			if (is_dir($file))
 			{
-				if (!self::rm_recursive($file)) return false;
+				if (!self::rm_recursive($file))
+				{
+					return false;
+				}
 			}
 			else
 			{
-				if (!unlink($file)) return false;
+				if (!unlink($file))
+				{
+					return false;
+				}
 			}
 		}
 		return rmdir($path);

api/src/Cache/Exception.php

Braces +3 added lines, -1 removed lines

@@ -18,4 +18,6 @@
 /**
  * Exception thrown by Cache classes
  */
-class Exception extends Api\Exception {}
+class Exception extends Api\Exception
+{
+}

api/src/Country.php

Braces +21 added lines, -5 removed lines

@@ -368,7 +368,10 @@ 
 	{
 		if ($translated)
 		{
-			if (!self::$countries_translated) self::_translate_countries();
+			if (!self::$countries_translated)
+			{
+				self::_translate_countries();
+			}
 
 			return self::$countries_translated[strtoupper($code)];
 		}
@@ -383,7 +386,11 @@ 
 	 */
 	public static function country_code($name)
 	{
-		if (!$name) return '';	// nothing to do
+		if (!$name)
+		{
+			return '';
+		}
+		// nothing to do
 
 		if (strlen($name) == 2 && isset(self::$country_array[$name]))
 		{
@@ -394,7 +401,10 @@ 
 		{
 			return $code;
 		}
-		if (!self::$countries_translated) self::_translate_countries();
+		if (!self::$countries_translated)
+		{
+			self::_translate_countries();
+		}
 
 		if (($code = array_search(strtoupper($name),self::$countries_translated)) !== false ||
 			($code = array_search($name,self::$countries_translated)) !== false)
@@ -429,7 +439,10 @@ 
 	{
 		if ($translated)
 		{
-			if (!self::$countries_translated) self::_translate_countries();
+			if (!self::$countries_translated)
+			{
+				self::_translate_countries();
+			}
 
 			return self::$countries_translated;
 		}
@@ -443,7 +456,10 @@ 
 	 */
 	protected static function _translate_countries()
 	{
-		if (self::$countries_translated) return;
+		if (self::$countries_translated)
+		{
+			return;
+		}
 
 		self::$countries_translated = self::$country_array;
 		// try to translate them and sort alphabetic

api/src/Auth/Ads.php

Braces +16 added lines, -4 removed lines

@@ -90,7 +90,10 @@ 
 			if (($id = $GLOBALS['egw']->accounts->name2id($username,'account_lid','u')))
 			{
 				$ret = $GLOBALS['egw']->accounts->id2name($id,'account_status') == 'A';
-				if (!$ret) error_log(__METHOD__."('$username') account_status check returning ".array2string($ret));
+				if (!$ret)
+				{
+					error_log(__METHOD__."('$username') account_status check returning ".array2string($ret));
+				}
 				return $ret;
 			}
 			// store homedirectory for egw_session->read_repositories
@@ -158,7 +161,10 @@ 
 	static function setLastPwdChange($account_id=0, $passwd=NULL, $lastpwdchange=NULL, $return_mod=false)
 	{
 		unset($passwd);	// not used but required by function signature
-		if (!($adldap = Api\Accounts\Ads::get_adldap())) return false;
+		if (!($adldap = Api\Accounts\Ads::get_adldap()))
+		{
+			return false;
+		}
 
 		if ($lastpwdchange)
 		{
@@ -166,7 +172,10 @@ 
 			$ldapServerInfo = Api\Ldap\ServerInfo::get($adldap->getLdapConnection(), $GLOBALS['egw_info']['server']['ads_host']);
 			if ($ldapServerInfo->serverType == Api\Ldap\ServerInfo::SAMBA4)
 			{
-				if ($lastpwdchange == -1) $lastpwdchange = time();
+				if ($lastpwdchange == -1)
+				{
+					$lastpwdchange = time();
+				}
 			}
 			// while Windows only allows to set -1 for current time (or 0 to force user to change password)
 			else
@@ -179,7 +188,10 @@ 
 			$lastpwdchange = Api\Accounts\Ads::convertUnixTimeToWindowsTime($lastpwdchange);
 		}
 		$mod = array('pwdlastset' => $lastpwdchange);
-		if ($return_mod) return $mod;
+		if ($return_mod)
+		{
+			return $mod;
+		}
 
 		$ret = false;
 		if ($account_id && ($username = Api\Accounts::id2name($account_id, 'account_lid')) &&

api/src/Auth/Mail.php

Braces +24 added lines, -6 removed lines

@@ -70,7 +70,10 @@ 
 			}
 			catch(Horde_Imap_Client_Exception $e) {
 				// throw everything but authentication failed as exception
-				if ($e->getCode() != Horde_Imap_Client_Exception::LOGIN_AUTHENTICATIONFAILED) throw $e;
+				if ($e->getCode() != Horde_Imap_Client_Exception::LOGIN_AUTHENTICATIONFAILED)
+				{
+					throw $e;
+				}
 
 				$mailauth = false;
 			}
@@ -84,23 +87,38 @@ 
 			{
 				case 'imap':
 				default:
-					if (!isset($port)) $port = 143;
+					if (!isset($port))
+					{
+						$port = 143;
+					}
 					$mailauth = imap_open('{'.$host.':'.$port.'/imap/novalidate-cert}INBOX', $username , $passwd);
 					break;
 				case 'imaps':
-					if (!isset($port)) $port = 993;
+					if (!isset($port))
+					{
+						$port = 993;
+					}
 					$mailauth = imap_open('{'.$host.'/imap/ssl/novalidate-cert:'.$port.'}INBOX', $username , $passwd);
 					break;
 				case 'pop3':
-					if (!isset($port)) $port = 110;
+					if (!isset($port))
+					{
+						$port = 110;
+					}
 					$mailauth = imap_open('{'.$host.'/pop3/novalidate-cert:'.$port.'}INBOX', $username , $passwd);
 					break;
 				case 'pop3s':
-					if (!isset($port)) $port = 995;
+					if (!isset($port))
+					{
+						$port = 995;
+					}
 					$mailauth = imap_open('{'.$host.'/pop3/ssl/novalidate-cert:'.$port.'}INBOX', $username , $passwd);
 					break;
 			}
-			if ($mailauth) imap_close($mailauth);
+			if ($mailauth)
+			{
+				imap_close($mailauth);
+			}
 		}
 		return !!$mailauth;
 	}

api/src/Auth/Cas.php

Braces +4 added lines, -1 removed lines

@@ -33,13 +33,16 @@
 	{
 		/* if program goes here, authenticate is, normaly, already verified by CAS */
 		if ($GLOBALS['egw_info']['server']['account_repository'] != 'ldap' &&
-			$GLOBALS['egw_info']['server']['account_repository'] != 'ldsq') /* For anonymous LDAP connection */
+			$GLOBALS['egw_info']['server']['account_repository'] != 'ldsq')
+		{
+			/* For anonymous LDAP connection */
 		{
 			if (!($id = $GLOBALS['egw']->accounts->name2id($username,'account_lid','u')) &&
 				$GLOBALS['egw_info']['server']['auto_create_acct'])
 			{
 				// create a global array with all availible info about that account
 				$GLOBALS['auto_create_acct'] = array();
+		}
 				foreach(array(
 					'givenname' => 'firstname',
 					'sn'        => 'lastname',

api/src/Csrf.php

Braces +8 added lines, -2 removed lines

@@ -64,10 +64,16 @@
 		if (!isset($stored_purpose) || $stored_purpose !== $_purpose)
 		{
 			error_log('CSRF detected from IP '.$_SERVER['REMOTE_ADDR'].' to '.$_SERVER['REQUEST_METHOD'].' '.$_SERVER['REQUEST_URI']);
-			if ($_POST) error_log(array2string($_POST));
+			if ($_POST)
+			{
+				error_log(array2string($_POST));
+			}
 			// we are not throwing an exception here, but die, to not allow catching it!
 			die("CSRF detected, request terminated!");
 		}
-		if ($_delete_token) Cache::unsetSession(__CLASS__, $_token);
+		if ($_delete_token)
+		{
+			Cache::unsetSession(__CLASS__, $_token);
+		}
 	}
 }
\ No newline at end of file

api/src/Image.php

Braces +48 added lines, -12 removed lines

@@ -33,7 +33,10 @@ 
 	{
 		$svg = Header\UserAgent::mobile() ? null : $_svg; // ATM we use svg icons only for mobile theme
 		static $image_map_no_svg = null, $image_map_svg = null;
-		if (is_null($svg)) $svg = self::svg_usable ();
+		if (is_null($svg))
+		{
+			$svg = self::svg_usable ();
+		}
 		if ($svg)
 		{
 			$image_map =& $image_map_svg;
@@ -42,7 +45,10 @@ 
 		{
 			$image_map =& $image_map_no_svg;
 		}
-		if (is_null($image_map)) $image_map = self::map(null, $svg);
+		if (is_null($image_map))
+		{
+			$image_map = self::map(null, $svg);
+		}
 
 		// array of images in descending precedence
 		if (is_array($image))
@@ -152,15 +158,24 @@ 
 		$img_types = array('png','jpg','gif','ico');
 
 		// if we want svg, prepend it to img-types
-		if ($svg) array_unshift ($img_types, 'svg');
+		if ($svg)
+		{
+			array_unshift ($img_types, 'svg');
+		}
 
 		$map = array();
 		foreach(scandir(EGW_SERVER_ROOT) as $app)
 		{
-			if ($app[0] == '.' || !is_dir(EGW_SERVER_ROOT.'/'.$app) || !file_exists(EGW_SERVER_ROOT.'/'.$app.'/templates')) continue;
+			if ($app[0] == '.' || !is_dir(EGW_SERVER_ROOT.'/'.$app) || !file_exists(EGW_SERVER_ROOT.'/'.$app.'/templates'))
+			{
+				continue;
+			}
 
 			$app_map =& $map[$app];
-			if (true) $app_map = array();
+			if (true)
+			{
+				$app_map = array();
+			}
 			$imagedirs = array();
 			if (Header\UserAgent::mobile())
 			{
@@ -174,24 +189,39 @@ 
 			{
 				$imagedirs[] = '/'.$app.'/templates/'.$template_set.'/images';
 			}
-			if ($template_set != 'idots') $imagedirs[] = '/'.$app.'/templates/idots/images';
+			if ($template_set != 'idots')
+			{
+				$imagedirs[] = '/'.$app.'/templates/idots/images';
+			}
 			$imagedirs[] = '/'.$app.'/templates/default/images';
 
 			foreach($imagedirs as $imagedir)
 			{
-				if (!file_exists($dir = EGW_SERVER_ROOT.$imagedir) || !is_readable($dir)) continue;
+				if (!file_exists($dir = EGW_SERVER_ROOT.$imagedir) || !is_readable($dir))
+				{
+					continue;
+				}
 
 				foreach(scandir($dir) as $img)
 				{
-					if ($img[0] == '.') continue;
+					if ($img[0] == '.')
+					{
+						continue;
+					}
 
 					$subdir = null;
 					foreach(is_dir($dir.'/'.$img) ? scandir($dir.'/'.($subdir=$img)) : (array) $img as $img)
 					{
 						$name = null;
-						if (!in_array($ext = self::get_extension($img, $name), $img_types) || empty($name)) continue;
+						if (!in_array($ext = self::get_extension($img, $name), $img_types) || empty($name))
+						{
+							continue;
+						}
 
-						if (isset($subdir)) $name = $subdir.'/'.$name;
+						if (isset($subdir))
+						{
+							$name = $subdir.'/'.$name;
+						}
 
 						if (!isset($app_map[$name]) || array_search($ext, $img_types) < array_search(self::get_extension($app_map[$name]), $img_types))
 						{
@@ -202,12 +232,18 @@ 
 			}
 		}
 		$app_map =& $map['vfs'];
-		if (true) $app_map = array();
+		if (true)
+		{
+			$app_map = array();
+		}
 		if (($dir = $GLOBALS['egw_info']['server']['vfs_image_dir']) && Vfs::file_exists($dir) && Vfs::is_readable($dir))
 		{
 			foreach(Vfs::find($dir) as $img)
 			{
-				if (!in_array($ext = self::get_extension($img, $name), $img_types) || empty($name)) continue;
+				if (!in_array($ext = self::get_extension($img, $name), $img_types) || empty($name))
+				{
+					continue;
+				}
 
 				if (!isset($app_map[$name]) || array_search($ext, $img_types) < array_search(self::get_extension($app_map[$name]), $img_types))
 				{

api/src/Mail/Smtp.php

Braces +12 added lines, -3 removed lines

@@ -235,8 +235,14 @@ 
 	 */
 	/*static*/ public function mailbox_addr($account,$domain=null,$mail_login_type=null)
 	{
-		if (is_null($domain)) $domain = $this->defaultDomain;
-		if (is_null($mail_login_type)) $mail_login_type = $this->loginType;
+		if (is_null($domain))
+		{
+			$domain = $this->defaultDomain;
+		}
+		if (is_null($mail_login_type))
+		{
+			$mail_login_type = $this->loginType;
+		}
 
 		switch($mail_login_type)
 		{
@@ -245,7 +251,10 @@ 
 				break;
 
 			case 'uidNumber':
-				if (is_array($account)) $account = $account['account_id'];
+				if (is_array($account))
+				{
+					$account = $account['account_id'];
+				}
 				$mbox = 'u'.$account.'@'.$domain;
 				break;