EGroupware / egroupware

api/src/Mail/Cache.php

Spacing +2 added lines, -2 removed lines

@@ -39,7 +39,7 @@ 
      */
     public function get($key, $lifetime = 0)
 	{
-		unset($lifetime);	// not (yet) used, but required by function signature
+		unset($lifetime); // not (yet) used, but required by function signature
 
 		$ret = Api\Cache::getCache(self::LEVEL, 'mail', $key);
 
@@ -71,7 +71,7 @@ 
      */
     public function exists($key, $lifetime = 0)
 	{
-		unset($lifetime);	// not (yet) used, but required by function signature
+		unset($lifetime); // not (yet) used, but required by function signature
 
 		return !is_null(Api\Cache::getCache(self::LEVEL, 'mail', $key));
 	}

api/src/Mail/Smtp/Ads.php

Spacing +2 added lines, -2 removed lines

@@ -133,7 +133,7 @@ 
 	 */
 	protected function getLdapConnection()
 	{
-		static $ldap=null;
+		static $ldap = null;
 
 		if (is_null($ldap))
 		{
@@ -151,7 +151,7 @@ 
 	 *
 	 * @param string $defaultDomain =null
 	 */
-	function __construct($defaultDomain=null)
+	function __construct($defaultDomain = null)
 	{
 		parent::__construct($defaultDomain);
 

api/src/Mail/Smtp/Oldqmailuser.php

Spacing +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
 	/**
 	 * Primary mail address required as an alias too: true or false
 	 */
-	const REQUIRE_MAIL_AS_ALIAS=false;
+	const REQUIRE_MAIL_AS_ALIAS = false;
 
 	/**
 	 * Attribute for forwards OR false if not possible

api/src/Mail/Smtp/Univention.php

Spacing +2 added lines, -2 removed lines

@@ -50,7 +50,7 @@ 
 	/**
 	 * Primary mail address required as an alias too: true or false
 	 */
-	const REQUIRE_MAIL_AS_ALIAS=false;
+	const REQUIRE_MAIL_AS_ALIAS = false;
 
 	/**
 	 * Attribute for forwards OR false if not possible
@@ -95,7 +95,7 @@ 
 	 */
 	function addAccountExtra(array $_hookValues, array $allValues, array &$newData)
 	{
-		unset($_hookValues);	// not used, but required by function signature
+		unset($_hookValues); // not used, but required by function signature
 
 		if (empty($allValues['univentionmailhomeserver'][0]) && $newData[self::MAIL_ENABLE_ATTR])
 		{

api/src/Mail/Hooks.php

Spacing +3 added lines, -3 removed lines

@@ -82,9 +82,9 @@ 
 	 */
 	protected static function run_plugin_hooks($method, array $data)
 	{
-		foreach(Account::search((int)$data['account_id'], 'params') as $params)
+		foreach (Account::search((int)$data['account_id'], 'params') as $params)
 		{
-			if (!Account::is_multiple($params)) continue;	// no need to waste time on personal accounts
+			if (!Account::is_multiple($params)) continue; // no need to waste time on personal accounts
 
 			try {
 				$account = new Account($params);
@@ -100,7 +100,7 @@ 
 					$imap->$method($data);
 				}
 			}
-			catch(\Exception $e) {
+			catch (\Exception $e) {
 				_egw_log_exception($e);
 				// ignore exception, without stalling other hooks
 			}

api/src/Mail/Types.php

Spacing +7 added lines, -7 removed lines

@@ -25,10 +25,10 @@ 
 	 *
 	 * @return array classname => label pairs
 	 */
-	static public function getSMTPServerTypes($extended=true)
+	static public function getSMTPServerTypes($extended = true)
 	{
 		$retData = self::server_types(false, $extended);
-		foreach(Api\Hooks::process(array(
+		foreach (Api\Hooks::process(array(
 			'location' => 'smtp_server_types',
 			'extended' => $extended,
 		), array('managementserver'), true) as $app => $data)
@@ -55,10 +55,10 @@ 
 	 * @param boolean $extended =true
 	 * @return array classname => label pairs
 	 */
-	static public function getIMAPServerTypes($extended=true)
+	static public function getIMAPServerTypes($extended = true)
 	{
 		$retData = self::server_types(true, $extended);
-		foreach(Api\Hooks::process(array(
+		foreach (Api\Hooks::process(array(
 			'location' => 'imap_server_types',
 			'extended' => $extended,
 		), array('managementserver'), true) as $app => $data)
@@ -84,11 +84,11 @@ 
 	 * @param boolean $extended =false
 	 * @return array
 	 */
-	protected static function server_types($imap=true, $extended=false)
+	protected static function server_types($imap = true, $extended = false)
 	{
 		$types = array();
 		$prefix = $imap ? 'Imap' : 'Smtp';
-		foreach(scandir($dir=__DIR__.'/'.$prefix) as $file)
+		foreach (scandir($dir = __DIR__.'/'.$prefix) as $file)
 		{
 			if ($file == '..')
 			{
@@ -106,7 +106,7 @@ 
 
 			$type = array(
 				'classname' => $class_name,
-				'description' => is_callable($function=$class_name.'::description') ? call_user_func($function) : $class_name,
+				'description' => is_callable($function = $class_name.'::description') ? call_user_func($function) : $class_name,
 			);
 			if ($imap) $type['protocol'] = 'imap';
 

api/src/Mail/Notifications.php

Spacing +9 added lines, -9 removed lines

@@ -52,7 +52,7 @@ 
 	 * @param boolean $return_empty_marker =false should we return null
 	 * @return array with values for "notify_folders", "notify_use_default"
 	 */
-	public static function read($acc_id, $account_id=null, $return_empty_marker=false)
+	public static function read($acc_id, $account_id = null, $return_empty_marker = false)
 	{
 		if (is_null($account_id))
 		{
@@ -71,7 +71,7 @@ 
 			//error_log(__METHOD__."($acc_id, ".array2string($account_id).") nothing in cache");
 		}
 		$account_specific = 0;
-		foreach($rows as $row)
+		foreach ($rows as $row)
 		{
 			if ($row['account_id'])
 			{
@@ -109,19 +109,19 @@ 
 	{
 		if (!is_numeric($account_id) || !($account_id >= 0))
 		{
-			return 0;	// nothing to save, happens eg. in setup
+			return 0; // nothing to save, happens eg. in setup
 		}
 
 		if ($account_id && !$folders && ($default = self::read($acc_id, 0)) && $default['notify_folders'])
 		{
-			$folders[] = null;	// we need to write a marker, that user wants no notifications!
+			$folders[] = null; // we need to write a marker, that user wants no notifications!
 		}
-		$old = self::read($acc_id, $account_id, true);	// true = return empty marker
-		if ($account_id && !$old['notify_account_id']) $old['notify_folders'] = array();	// ignore returned default
+		$old = self::read($acc_id, $account_id, true); // true = return empty marker
+		if ($account_id && !$old['notify_account_id']) $old['notify_folders'] = array(); // ignore returned default
 
 		$changed = 0;
 		// insert newly added ones
-		foreach(array_diff($folders, $old['notify_folders']) as $folder)
+		foreach (array_diff($folders, $old['notify_folders']) as $folder)
 		{
 			self::$db->insert(self::TABLE, array(
 				'acc_id' => $acc_id,
@@ -156,7 +156,7 @@ 
 	 * @param int|array $account_id =null
 	 * @return int number of rows deleted
 	 */
-	public static function delete($acc_id, $account_id=null)
+	public static function delete($acc_id, $account_id = null)
 	{
 		if (!($acc_id > 0) && !isset($account_id))
 		{
@@ -169,7 +169,7 @@ 
 		self::$db->delete(self::TABLE, $where, __LINE__, __FILE__, self::APP);
 
 		// invalidate cache: we allways unset everything about an account to simplify cache handling
-		foreach($acc_id > 0 ? (array)$acc_id : array_keys(self::$cache) as $acc_id)
+		foreach ($acc_id > 0 ? (array)$acc_id : array_keys(self::$cache) as $acc_id)
 		{
 			unset(self::$cache[$acc_id]);
 		}

api/config.php

Spacing +2 added lines, -2 removed lines

@@ -32,7 +32,7 @@ 
 $etag = '"'.md5($config.$link_registry).'"';
 
 // headers to allow caching, egw_framework specifies etag on url to force reload, even with Expires header
-Api\Session::cache_control(86400);	// cache for one day
+Api\Session::cache_control(86400); // cache for one day
 Header('Content-Type: text/javascript; charset=utf-8');
 Header('ETag: '.$etag);
 
@@ -47,7 +47,7 @@ 
 $content .= 'egw.set_link_registry('.$link_registry.", undefined, egw && egw.window !== window);\n";
 
 // we run our own gzip compression, to set a correct Content-Length of the encoded content
-if (in_array('gzip', explode(',',$_SERVER['HTTP_ACCEPT_ENCODING'])) && function_exists('gzencode'))
+if (in_array('gzip', explode(',', $_SERVER['HTTP_ACCEPT_ENCODING'])) && function_exists('gzencode'))
 {
 	$content = gzencode($content);
 	header('Content-Encoding: gzip');

api/js/etemplate/test/test.php

Spacing +5 added lines, -5 removed lines

@@ -16,18 +16,18 @@
 
 use EGroupware\Api;
 
-Api\Framework::includeJS('.','etemplate2','etemplate');
-Api\Framework::includeJS('jquery','jquery.tools.min','phpgwapi'); // Not needed once JS require works for files like this
-Api\Framework::includeJS('jquery','jquery.html5_upload','phpgwapi'); // Not needed once JS require works for files like this
+Api\Framework::includeJS('.', 'etemplate2', 'etemplate');
+Api\Framework::includeJS('jquery', 'jquery.tools.min', 'phpgwapi'); // Not needed once JS require works for files like this
+Api\Framework::includeJS('jquery', 'jquery.html5_upload', 'phpgwapi'); // Not needed once JS require works for files like this
 Api\Framework::includeCSS('/api/js/etemplate/test/test.css');
 
 /*
 * Test using any actual template
 */
 	$template = 'etemplate.et2_test_file_upload';
-	if($template) {
+	if ($template) {
 		$etemplate = new Api\Etemplate('etemplate.et2_test_file_upload');
-		$etemplate->exec('',array());
+		$etemplate->exec('', array());
 		return;
 	}
 	echo $GLOBALS['egw']->framework->header();