EGroupware / egroupware

api/src/Storage/Base2.php

Spacing +7 added lines, -7 removed lines

@@ -53,9 +53,9 @@ 
 	 *
 	 * @return so_sql2
 	 */
-	function __construct($app='',$table='',Api\Db $db=null,$column_prefix='',$no_clone=false)
+	function __construct($app = '', $table = '', Api\Db $db = null, $column_prefix = '', $no_clone = false)
 	{
-		parent::__construct($app,$table,$db,$column_prefix,$no_clone);
+		parent::__construct($app, $table, $db, $column_prefix, $no_clone);
 	}
 
 	/**
@@ -68,13 +68,13 @@ 
 	 */
 	function __get($property)
 	{
-		switch($property)
+		switch ($property)
 		{
 			case 'id':
 				$property = $this->autoinc_id;
 				break;
 		}
-		if (in_array($property,$this->db_cols) || in_array($property,$this->non_db_cols))
+		if (in_array($property, $this->db_cols) || in_array($property, $this->non_db_cols))
 		{
 			return $this->data[$property];
 		}
@@ -88,15 +88,15 @@ 
 	 * @param string $property
 	 * @param mixed $value
 	 */
-	function __set($property,$value)
+	function __set($property, $value)
 	{
-		switch($property)
+		switch ($property)
 		{
 			case 'id':
 				$property = $this->autoinc_id;
 				break;
 		}
-		if (in_array($property,$this->db_cols) || in_array($property,$this->non_db_cols))
+		if (in_array($property, $this->db_cols) || in_array($property, $this->non_db_cols))
 		{
 			$this->data[$property] = $value;
 		}

api/src/Exception/NoPermission/App.php

Spacing +2 added lines, -2 removed lines

@@ -21,7 +21,7 @@ 
  */
 class App extends Exception\NoPermission
 {
-	function __construct($msg=null,$code=101)
+	function __construct($msg = null, $code = 101)
 	{
 		if (isset($GLOBALS['egw_info']['apps'][$msg]))
 		{
@@ -39,6 +39,6 @@ 
 						'"'.$app.'"');
 			}
 		}
-		parent::__construct($msg,$code);
+		parent::__construct($msg, $code);
 	}
 }

api/src/Exception/NoPermission/Admin.php

Spacing +2 added lines, -2 removed lines

@@ -19,10 +19,10 @@
  */
 class Admin extends App
 {
-	function __construct($msg=null,$code=102)
+	function __construct($msg = null, $code = 102)
 	{
 		if (is_null($msg)) $msg = 'admin';
 
-		parent::__construct($msg,$code);
+		parent::__construct($msg, $code);
 	}
 }

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/NotFound.php

Spacing +2 added lines, -2 removed lines

@@ -26,10 +26,10 @@
 	 * @param string $msg =null message, default "Entry not found!"
 	 * @param int $code =99 numerical code, default 2
 	 */
-	function __construct($msg=null,$code=2)
+	function __construct($msg = null, $code = 2)
 	{
 		if (is_null($msg)) $msg = lang('Entry not found!');
 
-		parent::__construct($msg,$code);
+		parent::__construct($msg, $code);
 	}
 }

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/Redirect.php

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
 	 * @param string $msg
 	 * @param int $code
 	 */
-	function __construct($url,$app=null,$msg=null,$code=301)
+	function __construct($url, $app = null, $msg = null, $code = 301)
 	{
 		$this->url = $url;
 		$this->app = $app;

api/src/Exception/NoPermission.php

Spacing +2 added lines, -2 removed lines

@@ -27,10 +27,10 @@
 	 * @param string $msg =null message, default "Permission denied!"
 	 * @param int $code =100 numerical code, default 100
 	 */
-	function __construct($msg=null,$code=100)
+	function __construct($msg = null, $code = 100)
 	{
 		if (is_null($msg)) $msg = lang('Permisson denied!');
 
-		parent::__construct($msg,$code);
+		parent::__construct($msg, $code);
 	}
 }

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/Db/CallbackIterator.php

Spacing +9 added lines, -10 removed lines

@@ -99,13 +99,13 @@ 
 	 * @param array $params =array() additional parameters, row is always first parameter
 	 * @param $key_callback =null optional callback, if you want different keys
 	 */
