EGroupware / egroupware

api/src/Hooks.php

Spacing +14 added lines, -14 removed lines

@@ -64,22 +64,22 @@ 
 
 		if (!isset(self::$locations)) self::read();
 		$hooks = self::$locations[$location];
-		if (!isset($hooks) || empty($hooks)) return array();	// not a single app implements that hook
+		if (!isset($hooks) || empty($hooks)) return array(); // not a single app implements that hook
 
 		$apps = array_keys($hooks);
 		if (!$no_permission_check)
 		{
 			// on install of a new egroupware both hook-apps and user apps may be empty/not set
-			$apps = array_intersect((array)$apps,array_keys((array)$GLOBALS['egw_info']['user']['apps']));
+			$apps = array_intersect((array)$apps, array_keys((array)$GLOBALS['egw_info']['user']['apps']));
 		}
 		if ($order)
 		{
-			$apps = array_unique(array_merge((array)$order,$apps));
+			$apps = array_unique(array_merge((array)$order, $apps));
 		}
 		$results = array();
-		foreach((array)$apps as $appname)
+		foreach ((array)$apps as $appname)
 		{
-			$results[$appname] = self::single($args,$appname,$no_permission_check);
+			$results[$appname] = self::single($args, $appname, $no_permission_check);
 		}
 		return $results;
 	}
@@ -115,7 +115,7 @@ 
 		}
 
 		$ret = array();
