EGroupware / egroupware

api/src/Storage/History.php

Braces +16 added lines, -5 removed lines

@@ -142,7 +142,10 @@ 
 	 */
 	function search($filter,$order='history_id',$sort='DESC',$limit=null)
 	{
-		if (!is_array($filter)) $filter = is_numeric($filter) ? array('history_record_id' => $filter) : array();
+		if (!is_array($filter))
+		{
+			$filter = is_numeric($filter) ? array('history_record_id' => $filter) : array();
+		}
 
 		if (!$order || !preg_match('/^[a-z0-9_]+$/i',$order) || !preg_match('/^(asc|desc)?$/i',$sort))
 		{
@@ -160,10 +163,16 @@ 
 				unset($filter[$col]);
 			}
 		}
-		if (!isset($filter['history_appname'])) $filter['history_appname'] = $this->appname;
+		if (!isset($filter['history_appname']))
+		{
+			$filter['history_appname'] = $this->appname;
+		}
 
 		// do not try to read all history entries of an app
-		if (!$filter['history_record_id']) return array();
+		if (!$filter['history_record_id'])
+		{
+			return array();
+		}
 
 		$rows = array();
 		foreach($this->db->select(self::TABLE, '*', $filter, __LINE__, __FILE__,
@@ -188,8 +197,10 @@ 
 		$rows = array();
 		$filter['history_appname'] = $query['appname'];
 		$filter['history_record_id'] = $query['record_id'];
-		if(is_array($query['colfilter'])) {
-			foreach($query['colfilter'] as $column => $value) {
+		if(is_array($query['colfilter']))
+		{
+			foreach($query['colfilter'] as $column => $value)
+			{
 				$filter[$column] = $value;
 			}
 		}

api/src/Exception/NoPermission/Admin.php

Braces +4 added lines, -1 removed lines

@@ -21,7 +21,10 @@
 {
 	function __construct($msg=null,$code=102)
 	{
-		if (is_null($msg)) $msg = 'admin';
+		if (is_null($msg))
+		{
+			$msg = 'admin';
+		}
 
 		parent::__construct($msg,$code);
 	}

api/src/Exception/NoPermission/Record.php

Braces +3 added lines, -1 removed lines

@@ -19,4 +19,6 @@
  * User lacks a record level permission, eg. he's not the owner and has no grant from the owner
  *
  */
-class Record extends Exception\NoPermission { }
+class Record extends Exception\NoPermission
+{
+}

api/src/Exception/NotFound.php

Braces +4 added lines, -1 removed lines

@@ -28,7 +28,10 @@
 	 */
 	function __construct($msg=null,$code=2)
 	{
-		if (is_null($msg)) $msg = lang('Entry not found!');
+		if (is_null($msg))
+		{
+			$msg = lang('Entry not found!');
+		}
 
 		parent::__construct($msg,$code);
 	}

api/src/Exception/WrongUserinput.php

Braces +3 added lines, -1 removed lines

@@ -17,4 +17,6 @@
  * Wrong or missing required user input: message should be translated so it can be shown directly to the user
  *
  */
-class WrongUserinput extends AssertionFailed { }
+class WrongUserinput extends AssertionFailed
+{
+}

api/src/Exception/WrongParameter.php

Braces +3 added lines, -1 removed lines

@@ -18,4 +18,6 @@
  *
  * As you get this only by an error in the code or during development, the message does not need to be translated
  */
-class WrongParameter extends AssertionFailed { }
+class WrongParameter extends AssertionFailed
+{
+}

api/src/Exception/AssertionFailed.php

Braces +3 added lines, -1 removed lines

@@ -20,4 +20,6 @@
  *
  * As you get this only by an error in the code or during development, the message does not need to be translated
  */
-class AssertionFailed extends Api\Exception { }
+class AssertionFailed extends Api\Exception
+{
+}

api/src/Exception/NoPermission.php

Braces +4 added lines, -1 removed lines

@@ -29,7 +29,10 @@
 	 */
 	function __construct($msg=null,$code=100)
 	{
-		if (is_null($msg)) $msg = lang('Permisson denied!');
+		if (is_null($msg))
+		{
+			$msg = lang('Permisson denied!');
+		}
 
 		parent::__construct($msg,$code);
 	}

api/src/loader.php

Braces +20 added lines, -5 removed lines

@@ -36,7 +36,11 @@ 
 	die("EGroupware requires PHP $egw_min_php_version or greater.<br />Please contact your System Administrator to upgrade PHP!");
 }
 
-if (!defined('EGW_API_INC')) define('EGW_API_INC',PHPGW_API_INC);	// this is to support the header upgrade
+if (!defined('EGW_API_INC'))
+{
+	define('EGW_API_INC',PHPGW_API_INC);
+}
+// this is to support the header upgrade
 
 /* Make sure the header.inc.php is current. */
 if (!isset($GLOBALS['egw_domain']) || $GLOBALS['egw_info']['server']['versions']['header'] < $GLOBALS['egw_info']['server']['versions']['current_header'])
@@ -70,21 +74,32 @@ 
 			$GLOBALS['egw_info'] = array_merge($_SESSION[Session::EGW_INFO_CACHE],array('flags' => $GLOBALS['egw_info']['flags']));
 
 			// include required class-definitions
-			if (is_array($_SESSION[Session::EGW_REQUIRED_FILES]))	// all classes, which can not be autoloaded
+			if (is_array($_SESSION[Session::EGW_REQUIRED_FILES]))
+			{
+				// all classes, which can not be autoloaded
 			{
 				foreach($_SESSION[Session::EGW_REQUIRED_FILES] as $file)
 				{
 					require_once($file);
+			}
 				}
 			}
 			$GLOBALS['egw'] = unserialize($_SESSION[Session::EGW_OBJECT_CACHE]);
 
-			if (is_object($GLOBALS['egw']) && ($GLOBALS['egw'] instanceof Egw))	// only egw object has wakeup2, setups egw_minimal eg. has not!
+			if (is_object($GLOBALS['egw']) && ($GLOBALS['egw'] instanceof Egw))
 			{
-				$GLOBALS['egw']->wakeup2();	// adapt the restored egw-object/enviroment to this request (eg. changed current app)
+				// only egw object has wakeup2, setups egw_minimal eg. has not!
+			{
+				$GLOBALS['egw']->wakeup2();
+			}
+			// adapt the restored egw-object/enviroment to this request (eg. changed current app)
 
 				$GLOBALS['egw_info']['flags']['session_restore_time'] = microtime(true) - $GLOBALS['egw_info']['flags']['page_start_time'];
-				if (is_object($GLOBALS['egw']->translation)) return;	// exit this file, as the rest of the file creates a new egw-object and -enviroment
+				if (is_object($GLOBALS['egw']->translation))
+				{
+					return;
+				}
+				// exit this file, as the rest of the file creates a new egw-object and -enviroment
 			}
 			// egw object could NOT be restored from the session, create a new one
 			unset($GLOBALS['egw']);