-	public function __construct(\Traversable $rs, $callback, $params=array(), $key_callback=null)
+	public function __construct(\Traversable $rs, $callback, $params = array(), $key_callback = null)
 	{
 		$this->callback = $callback;
 		$this->params = $params;
 		$this->key_callback = $key_callback;
 
-		if (is_a($rs,'IteratorAggregate'))
+		if (is_a($rs, 'IteratorAggregate'))
 		{
 			$this->rs = $rs->getIterator();
 		}
@@ -122,7 +122,7 @@ 
 	 */
 	public function current()
 	{
-		if (is_a($this->rs,'iterator'))
+		if (is_a($this->rs, 'iterator'))
 		{
 			$params = $this->params;
 			array_unshift($params, $this->rs->current());
@@ -138,11 +138,10 @@ 
 	 */
 	public function key()
 	{
-		if (is_a($this->rs,'iterator'))
+		if (is_a($this->rs, 'iterator'))
 		{
 			return $this->key_callback ?
-				call_user_func($this->key_callback, $this->rs->current()) :
-				$this->rs->key();
+				call_user_func($this->key_callback, $this->rs->current()) : $this->rs->key();
 		}
 		return 0;
 	}
@@ -152,7 +151,7 @@ 
 	 */
 	public function next()
 	{
-		if (is_a($this->rs,'iterator'))
+		if (is_a($this->rs, 'iterator'))
 		{
 			return $this->rs->next();
 		}
@@ -163,7 +162,7 @@ 
 	 */
 	public function rewind()
 	{
-		if (is_a($this->rs,'iterator'))
+		if (is_a($this->rs, 'iterator'))
 		{
 			return $this->rs->rewind();
 		}
@@ -174,9 +173,9 @@ 
 	 *
 	 * @return boolean
 	 */
-	public function valid ()
+	public function valid()
 	{
-		if (is_a($this->rs,'iterator'))
+		if (is_a($this->rs, 'iterator'))
 		{
 			return $this->rs->valid();
 		}

api/src/Db/Backup.php

Indentation +5 added lines, -5 removed lines

@@ -381,7 +381,7 @@ 
 		$zip = NULL;
 		$_f = NULL;
 		if($type == 'zip')
-	    {
+		{
 			// has already been verified to be available in fopen_backup
 			$zip = new ZipArchive;
 			if(($zip->open($filename)) !== TRUE)
@@ -719,9 +719,9 @@ 
 		while($file = $list[0])
 		{
 			if(is_dir($file) && $file != '.' && $file != '..')
-			    self::remove_dir_content($dir.'/'.$file);
+				self::remove_dir_content($dir.'/'.$file);
 			if(is_file($file) && $file != '.' && $file != '..')
-			    unlink($dir.'/'.$file);
+				unlink($dir.'/'.$file);
 			array_shift($list);
 		}
 		//rmdir($dir);  // dont remove own dir
@@ -967,8 +967,8 @@ 
 				echo '<center>'.lang("Cant open %1, needs ZipArchive", $name)."<br>\n".'</center>';
 			}
 
-		    fclose($f);
-		    if (file_exists($name)) unlink($name);
+			fclose($f);
+			if (file_exists($name)) unlink($name);
 			return TRUE;
 		}
 		// save files ....

Spacing +151 added lines, -151 removed lines

@@ -50,9 +50,9 @@ 
 	 * @var array
 	 */
 	var $exclude_tables = array(
-		'egw_sessions','egw_app_sessions','phpgw_sessions','phpgw_app_sessions',	// eGW's session-tables
-		'phpgw_anglemail',	// email's cache
-		'egw_felamimail_cache','egw_felamimail_folderstatus','phpgw_felamimail_cache','phpgw_felamimail_folderstatus',	// felamimail's cache
+		'egw_sessions', 'egw_app_sessions', 'phpgw_sessions', 'phpgw_app_sessions', // eGW's session-tables
+		'phpgw_anglemail', // email's cache
+		'egw_felamimail_cache', 'egw_felamimail_folderstatus', 'phpgw_felamimail_cache', 'phpgw_felamimail_folderstatus', // felamimail's cache
 		'egw_phpfreechat', // as of the fieldnames of the table a restore would fail within egroupware, and chatcontent is of no particular intrest
 	);
 	/**
@@ -84,7 +84,7 @@ 
 	 *
 	 * @var boolean
 	 */
-	var $backup_files = false ;
+	var $backup_files = false;
 	/**
 	 * Reference to schema_proc's Api\Db object
 	 *
@@ -99,7 +99,7 @@ 
 	{
 		if (isset($GLOBALS['egw_setup']) && is_object($GLOBALS['egw_setup']) && !isset($GLOBALS['egw_setup']->db))
 		{
-			$GLOBALS['egw_setup']->loaddb();	// we run inside setup, but db object is not loaded
+			$GLOBALS['egw_setup']->loaddb(); // we run inside setup, but db object is not loaded
 		}
 		if (isset($GLOBALS['egw_setup']->oProc) && is_object($GLOBALS['egw_setup']->oProc))	// schema_proc already instanciated, use it
 		{
@@ -117,21 +117,21 @@ 
 		{
 			if ($GLOBALS['egw_setup']->config_table && $GLOBALS['egw_setup']->table_exist(array($GLOBALS['egw_setup']->config_table)))
 			{
-				if (!($this->files_dir = $this->db->query("SELECT config_value FROM {$GLOBALS['egw_setup']->config_table} WHERE config_app='phpgwapi' AND config_name='files_dir'",__LINE__,__FILE__)->fetchColumn()))
+				if (!($this->files_dir = $this->db->query("SELECT config_value FROM {$GLOBALS['egw_setup']->config_table} WHERE config_app='phpgwapi' AND config_name='files_dir'", __LINE__, __FILE__)->fetchColumn()))
 				{
 					error_log(__METHOD__."->"."No files Directory set/found");
 				}
-				if (!($this->backup_dir = $this->db->query("SELECT config_value FROM {$GLOBALS['egw_setup']->config_table} WHERE config_app='phpgwapi' AND config_name='backup_dir'",__LINE__,__FILE__)->fetchColumn()))
+				if (!($this->backup_dir = $this->db->query("SELECT config_value FROM {$GLOBALS['egw_setup']->config_table} WHERE config_app='phpgwapi' AND config_name='backup_dir'", __LINE__, __FILE__)->fetchColumn()))
 				{
 					$this->backup_dir = $this->files_dir.'/db_backup';
 				}
-				$this->charset = $this->db->query("SELECT config_value FROM {$GLOBALS['egw_setup']->config_table} WHERE config_app='phpgwapi' AND config_name='system_charset'",__LINE__,__FILE__)->fetchColumn();
-				$this->api_version = $this->db->select($GLOBALS['egw_setup']->applications_table,'app_version',array('app_name'=>array('api','phpgwapi')),
-					__LINE__,__FILE__,0,'ORDER BY app_name ASC')->fetchColumn();
+				$this->charset = $this->db->query("SELECT config_value FROM {$GLOBALS['egw_setup']->config_table} WHERE config_app='phpgwapi' AND config_name='system_charset'", __LINE__, __FILE__)->fetchColumn();
+				$this->api_version = $this->db->select($GLOBALS['egw_setup']->applications_table, 'app_version', array('app_name'=>array('api', 'phpgwapi')),
+					__LINE__, __FILE__, 0, 'ORDER BY app_name ASC')->fetchColumn();
 				// Backup settings
-				$this->backup_mincount = $this->db->query("SELECT config_value FROM {$GLOBALS['egw_setup']->config_table} WHERE config_app='phpgwapi' AND config_name='backup_mincount'",__LINE__,__FILE__)->fetchColumn();
+				$this->backup_mincount = $this->db->query("SELECT config_value FROM {$GLOBALS['egw_setup']->config_table} WHERE config_app='phpgwapi' AND config_name='backup_mincount'", __LINE__, __FILE__)->fetchColumn();
 				// backup files too
-				$this->backup_files = (bool)$this->db->query("SELECT config_value FROM {$GLOBALS['egw_setup']->config_table} WHERE config_app='phpgwapi' AND config_name='backup_files'",__LINE__,__FILE__)->fetchColumn();
+				$this->backup_files = (bool)$this->db->query("SELECT config_value FROM {$GLOBALS['egw_setup']->config_table} WHERE config_app='phpgwapi' AND config_name='backup_files'", __LINE__, __FILE__)->fetchColumn();
 			}
 			if (!$this->charset) $this->charset = 'utf-8';
 		}
@@ -146,7 +146,7 @@ 
 			$this->backup_files = $GLOBALS['egw_info']['server']['backup_files'];
 			$this->charset = Api\Translation::charset();
 
-			$this->api_version = $GLOBALS['egw_info']['apps'][isset($GLOBALS['egw_info']['apps']['api'])?'api':'phpgwapi']['version'];
+			$this->api_version = $GLOBALS['egw_info']['apps'][isset($GLOBALS['egw_info']['apps']['api']) ? 'api' : 'phpgwapi']['version'];
 		}
 		// Set a default value if not set.
 		if (!isset($this->backup_mincount))
@@ -162,7 +162,7 @@ 
 		{
 			mkdir($this->backup_dir);
 		}
-		switch($this->db->Type)
+		switch ($this->db->Type)
 		{
 			case 'sapdb':
 			case 'maxdb':
@@ -179,7 +179,7 @@ 
 	 * @param $reading =false opening for reading ('rb') or writing ('wb')
 	 * @return string/resource/zip error-msg of file-handle
 	 */
-	function fopen_backup($name=false,$reading=false)
+	function fopen_backup($name = false, $reading = false)
 	{
 		//echo "function fopen_backup($name,$reading)<br>";	// !
 		if (!$name)
@@ -188,27 +188,27 @@ 
 			if (!$this->backup_dir || !is_writable($this->backup_dir))
 			{
 				//echo '   -> !$this->backup_dir || !is_writable($this->backup_dir)<br>';	// !
-				return lang("backupdir '%1' is not writeable by the webserver",$this->backup_dir);
+				return lang("backupdir '%1' is not writeable by the webserver", $this->backup_dir);
 			}
 			$name = $this->backup_dir.'/db_backup-'.date('YmdHi');
 		}
 		else	// remove the extension, to use the correct wrapper based on the extension
 		{
 			//echo '-> else<br>';	// !
-			$name = preg_replace('/\.(bz2|gz)$/i','',$name);
+			$name = preg_replace('/\.(bz2|gz)$/i', '', $name);
 		}
 		$mode = $reading ? 'rb' : 'wb';
-		list( , $type) = explode('.', basename($name));
-		if($type == 'zip' && $reading && $this->backup_files)
+		list(, $type) = explode('.', basename($name));
+		if ($type == 'zip' && $reading && $this->backup_files)
 		{
 			//echo '-> $type == "zip" && $reading<br>';	// !
-			if(!class_exists('ZipArchive', false))
+			if (!class_exists('ZipArchive', false))
 			{
 				$this->backup_files = false;
 				//echo '   -> (new ZipArchive) == NULL<br>';	// !
 				return lang("Cant open %1, needs ZipArchive", $name)."<br>\n";
 			}
-			if(!($f = fopen($name, $mode)))
+			if (!($f = fopen($name, $mode)))
 			{
 				//echo '   -> !($f = fopen($name, $mode))<br>';	// !
 				$lang_mode = $reading ? lang("reading") : lang("writing");
@@ -216,10 +216,10 @@ 
 			}
 			return $f;
 		}
-		if(class_exists('ZipArchive', false) && !$reading && $this->backup_files)
+		if (class_exists('ZipArchive', false) && !$reading && $this->backup_files)
 		{
 			//echo '-> (new ZipArchive) != NULL && !$reading; '.$name.'<br>';	// !
-			if(!($f = fopen($name, $mode)))
+			if (!($f = fopen($name, $mode)))
 			{
 				//echo '   -> !($f = fopen($name, $mode))<br>';	// !
 				$lang_mode = $reading ? lang("reading") : lang("writing");
@@ -227,9 +227,9 @@ 
 			}
 			return $f;
 		}
-		if(!($f = fopen("compress.bzip2://$name.bz2", $mode)) &&
-	 		!($f = fopen("compress.zlib://$name.gz",$mode)) &&
- 		 	!($f = fopen($name,$mode))
+		if (!($f = fopen("compress.bzip2://$name.bz2", $mode)) &&
+	 		!($f = fopen("compress.zlib://$name.gz", $mode)) &&
+ 		 	!($f = fopen($name, $mode))
 		)
 		{
 			//echo '-> !($f = fopen("compress.bzip2://$name.bz2", $mode))<br>';	// !
@@ -256,13 +256,13 @@ 
 		/* Search the backup directory for matching files. */
 		$handle = @opendir($this->backup_dir);
 		$files = array();
-		while($handle && ($file = readdir($handle)))
+		while ($handle && ($file = readdir($handle)))
 		{
 			/* Filter for only the files with the regular name (un-renamed).
 			 * Leave special backup files (renamed) in place.
 			 * Note that this also excludes "." and "..".
 			 */
-			if (preg_match("/^db_backup-[0-9]{12}(\.bz2|\.gz|\.zip|)$/",$file))
+			if (preg_match("/^db_backup-[0-9]{12}(\.bz2|\.gz|\.zip|)$/", $file))
 			{
 				$files[filectime($this->backup_dir.'/'.$file)] = $file;
 			}
@@ -272,7 +272,7 @@ 
 		/* Sort the files by ctime. */
 		krsort($files);
 		$count = 0;
-		foreach($files as $file)
+		foreach ($files as $file)
 		{
 			if ($count >= $this->backup_mincount)//
 			{
@@ -282,7 +282,7 @@ 
 					array_push($files_return, $file);
 				}
 			}
-			$count ++;
+			$count++;
 		}
 	}
 
@@ -292,13 +292,13 @@ 
 	 * @param int $minCount Minimum number of backups to keep.
 	 * @param boolean $backupFiles include files in backup or not, default dont change!
 	 */
-	function saveConfig($minCount,$backupFiles=null)
+	function saveConfig($minCount, $backupFiles = null)
 	{
-		Api\Config::save_value('backup_mincount',$this->backup_mincount=(int)$minCount,'phpgwapi');
+		Api\Config::save_value('backup_mincount', $this->backup_mincount = (int)$minCount, 'phpgwapi');
 
 		if (!is_null($backupFiles))
 		{
-			Api\Config::save_value('backup_files',$this->backup_files=(boolean)$backupFiles,'phpgwapi');
+			Api\Config::save_value('backup_files', $this->backup_files = (boolean)$backupFiles, 'phpgwapi');
 		}
 	}
 
@@ -321,7 +321,7 @@ 
 		'httpproxy_server_password',
 		'system_charset',
 		'usecookies',
-		'install_id',	// do not restore install_id, as that would give two systems with identical install_id
+		'install_id', // do not restore install_id, as that would give two systems with identical install_id
 	);
 
 	/**
@@ -335,20 +335,20 @@ 
 	 *
 	 * @returns An empty string or an error message in case of failure.
 	 */
-	function restore($f,$convert_to_system_charset=true,$filename='',$protect_system_config=true, $insert_n_rows=500)
+	function restore($f, $convert_to_system_charset = true, $filename = '', $protect_system_config = true, $insert_n_rows = 500)
 	{
 		@set_time_limit(0);
-		ini_set('auto_detect_line_endings',true);
+		ini_set('auto_detect_line_endings', true);
 
-		if (true) $convert_to_system_charset = true;	// enforce now utf-8 as system charset restores of old backups
+		if (true) $convert_to_system_charset = true; // enforce now utf-8 as system charset restores of old backups
 
 		if ($protect_system_config)
 		{
 			$system_config = array();
-			foreach($this->db->select(self::TABLE,'*',array(
+			foreach ($this->db->select(self::TABLE, '*', array(
 				'config_app' => 'phpgwapi',
 				'config_name' => self::$system_config,
-			),__LINE__,__FILE__) as $row)
+			), __LINE__, __FILE__) as $row)
 			{
 				$system_config[] = $row;
 			}
@@ -357,7 +357,7 @@ 
 		// as MySQL 5.7+ has sql_mode STRICT_(ALL|TRANS)_TABLES enabled by default,
 		// it will no longer restore '0000-00-00 00:00:00' in timestamps it created before,
 		// so switching strict-mode off temporary for the restore (we dont create these!)
-		if (substr($this->db->Type,0,5) == 'mysql')
+		if (substr($this->db->Type, 0, 5) == 'mysql')
 		{
 			$this->db->query("SET SESSION sql_mode=(SELECT REPLACE(REPLACE(@@sql_mode,'STRICT_ALL_TABLES',''),'STRICT_TRANS_TABLES',''))", __LINE__, __FILE__);
 		}
@@ -368,17 +368,17 @@ 
 
 
 		// drop all existing tables
-		foreach($this->adodb->MetaTables('TABLES') as $table)
+		foreach ($this->adodb->MetaTables('TABLES') as $table)
 		{
-			if ($this->system_tables && preg_match($this->system_tables,$table) ||
-				$this->egw_tables && !preg_match($this->egw_tables,$table))
+			if ($this->system_tables && preg_match($this->system_tables, $table) ||
+				$this->egw_tables && !preg_match($this->egw_tables, $table))
 			{
 				 continue;
 			}
 			$this->schema_proc->DropTable($table);
 		}
 		// it could be an old backup
-		list( , $type) = explode('.', basename($filename));
+		list(, $type) = explode('.', basename($filename));
 		$dir = $this->files_dir; // $GLOBALS['egw_info']['server']['files_dir'];
 		// we may have to clean up old backup - left overs
 		if (is_dir($dir.'/database_backup'))
@@ -391,20 +391,20 @@ 
 		$name = "";
 		$zip = NULL;
 		$_f = NULL;
-		if($type == 'zip')
+		if ($type == 'zip')
 	    {
 			// has already been verified to be available in fopen_backup
 			$zip = new ZipArchive;
-			if(($zip->open($filename)) !== TRUE)
+			if (($zip->open($filename)) !== TRUE)
 			{
 				return lang("Cant open '%1' for %2", $filename, lang("reading"))."<br>\n";
 			}
-			self::remove_dir_content($dir);  // removes the files-dir
+			self::remove_dir_content($dir); // removes the files-dir
 			$zip->extractTo($dir);
 			$_f = $f;
 			$list = $this->get_file_list($dir.'/database_backup/');
 			$name = $dir.'/database_backup/'.basename($list[0]);
-			if(!($f = fopen($name, 'rb')))
+			if (!($f = fopen($name, 'rb')))
 			{
 				return lang("Cant open '%1' for %2", $filename, lang("reading"))."<br>\n";
 			}
@@ -415,56 +415,56 @@ 
 		{
 			$this->db->insert(Api\Config::TABLE, array(
 				'config_value' => $this->schema_proc->system_charset,
-			),array(
+			), array(
 				'config_app' => 'phpgwapi',
 				'config_name' => 'system_charset',
-			),__LINE__,__FILE__);
+			), __LINE__, __FILE__);
 		}
 		// restore protected system config
 		if ($protect_system_config)
 		{
-			foreach($system_config as $row)
+			foreach ($system_config as $row)
 			{
-				$this->db->insert(self::TABLE,array('config_value'=>$row['config_value']),array(
+				$this->db->insert(self::TABLE, array('config_value'=>$row['config_value']), array(
 					'config_name' => $row['config_name'],
 					'config_app'  => $row['config_app'],
-				),__LINE__,__FILE__);
+				), __LINE__, __FILE__);
 			}
 			// check and reset cookie configuration, if it does not match current enviroment
 			// if $_SERVER[HTTP_HOST] does not end with cookiedomain --> delete cookiedomain
-			if (($cookiedomain = $this->db->select(self::TABLE,'config_value',array(
+			if (($cookiedomain = $this->db->select(self::TABLE, 'config_value', array(
 					'config_app' => 'phpgwapi',
 					'config_name' => 'cookiedomain',
-				),__LINE__,__FILE__)->fetchColumn()) && isset($_SERVER['HTTP_HOST']) &&
-				(list($hostname) = explode(':',$_SERVER['HTTP_HOST'])) &&
-				substr($hostname,-strlen($cookiedomain) !== $cookiedomain))
+				), __LINE__, __FILE__)->fetchColumn()) && isset($_SERVER['HTTP_HOST']) &&
+				(list($hostname) = explode(':', $_SERVER['HTTP_HOST'])) &&
+				substr($hostname, -strlen($cookiedomain) !== $cookiedomain))
 			{
-				$this->db->delete(self::TABLE,array(
+				$this->db->delete(self::TABLE, array(
 					'config_app' => 'phpgwapi',
 					'config_name' => 'cookiedomain',
-				),__LINE__,__FILE__);
+				), __LINE__, __FILE__);
 			}
 			// if configured webserver_url does NOT start with cookiepath --> delete cookiepath
-			if (($cookiepath = $this->db->select(self::TABLE,'config_value',array(
+			if (($cookiepath = $this->db->select(self::TABLE, 'config_value', array(
 					'config_app' => 'phpgwapi',
 					'config_name' => 'cookiepath',
-				),__LINE__,__FILE__)->fetchColumn()) &&
-				substr(parse_url($system_config['webserver_url'], PHP_URL_PATH),0,strlen($cookiepath) !== $cookiepath))
+				), __LINE__, __FILE__)->fetchColumn()) &&
+				substr(parse_url($system_config['webserver_url'], PHP_URL_PATH), 0, strlen($cookiepath) !== $cookiepath))
 			{
-				$this->db->delete(self::TABLE,array(
+				$this->db->delete(self::TABLE, array(
 					'config_app' => 'phpgwapi',
 					'config_name' => 'cookiepath',
-				),__LINE__,__FILE__);
+				), __LINE__, __FILE__);
 			}
 		}
 		// zip?
-		if($type == 'zip')
+		if ($type == 'zip')
 		{
 			fclose($f);
 			unlink($name);
 			rmdir($dir.'/database_backup');
 		}
-		if (substr($this->db->Type,0,5) != 'mysql')
+		if (substr($this->db->Type, 0, 5) != 'mysql')
 		{
 			if (!$this->db->transaction_commit())
 			{
@@ -499,26 +499,26 @@ 
 	 * @param int|string $insert_n_rows =500 how many rows to insert in one sql statement, or string with column-name used as unique key for insert
 	 * @returns int number of rows read from csv file
 	 */
-	function db_restore($f, $insert_n_rows=500)
+	function db_restore($f, $insert_n_rows = 500)
 	{
 		$convert_to_system_charset = true;
 		$table = null;
 		$n = 0;
 		$rows = array();
-		while(!feof($f))
+		while (!feof($f))
 		{
 			$line = trim(fgets($f)); ++$n;
 
 			if (empty($line)) continue;
 
-			if (substr($line,0,9) == 'version: ')
+			if (substr($line, 0, 9) == 'version: ')
 			{
 				// currenty not used: $api_version = trim(substr($line,9));
 				continue;
 			}
-			if (substr($line,0,9) == 'charset: ')
+			if (substr($line, 0, 9) == 'charset: ')
 			{
-				$charset = trim(substr($line,9));
+				$charset = trim(substr($line, 9));
 				// needed if mbstring.func_overload > 0, else eg. substr does not work with non ascii chars
 				$ini_default_charset = version_compare(PHP_VERSION, '5.6', '<') ? 'mbstring.internal_encoding' : 'default_charset';
 				@ini_set($ini_default_charset, $charset);
@@ -526,18 +526,18 @@ 
 				// check if we really need to convert the charset, as it's not perfect and can do some damage
 				if ($convert_to_system_charset && !strcasecmp($this->schema_proc->system_charset, $charset))
 				{
-					$convert_to_system_charset = false;	// no conversation necessary
+					$convert_to_system_charset = false; // no conversation necessary
 				}
 				continue;
 			}
-			if (substr($line,0,8) == 'schema: ')
+			if (substr($line, 0, 8) == 'schema: ')
 			{
 				// create the tables in the backup set
-				$this->schemas = json_php_unserialize(trim(substr($line,8)));
-				foreach($this->schemas as $table_name => $schema)
+				$this->schemas = json_php_unserialize(trim(substr($line, 8)));
+				foreach ($this->schemas as $table_name => $schema)
 				{
 					// if column is longtext in current schema, convert text to longtext, in case user already updated column
-					foreach($schema['fd'] as $col => &$def)
+					foreach ($schema['fd'] as $col => &$def)
 					{
 						if ($def['type'] == 'text' && $this->db->get_column_attribute($col, $table_name, true, 'type') == 'longtext')
 						{
@@ -549,27 +549,27 @@ 
 				}
 				continue;
 			}
-			if (substr($line,0,7) == 'table: ')
+			if (substr($line, 0, 7) == 'table: ')
 			{
 				if ($rows)	// flush pending rows of last table
 				{
 					$this->insert_multiple($table, $rows, $this->schemas[$table]);
 				}
 				$rows = array();
-				$table = substr($line,7);
+				$table = substr($line, 7);
 				if (!isset($this->schemas[$table])) $this->schemas[$table] = $this->db->get_table_definitions(true, $table);
 				$auto_id = count($this->schemas[$table]['pk']) == 1 ? $this->schemas[$table]['pk'][0] : null;
 
-				$cols = self::csv_split($line=fgets($f)); ++$n;
+				$cols = self::csv_split($line = fgets($f)); ++$n;
 				$blobs = array();
-				foreach($this->schemas[$table]['fd'] as $col => $data)
+				foreach ($this->schemas[$table]['fd'] as $col => $data)
 				{
 					if ($data['type'] == 'blob') $blobs[] = $col;
 				}
 				// check if we have an old PostgreSQL backup useing 't'/'f' for bool values
 				// --> convert them to MySQL and our new PostgreSQL format of 1/0
 				$bools = array();
-				foreach($this->schemas[$table]['fd'] as $col => $def)
+				foreach ($this->schemas[$table]['fd'] as $col => $def)
 				{
 					if ($def['type'] === 'bool') $bools[] = $col;
 				}
@@ -582,15 +582,15 @@ 
 				$import = true;
 				$data = self::csv_split($line, $cols, $blobs, $bools);
 
-				if ($table == 'egw_async' && in_array('##last-check-run##',$data))
+				if ($table == 'egw_async' && in_array('##last-check-run##', $data))
 				{
 					//echo '<p>'.lang("Line %1: '%2'<br><b>csv data does contain ##last-check-run## of table %3 ==> ignored</b>",$n,$line,$table)."</p>\n";
 					//echo 'data=<pre>'.print_r($data,true)."</pre>\n";
 					$import = false;
 				}
-				if (in_array($table,$this->exclude_tables))
+				if (in_array($table, $this->exclude_tables))
 				{
-					echo '<p><b>'.lang("Table %1 is excluded from backup and restore. Data will not be restored.",$table)."</b></p>\n";
+					echo '<p><b>'.lang("Table %1 is excluded from backup and restore. Data will not be restored.", $table)."</b></p>\n";
 					$import = false; // dont restore data of excluded tables
 				}
 				if ($import)
@@ -599,7 +599,7 @@ 
 					{
 						if ($convert_to_system_charset)
 						{
-							$data = Api\Translation::convert($data,$charset);
+							$data = Api\Translation::convert($data, $charset);
 						}
 						if ($insert_n_rows > 1)
 						{
@@ -613,27 +613,27 @@ 
 							}
 						}
 						// update existing table using given unique key in $insert_n_rows (also removing auto-id/sequence)
-						elseif(!is_numeric($insert_n_rows))
+						elseif (!is_numeric($insert_n_rows))
 						{
 							$where = array($insert_n_rows => $data[$insert_n_rows]);
 							unset($data[$insert_n_rows]);
 							if ($auto_id) unset($data[$auto_id]);
-							$this->db->insert($table,$data,$where,__LINE__,__FILE__,false,false,$this->schemas[$table]);
+							$this->db->insert($table, $data, $where, __LINE__, __FILE__, false, false, $this->schemas[$table]);
 						}
 						else
 						{
 							try {
-								$this->db->insert($table,$data,False,__LINE__,__FILE__,false,false,$this->schemas[$table]);
+								$this->db->insert($table, $data, False, __LINE__, __FILE__, false, false, $this->schemas[$table]);
 							}
-							catch(Exception\InvalidSql $e) {
+							catch (Exception\InvalidSql $e) {
 								echo "<p>".$e->getMessage()."</p>\n";
 							}
 						}
 					}
 					else
 					{
-						echo '<p>'.lang("Line %1: '%2'<br><b>csv data does not match column-count of table %3 ==> ignored</b>",$n,$line,$table)."</p>\n";
-						echo 'data=<pre>'.print_r($data,true)."</pre>\n";
+						echo '<p>'.lang("Line %1: '%2'<br><b>csv data does not match column-count of table %3 ==> ignored</b>", $n, $line, $table)."</p>\n";
+						echo 'data=<pre>'.print_r($data, true)."</pre>\n";
 					}
 				}
 			}
@@ -643,14 +643,14 @@ 
 			$this->insert_multiple($table, $rows, $this->schemas[$table]);
 		}
 		// updated the sequences, if the DB uses them
-		foreach($this->schemas as $table => $schema)
+		foreach ($this->schemas as $table => $schema)
 		{
-			foreach($schema['fd'] as $column => $definition)
+			foreach ($schema['fd'] as $column => $definition)
 			{
 				if ($definition['type'] == 'auto')
 				{
-					$this->schema_proc->UpdateSequence($table,$column);
-					break;	// max. one per table
+					$this->schema_proc->UpdateSequence($table, $column);
+					break; // max. one per table
 				}
 			}
 		}
@@ -673,15 +673,15 @@ 
 		try {
 			$this->db->insert($table, $rows, False, __LINE__, __FILE__, false, false, $schema);
 		}
-		catch(Exception\InvalidSql $e)
+		catch (Exception\InvalidSql $e)
 		{
 			// try inserting them one by one, ignoring doublicates
-			foreach($rows as $data)
+			foreach ($rows as $data)
 			{
 				try {
 					$this->db->insert($table, $data, False, __LINE__, __FILE__, false, false, $schema);
 				}
-				catch(Exception\InvalidSql $e) {
+				catch (Exception\InvalidSql $e) {
 					echo "<p>".$e->getMessage()."</p>\n";
 				}
 			}
@@ -696,11 +696,11 @@ 
 	private static function remove_dir_content($dir)
 	{
 		$list = scandir($dir);
-		while($file = $list[0])
+		while ($file = $list[0])
 		{
-			if(is_dir($file) && $file != '.' && $file != '..')
+			if (is_dir($file) && $file != '.' && $file != '..')
 			    self::remove_dir_content($dir.'/'.$file);
-			if(is_file($file) && $file != '.' && $file != '..')
+			if (is_file($file) && $file != '.' && $file != '..')
 			    unlink($dir.'/'.$file);
 			array_shift($list);
 		}
@@ -725,7 +725,7 @@ 
 	 * @param array $bools =array() bool columns, values might be 't'/'f' for old PostgreSQL backups
 	 * @return array
 	 */
-	public static function csv_split($line, $keys=null, $blobs=array(), $bools=array())
+	public static function csv_split($line, $keys = null, $blobs = array(), $bools = array())
 	{
 		if (function_exists('str_getcsv'))	// php5.3+
 		{
@@ -738,9 +738,9 @@ 
 				'\\n'    => "\n",
 				'\\r'    => "\r")), ',', '"', '\0');
 			// replace NULL-token again with 'NULL', 'NULL' with null and BACKSLASH-token with a backslash
-			foreach($fields as &$field)
+			foreach ($fields as &$field)
 			{
-				switch($field)
+				switch ($field)
 				{
 					case self::NULL_TOKEN:
 						$field = 'NULL';
@@ -757,7 +757,7 @@ 
 			{
 				$fields = array_combine($keys, $fields);
 				// base64-decode blob columns, if they are base64 encoded
-				foreach($blobs as $key)
+				foreach ($blobs as $key)
 				{
 					if (!is_null($fields[$key]) && ($tmp = base64_decode($fields[$key], true)) !== false)
 					{
@@ -765,7 +765,7 @@ 
 					}
 				}
 				// decode bool columns, they might be 't'/'f' for old PostgreSQL backups
-				foreach($bools as $key)
+				foreach ($bools as $key)
 				{
 					if (isset($fields[$key]))	// do NOT replace NULL with false/0
 					{
@@ -776,11 +776,11 @@ 
 			return $fields;
 		}
 		// pre 5.3 implementation
-		$fields = explode(',',trim($line));
+		$fields = explode(',', trim($line));
 
 		$str_pending = False;
 		$n = 0;
-		foreach($fields as $field)
+		foreach ($fields as $field)
 		{
 			if ($str_pending !== False)
 			{
@@ -791,12 +791,12 @@ 
 
 			if ($field[0] == '"')
 			{
-				if (substr($field,-1) !== '"' || $field === '"' || !preg_match('/[^\\\\]+(\\\\\\\\)*"$/',$field))
+				if (substr($field, -1) !== '"' || $field === '"' || !preg_match('/[^\\\\]+(\\\\\\\\)*"$/', $field))
 				{
 					$str_pending = $field;
 					continue;
 				}
-				$arr[$key] = str_replace(self::BACKSLASH_TOKEN,'\\',str_replace(array('\\\\','\\n','\\r','\\"'),array(self::BACKSLASH_TOKEN,"\n","\r",'"'),substr($field,1,-1)));
+				$arr[$key] = str_replace(self::BACKSLASH_TOKEN, '\\', str_replace(array('\\\\', '\\n', '\\r', '\\"'), array(self::BACKSLASH_TOKEN, "\n", "\r", '"'), substr($field, 1, -1)));
 			}
 			elseif ($keys && strlen($field) > 26)
 			{
@@ -818,7 +818,7 @@ 
 	/**
 	 * escape data for csv
 	 */
-	public static function escape_data(&$data,$col,$defs)
+	public static function escape_data(&$data, $col, $defs)
 	{
 		if (is_null($data))
 		{
@@ -826,7 +826,7 @@ 
 		}
 		else
 		{
-			switch($defs[$col]['type'])
+			switch ($defs[$col]['type'])
 			{
 				case 'int':
 				case 'auto':
@@ -841,7 +841,7 @@ 
 					$data = (int)Api\Db::from_bool($data);
 					break;
 				default:
-					$data = '"'.str_replace(array('\\',"\n","\r",'"'),array('\\\\','\\n','\\r','\\"'),$data).'"';
+					$data = '"'.str_replace(array('\\', "\n", "\r", '"'), array('\\\\', '\\n', '\\r', '\\"'), $data).'"';
 					break;
 			}
 		}
@@ -860,7 +860,7 @@ 
 	 *	default of null limits memory usage if there is no primary key, by locking table and use ROW_CHUCK
 	 * @todo use https://github.com/maennchen/ZipStream-PHP to not assemble all files in memmory
 	 */
-	function backup($f, $lock_table=null)
+	function backup($f, $lock_table = null)
 	{
 		//echo "function backup($f)<br>";	// !
 		@set_time_limit(0);
@@ -876,15 +876,15 @@ 
 		$name = $this->backup_dir.'/db_backup-'.date('YmdHi');
 		$filename = $name.'.zip';
 		$zippresent = false;
-		if(class_exists('ZipArchive') && $this->backup_files)
+		if (class_exists('ZipArchive') && $this->backup_files)
 		{
 			$zip = new ZipArchive;
-			if(is_object($zip))
+			if (is_object($zip))
 			{
 				$zippresent = true;
 				//echo '-> is_object($zip); '.$filename.'<br>';	// !
 				$res = $zip->open($filename, ZipArchive::CREATE);
-				if($res !== TRUE)
+				if ($res !== TRUE)
 				{
 					//echo '   -> !$res<br>';	// !
 					return lang("Cant open '%1' for %2", $filename, lang("writing"))."<br>\n";
@@ -892,22 +892,22 @@ 
 				$file_list = $this->get_file_list($dir);
 			}
 		}
-		fwrite($f,"EGroupware backup from ".date('Y-m-d H:i:s')."\n\n");
+		fwrite($f, "EGroupware backup from ".date('Y-m-d H:i:s')."\n\n");
 
-		fwrite($f,"version: $this->api_version\n\n");
+		fwrite($f, "version: $this->api_version\n\n");
 
-		fwrite($f,"charset: $this->charset\n\n");
+		fwrite($f, "charset: $this->charset\n\n");
 
-		$this->schema_backup($f);	// add the schema in a human readable form too
+		$this->schema_backup($f); // add the schema in a human readable form too
 
-		fwrite($f,"\nschema: ".json_encode($this->schemas)."\n");
+		fwrite($f, "\nschema: ".json_encode($this->schemas)."\n");
 
 		// let apps know that backup is about to start
 		Api\Hooks::process('backup_starts', array(), true);
 
-		foreach($this->schemas as $table => $schema)
+		foreach ($this->schemas as $table => $schema)
 		{
-			if (in_array($table,$this->exclude_tables)) continue;	// dont backup
+			if (in_array($table, $this->exclude_tables)) continue; // dont backup
 
 			// do we have a primary key?
 			// --> use it to order and limit rows, to kope with rows being added during backup
@@ -923,7 +923,7 @@ 
 			do {
 				$num_rows = 0;
 				// querying only chunks for 10000 rows, to not run into memory limit on huge tables
-				foreach($this->db->select($table, '*',
+				foreach ($this->db->select($table, '*',
 					// limit by maximum primary key already received
 					empty($pk) || !$max ? false : $pk.' > '.$this->db->quote($max, $schema['fd'][$pk]['type']),
 					__LINE__, __FILE__,
@@ -934,15 +934,15 @@ 
 					false, self::ROW_CHUNK) as $row)
 				{
 					if (!empty($pk)) $max = $row[$pk];
-					if ($total === 0) fwrite($f,"\ntable: $table\n".implode(',',array_keys($row))."\n");
+					if ($total === 0) fwrite($f, "\ntable: $table\n".implode(',', array_keys($row))."\n");
 
 					array_walk($row, array(__CLASS__, 'escape_data'), $schema['fd']);
-					fwrite($f,implode(',',$row)."\n");
+					fwrite($f, implode(',', $row)."\n");
 					++$total;
 					++$num_rows;
 				}
 			}
-			while((!empty($pk) || $lock_table !== false) && !($total % self::ROW_CHUNK) && $num_rows);
+			while ((!empty($pk) || $lock_table !== false) && !($total % self::ROW_CHUNK) && $num_rows);
 
 			if (!$pk) $this->db->rollback_lock($table);
 		}
@@ -950,7 +950,7 @@ 
 		// let apps know that backup is finished
 		Api\Hooks::process('backup_finished', array(), true);
 
-		if(!$zippresent)  // save without files
+		if (!$zippresent)  // save without files
 		{
 			if ($this->backup_files)
 			{
@@ -965,16 +965,16 @@ 
 		//echo $name.'<br>';
 		$zip->addFile($name, 'database_backup/'.basename($name));
 		$count = 1;
-		foreach($file_list as $file)
+		foreach ($file_list as $file)
 		{
 			//echo substr($file,strlen($dir)+1).'<br>';
 			//echo $file.'<br>';
-			$zip->addFile($file,substr($file,strlen($dir)+1));//,substr($file);
-			if(($count++) == 100) { // the file descriptor limit
+			$zip->addFile($file, substr($file, strlen($dir) + 1)); //,substr($file);
+			if (($count++) == 100) { // the file descriptor limit
 				$zip->close();
-				if(($zip = new ZipArchive())) {
+				if (($zip = new ZipArchive())) {
 					$zip->open($filename);
-					$count =0;
+					$count = 0;
 				}
 			}
 		}
@@ -997,17 +997,17 @@ 
 	{
 		//chdir($f);
 		//echo "Processing $f <br>";
-		if ($path_name =='') $path_name = $f;
+		if ($path_name == '') $path_name = $f;
 		$tlist = scandir($f);
 		$list = array();
 		$i = $cnt;
-		while($file = $tlist[0]) // remove all '.' and '..' and transfer to $list
+		while ($file = $tlist[0]) // remove all '.' and '..' and transfer to $list
 		{
-			if($file == '.' || $file == '..')
+			if ($file == '.' || $file == '..')
 			{
 				array_shift($tlist);
 			}
-			elseif ($file == 'debug.txt' && stripos($f,'activesync')!==false)
+			elseif ($file == 'debug.txt' && stripos($f, 'activesync') !== false)
 			{
 				// skip activesync debug.txt on backupFiles
 				//error_log(__METHOD__.__LINE__.'->'.$f.'/'.$file);
@@ -1015,7 +1015,7 @@ 
 			}
 			else
 			{
-				if(is_dir($f.'/'.$file))
+				if (is_dir($f.'/'.$file))
 				{
 					$nlist = $this->get_file_list($f.'/'.$file, $i);
 					$list += $nlist;
@@ -1036,12 +1036,12 @@ 
 	 *
 	 * @param resource|boolean $f
 	 */
-	function schema_backup($f=False)
+	function schema_backup($f = False)
 	{
-		foreach($this->adodb->MetaTables('TABLES') as $table)
+		foreach ($this->adodb->MetaTables('TABLES') as $table)
 		{
-			if ($this->system_tables && preg_match($this->system_tables,$table) ||
-				$this->egw_tables && !preg_match($this->egw_tables,$table))
+			if ($this->system_tables && preg_match($this->system_tables, $table) ||
+				$this->egw_tables && !preg_match($this->egw_tables, $table))
 			{
 				continue;
 			}
@@ -1060,17 +1060,17 @@ 
 			}
 		}
 		$def = "\t\$phpgw_baseline = ";
-		$def .= self::write_array($this->schemas,1);
+		$def .= self::write_array($this->schemas, 1);
 		$def .= ";\n";
 
 		if ($f)
 		{
-			fwrite($f,$def);
+			fwrite($f, $def);
 		}
 		else
 		{
 			$def = "<?php\n\t/* EGroupware schema-backup from ".date('Y-m-d H:i:s')." */\n\n".$def;
-			Api\Header\Content::type('schema-backup-'.date('YmdHi').'.inc.php','text/plain',bytes($def));
+			Api\Header\Content::type('schema-backup-'.date('YmdHi').'.inc.php', 'text/plain', bytes($def));
 			echo $def;
 		}
 	}
@@ -1080,9 +1080,9 @@ 
 	 *
 	 * copied from etemplate/inc/class.db_tools.inc.php
 	 */
-	private static function write_array($arr,$depth,$parent='')
+	private static function write_array($arr, $depth, $parent = '')
 	{
-		if (in_array($parent,array('pk','fk','ix','uc')))
+		if (in_array($parent, array('pk', 'fk', 'ix', 'uc')))
 		{
 			$depth = 0;
 		}
@@ -1098,7 +1098,7 @@ 
 		$def = "array($tabs".($tabs ? "\t" : '');
 
 		$n = 0;
-		foreach($arr as $key => $val)
+		foreach ($arr as $key => $val)
 		{
 			if (!is_int($key))
 			{
@@ -1106,7 +1106,7 @@ 
 			}
 			if (is_array($val))
 			{
-				$def .= self::write_array($val,$parent == 'fd' ? 0 : $depth,$key);
+				$def .= self::write_array($val, $parent == 'fd' ? 0 : $depth, $key);
 			}
 			else
 			{
@@ -1119,7 +1119,7 @@ 
 					$def .= "'$val'";
 				}
 			}
-			if ($n < count($arr)-1)
+			if ($n < count($arr) - 1)
 			{
 				$def .= ",$tabs".($tabs ? "\t" : '');
 			}

Braces +127 added lines, -35 removed lines

@@ -101,25 +101,34 @@ 
 		{
 			$GLOBALS['egw_setup']->loaddb();	// we run inside setup, but db object is not loaded
 		}
-		if (isset($GLOBALS['egw_setup']->oProc) && is_object($GLOBALS['egw_setup']->oProc))	// schema_proc already instanciated, use it
+		if (isset($GLOBALS['egw_setup']->oProc) && is_object($GLOBALS['egw_setup']->oProc))
+		{
+			// schema_proc already instanciated, use it
 		{
 			$this->schema_proc = $GLOBALS['egw_setup']->oProc;
 		}
+		}
 		else
 		{
 			$this->schema_proc = new Api\Db\Schema();
 		}
 
 		$this->db = $this->schema_proc->m_odb;
-		if (!$this->db->Link_ID) $this->db->connect();
+		if (!$this->db->Link_ID)
+		{
+			$this->db->connect();
+		}
 		$this->adodb = $this->db->Link_ID;
-		if (isset($GLOBALS['egw_setup']) && is_object($GLOBALS['egw_setup']))		// called from setup
+		if (isset($GLOBALS['egw_setup']) && is_object($GLOBALS['egw_setup']))
+		{
+			// called from setup
 		{
 			if ($GLOBALS['egw_setup']->config_table && $GLOBALS['egw_setup']->table_exist(array($GLOBALS['egw_setup']->config_table)))
 			{
 				if (!($this->files_dir = $this->db->query("SELECT config_value FROM {$GLOBALS['egw_setup']->config_table} WHERE config_app='phpgwapi' AND config_name='files_dir'",__LINE__,__FILE__)->fetchColumn()))
 				{
 					error_log(__METHOD__."->"."No files Directory set/found");
+		}
 				}
 				if (!($this->backup_dir = $this->db->query("SELECT config_value FROM {$GLOBALS['egw_setup']->config_table} WHERE config_app='phpgwapi' AND config_name='backup_dir'",__LINE__,__FILE__)->fetchColumn()))
 				{
@@ -133,7 +142,10 @@ 
 				// backup files too
 				$this->backup_files = (bool)$this->db->query("SELECT config_value FROM {$GLOBALS['egw_setup']->config_table} WHERE config_app='phpgwapi' AND config_name='backup_files'",__LINE__,__FILE__)->fetchColumn();
 			}
-			if (!$this->charset) $this->charset = 'utf-8';
+			if (!$this->charset)
+			{
+				$this->charset = 'utf-8';
+			}
 		}
 		else	// called from eGW
 		{
@@ -230,8 +242,7 @@ 
 		if(!($f = fopen("compress.bzip2://$name.bz2", $mode)) &&
 	 		!($f = fopen("compress.zlib://$name.gz",$mode)) &&
  		 	!($f = fopen($name,$mode))
-		)
-		{
+		) {
 			//echo '-> !($f = fopen("compress.bzip2://$name.bz2", $mode))<br>';	// !
 			$lang_mode = $reading ? lang("reading") : lang("writing");
 			return lang("Cant open '%1' for %2", $name, $lang_mode)."<br>";
@@ -267,16 +278,22 @@ 
 				$files[filectime($this->backup_dir.'/'.$file)] = $file;
 			}
 		}
-		if ($handle) closedir($handle);
+		if ($handle)
+		{
+			closedir($handle);
+		}
 
 		/* Sort the files by ctime. */
 		krsort($files);
 		$count = 0;
 		foreach($files as $file)
 		{
-			if ($count >= $this->backup_mincount)//
+			if ($count >= $this->backup_mincount)
+			{
+				//
 			{
 				$ret = unlink($this->backup_dir.'/'.$file);
+			}
 				if (($ret) && (is_array($files_return)))
 				{
 					array_push($files_return, $file);
@@ -340,7 +357,11 @@ 
 		@set_time_limit(0);
 		ini_set('auto_detect_line_endings',true);
 
-		if (true) $convert_to_system_charset = true;	// enforce now utf-8 as system charset restores of old backups
+		if (true)
+		{
+			$convert_to_system_charset = true;
+		}
+		// enforce now utf-8 as system charset restores of old backups
 
 		if ($protect_system_config)
 		{
@@ -392,7 +413,7 @@ 
 		$zip = NULL;
 		$_f = NULL;
 		if($type == 'zip')
-	    {
+		{
 			// has already been verified to be available in fopen_backup
 			$zip = new ZipArchive;
 			if(($zip->open($filename)) !== TRUE)
@@ -411,7 +432,9 @@ 
 		}
 		$this->db_restore($f, $insert_n_rows);
 
-		if ($convert_to_system_charset)	// store the changed charset
+		if ($convert_to_system_charset)
+		{
+			// store the changed charset
 		{
 			$this->db->insert(Api\Config::TABLE, array(
 				'config_value' => $this->schema_proc->system_charset,
@@ -420,6 +443,7 @@ 
 				'config_name' => 'system_charset',
 			),__LINE__,__FILE__);
 		}
+		}
 		// restore protected system config
 		if ($protect_system_config)
 		{
@@ -509,7 +533,10 @@ 
 		{
 			$line = trim(fgets($f)); ++$n;
 
-			if (empty($line)) continue;
+			if (empty($line))
+			{
+				continue;
+			}
 
 			if (substr($line,0,9) == 'version: ')
 			{
@@ -551,35 +578,53 @@ 
 			}
 			if (substr($line,0,7) == 'table: ')
 			{
-				if ($rows)	// flush pending rows of last table
+				if ($rows)
+				{
+					// flush pending rows of last table
 				{
 					$this->insert_multiple($table, $rows, $this->schemas[$table]);
 				}
+				}
 				$rows = array();
 				$table = substr($line,7);
-				if (!isset($this->schemas[$table])) $this->schemas[$table] = $this->db->get_table_definitions(true, $table);
+				if (!isset($this->schemas[$table]))
+				{
+					$this->schemas[$table] = $this->db->get_table_definitions(true, $table);
+				}
 				$auto_id = count($this->schemas[$table]['pk']) == 1 ? $this->schemas[$table]['pk'][0] : null;
 
 				$cols = self::csv_split($line=fgets($f)); ++$n;
 				$blobs = array();
 				foreach($this->schemas[$table]['fd'] as $col => $data)
 				{
-					if ($data['type'] == 'blob') $blobs[] = $col;
+					if ($data['type'] == 'blob')
+					{
+						$blobs[] = $col;
+					}
 				}
 				// check if we have an old PostgreSQL backup useing 't'/'f' for bool values
 				// --> convert them to MySQL and our new PostgreSQL format of 1/0
 				$bools = array();
 				foreach($this->schemas[$table]['fd'] as $col => $def)
 				{
-					if ($def['type'] === 'bool') $bools[] = $col;
+					if ($def['type'] === 'bool')
+					{
+						$bools[] = $col;
+					}
 				}
 
-				if (feof($f)) break;
+				if (feof($f))
+				{
+					break;
+				}
 				continue;
 			}
-			if ($table)	// do we already reached the data part
+			if ($table)
+			{
+				// do we already reached the data part
 			{
 				$import = true;
+			}
 				$data = self::csv_split($line, $cols, $blobs, $bools);
 
 				if ($table == 'egw_async' && in_array('##last-check-run##',$data))
@@ -617,7 +662,10 @@ 
 						{
 							$where = array($insert_n_rows => $data[$insert_n_rows]);
 							unset($data[$insert_n_rows]);
-							if ($auto_id) unset($data[$auto_id]);
+							if ($auto_id)
+							{
+								unset($data[$auto_id]);
+							}
 							$this->db->insert($table,$data,$where,__LINE__,__FILE__,false,false,$this->schemas[$table]);
 						}
 						else
@@ -638,10 +686,13 @@ 
 				}
 			}
 		}
-		if ($rows)	// flush pending rows
+		if ($rows)
+		{
+			// flush pending rows
 		{
 			$this->insert_multiple($table, $rows, $this->schemas[$table]);
 		}
+		}
 		// updated the sequences, if the DB uses them
 		foreach($this->schemas as $table => $schema)
 		{
@@ -699,9 +750,13 @@ 
 		while($file = $list[0])
 		{
 			if(is_dir($file) && $file != '.' && $file != '..')
-			    self::remove_dir_content($dir.'/'.$file);
+			{
+						    self::remove_dir_content($dir.'/'.$file);
+			}
 			if(is_file($file) && $file != '.' && $file != '..')
-			    unlink($dir.'/'.$file);
+			{
+						    unlink($dir.'/'.$file);
+			}
 			array_shift($list);
 		}
 		//rmdir($dir);  // dont remove own dir
@@ -727,7 +782,9 @@ 
 	 */
 	public static function csv_split($line, $keys=null, $blobs=array(), $bools=array())
 	{
-		if (function_exists('str_getcsv'))	// php5.3+
+		if (function_exists('str_getcsv'))
+		{
+			// php5.3+
 		{
 			// we need to take care of literal "NULL" values, replacing them we a special token as str_getcsv removes enclosures around strings
 			// str_getcsv uses '""' for '"' instead of '\\"' and does not unescape '\\n', '\\r' or '\\\\' (two backslashes)
@@ -737,6 +794,7 @@ 
 				'\\"'    => '""',
 				'\\n'    => "\n",
 				'\\r'    => "\r")), ',', '"', '\0');
+		}
 			// replace NULL-token again with 'NULL', 'NULL' with null and BACKSLASH-token with a backslash
 			foreach($fields as &$field)
 			{
@@ -753,9 +811,12 @@ 
 						break;
 				}
 			}
-			if ($keys)	// if string keys are to be used --> combine keys and values
+			if ($keys)
+			{
+				// if string keys are to be used --> combine keys and values
 			{
 				$fields = array_combine($keys, $fields);
+			}
 				// base64-decode blob columns, if they are base64 encoded
 				foreach($blobs as $key)
 				{
@@ -767,10 +828,13 @@ 
 				// decode bool columns, they might be 't'/'f' for old PostgreSQL backups
 				foreach($bools as $key)
 				{
-					if (isset($fields[$key]))	// do NOT replace NULL with false/0
+					if (isset($fields[$key]))
+					{
+						// do NOT replace NULL with false/0
 					{
 						$fields[$key] = Api\Db::from_bool($fields[$key]);
 					}
+					}
 				}
 			}
 			return $fields;
@@ -907,7 +971,11 @@ 
 
 		foreach($this->schemas as $table => $schema)
 		{
-			if (in_array($table,$this->exclude_tables)) continue;	// dont backup
+			if (in_array($table,$this->exclude_tables))
+			{
+				continue;
+			}
+			// dont backup
 
 			// do we have a primary key?
 			// --> use it to order and limit rows, to kope with rows being added during backup
@@ -933,8 +1001,14 @@ 
 					empty($pk) ? '' : 'ORDER BY '.$this->db->name_quote($pk).' ASC',
 					false, self::ROW_CHUNK) as $row)
 				{
-					if (!empty($pk)) $max = $row[$pk];
-					if ($total === 0) fwrite($f,"\ntable: $table\n".implode(',',array_keys($row))."\n");
+					if (!empty($pk))
+					{
+						$max = $row[$pk];
+					}
+					if ($total === 0)
+					{
+						fwrite($f,"\ntable: $table\n".implode(',',array_keys($row))."\n");
+					}
 
 					array_walk($row, array(__CLASS__, 'escape_data'), $schema['fd']);
 					fwrite($f,implode(',',$row)."\n");
@@ -944,21 +1018,30 @@ 
 			}
 			while((!empty($pk) || $lock_table !== false) && !($total % self::ROW_CHUNK) && $num_rows);
 
-			if (!$pk) $this->db->rollback_lock($table);
+			if (!$pk)
+			{
+				$this->db->rollback_lock($table);
+			}
 		}
 
 		// let apps know that backup is finished
 		Api\Hooks::process('backup_finished', array(), true);
 
-		if(!$zippresent)  // save without files
+		if(!$zippresent)
+		{
+			// save without files
 		{
 			if ($this->backup_files)
 			{
 				echo '<center>'.lang("Cant open %1, needs ZipArchive", $name)."<br>\n".'</center>';
+		}
 			}
 
 		    fclose($f);
-		    if (file_exists($name)) unlink($name);
+		    if (file_exists($name))
+		    {
+		    	unlink($name);
+		    }
 			return TRUE;
 		}
 		// save files ....
@@ -970,9 +1053,12 @@ 
 			//echo substr($file,strlen($dir)+1).'<br>';
 			//echo $file.'<br>';
 			$zip->addFile($file,substr($file,strlen($dir)+1));//,substr($file);
-			if(($count++) == 100) { // the file descriptor limit
+			if(($count++) == 100)
+			{
+// the file descriptor limit
 				$zip->close();
-				if(($zip = new ZipArchive())) {
+				if(($zip = new ZipArchive()))
+				{
 					$zip->open($filename);
 					$count =0;
 				}
@@ -997,15 +1083,21 @@ 
 	{
 		//chdir($f);
 		//echo "Processing $f <br>";
-		if ($path_name =='') $path_name = $f;
+		if ($path_name =='')
+		{
+			$path_name = $f;
+		}
 		$tlist = scandir($f);
 		$list = array();
 		$i = $cnt;
-		while($file = $tlist[0]) // remove all '.' and '..' and transfer to $list
+		while($file = $tlist[0])
+		{
+			// remove all '.' and '..' and transfer to $list
 		{
 			if($file == '.' || $file == '..')
 			{
 				array_shift($tlist);
+		}
 			}
 			elseif ($file == 'debug.txt' && stripos($f,'activesync')!==false)
 			{

api/src/Db/Exception.php

Spacing +2 added lines, -2 removed lines

@@ -27,10 +27,10 @@
 	 * @param string $msg =null message, default "Database error!"
 	 * @param int $code =100
 	 */
-	function __construct($msg=null,$code=100)
+	function __construct($msg = null, $code = 100)
 	{
 		if (is_null($msg)) $msg = lang('Database error!');
 
-		parent::__construct($msg,$code);
+		parent::__construct($msg, $code);
 	}
 }

Braces +4 added lines, -1 removed lines

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