-		foreach((array)self::$locations[$location][$appname] as $hook)
+		foreach ((array)self::$locations[$location][$appname] as $hook)
 		{
 			try {
 				// old style file hook
@@ -149,7 +149,7 @@ 
 		}
 
 		// hooks only existing in filesystem used by setup
-		if (!$ret && $try_unregistered && file_exists(EGW_SERVER_ROOT.($hook='/'.$appname.'/inc/hook_'.$location.'.inc.php')))
+		if (!$ret && $try_unregistered && file_exists(EGW_SERVER_ROOT.($hook = '/'.$appname.'/inc/hook_'.$location.'.inc.php')))
 		{
 			include(EGW_SERVER_ROOT.$hook);
 			return true;
@@ -181,7 +181,7 @@ 
 	 * @param boolean $return_methods =false true: return hook-method(s)
 	 * @return int|array the number of found hooks or for $return_methods array with methods
 	 */
-	public static function exists($location, $app, $return_methods=false)
+	public static function exists($location, $app, $return_methods = false)
 	{
 		if (!isset(self::$locations)) self::read();
 
@@ -208,7 +208,7 @@ 
 	 *
 	 * @param boolan $force_rescan =false true: do not use instance cache
 	 */
-	public static function read($force_rescan=false)
+	public static function read($force_rescan = false)
 	{
 		//$starttime = microtime(true);
 		if ($force_rescan) Cache::unsetInstance(__CLASS__, 'locations');
@@ -224,18 +224,18 @@ 
 
 			// read all apps using just filesystem data
 			$locations = array();
-			foreach(array_merge(array('api'), array_keys($GLOBALS['egw_info']['apps'])) as $appname)
+			foreach (array_merge(array('api'), array_keys($GLOBALS['egw_info']['apps'])) as $appname)
 			{
 				if ($appname[0] == '.' || !is_dir(EGW_SERVER_ROOT.'/'.$appname)) continue;
 
-				$f = EGW_SERVER_ROOT . '/' . $appname . '/setup/setup.inc.php';
+				$f = EGW_SERVER_ROOT.'/'.$appname.'/setup/setup.inc.php';
 				$setup_info = array($appname => array());
-				if(@file_exists($f)) include($f);
+				if (@file_exists($f)) include($f);
 
 				// some apps have setup_info for more then themselfs (eg. api for groupdav)
-				foreach($setup_info as $appname => $data)
+				foreach ($setup_info as $appname => $data)
 				{
-					foreach((array)$data['hooks'] as $location => $methods)
+					foreach ((array)$data['hooks'] as $location => $methods)
 					{
 						if (is_int($location))
 						{

api/src/Country.php

Spacing +12 added lines, -12 removed lines

@@ -364,7 +364,7 @@ 
 	 * @param boolean $translated =true use translated name or english
 	 * @return string
 	 */
-	public static function get_full_name($code,$translated=true)
+	public static function get_full_name($code, $translated = true)
 	{
 		if ($translated)
 		{
@@ -383,36 +383,36 @@ 
 	 */
 	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]))
 		{
-			return $name;	// $name is already a country-code
+			return $name; // $name is already a country-code
 		}
 
-		if (($code = array_search(strtoupper($name),self::$country_array)) !== false)
+		if (($code = array_search(strtoupper($name), self::$country_array)) !== false)
 		{
 			return $code;
 		}
 		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)
+		if (($code = array_search(strtoupper($name), self::$countries_translated)) !== false ||
+			($code = array_search($name, self::$countries_translated)) !== false)
 		{
 			return $code;
 		}
 		// search case-insensitive all translations for the english phrase of given country $name
 		// we do that to catch all possible cases of translations
-		static $en_names = array();	// we do some caching to minimize db-accesses
+		static $en_names = array(); // we do some caching to minimize db-accesses
 		if (isset($en_names[$name]))
 		{
 			$name = $en_names[$name];
 		}
-		elseif (($name_en = Translation::get_message_id($name,'common')))
+		elseif (($name_en = Translation::get_message_id($name, 'common')))
 		{
 			$name = $en_names[$name] = strtoupper($name_en);
 		}
-		if (($code = array_search(strtoupper($name),self::$country_array)) !== false)
+		if (($code = array_search(strtoupper($name), self::$country_array)) !== false)
 		{
 			return $code;
 		}
@@ -425,7 +425,7 @@ 
 	 * @param boolean $translated =true use translated names or english
 	 * @return array with 2-letter code => name pairs
 	 */
-	public static function countries($translated=true)
+	public static function countries($translated = true)
 	{
 		if ($translated)
 		{
@@ -447,12 +447,12 @@ 
 
 		self::$countries_translated = self::$country_array;
 		// try to translate them and sort alphabetic
-		foreach(self::$countries_translated as $k => $name)
+		foreach (self::$countries_translated as $k => $name)
 		{
 			self::$countries_translated[$k] = lang($name);
 		}
 
-		if(class_exists('Collator') && class_exists('Locale'))
+		if (class_exists('Collator') && class_exists('Locale'))
 		{
 			$col = new \Collator(Preferences::setlocale());
 			$col->asort(self::$countries_translated);

api/src/Auth/Fallbackmail2sql.php

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
 	/**
 	 * Constructor
 	 */
-	function __construct($primary='mail', $fallback='sql')
+	function __construct($primary = 'mail', $fallback = 'sql')
 	{
 		parent::__construct($primary, $fallback);
 	}

api/src/Auth/Nis.php

Spacing +8 added lines, -8 removed lines

@@ -26,22 +26,22 @@ 
 	 * @param string $passwd_type ='text' 'text' for cleartext passwords (default)
 	 * @return boolean true if successful authenticated, false otherwise
 	 */
-	function authenticate($username, $passwd, $passwd_type='text')
+	function authenticate($username, $passwd, $passwd_type = 'text')
 	{
-		unset($passwd_type);	// not used but required by interface
+		unset($passwd_type); // not used but required by interface
 
 		$domain = yp_get_default_domain();
-		if(!empty($GLOBALS['egw_info']['server']['nis_domain']))
+		if (!empty($GLOBALS['egw_info']['server']['nis_domain']))
 		{
 			$domain = $GLOBALS['egw_info']['server']['nis_domain'];
 		}
 
 		$map = "passwd.byname";
-		if(!empty($GLOBALS['egw_info']['server']['nis_map']))
+		if (!empty($GLOBALS['egw_info']['server']['nis_map']))
 		{
 			$map = $GLOBALS['egw_info']['server']['nis_map'];
 		}
-		$entry = yp_match( $domain, $map, $username );
+		$entry = yp_match($domain, $map, $username);
 
 		/*
 		 * we assume that the map is structured in the usual
@@ -63,11 +63,11 @@ 
 	 * @param int $account_id =0 account id of user whose passwd should be changed
 	 * @return boolean true if password successful changed, false otherwise
 	 */
-	function change_password($old_passwd, $new_passwd, $account_id=0)
+	function change_password($old_passwd, $new_passwd, $account_id = 0)
 	{
-		unset($old_passwd, $new_passwd, $account_id);	// not used but required by interface
+		unset($old_passwd, $new_passwd, $account_id); // not used but required by interface
 
 		// can't change passwords unless server runs as root (bad idea)
-		return( False );
+		return(False);
 	}
 }

api/src/Auth/Sqlssl.php

Spacing +8 added lines, -8 removed lines

@@ -29,21 +29,21 @@ 
 	 * @param string $passwd_type ='text' 'text' for cleartext passwords (default)
 	 * @return boolean true if successful authenticated, false otherwise
 	 */
-	function authenticate($username, $passwd, $passwd_type='text')
+	function authenticate($username, $passwd, $passwd_type = 'text')
 	{
-		unset($passwd_type);	// not used but required by interface
+		unset($passwd_type); // not used but required by interface
 
 		$local_debug = False;
 
-		if($local_debug)
+		if ($local_debug)
 		{
 			echo "<b>Debug SQL: uid - $username passwd - $passwd</b>";
 		}
-		if (!($row = $this->db->select($this->table,'account_lid,account_pwd',array(
+		if (!($row = $this->db->select($this->table, 'account_lid,account_pwd', array(
 			'account_lid' => $username,
 			'account_status' => 'A',
 			'account_type'   => 'u',
-		),__LINE__,__FILE__)->fetch()) || $GLOBALS['egw_info']['server']['case_sensitive_username'] && $row['account_lid'] != $username)
+		), __LINE__, __FILE__)->fetch()) || $GLOBALS['egw_info']['server']['case_sensitive_username'] && $row['account_lid'] != $username)
 		{
 			return false;
 		}
@@ -51,7 +51,7 @@ 
 		# Apache + mod_ssl provide the data in the environment
 		# Certificate (chain) verification occurs inside mod_ssl
 		# see http://www.modssl.org/docs/2.8/ssl_howto.html#ToC6
-		if(!isset($_SERVER['SSL_CLIENT_S_DN']))
+		if (!isset($_SERVER['SSL_CLIENT_S_DN']))
 		{
 			# if we're not doing SSL authentication, behave like auth_sql
 			return Api\Auth::compare_password($passwd, $row['account_pwd'], 'md5', strtolower($username));
@@ -67,9 +67,9 @@ 
 	 * @param int $account_id =0 account id of user whose passwd should be changed
 	 * @return boolean true if password successful changed, false otherwise
 	 */
-	function change_password($old_passwd, $new_passwd, $account_id=0)
+	function change_password($old_passwd, $new_passwd, $account_id = 0)
 	{
-		unset($old_passwd, $new_passwd, $account_id);	// not used but required by interface
+		unset($old_passwd, $new_passwd, $account_id); // not used but required by interface
 
 		// deny password changes.
 		return False;

api/src/Auth/Pam.php

Spacing +7 added lines, -7 removed lines

@@ -30,9 +30,9 @@ 
 	 * @param string $passwd_type ='text' 'text' for cleartext passwords (default)
 	 * @return boolean true if successful authenticated, false otherwise
 	 */
-	function authenticate($username, $passwd, $passwd_type='text')
+	function authenticate($username, $passwd, $passwd_type = 'text')
 	{
-		unset($passwd_type);	// not used but required by interface
+		unset($passwd_type); // not used but required by interface
 
 		if (pam_auth($username, get_magic_quotes_gpc() ? stripslashes($passwd) : $passwd))
 		{
@@ -40,12 +40,12 @@ 
 			if (!$GLOBALS['egw']->accounts->name2id($username) &&
 				function_exists('posix_getpwnam') && ($data = posix_getpwnam($username)))
 			{
-				list($fullname) = explode(',',$data['gecos']);
-				$parts = explode(' ',$fullname);
+				list($fullname) = explode(',', $data['gecos']);
+				$parts = explode(' ', $fullname);
 				if (count($parts) > 1)
 				{
 					$lastname = array_pop($parts);
-					$firstname = implode(' ',$parts);
+					$firstname = implode(' ', $parts);
 					$email = Api\Accounts::email($firstname, $lastname, $username);
 
 					$GLOBALS['auto_create_acct'] = array(
@@ -69,9 +69,9 @@ 
 	 * @param int $account_id =0 account id of user whose passwd should be changed
 	 * @return boolean true if password successful changed, false otherwise
 	 */
-	function change_password($old_passwd, $new_passwd, $account_id=0)
+	function change_password($old_passwd, $new_passwd, $account_id = 0)
 	{
-		unset($old_passwd, $new_passwd, $account_id);	// not used but required by interface
+		unset($old_passwd, $new_passwd, $account_id); // not used but required by interface
 
 		// deny password changes.
 		return False;

api/src/Auth/Mail.php

Spacing +10 added lines, -10 removed lines

@@ -34,14 +34,14 @@ 
 	 * @param string $passwd_type ='text' 'text' for cleartext passwords (default)
 	 * @return boolean true if successful authenticated, false otherwise
 	 */
-	function authenticate($username, $passwd, $passwd_type='text')
+	function authenticate($username, $passwd, $passwd_type = 'text')
 	{
-		unset($passwd_type);	// not used but required by function signature
+		unset($passwd_type); // not used but required by function signature
 
 		switch ($GLOBALS['egw_info']['server']['mail_login_type'])
 		{
 			case 'vmailmgr':
-				$username = $username . '@' . $GLOBALS['egw_info']['server']['mail_suffix'];
+				$username = $username.'@'.$GLOBALS['egw_info']['server']['mail_suffix'];
 				break;
 			case 'email':
 				$username = $GLOBALS['egw']->accounts->id2name($username, 'account_email');
@@ -68,7 +68,7 @@ 
 				$mailauth = true;
 				$imap->logout();
 			}
-			catch(Horde_Imap_Client_Exception $e) {
+			catch (Horde_Imap_Client_Exception $e) {
 				// throw everything but authentication failed as exception
 				if ($e->getCode() != Horde_Imap_Client_Exception::LOGIN_AUTHENTICATIONFAILED) throw $e;
 
@@ -85,19 +85,19 @@ 
 				case 'imap':
 				default:
 					if (!isset($port)) $port = 143;
-					$mailauth = imap_open('{'.$host.':'.$port.'/imap/novalidate-cert}INBOX', $username , $passwd);
+					$mailauth = imap_open('{'.$host.':'.$port.'/imap/novalidate-cert}INBOX', $username, $passwd);
 					break;
 				case 'imaps':
 					if (!isset($port)) $port = 993;
-					$mailauth = imap_open('{'.$host.'/imap/ssl/novalidate-cert:'.$port.'}INBOX', $username , $passwd);
+					$mailauth = imap_open('{'.$host.'/imap/ssl/novalidate-cert:'.$port.'}INBOX', $username, $passwd);
 					break;
 				case 'pop3':
 					if (!isset($port)) $port = 110;
-					$mailauth = imap_open('{'.$host.'/pop3/novalidate-cert:'.$port.'}INBOX', $username , $passwd);
+					$mailauth = imap_open('{'.$host.'/pop3/novalidate-cert:'.$port.'}INBOX', $username, $passwd);
 					break;
 				case 'pop3s':
 					if (!isset($port)) $port = 995;
-					$mailauth = imap_open('{'.$host.'/pop3/ssl/novalidate-cert:'.$port.'}INBOX', $username , $passwd);
+					$mailauth = imap_open('{'.$host.'/pop3/ssl/novalidate-cert:'.$port.'}INBOX', $username, $passwd);
 					break;
 			}
 			if ($mailauth) imap_close($mailauth);
@@ -113,9 +113,9 @@ 
 	 * @param int $account_id =0 account id of user whose passwd should be changed
 	 * @return boolean true if password successful changed, false otherwise
 	 */
-	function change_password($old_passwd, $new_passwd, $account_id=0)
+	function change_password($old_passwd, $new_passwd, $account_id = 0)
 	{
-		unset($old_passwd, $new_passwd, $account_id);	// not used but required by function sigature
+		unset($old_passwd, $new_passwd, $account_id); // not used but required by function sigature
 
 		return False;
 	}

api/src/Auth/Backend.php

Spacing +2 added lines, -2 removed lines

@@ -25,7 +25,7 @@ 
 	 * @param string $passwd_type ='text' 'text' for cleartext passwords (default)
 	 * @return boolean true if successful authenticated, false otherwise
 	 */
-	function authenticate($username, $passwd, $passwd_type='text');
+	function authenticate($username, $passwd, $passwd_type = 'text');
 
 	/**
 	 * changes password in sql datababse
@@ -36,5 +36,5 @@ 
 	 * @throws Exception to give a verbose error, why changing password failed
 	 * @return boolean true if password successful changed, false otherwise
 	 */
-	function change_password($old_passwd, $new_passwd, $account_id=0);
+	function change_password($old_passwd, $new_passwd, $account_id = 0);
 }

api/src/Auth/Cas.php

Spacing +6 added lines, -6 removed lines

@@ -29,18 +29,18 @@ 
 	 * @param string $passwd_type ='text' 'text' for cleartext passwords (default)
 	 * @return boolean true if successful authenticated, false otherwise
 	 */
-	function authenticate($username, $passwd, $passwd_type='text')
+	function authenticate($username, $passwd, $passwd_type = 'text')
 	{
 		/* 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 */
 		{
-			if (!($id = $GLOBALS['egw']->accounts->name2id($username,'account_lid','u')) &&
+			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(
+				foreach (array(
 					'givenname' => 'firstname',
 					'sn'        => 'lastname',
 					'uidnumber' => 'id',
@@ -48,11 +48,11 @@ 
 					'gidnumber' => 'primary_group',
 					) as $ldap_name => $acct_name)
 				{
-					$GLOBALS['auto_create_acct'][$acct_name] = Api\Translation::convert($allValues[0][$ldap_name][0],'utf-8');
+					$GLOBALS['auto_create_acct'][$acct_name] = Api\Translation::convert($allValues[0][$ldap_name][0], 'utf-8');
 				}
 				return True;
 			}
-			return $id && $GLOBALS['egw']->accounts->id2name($id,'account_status') == 'A' && phpCAS::checkAuthentication();
+			return $id && $GLOBALS['egw']->accounts->id2name($id, 'account_status') == 'A' && phpCAS::checkAuthentication();
 		}
 		return phpCAS::checkAuthentication();
 	}
@@ -65,7 +65,7 @@ 
 	 * @param int $account_id =0 account id of user whose passwd should be changed
 	 * @return boolean true if password successful changed, false otherwise
 	 */
-	function change_password($old_passwd, $new_passwd, $account_id=0)
+	function change_password($old_passwd, $new_passwd, $account_id = 0)
 	{
 		/* Not allowed */
 		return false;