EGroupware / egroupware

api/src/Db/Backup.php

Doc Comments +3 added lines, -2 removed lines

@@ -496,7 +496,7 @@ 
 	 * Restore data from a (compressed) csv file
 	 *
 	 * @param resource $f file opened with fopen for reading
-	 * @param int|string $insert_n_rows =10 how many rows to insert in one sql statement, or string with column-name used as unique key for insert
+	 * @param integer $insert_n_rows =10 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=10)
@@ -1044,7 +1044,7 @@ 
 	/**
 	 * Backup all schemas in the form of a setup/tables_current.inc.php file
 	 *
-	 * @param resource|boolean $f
+	 * @param resource $f
 	 */
 	function schema_backup($f=False)
 	{
@@ -1089,6 +1089,7 @@ 
 	 * Dump an array as php source
 	 *
 	 * copied from etemplate/inc/class.db_tools.inc.php
+	 * @param integer $depth
 	 */
 	private static function write_array($arr,$depth,$parent='')
 	{

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 ....

Braces +123 added lines, -34 removed lines

@@ -102,25 +102,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()))
 				{
@@ -134,7 +143,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
 		{
@@ -231,8 +243,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>";
@@ -268,16 +279,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);
@@ -341,7 +358,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)
 		{
@@ -393,7 +414,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)
@@ -412,7 +433,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,
@@ -421,6 +444,7 @@ 
 				'config_name' => 'system_charset',
 			),__LINE__,__FILE__);
 		}
+		}
 		// restore protected system config
 		if ($protect_system_config)
 		{
@@ -510,7 +534,10 @@ 
 		{
 			$line = trim(fgets($f)); ++$n;
 
-			if (empty($line)) continue;
+			if (empty($line))
+			{
+				continue;
+			}
 
 			if (substr($line,0,9) == 'version: ')
 			{
@@ -563,30 +590,45 @@ 
 			}
 			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 ($convert_to_system_charset && !$this->db->capabilities['client_encoding'])
@@ -599,9 +641,12 @@ 
 					}
 				}
 			}
-			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))
@@ -637,7 +682,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
@@ -658,10 +706,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)
 		{
@@ -719,9 +770,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
@@ -747,7 +802,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)
@@ -757,6 +814,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)
 			{
@@ -773,9 +831,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)
 				{
@@ -924,7 +985,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
@@ -950,8 +1015,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");
@@ -961,21 +1032,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 ....
@@ -987,9 +1067,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;
 				}
@@ -1014,15 +1097,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)
 			{

Spacing +155 added lines, -155 removed lines

@@ -51,9 +51,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
 	);
 	/**
@@ -85,7 +85,7 @@ 
 	 *
 	 * @var boolean
 	 */
-	var $backup_files = false ;
+	var $backup_files = false;
 	/**
 	 * Reference to schema_proc's Api\Db object
 	 *
@@ -100,7 +100,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
 		{
@@ -118,21 +118,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';
 		}
@@ -147,7 +147,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))
@@ -163,7 +163,7 @@ 
 		{
 			mkdir($this->backup_dir);
 		}
-		switch($this->db->Type)
+		switch ($this->db->Type)
 		{
 			case 'sapdb':
 			case 'maxdb':
@@ -180,7 +180,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)
@@ -189,27 +189,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");
@@ -217,10 +217,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");
@@ -228,9 +228,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>';	// !
@@ -257,13 +257,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;
 			}
@@ -273,7 +273,7 @@ 
 		/* Sort the files by ctime. */
 		krsort($files);
 		$count = 0;
-		foreach($files as $file)
+		foreach ($files as $file)
 		{
 			if ($count >= $this->backup_mincount)//
 			{
@@ -283,7 +283,7 @@ 
 					array_push($files_return, $file);
 				}
 			}
-			$count ++;
+			$count++;
 		}
 	}
 
@@ -293,13 +293,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');
 		}
 	}
 
@@ -322,7 +322,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
 	);
 
 	/**
@@ -336,20 +336,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=10)
+	function restore($f, $convert_to_system_charset = true, $filename = '', $protect_system_config = true, $insert_n_rows = 10)
 	{
 		@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;
 			}
@@ -358,7 +358,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__);
 		}
@@ -369,17 +369,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'))
@@ -392,20 +392,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";
 			}
@@ -422,46 +422,46 @@ 
 		{
 			$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__);
 			}
 		}
 		// restore original Halt_On_Error state (if changed)
@@ -470,13 +470,13 @@ 
 			$this->db->Halt_On_Error = $backup_db_halt_on_error;
 		}
 		// 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())
 			{
@@ -511,26 +511,26 @@ 
 	 * @param int|string $insert_n_rows =10 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=10)
+	function db_restore($f, $insert_n_rows = 10)
 	{
 		$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: ')
 			{
-				$api_version = trim(substr($line,9));
+				$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);
@@ -538,29 +538,29 @@ 
 				// 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
 				}
 				// set the DB's client encoding (for mysql only if api_version >= 1.0.1.019)
 				if ((!$convert_to_system_charset || $this->db->capabilities['client_encoding']) &&
-					(substr($this->db->Type,0,5) != 'mysql' || !is_object($GLOBALS['egw_setup']) ||
-					$api_version && !$GLOBALS['egw_setup']->alessthanb($api_version,'1.0.1.019')))
+					(substr($this->db->Type, 0, 5) != 'mysql' || !is_object($GLOBALS['egw_setup']) ||
+					$api_version && !$GLOBALS['egw_setup']->alessthanb($api_version, '1.0.1.019')))
 				{
 					$this->db->Link_ID->SetCharSet($charset);
 					if (!$convert_to_system_charset)
 					{
-						$this->schema_proc->system_charset = $charset;	// so schema_proc uses it for the creation of the tables
+						$this->schema_proc->system_charset = $charset; // so schema_proc uses it for the creation of the tables
 					}
 				}
 				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')
 						{
@@ -572,27 +572,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;
 				}
@@ -615,15 +615,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)
@@ -632,7 +632,7 @@ 
 					{
 						if ($convert_to_system_charset && !$this->db->capabilities['client_encoding'])
 						{
-							$data = Api\Translation::convert($data,$charset);
+							$data = Api\Translation::convert($data, $charset);
 						}
 						if ($insert_n_rows > 1)
 						{
@@ -644,27 +644,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";
 					}
 				}
 			}
@@ -674,14 +674,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
 				}
 			}
 		}
@@ -704,15 +704,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";
 				}
 			}
@@ -727,11 +727,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);
 		}
@@ -756,7 +756,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+
 		{
@@ -769,9 +769,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';
@@ -788,7 +788,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)
 					{
@@ -796,7 +796,7 @@ 
 					}
 				}
 				// decode bool columns, they might be 't'/'f' for old PostgreSQL backups
-				foreach($bools as $key)
+				foreach ($bools as $key)
 				{
 					$fields[$key] = Api\Db::from_bool($fields[$key]);
 				}
@@ -804,11 +804,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)
 			{
@@ -819,12 +819,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)
 			{
@@ -846,7 +846,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))
 		{
@@ -854,7 +854,7 @@ 
 		}
 		else
 		{
-			switch($defs[$col]['type'])
+			switch ($defs[$col]['type'])
 			{
 				case 'int':
 				case 'auto':
@@ -869,7 +869,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;
 			}
 		}
@@ -888,7 +888,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);
@@ -904,15 +904,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";
@@ -920,22 +920,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
@@ -951,7 +951,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__,
@@ -962,15 +962,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);
 		}
@@ -978,7 +978,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)
 			{
@@ -993,16 +993,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;
 				}
 			}
 		}
@@ -1025,17 +1025,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);
@@ -1043,7 +1043,7 @@ 
 			}
 			else
 			{
-				if(is_dir($f.'/'.$file))
+				if (is_dir($f.'/'.$file))
 				{
 					$nlist = $this->get_file_list($f.'/'.$file, $i);
 					$list += $nlist;
@@ -1064,12 +1064,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;
 			}
@@ -1088,17 +1088,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;
 		}
 	}
@@ -1108,9 +1108,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;
 		}
@@ -1126,7 +1126,7 @@ 
 		$def = "array($tabs".($tabs ? "\t" : '');
 
 		$n = 0;
-		foreach($arr as $key => $val)
+		foreach ($arr as $key => $val)
 		{
 			if (!is_int($key))
 			{
@@ -1134,7 +1134,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
 			{
@@ -1147,7 +1147,7 @@ 
 					$def .= "'$val'";
 				}
 			}
-			if ($n < count($arr)-1)
+			if ($n < count($arr) - 1)
 			{
 				$def .= ",$tabs".($tabs ? "\t" : '');
 			}

api/src/Db/CallbackIterator.php

Doc Comments +2 added lines, -2 removed lines

@@ -94,10 +94,10 @@
 	/**
 	 * Constructor
 	 *
-	 * @param Traversable $rs
+	 * @param \Traversable $rs
 	 * @param callback $callback
 	 * @param array $params =array() additional parameters, row is always first parameter
-	 * @param $key_callback =null optional callback, if you want different keys
+	 * @param \Closure $key_callback =null optional callback, if you want different keys
 	 */
 	public function __construct(\Traversable $rs, $callback, $params=array(), $key_callback=null)
 	{

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

Doc Comments +2 added lines, -2 removed lines

@@ -185,7 +185,7 @@ 
 	* @deprecated not used anymore as it costs to much performance, use transactions if needed
 	* @param string $table name of table to lock
 	* @param string $mode type of lock required (optional), default write
-	* @return bool True if sucessful, False if fails
+	* @return boolean|null True if sucessful, False if fails
 	*/
 	function lock($table, $mode='write')
 	{
@@ -196,7 +196,7 @@ 
 	* Unlock a table
 	*
 	* @deprecated not used anymore as it costs to much performance, use transactions if needed
-	* @return bool True if sucessful, False if fails
+	* @return boolean|null True if sucessful, False if fails
 	*/
 	function unlock()
 	{}

Indentation +81 added lines, -81 removed lines

@@ -41,19 +41,19 @@ 
 	var $Query_ID = 0;
 
 	/**
-	* Execute a query
-	*
-	* @param string $Query_String the query to be executed
-	* @param int $line the line method was called from - use __LINE__
-	* @param string $file the file method was called from - use __FILE__
-	* @param int $offset row to start from, default 0
-	* @param int $num_rows number of rows to return (optional), default -1 = all, 0 will use $GLOBALS['egw_info']['user']['preferences']['common']['maxmatchs']
-	* @param array|boolean $inputarr array for binding variables to parameters or false (default)
-	* @param int $fetchmode =self::FETCH_BOTH self::FETCH_BOTH (default), self::FETCH_ASSOC or self::FETCH_NUM
-	* @param boolean $reconnect =true true: try reconnecting if server closes connection, false: dont (mysql only!)
-	* @return ADORecordSet or false, if the query fails
-	* @throws EGroupware\Api\Db\Exception\InvalidSql with $this->Link_ID->ErrorNo() as code
-	*/
+	 * Execute a query
+	 *
+	 * @param string $Query_String the query to be executed
+	 * @param int $line the line method was called from - use __LINE__
+	 * @param string $file the file method was called from - use __FILE__
+	 * @param int $offset row to start from, default 0
+	 * @param int $num_rows number of rows to return (optional), default -1 = all, 0 will use $GLOBALS['egw_info']['user']['preferences']['common']['maxmatchs']
+	 * @param array|boolean $inputarr array for binding variables to parameters or false (default)
+	 * @param int $fetchmode =self::FETCH_BOTH self::FETCH_BOTH (default), self::FETCH_ASSOC or self::FETCH_NUM
+	 * @param boolean $reconnect =true true: try reconnecting if server closes connection, false: dont (mysql only!)
+	 * @return ADORecordSet or false, if the query fails
+	 * @throws EGroupware\Api\Db\Exception\InvalidSql with $this->Link_ID->ErrorNo() as code
+	 */
 	function query($Query_String, $line = '', $file = '', $offset=0, $num_rows=-1, $inputarr=false, $fetchmode=self::FETCH_BOTH, $reconnect=true)
 	{
 		// New query, discard previous result.
@@ -76,12 +76,12 @@ 
 	}
 
 	/**
-	* Escape strings before sending them to the database
-	*
-	* @deprecated use quote($value,$type='') instead
-	* @param string $str the string to be escaped
-	* @return string escaped sting
-	*/
+	 * Escape strings before sending them to the database
+	 *
+	 * @deprecated use quote($value,$type='') instead
+	 * @param string $str the string to be escaped
+	 * @return string escaped sting
+	 */
 	function db_addslashes($str)
 	{
 		if (!isset($str) || $str == '')
@@ -107,14 +107,14 @@ 
 	}
 
 	/**
-	* Move to the next row in the results set
-	*
-	* Specifying a fetch_mode only works for newly fetched rows, the first row always gets fetched by query!!!
-	*
-	* @deprecated use foreach(query() or foreach(select() to loop over the query using the global db object
-	* @param int $fetch_mode self::FETCH_BOTH = numerical+assoc keys (eGW default), self::FETCH_ASSOC or self::FETCH_NUM
-	* @return bool was another row found?
-	*/
+	 * Move to the next row in the results set
+	 *
+	 * Specifying a fetch_mode only works for newly fetched rows, the first row always gets fetched by query!!!
+	 *
+	 * @deprecated use foreach(query() or foreach(select() to loop over the query using the global db object
+	 * @param int $fetch_mode self::FETCH_BOTH = numerical+assoc keys (eGW default), self::FETCH_ASSOC or self::FETCH_NUM
+	 * @return bool was another row found?
+	 */
 	function next_record($fetch_mode=self::FETCH_BOTH)
 	{
 		if (!$this->Query_ID)
@@ -164,12 +164,12 @@ 
 	}
 
 	/**
-	* Move to position in result set
-	*
-	* @deprecated use the result-object returned by query() or select() direct, so you can use the global db-object and not a clone
-	* @param int $pos required row (optional), default first row
-	* @return boolean true if sucessful or false if not found
-	*/
+	 * Move to position in result set
+	 *
+	 * @deprecated use the result-object returned by query() or select() direct, so you can use the global db-object and not a clone
+	 * @param int $pos required row (optional), default first row
+	 * @return boolean true if sucessful or false if not found
+	 */
 	function seek($pos = 0)
 	{
 		if (!$this->Query_ID  || !$this->Query_ID->Move($this->Row = $pos))
@@ -180,74 +180,74 @@ 
 	}
 
 	/**
-	* Lock a table
-	*
-	* @deprecated not used anymore as it costs to much performance, use transactions if needed
-	* @param string $table name of table to lock
-	* @param string $mode type of lock required (optional), default write
-	* @return bool True if sucessful, False if fails
-	*/
+	 * Lock a table
+	 *
+	 * @deprecated not used anymore as it costs to much performance, use transactions if needed
+	 * @param string $table name of table to lock
+	 * @param string $mode type of lock required (optional), default write
+	 * @return bool True if sucessful, False if fails
+	 */
 	function lock($table, $mode='write')
 	{
 		unset($table, $mode);	// not used anymore
 	}
 
 	/**
-	* Unlock a table
-	*
-	* @deprecated not used anymore as it costs to much performance, use transactions if needed
-	* @return bool True if sucessful, False if fails
-	*/
+	 * Unlock a table
+	 *
+	 * @deprecated not used anymore as it costs to much performance, use transactions if needed
+	 * @return bool True if sucessful, False if fails
+	 */
 	function unlock()
 	{}
 
 	/**
-	* Number of rows in current result set
-	*
-	* @deprecated use the result-object returned by query/select()->NumRows(), so you can use the global db-object and not a clone
-	* @return int number of rows
-	*/
+	 * Number of rows in current result set
+	 *
+	 * @deprecated use the result-object returned by query/select()->NumRows(), so you can use the global db-object and not a clone
+	 * @return int number of rows
+	 */
 	function num_rows()
 	{
 		return $this->Query_ID ? $this->Query_ID->RecordCount() : False;
 	}
 
 	/**
-	* Number of fields in current row
-	*
-	* @deprecated use the result-object returned by query() or select() direct, so you can use the global db-object and not a clone
-	* @return int number of fields
-	*/
+	 * Number of fields in current row
+	 *
+	 * @deprecated use the result-object returned by query() or select() direct, so you can use the global db-object and not a clone
+	 * @return int number of fields
+	 */
 	function num_fields()
 	{
 		return $this->Query_ID ? $this->Query_ID->FieldCount() : False;
 	}
 
 	/**
-	* @deprecated use num_rows()
-	*/
+	 * @deprecated use num_rows()
+	 */
 	function nf()
 	{
 		return $this->num_rows();
 	}
 
 	/**
-	* @deprecated use print num_rows()
-	*/
+	 * @deprecated use print num_rows()
+	 */
 	function np()
 	{
 		print $this->num_rows();
 	}
 
 	/**
-	* Return the value of a column
-	*
-	* @deprecated use the result-object returned by query() or select() direct, so you can use the global db-object and not a clone
-	* @param string|integer $Name name of field or positional index starting from 0
-	* @param bool $strip_slashes string escape chars from field(optional), default false
-	*	depricated param, as correctly quoted values dont need any stripslashes!
-	* @return string the field value
-	*/
+	 * Return the value of a column
+	 *
+	 * @deprecated use the result-object returned by query() or select() direct, so you can use the global db-object and not a clone
+	 * @param string|integer $Name name of field or positional index starting from 0
+	 * @param bool $strip_slashes string escape chars from field(optional), default false
+	 *	depricated param, as correctly quoted values dont need any stripslashes!
+	 * @return string the field value
+	 */
 	function f($Name, $strip_slashes = False)
 	{
 		if ($strip_slashes)
@@ -258,26 +258,26 @@ 
 	}
 
 	/**
-	* Print the value of a field
-	*
-	* @deprecated use the result-object returned by query() or select() direct, so you can use the global db-object and not a clone
-	* @param string $Name name of field to print
-	* @param bool $strip_slashes string escape chars from field(optional), default false
-	*	depricated param, as correctly quoted values dont need any stripslashes!
-	*/
+	 * Print the value of a field
+	 *
+	 * @deprecated use the result-object returned by query() or select() direct, so you can use the global db-object and not a clone
+	 * @param string $Name name of field to print
+	 * @param bool $strip_slashes string escape chars from field(optional), default false
+	 *	depricated param, as correctly quoted values dont need any stripslashes!
+	 */
 	function p($Name, $strip_slashes = True)
 	{
 		print $this->f($Name, $strip_slashes);
 	}
 
 	/**
-	* Returns a query-result-row as an associative array (no numerical keys !!!)
-	*
-	* @deprecated use foreach(query() or foreach(select() to loop over the query using the global db object
-	* @param bool $do_next_record should next_record() be called or not (default not)
-	* @param string $strip ='' string to strip of the column-name, default ''
-	* @return array/bool the associative array or False if no (more) result-row is availible
-	*/
+	 * Returns a query-result-row as an associative array (no numerical keys !!!)
+	 *
+	 * @deprecated use foreach(query() or foreach(select() to loop over the query using the global db object
+	 * @param bool $do_next_record should next_record() be called or not (default not)
+	 * @param string $strip ='' string to strip of the column-name, default ''
+	 * @return array/bool the associative array or False if no (more) result-row is availible
+	 */
 	function row($do_next_record=False,$strip='')
 	{
 		if ($do_next_record && !$this->next_record(self::FETCH_ASSOC) || !is_array($this->Record))

Spacing +11 added lines, -11 removed lines

@@ -54,7 +54,7 @@ 
 	* @return ADORecordSet or false, if the query fails
 	* @throws EGroupware\Api\Db\Exception\InvalidSql with $this->Link_ID->ErrorNo() as code
 	*/
-	function query($Query_String, $line = '', $file = '', $offset=0, $num_rows=-1, $inputarr=false, $fetchmode=self::FETCH_BOTH, $reconnect=true)
+	function query($Query_String, $line = '', $file = '', $offset = 0, $num_rows = -1, $inputarr = false, $fetchmode = self::FETCH_BOTH, $reconnect = true)
 	{
 		// New query, discard previous result.
 		if ($this->Query_ID)
@@ -102,7 +102,7 @@ 
 	 */
 	function free()
 	{
-		unset($this->Query_ID);	// else copying of the db-object does not work
+		unset($this->Query_ID); // else copying of the db-object does not work
 		$this->Query_ID = 0;
 	}
 
@@ -115,7 +115,7 @@ 
 	* @param int $fetch_mode self::FETCH_BOTH = numerical+assoc keys (eGW default), self::FETCH_ASSOC or self::FETCH_NUM
 	* @return bool was another row found?
 	*/
-	function next_record($fetch_mode=self::FETCH_BOTH)
+	function next_record($fetch_mode = self::FETCH_BOTH)
 	{
 		if (!$this->Query_ID)
 		{
@@ -135,7 +135,7 @@ 
 		}
 		if ($this->capabilities[self::CAPABILITY_NAME_CASE] == 'upper')	// maxdb, oracle, ...
 		{
-			switch($fetch_mode)
+			switch ($fetch_mode)
 			{
 				case self::FETCH_ASSOC:
 					$this->Record = array_change_key_case($this->Record);
@@ -172,9 +172,9 @@ 
 	*/
 	function seek($pos = 0)
 	{
-		if (!$this->Query_ID  || !$this->Query_ID->Move($this->Row = $pos))
+		if (!$this->Query_ID || !$this->Query_ID->Move($this->Row = $pos))
 		{
-			throw new Exception("seek($pos) failed: resultset has " . $this->num_rows() . " rows");
+			throw new Exception("seek($pos) failed: resultset has ".$this->num_rows()." rows");
 		}
 		return True;
 	}
@@ -187,9 +187,9 @@ 
 	* @param string $mode type of lock required (optional), default write
 	* @return bool True if sucessful, False if fails
 	*/
-	function lock($table, $mode='write')
+	function lock($table, $mode = 'write')
 	{
-		unset($table, $mode);	// not used anymore
+		unset($table, $mode); // not used anymore
 	}
 
 	/**
@@ -278,18 +278,18 @@ 
 	* @param string $strip ='' string to strip of the column-name, default ''
 	* @return array/bool the associative array or False if no (more) result-row is availible
 	*/
-	function row($do_next_record=False,$strip='')
+	function row($do_next_record = False, $strip = '')
 	{
 		if ($do_next_record && !$this->next_record(self::FETCH_ASSOC) || !is_array($this->Record))
 		{
 			return False;
 		}
 		$result = array();
-		foreach($this->Record as $column => $value)
+		foreach ($this->Record as $column => $value)
 		{
 			if (!is_numeric($column))
 			{
-				if ($strip) $column = str_replace($strip,'',$column);
+				if ($strip) $column = str_replace($strip, '', $column);
 
 				$result[$column] = $value;
 			}

Braces +17 added lines, -5 removed lines

@@ -121,10 +121,13 @@ 
 		{
 			throw new Exception('next_record called with no query pending.');
 		}
-		if ($this->Row)	// first row is already fetched
+		if ($this->Row)
+		{
+			// first row is already fetched
 		{
 			$this->Query_ID->MoveNext();
 		}
+		}
 		++$this->Row;
 
 		$this->Record = $this->Query_ID->fields;
@@ -133,12 +136,15 @@ 
 		{
 			return False;
 		}
-		if ($this->capabilities[self::CAPABILITY_NAME_CASE] == 'upper')	// maxdb, oracle, ...
+		if ($this->capabilities[self::CAPABILITY_NAME_CASE] == 'upper')
+		{
+			// maxdb, oracle, ...
 		{
 			switch($fetch_mode)
 			{
 				case self::FETCH_ASSOC:
 					$this->Record = array_change_key_case($this->Record);
+		}
 					break;
 				case self::FETCH_NUM:
 					$this->Record = array_values($this->Record);
@@ -199,7 +205,8 @@ 
 	* @return bool True if sucessful, False if fails
 	*/
 	function unlock()
-	{}
+	{
+}
 
 	/**
 	* Number of rows in current result set
@@ -289,7 +296,10 @@ 
 		{
 			if (!is_numeric($column))
 			{
-				if ($strip) $column = str_replace($strip,'',$column);
+				if ($strip)
+				{
+					$column = str_replace($strip,'',$column);
+				}
 
 				$result[$column] = $value;
 			}
@@ -301,4 +311,6 @@ 
 /**
  * @deprecated use EGroupware\Api\Db\CallbackIterator
  */
-class egw_db_callback_iterator extends Api\Db\CallbackIterator {}
+class egw_db_callback_iterator extends Api\Db\CallbackIterator
+{
+}

api/src/Db/Schema.php

Doc Comments +7 added lines, -14 removed lines

@@ -96,7 +96,7 @@ 
 	 * Constructor of schema-processor
 	 *
 	 * @param string $dbms type of the database: 'mysql','pgsql','mssql','maxdb'
-	 * @param Db $db =null database class, if null we use $GLOBALS['egw']->db
+	 * @param  $db =null database class, if null we use $GLOBALS['egw']->db
 	 * @return schema_proc
 	 */
 	function __construct($dbms=False, Api\Db $db=null)
@@ -396,7 +396,7 @@ 
 	 *
 	 * @param string $sTableName
 	 * @param bool $preserveValue
-	 * @return boolean|string sequence-name or false
+	 * @return false|string sequence-name or false
 	 */
 	function _PostgresHasOldSequence($sTableName,$preserveValue=False)
 	{
@@ -422,7 +422,7 @@ 
 	/**
 	 * Check if we have an old, not automaticaly droped sequence and drop it
 	 *
-	 * @param $sTableName
+	 * @param string $sTableName
 	 */
 	function _PostgresTestDropOldSequence($sTableName)
 	{
@@ -877,12 +877,7 @@ 
 	/**
 	* Execute a query
 	*
-	* @param string $Query_String the query to be executed
-	* @param mixed $line the line method was called from - use __LINE__
-	* @param string $file the file method was called from - use __FILE__
-	* @param int $offset row to start from
-	* @param int $num_rows number of rows to return (optional), if unset will use $GLOBALS['phpgw_info']['user']['preferences']['common']['maxmatchs']
-	* @return ADORecordSet or false, if the query fails
+	* @param integer $line
 	*/
 	function query($sQuery, $line='', $file='')
 	{
@@ -898,8 +893,6 @@ 
 	*	if the row exists db::update is called else a new row with $date merged with $where gets inserted (data has precedence)
 	* @param int $line line-number to pass to query
 	* @param string $file file-name to pass to query
-	* @param string $app=false string with name of app, this need to be set in setup anyway!!!
-	* @return ADORecordSet or false, if the query fails
 	*/
 	function insert($table,$data,$where,$line,$file,$app=False,$use_prepared_statement=false)
 	{
@@ -910,7 +903,6 @@ 
 	 * Execute the Sql statements in an array and give diagnostics, if any error occures
 	 *
 	 * @param array $aSql array of SQL strings to execute
-	 * @param int $debug_level for which debug_level (and higher) should the diagnostics always been printed
 	 * @param string $debug variable number of arguments for the debug_message functions in case of an error
 	 * @return int 2: no error, 1: errors, but continued, 0: errors aborted
 	 */
@@ -953,7 +945,7 @@ 
 	 * b) we use the table-names plus all column-names and remove dublicate parts
 	 *
 	 * @internal
-	 * @param $sTableName string name of the table
+	 * @param string $sTableName string name of the table
 	 * @param $aColumnNames array of column-names or string with a single column-name
 	 * @return string the index-name
 	 */
@@ -1010,6 +1002,7 @@ 
 
 	/**
 	 * Giving a non-fatal error-message
+	 * @param string $str
 	 */
 	function error($str)
 	{
@@ -1038,7 +1031,7 @@ 
 	 *
 	 * The parameters get formated depending on their type.
 	 *
-	 * @param $msg string message with parameters/variables like lang(), eg. '%1'
+	 * @param string $msg string message with parameters/variables like lang(), eg. '%1'
 	 * @param $backtrace include a function-backtrace, default True=On
 	 *	should only be set to False=Off, if your code ensures a call with backtrace=On was made before !!!
 	 * @param $param mixed a variable number of parameters, to be inserted in $msg

Indentation +50 added lines, -50 removed lines

@@ -101,19 +101,19 @@ 
 	 */
 	function __construct($dbms=False, Api\Db $db=null)
 	{
-	    if(is_object($db))
+		if(is_object($db))
 		{
 			$this->m_odb = $db;
-	    }
-	    else
-	    {
+		}
+		else
+		{
 			$this->m_odb = isset($GLOBALS['egw']->db) && is_object($GLOBALS['egw']->db) ? $GLOBALS['egw']->db : $GLOBALS['egw_setup']->db;
-	    }
-	    if (!($this->m_odb instanceof Api\Db))
-	    {
-	    	throw new Api\Exception\AssertionFailed('no EGroupware\Api\Db object!');
-	    }
-	    $this->m_odb->connect();
+		}
+		if (!($this->m_odb instanceof Api\Db))
+		{
+			throw new Api\Exception\AssertionFailed('no EGroupware\Api\Db object!');
+		}
+		$this->m_odb->connect();
 		$this->capabilities =& $this->m_odb->capabilities;
 
 		$this->sType = $dbms ? $dbms : $this->m_odb->Type;
@@ -828,79 +828,79 @@ 
 	}
 
 	/**
-	* Return the value of a column
-	*
-	* @param string|integer $value name of field or positional index starting from 0
-	* @param bool $strip_slashes string escape chars from field(optional), default false
-	* @deprecated use result-set returned by query/select
-	* @return string the field value
-	*/
+	 * Return the value of a column
+	 *
+	 * @param string|integer $value name of field or positional index starting from 0
+	 * @param bool $strip_slashes string escape chars from field(optional), default false
+	 * @deprecated use result-set returned by query/select
+	 * @return string the field value
+	 */
 	function f($value,$strip_slashes=False)
 	{
 		if (!($this->m_odb instanceof Deprecated))
 		{
-	    	throw new Api\Exception\AssertionFailed(__METHOD__.' requires an EGroupware\Api\Db\Deprecated object!');
+			throw new Api\Exception\AssertionFailed(__METHOD__.' requires an EGroupware\Api\Db\Deprecated object!');
 		}
 		return $this->m_odb->f($value,$strip_slashes);
 	}
 
 	/**
-	* Number of rows in current result set
-	*
-	* @deprecated use result-set returned by query/select
-	* @return int number of rows
-	*/
+	 * Number of rows in current result set
+	 *
+	 * @deprecated use result-set returned by query/select
+	 * @return int number of rows
+	 */
 	function num_rows()
 	{
 		if (!($this->m_odb instanceof Deprecated))
 		{
-	    	throw new Api\Exception\AssertionFailed(__METHOD__.' requires an EGroupware\Api\Db\Deprecated object!');
+			throw new Api\Exception\AssertionFailed(__METHOD__.' requires an EGroupware\Api\Db\Deprecated object!');
 		}
 		return $this->m_odb->num_rows();
 	}
 
 	/**
-	* Move to the next row in the results set
-	*
-	* @deprecated use result-set returned by query/select
-	* @return bool was another row found?
-	*/
+	 * Move to the next row in the results set
+	 *
+	 * @deprecated use result-set returned by query/select
+	 * @return bool was another row found?
+	 */
 	function next_record()
 	{
 		if (!($this->m_odb instanceof Deprecated))
 		{
-	    	throw new Api\Exception\AssertionFailed(__METHOD__.' requires an EGroupware\Api\Db\Deprecated object!');
+			throw new Api\Exception\AssertionFailed(__METHOD__.' requires an EGroupware\Api\Db\Deprecated object!');
 		}
 		return $this->m_odb->next_record();
 	}
 
 	/**
-	* Execute a query
-	*
-	* @param string $Query_String the query to be executed
-	* @param mixed $line the line method was called from - use __LINE__
-	* @param string $file the file method was called from - use __FILE__
-	* @param int $offset row to start from
-	* @param int $num_rows number of rows to return (optional), if unset will use $GLOBALS['phpgw_info']['user']['preferences']['common']['maxmatchs']
-	* @return ADORecordSet or false, if the query fails
-	*/
+	 * Execute a query
+	 *
+	 * @param string $Query_String the query to be executed
+	 * @param mixed $line the line method was called from - use __LINE__
+	 * @param string $file the file method was called from - use __FILE__
+	 * @param int $offset row to start from
+	 * @param int $num_rows number of rows to return (optional), if unset will use $GLOBALS['phpgw_info']['user']['preferences']['common']['maxmatchs']
+	 * @return ADORecordSet or false, if the query fails
+	 */
 	function query($sQuery, $line='', $file='')
 	{
 		return $this->m_odb->query($sQuery, $line, $file);
 	}
 
 	/**
-	* Insert a row of data into a table or updates it if $where is given, all data is quoted according to it's type
-	*
-	* @param string $table name of the table
-	* @param array $data with column-name / value pairs
-	* @param mixed $where string with where clause or array with column-name / values pairs to check if a row with that keys already exists, or false for an unconditional insert
-	*	if the row exists db::update is called else a new row with $date merged with $where gets inserted (data has precedence)
-	* @param int $line line-number to pass to query
-	* @param string $file file-name to pass to query
-	* @param string $app=false string with name of app, this need to be set in setup anyway!!!
-	* @return ADORecordSet or false, if the query fails
-	*/
+	 * Insert a row of data into a table or updates it if $where is given, all data is quoted according to it's type
+	 *
+	 * @param string $table name of the table
+	 * @param array $data with column-name / value pairs
+	 * @param mixed $where string with where clause or array with column-name / values pairs to check if a row with that keys already exists, or false for an unconditional insert
+	 *	if the row exists db::update is called else a new row with $date merged with $where gets inserted (data has precedence)
+	 * @param int $line line-number to pass to query
+	 * @param string $file file-name to pass to query
+	 * @param string $app=false string with name of app, this need to be set in setup anyway!!!
+	 * @return ADORecordSet or false, if the query fails
+	 */
 	function insert($table,$data,$where,$line,$file,$app=False,$use_prepared_statement=false)
 	{
 		return $this->m_odb->insert($table,$data,$where,$line,$file,$app,$use_prepared_statement);

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -797,7 +797,7 @@
 	 */
 	function ExecuteScripts($aTables, $bOutputHTML=False)
 	{
-		if(!is_array($aTables) || !IsSet($this->m_odb))
+		if(!is_array($aTables) || !isset($this->m_odb))
 		{
 			return False;
 		}

Spacing +181 added lines, -182 removed lines

@@ -57,7 +57,7 @@ 
 	 *
 	 * @var array
 	 */
-	var $max_index_length=array(
+	var $max_index_length = array(
 		'maxdb'  => 32,
 		'oracle' => 30,
 	);
@@ -99,9 +99,9 @@ 
 	 * @param Db $db =null database class, if null we use $GLOBALS['egw']->db
 	 * @return schema_proc
 	 */
-	function __construct($dbms=False, Api\Db $db=null)
+	function __construct($dbms = False, Api\Db $db = null)
 	{
-	    if(is_object($db))
+	    if (is_object($db))
 		{
 			$this->m_odb = $db;
 	    }
@@ -114,7 +114,7 @@ 
 	    	throw new Api\Exception\AssertionFailed('no EGroupware\Api\Db object!');
 	    }
 	    $this->m_odb->connect();
-		$this->capabilities =& $this->m_odb->capabilities;
+		$this->capabilities = & $this->m_odb->capabilities;
 
 		$this->sType = $dbms ? $dbms : $this->m_odb->Type;
 		$this->adodb = &$this->m_odb->Link_ID;
@@ -126,7 +126,7 @@ 
 		// to allow some of the former translator-functions to be called, we assign ourself as the translator
 		$this->m_oTranslator = &$this;
 
-		switch($this->sType)
+		switch ($this->sType)
 		{
 			case 'maxdb':
 				$this->max_varchar_length = 8000;
@@ -142,7 +142,7 @@ 
 		}
 		if (is_object($GLOBALS['egw_setup']))
 		{
-			$this->system_charset =& $GLOBALS['egw_setup']->system_charset;
+			$this->system_charset = & $GLOBALS['egw_setup']->system_charset;
 		}
 		elseif (isset($GLOBALS['egw_info']['server']['system_charset']))
 		{
@@ -158,20 +158,20 @@ 
 	 * @param boolean $ignore_length_limit =false should we take lenght-limits of indexes into account or not
 	 * @return boolean true if index over $columns exist in the $indexes array
 	 */
-	function _in_index($columns, $indexs, $ignore_length_limit=false)
+	function _in_index($columns, $indexs, $ignore_length_limit = false)
 	{
 		if (is_array($columns))
 		{
 			unset($columns['options']);
-			$columns = implode('-',$columns);
+			$columns = implode('-', $columns);
 			if ($ignore_length_limit) $columns = preg_replace('/\(\d+\)/', '', $columns);
 		}
-		foreach($indexs as $index)
+		foreach ($indexs as $index)
 		{
 			if (is_array($index))
 			{
 				unset($index['options']);
-				$index = implode('-',$index);
+				$index = implode('-', $index);
 			}
 			if ($ignore_length_limit) $index = preg_replace('/\(\d+\)/', '', $index);
 			if ($columns == $index) return true;
@@ -187,21 +187,21 @@ 
 	 * @param bool $preserveSequence
 	 * @return int 2: no error, 1: errors, but continued, 0: errors aborted
 	 */
-	function CreateTable($sTableName, $aTableDef, $preserveSequence=False)
+	function CreateTable($sTableName, $aTableDef, $preserveSequence = False)
 	{
 		if ($this->debug)
 		{
-			$this->debug_message('schema_proc::CreateTable(%1,%2)',False,$sTableName, $aTableDef);
+			$this->debug_message('schema_proc::CreateTable(%1,%2)', False, $sTableName, $aTableDef);
 		}
 		// for mysql 4.0+ we set the charset for the table
-		if ($this->system_charset && substr($this->sType,0,5) == 'mysql' &&
-			(float) $this->m_odb->ServerInfo['version'] >= 4.0)
+		if ($this->system_charset && substr($this->sType, 0, 5) == 'mysql' &&
+			(float)$this->m_odb->ServerInfo['version'] >= 4.0)
 		{
 			$set_table_charset = array($this->sType => 'CHARACTER SET utf8');
 		}
 		// creating the table
-		$aSql = $this->dict->CreateTableSQL($sTableName,$ado_cols = $this->_egw2adodb_columndef($aTableDef),$set_table_charset);
-		if (!($retVal = $this->ExecuteSQLArray($aSql,2,'CreateTableSQL(%1,%2) sql=%3',False,$sTableName,$ado_cols,$aSql)))
+		$aSql = $this->dict->CreateTableSQL($sTableName, $ado_cols = $this->_egw2adodb_columndef($aTableDef), $set_table_charset);
+		if (!($retVal = $this->ExecuteSQLArray($aSql, 2, 'CreateTableSQL(%1,%2) sql=%3', False, $sTableName, $ado_cols, $aSql)))
 		{
 			return $retVal;
 		}
@@ -210,13 +210,13 @@ 
 		{
 			if (empty($mFields))
 			{
-				continue;	// cant create an index without fields (was observed in broken backups)
+				continue; // cant create an index without fields (was observed in broken backups)
 			}
-			if ($this->_in_index($mFields,array($aTableDef['pk'])))
+			if ($this->_in_index($mFields, array($aTableDef['pk'])))
 			{
-				continue;	// is already created as primary key
+				continue; // is already created as primary key
 			}
-			if (!($retVal = $this->CreateIndex($sTableName,$mFields,true,'',$name)))
+			if (!($retVal = $this->CreateIndex($sTableName, $mFields, true, '', $name)))
 			{
 				return $retVal;
 			}
@@ -226,12 +226,12 @@ 
 		{
 			if (empty($mFields))
 			{
-				continue;	// cant create an index without fields (was observed in broken backups)
+				continue; // cant create an index without fields (was observed in broken backups)
 			}
-			if ($this->_in_index($mFields,array($aTableDef['pk'])) ||
-				$this->_in_index($mFields,$aTableDef['uc']))
+			if ($this->_in_index($mFields, array($aTableDef['pk'])) ||
+				$this->_in_index($mFields, $aTableDef['uc']))
 			{
-				continue;	// is already created as primary key or unique index
+				continue; // is already created as primary key or unique index
 			}
 			$options = False;
 			if (is_array($mFields))
@@ -240,21 +240,21 @@ 
 				{
 					if (isset($mFields['options'][$this->sType]))
 					{
-						$options = $mFields['options'][$this->sType];	// db-specific options, eg. index-type
-						if (!$options) continue;	// no index for our db-type
+						$options = $mFields['options'][$this->sType]; // db-specific options, eg. index-type
+						if (!$options) continue; // no index for our db-type
 					}
 					unset($mFields['options']);
 				}
 			}
-			foreach((array)$mFields as $k => $col)
+			foreach ((array)$mFields as $k => $col)
 			{
 				// only create indexes on text-columns, if (db-)specifiy options are given or FULLTEXT for mysql
 				// most DB's cant do them and give errors
-				if (in_array($aTableDef['fd'][$col]['type'],array('text','longtext')))
+				if (in_array($aTableDef['fd'][$col]['type'], array('text', 'longtext')))
 				{
 					if (is_array($mFields))	// index over multiple columns including a text column
 					{
-						$mFields[$k] .= '(32)';	// 32=limit of egw_addressbook_extra.extra_value to fix old backups
+						$mFields[$k] .= '(32)'; // 32=limit of egw_addressbook_extra.extra_value to fix old backups
 					}
 					elseif (!$options)	// index over a single text column and no options given
 					{
@@ -264,12 +264,12 @@ 
 						}
 						else
 						{
-							continue 2;	// ignore that index, 2=not just column but whole index!
+							continue 2; // ignore that index, 2=not just column but whole index!
 						}
 					}
 				}
 			}
-			if (!($retVal = $this->CreateIndex($sTableName,$mFields,false,$options,$name)))
+			if (!($retVal = $this->CreateIndex($sTableName, $mFields, false, $options, $name)))
 			{
 				return $retVal;
 			}
@@ -280,7 +280,7 @@ 
 			if (($seq = $this->_PostgresHasOldSequence($sTableName)))
 			{
 				$this->pgsql_old_seq = $this->pgsql_old_seq + 1;
-				$this->m_odb->query("ALTER SEQUENCE $seq RESTART WITH " . $this->pgsql_old_seq,__LINE__,__FILE__);
+				$this->m_odb->query("ALTER SEQUENCE $seq RESTART WITH ".$this->pgsql_old_seq, __LINE__, __FILE__);
 			}
 			$this->pgsql_old_seq = 0;
 		}
@@ -294,9 +294,9 @@ 
 	 * @param boolean $bOutputHTML should we give diagnostics, default False
 	 * @return boolean True if no error, else False
 	 */
-	function DropAllTables($aTables, $bOutputHTML=False)
+	function DropAllTables($aTables, $bOutputHTML = False)
 	{
-		if(!is_array($aTables) || !isset($this->m_odb))
+		if (!is_array($aTables) || !isset($this->m_odb))
 		{
 			return False;
 		}
@@ -304,13 +304,13 @@ 
 		if ($this->debug) $bOutputHTML = True;
 		if ($bOutputHTML && !$this->debug) $this->debug = 2;
 
-		foreach(array_keys($aTables) as $sTableName)
+		foreach (array_keys($aTables) as $sTableName)
 		{
-			if($this->DropTable($sTableName))
+			if ($this->DropTable($sTableName))
 			{
-				if($bOutputHTML)
+				if ($bOutputHTML)
 				{
-					echo '<br>Drop Table <b>' . $sTableName . '</b>';
+					echo '<br>Drop Table <b>'.$sTableName.'</b>';
 				}
 			}
 			else
@@ -333,7 +333,7 @@ 
 
 		$aSql = $this->dict->DropTableSql($sTableName);
 
-		return $this->ExecuteSQLArray($aSql,2,'DropTable(%1) sql=%2',False,$sTableName,$aSql);
+		return $this->ExecuteSQLArray($aSql, 2, 'DropTable(%1) sql=%2', False, $sTableName, $aSql);
 	}
 
 	/**
@@ -346,14 +346,14 @@ 
 	 */
 	function DropColumn($sTableName, $aTableDef, $sColumnName)
 	{
-		unset($aTableDef);	// not used, but required by function signature
+		unset($aTableDef); // not used, but required by function signature
 
 		if (!($table_def = $this->GetTableDefinition($sTableName))) return 0;
 		unset($table_def['fd'][$sColumnName]);
 
-		$aSql = $this->dict->DropColumnSql($sTableName,$sColumnName,$ado_table=$this->_egw2adodb_columndef($table_def));
+		$aSql = $this->dict->DropColumnSql($sTableName, $sColumnName, $ado_table = $this->_egw2adodb_columndef($table_def));
 
-		return $this->ExecuteSQLArray($aSql,2,'DropColumnSQL(%1,%2,%3) sql=%4',False,$sTableName,$sColumnName,$ado_table,$aSql);
+		return $this->ExecuteSQLArray($aSql, 2, 'DropColumnSQL(%1,%2,%3) sql=%4', False, $sTableName, $sColumnName, $ado_table, $aSql);
 	}
 
 	/**
@@ -372,12 +372,12 @@ 
 			if (!($table_def = $this->GetTableDefinition($sOldTableName))) return 0;
 
 			// only use old PostgreSQL stuff, if we have an old sequence, otherwise rely on it being new enough
-			if ($this->_PostgresHasOldSequence($sOldTableName,True) &&
+			if ($this->_PostgresHasOldSequence($sOldTableName, True) &&
 				(count($table_def['pk']) || count($table_def['ix']) || count($table_def['uc'])))
 			{
 				if ($this->adodb->BeginTrans() &&
-					$this->CreateTable($sNewTableName,$table_def,True) &&
-					$this->m_odb->query("INSERT INTO $sNewTableName SELECT * FROM $sOldTableName",__LINE__,__FILE__) &&
+					$this->CreateTable($sNewTableName, $table_def, True) &&
+					$this->m_odb->query("INSERT INTO $sNewTableName SELECT * FROM $sOldTableName", __LINE__, __FILE__) &&
 					// sequence must be updated, after inserts containing pkey, otherwise new inserst will fail!
 					(count($table_def['pk']) !== 1 || $this->UpdateSequence($sNewTableName, $table_def['pk'][0])) &&
 					$this->DropTable($sOldTableName))
@@ -391,7 +391,7 @@ 
 		}
 		$aSql = $this->dict->RenameTableSQL($sOldTableName, $sNewTableName);
 
-		return $this->ExecuteSQLArray($aSql,2,'RenameTableSQL(%1,%2) sql=%3',False,$sOldTableName,$sNewTableName,$aSql);
+		return $this->ExecuteSQLArray($aSql, 2, 'RenameTableSQL(%1,%2) sql=%3', False, $sOldTableName, $sNewTableName, $aSql);
 	}
 
 	/**
@@ -401,7 +401,7 @@ 
 	 * @param bool $preserveValue
 	 * @return boolean|string sequence-name or false
 	 */
-	function _PostgresHasOldSequence($sTableName,$preserveValue=False)
+	function _PostgresHasOldSequence($sTableName, $preserveValue = False)
 	{
 		if ($this->sType != 'pgsql') return false;
 
@@ -409,14 +409,14 @@ 
 		$seq2 = $this->adodb->GetOne("SELECT d.adsrc FROM pg_attribute a, pg_class c, pg_attrdef d WHERE c.relname='$sTableName' AND c.oid=d.adrelid AND d.adsrc LIKE '%$sTableName%_seq''::text)' AND a.attrelid=c.oid AND d.adnum=a.attnum");
 
 		$matches = null;
-		if ($seq && preg_match('/^nextval\(\'(.*)\'/',$seq,$matches))
+		if ($seq && preg_match('/^nextval\(\'(.*)\'/', $seq, $matches))
 		{
-			if ($preserveValue) $this->pgsql_old_seq = $this->adodb->GetOne("SELECT last_value FROM " . $matches[1]);
+			if ($preserveValue) $this->pgsql_old_seq = $this->adodb->GetOne("SELECT last_value FROM ".$matches[1]);
 			return $matches[1];
 		}
-		if ($seq2 && preg_match('/^nextval\(\'public\.(.*)\'/',$seq2,$matches))
+		if ($seq2 && preg_match('/^nextval\(\'public\.(.*)\'/', $seq2, $matches))
 		{
-			if ($preserveValue) $this->pgsql_old_seq = $this->adodb->GetOne("SELECT last_value FROM " . $matches[1]);
+			if ($preserveValue) $this->pgsql_old_seq = $this->adodb->GetOne("SELECT last_value FROM ".$matches[1]);
 			return $matches[1];
 		}
 		return false;
@@ -435,7 +435,7 @@ 
 			// only drop sequence, if there is no dependency on it
 			if (!$this->adodb->GetOne("SELECT relname FROM pg_class JOIN pg_depend ON pg_class.relfilenode=pg_depend.objid WHERE relname='$seq' AND relkind='S' AND deptype='i'"))
 			{
-				$this->query('DROP SEQUENCE '.$seq,__LINE__,__FILE__);
+				$this->query('DROP SEQUENCE '.$seq, __LINE__, __FILE__);
 			}
 		}
 	}
@@ -453,7 +453,7 @@ 
 		if (!($table_def = $this->GetTableDefinition($sTableName))) return 0;
 
 		// PostgreSQL: varchar or ascii column shortened, use substring to avoid error if current content is to long
-		if($this->sType == 'pgsql' && in_array($table_def['fd'][$sColumnName]['type'], array('varchar', 'ascii')) &&
+		if ($this->sType == 'pgsql' && in_array($table_def['fd'][$sColumnName]['type'], array('varchar', 'ascii')) &&
 			in_array($aColumnDef['type'], array('varchar', 'ascii')) &&
 			$table_def['fd'][$sColumnName]['precision'] > $aColumnDef['precision'])
 		{
@@ -468,12 +468,12 @@ 
 		}
 		$table_def['fd'][$sColumnName] = $aColumnDef;
 
-		$aSql = $this->dict->AlterColumnSQL($sTableName,$ado_col = $this->_egw2adodb_columndef(array(
+		$aSql = $this->dict->AlterColumnSQL($sTableName, $ado_col = $this->_egw2adodb_columndef(array(
 				'fd' => array($sColumnName => $aColumnDef),
 				'pk' => array(),
-			)),$ado_table=$this->_egw2adodb_columndef($table_def));
+			)), $ado_table = $this->_egw2adodb_columndef($table_def));
 
-		return $this->ExecuteSQLArray($aSql,2,'AlterColumnSQL(%1,%2,%3) sql=%4',False,$sTableName,$ado_col,$ado_table,$aSql);
+		return $this->ExecuteSQLArray($aSql, 2, 'AlterColumnSQL(%1,%2,%3) sql=%4', False, $sTableName, $ado_col, $ado_table, $aSql);
 	}
 
 	/**
@@ -495,7 +495,7 @@ 
 		}
 		else
 		{
-			foreach($table_def['fd'] as $col => $def)
+			foreach ($table_def['fd'] as $col => $def)
 			{
 				if (strtolower($col) == strtolower($sOldColumnName))
 				{
@@ -509,9 +509,9 @@ 
 				'pk' => array(),
 			));
 
-		$aSql = $this->dict->RenameColumnSQL($sTableName,$sOldColumnName,$sNewColumnName,$col_def);
+		$aSql = $this->dict->RenameColumnSQL($sTableName, $sOldColumnName, $sNewColumnName, $col_def);
 
-		return $this->ExecuteSQLArray($aSql,2,'RenameColumnSQL(%1,%2,%3) sql=%4',False,$sTableName,$sOldColumnName, $sNewColumnName,$aSql);
+		return $this->ExecuteSQLArray($aSql, 2, 'RenameColumnSQL(%1,%2,%3) sql=%4', False, $sTableName, $sOldColumnName, $sNewColumnName, $aSql);
 	}
 
 	/**
@@ -524,12 +524,12 @@ 
 	 */
 	function AddColumn($sTableName, $sColumnName, $aColumnDef)
 	{
-		$aSql = $this->dict->AddColumnSQL($sTableName,$ado_cols = $this->_egw2adodb_columndef(array(
+		$aSql = $this->dict->AddColumnSQL($sTableName, $ado_cols = $this->_egw2adodb_columndef(array(
 				'fd' => array($sColumnName => $aColumnDef),
 				'pk' => array(),
 			)));
 
-		return $this->ExecuteSQLArray($aSql,2,'AlterColumnSQL(%1,%2,%3) sql=%4',False,$sTableName,$sColumnName, $aColumnDef,$aSql);
+		return $this->ExecuteSQLArray($aSql, 2, 'AlterColumnSQL(%1,%2,%3) sql=%4', False, $sTableName, $sColumnName, $aColumnDef, $aSql);
 	}
 
 	/**
@@ -542,23 +542,23 @@ 
 	 * @param string $sIdxName ='' name of the index, if not given (default) its created automaticaly
 	 * @return int 2: no error, 1: errors, but continued, 0: errors aborted
 	 */
-	function CreateIndex($sTableName,$aColumnNames,$bUnique=false,$options='',$sIdxName='')
+	function CreateIndex($sTableName, $aColumnNames, $bUnique = false, $options = '', $sIdxName = '')
 	{
 		// remove length limits from column names, if DB type is NOT MySQL
 		if ($this->sType != 'mysql')
 		{
-			$aColumnNames = preg_replace('/ *\(\d+\)$/','',$aColumnNames);
+			$aColumnNames = preg_replace('/ *\(\d+\)$/', '', $aColumnNames);
 		}
 		if (!$sIdxName || is_numeric($sIdxName))
 		{
-			$sIdxName = $this->_index_name($sTableName,$aColumnNames);
+			$sIdxName = $this->_index_name($sTableName, $aColumnNames);
 		}
 		if (!is_array($options)) $options = $options ? array($options) : array();
 		if ($bUnique) $options[] = 'UNIQUE';
 
-		$aSql = $this->dict->CreateIndexSQL($sIdxName,$sTableName,$aColumnNames,$options);
+		$aSql = $this->dict->CreateIndexSQL($sIdxName, $sTableName, $aColumnNames, $options);
 
-		return $this->ExecuteSQLArray($aSql,2,'CreateIndexSQL(%1,%2,%3,%4) sql=%5',False,$sTableName,$aColumnNames,$options,$sIdxName,$aSql);
+		return $this->ExecuteSQLArray($aSql, 2, 'CreateIndexSQL(%1,%2,%3,%4) sql=%5', False, $sTableName, $aColumnNames, $options, $sIdxName, $aSql);
 	}
 
 	/**
@@ -568,7 +568,7 @@ 
 	 * @param array|string $aColumnNames columns of the index or the name of the index
 	 * @return int 2: no error, 1: errors, but continued, 0: errors aborted
 	 */
-	function DropIndex($sTableName,$aColumnNames)
+	function DropIndex($sTableName, $aColumnNames)
 	{
 		if (is_array($aColumnNames))
 		{
@@ -578,13 +578,13 @@ 
 			{
 				// if MetaIndexes is not availible for the DB, we try the name the index was created with
 				// this fails if one of the columns have been renamed
-				$sIdxName = $this->_index_name($sTableName,$aColumnNames);
+				$sIdxName = $this->_index_name($sTableName, $aColumnNames);
 			}
 			else
 			{
-				foreach($indexes as $idx => $idx_data)
+				foreach ($indexes as $idx => $idx_data)
 				{
-					if (strtolower(implode(':',$idx_data['columns'])) == implode(':',$aColumnNames))
+					if (strtolower(implode(':', $idx_data['columns'])) == implode(':', $aColumnNames))
 					{
 						$sIdxName = $idx;
 						break;
@@ -596,13 +596,13 @@ 
 		{
 			$sIdxName = $aColumnNames;
 		}
-		if(!$sIdxName)
+		if (!$sIdxName)
 		{
 			return True;
 		}
-		$aSql = $this->dict->DropIndexSQL($sIdxName,$sTableName);
+		$aSql = $this->dict->DropIndexSQL($sIdxName, $sTableName);
 
-		return $this->ExecuteSQLArray($aSql,2,'DropIndexSQL(%1(%2),%3) sql=%4',False,$sIdxName,$aColumnNames,$sTableName,$aSql);
+		return $this->ExecuteSQLArray($aSql, 2, 'DropIndexSQL(%1(%2),%3) sql=%4', False, $sIdxName, $aColumnNames, $sTableName, $aSql);
 	}
 
 	/**
@@ -610,22 +610,22 @@ 
 	 * @param string $sTableName table-name
 	 * @param string $sColumnName column-name, which default is set to nextval()
 	 */
-	function UpdateSequence($sTableName,$sColumnName)
+	function UpdateSequence($sTableName, $sColumnName)
 	{
-		switch($this->sType)
+		switch ($this->sType)
 		{
 			case 'pgsql':
 				// identify the sequence name, ADOdb uses a different name or it might be renamed
 				$columns = $this->dict->MetaColumns($sTableName);
 				$seq_name = 'seq_'.$sTableName;
 				$matches = null;
-				if (preg_match("/nextval\('([^']+)'::(text|regclass)\)/",$columns[strtoupper($sColumnName)]->default_value,$matches))
+				if (preg_match("/nextval\('([^']+)'::(text|regclass)\)/", $columns[strtoupper($sColumnName)]->default_value, $matches))
 				{
 					$seq_name = $matches[1];
 				}
 				$sql = "SELECT setval('$seq_name',MAX($sColumnName)) FROM $sTableName";
-				if($this->debug) { echo "<br>Updating sequence '$seq_name using: $sql"; }
-				return $this->query($sql,__LINE__,__FILE__);
+				if ($this->debug) { echo "<br>Updating sequence '$seq_name using: $sql"; }
+				return $this->query($sql, __LINE__, __FILE__);
 		}
 		return True;
 	}
@@ -642,9 +642,9 @@ 
 	 * @param array $aTableDef eGW table-defintion
 	 * @param array|boolean $aDefaults array with default for the colums during copying, values are either (old) column-names or quoted string-literals
 	 */
-	function RefreshTable($sTableName, $aTableDef, $aDefaults=False)
+	function RefreshTable($sTableName, $aTableDef, $aDefaults = False)
 	{
-		if($this->debug) { echo "<p>schema_proc::RefreshTable('$sTableName',"._debug_array($aTableDef,False).")\n"; }
+		if ($this->debug) { echo "<p>schema_proc::RefreshTable('$sTableName',"._debug_array($aTableDef, False).")\n"; }
 
 		$old_table_def = $this->GetTableDefinition($sTableName);
 
@@ -653,7 +653,7 @@ 
 
 		$select = array();
 		$blob_column_included = $auto_column_included = False;
-		foreach($aTableDef['fd'] as $name => $data)
+		foreach ($aTableDef['fd'] as $name => $data)
 		{
 			// new auto column with no default or explicit NULL as default (can be an existing column too!)
 			if ($data['type'] == 'auto' &&
@@ -661,7 +661,7 @@ 
 				$aDefaults && strtoupper($aDefaults[$name]) == 'NULL'))
 			{
 				$sequence_name = $sTableName.'_'.$name.'_seq';
-				switch($GLOBALS['egw_setup']->db->Type)
+				switch ($GLOBALS['egw_setup']->db->Type)
 				{
 					case 'mysql':
 						$value = 'NULL'; break;
@@ -682,25 +682,25 @@ 
 				if ($this->sType == 'pgsql')			// some postgres specific code
 				{
 					// this is eg. necessary to change a varchar into an int column under postgres
-					if (in_array($old_table_def['fd'][$name]['type'],array('char','varchar','text','blob')) &&
-						in_array($data['type'],array('int','decimal')))
+					if (in_array($old_table_def['fd'][$name]['type'], array('char', 'varchar', 'text', 'blob')) &&
+						in_array($data['type'], array('int', 'decimal')))
 					{
 						$value = "to_number($name,'S9999999999999D99')";
 					}
 					// blobs cant be casted to text
-					elseif($old_table_def['fd'][$name]['type'] == 'blob' && $data['type'] == 'text')
+					elseif ($old_table_def['fd'][$name]['type'] == 'blob' && $data['type'] == 'text')
 					{
 						$value = "ENCODE($value,'escape')";
 					}
 					// varchar or ascii column shortened, use substring to avoid error if current content is to long
-					elseif(in_array($old_table_def['fd'][$name]['type'], array('varchar', 'ascii')) &&
+					elseif (in_array($old_table_def['fd'][$name]['type'], array('varchar', 'ascii')) &&
 						in_array($data['type'], array('varchar', 'ascii')) &&
 						$old_table_def['fd'][$name]['precision'] > $data['precision'])
 					{
 						$value = "SUBSTRING($value FROM 1 FOR ".(int)$data['precision'].')';
 					}
 					// cast everything which is a different type
-					elseif($old_table_def['fd'][$name]['type'] != $data['type'] && ($type_translated = $this->TranslateType($data['type'])))
+					elseif ($old_table_def['fd'][$name]['type'] != $data['type'] && ($type_translated = $this->TranslateType($data['type'])))
 					{
 						$value = "CAST($value AS $type_translated)";
 					}
@@ -713,42 +713,42 @@ 
 					$value = 'NULL';
 				}
 				// some stuff is NOT to be quoted
-				elseif (in_array(strtoupper($data['default']),array('CURRENT_TIMESTAMP','CURRENT_DATE','NULL','NOW()')))
+				elseif (in_array(strtoupper($data['default']), array('CURRENT_TIMESTAMP', 'CURRENT_DATE', 'NULL', 'NOW()')))
 				{
 					$value = $data['default'];
 				}
 				else
 				{
-					$value = $this->m_odb->quote(isset($data['default']) ? $data['default'] : '',$data['type']);
+					$value = $this->m_odb->quote(isset($data['default']) ? $data['default'] : '', $data['type']);
 				}
 				if ($this->sType == 'pgsql')
 				{
 					// fix for postgres error "no '<' operator for type 'unknown'"
-					if(($type_translated = $this->TranslateType($data['type'])))
+					if (($type_translated = $this->TranslateType($data['type'])))
 					{
 						$value = "CAST($value AS $type_translated)";
 					}
 				}
 			}
-			$blob_column_included = $blob_column_included || in_array($data['type'],array('blob','text','longtext'));
+			$blob_column_included = $blob_column_included || in_array($data['type'], array('blob', 'text', 'longtext'));
 			$auto_column_included = $auto_column_included || $data['type'] == 'auto';
 			$select[] = $value;
 		}
 
 		$extra = '';
 		$distinct = 'DISTINCT';
-		switch($this->sType)
+		switch ($this->sType)
 		{
 			case 'mssql':
 				if ($auto_column_included) $extra = "SET IDENTITY_INSERT $sTableName ON\n";
-				if ($blob_column_included) $distinct = '';	// no distinct on blob-columns
+				if ($blob_column_included) $distinct = ''; // no distinct on blob-columns
 				break;
 		}
 		// because of all the trouble with sequences and indexes in the global namespace,
 		// we use an additional temp. table for postgres and not rename the existing one, but drop it.
 		if ($this->sType == 'pgsql')
 		{
-			$Ok = $this->m_odb->query("SELEcT * INTO TEMPORARY TABLE $tmp_name FROM $sTableName",__LINE__,__FILE__) &&
+			$Ok = $this->m_odb->query("SELEcT * INTO TEMPORARY TABLE $tmp_name FROM $sTableName", __LINE__, __FILE__) &&
 				$this->DropTable($sTableName);
 		}
 		else
@@ -758,12 +758,12 @@ 
 			{
 				$this->DropTable($tmp_name);
 			}
-			$Ok = $this->RenameTable($sTableName,$tmp_name);
+			$Ok = $this->RenameTable($sTableName, $tmp_name);
 		}
-		$Ok = $Ok && $this->CreateTable($sTableName,$aTableDef) &&
-			$this->m_odb->query($sql_copy_data="$extra INSERT INTO $sTableName (".
-				implode(',',array_keys($aTableDef['fd'])).
-				") SELEcT $distinct ".implode(',',$select)." FROM $tmp_name",__LINE__,__FILE__) &&
+		$Ok = $Ok && $this->CreateTable($sTableName, $aTableDef) &&
+			$this->m_odb->query($sql_copy_data = "$extra INSERT INTO $sTableName (".
+				implode(',', array_keys($aTableDef['fd'])).
+				") SELEcT $distinct ".implode(',', $select)." FROM $tmp_name", __LINE__, __FILE__) &&
 			$this->DropTable($tmp_name);
 		//error_log($sql_copy_data);
 
@@ -775,7 +775,7 @@ 
 		// do we need to update the new sequences value ?
 		if (count($aTableDef['pk']) == 1 && $aTableDef['fd'][$aTableDef['pk'][0]]['type'] == 'auto')
 		{
-			$this->UpdateSequence($sTableName,$aTableDef['pk'][0]);
+			$this->UpdateSequence($sTableName, $aTableDef['pk'][0]);
 		}
 		$this->m_odb->transaction_commit();
 
@@ -798,9 +798,9 @@ 
 	 * @param boolean $bOutputHTML =false should we give diagnostics, default False
 	 * @return boolean True on success, False if an (fatal) error occured
 	 */
-	function ExecuteScripts($aTables, $bOutputHTML=False)
+	function ExecuteScripts($aTables, $bOutputHTML = False)
 	{
-		if(!is_array($aTables) || !IsSet($this->m_odb))
+		if (!is_array($aTables) || !IsSet($this->m_odb))
 		{
 			return False;
 		}
@@ -808,20 +808,20 @@ 
 		if ($this->debug) $bOutputHTML = True;
 		if ($bOutputHTML && !$this->debug) $this->debug = 2;
 
-		foreach($aTables as $sTableName => $aTableDef)
+		foreach ($aTables as $sTableName => $aTableDef)
 		{
-			if($this->CreateTable($sTableName, $aTableDef))
+			if ($this->CreateTable($sTableName, $aTableDef))
 			{
-				if($bOutputHTML)
+				if ($bOutputHTML)
 				{
-					echo '<br>Create Table <b>' . $sTableName . '</b>';
+					echo '<br>Create Table <b>'.$sTableName.'</b>';
 				}
 			}
 			else
 			{
-				if($bOutputHTML)
+				if ($bOutputHTML)
 				{
-					echo '<br>Create Table Failed For <b>' . $sTableName . '</b>';
+					echo '<br>Create Table Failed For <b>'.$sTableName.'</b>';
 				}
 
 				return False;
@@ -838,13 +838,13 @@ 
 	* @deprecated use result-set returned by query/select
 	* @return string the field value
 	*/
-	function f($value,$strip_slashes=False)
+	function f($value, $strip_slashes = False)
 	{
 		if (!($this->m_odb instanceof Deprecated))
 		{
 	    	throw new Api\Exception\AssertionFailed(__METHOD__.' requires an EGroupware\Api\Db\Deprecated object!');
 		}
-		return $this->m_odb->f($value,$strip_slashes);
+		return $this->m_odb->f($value, $strip_slashes);
 	}
 
 	/**
@@ -887,7 +887,7 @@ 
 	* @param int $num_rows number of rows to return (optional), if unset will use $GLOBALS['phpgw_info']['user']['preferences']['common']['maxmatchs']
 	* @return ADORecordSet or false, if the query fails
 	*/
-	function query($sQuery, $line='', $file='')
+	function query($sQuery, $line = '', $file = '')
 	{
 		return $this->m_odb->query($sQuery, $line, $file);
 	}
@@ -904,9 +904,9 @@ 
 	* @param string $app=false string with name of app, this need to be set in setup anyway!!!
 	* @return ADORecordSet or false, if the query fails
 	*/
-	function insert($table,$data,$where,$line,$file,$app=False,$use_prepared_statement=false)
+	function insert($table, $data, $where, $line, $file, $app = False, $use_prepared_statement = false)
 	{
-		return $this->m_odb->insert($table,$data,$where,$line,$file,$app,$use_prepared_statement);
+		return $this->m_odb->insert($table, $data, $where, $line, $file, $app, $use_prepared_statement);
 	}
 
 	/**
@@ -917,14 +917,14 @@ 
 	 * @param string $debug variable number of arguments for the debug_message functions in case of an error
 	 * @return int 2: no error, 1: errors, but continued, 0: errors aborted
 	 */
-	function ExecuteSqlArray($aSql,$debug_level)
+	function ExecuteSqlArray($aSql, $debug_level)
 	{
 		if ($this->m_odb->query_log)	// we use Db::query to log the queries
 		{
 			$retval = 2;
-			foreach($aSql as $sql)
+			foreach ($aSql as $sql)
 			{
-				if (!$this->m_odb->query($sql,__LINE__,__FILE__))
+				if (!$this->m_odb->query($sql, __LINE__, __FILE__))
 				{
 					$retval = 1;
 				}
@@ -939,7 +939,7 @@ 
 			$debug_params = func_get_args();
 			array_shift($debug_params);
 			array_shift($debug_params);
-			call_user_func_array(array($this,'debug_message'),$debug_params);
+			call_user_func_array(array($this, 'debug_message'), $debug_params);
 			if ($retval < 2 && !$this->dict->debug)
 			{
 				echo '<p><b>'.$this->adodb->ErrorMsg()."</b></p>\n";
@@ -960,7 +960,7 @@ 
 	 * @param $aColumnNames array of column-names or string with a single column-name
 	 * @return string the index-name
 	 */
-	function _index_name($sTableName,$aColumnNames)
+	function _index_name($sTableName, $aColumnNames)
 	{
 		// this code creates extrem short index-names, eg. for MaxDB
 //			if (isset($this->max_index_length[$this->sType]) && $this->max_index_length[$this->sType] <= 32)
@@ -985,28 +985,28 @@ 
 //				return $name;
 //			}
 		// This code creates longer index-names incl. the table-names and the used columns
-		$table = str_replace(array('phpgw_','egw_'),'',$sTableName);
+		$table = str_replace(array('phpgw_', 'egw_'), '', $sTableName);
 		// if the table-name or a part of it is repeated in the column-name, remove it there
 		$remove[] = $table.'_';
 		// also remove 3 or 4 letter shortcuts of the table- or app-name
-		$remove[] = substr($table,0,3).'_';
-		$remove[] = substr($table,0,4).'_';
+		$remove[] = substr($table, 0, 3).'_';
+		$remove[] = substr($table, 0, 4).'_';
 		// if the table-name consists of '_' limtied parts, remove occurences of these parts too
-		foreach (explode('_',$table) as $part)
+		foreach (explode('_', $table) as $part)
 		{
 			$remove[] = $part.'_';
 		}
-		$cols = str_replace($remove,'',$aColumnNames);
+		$cols = str_replace($remove, '', $aColumnNames);
 
-		$name = $sTableName.'_'.(is_array($cols) ? implode('_',$cols) : $cols);
+		$name = $sTableName.'_'.(is_array($cols) ? implode('_', $cols) : $cols);
 		// remove length limits from column names
-		$name = preg_replace('/ *\(\d+\)/','',$name);
+		$name = preg_replace('/ *\(\d+\)/', '', $name);
 
 		// this code creates a fixed short index-names (30 chars) from the long and unique name, eg. for MaxDB or Oracle
 		if (isset($this->max_index_length[$this->sType]) && $this->max_index_length[$this->sType] <= 32 && strlen($name) > 30 ||
 			strlen($name) >= 64)	// even mysql has a limit here ;-)
 		{
-			$name = "i".substr(md5($name),0,29);
+			$name = "i".substr(md5($name), 0, 29);
 		}
 		return $name;
 	}
@@ -1047,19 +1047,19 @@ 
 	 * @param $param mixed a variable number of parameters, to be inserted in $msg
 	 *	arrays get serialized with print_r() !
 	 */
-	function debug_message($msg,$backtrace=True)
+	function debug_message($msg, $backtrace = True)
 	{
-		for($i = 2; $i < func_num_args(); ++$i)
+		for ($i = 2; $i < func_num_args(); ++$i)
 		{
 			$param = func_get_arg($i);
 
 			if (is_null($param))
 			{
-				$param='NULL';
+				$param = 'NULL';
 			}
 			else
 			{
-				switch(gettype($param))
+				switch (gettype($param))
 				{
 					case 'string':
 						$param = "'$param'";
@@ -1067,15 +1067,15 @@ 
 					case 'array':
 					case 'object':
 						list(,$content) = @each($param);
-						$do_pre = is_array($param) ? count($param) > 6 || is_array($content)&&count($content) : True;
-						$param = ($do_pre ? '<pre>' : '').print_r($param,True).($do_pre ? '</pre>' : '');
+						$do_pre = is_array($param) ? count($param) > 6 || is_array($content) && count($content) : True;
+						$param = ($do_pre ? '<pre>' : '').print_r($param, True).($do_pre ? '</pre>' : '');
 						break;
 					case 'boolean':
 						$param = $param ? 'True' : 'False';
 						break;
 				}
 			}
-			$msg = str_replace('%'.($i-1),$param,$msg);
+			$msg = str_replace('%'.($i - 1), $param, $msg);
 		}
 		echo '<p>'.$msg."<br>\n".($backtrace ? 'Backtrace: '.function_backtrace(1)."</p>\n" : '');
 	}
@@ -1090,15 +1090,15 @@ 
 	function _egw2adodb_columndef($aTableDef)
 	{
 		$ado_defs = array();
-		foreach($aTableDef['fd'] as $col => $col_data)
+		foreach ($aTableDef['fd'] as $col => $col_data)
 		{
 			$ado_col = False;
 
-			switch($col_data['type'])
+			switch ($col_data['type'])
 			{
 				case 'auto':
 					$ado_col = 'I AUTOINCREMENT NOTNULL';
-					unset($col_data['nullable']);	// else we set it twice
+					unset($col_data['nullable']); // else we set it twice
 					break;
 				case 'blob':
 					$ado_col = 'B';
@@ -1111,11 +1111,11 @@ 
 				case 'ascii':
 				case 'varchar':
 					$ado_col = "C";
-					if(0 < $col_data['precision'] && $col_data['precision'] <= $this->max_varchar_length)
+					if (0 < $col_data['precision'] && $col_data['precision'] <= $this->max_varchar_length)
 					{
 						$ado_col .= "($col_data[precision])";
 					}
-					if($col_data['precision'] > $this->max_varchar_length)
+					if ($col_data['precision'] > $this->max_varchar_length)
 					{
 						$ado_col = 'X';
 					}
@@ -1127,7 +1127,7 @@ 
 				case 'date':
 					$ado_col = 'D';
 					// allow to use now() beside current_date, as Postgres backups contain it and it's easier to remember anyway
-					if (in_array($col_data['default'],array('current_date','now()')))
+					if (in_array($col_data['default'], array('current_date', 'now()')))
 					{
 						$ado_col .= ' DEFDATE';
 						unset($col_data['default']);
@@ -1143,7 +1143,7 @@ 
 					break;
 				case 'int':
 					$ado_col = 'I';
-					switch($col_data['precision'])
+					switch ($col_data['precision'])
 					{
 						case 1:
 						case 2:
@@ -1162,7 +1162,7 @@ 
 				case 'timestamp':
 					$ado_col = 'T';
 					// allow to use now() beside current_timestamp, as Postgres backups contain it and it's easier to remember anyway
-					if (in_array($col_data['default'],array('current_timestamp','now()')))
+					if (in_array($col_data['default'], array('current_timestamp', 'now()')))
 					{
 						$ado_col .= ' DEFTIMESTAMP';
 						unset($col_data['default']);
@@ -1180,17 +1180,17 @@ 
 			}
 			if (isset($col_data['default']))
 			{
-				$ado_col .= (in_array($col_data['type'],array('bool','int','decimal','float','double')) && $col_data['default'] != 'NULL' ? ' NOQUOTE' : '').
-					' DEFAULT '.$this->m_odb->quote($col_data['default'],$col_data['type']);
+				$ado_col .= (in_array($col_data['type'], array('bool', 'int', 'decimal', 'float', 'double')) && $col_data['default'] != 'NULL' ? ' NOQUOTE' : '').
+					' DEFAULT '.$this->m_odb->quote($col_data['default'], $col_data['type']);
 			}
-			if (in_array($col,$aTableDef['pk']))
+			if (in_array($col, $aTableDef['pk']))
 			{
 				$ado_col .= ' PRIMARY';
 			}
-			$ado_defs[] = $col . ' ' . $ado_col;
+			$ado_defs[] = $col.' '.$ado_col;
 		}
 		//print_r($aTableDef); echo implode(",\n",$ado_defs)."\n";
-		return implode(",\n",$ado_defs);
+		return implode(",\n", $ado_defs);
 	}
 
 	/**
@@ -1206,7 +1206,7 @@ 
 			'pk' => array(),
 		));
 		$matches = null;
-		return preg_match('/test ([A-Z0-9]+)/i',$ado_col,$matches) ? $this->dict->ActualType($matches[1]) : false;
+		return preg_match('/test ([A-Z0-9]+)/i', $ado_col, $matches) ? $this->dict->ActualType($matches[1]) : false;
 	}
 
 	/**
@@ -1222,7 +1222,7 @@ 
 		// MetaType returns all varchar >= blobSize as blob, it's by default 100, which is wrong
 		$this->dict->blobSize = $this->max_varchar_length;
 
-		if (!method_exists($this->dict,'MetaColumns') ||
+		if (!method_exists($this->dict, 'MetaColumns') ||
 			!($columns = $this->dict->MetaColumns($sTableName)))
 		{
 			return False;
@@ -1235,11 +1235,11 @@ 
 			'uc' => array(),
 		);
 		//echo "$sTableName: <pre>".print_r($columns,true)."</pre>";
-		foreach($columns as $column)
+		foreach ($columns as $column)
 		{
 			$name = $this->capabilities['name_case'] == 'upper' ? strtolower($column->name) : $column->name;
 
-			$type = method_exists($this->dict,'MetaType') ? $this->dict->MetaType($column) : strtoupper($column->type);
+			$type = method_exists($this->dict, 'MetaType') ? $this->dict->MetaType($column) : strtoupper($column->type);
 
 			// fix longtext not correctly handled by ADOdb
 			if ($type == 'X' && $column->type == 'longtext') $type = 'XL';
@@ -1261,11 +1261,11 @@ 
 			);
 			$definition['fd'][$name]['type'] = $ado_type2egw[$type];
 
-			switch($type)
+			switch ($type)
 			{
 				case 'D': case 'T':
 					// detecting the automatic timestamps again
-					if ($column->has_default && preg_match('/(0000-00-00|timestamp)/i',$column->default_value))
+					if ($column->has_default && preg_match('/(0000-00-00|timestamp)/i', $column->default_value))
 					{
 						$column->default_value = $type == 'D' ? 'current_date' : 'current_timestamp';
 					}
@@ -1292,10 +1292,10 @@ 
 					$column->auto_increment = true;
 					// fall-through
 				case 'I': case 'I1': case 'I2': case 'I4': case 'I8':
-					switch($type)
+					switch ($type)
 					{
 						case 'I1': case 'I2': case 'I4': case 'I8':
-							$definition['fd'][$name]['precision'] = (int) $type[1];
+							$definition['fd'][$name]['precision'] = (int)$type[1];
 							break;
 						default:
 							if ($column->max_length > 11)
@@ -1331,7 +1331,7 @@ 
 						$definition['fd'][$name]['type'] = 'int';
 						// detect postgres type-spec and remove it
 						$matches = null;
-						if ($this->sType == 'pgsql' && $column->has_default && preg_match('/\(([^)])\)::/',$column->default_value,$matches))
+						if ($this->sType == 'pgsql' && $column->has_default && preg_match('/\(([^)])\)::/', $column->default_value, $matches))
 						{
 							$definition['fd'][$name]['default'] = $matches[1];
 							$column->has_default = False;
@@ -1349,7 +1349,7 @@ 
 			}
 			if ($column->has_default)
 			{
-				if (preg_match("/^'(.*)'::.*$/",$column->default_value,$matches))	// postgres
+				if (preg_match("/^'(.*)'::.*$/", $column->default_value, $matches))	// postgres
 				{
 					$column->default_value = $matches[1];
 				}
@@ -1359,18 +1359,18 @@ 
 			{
 				$definition['fd'][$name]['nullable'] = False;
 			}
-			if ($column->primary_key && !in_array($name,$definition['pk']))
+			if ($column->primary_key && !in_array($name, $definition['pk']))
 			{
 				$definition['pk'][] = $name;
 			}
 		}
-		if ($this->debug > 2) $this->debug_message("schema_proc::GetTableDefintion: MetaColumns(%1) = %2",False,$sTableName,$columns);
+		if ($this->debug > 2) $this->debug_message("schema_proc::GetTableDefintion: MetaColumns(%1) = %2", False, $sTableName, $columns);
 
 		// not all DB's (odbc) return the primary keys via MetaColumns
-		if (!count($definition['pk']) && method_exists($this->dict,'MetaPrimaryKeys') &&
+		if (!count($definition['pk']) && method_exists($this->dict, 'MetaPrimaryKeys') &&
 			is_array($primary = $this->dict->MetaPrimaryKeys($sTableName)) && count($primary))
 		{
-			if($this->capabilities['name_case'] == 'upper')
+			if ($this->capabilities['name_case'] == 'upper')
 			{
 				array_walk($primary, function(&$s)
 				{
@@ -1379,10 +1379,10 @@ 
 			}
 			$definition['pk'] = $primary;
 		}
-		if ($this->debug > 1) $this->debug_message("schema_proc::GetTableDefintion: MetaPrimaryKeys(%1) = %2",False,$sTableName,$primary);
+		if ($this->debug > 1) $this->debug_message("schema_proc::GetTableDefintion: MetaPrimaryKeys(%1) = %2", False, $sTableName, $primary);
 
 		$this->GetIndexes($sTableName, $definition);
-		if ($this->debug > 1) $this->debug_message("schema_proc::GetTableDefintion(%1) = %2",False,$sTableName,$definition);
+		if ($this->debug > 1) $this->debug_message("schema_proc::GetTableDefintion(%1) = %2", False, $sTableName, $definition);
 
 		return $definition;
 	}
@@ -1394,35 +1394,35 @@ 
 	 * @param array& $definition=array()
 	 * @return array of arrays with keys 'ix' and 'uc'
 	 */
-	public function GetIndexes($sTableName, array &$definition=array())
+	public function GetIndexes($sTableName, array &$definition = array())
 	{
-		if (method_exists($this->dict,'MetaIndexes') &&
+		if (method_exists($this->dict, 'MetaIndexes') &&
 			is_array($indexes = $this->dict->MetaIndexes($sTableName)) && count($indexes))
 		{
-			foreach($indexes as $index)
+			foreach ($indexes as $index)
 			{
 				// append (optional) length of index in brackets to column
-				foreach((array)$index['length'] as $col => $length)
+				foreach ((array)$index['length'] as $col => $length)
 				{
 					if (($key = array_search($col, $index['columns']))) $index['columns'][$key] .= '('.$length.')';
 				}
-				if($this->capabilities['name_case'] == 'upper')
+				if ($this->capabilities['name_case'] == 'upper')
 				{
 					array_walk($index['columns'], function(&$s)
 					{
 						$s = strtolower($s);
 					});
 				}
-				if (count($definition['pk']) && (implode(':',$definition['pk']) == implode(':',$index['columns']) ||
-					$index['unique'] && count(array_intersect($definition['pk'],$index['columns'])) == count($definition['pk'])))
+				if (count($definition['pk']) && (implode(':', $definition['pk']) == implode(':', $index['columns']) ||
+					$index['unique'] && count(array_intersect($definition['pk'], $index['columns'])) == count($definition['pk'])))
 				{
-					continue;	// is already the primary key => ignore it
+					continue; // is already the primary key => ignore it
 				}
 				$kind = $index['unique'] ? 'uc' : 'ix';
 
 				$definition[$kind][] = count($index['columns']) > 1 ? $index['columns'] : $index['columns'][0];
 			}
-			if ($this->debug > 2) $this->debug_message("schema_proc::GetTableDefintion: MetaIndexes(%1) = %2",False,$sTableName,$indexes);
+			if ($this->debug > 2) $this->debug_message("schema_proc::GetTableDefintion: MetaIndexes(%1) = %2", False, $sTableName, $indexes);
 		}
 		return $definition;
 	}
@@ -1434,7 +1434,7 @@ 
 	 */
 	function CheckCreateIndexes()
 	{
-		foreach($this->adodb->MetaTables('TABLES') as $table)
+		foreach ($this->adodb->MetaTables('TABLES') as $table)
 		{
 			if (!($table_def = $this->m_odb->get_table_definitions(true, $table))) continue;
 
@@ -1442,9 +1442,9 @@ 
 			$this->GetIndexes($table, $definition);
 
 			// iterate though indexes we should have according to tables_current
-			foreach(array('uc', 'ix') as $type)
+			foreach (array('uc', 'ix') as $type)
 			{
-				foreach($table_def[$type] as $columns)
+				foreach ($table_def[$type] as $columns)
 				{
 					// sometimes primary key is listed as (unique) index too --> ignore it
 					if ($this->_in_index($columns, array($table_def['pk']), true)) continue;
@@ -1475,27 +1475,26 @@ 
 	 * old translator function, use GetTableDefition() instead
 	 * @depricated
 	 */
-	function _GetColumns($oProc,$sTableName,&$sColumns)
+	function _GetColumns($oProc, $sTableName, &$sColumns)
 	{
-		unset($oProc);	// unused, but required by function signature
+		unset($oProc); // unused, but required by function signature
 		$this->sCol = $this->pk = $this->fk = $this->ix = $this->uc = array();
 
 		$tabledef = $this->GetTableDefinition($sTableName);
 
-		$sColumns = implode(',',array_keys($tabledef['fd']));
+		$sColumns = implode(',', array_keys($tabledef['fd']));
 
-		foreach($tabledef['fd'] as $column => $data)
+		foreach ($tabledef['fd'] as $column => $data)
 		{
 			$col_def = "'type' => '$data[type]'";
 			unset($data['type']);
-			foreach($data as $key => $val)
+			foreach ($data as $key => $val)
 			{
-				$col_def .= ", '$key' => ".(is_bool($val) ? ($val ? 'true' : 'false') :
-					(is_int($val) ? $val : "'$val'"));
+				$col_def .= ", '$key' => ".(is_bool($val) ? ($val ? 'true' : 'false') : (is_int($val) ? $val : "'$val'"));
 			}
 			$this->sCol[] = "\t\t\t\t'$column' => array($col_def),\n";
 		}
-		foreach(array('pk','fk','ix','uc') as $kind)
+		foreach (array('pk', 'fk', 'ix', 'uc') as $kind)
 		{
 			$this->$kind = $tabledef[$kind];
 		}

Braces +165 added lines, -43 removed lines

@@ -102,7 +102,7 @@ 
 	function __construct($dbms=False, Api\Db $db=null)
 	{
 	    if(is_object($db))
-		{
+	    {
 			$this->m_odb = $db;
 	    }
 	    else
@@ -121,7 +121,10 @@ 
 		$this->dict = NewDataDictionary($this->adodb);
 
 		// enable the debuging in ADOdb's datadictionary if the debug-level is greater then 1
-		if ($this->debug > 1) $this->dict->debug = True;
+		if ($this->debug > 1)
+		{
+			$this->dict->debug = True;
+		}
 
 		// to allow some of the former translator-functions to be called, we assign ourself as the translator
 		$this->m_oTranslator = &$this;
@@ -164,7 +167,10 @@ 
 		{
 			unset($columns['options']);
 			$columns = implode('-',$columns);
-			if ($ignore_length_limit) $columns = preg_replace('/\(\d+\)/', '', $columns);
+			if ($ignore_length_limit)
+			{
+				$columns = preg_replace('/\(\d+\)/', '', $columns);
+			}
 		}
 		foreach($indexs as $index)
 		{
@@ -173,8 +179,14 @@ 
 				unset($index['options']);
 				$index = implode('-',$index);
 			}
-			if ($ignore_length_limit) $index = preg_replace('/\(\d+\)/', '', $index);
-			if ($columns == $index) return true;
+			if ($ignore_length_limit)
+			{
+				$index = preg_replace('/\(\d+\)/', '', $index);
+			}
+			if ($columns == $index)
+			{
+				return true;
+			}
 		}
 		return false;
 	}
@@ -236,12 +248,20 @@ 
 			$options = False;
 			if (is_array($mFields))
 			{
-				if (isset($mFields['options']))		// array sets additional options
+				if (isset($mFields['options']))
+				{
+					// array sets additional options
 				{
 					if (isset($mFields['options'][$this->sType]))
 					{
-						$options = $mFields['options'][$this->sType];	// db-specific options, eg. index-type
-						if (!$options) continue;	// no index for our db-type
+						$options = $mFields['options'][$this->sType];
+				}
+				// db-specific options, eg. index-type
+						if (!$options)
+						{
+							continue;
+						}
+						// no index for our db-type
 					}
 					unset($mFields['options']);
 				}
@@ -252,15 +272,22 @@ 
 				// most DB's cant do them and give errors
 				if (in_array($aTableDef['fd'][$col]['type'],array('text','longtext')))
 				{
-					if (is_array($mFields))	// index over multiple columns including a text column
+					if (is_array($mFields))
 					{
-						$mFields[$k] .= '(32)';	// 32=limit of egw_addressbook_extra.extra_value to fix old backups
+						// index over multiple columns including a text column
+					{
+						$mFields[$k] .= '(32)';
+					}
+					// 32=limit of egw_addressbook_extra.extra_value to fix old backups
 					}
-					elseif (!$options)	// index over a single text column and no options given
+					elseif (!$options)
+					{
+						// index over a single text column and no options given
 					{
 						if ($this->sType == 'mysql')
 						{
 							$options = 'FULLTEXT';
+					}
 						}
 						else
 						{
@@ -301,8 +328,14 @@ 
 			return False;
 		}
 		// set our debug-mode or $bOutputHTML is the other one is set
-		if ($this->debug) $bOutputHTML = True;
-		if ($bOutputHTML && !$this->debug) $this->debug = 2;
+		if ($this->debug)
+		{
+			$bOutputHTML = True;
+		}
+		if ($bOutputHTML && !$this->debug)
+		{
+			$this->debug = 2;
+		}
 
 		foreach(array_keys($aTables) as $sTableName)
 		{
@@ -329,7 +362,10 @@ 
 	 */
 	function DropTable($sTableName)
 	{
-		if ($this->sType == 'pgsql') $this->_PostgresTestDropOldSequence($sTableName);
+		if ($this->sType == 'pgsql')
+		{
+			$this->_PostgresTestDropOldSequence($sTableName);
+		}
 
 		$aSql = $this->dict->DropTableSql($sTableName);
 
@@ -348,7 +384,10 @@ 
 	{
 		unset($aTableDef);	// not used, but required by function signature
 
-		if (!($table_def = $this->GetTableDefinition($sTableName))) return 0;
+		if (!($table_def = $this->GetTableDefinition($sTableName)))
+		{
+			return 0;
+		}
 		unset($table_def['fd'][$sColumnName]);
 
 		$aSql = $this->dict->DropColumnSql($sTableName,$sColumnName,$ado_table=$this->_egw2adodb_columndef($table_def));
@@ -369,7 +408,10 @@ 
 		// we create a new table, copy the content and drop the old one
 		if ($this->sType == 'pgsql')
 		{
-			if (!($table_def = $this->GetTableDefinition($sOldTableName))) return 0;
+			if (!($table_def = $this->GetTableDefinition($sOldTableName)))
+			{
+				return 0;
+			}
 
 			// only use old PostgreSQL stuff, if we have an old sequence, otherwise rely on it being new enough
 			if ($this->_PostgresHasOldSequence($sOldTableName,True) &&
@@ -403,7 +445,10 @@ 
 	 */
 	function _PostgresHasOldSequence($sTableName,$preserveValue=False)
 	{
-		if ($this->sType != 'pgsql') return false;
+		if ($this->sType != 'pgsql')
+		{
+			return false;
+		}
 
 		$seq = $this->adodb->GetOne("SELECT d.adsrc FROM pg_attribute a, pg_class c, pg_attrdef d WHERE c.relname='$sTableName' AND c.oid=d.adrelid AND d.adsrc LIKE '%seq_$sTableName''::text)' AND a.attrelid=c.oid AND d.adnum=a.attnum");
 		$seq2 = $this->adodb->GetOne("SELECT d.adsrc FROM pg_attribute a, pg_class c, pg_attrdef d WHERE c.relname='$sTableName' AND c.oid=d.adrelid AND d.adsrc LIKE '%$sTableName%_seq''::text)' AND a.attrelid=c.oid AND d.adnum=a.attnum");
@@ -411,12 +456,18 @@ 
 		$matches = null;
 		if ($seq && preg_match('/^nextval\(\'(.*)\'/',$seq,$matches))
 		{
-			if ($preserveValue) $this->pgsql_old_seq = $this->adodb->GetOne("SELECT last_value FROM " . $matches[1]);
+			if ($preserveValue)
+			{
+				$this->pgsql_old_seq = $this->adodb->GetOne("SELECT last_value FROM " . $matches[1]);
+			}
 			return $matches[1];
 		}
 		if ($seq2 && preg_match('/^nextval\(\'public\.(.*)\'/',$seq2,$matches))
 		{
-			if ($preserveValue) $this->pgsql_old_seq = $this->adodb->GetOne("SELECT last_value FROM " . $matches[1]);
+			if ($preserveValue)
+			{
+				$this->pgsql_old_seq = $this->adodb->GetOne("SELECT last_value FROM " . $matches[1]);
+			}
 			return $matches[1];
 		}
 		return false;
@@ -450,7 +501,10 @@ 
 	 */
 	function AlterColumn($sTableName, $sColumnName, $aColumnDef)
 	{
-		if (!($table_def = $this->GetTableDefinition($sTableName))) return 0;
+		if (!($table_def = $this->GetTableDefinition($sTableName)))
+		{
+			return 0;
+		}
 
 		// PostgreSQL: varchar or ascii column shortened, use substring to avoid error if current content is to long
 		if($this->sType == 'pgsql' && in_array($table_def['fd'][$sColumnName]['type'], array('varchar', 'ascii')) &&
@@ -553,8 +607,14 @@ 
 		{
 			$sIdxName = $this->_index_name($sTableName,$aColumnNames);
 		}
-		if (!is_array($options)) $options = $options ? array($options) : array();
-		if ($bUnique) $options[] = 'UNIQUE';
+		if (!is_array($options))
+		{
+			$options = $options ? array($options) : array();
+		}
+		if ($bUnique)
+		{
+			$options[] = 'UNIQUE';
+		}
 
 		$aSql = $this->dict->CreateIndexSQL($sIdxName,$sTableName,$aColumnNames,$options);
 
@@ -624,7 +684,9 @@ 
 					$seq_name = $matches[1];
 				}
 				$sql = "SELECT setval('$seq_name',MAX($sColumnName)) FROM $sTableName";
-				if($this->debug) { echo "<br>Updating sequence '$seq_name using: $sql"; }
+				if($this->debug)
+				{
+echo "<br>Updating sequence '$seq_name using: $sql"; }
 				return $this->query($sql,__LINE__,__FILE__);
 		}
 		return True;
@@ -644,7 +706,9 @@ 
 	 */
 	function RefreshTable($sTableName, $aTableDef, $aDefaults=False)
 	{
-		if($this->debug) { echo "<p>schema_proc::RefreshTable('$sTableName',"._debug_array($aTableDef,False).")\n"; }
+		if($this->debug)
+		{
+echo "<p>schema_proc::RefreshTable('$sTableName',"._debug_array($aTableDef,False).")\n"; }
 
 		$old_table_def = $this->GetTableDefinition($sTableName);
 
@@ -671,21 +735,30 @@ 
 						$value = "nextval('$sequence_name')"; break;
 				}
 			}
-			elseif ($aDefaults && isset($aDefaults[$name]))	// use given default
+			elseif ($aDefaults && isset($aDefaults[$name]))
+			{
+				// use given default
 			{
 				$value = $aDefaults[$name];
 			}
-			elseif (isset($old_table_def['fd'][$name]))	// existing column, use its value => column-name in query
+			}
+			elseif (isset($old_table_def['fd'][$name]))
+			{
+				// existing column, use its value => column-name in query
 			{
 				$value = $name;
+			}
 
-				if ($this->sType == 'pgsql')			// some postgres specific code
+				if ($this->sType == 'pgsql')
+				{
+					// some postgres specific code
 				{
 					// this is eg. necessary to change a varchar into an int column under postgres
 					if (in_array($old_table_def['fd'][$name]['type'],array('char','varchar','text','blob')) &&
 						in_array($data['type'],array('int','decimal')))
 					{
 						$value = "to_number($name,'S9999999999999D99')";
+				}
 					}
 					// blobs cant be casted to text
 					elseif($old_table_def['fd'][$name]['type'] == 'blob' && $data['type'] == 'text')
@@ -740,8 +813,15 @@ 
 		switch($this->sType)
 		{
 			case 'mssql':
-				if ($auto_column_included) $extra = "SET IDENTITY_INSERT $sTableName ON\n";
-				if ($blob_column_included) $distinct = '';	// no distinct on blob-columns
+				if ($auto_column_included)
+				{
+					$extra = "SET IDENTITY_INSERT $sTableName ON\n";
+				}
+				if ($blob_column_included)
+				{
+					$distinct = '';
+				}
+				// no distinct on blob-columns
 				break;
 		}
 		// because of all the trouble with sequences and indexes in the global namespace,
@@ -805,8 +885,14 @@ 
 			return False;
 		}
 		// set our debug-mode or $bOutputHTML is the other one is set
-		if ($this->debug) $bOutputHTML = True;
-		if ($bOutputHTML && !$this->debug) $this->debug = 2;
+		if ($this->debug)
+		{
+			$bOutputHTML = True;
+		}
+		if ($bOutputHTML && !$this->debug)
+		{
+			$this->debug = 2;
+		}
 
 		foreach($aTables as $sTableName => $aTableDef)
 		{
@@ -919,9 +1005,12 @@ 
 	 */
 	function ExecuteSqlArray($aSql,$debug_level)
 	{
-		if ($this->m_odb->query_log)	// we use Db::query to log the queries
+		if ($this->m_odb->query_log)
+		{
+			// we use Db::query to log the queries
 		{
 			$retval = 2;
+		}
 			foreach($aSql as $sql)
 			{
 				if (!$this->m_odb->query($sql,__LINE__,__FILE__))
@@ -1004,10 +1093,13 @@ 
 
 		// this code creates a fixed short index-names (30 chars) from the long and unique name, eg. for MaxDB or Oracle
 		if (isset($this->max_index_length[$this->sType]) && $this->max_index_length[$this->sType] <= 32 && strlen($name) > 30 ||
-			strlen($name) >= 64)	// even mysql has a limit here ;-)
+			strlen($name) >= 64)
+		{
+			// even mysql has a limit here ;-)
 		{
 			$name = "i".substr(md5($name),0,29);
 		}
+		}
 		return $name;
 	}
 
@@ -1242,7 +1334,10 @@ 
 			$type = method_exists($this->dict,'MetaType') ? $this->dict->MetaType($column) : strtoupper($column->type);
 
 			// fix longtext not correctly handled by ADOdb
-			if ($type == 'X' && $column->type == 'longtext') $type = 'XL';
+			if ($type == 'X' && $column->type == 'longtext')
+			{
+				$type = 'XL';
+			}
 
 			static $ado_type2egw = array(
 				'C'		=> 'varchar',
@@ -1349,10 +1444,13 @@ 
 			}
 			if ($column->has_default)
 			{
-				if (preg_match("/^'(.*)'::.*$/",$column->default_value,$matches))	// postgres
+				if (preg_match("/^'(.*)'::.*$/",$column->default_value,$matches))
+				{
+					// postgres
 				{
 					$column->default_value = $matches[1];
 				}
+				}
 				$definition['fd'][$name]['default'] = $column->default_value;
 			}
 			if ($column->not_null)
@@ -1364,7 +1462,10 @@ 
 				$definition['pk'][] = $name;
 			}
 		}
-		if ($this->debug > 2) $this->debug_message("schema_proc::GetTableDefintion: MetaColumns(%1) = %2",False,$sTableName,$columns);
+		if ($this->debug > 2)
+		{
+			$this->debug_message("schema_proc::GetTableDefintion: MetaColumns(%1) = %2",False,$sTableName,$columns);
+		}
 
 		// not all DB's (odbc) return the primary keys via MetaColumns
 		if (!count($definition['pk']) && method_exists($this->dict,'MetaPrimaryKeys') &&
@@ -1379,10 +1480,16 @@ 
 			}
 			$definition['pk'] = $primary;
 		}
-		if ($this->debug > 1) $this->debug_message("schema_proc::GetTableDefintion: MetaPrimaryKeys(%1) = %2",False,$sTableName,$primary);
+		if ($this->debug > 1)
+		{
+			$this->debug_message("schema_proc::GetTableDefintion: MetaPrimaryKeys(%1) = %2",False,$sTableName,$primary);
+		}
 
 		$this->GetIndexes($sTableName, $definition);
-		if ($this->debug > 1) $this->debug_message("schema_proc::GetTableDefintion(%1) = %2",False,$sTableName,$definition);
+		if ($this->debug > 1)
+		{
+			$this->debug_message("schema_proc::GetTableDefintion(%1) = %2",False,$sTableName,$definition);
+		}
 
 		return $definition;
 	}
@@ -1404,7 +1511,10 @@ 
 				// append (optional) length of index in brackets to column
 				foreach((array)$index['length'] as $col => $length)
 				{
-					if (($key = array_search($col, $index['columns']))) $index['columns'][$key] .= '('.$length.')';
+					if (($key = array_search($col, $index['columns'])))
+					{
+						$index['columns'][$key] .= '('.$length.')';
+					}
 				}
 				if($this->capabilities['name_case'] == 'upper')
 				{
@@ -1422,7 +1532,10 @@ 
 
 				$definition[$kind][] = count($index['columns']) > 1 ? $index['columns'] : $index['columns'][0];
 			}
-			if ($this->debug > 2) $this->debug_message("schema_proc::GetTableDefintion: MetaIndexes(%1) = %2",False,$sTableName,$indexes);
+			if ($this->debug > 2)
+			{
+				$this->debug_message("schema_proc::GetTableDefintion: MetaIndexes(%1) = %2",False,$sTableName,$indexes);
+			}
 		}
 		return $definition;
 	}
@@ -1436,7 +1549,10 @@ 
 	{
 		foreach($this->adodb->MetaTables('TABLES') as $table)
 		{
-			if (!($table_def = $this->m_odb->get_table_definitions(true, $table))) continue;
+			if (!($table_def = $this->m_odb->get_table_definitions(true, $table)))
+			{
+				continue;
+			}
 
 			$definition = array();
 			$this->GetIndexes($table, $definition);
@@ -1447,7 +1563,10 @@ 
 				foreach($table_def[$type] as $columns)
 				{
 					// sometimes primary key is listed as (unique) index too --> ignore it
-					if ($this->_in_index($columns, array($table_def['pk']), true)) continue;
+					if ($this->_in_index($columns, array($table_def['pk']), true))
+					{
+						continue;
+					}
 
 					// check if they exist in real table and create them if not
 					if (!$this->_in_index($columns, $definition[$type]) &&
@@ -1458,7 +1577,10 @@ 
 						if ($this->_in_index($columns, $definition[$type], true))
 						{
 							// for PostgreSQL we dont use length-limited indexes --> nothing to do
-							if ($this->m_odb->Type == 'pgsql') continue;
+							if ($this->m_odb->Type == 'pgsql')
+							{
+								continue;
+							}
 							// for MySQL we drop current index and create it with correct length
 							$this->DropIndex($table, $columns);
 						}

api/src/Etemplate.php

Doc Comments +1 added lines, -1 removed lines

@@ -543,7 +543,7 @@
 	 *
 	 * @param string $name cell-name
 	 * @param string $attr attribute-name
-	 * @param mixed $val if not NULL sets attribute else returns it
+	 * @param boolean|string $val if not NULL sets attribute else returns it
 	 * @return reference to attribute
 	 * @deprecated use setElementAttribute($name, $attr, $val)
 	 */

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -158,7 +158,7 @@
 		// instanciate template to fill self::$request->sel_options for select-* widgets
 		// not sure if we want to handle it this way, thought otherwise we will have a few ajax request for each dialog fetching predefined selectboxes
 		$template = self::instance($this->name, $this->template_set, $this->version, $this->laod_via);
-		if (!$template) throw new Exception\AssertionFailed("Template $this->name not instanciable! Maybe you forgot to rename template id.");
+		if (!$template) throw new Exception\AssertionFailed("template $this->name not instanciable! Maybe you forgot to rename template id.");
 		$template->run('beforeSendToClient', array('', array('cont'=>$content)));
 
 		// some apps (eg. InfoLog) set app_header only in get_rows depending on filter settings

Spacing +58 added lines, -59 removed lines

@@ -29,7 +29,7 @@ 
 	 *
 	 * @public boolean
 	 */
-	public $sitemgr=false;
+	public $sitemgr = false;
 
 	/**
 	 * Tell egw framework it's ok to call this
@@ -44,17 +44,17 @@ 
 	 * @param string $name of etemplate or array with name and other keys
 	 * @param string|array $load_via with keys of other etemplate to load in order to get $name
 	 */
-	function __construct($name='',$load_via='')
+	function __construct($name = '', $load_via = '')
 	{
 		// we do NOT call parent consturctor, as we only want to enherit it's (static) methods
-		if (false) parent::__construct ($name);	// satisfy IDE, as we dont call parent constructor
+		if (false) parent::__construct($name); // satisfy IDE, as we dont call parent constructor
 
 		$this->sitemgr = isset($GLOBALS['Common_BO']) && is_object($GLOBALS['Common_BO']);
 
-		if ($name) $this->read($name,$template='default','default',0,'',$load_via);
+		if ($name) $this->read($name, $template = 'default', 'default', 0, '', $load_via);
 
 		// generate new etemplate request object, if not already existing
-		if(!isset(self::$request)) self::$request = Etemplate\Request::read();
+		if (!isset(self::$request)) self::$request = Etemplate\Request::read();
 	}
 
 	/**
@@ -65,7 +65,7 @@ 
 	 *
 	 * @param string|array $params url or array with get-params incl. menuaction
 	 */
-	static function location($params='')
+	static function location($params = '')
 	{
 		Framework::redirect_link(is_array($params) ? '/index.php' : $params,
 			is_array($params) ? $params : '');
@@ -100,7 +100,7 @@ 
 	 * @param array $changes change made in the last call if looping, only used internaly by process_exec
 	 * @return string html for $output_mode == 1, else nothing
 	 */
-	function exec($method,array $content,array $sel_options=null,array $readonlys=null,array $preserv=null,$output_mode=0,$ignore_validation='',array $changes=null)
+	function exec($method, array $content, array $sel_options = null, array $readonlys = null, array $preserv = null, $output_mode = 0, $ignore_validation = '', array $changes = null)
 	{
 		$hook_data = Hooks::process(
 			array('hook_location'   => 'etemplate2_before_exec') +
@@ -109,11 +109,11 @@ 
 			$content
 		);
 
-		foreach($hook_data as $extras)
+		foreach ($hook_data as $extras)
 		{
 			if (!$extras) continue;
 
-			foreach(isset($extras[0]) ? $extras : array($extras) as $extra)
+			foreach (isset($extras[0]) ? $extras : array($extras) as $extra)
 			{
 				if ($extra['data'] && is_array($extra['data']))
 				{
@@ -143,7 +143,7 @@ 
 			$output_mode = 0;
 			self::$response = Json\Response::get();
 		}
-		self::$request->output_mode = $output_mode;	// let extensions "know" they are run eg. in a popup
+		self::$request->output_mode = $output_mode; // let extensions "know" they are run eg. in a popup
 		self::$request->content = self::$cont = $content;
 		self::$request->changes = $changes;
 		self::$request->sel_options = is_array($sel_options) ? self::fix_sel_options($sel_options) : array();
@@ -166,15 +166,15 @@ 
 
 		// compile required translations translations
 		$currentapp = $GLOBALS['egw_info']['flags']['currentapp'];
-		$langRequire = array('common' => array(), 'etemplate' => array());	// keep that order
-		foreach(Translation::$loaded_apps as $l_app => $lang)
+		$langRequire = array('common' => array(), 'etemplate' => array()); // keep that order
+		foreach (Translation::$loaded_apps as $l_app => $lang)
 		{
 			if (!in_array($l_app, array($currentapp, 'custom')))
 			{
 				$langRequire[$l_app] = array('app' => $l_app, 'lang' => $lang, 'etag' => Translation::etag($l_app, $lang));
 			}
 		}
-		foreach(array($currentapp, 'custom') as $l_app)
+		foreach (array($currentapp, 'custom') as $l_app)
 		{
 			if (isset(Translation::$loaded_apps[$l_app]))
 			{
@@ -195,7 +195,7 @@ 
 		);
 
 		// Info required to load the etemplate client-side
-		$dom_id = str_replace('.','-',$this->dom_id);
+		$dom_id = str_replace('.', '-', $this->dom_id);
 		$load_array = array(
 			'name' => $this->name,
 			'url' => self::rel2url($this->rel_path),
@@ -205,16 +205,16 @@ 
 		if (self::$response)	// call is within an ajax event / form submit
 		{
 			//error_log("Ajax " . __LINE__);
-			self::$response->generic('et2_load', $load_array+Framework::get_extra());
-			Framework::clear_extra();	// to not send/set it twice for multiple etemplates (eg. CRM view)
+			self::$response->generic('et2_load', $load_array + Framework::get_extra());
+			Framework::clear_extra(); // to not send/set it twice for multiple etemplates (eg. CRM view)
 		}
 		else	// first call
 		{
 			// check if application of template has a app.js file --> load it
-			list($app) = explode('.',$this->name);
+			list($app) = explode('.', $this->name);
 			if (file_exists(EGW_SERVER_ROOT.'/'.$app.'/js/app.js'))
 			{
-				Framework::includeJS('.','app',$app,false);
+				Framework::includeJS('.', 'app', $app, false);
 			}
 			// Category styles
 			Categories::css($app);
@@ -232,8 +232,8 @@ 
 				$content = '<form target="egw_iframe_autocomplete_helper" action="'.$form_action.'" id="'.$dom_id.'" class="et2_container"></form>'."\n".
 					'<iframe name="egw_iframe_autocomplete_helper" style="width:0;height:0;position: absolute;visibility:hidden;"></iframe>';
 				$GLOBALS['egw']->framework->response->generic("data", array($content));
-				$GLOBALS['egw']->framework->response->generic('et2_load',$load_array+Framework::get_extra());
-				Framework::clear_extra();	// to not send/set it twice for multiple etemplates (eg. CRM view)
+				$GLOBALS['egw']->framework->response->generic('et2_load', $load_array + Framework::get_extra());
+				Framework::clear_extra(); // to not send/set it twice for multiple etemplates (eg. CRM view)
 				self::$request = null;
 				return;
 			}
@@ -252,7 +252,7 @@ 
 				echo '<div id="popupMainDiv" class="popupMainDiv">'."\n";
 			}
 			// Send any accumulated json responses - after flush to avoid sending the buffer as a response
-			if(Json\Response::isJSONResponse())
+			if (Json\Response::isJSONResponse())
 			{
 				$load_array['response'] = Json\Response::get()->returnResult();
 			}
@@ -279,10 +279,10 @@ 
 	 */
 	static protected function fix_sel_options(array $sel_options)
 	{
-		foreach($sel_options as &$options)
+		foreach ($sel_options as &$options)
 		{
-			if (!is_array($options)||empty($options)) continue;
-			foreach($options as $key => $value)
+			if (!is_array($options) || empty($options)) continue;
+			foreach ($options as $key => $value)
 			{
 				if (is_numeric($key) && (!is_array($value) || !isset($value['value'])))
 				{
@@ -321,9 +321,9 @@ 
 		$GLOBALS['egw']->session->set_action('Etemplate: '.self::$request->method);
 
 		// Set current app for validation
-		list($app) = explode('.',self::$request->method);
-		if(!$app) list($app) = explode('::',self::$request->method);
-		if($app)
+		list($app) = explode('.', self::$request->method);
+		if (!$app) list($app) = explode('::', self::$request->method);
+		if ($app)
 		{
 			Translation::add_app($app);
 			$GLOBALS['egw_info']['flags']['currentapp'] = $app;
@@ -332,7 +332,7 @@ 
 		$expand = array(
 			'cont' => &self::$request->content,
 		);
-		$template->run('validate', array('', $expand, $_content, &$validated), true);	// $respect_disabled=true: do NOT validate disabled widgets and children
+		$template->run('validate', array('', $expand, $_content, &$validated), true); // $respect_disabled=true: do NOT validate disabled widgets and children
 
 		if ($no_validation)
 		{
@@ -348,14 +348,14 @@ 
 		// tell request call to remove request, if it is not modified eg. by call to exec in callback
 		self::$request->remove_if_not_modified();
 
-		foreach(Hooks::process(array(
+		foreach (Hooks::process(array(
 			'hook_location'   => 'etemplate2_before_process',
 			'location_name'   => $template->id,
 		) + self::complete_array_merge(self::$request->preserv, $validated)) as $extras)
 		{
 			if (!$extras) continue;
 
-			foreach(isset($extras[0]) ? $extras : array($extras) as $extra)
+			foreach (isset($extras[0]) ? $extras : array($extras) as $extra)
 			{
 				if ($extra['data'] && is_array($extra['data']))
 				{
@@ -368,8 +368,7 @@ 
 		//error_log(' validated='.array2string($validated));
 		$content = ExecMethod(self::$request->method, self::complete_array_merge(self::$request->preserv, $validated));
 
-		$tcontent = is_array($content) ? $content :
-			self::complete_array_merge(self::$request->preserv, $validated);
+		$tcontent = is_array($content) ? $content : self::complete_array_merge(self::$request->preserv, $validated);
 
 		$hook_data = Hooks::process(
 			array(
@@ -381,11 +380,11 @@ 
 
 		if (is_array($content))
 		{
-			foreach($hook_data as $extras)
+			foreach ($hook_data as $extras)
 			{
 				if (!$extras) continue;
 
-				foreach(isset($extras[0]) ? $extras : array($extras) as $extra) {
+				foreach (isset($extras[0]) ? $extras : array($extras) as $extra) {
 					if ($extra['data'] && is_array($extra['data'])) {
 						$content = array_merge($content, $extra['data']);
 					}
@@ -397,7 +396,7 @@ 
 		if (isset($GLOBALS['egw_info']['flags']['java_script']))
 		{
 			// Strip out any script tags
-			$GLOBALS['egw_info']['flags']['java_script'] = preg_replace(array('/(<script[^>]*>)([^<]*)/is','/<\/script>/'),array('$2',''),$GLOBALS['egw_info']['flags']['java_script']);
+			$GLOBALS['egw_info']['flags']['java_script'] = preg_replace(array('/(<script[^>]*>)([^<]*)/is', '/<\/script>/'), array('$2', ''), $GLOBALS['egw_info']['flags']['java_script']);
 			self::$response->script($GLOBALS['egw_info']['flags']['java_script']);
 			//error_log($app .' added javascript to $GLOBALS[egw_info][flags][java_script] - use Json\Response->script() instead.');
 		}
@@ -426,7 +425,7 @@ 
 	static public function process_exec()
 	{
 		if (get_magic_quotes_gpc()) $_POST['value'] = stripslashes($_POST['value']);
-		$content = json_decode($_POST['value'],true);
+		$content = json_decode($_POST['value'], true);
 		//error_log(__METHOD__."(".array2string($content).")");
 
 		self::$request = Etemplate\Request::read($_POST['etemplate_exec_id']);
@@ -440,7 +439,7 @@ 
 		$expand = array(
 			'cont' => &self::$request->content,
 		);
-		$template->run('validate', array('', $expand, $content, &$validated), true);	// $respect_disabled=true: do NOT validate disabled widgets and children
+		$template->run('validate', array('', $expand, $content, &$validated), true); // $respect_disabled=true: do NOT validate disabled widgets and children
 		if (self::validation_errors(self::$request->ignore_validation))
 		{
 			error_log(__METHOD__."(,".array2string($content).') validation_errors='.array2string(self::$validation_errors));
@@ -476,7 +475,7 @@ 
 	 *
 	 * @ToDo supported customized templates stored in DB
 	 */
-	public function read($name,$template_set=null,$lang='default',$group=0,$version='',$load_via='')
+	public function read($name, $template_set = null, $lang = 'default', $group = 0, $version = '', $load_via = '')
 	{
 
 		// For mobile experience try to load custom mobile templates
@@ -485,9 +484,9 @@ 
 			$template_set = "mobile";
 		}
 
-		unset($lang); unset($group);	// not used, but in old signature
-		$this->rel_path = self::relPath($this->name=$name, $this->template_set=$template_set,
-			$this->version=$version, $this->laod_via = $load_via);
+		unset($lang); unset($group); // not used, but in old signature
+		$this->rel_path = self::relPath($this->name = $name, $this->template_set = $template_set,
+			$this->version = $version, $this->laod_via = $load_via);
 		//error_log(__METHOD__."('$name', '$template_set', '$lang', $group, '$version', '$load_via') rel_path=".array2string($this->rel_path));
 
 		$this->dom_id = $name;
@@ -530,7 +529,7 @@ 
 	 * @return mixed reference to attribute, usually NULL
 	 * @deprecated use getElementAttribute($name, $attr)
 	 */
-	public function &get_cell_attribute($name,$attr)
+	public function &get_cell_attribute($name, $attr)
 	{
 		return self::getElementAttribute($name, $attr);
 	}
@@ -544,7 +543,7 @@ 
 	 * @return reference to attribute
 	 * @deprecated use setElementAttribute($name, $attr, $val)
 	 */
-	public function &set_cell_attribute($name,$attr,$val)
+	public function &set_cell_attribute($name, $attr, $val)
 	{
 		return self::setElementAttribute($name, $attr, $val);
 	}
@@ -557,7 +556,7 @@ 
 	 * @return reference to attribute
 	 * @deprecated use disableElement($name, $disabled=true)
 	 */
-	public function disable_cells($name,$disabled=True)
+	public function disable_cells($name, $disabled = True)
 	{
 		return self::disableElement($name, $disabled);
 	}
@@ -573,23 +572,23 @@ 
 	 * @param array $new
 	 * @return array the merged array
 	 */
-	public static function complete_array_merge($old,$new)
+	public static function complete_array_merge($old, $new)
 	{
 		if (is_array($new))
 		{
-			if (!is_array($old)) $old = (array) $old;
+			if (!is_array($old)) $old = (array)$old;
 
-			foreach($new as $k => $v)
+			foreach ($new as $k => $v)
 			{
-				if (!is_array($v) || !isset($old[$k]) || 	// no array or a new array
-					isset($v[0]) && !is_array($v[0]) && isset($v[count($v)-1])	|| // or no associative array, eg. selecting multiple accounts
+				if (!is_array($v) || !isset($old[$k]) || // no array or a new array
+					isset($v[0]) && !is_array($v[0]) && isset($v[count($v) - 1]) || // or no associative array, eg. selecting multiple accounts
 					is_array($v) && count($v) == 0)			// Empty array replacing non-empty
 				{
 					$old[$k] = $v;
 				}
 				else
 				{
-					$old[$k] = self::complete_array_merge($old[$k],$v);
+					$old[$k] = self::complete_array_merge($old[$k], $v);
 				}
 			}
 		}
@@ -601,7 +600,7 @@ 
 	 *
 	 * @param array $content =null
 	 */
-	public function debug(array $content=null)
+	public function debug(array $content = null)
 	{
 		$GLOBALS['egw']->framework->render(print_r($content, true));
 	}
@@ -615,11 +614,11 @@ 
 	 * @param int &$max_upload=null on return max. upload size in byte
 	 * @return string
 	 */
-	static function max_upload_size_message(&$max_upload=null)
+	static function max_upload_size_message(&$max_upload = null)
 	{
 		$upload_max_filesize = ini_get('upload_max_filesize');
 		$post_max_size = ini_get('post_max_size');
-		$max_upload = min(self::km2int($upload_max_filesize),self::km2int($post_max_size)-2800);
+		$max_upload = min(self::km2int($upload_max_filesize), self::km2int($post_max_size) - 2800);
 
 		return lang('Maximum size for uploads').': '.Vfs::hsize($max_upload).
 			" (php.ini: upload_max_filesize=$upload_max_filesize, post_max_size=$post_max_size)";
@@ -633,9 +632,9 @@ 
 	 * @param boolean $readonly =true
 	 * @return string
 	 */
-	static public function number_format($number,$num_decimal_places=2,$readonly=true)
+	static public function number_format($number, $num_decimal_places = 2, $readonly = true)
 	{
-		static $dec_separator=null,$thousands_separator=null;
+		static $dec_separator = null, $thousands_separator = null;
 		if (is_null($dec_separator))
 		{
 			$dec_separator = $GLOBALS['egw_info']['user']['preferences']['common']['number_format'][0];
@@ -644,7 +643,7 @@ 
 		}
 		if ((string)$number === '') return '';
 
-		return number_format(str_replace(' ','',$number),$num_decimal_places,$dec_separator,$readonly ? $thousands_separator : '');
+		return number_format(str_replace(' ', '', $number), $num_decimal_places, $dec_separator, $readonly ? $thousands_separator : '');
 	}
 
 	/**
@@ -657,13 +656,13 @@ 
 	{
 		if (!is_numeric($size))
 		{
-			switch(strtolower(substr($size,-1)))
+			switch (strtolower(substr($size, -1)))
 			{
 				case 'm':
-					$size = 1024*1024*(int)$size;
+					$size = 1024 * 1024 * (int)$size;
 					break;
 				case 'k':
-					$size = 1024*(int)$size;
+					$size = 1024 * (int)$size;
 					break;
 			}
 		}

Braces +81 added lines, -21 removed lines

@@ -47,14 +47,24 @@ 
 	function __construct($name='',$load_via='')
 	{
 		// we do NOT call parent consturctor, as we only want to enherit it's (static) methods
-		if (false) parent::__construct ($name);	// satisfy IDE, as we dont call parent constructor
+		if (false)
+		{
+			parent::__construct ($name);
+		}
+		// satisfy IDE, as we dont call parent constructor
 
 		$this->sitemgr = isset($GLOBALS['Common_BO']) && is_object($GLOBALS['Common_BO']);
 
-		if ($name) $this->read($name,$template='default','default',0,'',$load_via);
+		if ($name)
+		{
+			$this->read($name,$template='default','default',0,'',$load_via);
+		}
 
 		// generate new etemplate request object, if not already existing
-		if(!isset(self::$request)) self::$request = Etemplate\Request::read();
+		if(!isset(self::$request))
+		{
+			self::$request = Etemplate\Request::read();
+		}
 	}
 
 	/**
@@ -111,7 +121,10 @@ 
 
 		foreach($hook_data as $extras)
 		{
-			if (!$extras) continue;
+			if (!$extras)
+			{
+				continue;
+			}
 
 			foreach(isset($extras[0]) ? $extras : array($extras) as $extra)
 			{
@@ -136,7 +149,10 @@ 
 		// Include the etemplate2 javascript code
 		Framework::includeJS('etemplate', 'etemplate2', 'api');
 
-		if (!$this->rel_path) throw new Exception\AssertionFailed("No (valid) template '$this->name' found!");
+		if (!$this->rel_path)
+		{
+			throw new Exception\AssertionFailed("No (valid) template '$this->name' found!");
+		}
 
 		if ($output_mode == 4)
 		{
@@ -151,14 +167,23 @@ 
 		self::$request->preserv = $preserv ? $preserv : array();
 		self::$request->method = $method;
 		self::$request->ignore_validation = $ignore_validation;
-		if (self::$request->output_mode == -1) self::$request->output_mode = 0;
+		if (self::$request->output_mode == -1)
+		{
+			self::$request->output_mode = 0;
+		}
 		self::$request->template = $this->as_array();
 
-		if (empty($this->name)) throw new Exception\AssertionFailed("Template  name is not set '$this->name' !");
+		if (empty($this->name))
+		{
+			throw new Exception\AssertionFailed("Template  name is not set '$this->name' !");
+		}
 		// instanciate template to fill self::$request->sel_options for select-* widgets
 		// not sure if we want to handle it this way, thought otherwise we will have a few ajax request for each dialog fetching predefined selectboxes
 		$template = self::instance($this->name, $this->template_set, $this->version, $this->laod_via);
-		if (!$template) throw new Exception\AssertionFailed("Template $this->name not instanciable! Maybe you forgot to rename template id.");
+		if (!$template)
+		{
+			throw new Exception\AssertionFailed("Template $this->name not instanciable! Maybe you forgot to rename template id.");
+		}
 		$template->run('beforeSendToClient', array('', array('cont'=>$content)));
 
 		// some apps (eg. InfoLog) set app_header only in get_rows depending on filter settings
@@ -202,10 +227,13 @@ 
 			'data' => $data,
 			'DOMNodeID' => $dom_id,
 		);
-		if (self::$response)	// call is within an ajax event / form submit
+		if (self::$response)
+		{
+			// call is within an ajax event / form submit
 		{
 			//error_log("Ajax " . __LINE__);
 			self::$response->generic('et2_load', $load_array+Framework::get_extra());
+		}
 			Framework::clear_extra();	// to not send/set it twice for multiple etemplates (eg. CRM view)
 		}
 		else	// first call
@@ -268,7 +296,10 @@ 
 				echo $GLOBALS['egw']->framework->footer();
 			}
 			ob_flush();
-			if ($output_mode == 1)	return $form;
+			if ($output_mode == 1)
+			{
+				return $form;
+			}
 		}
 		self::$request = null;
 	}
@@ -283,7 +314,10 @@ 
 	{
 		foreach($sel_options as &$options)
 		{
-			if (!is_array($options)||empty($options)) continue;
+			if (!is_array($options)||empty($options))
+			{
+				continue;
+			}
 			foreach($options as $key => $value)
 			{
 				if (is_numeric($key) && (!is_array($value) || !isset($value['value'])))
@@ -324,7 +358,10 @@ 
 
 		// Set current app for validation
 		list($app) = explode('.',self::$request->method);
-		if(!$app) list($app) = explode('::',self::$request->method);
+		if(!$app)
+		{
+			list($app) = explode('::',self::$request->method);
+		}
 		if($app)
 		{
 			Translation::add_app($app);
@@ -355,7 +392,10 @@ 
 			'location_name'   => $template->id,
 		) + self::complete_array_merge(self::$request->preserv, $validated)) as $extras)
 		{
-			if (!$extras) continue;
+			if (!$extras)
+			{
+				continue;
+			}
 
 			foreach(isset($extras[0]) ? $extras : array($extras) as $extra)
 			{
@@ -385,10 +425,15 @@ 
 		{
 			foreach($hook_data as $extras)
 			{
-				if (!$extras) continue;
+				if (!$extras)
+				{
+					continue;
+				}
 
-				foreach(isset($extras[0]) ? $extras : array($extras) as $extra) {
-					if ($extra['data'] && is_array($extra['data'])) {
+				foreach(isset($extras[0]) ? $extras : array($extras) as $extra)
+				{
+					if ($extra['data'] && is_array($extra['data']))
+					{
 						$content = array_merge($content, $extra['data']);
 					}
 				}
@@ -427,7 +472,10 @@ 
 	 */
 	static public function process_exec()
 	{
-		if (get_magic_quotes_gpc()) $_POST['value'] = stripslashes($_POST['value']);
+		if (get_magic_quotes_gpc())
+		{
+			$_POST['value'] = stripslashes($_POST['value']);
+		}
 		$content = json_decode($_POST['value'],true);
 		//error_log(__METHOD__."(".array2string($content).")");
 
@@ -579,16 +627,22 @@ 
 	{
 		if (is_array($new))
 		{
-			if (!is_array($old)) $old = (array) $old;
+			if (!is_array($old))
+			{
+				$old = (array) $old;
+			}
 
 			foreach($new as $k => $v)
 			{
 				if (!is_array($v) || !isset($old[$k]) || 	// no array or a new array
 					isset($v[0]) && !is_array($v[0]) && isset($v[count($v)-1])	|| // or no associative array, eg. selecting multiple accounts
-					is_array($v) && count($v) == 0)			// Empty array replacing non-empty
+					is_array($v) && count($v) == 0)
+				{
+					// Empty array replacing non-empty
 				{
 					$old[$k] = $v;
 				}
+				}
 				else
 				{
 					$old[$k] = self::complete_array_merge($old[$k],$v);
@@ -641,10 +695,16 @@ 
 		if (is_null($dec_separator))
 		{
 			$dec_separator = $GLOBALS['egw_info']['user']['preferences']['common']['number_format'][0];
-			if (empty($dec_separator)) $dec_separator = '.';
+			if (empty($dec_separator))
+			{
+				$dec_separator = '.';
+			}
 			$thousands_separator = $GLOBALS['egw_info']['user']['preferences']['common']['number_format'][1];
 		}
-		if ((string)$number === '') return '';
+		if ((string)$number === '')
+		{
+			return '';
+		}
 
 		return number_format(str_replace(' ','',$number),$num_decimal_places,$dec_separator,$readonly ? $thousands_separator : '');
 	}

api/src/Etemplate/Widget.php

Doc Comments +1 added lines, -1 removed lines

@@ -155,7 +155,7 @@
 	 *
 	 * @param string|XMLReader $xml
 	 * @param boolean $cloned =true true: object does NOT need to be cloned, false: to set attribute, set them in cloned object
-	 * @return Template current object or clone, if any attribute was set
+	 * @return Widget current object or clone, if any attribute was set
 	 */
 	public function set_attrs($xml, $cloned=true)
 	{

Indentation +6 added lines, -6 removed lines

@@ -903,12 +903,12 @@
 	}
 
 	/**
-	* Check if we have not ignored validation errors
-	*
-	* @param string $ignore_validation ='' if not empty regular expression for validation-errors to ignore
-	* @param string $cname =null name-prefix, which need to be ignored, default self::$name_vars
-	* @return boolean true if there are not ignored validation errors, false otherwise
-	*/
+	 * Check if we have not ignored validation errors
+	 *
+	 * @param string $ignore_validation ='' if not empty regular expression for validation-errors to ignore
+	 * @param string $cname =null name-prefix, which need to be ignored, default self::$name_vars
+	 * @return boolean true if there are not ignored validation errors, false otherwise
+	 */
 	public static function validation_errors($ignore_validation='',$cname='')
 	{
 		// not yet used: if (is_null($cname)) $cname = self::$name_vars;

Spacing +104 added lines, -104 removed lines

@@ -99,9 +99,9 @@ 
 		$this->id = $reader->getAttribute('id');
 
 		// Update content?
-		if(self::$cont == null)
+		if (self::$cont == null)
 			self::$cont = is_array(self::$request->content) ? self::$request->content : array();
-		if($this->id && is_array(self::$cont[$this->id]))
+		if ($this->id && is_array(self::$cont[$this->id]))
 		{
 			$old_cont = self::$cont;
 			self::$cont = self::$cont[$this->id];
@@ -110,7 +110,7 @@ 
 		// read all attributes
 		$this->set_attrs($reader);
 
-		while($reader->read() && $reader->depth > $depth)
+		while ($reader->read() && $reader->depth > $depth)
 		{
 			if ($reader->nodeType == XMLReader::ELEMENT && $reader->depth > $depth)
 			{
@@ -119,7 +119,7 @@ 
 		}
 
 		// Reset content as we leave
-		if($old_cont) {
+		if ($old_cont) {
 			self::$cont = $old_cont;
 		}
 	}
@@ -157,7 +157,7 @@ 
 	 * @param boolean $cloned =true true: object does NOT need to be cloned, false: to set attribute, set them in cloned object
 	 * @return Template current object or clone, if any attribute was set
 	 */
-	public function set_attrs($xml, $cloned=true)
+	public function set_attrs($xml, $cloned = true)
 	{
 		$reader = self::get_reader($xml);
 
@@ -174,14 +174,14 @@ 
 
 		// read and set all attributes
 		$template = $this;
-		while($reader->moveToNextAttribute())
+		while ($reader->moveToNextAttribute())
 		{
 			if ($reader->name != 'id' && $template->attr[$reader->name] != $reader->value)
 			{
 				if (!$cloned)
 				{
 					$template = clone($this);
-					$cloned = true;	// only clone it once, otherwise we loose attributes!
+					$cloned = true; // only clone it once, otherwise we loose attributes!
 				}
 				// $reader->value is an object and therefore assigned by reference
 				// this is important to not loose content when validating dynamic generated tabs as in settings!
@@ -198,7 +198,7 @@ 
 				if ($legacy_options && $reader->name == 'options')
 				{
 					$legacy_options = explode(',', $legacy_options);
-					foreach(self::csv_split($value, count($legacy_options)) as $n => $val)
+					foreach (self::csv_split($value, count($legacy_options)) as $n => $val)
 					{
 						if ($legacy_options[$n] && (string)$val !== '') $template->attrs[$legacy_options[$n]] = $val;
 					}
@@ -207,9 +207,9 @@ 
 		}
 
 		// Add in anything in the modification array
-		if(is_array(self::$request->modifications[$this->id]))
+		if (is_array(self::$request->modifications[$this->id]))
 		{
-			$this->attrs = array_merge($this->attrs,self::$request->modifications[$this->id]);
+			$this->attrs = array_merge($this->attrs, self::$request->modifications[$this->id]);
 		}
 
 		return $template;
@@ -230,32 +230,32 @@ 
 	 * @param string $enclosure ='"'
 	 * @return array
 	 */
-	public static function csv_split($str,$num=null,$delimiter=',',$enclosure='"')
+	public static function csv_split($str, $num = null, $delimiter = ',', $enclosure = '"')
 	{
-		if (strpos($str,$enclosure) === false)
+		if (strpos($str, $enclosure) === false)
 		{
-			return is_null($num) ? explode($delimiter,$str) : explode($delimiter,$str,$num);	// no need to run this more expensive code
+			return is_null($num) ? explode($delimiter, $str) : explode($delimiter, $str, $num); // no need to run this more expensive code
 		}
-		$parts = explode($delimiter,$str);
-		for($n = 0; isset($parts[$n]); ++$n)
+		$parts = explode($delimiter, $str);
+		for ($n = 0; isset($parts[$n]); ++$n)
 		{
-			$part =& $parts[$n];
+			$part = & $parts[$n];
 			if ($part[0] === $enclosure)
 			{
-				while (isset($parts[$n+1]) && substr($part,-1) !== $enclosure)
+				while (isset($parts[$n + 1]) && substr($part, -1) !== $enclosure)
 				{
 					$part .= $delimiter.$parts[++$n];
 					unset($parts[$n]);
 				}
-				$part = substr(str_replace($enclosure.$enclosure,$enclosure,$part),1,-1);
+				$part = substr(str_replace($enclosure.$enclosure, $enclosure, $part), 1, -1);
 			}
 		}
-		$parts_renum = array_values($parts);	// renumber the parts (in case we had to concat them)
+		$parts_renum = array_values($parts); // renumber the parts (in case we had to concat them)
 
 		if ($num > 0 && count($parts_renum) > $num)
 		{
-			$parts_renum[$num-1] = implode($delimiter,array_slice($parts_renum,$num-1,count($parts_renum)-$num+1));
-			$parts_renum = array_slice($parts_renum,0,$num);
+			$parts_renum[$num - 1] = implode($delimiter, array_slice($parts_renum, $num - 1, count($parts_renum) - $num + 1));
+			$parts_renum = array_slice($parts_renum, 0, $num);
 		}
 		return $parts_renum;
 	}
@@ -282,7 +282,7 @@ 
 		{
 			throw new Api\Exception\WrongParameter(__METHOD__."('$class', ".array2string($widgets).") $class is no subclass of ".__CLASS__.'!');
 		}
-		foreach((array)$widgets as $widget)
+		foreach ((array)$widgets as $widget)
 		{
 			self::$widget_registry[$widget] = $class;
 		}
@@ -307,15 +307,15 @@ 
 
 		if (!$widget_registry)	// not in instance cache --> rescan from filesystem
 		{
-			foreach(new \RecursiveIteratorIterator(new \RecursiveDirectoryIterator(__DIR__.'/Widget')) as $path)
+			foreach (new \RecursiveIteratorIterator(new \RecursiveDirectoryIterator(__DIR__.'/Widget')) as $path)
 			{
-				if(substr($path, -4) == '.php')
+				if (substr($path, -4) == '.php')
 				{
 					try
 					{
 						include_once($path);
 					}
-					catch(Exception $e)
+					catch (Exception $e)
 					{
 						error_log($e->getMessage());
 					}
@@ -323,18 +323,18 @@ 
 			}
 
 			// Use hook to load custom widgets from other apps
-			$widgets = Api\Hooks::process('etemplate2_register_widgets',array(),true);
-			foreach($widgets as $list)
+			$widgets = Api\Hooks::process('etemplate2_register_widgets', array(), true);
+			foreach ($widgets as $list)
 			{
 				if (is_array($list))
 				{
-					foreach($list as $class)
+					foreach ($list as $class)
 					{
 						try
 						{
-							class_exists($class);	// trigger autoloader
+							class_exists($class); // trigger autoloader
 						}
-						catch(Exception $e)
+						catch (Exception $e)
 						{
 							error_log($e->getMessage());
 						}
@@ -357,22 +357,22 @@ 
 	 * @param string|XMLReader $xml
 	 * @param string $id =null
 	 */
-	public static function factory($type, $xml, $id=null)
+	public static function factory($type, $xml, $id = null)
 	{
-		$class_name =& self::$widget_registry[$type];
+		$class_name = & self::$widget_registry[$type];
 
 		if (!isset($class_name))
 		{
-			list($basetype) = explode('-',$type);
+			list($basetype) = explode('-', $type);
 			if (//dont think this is used: !class_exists($class_name = 'etemplate_widget_'.str_replace('-','_',$type)) &&
 				!class_exists($class_name = __CLASS__.'\\'.ucfirst($basetype)) &&
 				// widgets supplied by application in class ${app}_widget_etemplate or ${app}_${subtype}_widget_etemplate
 				!(isset($GLOBALS['egw_info']['apps'][$basetype]) &&
-					(class_exists($class_name = str_replace('-','_',$type).'_etemplate_widget') ||
+					(class_exists($class_name = str_replace('-', '_', $type).'_etemplate_widget') ||
 					 class_exists($class_name = $basetype.'_etemplate_widget'))))
 			{
 				// Try for base type, it's probably better than the root
-				if(self::$widget_registry[$basetype] && self::$widget_registry[$basetype] != $class_name)
+				if (self::$widget_registry[$basetype] && self::$widget_registry[$basetype] != $class_name)
 				{
 					$class_name = self::$widget_registry[$basetype];
 				}
@@ -393,22 +393,22 @@ 
 		{
 			// Class in widget registry, but not loaded
 			// Try for a base class
-			$subtypes = explode('-',$type);
+			$subtypes = explode('-', $type);
 			$basetype = $subtypes[0];
-			$class_name = __CLASS__.'\\'.implode('\\',array_map('ucfirst',$subtypes));
-			if(!class_exists($class_name) && self::$widget_registry[$basetype] && self::$widget_registry[$basetype] != $class_name)
+			$class_name = __CLASS__.'\\'.implode('\\', array_map('ucfirst', $subtypes));
+			if (!class_exists($class_name) && self::$widget_registry[$basetype] && self::$widget_registry[$basetype] != $class_name)
 			{
 				// Try for base type, it's probably better than the root
 				$class_name = self::$widget_registry[$basetype];
 			}
-			if(!class_exists($class_name))
+			if (!class_exists($class_name))
 			{
 				// Fall back to widget class, we can not ignore it, as the widget may contain other widgets
 				$class_name = __CLASS__;
 			}
 		}
 
-		if(!$xml)
+		if (!$xml)
 		{
 			if (empty($type)) $type = 'widget';
 			$xml = "<$type id='$id'/>";
@@ -419,7 +419,7 @@ 
 		if ($type == 'template' && $id && ($template = Widget\Template::instance($id)))
 		{
 			// references can set different attributes like: class, span, content (namespace)
-			return $template->set_attrs($xml, false);	// false = need to clone template, if attributs are set!
+			return $template->set_attrs($xml, false); // false = need to clone template, if attributs are set!
 		}
 		return new $class_name($xml);
 	}
@@ -431,9 +431,9 @@ 
 	 * @param string $type =null
 	 * @return Widget|NULL
 	 */
-	public function getElementById($id, $type=null)
+	public function getElementById($id, $type = null)
 	{
-		foreach($this->children as $child)
+		foreach ($this->children as $child)
 		{
 			if ($child->id === $id && (is_null($type) || $child->type === $type))
 			{
@@ -456,7 +456,7 @@ 
 	public function getElementsByType($type)
 	{
 		$elements = array();
-		foreach($this->children as $child)
+		foreach ($this->children as $child)
 		{
 			if ($child->type === $type)
 			{
@@ -476,14 +476,14 @@ 
 	 * @param array $params =array('') parameter(s) first parameter has to be the cname, second $expand!
 	 * @param boolean $respect_disabled =false false (default): ignore disabled, true: method is NOT run for disabled widgets AND their children
 	 */
-	public function run($method_name, $params=array(''), $respect_disabled=false)
+	public function run($method_name, $params = array(''), $respect_disabled = false)
 	{
 		// maintain $expand array name-expansion
 		$cname = $params[0];
-		$expand =& $params[1];
+		$expand = & $params[1];
 		if ($expand['cname'] && $expand['cname'] !== $cname)
 		{
-			$expand['cont'] =& self::get_array(self::$request->content, $cname);
+			$expand['cont'] = & self::get_array(self::$request->content, $cname);
 			$expand['cname'] = $cname;
 		}
 		if ($respect_disabled && ($disabled = $this->attrs['disabled'] && self::check_disabled($this->attrs['disabled'], $expand)))
@@ -496,22 +496,22 @@ 
 			// Some parameter checking to avoid fatal errors
 			$call = true;
 			$method = new ReflectionMethod($this, $method_name);
-			foreach($method->getParameters() as $index => $param)
+			foreach ($method->getParameters() as $index => $param)
 			{
-				if(!$param->isOptional() && !array_key_exists($index,$params))
+				if (!$param->isOptional() && !array_key_exists($index, $params))
 				{
 					error_log("Missing required parameter {$param->getPosition()}: {$param->getName()}");
 					$call = false;
 				}
-				if($param->isArray() && !is_array($params[$index]))
+				if ($param->isArray() && !is_array($params[$index]))
 				{
 					error_log("$method_name expects an array for {$param->getPosition()}: {$param->getName()}");
 					$params[$index] = (array)$params[$index];
 				}
 			}
-			if($call) call_user_func_array(array($this, $method_name), $params);
+			if ($call) call_user_func_array(array($this, $method_name), $params);
 		}
-		foreach($this->children as $child)
+		foreach ($this->children as $child)
 		{
 			// If type has something that can be expanded, we need to expand it so the correct method is run
 			$this->expand_widget($child, $expand);
@@ -526,15 +526,15 @@ 
 	 * @param Widget& $child Widget to check & expand if needed
 	 * @param array& $expand Expansion array
 	 */
-	protected function expand_widget(Widget &$child, array &$expand)
+	protected function expand_widget(Widget&$child, array &$expand)
 	{
-		if(strpos($child->attrs['type'], '@') !== false || strpos($child->attrs['type'], '$') !== false)
+		if (strpos($child->attrs['type'], '@') !== false || strpos($child->attrs['type'], '$') !== false)
 		{
-			$type = self::expand_name($child->attrs['type'],$expand['c'], $expand['row'], $expand['c_'], $expand['row_'], $expand['cont']);
-			$id = self::expand_name($child->id,$expand['c'], $expand['row'], $expand['c_'], $expand['row_'], $expand['cont']);
+			$type = self::expand_name($child->attrs['type'], $expand['c'], $expand['row'], $expand['c_'], $expand['row_'], $expand['cont']);
+			$id = self::expand_name($child->id, $expand['c'], $expand['row'], $expand['c_'], $expand['row_'], $expand['cont']);
 			$attrs = $child->attrs;
 			unset($attrs['type']);
-			$expanded_child = self::factory($type, false,$id);
+			$expanded_child = self::factory($type, false, $id);
 			$expanded_child->id = $id;
 			$expanded_child->type = $type;
 			$expanded_child->attrs = $attrs + array('type' => $type);
@@ -557,14 +557,14 @@ 
 	{
 		if (($not = $disabled[0] == '!'))
 		{
-			$disabled = substr($disabled,1);
+			$disabled = substr($disabled, 1);
 		}
-		list($value,$check) = $vals = explode('=',$disabled);
+		list($value, $check) = $vals = explode('=', $disabled);
 
 		// use expand_name to be able to use @ or $
 		$val = self::expand_name($value, $expand['c'], $expand['row'], $expand['c_'], $expand['row_'], $expand['cont']);
 		$check_val = self::expand_name($check, $expand['c'], $expand['row'], $expand['c_'], $expand['row_'], $expand['cont']);
-		$result = count($vals) == 1 ? $val != '' : ($check_val[0] == '/' ? preg_match($check_val,$val) : $val == $check_val);
+		$result = count($vals) == 1 ? $val != '' : ($check_val[0] == '/' ? preg_match($check_val, $val) : $val == $check_val);
 		if ($not) $result = !$result;
 
 		//error_log(__METHOD__."('".($not?'!':'')."$disabled' = '$val' ".(count($vals) == 1 ? '' : ($not?'!':'=')."= '$check_val'")." = ".($result?'True':'False'));
@@ -606,33 +606,33 @@ 
 	 * 	"del[$cont[id]]" expands to "del[123]" if $cont = array('id' => 123)
 	 * @return string the expanded name
 	 */
-	protected static function expand_name($name,$c,$row,$c_='',$row_='',$cont='')
+	protected static function expand_name($name, $c, $row, $c_ = '', $row_ = '', $cont = '')
 	{
 		$is_index_in_content = $name[0] == '@';
-		if (($pos_var=strpos($name,'$')) !== false)
+		if (($pos_var = strpos($name, '$')) !== false)
 		{
 			if (!$cont)
 			{
 				$cont = array();
 			}
 			if (!is_numeric($c)) $c = self::chrs2num($c);
-			$col = self::num2chrs($c-1);	// $c-1 to get: 0:'@', 1:'A', ...
-			$col_ = self::num2chrs($c_-1);
+			$col = self::num2chrs($c - 1); // $c-1 to get: 0:'@', 1:'A', ...
+			$col_ = self::num2chrs($c_ - 1);
 			$row_cont = $cont[$row];
 			$col_row_cont = $cont[$col.$row];
 
-			eval('$name = "'.str_replace('"','\\"',$name).'";');
-			unset($col_, $row_, $row_cont, $col_row_cont);	// quiten IDE warning about used vars, they might be used in above eval!
+			eval('$name = "'.str_replace('"', '\\"', $name).'";');
+			unset($col_, $row_, $row_cont, $col_row_cont); // quiten IDE warning about used vars, they might be used in above eval!
 		}
 		if ($is_index_in_content)
 		{
 			if ($name[1] == '@' && is_array(self::$request->content))
 			{
-				$name = self::get_array(self::$request->content,substr($name,2));
+				$name = self::get_array(self::$request->content, substr($name, 2));
 			}
-			elseif(is_array($cont))
+			elseif (is_array($cont))
 			{
-				$name = self::get_array($cont,substr($name,1));
+				$name = self::get_array($cont, substr($name, 1));
 			}
 			else
 			{
@@ -654,11 +654,11 @@ 
 		$min = ord('A');
 		$max = ord('Z') - $min + 1;
 
-		$num = 1+ord($chrs{0})-$min;
+		$num = 1 + ord($chrs{0}) - $min;
 		if (strlen($chrs) > 1)
 		{
 			$num *= 1 + $max - $min;
-			$num += 1+ord($chrs{1})-$min;
+			$num += 1 + ord($chrs{1}) - $min;
 		}
 		return $num;
 	}
@@ -689,7 +689,7 @@ 
 	 */
 	public function __toString()
 	{
-		return '['.get_class($this).'] ' .
+		return '['.get_class($this).'] '.
 			$this->type.($this->attrs['type'] && $this->attrs['type'] != $this->type ? '('.$this->attrs['type'].')' : '').'#'.$this->id;
 	}
 
@@ -698,7 +698,7 @@ 
 	 */
 	public function __clone()
 	{
-		foreach($this->children as $child_num => $child) {
+		foreach ($this->children as $child_num => $child) {
 			$this->children[$child_num] = clone $child;
 		}
 	}
@@ -709,16 +709,16 @@ 
 	 * @param string $indent =''
 	 * @return string
 	 */
-	public function toXml($indent='')
+	public function toXml($indent = '')
 	{
 		echo "$indent<$this->type";
 		if ($this->id) echo ' id="'.htmlspecialchars($this->id).'"';
-		foreach($this->attrs as $name => $value)
+		foreach ($this->attrs as $name => $value)
 		{
 			if ($name == 'options' && $this->legacy_options && (!is_array($this->legacy_options) ||
 				isset($this->legacy_options[$this->attrs['type'] ? $this->attrs['type'] : $this->type])))
 			{
-				continue;	// do NOT output already converted legacy options
+				continue; // do NOT output already converted legacy options
 			}
 			echo ' '.$name.'="'.htmlspecialchars($value).'"';
 		}
@@ -727,7 +727,7 @@ 
 		if ($this->children)
 		{
 			echo ">\n";
-			foreach($this->children as $child)
+			foreach ($this->children as $child)
 			{
 				$child->toXml($indent."\t");
 			}
@@ -750,7 +750,7 @@ 
 	 * @param array $expand =null values for keys 'c', 'row', 'c_', 'row_', 'cont'
 	 * @return string complete form-name
 	 */
-	static function form_name($cname,$name,array $expand=null)
+	static function form_name($cname, $name, array $expand = null)
 	{
 		if ($expand && !empty($name))
 		{
@@ -758,18 +758,18 @@ 
 		}
 		if (count($name_parts = explode('[', $name, 2)) > 1)
 		{
-			$name_parts = array_merge(array($name_parts[0]), explode('][', substr($name_parts[1],0,-1)));
+			$name_parts = array_merge(array($name_parts[0]), explode('][', substr($name_parts[1], 0, -1)));
 		}
 		if (!empty($cname))
 		{
-			array_unshift($name_parts,$cname);
+			array_unshift($name_parts, $cname);
 		}
 		$form_name = array_shift($name_parts);
 		if (count($name_parts))
 		{
 			// RB: not sure why this business with entity encoding for square brakets, it messes up validation
 			//$form_name .= '&#x5B;'.implode('&#x5D;&#x5B;',$name_parts).'&#x5D;';
-			$form_name .= '['.implode('][',$name_parts).']';
+			$form_name .= '['.implode('][', $name_parts).']';
 		}
 		return $form_name;
 	}
@@ -786,36 +786,36 @@ 
 	 * @param bool $skip_empty returns false if $idx is not present in $arr
 	 * @return mixed reference to $arr[$idx] or null if $idx is not set and not $reference_into
 	 */
-	static function &get_array(&$arr,$_idx,$reference_into=False,$skip_empty=False)
+	static function &get_array(&$arr, $_idx, $reference_into = False, $skip_empty = False)
 	{
 		if (!is_array($arr))
 		{
 			throw new Api\Exception\AssertionFailed(__METHOD__."(\$arr,'$_idx',$reference_into,$skip_empty) \$arr is no array!");
 		}
-		if (is_object($_idx)) return false;	// given an error in php5.2
+		if (is_object($_idx)) return false; // given an error in php5.2
 
 		// Make sure none of these are left
-		$idx = str_replace(array('&#x5B;','&#x5D;'), array('[',']'), $_idx);
+		$idx = str_replace(array('&#x5B;', '&#x5D;'), array('[', ']'), $_idx);
 
 		// Handle things expecting arrays - ends in []
-		if(substr($idx,-2) == "[]")
+		if (substr($idx, -2) == "[]")
 		{
-			$idx = substr($idx,0,-2);
+			$idx = substr($idx, 0, -2);
 		}
 
 		if (count($idxs = explode('[', $idx, 2)) > 1)
 		{
-			$idxs = array_merge(array($idxs[0]), explode('][', substr($idxs[1],0,-1)));
+			$idxs = array_merge(array($idxs[0]), explode('][', substr($idxs[1], 0, -1)));
 		}
 		$pos = &$arr;
-		foreach($idxs as $idx)
+		foreach ($idxs as $idx)
 		{
 			if (!is_array($pos) && (!$reference_into || $reference_into && isset($pos)))
 			{
 				//if ($reference_into) error_log(__METHOD__."(".(strlen($s=array2string($arr))>512?substr($s,0,512).'...':$s).", '$idx', ".array2string($reference_into).", ".array2string($skip_empty).") ".function_backtrace());
 				return null;
 			}
-			if($skip_empty && (!is_array($pos) || !isset($pos[$idx]))) return null;
+			if ($skip_empty && (!is_array($pos) || !isset($pos[$idx]))) return null;
 			$pos = &$pos[$idx];
 		}
 		return $pos;
@@ -833,7 +833,7 @@ 
 	 */
 	static function set_array(&$_arr, $_idx, $_value)
 	{
-		$ref =& self::get_array($_arr, $_idx, true);
+		$ref = & self::get_array($_arr, $_idx, true);
 		if (true) $ref = $_value;
 	}
 
@@ -847,7 +847,7 @@ 
 	 * @param string $form_name =null form_name, to not calculate him again
 	 * @return boolean
 	 */
-	public function is_readonly($cname='', $form_name=null)
+	public function is_readonly($cname = '', $form_name = null)
 	{
 		if (!isset($form_name))
 		{
@@ -857,11 +857,11 @@ 
 			$form_name = self::form_name($cname, $this->id, $expand);
 		}
 		$readonly = $this->attrs['readonly'] || self::$request->readonlys[$form_name] ||
-			self::get_array(self::$request->readonlys,$form_name) === true ||
+			self::get_array(self::$request->readonlys, $form_name) === true ||
 			isset(self::$request->readonlys['__ALL__']) && (
 				// Exceptions to all
 				self::$request->readonlys[$form_name] !== false &&
-				self::get_array(self::$request->readonlys,$form_name) !== false
+				self::get_array(self::$request->readonlys, $form_name) !== false
 			);
 
 		//error_log(__METHOD__."('$cname') this->id='$this->id' --> form_name='$form_name': attrs[readonly]=".array2string($this->attrs['readonly']).", readonlys['$form_name']=".array2string(self::$request->readonlys[$form_name]).", readonlys[$form_name]=".array2string(self::get_array(self::$request->readonlys,$form_name)).", readonlys['__ALL__']=".array2string(self::$request->readonlys['__ALL__'])." returning ".array2string($readonly));
@@ -881,12 +881,12 @@ 
 	 * @param string|boolean $error error-message already translated or false to reset all existing error for given name
 	 * @param string $cname =null set it to '', if the name is already a form-name, defaults to self::$name_vars
 	 */
-	public static function set_validation_error($name,$error,$cname=null)
+	public static function set_validation_error($name, $error, $cname = null)
 	{
 		// not yet used: if (is_null($cname)) $cname = self::$name_vars;
 		error_log(__METHOD__."('$name','$error','$cname') ".function_backtrace());
 
-		if ($cname) $name = self::form_name($cname,$name);
+		if ($cname) $name = self::form_name($cname, $name);
 
 		if ($error === false)
 		{
@@ -909,18 +909,18 @@ 
 	* @param string $cname =null name-prefix, which need to be ignored, default self::$name_vars
 	* @return boolean true if there are not ignored validation errors, false otherwise
 	*/
-	public static function validation_errors($ignore_validation='',$cname='')
+	public static function validation_errors($ignore_validation = '', $cname = '')
 	{
 		// not yet used: if (is_null($cname)) $cname = self::$name_vars;
 		//echo "<p>uietemplate::validation_errors('$ignore_validation','$cname') validation_error="; _debug_array(self::$validation_errors);
 		if (!$ignore_validation) return count(self::$validation_errors) > 0;
 
-		foreach(array_values(self::$validation_errors) as $name)
+		foreach (array_values(self::$validation_errors) as $name)
 		{
-			if ($cname) $name = preg_replace('/^'.$cname.'\[([^\]]+)\](.*)$/','\\1\\2',$name);
+			if ($cname) $name = preg_replace('/^'.$cname.'\[([^\]]+)\](.*)$/', '\\1\\2', $name);
 
 			// treat $ignoare_validation only as regular expression, if it starts with a slash
-			if ($ignore_validation[0] == '/' && !preg_match($ignore_validation,$name) ||
+			if ($ignore_validation[0] == '/' && !preg_match($ignore_validation, $name) ||
 				$ignore_validation[0] != '/' && $ignore_validation != $name)
 			{
 				//echo "<p>uietemplate::validation_errors('$ignore_validation','$cname') name='$name' ($error) not ignored!!!</p>\n";
@@ -957,14 +957,14 @@ 
 	 * @param mixed $val if not NULL sets attribute else returns it
 	 * @return reference to attribute
 	 */
-	public static function &setElementAttribute($name,$attr,$val)
+	public static function &setElementAttribute($name, $attr, $val)
 	{
 		//error_log(__METHOD__."('$name', '$attr', ...) request=".get_class(self::$request).", response=".get_class(self::$response).function_backtrace());
-		$ref =& self::$request->modifications[$name][$attr];
-		if(self::$request && self::$response && (!isset($this) || $val != $this->attrs[$attr]))
+		$ref = & self::$request->modifications[$name][$attr];
+		if (self::$request && self::$response && (!isset($this) || $val != $this->attrs[$attr]))
 		{
 			// In an AJAX response - automatically add
-			self::$response->generic('assign',array(
+			self::$response->generic('assign', array(
 				'etemplate_exec_id' => self::$request->id(),
 				'id' => $name,
 				'key' => $attr,
@@ -988,7 +988,7 @@ 
 	 * @param boolean $disabled =true disable or enable a cell, default true=disable
 	 * @return reference to attribute
 	 */
-	public function disableElement($name,$disabled=True)
+	public function disableElement($name, $disabled = True)
 	{
 		return self::setElementAttribute($name, 'disabled', $disabled);
 	}

Braces +68 added lines, -18 removed lines

@@ -100,7 +100,9 @@ 
 
 		// Update content?
 		if(self::$cont == null)
-			self::$cont = is_array(self::$request->content) ? self::$request->content : array();
+		{
+					self::$cont = is_array(self::$request->content) ? self::$request->content : array();
+		}
 		if($this->id && is_array(self::$cont[$this->id]))
 		{
 			$old_cont = self::$cont;
@@ -119,7 +121,8 @@ 
 		}
 
 		// Reset content as we leave
-		if($old_cont) {
+		if($old_cont)
+		{
 			self::$cont = $old_cont;
 		}
 	}
@@ -200,7 +203,10 @@ 
 					$legacy_options = explode(',', $legacy_options);
 					foreach(self::csv_split($value, count($legacy_options)) as $n => $val)
 					{
-						if ($legacy_options[$n] && (string)$val !== '') $template->attrs[$legacy_options[$n]] = $val;
+						if ($legacy_options[$n] && (string)$val !== '')
+						{
+							$template->attrs[$legacy_options[$n]] = $val;
+						}
 					}
 				}
 			}
@@ -305,7 +311,9 @@ 
 
 		$widget_registry = Api\Cache::getInstance('etemplate', 'widget_registry');
 
-		if (!$widget_registry)	// not in instance cache --> rescan from filesystem
+		if (!$widget_registry)
+		{
+			// not in instance cache --> rescan from filesystem
 		{
 			foreach(new \RecursiveIteratorIterator(new \RecursiveDirectoryIterator(__DIR__.'/Widget')) as $path)
 			{
@@ -314,6 +322,7 @@ 
 					try
 					{
 						include_once($path);
+		}
 					}
 					catch(Exception $e)
 					{
@@ -410,7 +419,10 @@ 
 
 		if(!$xml)
 		{
-			if (empty($type)) $type = 'widget';
+			if (empty($type))
+			{
+				$type = 'widget';
+			}
 			$xml = "<$type id='$id'/>";
 		}
 		//error_log(__METHOD__."('$type', ..., '$id') using $class_name");
@@ -509,7 +521,10 @@ 
 					$params[$index] = (array)$params[$index];
 				}
 			}
-			if($call) call_user_func_array(array($this, $method_name), $params);
+			if($call)
+			{
+				call_user_func_array(array($this, $method_name), $params);
+			}
 		}
 		foreach($this->children as $child)
 		{
@@ -565,7 +580,10 @@ 
 		$val = self::expand_name($value, $expand['c'], $expand['row'], $expand['c_'], $expand['row_'], $expand['cont']);
 		$check_val = self::expand_name($check, $expand['c'], $expand['row'], $expand['c_'], $expand['row_'], $expand['cont']);
 		$result = count($vals) == 1 ? $val != '' : ($check_val[0] == '/' ? preg_match($check_val,$val) : $val == $check_val);
-		if ($not) $result = !$result;
+		if ($not)
+		{
+			$result = !$result;
+		}
 
 		//error_log(__METHOD__."('".($not?'!':'')."$disabled' = '$val' ".(count($vals) == 1 ? '' : ($not?'!':'=')."= '$check_val'")." = ".($result?'True':'False'));
 		return $result;
@@ -615,7 +633,10 @@ 
 			{
 				$cont = array();
 			}
-			if (!is_numeric($c)) $c = self::chrs2num($c);
+			if (!is_numeric($c))
+			{
+				$c = self::chrs2num($c);
+			}
 			$col = self::num2chrs($c-1);	// $c-1 to get: 0:'@', 1:'A', ...
 			$col_ = self::num2chrs($c_-1);
 			$row_cont = $cont[$row];
@@ -698,7 +719,8 @@ 
 	 */
 	public function __clone()
 	{
-		foreach($this->children as $child_num => $child) {
+		foreach($this->children as $child_num => $child)
+		{
 			$this->children[$child_num] = clone $child;
 		}
 	}
@@ -712,7 +734,10 @@ 
 	public function toXml($indent='')
 	{
 		echo "$indent<$this->type";
-		if ($this->id) echo ' id="'.htmlspecialchars($this->id).'"';
+		if ($this->id)
+		{
+			echo ' id="'.htmlspecialchars($this->id).'"';
+		}
 		foreach($this->attrs as $name => $value)
 		{
 			if ($name == 'options' && $this->legacy_options && (!is_array($this->legacy_options) ||
@@ -792,7 +817,11 @@ 
 		{
 			throw new Api\Exception\AssertionFailed(__METHOD__."(\$arr,'$_idx',$reference_into,$skip_empty) \$arr is no array!");
 		}
-		if (is_object($_idx)) return false;	// given an error in php5.2
+		if (is_object($_idx))
+		{
+			return false;
+		}
+		// given an error in php5.2
 
 		// Make sure none of these are left
 		$idx = str_replace(array('&#x5B;','&#x5D;'), array('[',']'), $_idx);
@@ -815,7 +844,10 @@ 
 				//if ($reference_into) error_log(__METHOD__."(".(strlen($s=array2string($arr))>512?substr($s,0,512).'...':$s).", '$idx', ".array2string($reference_into).", ".array2string($skip_empty).") ".function_backtrace());
 				return null;
 			}
-			if($skip_empty && (!is_array($pos) || !isset($pos[$idx]))) return null;
+			if($skip_empty && (!is_array($pos) || !isset($pos[$idx])))
+			{
+				return null;
+			}
 			$pos = &$pos[$idx];
 		}
 		return $pos;
@@ -834,7 +866,10 @@ 
 	static function set_array(&$_arr, $_idx, $_value)
 	{
 		$ref =& self::get_array($_arr, $_idx, true);
-		if (true) $ref = $_value;
+		if (true)
+		{
+			$ref = $_value;
+		}
 	}
 
 	/**
@@ -886,7 +921,10 @@ 
 		// not yet used: if (is_null($cname)) $cname = self::$name_vars;
 		error_log(__METHOD__."('$name','$error','$cname') ".function_backtrace());
 
-		if ($cname) $name = self::form_name($cname,$name);
+		if ($cname)
+		{
+			$name = self::form_name($cname,$name);
+		}
 
 		if ($error === false)
 		{
@@ -913,11 +951,17 @@ 
 	{
 		// not yet used: if (is_null($cname)) $cname = self::$name_vars;
 		//echo "<p>uietemplate::validation_errors('$ignore_validation','$cname') validation_error="; _debug_array(self::$validation_errors);
-		if (!$ignore_validation) return count(self::$validation_errors) > 0;
+		if (!$ignore_validation)
+		{
+			return count(self::$validation_errors) > 0;
+		}
 
 		foreach(array_values(self::$validation_errors) as $name)
 		{
-			if ($cname) $name = preg_replace('/^'.$cname.'\[([^\]]+)\](.*)$/','\\1\\2',$name);
+			if ($cname)
+			{
+				$name = preg_replace('/^'.$cname.'\[([^\]]+)\](.*)$/','\\1\\2',$name);
+			}
 
 			// treat $ignoare_validation only as regular expression, if it starts with a slash
 			if ($ignore_validation[0] == '/' && !preg_match($ignore_validation,$name) ||
@@ -974,8 +1018,14 @@ 
 			self::$request->unset_to_process('');
 			//error_log(__METHOD__."('$name', '$attr', ...) ".function_backtrace());
 		}
-		if (isset($this)) $this->attrs[$attr] = $val;
-		if (!is_null($val)) $ref = $val;
+		if (isset($this))
+		{
+			$this->attrs[$attr] = $val;
+		}
+		if (!is_null($val))
+		{
+			$ref = $val;
+		}
 
 		//error_log(__METHOD__."('$name', '$attr', ".array2string($val).')');
 		return $ref;

api/src/Etemplate/Widget/Checkbox.php

Doc Comments +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
 	 * @param array $expand values for keys 'c', 'row', 'c_', 'row_', 'cont'
 	 * @param array $content
 	 * @param array &$validated=array() validated content
-	 * @return boolean true if no validation error, false otherwise
+	 * @return boolean|null true if no validation error, false otherwise
 	 */
 	public function validate($cname, array $expand, array $content, &$validated=array())
 	{

Spacing +11 added lines, -11 removed lines

@@ -44,7 +44,7 @@ 
 	 * @param array &$validated=array() validated content
 	 * @return boolean true if no validation error, false otherwise
 	 */
-	public function validate($cname, array $expand, array $content, &$validated=array())
+	public function validate($cname, array $expand, array $content, &$validated = array())
 	{
 		$form_name = self::form_name($cname, $this->id, $expand);
 
@@ -56,28 +56,28 @@ 
 		if (!$this->is_readonly($cname, $form_name))
 		{
 			$value = self::get_array($content, $form_name);
-			if (!isset($value))	return;	// value not transmitted --> nothing to validate
-			$valid =& self::get_array($validated, $form_name, true);
+			if (!isset($value))	return; // value not transmitted --> nothing to validate
+			$valid = & self::get_array($validated, $form_name, true);
 
 			if (!$value && $this->attrs['needed'])
 			{
-				self::set_validation_error($form_name,lang('Field must not be empty !!!'),'');
+				self::set_validation_error($form_name, lang('Field must not be empty !!!'), '');
 			}
 			$type = $this->type ? $this->type : $this->attrs['type'];
 			$value_attr = $type == 'radio' ? 'set_value' : 'selected_value';
 			// defaults for set and unset values
 			$selected_value = true;
 			$unselected_value = false;
-			if (array_key_exists($value_attr, $this->attrs) || array_key_exists('unselected_value',$this->attrs))
+			if (array_key_exists($value_attr, $this->attrs) || array_key_exists('unselected_value', $this->attrs))
 			{
-				if(array_key_exists($value_attr, $this->attrs))
+				if (array_key_exists($value_attr, $this->attrs))
 				{
 					// Expand any content stuff
-					$selected_value = self::expand_name($this->attrs[$value_attr], $expand['c'], $expand['row'], $expand['c_'], $expand['row_'],$expand['cont']);
+					$selected_value = self::expand_name($this->attrs[$value_attr], $expand['c'], $expand['row'], $expand['c_'], $expand['row_'], $expand['cont']);
 				}
-				if(array_key_exists('unselected_value',$this->attrs))
+				if (array_key_exists('unselected_value', $this->attrs))
 				{
-					$unselected_value = self::expand_name($this->attrs['unselected_value'], $expand['c'], $expand['row'], $expand['c_'], $expand['row_'],$expand['cont']);
+					$unselected_value = self::expand_name($this->attrs['unselected_value'], $expand['c'], $expand['row'], $expand['c_'], $expand['row_'], $expand['cont']);
 				}
 			}
 			if ($type == 'radio')
@@ -87,14 +87,14 @@ 
 				{
 					$valid = $value;
 				}
-				elseif ( !$options && $selected_value && $value == $selected_value)
+				elseif (!$options && $selected_value && $value == $selected_value)
 				{
 					// Options set in template, not in sel_options
 					$valid = $selected_value;
 				}
 				elseif (!isset($valid))
 				{
-					$valid = '';	// do not overwrite value of an other radio-button of the same group (identical name)!
+					$valid = ''; // do not overwrite value of an other radio-button of the same group (identical name)!
 				}
 			}
 			elseif (in_array((string)$selected_value, (array)$value))

Braces +13 added lines, -3 removed lines

@@ -56,7 +56,11 @@ 
 		if (!$this->is_readonly($cname, $form_name))
 		{
 			$value = self::get_array($content, $form_name);
-			if (!isset($value))	return;	// value not transmitted --> nothing to validate
+			if (!isset($value))
+			{
+				return;
+			}
+			// value not transmitted --> nothing to validate
 			$valid =& self::get_array($validated, $form_name, true);
 
 			if (!$value && $this->attrs['needed'])
@@ -101,7 +105,10 @@ 
 			{
 				if ($multiple)
 				{
-					if (!isset($valid)) $valid = array();
+					if (!isset($valid))
+					{
+						$valid = array();
+					}
 					$valid[] = $selected_value;
 				}
 				else
@@ -113,7 +120,10 @@ 
 			{
 				if ($multiple)
 				{
-					if (!isset($valid)) $valid = array();
+					if (!isset($valid))
+					{
+						$valid = array();
+					}
 				}
 				elseif ($value === 'true')
 				{

api/src/Etemplate/Widget/Date.php

Doc Comments +1 added lines, -2 removed lines

@@ -101,7 +101,6 @@ 
 	/**
 	 * Put date in the proper format for sending to client
 	 * @param string|int $value
-	 * @param string $format
 	 */
 	public function format_date($value)
 	{
@@ -139,7 +138,7 @@ 
 	 * @param array $expand values for keys 'c', 'row', 'c_', 'row_', 'cont'
 	 * @param array $content
 	 * @param array &$validated=array() validated content
-	 * @return boolean true if no validation error, false otherwise
+	 * @return boolean|null true if no validation error, false otherwise
 	 */
 	public function validate($cname, array $expand, array $content, &$validated=array())
 	{

Spacing +25 added lines, -25 removed lines

@@ -60,17 +60,17 @@ 
 	 * @param string $cname
 	 * @param array $expand values for keys 'c', 'row', 'c_', 'row_', 'cont'
 	 */
-	public function beforeSendToClient($cname, array $expand=null)
+	public function beforeSendToClient($cname, array $expand = null)
 	{
-		if($this->type == 'date-houronly')
+		if ($this->type == 'date-houronly')
 		{
 			return parent::beforeSendToClient($cname, $expand);
 		}
 
 		$form_name = self::form_name($cname, $this->id, $expand);
-		$value =& self::get_array(self::$request->content, $form_name, false, true);
+		$value = & self::get_array(self::$request->content, $form_name, false, true);
 
-		if($this->type != 'date-duration' && $value)
+		if ($this->type != 'date-duration' && $value)
 		{
 			$value = $this->format_date($value);
 		}
@@ -90,10 +90,10 @@ 
 	 */
 	public function set_row_value($cname, Array $expand, Array &$data)
 	{
-		if($this->type == 'date-duration') return;
+		if ($this->type == 'date-duration') return;
 
 		$form_name = self::form_name($cname, $this->id, $expand);
-		$value =& $this->get_array($data, $form_name, true);
+		$value = & $this->get_array($data, $form_name, true);
 
 		if (true) $value = $this->format_date($value);
 	}
@@ -105,7 +105,7 @@ 
 	 */
 	public function format_date($value)
 	{
-		if (!$value) return $value;	// otherwise we will get current date or 1970-01-01 instead of an empty value
+		if (!$value) return $value; // otherwise we will get current date or 1970-01-01 instead of an empty value
 
 		if ($this->attrs['dataformat'] && !is_numeric($value))
 		{
@@ -115,11 +115,11 @@ 
 		{
 			$date = new Api\DateTime($value);
 		}
-		if($this->type == 'date-timeonly')
+		if ($this->type == 'date-timeonly')
 		{
 			$date->setDate(1970, 1, 1);
 		}
-		if($date)
+		if ($date)
 		{
 			// postfix date-string with "Z" so javascript doesn't add/subtract anything
 			$value = $date->format('Y-m-d\TH:i:s\Z');
@@ -141,18 +141,18 @@ 
 	 * @param array &$validated=array() validated content
 	 * @return boolean true if no validation error, false otherwise
 	 */
-	public function validate($cname, array $expand, array $content, &$validated=array())
+	public function validate($cname, array $expand, array $content, &$validated = array())
 	{
 		$form_name = self::form_name($cname, $this->id, $expand);
 
 		if (!$this->is_readonly($cname, $form_name) && $this->type != 'date-since')	// date-since is always readonly
 		{
 			$value = self::get_array($content, $form_name);
-			$valid =& self::get_array($validated, $form_name, true);
+			$valid = & self::get_array($validated, $form_name, true);
 
 			if ((string)$value === '' && $this->attrs['needed'])
 			{
-				self::set_validation_error($form_name,lang('Field must not be empty !!!'));
+				self::set_validation_error($form_name, lang('Field must not be empty !!!'));
 			}
 			elseif (is_null($value))
 			{
@@ -162,49 +162,49 @@ 
 			{
 				$valid = (string)$value === '' ? '' : (int)$value;
 			}
-			if($value)
+			if ($value)
 			{
 				$date = new Api\DateTime($value);
 			}
 			if (!empty($this->attrs['min']))
 			{
-				if(is_numeric($this->attrs['min']))
+				if (is_numeric($this->attrs['min']))
 				{
-					$min = new Api\DateTime(strtotime( $this->attrs['min'] . 'days'));
+					$min = new Api\DateTime(strtotime($this->attrs['min'].'days'));
 				}
 				else
 				{
 					$min = new Api\DateTime(strtotime($this->attrs['min']));
 				}
-				if($value < $min)
+				if ($value < $min)
 				{
-					self::set_validation_error($form_name,lang(
+					self::set_validation_error($form_name, lang(
 						"Value has to be at least '%1' !!!",
 						$min->format($this->type != 'date')
-					),'');
+					), '');
 					$value = $min;
 				}
 			}
 			if (!empty($this->attrs['max']))
 			{
-				if(is_numeric($this->attrs['max']))
+				if (is_numeric($this->attrs['max']))
 				{
-					$max = new Api\DateTime(strtotime( $this->attrs['max'] . 'days'));
+					$max = new Api\DateTime(strtotime($this->attrs['max'].'days'));
 				}
 				else
 				{
 					$max = new Api\DateTime(strtotime($this->attrs['max']));
 				}
-				if($value < $max)
+				if ($value < $max)
 				{
-					self::set_validation_error($form_name,lang(
+					self::set_validation_error($form_name, lang(
 						"Value has to be at maximum '%1' !!!",
 						$max->format($this->type != 'date')
-					),'');
+					), '');
 					$value = $max;
 				}
 			}
-			if(!$value)
+			if (!$value)
 			{
 				// Not null, blank
 				$value = '';
@@ -221,7 +221,7 @@ 
 			else
 			{
 				// this is not really a user error, but one of the clientside engine
-				self::set_validation_error($form_name,lang("'%1' is not a valid date !!!", $value).' '.$this->dataformat);
+				self::set_validation_error($form_name, lang("'%1' is not a valid date !!!", $value).' '.$this->dataformat);
 			}
 			//error_log("$this : ($valid)" . Api\DateTime::to($valid));
 		}

Braces +21 added lines, -5 removed lines

@@ -90,12 +90,18 @@ 
 	 */
 	public function set_row_value($cname, Array $expand, Array &$data)
 	{
-		if($this->type == 'date-duration') return;
+		if($this->type == 'date-duration')
+		{
+			return;
+		}
 
 		$form_name = self::form_name($cname, $this->id, $expand);
 		$value =& $this->get_array($data, $form_name, true);
 
-		if (true) $value = $this->format_date($value);
+		if (true)
+		{
+			$value = $this->format_date($value);
+		}
 	}
 
 	/**
@@ -105,7 +111,11 @@ 
 	 */
 	public function format_date($value)
 	{
-		if (!$value) return $value;	// otherwise we will get current date or 1970-01-01 instead of an empty value
+		if (!$value)
+		{
+			return $value;
+		}
+		// otherwise we will get current date or 1970-01-01 instead of an empty value
 
 		if ($this->attrs['dataformat'] && !is_numeric($value))
 		{
@@ -145,9 +155,12 @@ 
 	{
 		$form_name = self::form_name($cname, $this->id, $expand);
 
-		if (!$this->is_readonly($cname, $form_name) && $this->type != 'date-since')	// date-since is always readonly
+		if (!$this->is_readonly($cname, $form_name) && $this->type != 'date-since')
+		{
+			// date-since is always readonly
 		{
 			$value = self::get_array($content, $form_name);
+		}
 			$valid =& self::get_array($validated, $form_name, true);
 
 			if ((string)$value === '' && $this->attrs['needed'])
@@ -209,10 +222,13 @@ 
 				// Not null, blank
 				$value = '';
 			}
-			elseif (empty($this->attrs['dataformat']))	// integer timestamp
+			elseif (empty($this->attrs['dataformat']))
+			{
+				// integer timestamp
 			{
 				$valid = $date->format('ts');
 			}
+			}
 			// string with formatting letters like for php's date() method
 			elseif ($date && ($valid = $date->format($this->attrs['dataformat'])))
 			{

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -88,7 +88,7 @@
 	 * @param array $data Row data
 	 * @return type
 	 */
-	public function set_row_value($cname, Array $expand, Array &$data)
+	public function set_row_value($cname, array $expand, array &$data)
 	{
 		if($this->type == 'date-duration') return;
 

api/src/Etemplate/Widget/Tabbox.php

Doc Comments -1 removed lines

@@ -102,7 +102,6 @@
 	 * @param array $expand values for keys 'c', 'row', 'c_', 'row_', 'cont'
 	 * @param array $content
 	 * @param array &$validated=array() validated content
-	 * @param array $expand=array values for keys 'c', 'row', 'c_', 'row_', 'cont'
 	 */
 	public function validate($cname, array $expand, array $content, &$validated=array())
 	{

Braces +8 added lines, -2 removed lines

@@ -65,7 +65,10 @@ 
 			foreach($tabs as &$tab)
 			{
 				$template= clone Template::instance($tab['template']);
-				if($tab['id']) $template->attrs['content'] = $tab['id'];
+				if($tab['id'])
+				{
+					$template->attrs['content'] = $tab['id'];
+				}
 				$this->children[1]->children[] = $template;
 				$tab['url'] = Template::rel2url($template->rel_path);
 				//$this->tabs[] = $tab;
@@ -112,7 +115,10 @@ 
 		{
 			$value = self::get_array($content, $form_name);
 			$valid =& self::get_array($validated, $form_name, true);
-			if (true) $valid = $value;
+			if (true)
+			{
+				$valid = $value;
+			}
 		}
 	}
 }

Spacing +16 added lines, -16 removed lines

@@ -42,30 +42,30 @@ 
 	 * @param array $params =array('') parameter(s) first parameter has to be the cname, second $expand!
 	 * @param boolean $respect_disabled =false false (default): ignore disabled, true: method is NOT run for disabled widgets AND their children
 	 */
-	public function run($method_name, $params=array(''), $respect_disabled=false)
+	public function run($method_name, $params = array(''), $respect_disabled = false)
 	{
 		$form_name = self::form_name($params[0], $this->id, $params[1]);
 
 		// Make sure additional tabs are processed for any method
-		if (!($tabs =& self::getElementAttribute($form_name, 'tabs')))
+		if (!($tabs = & self::getElementAttribute($form_name, 'tabs')))
 		{
 			$tabs = $this->attrs['tabs'];
 		}
-		if($tabs && !$this->tabs_attr_evaluated)
+		if ($tabs && !$this->tabs_attr_evaluated)
 		{
-			$this->tabs_attr_evaluated = true;	// we must not evaluate tabs attribte more then once!
+			$this->tabs_attr_evaluated = true; // we must not evaluate tabs attribte more then once!
 
 			// add_tabs toggles replacing or adding to existing tabs
-			if(!$this->attrs['add_tabs'])
+			if (!$this->attrs['add_tabs'])
 			{
 				$this->children[1]->children = array();
 			}
 
 			//$this->tabs = array();
-			foreach($tabs as &$tab)
+			foreach ($tabs as &$tab)
 			{
-				$template= clone Template::instance($tab['template']);
-				if($tab['id']) $template->attrs['content'] = $tab['id'];
+				$template = clone Template::instance($tab['template']);
+				if ($tab['id']) $template->attrs['content'] = $tab['id'];
 				$this->children[1]->children[] = $template;
 				$tab['url'] = Template::rel2url($template->rel_path);
 				//$this->tabs[] = $tab;
@@ -83,22 +83,22 @@ 
 		// to avoid checking for server side validation
 		if ($form_name == 'tabs' && is_array($readonlys))
 		{
-			foreach($this->children[1]->children as $tab)
+			foreach ($this->children[1]->children as $tab)
 			{
-				if($readonlys[$tab->id])
+				if ($readonlys[$tab->id])
 				{
 					$tab->attrs['disabled'] = $readonlys[$tab->id];
 				}
 			}
 		}
 		
-		if($respect_disabled && $readonlys)
+		if ($respect_disabled && $readonlys)
 		{
-			foreach($this->children[1]->children as $tab)
+			foreach ($this->children[1]->children as $tab)
 			{
-				$parts = explode('.',$tab->template ? $tab->template : $tab->id);
+				$parts = explode('.', $tab->template ? $tab->template : $tab->id);
 				$ro_id = array_pop($parts);
-				if($readonlys[$ro_id])
+				if ($readonlys[$ro_id])
 				{
 					$tab->attrs['disabled'] = $readonlys[$ro_id];
 				}
@@ -118,14 +118,14 @@ 
 	 * @param array &$validated=array() validated content
 	 * @param array $expand=array values for keys 'c', 'row', 'c_', 'row_', 'cont'
 	 */
-	public function validate($cname, array $expand, array $content, &$validated=array())
+	public function validate($cname, array $expand, array $content, &$validated = array())
 	{
 		$form_name = self::form_name($cname, $this->id, $expand);
 
 		if (!empty($form_name))
 		{
 			$value = self::get_array($content, $form_name);
-			$valid =& self::get_array($validated, $form_name, true);
+			$valid = & self::get_array($validated, $form_name, true);
 			if (true) $valid = $value;
 		}
 	}