EGroupware / egroupware

importexport/inc/class.importexport_definitions_bo.inc.php

Doc Comments +1 added lines, -3 removed lines

@@ -139,7 +139,6 @@ 
 	/**
 	* Save a definition
 	*
-	* @param definition $definition
 	*/
 	public function save(Array $data) {
 		$definition = new importexport_definition();
@@ -209,7 +208,7 @@ 
 	/**
 	 * imports definitions from file
 	 *
-	 * @param string $import_file
+	 * @param string $_import_file
 	 * @throws Exeption
 	 * @return void
 	 */
@@ -268,7 +267,6 @@ 
 	 * It is not possible to handle some plugin options automatically, because they
 	 * just don't have equivalents.  (eg: What to do with unknown categories)
 	 *
-	 * @param importexport_definition $definition Import definition
 	 *
 	 * @return importexport_definition Export definition
 	 */

Indentation +4 added lines, -4 removed lines

@@ -137,10 +137,10 @@
 	}
 
 	/**
-	* Save a definition
-	*
-	* @param definition $definition
-	*/
+	 * Save a definition
+	 *
+	 * @param definition $definition
+	 */
 	public function save(Array $data) {
 		$definition = new importexport_definition();
 		$definition->set_record($data);

Spacing +49 added lines, -49 removed lines

@@ -31,13 +31,13 @@ 
 	 */
 	private $definitions;
 
-	public function __construct($_query=false, $ignore_acl = false)
+	public function __construct($_query = false, $ignore_acl = false)
 	{
-		$this->so_sql = new Api\Storage\Base(self::_appname, self::_defintion_table );
+		$this->so_sql = new Api\Storage\Base(self::_appname, self::_defintion_table);
 		if ($_query) {
 			$definitions = $this->so_sql->search($_query, false);
 			foreach ((array)$definitions as $definition) {
-				if(self::is_permitted($definition) || $ignore_acl) $this->definitions[] = $definition['definition_id'];
+				if (self::is_permitted($definition) || $ignore_acl) $this->definitions[] = $definition['definition_id'];
 			}
 		}
 	}
@@ -45,16 +45,16 @@ 
 	public function get_rows(&$query, &$rows, &$readonlys)
 	{
 		// Filter only definitions user is allowed to use
-		if(!$GLOBALS['egw_info']['user']['apps']['admin']) {
+		if (!$GLOBALS['egw_info']['user']['apps']['admin']) {
 			$this_membership = $GLOBALS['egw']->accounts->memberships($GLOBALS['egw_info']['user']['account_id'], true);
 			$this_membership[] = $GLOBALS['egw_info']['user']['account_id'];
 			$sql .= ' (';
 			$read = array();
-			foreach($this_membership as $id)
+			foreach ($this_membership as $id)
 			{
 				$read[] = 'allowed_users '.
 					$GLOBALS['egw']->db->capabilities['case_insensitive_like'].' '.
-					$GLOBALS['egw']->db->quote('%,'.str_replace('_','\\_',$id) .',%');
+					$GLOBALS['egw']->db->quote('%,'.str_replace('_', '\\_', $id).',%');
 			}
 			$sql .= implode(' OR ', $read);
 			$sql .= ') OR owner = '.$GLOBALS['egw_info']['user']['account_id'];
@@ -62,37 +62,37 @@ 
 		}
 		
 		// Handle allowed filter
-		if($query['col_filter']['allowed_users'])
+		if ($query['col_filter']['allowed_users'])
 		{
 			$allowed = array();
-			foreach((array)$query['col_filter']['allowed_users'] as $id)
+			foreach ((array)$query['col_filter']['allowed_users'] as $id)
 			{
 				$allowed[] = 'allowed_users '.
 					$GLOBALS['egw']->db->capabilities['case_insensitive_like'].' '.
-					$GLOBALS['egw']->db->quote('%,'.str_replace('_','\\_',$id) .',%');
+					$GLOBALS['egw']->db->quote('%,'.str_replace('_', '\\_', $id).',%');
 			}
-			if($allowed)
+			if ($allowed)
 			{
 				unset($query['col_filter']['allowed_users']);
-				$query['col_filter'][] = '('.implode(' OR ', $allowed) . ')';
+				$query['col_filter'][] = '('.implode(' OR ', $allowed).')';
 			}
 		}
 		
 		$total = $this->so_sql->get_rows($query, $rows, $readonlys);
 		$ro_count = 0;
-		foreach($rows as &$row) {
+		foreach ($rows as &$row) {
 			// Strip off leading + trailing ,
-			$row['allowed_users'] = substr($row['allowed_users'],1,-1);
+			$row['allowed_users'] = substr($row['allowed_users'], 1, -1);
 
 			$readonlys["edit[{$row['definition_id']}]"] = $readonlys["delete[{$row['definition_id']}]"] =
 				($row['owner'] != $GLOBALS['egw_info']['user']['account_id']) &&
 				!$GLOBALS['egw_info']['user']['apps']['admin'];
-			if($readonlys["edit[{$row['definition_id']}]"])
+			if ($readonlys["edit[{$row['definition_id']}]"])
 			{
 				$row['class'] .= 'rowNoEdit rowNoDelete';
 				$ro_count++;
 			}
-			$row['class'] .= ' ' . $row['type'];
+			$row['class'] .= ' '.$row['type'];
 		}
 		$readonlys['delete_selected'] = $ro_count == count($rows);
 		return $total;
@@ -107,11 +107,11 @@ 
 		return $this->definitions;
 	}
 	public function read($definition_id) {
-		if(is_numeric($definition_id)) {
+		if (is_numeric($definition_id)) {
 			$this->so_sql->read($definition_id);
 			$definition = new importexport_definition($this->so_sql->data['name']);
 		} else {
-			$definition = new importexport_definition( $definition_id['name'] );
+			$definition = new importexport_definition($definition_id['name']);
 		}
 		return $definition->get_record_array();
 	}
@@ -124,14 +124,14 @@ 
 		foreach ($keys as $index => $key) {
 			// Check for ownership
 			$definition = $this->read($key);
-			if($definition['owner'] && $definition['owner'] == $GLOBALS['egw_info']['user']['account_id'] || $GLOBALS['egw_info']['user']['apps']['admin']) {
+			if ($definition['owner'] && $definition['owner'] == $GLOBALS['egw_info']['user']['account_id'] || $GLOBALS['egw_info']['user']['apps']['admin']) {
 				// clear private cache
-				unset($this->definitions[array_search($key,$this->definitions)]);
+				unset($this->definitions[array_search($key, $this->definitions)]);
 			} else {
 				unset($keys[$index]);
 			}
 		}
-		if(count($keys) > 0) {
+		if (count($keys) > 0) {
 			$this->so_sql->delete(array('definition_id' => $keys));
 		}
 	}
@@ -154,12 +154,12 @@ 
 	 * @return bool
 	 */
 	static public function is_permitted($_definition) {
-		$allowed_user = is_array($_definition['allowed_users']) ? $_definition['allowed_users'] : explode(',',$_definition['allowed_users']);
+		$allowed_user = is_array($_definition['allowed_users']) ? $_definition['allowed_users'] : explode(',', $_definition['allowed_users']);
 		$this_user_id = $GLOBALS['egw_info']['user']['account_id'];
 		$this_membership = $GLOBALS['egw']->accounts->memberships($this_user_id, true);
 		$this_membership[] = $this_user_id;
 		$this_membership[] = 'all';
-		$alluser = array_intersect($allowed_user,$this_membership);
+		$alluser = array_intersect($allowed_user, $this_membership);
 		return ($this_user_id == $_definition['owner'] || count($alluser) > 0);
 	}
 
@@ -178,13 +178,13 @@ 
 
 		$export_data['definitions'] = array();
 		foreach ($keys as $definition_id) {
-			$definition = new importexport_definition( $definition_id );
+			$definition = new importexport_definition($definition_id);
 			$export_data['definitions'][$definition->name] = $definition->get_record_array();
 			$export_data['definitions'][$definition->name]['allowed_users'] =
 				importexport_helper_functions::account_id2name(
 					$export_data['definitions'][$definition->name]['allowed_users']
 				);
-			if($export_date['definitions'][$definition->name]['owner']) {
+			if ($export_date['definitions'][$definition->name]['owner']) {
 				$export_data['definitions'][$definition->name]['owner'] =
 					importexport_helper_functions::account_id2name(
 						$export_data['definitions'][$definition->name]['owner']
@@ -195,7 +195,7 @@ 
 			unset($export_data['definitions'][$definition->name]['definition_id']);
 			unset($export_data['definitions'][$definition->name]['description']);
 			unset($export_data['definitions'][$definition->name]['user_timezone_read']);
-			if(is_array($export_data['definitions'][$definition->name]['plugin_options'])) {
+			if (is_array($export_data['definitions'][$definition->name]['plugin_options'])) {
 				unset($export_data['definitions'][$definition->name]['plugin_options']['user_timezone_read']);
 			}
 			unset($definition);
@@ -213,17 +213,17 @@ 
 	 * @throws Exeption
 	 * @return void
 	 */
-	public static function import( $_import_file )
+	public static function import($_import_file)
 	{
-		if ( !is_file( $_import_file ) ) {
-			throw new Exception("'$_import_file' does not exist or is not readable" );
+		if (!is_file($_import_file)) {
+			throw new Exception("'$_import_file' does not exist or is not readable");
 		}
 
-		$data = importexport_arrayxml::xml2array( file_get_contents( $_import_file ) );
+		$data = importexport_arrayxml::xml2array(file_get_contents($_import_file));
 
 		$metainfo = $data['importExportDefinitions']['metainfo'];
 		$definitions = $data['importExportDefinitions']['definitions'];
-		unset ( $data );
+		unset ($data);
 
 		// convert charset into internal used charset
 		$definitions = Api\Translation::convert(
@@ -233,26 +233,26 @@ 
 		);
 
 		// Avoid warning if no definitions found
-		if(!is_array($definitions)) return lang('None found');
+		if (!is_array($definitions)) return lang('None found');
 
 		// save definition(s) into internal table
-		foreach ( $definitions as $name => $definition_data )
+		foreach ($definitions as $name => $definition_data)
 		{
 			// convert allowed_user
-			$definition_data['allowed_users'] = importexport_helper_functions::account_name2id( $definition_data['allowed_users'] );
-			if($definition_data['all_users'] && !$definition_data['allowed_users']) $definition_data['allowed_users'] = 'all';
+			$definition_data['allowed_users'] = importexport_helper_functions::account_name2id($definition_data['allowed_users']);
+			if ($definition_data['all_users'] && !$definition_data['allowed_users']) $definition_data['allowed_users'] = 'all';
 
-			$definition_data['owner'] = importexport_helper_functions::account_name2id( $definition_data['owner'] );
+			$definition_data['owner'] = importexport_helper_functions::account_name2id($definition_data['owner']);
 
-			$definition = new importexport_definition( $definition_data['name'] );
+			$definition = new importexport_definition($definition_data['name']);
 
 			// Only update if the imported is newer
-			if(strtotime($definition->modified) < strtotime($definition_data['modified']) || $definition->modified == 0)
+			if (strtotime($definition->modified) < strtotime($definition_data['modified']) || $definition->modified == 0)
 			{
 				$definition_id = $definition->get_identifier() ? $definition->get_identifier() : NULL;
 
-				$definition->set_record( $definition_data );
-				$definition->save( $definition_id );
+				$definition->set_record($definition_data);
+				$definition->save($definition_id);
 			}
 		}
 		return $definitions;
@@ -275,28 +275,28 @@ 
 	public static function export_from_import(importexport_definition $import)
 	{
 		// Only operates on import definitions
-		if($import->type != 'import') throw new Api\Exception\WrongParameter('Only import definitions');
+		if ($import->type != 'import') throw new Api\Exception\WrongParameter('Only import definitions');
 
 		// Find export plugin
-		$plugin = str_replace('import', 'export',$import->plugin);
+		$plugin = str_replace('import', 'export', $import->plugin);
 		$plugin_list = importexport_helper_functions::get_plugins($import->application, 'export');
-		foreach($plugin_list as $appname => $type)
+		foreach ($plugin_list as $appname => $type)
 		{
 			$plugins = $type['export'];
-			foreach($plugins as $name => $label)
+			foreach ($plugins as $name => $label)
 			{
-				if($plugin == $name) break;
+				if ($plugin == $name) break;
 			}
-			if($plugin !== $name) $plugin = $name;
+			if ($plugin !== $name) $plugin = $name;
 		}
 
 		$export = new importexport_definition();
 
 		// Common settings
-		$export->name = str_replace('import', 'export',$import->name);
-		if($export->name == $import->name) $export->name = $export->name . '-export';
+		$export->name = str_replace('import', 'export', $import->name);
+		if ($export->name == $import->name) $export->name = $export->name.'-export';
 		$test = new importexport_definition($export->name);
-		if($test->name) $export->name = $export->name .'-'.$GLOBALS['egw_info']['user']['account_lid'];
+		if ($test->name) $export->name = $export->name.'-'.$GLOBALS['egw_info']['user']['account_lid'];
 
 		$export->application = $import->application;
 		$export->plugin = $plugin;
@@ -312,7 +312,7 @@ 
 		);
 
 		// Mapping
-		foreach($i_options['field_mapping'] as $col_num => $field)
+		foreach ($i_options['field_mapping'] as $col_num => $field)
 		{
 			// Try to use heading from import file, if possible
 			$e_options['mapping'][$field] = $i_options['csv_fields'][$col_num] ? $i_options['csv_fields'][$col_num] : $field;

Braces +77 added lines, -29 removed lines

@@ -16,7 +16,8 @@ 
  *
  * @todo make this class an egw_record_pool!
  */
-class importexport_definitions_bo {
+class importexport_definitions_bo
+{
 
 	const _appname = 'importexport';
 	const _defintion_table = 'egw_importexport_definitions';
@@ -34,10 +35,15 @@ 
 	public function __construct($_query=false, $ignore_acl = false)
 	{
 		$this->so_sql = new Api\Storage\Base(self::_appname, self::_defintion_table );
-		if ($_query) {
+		if ($_query)
+		{
 			$definitions = $this->so_sql->search($_query, false);
-			foreach ((array)$definitions as $definition) {
-				if(self::is_permitted($definition) || $ignore_acl) $this->definitions[] = $definition['definition_id'];
+			foreach ((array)$definitions as $definition)
+			{
+				if(self::is_permitted($definition) || $ignore_acl)
+				{
+					$this->definitions[] = $definition['definition_id'];
+				}
 			}
 		}
 	}
@@ -45,7 +51,8 @@ 
 	public function get_rows(&$query, &$rows, &$readonlys)
 	{
 		// Filter only definitions user is allowed to use
-		if(!$GLOBALS['egw_info']['user']['apps']['admin']) {
+		if(!$GLOBALS['egw_info']['user']['apps']['admin'])
+		{
 			$this_membership = $GLOBALS['egw']->accounts->memberships($GLOBALS['egw_info']['user']['account_id'], true);
 			$this_membership[] = $GLOBALS['egw_info']['user']['account_id'];
 			$sql .= ' (';
@@ -80,7 +87,8 @@ 
 		
 		$total = $this->so_sql->get_rows($query, $rows, $readonlys);
 		$ro_count = 0;
-		foreach($rows as &$row) {
+		foreach($rows as &$row)
+		{
 			// Strip off leading + trailing ,
 			$row['allowed_users'] = substr($row['allowed_users'],1,-1);
 
@@ -103,14 +111,19 @@ 
 	 *
 	 * @return array
 	 */
-	public function get_definitions() {
+	public function get_definitions()
+	{
 		return $this->definitions;
 	}
-	public function read($definition_id) {
-		if(is_numeric($definition_id)) {
+	public function read($definition_id)
+	{
+		if(is_numeric($definition_id))
+		{
 			$this->so_sql->read($definition_id);
 			$definition = new importexport_definition($this->so_sql->data['name']);
-		} else {
+		}
+		else
+		{
 			$definition = new importexport_definition( $definition_id['name'] );
 		}
 		return $definition->get_record_array();
@@ -120,18 +133,24 @@ 
 	 *
 	 * @param array $keys
 	 */
-	public function delete($keys) {
-		foreach ($keys as $index => $key) {
+	public function delete($keys)
+	{
+		foreach ($keys as $index => $key)
+		{
 			// Check for ownership
 			$definition = $this->read($key);
-			if($definition['owner'] && $definition['owner'] == $GLOBALS['egw_info']['user']['account_id'] || $GLOBALS['egw_info']['user']['apps']['admin']) {
+			if($definition['owner'] && $definition['owner'] == $GLOBALS['egw_info']['user']['account_id'] || $GLOBALS['egw_info']['user']['apps']['admin'])
+			{
 				// clear private cache
 				unset($this->definitions[array_search($key,$this->definitions)]);
-			} else {
+			}
+			else
+			{
 				unset($keys[$index]);
 			}
 		}
-		if(count($keys) > 0) {
+		if(count($keys) > 0)
+		{
 			$this->so_sql->delete(array('definition_id' => $keys));
 		}
 	}
@@ -141,7 +160,8 @@ 
 	*
 	* @param definition $definition
 	*/
-	public function save(Array $data) {
+	public function save(Array $data)
+	{
 		$definition = new importexport_definition();
 		$definition->set_record($data);
 		$definition->save($data['definition_id']);
@@ -153,7 +173,8 @@ 
 	 * @param array $_definition
 	 * @return bool
 	 */
-	static public function is_permitted($_definition) {
+	static public function is_permitted($_definition)
+	{
 		$allowed_user = is_array($_definition['allowed_users']) ? $_definition['allowed_users'] : explode(',',$_definition['allowed_users']);
 		$this_user_id = $GLOBALS['egw_info']['user']['account_id'];
 		$this_membership = $GLOBALS['egw']->accounts->memberships($this_user_id, true);
@@ -177,25 +198,30 @@ 
 		));
 
 		$export_data['definitions'] = array();
-		foreach ($keys as $definition_id) {
+		foreach ($keys as $definition_id)
+		{
 			$definition = new importexport_definition( $definition_id );
 			$export_data['definitions'][$definition->name] = $definition->get_record_array();
 			$export_data['definitions'][$definition->name]['allowed_users'] =
 				importexport_helper_functions::account_id2name(
 					$export_data['definitions'][$definition->name]['allowed_users']
 				);
-			if($export_date['definitions'][$definition->name]['owner']) {
+			if($export_date['definitions'][$definition->name]['owner'])
+			{
 				$export_data['definitions'][$definition->name]['owner'] =
 					importexport_helper_functions::account_id2name(
 						$export_data['definitions'][$definition->name]['owner']
 					);
-			} else {
+			}
+			else
+			{
 				unset($export_data['definitions'][$definition->name]['owner']);
 			}
 			unset($export_data['definitions'][$definition->name]['definition_id']);
 			unset($export_data['definitions'][$definition->name]['description']);
 			unset($export_data['definitions'][$definition->name]['user_timezone_read']);
-			if(is_array($export_data['definitions'][$definition->name]['plugin_options'])) {
+			if(is_array($export_data['definitions'][$definition->name]['plugin_options']))
+			{
 				unset($export_data['definitions'][$definition->name]['plugin_options']['user_timezone_read']);
 			}
 			unset($definition);
@@ -215,7 +241,8 @@ 
 	 */
 	public static function import( $_import_file )
 	{
-		if ( !is_file( $_import_file ) ) {
+		if ( !is_file( $_import_file ) )
+		{
 			throw new Exception("'$_import_file' does not exist or is not readable" );
 		}
 
@@ -233,14 +260,20 @@ 
 		);
 
 		// Avoid warning if no definitions found
-		if(!is_array($definitions)) return lang('None found');
+		if(!is_array($definitions))
+		{
+			return lang('None found');
+		}
 
 		// save definition(s) into internal table
 		foreach ( $definitions as $name => $definition_data )
 		{
 			// convert allowed_user
 			$definition_data['allowed_users'] = importexport_helper_functions::account_name2id( $definition_data['allowed_users'] );
-			if($definition_data['all_users'] && !$definition_data['allowed_users']) $definition_data['allowed_users'] = 'all';
+			if($definition_data['all_users'] && !$definition_data['allowed_users'])
+			{
+				$definition_data['allowed_users'] = 'all';
+			}
 
 			$definition_data['owner'] = importexport_helper_functions::account_name2id( $definition_data['owner'] );
 
@@ -275,7 +308,10 @@ 
 	public static function export_from_import(importexport_definition $import)
 	{
 		// Only operates on import definitions
-		if($import->type != 'import') throw new Api\Exception\WrongParameter('Only import definitions');
+		if($import->type != 'import')
+		{
+			throw new Api\Exception\WrongParameter('Only import definitions');
+		}
 
 		// Find export plugin
 		$plugin = str_replace('import', 'export',$import->plugin);
@@ -285,18 +321,30 @@ 
 			$plugins = $type['export'];
 			foreach($plugins as $name => $label)
 			{
-				if($plugin == $name) break;
+				if($plugin == $name)
+				{
+					break;
+				}
+			}
+			if($plugin !== $name)
+			{
+				$plugin = $name;
 			}
-			if($plugin !== $name) $plugin = $name;
 		}
 
 		$export = new importexport_definition();
 
 		// Common settings
 		$export->name = str_replace('import', 'export',$import->name);
-		if($export->name == $import->name) $export->name = $export->name . '-export';
+		if($export->name == $import->name)
+		{
+			$export->name = $export->name . '-export';
+		}
 		$test = new importexport_definition($export->name);
-		if($test->name) $export->name = $export->name .'-'.$GLOBALS['egw_info']['user']['account_lid'];
+		if($test->name)
+		{
+			$export->name = $export->name .'-'.$GLOBALS['egw_info']['user']['account_lid'];
+		}
 
 		$export->application = $import->application;
 		$export->plugin = $plugin;

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

@@ -141,7 +141,7 @@
 	*
 	* @param definition $definition
 	*/
-	public function save(Array $data) {
+	public function save(array $data) {
 		$definition = new importexport_definition();
 		$definition->set_record($data);
 		$definition->save($data['definition_id']);

importexport/inc/class.importexport_definitions_ui.inc.php

Doc Comments +4 added lines, -3 removed lines

@@ -82,7 +82,7 @@ 
 	/**
 	 * List defined {im|ex}ports
 	 *
-	 * @param array $content=null
+	 * @param array $content
 	 */
 	function index($content = null,$msg='')
 	{
@@ -309,8 +309,9 @@ 
 	 * @param int &$success number of succeded actions
 	 * @param int &$failed number of failed actions (not enought permissions)
 	 * @param string &$action_msg translated verb for the actions, to be used in a message like %1 entries 'deleted'
-	 * @param string/array $session_name 'index' or 'email', or array with session-data depending if we are in the main list or the popup
-	 * @return boolean true if all actions succeded, false otherwise
+	 * @param string $session_name 'index' or 'email', or array with session-data depending if we are in the main list or the popup
+	 * @param string $msg
+	 * @return null|boolean true if all actions succeded, false otherwise
 	 */
 	function action($action,$selected,$use_all,&$success,&$failed,&$action_msg,$session_name,&$msg)
 	{

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

@@ -940,7 +940,7 @@
 	 * Determine if the user is allowed to edit the definition
 	 *
 	 */
-	protected function can_edit(Array $definition)
+	protected function can_edit(array $definition)
 	{
 		if($definition['owner'] && $definition['owner'] == $GLOBALS['egw_info']['user']['account_id'])
 		{

Braces +119 added lines, -44 removed lines

@@ -81,14 +81,18 @@ 
 	{
 		$filter = array('name' => '*');
 
-		if($GLOBALS['egw_info']['user']['apps']['admin']) {
+		if($GLOBALS['egw_info']['user']['apps']['admin'])
+		{
 			// Any public definition
 			$filter[] = '(owner=0 OR owner IS NULL OR allowed_users IS NOT NULL OR owner = ' . $GLOBALS['egw_info']['user']['account_id'] . ')';
-		} else {
+		}
+		else
+		{
 			// Filter private definitions
 			$filter['owner'] = $GLOBALS['egw_info']['user']['account_id'];
 			$config = Api\Config::read('phpgwapi');
-			if($config['export_limit'] == 'no' && !Api\Storage\Merge::is_export_limit_excepted()) {
+			if($config['export_limit'] == 'no' && !Api\Storage\Merge::is_export_limit_excepted())
+			{
 				$filter['type'] = 'import';
 			}
 		}
@@ -104,7 +108,10 @@ 
 			elseif(($button = array_search('pressed',$content['nm']['rows'])) !== false)
 			{
 				$selected = $content['nm']['rows']['selected'];
-				if(count($selected) < 1 || !is_array($selected)) exit();
+				if(count($selected) < 1 || !is_array($selected))
+				{
+					exit();
+				}
 				switch ($button)
 				{
 					case 'delete_selected' :
@@ -160,7 +167,8 @@ 
 			}
 		}
 
-		if(!is_array($content['nm'])) {
+		if(!is_array($content['nm']))
+		{
 			$content['nm'] = array(
 				'get_rows'	=> 'importexport.importexport_definitions_ui.get_rows',
 				'no_cat'	=> true,
@@ -171,7 +179,10 @@ 
 				'row_id'	=> 'definition_id',
 				'placeholder_actions' => array('add')
 			);
-			if($_GET['application']) $content['nm']['col_filter']['application'] = $_GET['application'];
+			if($_GET['application'])
+			{
+				$content['nm']['col_filter']['application'] = $_GET['application'];
+			}
 		}
 		if(Api\Cache::getSession('importexport', 'index'))
 		{
@@ -190,17 +201,22 @@ 
 		);
 		foreach ($this->plugins as $appname => $options)
 		{
-			if($GLOBALS['egw_info']['user']['apps'][$appname] || $GLOBALS['egw_info']['user']['apps']['admin']) {
+			if($GLOBALS['egw_info']['user']['apps'][$appname] || $GLOBALS['egw_info']['user']['apps']['admin'])
+			{
 				$sel_options['application'][$appname] = lang($appname);
 			}
 		}
-		if($msg) $content['msg'] = $msg;
+		if($msg)
+		{
+			$content['msg'] = $msg;
+		}
 
 		$etpl = new Etemplate(self::_appname.'.definition_index');
 		return $etpl->exec( self::_appname.'.importexport_definitions_ui.index', $content, $sel_options, $readonlys, $preserv );
 	}
 
-	private function get_actions() {
+	private function get_actions()
+	{
 		$group = 0;
 		$actions = array(
 			'edit' => array(
@@ -319,7 +335,8 @@ 
 			$this->get_rows($query,$rows,$readonlys);
 
 			$selected = array();
-			foreach($rows as $row) {
+			foreach($rows as $row)
+			{
 				$selected[] = $row['definition_id'];
 			}
 			if(!is_array($session_name))
@@ -334,18 +351,24 @@ 
 
 		$bodefinitions = new importexport_definitions_bo(false, true);
 
-		switch($action) {
+		switch($action)
+		{
 			case 'allowed':
 				// Field is allowed_users, popup doesn't like _
 				$action = 'allowed_users';
 				// Fall through
 			case 'owner':
 				$action_msg = lang('changed'. ' ' . $action);
-				foreach($selected as $id) {
+				foreach($selected as $id)
+				{
 					$definition = $bodefinitions->read((int)$id);
-					if($definition['definition_id']) {
+					if($definition['definition_id'])
+					{
 						// Prevent private with no owner
-						if(!$definition['owner'] && !$settings) $definition['owner'] = $GLOBALS['egw_info']['user']['account_id'];
+						if(!$definition['owner'] && !$settings)
+						{
+							$definition['owner'] = $GLOBALS['egw_info']['user']['account_id'];
+						}
 
 						$definition[$action] = $settings;
 						$bodefinitions->save($definition);
@@ -372,18 +395,22 @@ 
 			case 'copy':
 				$action_msg = lang('copied');
 				// Should only be one selected
-				foreach($selected as $id) {
+				foreach($selected as $id)
+				{
 					$definition = $bodefinitions->read((int)$id);
-					if($definition['definition_id']) {
+					if($definition['definition_id'])
+					{
 						unset($definition['definition_id']);
 						$definition['name'] = $settings ? $settings : $definition['name'] . ' copy';
 						try {
 							$bodefinitions->save($definition);
-						} catch (Exception $e) {
+						}
+						catch (Exception $e) {
 							try {
 								$definition['name'] .= ' ' . $GLOBALS['egw_info']['user']['account_lid'];
 								$bodefinitions->save($definition);
-							} catch (Exception $ex) {
+							}
+							catch (Exception $ex) {
 								$failed++;
 								$msg .= lang('Duplicate name, please choose another.');
 								continue;
@@ -396,18 +423,21 @@ 
 			case 'createexport':
 				$action_msg = lang('created');
 				// Should only be one selected
-				foreach($selected as $id) {
+				foreach($selected as $id)
+				{
 					$definition = new importexport_definition($id);
 					try {
 						$export = $bodefinitions->export_from_import($definition);
 						$export->save();
-					} catch (Exception $e) {
+					}
+					catch (Exception $e) {
 						if($export)
 						{
 							try {
 								$export->name = $export->name.' ' . $GLOBALS['egw_info']['user']['account_lid'];
 								$export->save();
-							} catch (Exception $ex) {
+							}
+							catch (Exception $ex) {
 								$failed++;
 								$msg .= lang('Duplicate name, please choose another.');
 								continue;
@@ -425,7 +455,8 @@ 
 		return !$failed;
 	}
 
-	public function get_rows(&$query, &$rows, &$readonlys) {
+	public function get_rows(&$query, &$rows, &$readonlys)
+	{
 		$rows = array();
 		Api\Cache::setSession('importexport', 'index', $query);
 
@@ -500,7 +531,10 @@ 
 
 		if(is_array($content) &&! $content['edit'])
 		{
-			if(self::_debug) error_log('importexport.wizard->$content '. print_r($content,true));
+			if(self::_debug)
+			{
+				error_log('importexport.wizard->$content '. print_r($content,true));
+			}
 			//foreach($content as $key => $val) error_log(" $key : ".array2string($val));
 			// fetch plugin object
 			if($content['plugin'] && $content['application'])
@@ -517,13 +551,18 @@ 
 					$wizard_plugin = $content['plugin'];
 				}
 				// App translations
-				if($content['application']) Api\Translation::add_app($content['application']);
+				if($content['application'])
+				{
+					Api\Translation::add_app($content['application']);
+				}
 
 				$this->plugin = is_object($GLOBALS['egw']->$wizard_plugin) ? $GLOBALS['egw']->$wizard_plugin : new $wizard_plugin;
 
 				// Global object needs to be the same, or references to plugin don't work
 				if(!is_object($GLOBALS['egw']->importexport_definitions_ui) || $GLOBALS['egw']->importexport_definitions_ui !== $this)
-					$GLOBALS['egw']->importexport_definitions_ui =& $this;
+				{
+									$GLOBALS['egw']->importexport_definitions_ui =& $this;
+				}
 			}
 			// deal with buttons even if we are not on ajax
 			if(isset($content['button']) && array_search('pressed',$content['button']) === false && count($content['button']) == 1)
@@ -653,7 +692,10 @@ 
 
 	function wizard_step10(&$content, &$sel_options, &$readonlys, &$preserv)
 	{
-		if(self::_debug) error_log('importexport.importexport_definitions_ui::wizard_step10->$content '.print_r($content,true));
+		if(self::_debug)
+		{
+			error_log('importexport.importexport_definitions_ui::wizard_step10->$content '.print_r($content,true));
+		}
 
 		// return from step10
 		if ($content['step'] == 'wizard_step10')
@@ -675,7 +717,8 @@ 
 			$content['text'] = $this->steps['wizard_step10'];
 			foreach ($this->plugins as $appname => $options)
 			{
-				if($GLOBALS['egw_info']['user']['apps'][$appname] || $GLOBALS['egw_info']['user']['apps']['admin']) {
+				if($GLOBALS['egw_info']['user']['apps'][$appname] || $GLOBALS['egw_info']['user']['apps']['admin'])
+				{
 					$sel_options['application'][$appname] = lang($appname);
 				}
 			}
@@ -691,7 +734,10 @@ 
 	// get plugin
 	function wizard_step20(&$content, &$sel_options, &$readonlys, &$preserv)
 	{
-		if(self::_debug) error_log('importexport.' . get_class($this) . '::wizard_step20->$content '.print_r($content,true));
+		if(self::_debug)
+		{
+			error_log('importexport.' . get_class($this) . '::wizard_step20->$content '.print_r($content,true));
+		}
 
 		// return from step20
 		if ($content['step'] == 'wizard_step20')
@@ -700,11 +746,16 @@ 
 			{
 				case 'next':
 					// There's no real reason the plugin has to come from any of these, as long as it has a $steps variable
-					if($this->plugin instanceof importexport_iface_import_plugin || $this->plugin instanceof importexport_wizard_basic_import_csv || strpos(get_class($this->plugin), 'import') !== false) {
+					if($this->plugin instanceof importexport_iface_import_plugin || $this->plugin instanceof importexport_wizard_basic_import_csv || strpos(get_class($this->plugin), 'import') !== false)
+					{
 						$content['type'] = 'import';
-					} elseif($this->plugin instanceof importexport_iface_export_plugin || $this->plugin instanceof importexport_wizard_basic_export_csv || strpos(get_class($this->plugin),'export') !== false) {
+					}
+					elseif($this->plugin instanceof importexport_iface_export_plugin || $this->plugin instanceof importexport_wizard_basic_export_csv || strpos(get_class($this->plugin),'export') !== false)
+					{
 						$content['type'] = 'export';
-					} else {
+					}
+					else
+					{
 						throw new Api\Exception('Invalid plugin');
 					}
 					return $this->get_step($content['step'],1);
@@ -722,9 +773,14 @@ 
 		{
 			$content['text'] = $this->steps['wizard_step20'];
 			$config = Api\Config::read('phpgwapi');
-			foreach ($this->plugins[$content['application']] as $type => $plugins) {
-				if($config['export_limit'] == 'no' && !$GLOBALS['egw_info']['user']['apps']['admin'] && $type == 'export') continue;
-				foreach($plugins as $plugin => $name) {
+			foreach ($this->plugins[$content['application']] as $type => $plugins)
+			{
+				if($config['export_limit'] == 'no' && !$GLOBALS['egw_info']['user']['apps']['admin'] && $type == 'export')
+				{
+					continue;
+				}
+				foreach($plugins as $plugin => $name)
+				{
 					$sel_options['plugin'][$plugin] = $name;
 				}
 			}
@@ -740,7 +796,10 @@ 
 	// name
 	function wizard_step21(&$content, &$sel_options, &$readonlys, &$preserv)
 	{
-		if(self::_debug) error_log('importexport.importexport_definitions_ui::wizard_step21->$content '.print_r($content,true));
+		if(self::_debug)
+		{
+			error_log('importexport.importexport_definitions_ui::wizard_step21->$content '.print_r($content,true));
+		}
 
 		// Check for duplicate name
 		$duplicate = isset($content['duplicate_error']);
@@ -752,7 +811,8 @@ 
 				{
 					throw new Exception('Already exists');
 				}
-			} catch (Exception $e) {
+			}
+			catch (Exception $e) {
 			//		throw new Exception('Already exists');
 				$content['duplicate_error'] = lang('Duplicate name, please choose another.');
 
@@ -763,13 +823,16 @@ 
 					$content['name'] .'-'. Api\DateTime::to('now', true),
 					//$content['name'] .'-'. rand(0,100),
 				);
-				foreach($suggestions as $key => $suggestion) {
+				foreach($suggestions as $key => $suggestion)
+				{
 					$sug_definition = new importexport_definition($suggestion);
-					if($sug_definition->definition_id) {
+					if($sug_definition->definition_id)
+					{
 						unset($suggestions[$key]);
 					}
 				}
-				if($suggestions) {
+				if($suggestions)
+				{
 					$content['duplicate_error'] .= '  '. lang('Try')." \n" . implode("\n", $suggestions);
 				}
 				return $this->get_step($content['step'],0);
@@ -806,7 +869,10 @@ 
 	// allowed users
 	function wizard_step90(&$content, &$sel_options, &$readonlys, &$preserv)
 	{
-		if(self::_debug) error_log('importexport.importexport_definitions_ui::wizard_step90->$content '.print_r($content,true));
+		if(self::_debug)
+		{
+			error_log('importexport.importexport_definitions_ui::wizard_step90->$content '.print_r($content,true));
+		}
 
 		// return from step90
 		if ($content['step'] == 'wizard_step90')
@@ -876,7 +942,10 @@ 
 
 	function wizard_finish(&$content)
 	{
-		if(self::_debug) error_log('importexport.importexport_definitions_ui::wizard_finish->$content '.print_r($content,true));
+		if(self::_debug)
+		{
+			error_log('importexport.importexport_definitions_ui::wizard_finish->$content '.print_r($content,true));
+		}
 		// Take out some UI leavings
 		unset($content['text']);
 		unset($content['step']);
@@ -905,7 +974,8 @@ 
 			if($content['update'])
 			{
 				$applist = importexport_helper_functions::get_apps('all', true);
-				foreach($applist as $appname) {
+				foreach($applist as $appname)
+				{
 					importexport_helper_functions::load_defaults($appname);
 				}
 				return $this->index();
@@ -971,7 +1041,9 @@ 
 			{
 				Api\Config::save_value($key, $value, 'importexport');
 			}
-		} elseif (isset($content['cancel'])) {
+		}
+		elseif (isset($content['cancel']))
+		{
 			$GLOBALS['egw']->redirect_link('/admin/index.php');
 		}
 
@@ -985,7 +1057,10 @@ 
 			unset($sel_options['import_charsets'][strtoupper($charset)]);
 		}
 
-		if(!$data['update']) $data['update'] = 'request';
+		if(!$data['update'])
+		{
+			$data['update'] = 'request';
+		}
 
 		$GLOBALS['egw_info']['flags']['app_header'] = lang('Site configuration') . ' - ' . lang(self::_appname);
 		$etpl = new etemplate(self::_appname.'.config');

Spacing +152 added lines, -153 removed lines

@@ -55,12 +55,12 @@ 
 	function __construct()
 	{
 		// we cant deal with notice and warnings, as we are on ajax!
-		error_reporting(E_ALL & ~E_NOTICE & ~E_WARNING);
+		error_reporting(E_ALL&~E_NOTICE&~E_WARNING);
 		Api\Translation::add_app(self::_appname);
 		$GLOBALS['egw_info']['flags']['currentapp'] = self::_appname;
 
 		$this->etpl = new Etemplate();
-		$this->clock = Api\Html::image(self::_appname,'clock');
+		$this->clock = Api\Html::image(self::_appname, 'clock');
 		$this->steps = array(
 			'wizard_step10' => lang('Choose an application'),
 			'wizard_step20' => lang('Choose a plugin'),
@@ -77,18 +77,18 @@ 
 	 *
 	 * @param array $content=null
 	 */
-	function index($content = null,$msg='')
+	function index($content = null, $msg = '')
 	{
 		$filter = array('name' => '*');
 
-		if($GLOBALS['egw_info']['user']['apps']['admin']) {
+		if ($GLOBALS['egw_info']['user']['apps']['admin']) {
 			// Any public definition
-			$filter[] = '(owner=0 OR owner IS NULL OR allowed_users IS NOT NULL OR owner = ' . $GLOBALS['egw_info']['user']['account_id'] . ')';
+			$filter[] = '(owner=0 OR owner IS NULL OR allowed_users IS NOT NULL OR owner = '.$GLOBALS['egw_info']['user']['account_id'].')';
 		} else {
 			// Filter private definitions
 			$filter['owner'] = $GLOBALS['egw_info']['user']['account_id'];
 			$config = Api\Config::read('phpgwapi');
-			if($config['export_limit'] == 'no' && !Api\Storage\Merge::is_export_limit_excepted()) {
+			if ($config['export_limit'] == 'no' && !Api\Storage\Merge::is_export_limit_excepted()) {
 				$filter['type'] = 'import';
 			}
 		}
@@ -99,12 +99,12 @@ 
 			if (isset($content['nm']['rows']['delete']))
 			{
 				$content['nm']['action'] = 'delete';
-				$content['nm']['selected'] = array_keys($content['nm']['rows']['delete'],'pressed');
+				$content['nm']['selected'] = array_keys($content['nm']['rows']['delete'], 'pressed');
 			}
-			elseif(($button = array_search('pressed',$content['nm']['rows'])) !== false)
+			elseif (($button = array_search('pressed', $content['nm']['rows'])) !== false)
 			{
 				$selected = $content['nm']['rows']['selected'];
-				if(count($selected) < 1 || !is_array($selected)) exit();
+				if (count($selected) < 1 || !is_array($selected)) exit();
 				switch ($button)
 				{
 					case 'delete_selected' :
@@ -126,54 +126,54 @@ 
 				else
 				{
 					// Action has an additional parameter
-					if(in_array($content['nm']['action'], array('owner', 'allowed')))
+					if (in_array($content['nm']['action'], array('owner', 'allowed')))
 					{
 						$action = $content['nm']['action'];
-						if($content['nm']['action'] == 'allowed')
+						if ($content['nm']['action'] == 'allowed')
 						{
 							$content['allowed'] = $content['allowed_popup']['allowed_private'] == 'true' ? null : (
-								$content['allowed_popup']['all_users']=='true' ? 'all' : implode(',',$content['allowed_popup']['allowed'])
+								$content['allowed_popup']['all_users'] == 'true' ? 'all' : implode(',', $content['allowed_popup']['allowed'])
 							);
 						}
 						else
 						{
 							$content['owner'] = $content['owner_popup']['owner'];
 						}
-						if(is_array($content[$content['nm']['action']]))
+						if (is_array($content[$content['nm']['action']]))
 						{
-							$content[$content['nm']['action']] = implode(',',$content[$content['nm']['action']]);
+							$content[$content['nm']['action']] = implode(',', $content[$content['nm']['action']]);
 						}
-						$content['nm']['action'] .= '_' . $content[$action];
+						$content['nm']['action'] .= '_'.$content[$action];
 						unset($content[$action]);
 						unset($content['allowed_popup']);
 					}
-					if ($this->action($content['nm']['action'],$content['nm']['selected'],$content['nm']['select_all'],
-						$success,$failed,$action_msg,'index',$msg))
+					if ($this->action($content['nm']['action'], $content['nm']['selected'], $content['nm']['select_all'],
+						$success, $failed, $action_msg, 'index', $msg))
 					{
-						$msg .= lang('%1 definition(s) %2',$success,$action_msg);
+						$msg .= lang('%1 definition(s) %2', $success, $action_msg);
 					}
-					elseif(empty($msg))
+					elseif (empty($msg))
 					{
-						$msg .= lang('%1 definition(s) %2, %3 failed because of insufficent rights !!!',$success,$action_msg,$failed);
+						$msg .= lang('%1 definition(s) %2, %3 failed because of insufficent rights !!!', $success, $action_msg, $failed);
 					}
 				}
 			}
 		}
 
-		if(!is_array($content['nm'])) {
+		if (!is_array($content['nm'])) {
 			$content['nm'] = array(
 				'get_rows'	=> 'importexport.importexport_definitions_ui.get_rows',
 				'no_cat'	=> true,
 				'no_filter'	=> true,
 				'no_filter2'	=> true,
-				'csv_fields'	=> false,	// Disable CSV export, uses own export
-				'default_cols'  => '!actions',  // switch legacy actions column and row off by default
+				'csv_fields'	=> false, // Disable CSV export, uses own export
+				'default_cols'  => '!actions', // switch legacy actions column and row off by default
 				'row_id'	=> 'definition_id',
 				'placeholder_actions' => array('add')
 			);
-			if($_GET['application']) $content['nm']['col_filter']['application'] = $_GET['application'];
+			if ($_GET['application']) $content['nm']['col_filter']['application'] = $_GET['application'];
 		}
-		if(Api\Cache::getSession('importexport', 'index'))
+		if (Api\Cache::getSession('importexport', 'index'))
 		{
 			$content['nm'] = array_merge($content['nm'], Api\Cache::getSession('importexport', 'index'));
 		}
@@ -190,14 +190,14 @@ 
 		);
 		foreach ($this->plugins as $appname => $options)
 		{
-			if($GLOBALS['egw_info']['user']['apps'][$appname] || $GLOBALS['egw_info']['user']['apps']['admin']) {
+			if ($GLOBALS['egw_info']['user']['apps'][$appname] || $GLOBALS['egw_info']['user']['apps']['admin']) {
 				$sel_options['application'][$appname] = lang($appname);
 			}
 		}
-		if($msg) $content['msg'] = $msg;
+		if ($msg) $content['msg'] = $msg;
 
 		$etpl = new Etemplate(self::_appname.'.definition_index');
-		return $etpl->exec( self::_appname.'.importexport_definitions_ui.index', $content, $sel_options, $readonlys, $preserv );
+		return $etpl->exec(self::_appname.'.importexport_definitions_ui.index', $content, $sel_options, $readonlys, $preserv);
 	}
 
 	private function get_actions() {
@@ -286,7 +286,7 @@ 
 		);
 
 		// Unset admin actions
-		if(!$GLOBALS['egw_info']['user']['apps']['admin'])
+		if (!$GLOBALS['egw_info']['user']['apps']['admin'])
 		{
 			unset($actions['schedule']);
 		}
@@ -305,7 +305,7 @@ 
 	 * @param string/array $session_name 'index' or 'email', or array with session-data depending if we are in the main list or the popup
 	 * @return boolean true if all actions succeded, false otherwise
 	 */
-	function action($action,$selected,$use_all,&$success,&$failed,&$action_msg,$session_name,&$msg)
+	function action($action, $selected, $use_all, &$success, &$failed, &$action_msg, $session_name, &$msg)
 	{
 		//error_log( __METHOD__."('$action', ".array2string($selected).', '.array2string($use_all).",,, '$session_name')");
 		if ($use_all)
@@ -313,16 +313,16 @@ 
 			// get the whole selection
 			$old_query = $query = is_array($session_name) ? $session_name : Api\Cache::getSession('importexport', $session_name);
 
-			@set_time_limit(0);				// switch off the execution time limit, as it's for big selections to small
-			$query['num_rows'] = -1;		// all
-			$query['csv_export'] = true;	// so get_rows method _can_ produce different content or not store state in the session
-			$this->get_rows($query,$rows,$readonlys);
+			@set_time_limit(0); // switch off the execution time limit, as it's for big selections to small
+			$query['num_rows'] = -1; // all
+			$query['csv_export'] = true; // so get_rows method _can_ produce different content or not store state in the session
+			$this->get_rows($query, $rows, $readonlys);
 
 			$selected = array();
-			foreach($rows as $row) {
+			foreach ($rows as $row) {
 				$selected[] = $row['definition_id'];
 			}
-			if(!is_array($session_name))
+			if (!is_array($session_name))
 			{
 				// Restore old query
 				Api\Cache::setSession('importexport', $session_name, $old_query);
@@ -334,18 +334,18 @@ 
 
 		$bodefinitions = new importexport_definitions_bo(false, true);
 
-		switch($action) {
+		switch ($action) {
 			case 'allowed':
 				// Field is allowed_users, popup doesn't like _
 				$action = 'allowed_users';
 				// Fall through
 			case 'owner':
-				$action_msg = lang('changed'. ' ' . $action);
-				foreach($selected as $id) {
+				$action_msg = lang('changed'.' '.$action);
+				foreach ($selected as $id) {
 					$definition = $bodefinitions->read((int)$id);
-					if($definition['definition_id']) {
+					if ($definition['definition_id']) {
 						// Prevent private with no owner
-						if(!$definition['owner'] && !$settings) $definition['owner'] = $GLOBALS['egw_info']['user']['account_id'];
+						if (!$definition['owner'] && !$settings) $definition['owner'] = $GLOBALS['egw_info']['user']['account_id'];
 
 						$definition[$action] = $settings;
 						$bodefinitions->save($definition);
@@ -363,7 +363,7 @@ 
 				$mime_type = ($GLOBALS['egw']->html->user_agent == 'msie' || $GLOBALS['egw']->html->user_agent == 'opera') ?
 					'application/octetstream' : 'application/octet-stream';
 				$name = 'importexport_definition.xml';
-				header('Content-Type: ' . $mime_type);
+				header('Content-Type: '.$mime_type);
 				header('Content-Disposition: attachment; filename="'.$name.'"');
 				echo $bodefinitions->export($selected);
 				exit();
@@ -372,16 +372,16 @@ 
 			case 'copy':
 				$action_msg = lang('copied');
 				// Should only be one selected
-				foreach($selected as $id) {
+				foreach ($selected as $id) {
 					$definition = $bodefinitions->read((int)$id);
-					if($definition['definition_id']) {
+					if ($definition['definition_id']) {
 						unset($definition['definition_id']);
-						$definition['name'] = $settings ? $settings : $definition['name'] . ' copy';
+						$definition['name'] = $settings ? $settings : $definition['name'].' copy';
 						try {
 							$bodefinitions->save($definition);
 						} catch (Exception $e) {
 							try {
-								$definition['name'] .= ' ' . $GLOBALS['egw_info']['user']['account_lid'];
+								$definition['name'] .= ' '.$GLOBALS['egw_info']['user']['account_lid'];
 								$bodefinitions->save($definition);
 							} catch (Exception $ex) {
 								$failed++;
@@ -396,16 +396,16 @@ 
 			case 'createexport':
 				$action_msg = lang('created');
 				// Should only be one selected
-				foreach($selected as $id) {
+				foreach ($selected as $id) {
 					$definition = new importexport_definition($id);
 					try {
 						$export = $bodefinitions->export_from_import($definition);
 						$export->save();
 					} catch (Exception $e) {
-						if($export)
+						if ($export)
 						{
 							try {
-								$export->name = $export->name.' ' . $GLOBALS['egw_info']['user']['account_lid'];
+								$export->name = $export->name.' '.$GLOBALS['egw_info']['user']['account_lid'];
 								$export->save();
 							} catch (Exception $ex) {
 								$failed++;
@@ -430,10 +430,10 @@ 
 		Api\Cache::setSession('importexport', 'index', $query);
 
 		// Special handling for allowed users 'private'
-		if($query['col_filter']['allowed_users'] == 'private')
+		if ($query['col_filter']['allowed_users'] == 'private')
 		{
 			unset($query['col_filter']['allowed_users']);
-			$query['col_filter'][] = 'allowed_users = ' . $GLOBALS['egw']->db->quote(',,');
+			$query['col_filter'][] = 'allowed_users = '.$GLOBALS['egw']->db->quote(',,');
 		}
 		$bodefinitions = new importexport_definitions_bo($query['col_filter'], true);
 		// We don't care about readonlys for the UI
@@ -448,7 +448,7 @@ 
 	 */
 	function edit()
 	{
-		if(!$_definition = $_GET['definition'])
+		if (!$_definition = $_GET['definition'])
 		{
 			$content = array(
 				'edit'		=> true,
@@ -457,11 +457,11 @@ 
 			);
 
 			// Jump to a step
-			if($_GET['step'])
+			if ($_GET['step'])
 			{
 				$content['edit'] = false;
 				// Wizard will process previous step, then advance
-				$content['step'] = $this->get_step($_GET['step'],-1);
+				$content['step'] = $this->get_step($_GET['step'], -1);
 				$content['button']['next'] = 'pressed';
 				$this->wizard($content);
 			}
@@ -472,7 +472,7 @@ 
 			}
 			return;
 		}
-		if(is_numeric($_GET['definition']))
+		if (is_numeric($_GET['definition']))
 		{
 			$definition = (int)$_GET['definition'];
 		}
@@ -484,31 +484,31 @@ 
 		$definition = $bodefinitions->read($definition);
 		$definition['edit'] = true;
 		// Jump to a step
-		if($_GET['step'])
+		if ($_GET['step'])
 		{
 			$definition['edit'] = false;
 			// Wizard will process previous step, then advance
-			$definition['step'] = $_GET['step'];;
+			$definition['step'] = $_GET['step']; ;
 			$definition['button'] = array('next' => 'pressed');
 		}
 		$this->wizard($definition);
 	}
 
-	function wizard($content = null, $msg='')
+	function wizard($content = null, $msg = '')
 	{
 		$this->etpl->read('importexport.wizardbox');
 
-		if(is_array($content) &&! $content['edit'])
+		if (is_array($content) && !$content['edit'])
 		{
-			if(self::_debug) error_log('importexport.wizard->$content '. print_r($content,true));
+			if (self::_debug) error_log('importexport.wizard->$content '.print_r($content, true));
 			//foreach($content as $key => $val) error_log(" $key : ".array2string($val));
 			// fetch plugin object
-			if($content['plugin'] && $content['application'])
+			if ($content['plugin'] && $content['application'])
 			{
-				$wizard_name = $content['application'] . '_wizard_' . str_replace($content['application'] . '_', '', $content['plugin']);
+				$wizard_name = $content['application'].'_wizard_'.str_replace($content['application'].'_', '', $content['plugin']);
 
 				// we need to deal with the wizard object if exists
-				if (file_exists(EGW_SERVER_ROOT . '/'. $content['application']."/inc/class.$wizard_name.inc.php"))
+				if (file_exists(EGW_SERVER_ROOT.'/'.$content['application']."/inc/class.$wizard_name.inc.php"))
 				{
 					$wizard_plugin = $wizard_name;
 				}
@@ -517,38 +517,38 @@ 
 					$wizard_plugin = $content['plugin'];
 				}
 				// App translations
-				if($content['application']) Api\Translation::add_app($content['application']);
+				if ($content['application']) Api\Translation::add_app($content['application']);
 
 				$this->plugin = is_object($GLOBALS['egw']->$wizard_plugin) ? $GLOBALS['egw']->$wizard_plugin : new $wizard_plugin;
 
 				// Global object needs to be the same, or references to plugin don't work
-				if(!is_object($GLOBALS['egw']->importexport_definitions_ui) || $GLOBALS['egw']->importexport_definitions_ui !== $this)
-					$GLOBALS['egw']->importexport_definitions_ui =& $this;
+				if (!is_object($GLOBALS['egw']->importexport_definitions_ui) || $GLOBALS['egw']->importexport_definitions_ui !== $this)
+					$GLOBALS['egw']->importexport_definitions_ui = & $this;
 			}
 			// deal with buttons even if we are not on ajax
-			if(isset($content['button']) && array_search('pressed',$content['button']) === false && count($content['button']) == 1)
+			if (isset($content['button']) && array_search('pressed', $content['button']) === false && count($content['button']) == 1)
 			{
 				$button = array_keys($content['button']);
 				$content['button'] = array($button[0] => 'pressed');
 			}
 
 			// post process submitted step
-			if($content['step'])
+			if ($content['step'])
 			{
-				if(!$this->can_edit($content))
+				if (!$this->can_edit($content))
 				{
 					// Each step changes definition, reload it
 					$bodefinitions = new importexport_definitions_bo();
 					$definition = $bodefinitions->read($content);
 					$content = $definition + array('step' => $content['step'], 'button' => $content['button']);
 				}
-				if(!key_exists($content['step'],$this->steps))
+				if (!key_exists($content['step'], $this->steps))
 				{
-					$next_step = $this->plugin->{$content['step']}($content,$sel_options,$readonlys,$preserv);
+					$next_step = $this->plugin->{$content['step']}($content, $sel_options, $readonlys, $preserv);
 				}
 				else
 				{
-					$next_step = $this->{$content['step']}($content,$sel_options,$readonlys,$preserv);
+					$next_step = $this->{$content['step']}($content, $sel_options, $readonlys, $preserv);
 				}
 			}
 			else
@@ -560,23 +560,23 @@ 
 			$sel_options = $readonlys = $preserv = array();
 
 			// Disable finish button if required fields are missing
-			if(!$content['name'] || !$content['type'] || !$content['plugin'])
+			if (!$content['name'] || !$content['type'] || !$content['plugin'])
 			{
 				$readonlys['button[finish]'] = true;
 			}
 			do
 			{
-				if(!key_exists($next_step,$this->steps))
+				if (!key_exists($next_step, $this->steps))
 				{
-					$this->wizard_content_template = $this->plugin->$next_step($content,$sel_options,$readonlys,$preserv);
+					$this->wizard_content_template = $this->plugin->$next_step($content, $sel_options, $readonlys, $preserv);
 				}
 				else
 				{
-					$this->wizard_content_template = $this->$next_step($content,$sel_options,$readonlys,$preserv);
+					$this->wizard_content_template = $this->$next_step($content, $sel_options, $readonlys, $preserv);
 				}
-				if($this->wizard_content_template == self::SKIP)
+				if ($this->wizard_content_template == self::SKIP)
 				{
-					if(!key_exists($content['step'],$this->steps))
+					if (!key_exists($content['step'], $this->steps))
 					{
 						$next_step = $this->plugin->{$content['step']}($content);
 					}
@@ -585,9 +585,9 @@ 
 						$next_step = $this->{$content['step']}($content);
 					}
 				}
-			} while($this->wizard_content_template == self::SKIP);
+			} while ($this->wizard_content_template == self::SKIP);
 
-			if(!$this->can_edit($content))
+			if (!$this->can_edit($content))
 			{
 				$readonlys[$this->wizard_content_template] = true;
 				$preserve = $content;
@@ -596,21 +596,21 @@ 
 
 			unset($content['button']);
 			$content['wizard_content'] = $this->wizard_content_template;
-			$this->etpl->exec(self::_appname.'.importexport_definitions_ui.wizard',$content,$sel_options,$readonlys,$preserv,2);
+			$this->etpl->exec(self::_appname.'.importexport_definitions_ui.wizard', $content, $sel_options, $readonlys, $preserv, 2);
 		}
 		else
 		{
 			// initial content
 			$sel_options = $readonlys = $preserv = array();
 			$readonlys['button[previous]'] = true;
-			if($content['edit'])
+			if ($content['edit'])
 			{
 				unset ($content['edit']);
 			}
 
 			$this->wizard_content_template = $this->wizard_step10($content, $sel_options, $readonlys, $preserv);
 
-			if(!$this->can_edit($content))
+			if (!$this->can_edit($content))
 			{
 				$readonlys[$this->wizard_content_template] = true;
 				$preserve = $content;
@@ -618,7 +618,7 @@ 
 			}
 
 			$content['wizard_content'] = $this->wizard_content_template;
-			$this->etpl->exec(self::_appname.'.importexport_definitions_ui.wizard',$content,$sel_options,$readonlys,$preserv,2);
+			$this->etpl->exec(self::_appname.'.importexport_definitions_ui.wizard', $content, $sel_options, $readonlys, $preserv, 2);
 		}
 	}
 
@@ -629,31 +629,31 @@ 
 	 * @param int $step_width
 	 * @return string containing function name of next step
 	 */
-	function get_step ($curr_step, $step_width)
+	function get_step($curr_step, $step_width)
 	{
 		/*if($content['plugin'] && $content['application']&& !is_object($this->plugin))
 		{
 			$plugin_definition =  $this->plugins[$content['application']][$content['plugin']]['definition'];
 			if($plugin_definition) $this->plugin = new $plugin_definition;
 		}*/
-		if(is_object($this->plugin) && is_array($this->plugin->steps))
+		if (is_object($this->plugin) && is_array($this->plugin->steps))
 		{
-			$steps = array_merge($this->steps,$this->plugin->steps);
-			$steps = array_flip($steps); asort($steps);	$steps = array_flip($steps);
+			$steps = array_merge($this->steps, $this->plugin->steps);
+			$steps = array_flip($steps); asort($steps); $steps = array_flip($steps);
 		}
 		else
 		{
 			$steps = $this->steps;
 		}
 		$step_keys = array_keys($steps);
-		$nn = array_search($curr_step,$step_keys)+(int)$step_width;
-		return (key_exists($nn,$step_keys)) ? $step_keys[$nn] : 'wizard_finish';
+		$nn = array_search($curr_step, $step_keys) + (int)$step_width;
+		return (key_exists($nn, $step_keys)) ? $step_keys[$nn] : 'wizard_finish';
 	}
 
 
 	function wizard_step10(&$content, &$sel_options, &$readonlys, &$preserv)
 	{
-		if(self::_debug) error_log('importexport.importexport_definitions_ui::wizard_step10->$content '.print_r($content,true));
+		if (self::_debug) error_log('importexport.importexport_definitions_ui::wizard_step10->$content '.print_r($content, true));
 
 		// return from step10
 		if ($content['step'] == 'wizard_step10')
@@ -661,11 +661,11 @@ 
 			switch (array_search('pressed', $content['button']))
 			{
 				case 'next':
-					return $this->get_step($content['step'],1);
+					return $this->get_step($content['step'], 1);
 				case 'finish':
 					return 'wizard_finish';
 				default :
-					return $this->wizard_step10($content,$sel_options,$readonlys,$preserv);
+					return $this->wizard_step10($content, $sel_options, $readonlys, $preserv);
 			}
 
 		}
@@ -675,7 +675,7 @@ 
 			$content['text'] = $this->steps['wizard_step10'];
 			foreach ($this->plugins as $appname => $options)
 			{
-				if($GLOBALS['egw_info']['user']['apps'][$appname] || $GLOBALS['egw_info']['user']['apps']['admin']) {
+				if ($GLOBALS['egw_info']['user']['apps'][$appname] || $GLOBALS['egw_info']['user']['apps']['admin']) {
 					$sel_options['application'][$appname] = lang($appname);
 				}
 			}
@@ -691,7 +691,7 @@ 
 	// get plugin
 	function wizard_step20(&$content, &$sel_options, &$readonlys, &$preserv)
 	{
-		if(self::_debug) error_log('importexport.' . get_class($this) . '::wizard_step20->$content '.print_r($content,true));
+		if (self::_debug) error_log('importexport.'.get_class($this).'::wizard_step20->$content '.print_r($content, true));
 
 		// return from step20
 		if ($content['step'] == 'wizard_step20')
@@ -700,21 +700,21 @@ 
 			{
 				case 'next':
 					// There's no real reason the plugin has to come from any of these, as long as it has a $steps variable
-					if($this->plugin instanceof importexport_iface_import_plugin || $this->plugin instanceof importexport_wizard_basic_import_csv || strpos(get_class($this->plugin), 'import') !== false) {
+					if ($this->plugin instanceof importexport_iface_import_plugin || $this->plugin instanceof importexport_wizard_basic_import_csv || strpos(get_class($this->plugin), 'import') !== false) {
 						$content['type'] = 'import';
-					} elseif($this->plugin instanceof importexport_iface_export_plugin || $this->plugin instanceof importexport_wizard_basic_export_csv || strpos(get_class($this->plugin),'export') !== false) {
+					} elseif ($this->plugin instanceof importexport_iface_export_plugin || $this->plugin instanceof importexport_wizard_basic_export_csv || strpos(get_class($this->plugin), 'export') !== false) {
 						$content['type'] = 'export';
 					} else {
 						throw new Api\Exception('Invalid plugin');
 					}
-					return $this->get_step($content['step'],1);
+					return $this->get_step($content['step'], 1);
 				case 'previous' :
 					$readonlys['button[previous]'] = true;
-					return $this->get_step($content['step'],-1);
+					return $this->get_step($content['step'], -1);
 				case 'finish':
 					return 'wizard_finish';
 				default :
-					return $this->wizard_step20($content,$sel_options,$readonlys,$preserv);
+					return $this->wizard_step20($content, $sel_options, $readonlys, $preserv);
 			}
 		}
 		// init step20
@@ -723,8 +723,8 @@ 
 			$content['text'] = $this->steps['wizard_step20'];
 			$config = Api\Config::read('phpgwapi');
 			foreach ($this->plugins[$content['application']] as $type => $plugins) {
-				if($config['export_limit'] == 'no' && !$GLOBALS['egw_info']['user']['apps']['admin'] && $type == 'export') continue;
-				foreach($plugins as $plugin => $name) {
+				if ($config['export_limit'] == 'no' && !$GLOBALS['egw_info']['user']['apps']['admin'] && $type == 'export') continue;
+				foreach ($plugins as $plugin => $name) {
 					$sel_options['plugin'][$plugin] = $name;
 				}
 			}
@@ -740,15 +740,15 @@ 
 	// name
 	function wizard_step21(&$content, &$sel_options, &$readonlys, &$preserv)
 	{
-		if(self::_debug) error_log('importexport.importexport_definitions_ui::wizard_step21->$content '.print_r($content,true));
+		if (self::_debug) error_log('importexport.importexport_definitions_ui::wizard_step21->$content '.print_r($content, true));
 
 		// Check for duplicate name
 		$duplicate = isset($content['duplicate_error']);
-		if($content['name'] && !$duplicate)
+		if ($content['name'] && !$duplicate)
 		{
 			try {
 				$check_definition = new importexport_definition($content['name']);
-				if($check_definition && $check_definition->definition_id && $check_definition->definition_id != $content['definition_id'])
+				if ($check_definition && $check_definition->definition_id && $check_definition->definition_id != $content['definition_id'])
 				{
 					throw new Exception('Already exists');
 				}
@@ -758,21 +758,21 @@ 
 
 				// Try some suggestions
 				$suggestions = array(
-					$content['name'] .'-'. $GLOBALS['egw_info']['user']['account_lid'],
-					$content['name'] .'-'. $GLOBALS['egw_info']['user']['account_id'],
-					$content['name'] .'-'. Api\DateTime::to('now', true),
+					$content['name'].'-'.$GLOBALS['egw_info']['user']['account_lid'],
+					$content['name'].'-'.$GLOBALS['egw_info']['user']['account_id'],
+					$content['name'].'-'.Api\DateTime::to('now', true),
 					//$content['name'] .'-'. rand(0,100),
 				);
-				foreach($suggestions as $key => $suggestion) {
+				foreach ($suggestions as $key => $suggestion) {
 					$sug_definition = new importexport_definition($suggestion);
-					if($sug_definition->definition_id) {
+					if ($sug_definition->definition_id) {
 						unset($suggestions[$key]);
 					}
 				}
-				if($suggestions) {
-					$content['duplicate_error'] .= '  '. lang('Try')." \n" . implode("\n", $suggestions);
+				if ($suggestions) {
+					$content['duplicate_error'] .= '  '.lang('Try')." \n".implode("\n", $suggestions);
 				}
-				return $this->get_step($content['step'],0);
+				return $this->get_step($content['step'], 0);
 			}
 		}
 
@@ -782,19 +782,19 @@ 
 			switch (array_search('pressed', $content['button']))
 			{
 				case 'next':
-					return $this->get_step($content['step'],1);
+					return $this->get_step($content['step'], 1);
 				case 'previous' :
-					return $this->get_step($content['step'],-1);
+					return $this->get_step($content['step'], -1);
 				case 'finish':
 					return 'wizard_finish';
 				default :
-					return $this->wizard_step21($content,$sel_options,$readonlys,$preserv);
+					return $this->wizard_step21($content, $sel_options, $readonlys, $preserv);
 			}
 		}
 		// init step21
 		else
 		{
-			$content['text'] = $this->steps['wizard_step21'] . ($duplicate ? "\n".$content['duplicate_error'] : '');
+			$content['text'] = $this->steps['wizard_step21'].($duplicate ? "\n".$content['duplicate_error'] : '');
 			$content['step'] = 'wizard_step21';
 			unset($content['duplicate_error']);
 			$preserv = $content;
@@ -806,17 +806,16 @@ 
 	// allowed users
 	function wizard_step90(&$content, &$sel_options, &$readonlys, &$preserv)
 	{
-		if(self::_debug) error_log('importexport.importexport_definitions_ui::wizard_step90->$content '.print_r($content,true));
+		if (self::_debug) error_log('importexport.importexport_definitions_ui::wizard_step90->$content '.print_r($content, true));
 
 		// return from step90
 		if ($content['step'] == 'wizard_step90')
 		{
-			if($this->can_edit($content))
+			if ($this->can_edit($content))
 			{
-				$content['owner'] = $content['just_me'] || !$GLOBALS['egw']->acl->check('share_definitions', Acl::READ,'importexport') ?
-					($content['owner'] ? $content['owner'] : $GLOBALS['egw_info']['user']['account_id']) :
-					null;
-				$content['allowed_users'] = $content['just_me'] ? '' : ($content['all_users'] ? 'all' : implode(',',$content['allowed_users']));
+				$content['owner'] = $content['just_me'] || !$GLOBALS['egw']->acl->check('share_definitions', Acl::READ, 'importexport') ?
+					($content['owner'] ? $content['owner'] : $GLOBALS['egw_info']['user']['account_id']) : null;
+				$content['allowed_users'] = $content['just_me'] ? '' : ($content['all_users'] ? 'all' : implode(',', $content['allowed_users']));
 				unset($content['just_me']);
 			}
 
@@ -824,12 +823,12 @@ 
 			switch (array_search('pressed', $content['button']))
 			{
 				case 'previous' :
-					return $this->get_step($content['step'],-1);
+					return $this->get_step($content['step'], -1);
 				case 'next':
 				case 'finish':
 					return 'wizard_finish';
 				default :
-					return $this->wizard_step90($content,$sel_options,$readonlys,$preserv);
+					return $this->wizard_step90($content, $sel_options, $readonlys, $preserv);
 			}
 		}
 		// init step90
@@ -840,10 +839,10 @@ 
 			$preserv = $content;
 
 			// Set owner for non-admins
-			$content['just_me'] = ((!$content['allowed_users'] || !$content['allowed_users'][0] && count($content['allowed_users']) ==1) && $content['owner']);
-			$content['all_users'] = is_array($content['allowed_users']) && array_key_exists('0',$content['allowed_users']) && $content['allowed_users'][0] == 'all' ||
+			$content['just_me'] = ((!$content['allowed_users'] || !$content['allowed_users'][0] && count($content['allowed_users']) == 1) && $content['owner']);
+			$content['all_users'] = is_array($content['allowed_users']) && array_key_exists('0', $content['allowed_users']) && $content['allowed_users'][0] == 'all' ||
 			$content['allowed_users'] == 'all';
-			if(!$GLOBALS['egw']->acl->check('share_definition', Acl::READ, 'importexport') && !$GLOBALS['egw_info']['user']['apps']['admin'])
+			if (!$GLOBALS['egw']->acl->check('share_definition', Acl::READ, 'importexport') && !$GLOBALS['egw_info']['user']['apps']['admin'])
 			{
 				$content['allowed_users'] = array();
 				$readonlys['allowed_users'] = true;
@@ -859,11 +858,11 @@ 
 				)
 			);
 			// Hide 'just me' checkbox, users get confused by read-only
-			if($readonlys['just_me'] || !$this->can_edit($content))
+			if ($readonlys['just_me'] || !$this->can_edit($content))
 			{
 				$content['no_just_me'] = true;
 			}
-			if($readonlys['all_users'] || !$this->can_edit($content))
+			if ($readonlys['all_users'] || !$this->can_edit($content))
 			{
 				$content['no_all_users'] = true;
 			}
@@ -876,7 +875,7 @@ 
 
 	function wizard_finish(&$content)
 	{
-		if(self::_debug) error_log('importexport.importexport_definitions_ui::wizard_finish->$content '.print_r($content,true));
+		if (self::_debug) error_log('importexport.importexport_definitions_ui::wizard_finish->$content '.print_r($content, true));
 		// Take out some UI leavings
 		unset($content['text']);
 		unset($content['step']);
@@ -886,26 +885,26 @@ 
 		$bodefinitions->save($content);
 		// This message is displayed if browser cant close window
 		$content['msg'] = lang('ImportExport wizard finished successfully!');
-		Framework::refresh_opener('','importexport');
+		Framework::refresh_opener('', 'importexport');
 		Framework::window_close();
 		return 'importexport.wizard_close';
 	}
 
-	function import_definition($content='')
+	function import_definition($content = '')
 	{
 		$bodefinitions = new importexport_definitions_bo();
 		if (is_array($content))
 		{
-			if($content['import_file']['tmp_name'])
+			if ($content['import_file']['tmp_name'])
 			{
 				$result = $bodefinitions->import($content['import_file']['tmp_name']);
-				$msg = lang('%1 definitions %2', count($result), lang('imported')) ."\n". implode("\n", array_keys($result));
+				$msg = lang('%1 definitions %2', count($result), lang('imported'))."\n".implode("\n", array_keys($result));
 				return $this->index(null, $msg);
 			}
-			if($content['update'])
+			if ($content['update'])
 			{
 				$applist = importexport_helper_functions::get_apps('all', true);
-				foreach($applist as $appname) {
+				foreach ($applist as $appname) {
 					importexport_helper_functions::load_defaults($appname);
 				}
 				return $this->index();
@@ -915,7 +914,7 @@ 
 		{
 			$content = array();
 			$etpl = new etemplate(self::_appname.'.import_definition');
-			return $etpl->exec(self::_appname.'.importexport_definitions_ui.import_definition',$content,array(),$readonlys,$preserv);
+			return $etpl->exec(self::_appname.'.importexport_definitions_ui.import_definition', $content, array(), $readonlys, $preserv);
 		}
 	}
 
@@ -925,17 +924,17 @@ 
 	 */
 	protected function can_edit(Array $definition)
 	{
-		if($definition['owner'] && $definition['owner'] == $GLOBALS['egw_info']['user']['account_id'])
+		if ($definition['owner'] && $definition['owner'] == $GLOBALS['egw_info']['user']['account_id'])
 		{
 			// Definition belongs to user
 			return true;
 		}
-		elseif($definition['definition_id'] && !$definition['owner'] && $GLOBALS['egw_info']['user']['apps']['admin'])
+		elseif ($definition['definition_id'] && !$definition['owner'] && $GLOBALS['egw_info']['user']['apps']['admin'])
 		{
 			// Definition is unowned, and user is an admin
 			return true;
 		}
-		elseif(!$definition['definition_id'])
+		elseif (!$definition['definition_id'])
 		{
 			// Definition is in-progress, not saved yet
 			return true;
@@ -948,26 +947,26 @@ 
 	 */
 	public function site_config($content = array())
 	{
-		if(!$GLOBALS['egw_info']['user']['apps']['admin'])
+		if (!$GLOBALS['egw_info']['user']['apps']['admin'])
 		{
 			Egw::redirect_link('/home');
 		}
-		if($content['save'])
+		if ($content['save'])
 		{
 			unset($content['save']);
 
 			// ACL
-			$GLOBALS['egw']->acl->delete_repository(self::_appname, 'definition',false);
-			$GLOBALS['egw']->acl->delete_repository(self::_appname, 'share_definition',false);
+			$GLOBALS['egw']->acl->delete_repository(self::_appname, 'definition', false);
+			$GLOBALS['egw']->acl->delete_repository(self::_appname, 'share_definition', false);
 
-			foreach($content['share_definition'] as $group)
+			foreach ($content['share_definition'] as $group)
 			{
-				$GLOBALS['egw']->acl->add_repository(self::_appname, 'share_definition', $group,Acl::READ);
+				$GLOBALS['egw']->acl->add_repository(self::_appname, 'share_definition', $group, Acl::READ);
 			}
 			unset($content['share_definition']);
 
 			// Other Api\Config
-			foreach($content as $key=>$value)
+			foreach ($content as $key=>$value)
 			{
 				Api\Config::save_value($key, $value, 'importexport');
 			}
@@ -977,18 +976,18 @@ 
 
 		$data = Api\Config::read(self::_appname);
 		$data['share_definition'] = $GLOBALS['egw']->acl->get_ids_for_location('share_definition', Acl::READ, self::_appname);
-		$sel_options['import_charsets'] = array_combine(mb_list_encodings(),mb_list_encodings());
+		$sel_options['import_charsets'] = array_combine(mb_list_encodings(), mb_list_encodings());
 
 		// Remove 'standard' encodings to prevent doubles
-		foreach(Api\Translation::get_installed_charsets() as $charset => $label)
+		foreach (Api\Translation::get_installed_charsets() as $charset => $label)
 		{
 			unset($sel_options['import_charsets'][strtoupper($charset)]);
 		}
 
-		if(!$data['update']) $data['update'] = 'request';
+		if (!$data['update']) $data['update'] = 'request';
 
-		$GLOBALS['egw_info']['flags']['app_header'] = lang('Site configuration') . ' - ' . lang(self::_appname);
+		$GLOBALS['egw_info']['flags']['app_header'] = lang('Site configuration').' - '.lang(self::_appname);
 		$etpl = new etemplate(self::_appname.'.config');
-		$etpl->exec(self::_appname.'.importexport_definitions_ui.site_config',$data,$sel_options,$readonlys,$preserv);
+		$etpl->exec(self::_appname.'.importexport_definitions_ui.site_config', $data, $sel_options, $readonlys, $preserv);
 	}
 }

importexport/inc/class.importexport_export_csv.inc.php

Doc Comments +3 added lines, -3 removed lines

@@ -96,7 +96,7 @@ 
 	 *
 	 * @param stram $_stream resource where records are exported to.
 	 * @param array _options options for specific backends
-	 * @return bool
+	 * @return boolean|null
 	 */
 	public function __construct( $_stream, array $_options ) {
 		if (!is_object($GLOBALS['egw']->translation)) {
@@ -151,7 +151,7 @@ 
 	 * exports a record into resource of handle
 	 *
 	 * @param importexport_iface_egw_record record
-	 * @return bool
+	 * @return boolean|null
 	 */
 	public function export_record( importexport_iface_egw_record $_record ) {
 		$this->record = $_record;
@@ -284,7 +284,7 @@ 
 	 *
 	 * Uses the static variable $types to convert various datatypes.
 	 *
-	 * @param record Record to be converted
+	 * @param record importexport_iface_egw_record to be converted
 	 * @parem fields List of field types => field names to be converted
 	 * @param appname Current appname if you want to do custom fields too
 	 */

Spacing +77 added lines, -77 removed lines

@@ -98,22 +98,22 @@ 
 	 * @param array _options options for specific backends
 	 * @return bool
 	 */
-	public function __construct( $_stream, array $_options ) {
+	public function __construct($_stream, array $_options) {
 		if (!is_object($GLOBALS['egw']->translation)) {
 			$GLOBALS['egw']->translation = new Api\Translation();
 		}
 		$this->translation = &$GLOBALS['egw']->translation;
 		$this->handle = $_stream;
-		if($_options['charset'] == 'user') $_options['charset'] = $GLOBALS['egw_info']['user']['preferences']['common']['csv_charset'];
+		if ($_options['charset'] == 'user') $_options['charset'] = $GLOBALS['egw_info']['user']['preferences']['common']['csv_charset'];
 		$this->csv_charset = $_options['charset'] ? $_options['charset'] : 'utf-8';
-		if ( !empty( $_options ) ) {
-			$this->csv_options = array_merge( $this->csv_options, $_options );
+		if (!empty($_options)) {
+			$this->csv_options = array_merge($this->csv_options, $_options);
 		}
 		//error_log(__METHOD__.__LINE__.array2string($_options['appname']));
-		if(!Api\Storage\Merge::is_export_limit_excepted()) {
+		if (!Api\Storage\Merge::is_export_limit_excepted()) {
 			$this->export_limit = Api\Storage\Merge::getExportLimit($_options['appname']);
 			//error_log(__METHOD__.__LINE__.' app:'.$_options['appname'].' limit:'.$this->export_limit);
-			if($this->export_limit == 'no') throw new Api\Exception\NoPermission\Admin('Export disabled');
+			if ($this->export_limit == 'no') throw new Api\Exception\NoPermission\Admin('Export disabled');
 		}
 	}
 	
@@ -122,15 +122,15 @@ 
 	 *
 	 * @param array $_mapping egw_field_name => csv_field_name
 	 */
-	public function set_mapping( array &$_mapping) {
+	public function set_mapping(array &$_mapping) {
 		if ($this->num_of_records > 0) {
 			throw new Exception('Error: Field mapping can\'t be set during ongoing export!');
 		}
-		if($_mapping['all_custom_fields']) {
+		if ($_mapping['all_custom_fields']) {
 			// Field value is the appname, so we can pull the fields
 			$custom = Api\Storage\Customfields::get($_mapping['all_custom_fields']);
 			unset($_mapping['all_custom_fields']);
-			foreach($custom as $field => $info) {
+			foreach ($custom as $field => $info) {
 				$_mapping['#'.$field] = $this->csv_options['begin_with_fieldnames'] == 'label' ? $info['label'] : $field;
 			}
 		}
@@ -143,7 +143,7 @@ 
 	 *
 	 * @param array $_conversion
 	 */
-	public function set_conversion( array $_conversion) {
+	public function set_conversion(array $_conversion) {
 		$this->conversion = $_conversion;
 	}
 	
@@ -153,59 +153,59 @@ 
 	 * @param importexport_iface_egw_record record
 	 * @return bool
 	 */
-	public function export_record( importexport_iface_egw_record $_record ) {
+	public function export_record(importexport_iface_egw_record $_record) {
 		$this->record = $_record;
 		$this->record_array = $_record->get_record_array();
 		
 		// begin with fieldnames ?
-		if ($this->num_of_records == 0 && $this->csv_options['begin_with_fieldnames'] ) {
-			if($this->csv_options['begin_with_fieldnames'] == 'label') {
+		if ($this->num_of_records == 0 && $this->csv_options['begin_with_fieldnames']) {
+			if ($this->csv_options['begin_with_fieldnames'] == 'label') {
 				// Load translations for app
 				list($appname, $part2) = explode('_', get_class($_record));
-				if(!$GLOBALS['egw_info']['apps'][$appname]) $appname .= $part2; // Handle apps with _ in the name
+				if (!$GLOBALS['egw_info']['apps'][$appname]) $appname .= $part2; // Handle apps with _ in the name
 
 				// Get translations from wizard, if possible
-				if(!$this->csv_options['no_header_translation'])
+				if (!$this->csv_options['no_header_translation'])
 				{
 					$backtrace = debug_backtrace();
 					$plugin = $backtrace[1]['class'];
-					$wizard_name = $appname . '_wizard_' . str_replace($appname . '_', '', $plugin);
+					$wizard_name = $appname.'_wizard_'.str_replace($appname.'_', '', $plugin);
 					try {
 						$wizard = new $wizard_name;
 						$fields = $wizard->get_export_fields();
-						foreach($this->mapping as $field => &$label)
+						foreach ($this->mapping as $field => &$label)
 						{
-							if($fields[$field])
+							if ($fields[$field])
 							{
 								$label = $label != $fields[$field] ? $fields[$field] : lang($label);
 							}
 							// Make sure no *
-							if(substr($label,-1) == '*') $label = substr($label,0,-1);
+							if (substr($label, -1) == '*') $label = substr($label, 0, -1);
 						}
 					} catch (Exception $e) {
 						Api\Translation::add_app($appname);
-						foreach($this->mapping as $field => &$label) {
+						foreach ($this->mapping as $field => &$label) {
 							$label = lang($label);
 						}
 					}
 				}
 			}
-			$mapping = ! empty( $this->mapping ) ? $this->mapping : array_keys ( $this->record_array );
-			self::fputcsv( $this->handle ,$mapping ,$this->csv_options['delimiter'], $this->csv_options['enclosure'] );
+			$mapping = !empty($this->mapping) ? $this->mapping : array_keys($this->record_array);
+			self::fputcsv($this->handle, $mapping, $this->csv_options['delimiter'], $this->csv_options['enclosure']);
 		}
 
 		// Check for limit
-		if($this->export_limit && $this->num_of_records >= $this->export_limit) {
+		if ($this->export_limit && $this->num_of_records >= $this->export_limit) {
 			return;
 		}
 		
 		// do conversions
-		if ( !empty( $this->conversion )) {
-			$this->record_array = importexport_helper_functions::conversion( $this->record_array, $this->conversion );
+		if (!empty($this->conversion)) {
+			$this->record_array = importexport_helper_functions::conversion($this->record_array, $this->conversion);
 		}
 		
 		// do fieldmapping
-		if ( !empty( $this->mapping ) ) {
+		if (!empty($this->mapping)) {
 			$record_data = $this->record_array;
 			$this->record_array = array();
 			foreach ($this->mapping as $egw_field => $csv_field) {
@@ -213,7 +213,7 @@ 
 			}
 		}
 		
-		self::fputcsv( $this->handle, $this->record_array, $this->csv_options['delimiter'], $this->csv_options['enclosure'] );
+		self::fputcsv($this->handle, $this->record_array, $this->csv_options['delimiter'], $this->csv_options['enclosure']);
 		$this->num_of_records++;
 	}
 
@@ -237,13 +237,13 @@ 
 	 * @return Array of fields to be added to list of fields needing conversion
 	 */
 	public static function convert_parse_custom_fields($appname, &$selects = array(), &$links = array(), &$methods = array()) {
-		if(!$appname) return;
+		if (!$appname) return;
 
 		$fields = array();
 		$custom = Api\Storage\Customfields::get($appname);
-		foreach($custom as $name => $c_field) {
-			$name = '#' . $name;
-			switch($c_field['type']) {
+		foreach ($custom as $name => $c_field) {
+			$name = '#'.$name;
+			switch ($c_field['type']) {
 				case 'date':
 					$fields['date'][] = $name;
 					break;
@@ -254,7 +254,7 @@ 
 					$fields['select-account'][] = $name;
 					break;
 				case 'ajax_select':
-					if($c_field['values']['get_title']) {
+					if ($c_field['values']['get_title']) {
 						$methods[$name] = $c_field['values']['get_title'];
 						break;
 					}
@@ -268,8 +268,8 @@ 
 					$selects[$name] = $c_field['values'];
 					break;
 				default:
-					list($type) = explode('-',$c_field['type'],2);
-					if(in_array($type, array_keys($GLOBALS['egw_info']['apps']))) {
+					list($type) = explode('-', $c_field['type'], 2);
+					if (in_array($type, array_keys($GLOBALS['egw_info']['apps']))) {
 						$fields['links'][] = $name;
 						$links[$name] = $c_field['type'];
 					}
@@ -288,18 +288,18 @@ 
 	 * @parem fields List of field types => field names to be converted
 	 * @param appname Current appname if you want to do custom fields too
 	 */
-	public static function convert(importexport_iface_egw_record &$record, Array $fields = array(), $appname = null, $selects = array()) {
-		if($appname) {
-			if(!self::$cf_parse_cache[$appname]) {
+	public static function convert(importexport_iface_egw_record&$record, Array $fields = array(), $appname = null, $selects = array()) {
+		if ($appname) {
+			if (!self::$cf_parse_cache[$appname]) {
 				$c_fields = self::convert_parse_custom_fields($appname, $selects, $links, $methods);
 				self::$cf_parse_cache[$appname] = array($c_fields, $selects, $links, $methods);
 			}
 			list($c_fields, $c_selects, $links, $methods) = self::$cf_parse_cache[$appname];
 
 			// Add in any fields that are keys to another app
-			foreach((array)$fields['links'] as $link_field => $app)
+			foreach ((array)$fields['links'] as $link_field => $app)
 			{
-				if(is_numeric($link_field)) continue;
+				if (is_numeric($link_field)) continue;
 				$links[$link_field] = $app;
 				// Set it as a normal link field
 				$fields['links'][] = $link_field;
@@ -307,8 +307,8 @@ 
 			}
 
 			// Not quite a recursive merge, since only one level
-			foreach($fields as $type => &$list) {
-				if($c_fields[$type]) {
+			foreach ($fields as $type => &$list) {
+				if ($c_fields[$type]) {
 					$list = array_merge($c_fields[$type], $list);
 					unset($c_fields[$type]);
 				}
@@ -316,12 +316,12 @@ 
 			$fields += $c_fields;
 			$selects += $c_selects;
 		}
-		foreach((array)$fields['select'] as $name) {
-			if($record->$name != null && is_array($selects) && $selects[$name]) {
+		foreach ((array)$fields['select'] as $name) {
+			if ($record->$name != null && is_array($selects) && $selects[$name]) {
 				$record->$name = explode(',', $record->$name);
-				if(is_array($record->$name)) {
+				if (is_array($record->$name)) {
 					$names = array();
-					foreach($record->$name as $_name) {
+					foreach ($record->$name as $_name) {
 						$names[] = lang($selects[$name][$_name]);
 					}
 					$record->$name = implode(', ', $names);
@@ -334,14 +334,14 @@ 
 				$record->$name = '';
 			}
 		}
-		foreach((array)$fields['links'] as $name) {
-			if($record->$name) {
-				if(is_numeric($record->$name) && !$links[$name]) {
+		foreach ((array)$fields['links'] as $name) {
+			if ($record->$name) {
+				if (is_numeric($record->$name) && !$links[$name]) {
 					$link = Link::get_link($record->$name);
 					$links[$name] = ($link['link_app1'] == $appname ? $link['link_app2'] : $link['link_app1']);
 					$record->$name = ($link['link_app1'] == $appname ? $link['link_id2'] : $link['link_id1']);
 				}
-				if($links[$name]) {
+				if ($links[$name]) {
 					$record->$name = Link::title($links[$name], $record->$name);
 				}
 			}
@@ -350,12 +350,12 @@ 
 				$record->$name = '';
 			}
 		}
-		foreach((array)$fields['select-account'] as $name) {
+		foreach ((array)$fields['select-account'] as $name) {
 			// Compare against null to deal with empty arrays
 			if ($record->$name !== null) {
-				if(is_array($record->$name)) {
+				if (is_array($record->$name)) {
 					$names = array();
-					foreach($record->$name as $_name) {
+					foreach ($record->$name as $_name) {
 						$names[] = Api\Accounts::username($_name);
 					}
 					$record->$name = implode(', ', $names);
@@ -368,64 +368,64 @@ 
 				$record->$name = '';
 			}
 		}
-		foreach((array)$fields['select-bool'] as $name) {
-			if($record->$name !== null) {
+		foreach ((array)$fields['select-bool'] as $name) {
+			if ($record->$name !== null) {
 				$record->$name = $record->$name ? lang('Yes') : lang('No');
 			}
 		}
-		foreach((array)$fields['date-time'] as $name) {
+		foreach ((array)$fields['date-time'] as $name) {
 			//if ($record->$name) $record->$name = date('Y-m-d H:i:s',$record->$name); // Standard date format
 			if ($record->$name && !is_numeric($record->$name)) $record->$name = strtotime($record->$name); // Custom fields stored as string
-			if ($record->$name && is_numeric($record->$name)) $record->$name = date($GLOBALS['egw_info']['user']['preferences']['common']['dateformat'] . ' '.
-				($GLOBALS['egw_info']['user']['preferences']['common']['timeformat'] == '24' ? 'H:i:s' : 'h:i:s a'),$record->$name); // User date format
+			if ($record->$name && is_numeric($record->$name)) $record->$name = date($GLOBALS['egw_info']['user']['preferences']['common']['dateformat'].' '.
+				($GLOBALS['egw_info']['user']['preferences']['common']['timeformat'] == '24' ? 'H:i:s' : 'h:i:s a'), $record->$name); // User date format
 			if (!$record->$name) $record->$name = '';
 		}
-		foreach((array)$fields['date'] as $name) {
+		foreach ((array)$fields['date'] as $name) {
 			//if ($record->$name) $record->$name = date('Y-m-d',$record->$name); // Standard date format
 			if ($record->$name && !is_numeric($record->$name)) $record->$name = strtotime($record->$name); // Custom fields stored as string
 			if ($record->$name && is_numeric($record->$name)) $record->$name = date($GLOBALS['egw_info']['user']['preferences']['common']['dateformat'], $record->$name); // User date format
 			if (!$record->$name) $record->$name = '';
 		}
-		foreach((array)$fields['float'] as $name)
+		foreach ((array)$fields['float'] as $name)
 		{
-			static $dec_separator,$thousands_separator;
+			static $dec_separator, $thousands_separator;
 			if (is_null($dec_separator))
 			{
 				$dec_separator = $GLOBALS['egw_info']['user']['preferences']['common']['number_format'][0];
 				if (empty($dec_separator)) $dec_separator = '.';
 				$thousands_separator = $GLOBALS['egw_info']['user']['preferences']['common']['number_format'][1];
 			}
-			if($record->$name && (string)$record->$name != '')
+			if ($record->$name && (string)$record->$name != '')
 			{
-				if(!is_numeric($record->$name))
+				if (!is_numeric($record->$name))
 				{
 					$record->$name = floatval(str_replace($dec_separator, '.', preg_replace('/[^\d'.preg_quote($dec_separator).']/', '', $record->$name)));
 				}
-				$record->$name = number_format(str_replace(' ','',$record->$name), 2,
-					$dec_separator,$thousands_separator
+				$record->$name = number_format(str_replace(' ', '', $record->$name), 2,
+					$dec_separator, $thousands_separator
 				);
 			}
 		}
 
 		// Some custom methods for conversion
-		foreach((array)$methods as $name => $method) {
-			if($record->$name) $record->$name = ExecMethod($method, $record->$name);
+		foreach ((array)$methods as $name => $method) {
+			if ($record->$name) $record->$name = ExecMethod($method, $record->$name);
 		}
 
 		static $cat_object;
-		if(is_null($cat_object)) $cat_object = new Api\Categories(false,$appname);
-		foreach((array)$fields['select-cat'] as $name) {
-			if($record->$name) {
+		if (is_null($cat_object)) $cat_object = new Api\Categories(false, $appname);
+		foreach ((array)$fields['select-cat'] as $name) {
+			if ($record->$name) {
 				$cats = array();
 				$ids = is_array($record->$name) ? $record->$name : explode(',', $record->$name);
-				foreach($ids as $n => $cat_id) {
+				foreach ($ids as $n => $cat_id) {
 
-					if ($cat_id && $cat_object->check_perms(Acl::READ,$cat_id))
+					if ($cat_id && $cat_object->check_perms(Acl::READ, $cat_id))
 					{
 						$cats[] = $cat_object->id2name($cat_id);
 					}
 				}
-				$record->$name = implode(', ',$cats);
+				$record->$name = implode(', ', $cats);
 			}
 			else
 			{
@@ -452,18 +452,18 @@ 
 	 * @param char $delimiter
 	 * @param char $enclosure
 	 */
-	protected function fputcsv($filePointer, Array $dataArray, $delimiter, $enclosure){
+	protected function fputcsv($filePointer, Array $dataArray, $delimiter, $enclosure) {
 		$string = "";
 		$writeDelimiter = false;
-		foreach($dataArray as $dataElement) {
-			if($writeDelimiter) $string .= $delimiter;
-			$string .= $enclosure . str_replace(array("\r\n", '"'), array("\n",'""'), $dataElement) . $enclosure;
+		foreach ($dataArray as $dataElement) {
+			if ($writeDelimiter) $string .= $delimiter;
+			$string .= $enclosure.str_replace(array("\r\n", '"'), array("\n", '""'), $dataElement).$enclosure;
 			$writeDelimiter = true;
 		}
 		$string .= "\n";
 		
 		// do charset translation
-		$string = $this->translation->convert( $string, $this->translation->charset(), $this->csv_charset );
+		$string = $this->translation->convert($string, $this->translation->charset(), $this->csv_charset);
 		
 		fwrite($filePointer, $string);
 			

Braces +180 added lines, -71 removed lines

@@ -98,22 +98,32 @@ 
 	 * @param array _options options for specific backends
 	 * @return bool
 	 */
-	public function __construct( $_stream, array $_options ) {
-		if (!is_object($GLOBALS['egw']->translation)) {
+	public function __construct( $_stream, array $_options )
+	{
+		if (!is_object($GLOBALS['egw']->translation))
+		{
 			$GLOBALS['egw']->translation = new Api\Translation();
 		}
 		$this->translation = &$GLOBALS['egw']->translation;
 		$this->handle = $_stream;
-		if($_options['charset'] == 'user') $_options['charset'] = $GLOBALS['egw_info']['user']['preferences']['common']['csv_charset'];
+		if($_options['charset'] == 'user')
+		{
+			$_options['charset'] = $GLOBALS['egw_info']['user']['preferences']['common']['csv_charset'];
+		}
 		$this->csv_charset = $_options['charset'] ? $_options['charset'] : 'utf-8';
-		if ( !empty( $_options ) ) {
+		if ( !empty( $_options ) )
+		{
 			$this->csv_options = array_merge( $this->csv_options, $_options );
 		}
 		//error_log(__METHOD__.__LINE__.array2string($_options['appname']));
-		if(!Api\Storage\Merge::is_export_limit_excepted()) {
+		if(!Api\Storage\Merge::is_export_limit_excepted())
+		{
 			$this->export_limit = Api\Storage\Merge::getExportLimit($_options['appname']);
 			//error_log(__METHOD__.__LINE__.' app:'.$_options['appname'].' limit:'.$this->export_limit);
-			if($this->export_limit == 'no') throw new Api\Exception\NoPermission\Admin('Export disabled');
+			if($this->export_limit == 'no')
+			{
+				throw new Api\Exception\NoPermission\Admin('Export disabled');
+			}
 		}
 	}
 	
@@ -122,15 +132,19 @@ 
 	 *
 	 * @param array $_mapping egw_field_name => csv_field_name
 	 */
-	public function set_mapping( array &$_mapping) {
-		if ($this->num_of_records > 0) {
+	public function set_mapping( array &$_mapping)
+	{
+		if ($this->num_of_records > 0)
+		{
 			throw new Exception('Error: Field mapping can\'t be set during ongoing export!');
 		}
-		if($_mapping['all_custom_fields']) {
+		if($_mapping['all_custom_fields'])
+		{
 			// Field value is the appname, so we can pull the fields
 			$custom = Api\Storage\Customfields::get($_mapping['all_custom_fields']);
 			unset($_mapping['all_custom_fields']);
-			foreach($custom as $field => $info) {
+			foreach($custom as $field => $info)
+			{
 				$_mapping['#'.$field] = $this->csv_options['begin_with_fieldnames'] == 'label' ? $info['label'] : $field;
 			}
 		}
@@ -143,7 +157,8 @@ 
 	 *
 	 * @param array $_conversion
 	 */
-	public function set_conversion( array $_conversion) {
+	public function set_conversion( array $_conversion)
+	{
 		$this->conversion = $_conversion;
 	}
 	
@@ -153,16 +168,23 @@ 
 	 * @param importexport_iface_egw_record record
 	 * @return bool
 	 */
-	public function export_record( importexport_iface_egw_record $_record ) {
+	public function export_record( importexport_iface_egw_record $_record )
+	{
 		$this->record = $_record;
 		$this->record_array = $_record->get_record_array();
 		
 		// begin with fieldnames ?
-		if ($this->num_of_records == 0 && $this->csv_options['begin_with_fieldnames'] ) {
-			if($this->csv_options['begin_with_fieldnames'] == 'label') {
+		if ($this->num_of_records == 0 && $this->csv_options['begin_with_fieldnames'] )
+		{
+			if($this->csv_options['begin_with_fieldnames'] == 'label')
+			{
 				// Load translations for app
 				list($appname, $part2) = explode('_', get_class($_record));
-				if(!$GLOBALS['egw_info']['apps'][$appname]) $appname .= $part2; // Handle apps with _ in the name
+				if(!$GLOBALS['egw_info']['apps'][$appname])
+				{
+					$appname .= $part2;
+				}
+				// Handle apps with _ in the name
 
 				// Get translations from wizard, if possible
 				if(!$this->csv_options['no_header_translation'])
@@ -180,11 +202,16 @@ 
 								$label = $label != $fields[$field] ? $fields[$field] : lang($label);
 							}
 							// Make sure no *
-							if(substr($label,-1) == '*') $label = substr($label,0,-1);
+							if(substr($label,-1) == '*')
+							{
+								$label = substr($label,0,-1);
+							}
 						}
-					} catch (Exception $e) {
+					}
+					catch (Exception $e) {
 						Api\Translation::add_app($appname);
-						foreach($this->mapping as $field => &$label) {
+						foreach($this->mapping as $field => &$label)
+						{
 							$label = lang($label);
 						}
 					}
@@ -195,20 +222,24 @@ 
 		}
 
 		// Check for limit
-		if($this->export_limit && $this->num_of_records >= $this->export_limit) {
+		if($this->export_limit && $this->num_of_records >= $this->export_limit)
+		{
 			return;
 		}
 		
 		// do conversions
-		if ( !empty( $this->conversion )) {
+		if ( !empty( $this->conversion ))
+		{
 			$this->record_array = importexport_helper_functions::conversion( $this->record_array, $this->conversion );
 		}
 		
 		// do fieldmapping
-		if ( !empty( $this->mapping ) ) {
+		if ( !empty( $this->mapping ) )
+		{
 			$record_data = $this->record_array;
 			$this->record_array = array();
-			foreach ($this->mapping as $egw_field => $csv_field) {
+			foreach ($this->mapping as $egw_field => $csv_field)
+			{
 				$this->record_array[$csv_field] = $record_data[$egw_field];
 			}
 		}
@@ -222,7 +253,8 @@ 
 	 *
 	 * @return int
 	 */
-	public function get_num_of_records() {
+	public function get_num_of_records()
+	{
 		return $this->num_of_records;
 	}
 
@@ -236,14 +268,20 @@ 
 	 *
 	 * @return Array of fields to be added to list of fields needing conversion
 	 */
-	public static function convert_parse_custom_fields($appname, &$selects = array(), &$links = array(), &$methods = array()) {
-		if(!$appname) return;
+	public static function convert_parse_custom_fields($appname, &$selects = array(), &$links = array(), &$methods = array())
+	{
+		if(!$appname)
+		{
+			return;
+		}
 
 		$fields = array();
 		$custom = Api\Storage\Customfields::get($appname);
-		foreach($custom as $name => $c_field) {
+		foreach($custom as $name => $c_field)
+		{
 			$name = '#' . $name;
-			switch($c_field['type']) {
+			switch($c_field['type'])
+			{
 				case 'date':
 					$fields['date'][] = $name;
 					break;
@@ -254,7 +292,8 @@ 
 					$fields['select-account'][] = $name;
 					break;
 				case 'ajax_select':
-					if($c_field['values']['get_title']) {
+					if($c_field['values']['get_title'])
+					{
 						$methods[$name] = $c_field['values']['get_title'];
 						break;
 					}
@@ -269,7 +308,8 @@ 
 					break;
 				default:
 					list($type) = explode('-',$c_field['type'],2);
-					if(in_array($type, array_keys($GLOBALS['egw_info']['apps']))) {
+					if(in_array($type, array_keys($GLOBALS['egw_info']['apps'])))
+					{
 						$fields['links'][] = $name;
 						$links[$name] = $c_field['type'];
 					}
@@ -288,9 +328,12 @@ 
 	 * @parem fields List of field types => field names to be converted
 	 * @param appname Current appname if you want to do custom fields too
 	 */
-	public static function convert(importexport_iface_egw_record &$record, Array $fields = array(), $appname = null, $selects = array()) {
-		if($appname) {
-			if(!self::$cf_parse_cache[$appname]) {
+	public static function convert(importexport_iface_egw_record &$record, Array $fields = array(), $appname = null, $selects = array())
+	{
+		if($appname)
+		{
+			if(!self::$cf_parse_cache[$appname])
+			{
 				$c_fields = self::convert_parse_custom_fields($appname, $selects, $links, $methods);
 				self::$cf_parse_cache[$appname] = array($c_fields, $selects, $links, $methods);
 			}
@@ -299,7 +342,10 @@ 
 			// Add in any fields that are keys to another app
 			foreach((array)$fields['links'] as $link_field => $app)
 			{
-				if(is_numeric($link_field)) continue;
+				if(is_numeric($link_field))
+				{
+					continue;
+				}
 				$links[$link_field] = $app;
 				// Set it as a normal link field
 				$fields['links'][] = $link_field;
@@ -307,8 +353,10 @@ 
 			}
 
 			// Not quite a recursive merge, since only one level
-			foreach($fields as $type => &$list) {
-				if($c_fields[$type]) {
+			foreach($fields as $type => &$list)
+			{
+				if($c_fields[$type])
+				{
 					$list = array_merge($c_fields[$type], $list);
 					unset($c_fields[$type]);
 				}
@@ -316,16 +364,22 @@ 
 			$fields += $c_fields;
 			$selects += $c_selects;
 		}
-		foreach((array)$fields['select'] as $name) {
-			if($record->$name != null && is_array($selects) && $selects[$name]) {
+		foreach((array)$fields['select'] as $name)
+		{
+			if($record->$name != null && is_array($selects) && $selects[$name])
+			{
 				$record->$name = explode(',', $record->$name);
-				if(is_array($record->$name)) {
+				if(is_array($record->$name))
+				{
 					$names = array();
-					foreach($record->$name as $_name) {
+					foreach($record->$name as $_name)
+					{
 						$names[] = lang($selects[$name][$_name]);
 					}
 					$record->$name = implode(', ', $names);
-				} else {
+				}
+				else
+				{
 					$record->$name = lang($selects[$name][$record->$name]);
 				}
 			}
@@ -334,14 +388,18 @@ 
 				$record->$name = '';
 			}
 		}
-		foreach((array)$fields['links'] as $name) {
-			if($record->$name) {
-				if(is_numeric($record->$name) && !$links[$name]) {
+		foreach((array)$fields['links'] as $name)
+		{
+			if($record->$name)
+			{
+				if(is_numeric($record->$name) && !$links[$name])
+				{
 					$link = Link::get_link($record->$name);
 					$links[$name] = ($link['link_app1'] == $appname ? $link['link_app2'] : $link['link_app1']);
 					$record->$name = ($link['link_app1'] == $appname ? $link['link_id2'] : $link['link_id1']);
 				}
-				if($links[$name]) {
+				if($links[$name])
+				{
 					$record->$name = Link::title($links[$name], $record->$name);
 				}
 			}
@@ -350,16 +408,22 @@ 
 				$record->$name = '';
 			}
 		}
-		foreach((array)$fields['select-account'] as $name) {
+		foreach((array)$fields['select-account'] as $name)
+		{
 			// Compare against null to deal with empty arrays
-			if ($record->$name !== null) {
-				if(is_array($record->$name)) {
+			if ($record->$name !== null)
+			{
+				if(is_array($record->$name))
+				{
 					$names = array();
-					foreach($record->$name as $_name) {
+					foreach($record->$name as $_name)
+					{
 						$names[] = Api\Accounts::username($_name);
 					}
 					$record->$name = implode(', ', $names);
-				} else {
+				}
+				else
+				{
 					$record->$name = Api\Accounts::username($record->$name);
 				}
 			}
@@ -368,23 +432,49 @@ 
 				$record->$name = '';
 			}
 		}
-		foreach((array)$fields['select-bool'] as $name) {
-			if($record->$name !== null) {
+		foreach((array)$fields['select-bool'] as $name)
+		{
+			if($record->$name !== null)
+			{
 				$record->$name = $record->$name ? lang('Yes') : lang('No');
 			}
 		}
-		foreach((array)$fields['date-time'] as $name) {
+		foreach((array)$fields['date-time'] as $name)
+		{
 			//if ($record->$name) $record->$name = date('Y-m-d H:i:s',$record->$name); // Standard date format
-			if ($record->$name && !is_numeric($record->$name)) $record->$name = strtotime($record->$name); // Custom fields stored as string
-			if ($record->$name && is_numeric($record->$name)) $record->$name = date($GLOBALS['egw_info']['user']['preferences']['common']['dateformat'] . ' '.
-				($GLOBALS['egw_info']['user']['preferences']['common']['timeformat'] == '24' ? 'H:i:s' : 'h:i:s a'),$record->$name); // User date format
-			if (!$record->$name) $record->$name = '';
+			if ($record->$name && !is_numeric($record->$name))
+			{
+				$record->$name = strtotime($record->$name);
+			}
+			// Custom fields stored as string
+			if ($record->$name && is_numeric($record->$name))
+			{
+				$record->$name = date($GLOBALS['egw_info']['user']['preferences']['common']['dateformat'] . ' '.
+				($GLOBALS['egw_info']['user']['preferences']['common']['timeformat'] == '24' ? 'H:i:s' : 'h:i:s a'),$record->$name);
+			}
+			// User date format
+			if (!$record->$name)
+			{
+				$record->$name = '';
+			}
 		}
-		foreach((array)$fields['date'] as $name) {
+		foreach((array)$fields['date'] as $name)
+		{
 			//if ($record->$name) $record->$name = date('Y-m-d',$record->$name); // Standard date format
-			if ($record->$name && !is_numeric($record->$name)) $record->$name = strtotime($record->$name); // Custom fields stored as string
-			if ($record->$name && is_numeric($record->$name)) $record->$name = date($GLOBALS['egw_info']['user']['preferences']['common']['dateformat'], $record->$name); // User date format
-			if (!$record->$name) $record->$name = '';
+			if ($record->$name && !is_numeric($record->$name))
+			{
+				$record->$name = strtotime($record->$name);
+			}
+			// Custom fields stored as string
+			if ($record->$name && is_numeric($record->$name))
+			{
+				$record->$name = date($GLOBALS['egw_info']['user']['preferences']['common']['dateformat'], $record->$name);
+			}
+			// User date format
+			if (!$record->$name)
+			{
+				$record->$name = '';
+			}
 		}
 		foreach((array)$fields['float'] as $name)
 		{
@@ -392,7 +482,10 @@ 
 			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($record->$name && (string)$record->$name != '')
@@ -408,17 +501,27 @@ 
 		}
 
 		// Some custom methods for conversion
-		foreach((array)$methods as $name => $method) {
-			if($record->$name) $record->$name = ExecMethod($method, $record->$name);
+		foreach((array)$methods as $name => $method)
+		{
+			if($record->$name)
+			{
+				$record->$name = ExecMethod($method, $record->$name);
+			}
 		}
 
 		static $cat_object;
-		if(is_null($cat_object)) $cat_object = new Api\Categories(false,$appname);
-		foreach((array)$fields['select-cat'] as $name) {
-			if($record->$name) {
+		if(is_null($cat_object))
+		{
+			$cat_object = new Api\Categories(false,$appname);
+		}
+		foreach((array)$fields['select-cat'] as $name)
+		{
+			if($record->$name)
+			{
 				$cats = array();
 				$ids = is_array($record->$name) ? $record->$name : explode(',', $record->$name);
-				foreach($ids as $n => $cat_id) {
+				foreach($ids as $n => $cat_id)
+				{
 
 					if ($cat_id && $cat_object->check_perms(Acl::READ,$cat_id))
 					{
@@ -439,7 +542,8 @@ 
 	 *
 	 * @return
 	 */
-	public function __destruct() {
+	public function __destruct()
+	{
 		
 	}
 
@@ -452,11 +556,16 @@ 
 	 * @param char $delimiter
 	 * @param char $enclosure
 	 */
-	protected function fputcsv($filePointer, Array $dataArray, $delimiter, $enclosure){
+	protected function fputcsv($filePointer, Array $dataArray, $delimiter, $enclosure)
+	{
 		$string = "";
 		$writeDelimiter = false;
-		foreach($dataArray as $dataElement) {
-			if($writeDelimiter) $string .= $delimiter;
+		foreach($dataArray as $dataElement)
+		{
+			if($writeDelimiter)
+			{
+				$string .= $delimiter;
+			}
 			$string .= $enclosure . str_replace(array("\r\n", '"'), array("\n",'""'), $dataElement) . $enclosure;
 			$writeDelimiter = true;
 		}

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

@@ -288,7 +288,7 @@ 
 	 * @parem fields List of field types => field names to be converted
 	 * @param appname Current appname if you want to do custom fields too
 	 */
-	public static function convert(importexport_iface_egw_record &$record, Array $fields = array(), $appname = null, $selects = array()) {
+	public static function convert(importexport_iface_egw_record &$record, array $fields = array(), $appname = null, $selects = array()) {
 		if($appname) {
 			if(!self::$cf_parse_cache[$appname]) {
 				$c_fields = self::convert_parse_custom_fields($appname, $selects, $links, $methods);
@@ -452,7 +452,7 @@ 
 	 * @param char $delimiter
 	 * @param char $enclosure
 	 */
-	protected function fputcsv($filePointer, Array $dataArray, $delimiter, $enclosure){
+	protected function fputcsv($filePointer, array $dataArray, $delimiter, $enclosure){
 		$string = "";
 		$writeDelimiter = false;
 		foreach($dataArray as $dataElement) {

importexport/inc/class.importexport_helper_functions.inc.php

Doc Comments +4 added lines, -3 removed lines

@@ -123,7 +123,7 @@ 
 	/**
 	 * converts accound_lid to account_id
 	 *
-	 * @param mixed $_account_lid comma seperated list or array with lids
+	 * @param mixed $_account_lids comma seperated list or array with lids
 	 * @return mixed comma seperated list or array with ids
 	 */
 	public static function account_name2id( &$_account_lids ) {
@@ -203,7 +203,7 @@ 
 	/**
 	 * converts account_ids to account_lids
 	 *
-	 * @param mixed $_account_ids comma seperated list or array with ids
+	 * @param mixed $_account_id comma seperated list or array with ids
 	 * @return mixed comma seperated list or array with lids
 	 */
 	public static function account_id2name( $_account_id ) {
@@ -419,7 +419,7 @@ 
 	/**
 	 * returns a list of importexport plugins
 	 *
-	 * @param string $_tpye {import | export | all}
+	 * @param string $_type {import | export | all}
 	 * @param string $_appname {<appname> | all}
 	 * @return array(<appname> => array( <type> => array(<plugin> => <title>)))
 	 */
@@ -591,6 +591,7 @@ 
 	 *
 	 * @param $app_name String name of app
 	 * @param $plugin_name Name of the plugin
+	 * @param importexport_wizard_basic_export_csv $wizard_plugin
 	 *
 	 * @return Array ([fieldname] => array(widget settings), ...)
 	 */

Indentation +21 added lines, -21 removed lines

@@ -32,25 +32,25 @@ 
 	 * Relative date ranges for filtering
 	 */
 	public static $relative_dates = array(      // Start: year,month,day,week, End: year,month,day,week
-                'Today'       => array(0,0,0,0,  0,0,1,0),
-                'Yesterday'   => array(0,0,-1,0, 0,0,0,0),
-                'This week'   => array(0,0,0,0,  0,0,0,1),
-                'Last week'   => array(0,0,0,-1, 0,0,0,0),
-                'This month'  => array(0,0,0,0,  0,1,0,0),
-                'Last month'  => array(0,-1,0,0, 0,0,0,0),
-                'Last 3 months' => array(0,-3,0,0, 0,0,0,0),
-                'This quarter'=> array(0,0,0,0,  0,0,0,0),      // Just a marker, needs special handling
-                'Last quarter'=> array(0,-4,0,0, 0,-4,0,0),     // Just a marker
-                'This year'   => array(0,0,0,0,  1,0,0,0),
-                'Last year'   => array(-1,0,0,0, 0,0,0,0),
-                '2 years ago' => array(-2,0,0,0, -1,0,0,0),
-                '3 years ago' => array(-3,0,0,0, -2,0,0,0),
-        );
+				'Today'       => array(0,0,0,0,  0,0,1,0),
+				'Yesterday'   => array(0,0,-1,0, 0,0,0,0),
+				'This week'   => array(0,0,0,0,  0,0,0,1),
+				'Last week'   => array(0,0,0,-1, 0,0,0,0),
+				'This month'  => array(0,0,0,0,  0,1,0,0),
+				'Last month'  => array(0,-1,0,0, 0,0,0,0),
+				'Last 3 months' => array(0,-3,0,0, 0,0,0,0),
+				'This quarter'=> array(0,0,0,0,  0,0,0,0),      // Just a marker, needs special handling
+				'Last quarter'=> array(0,-4,0,0, 0,-4,0,0),     // Just a marker
+				'This year'   => array(0,0,0,0,  1,0,0,0),
+				'Last year'   => array(-1,0,0,0, 0,0,0,0),
+				'2 years ago' => array(-2,0,0,0, -1,0,0,0),
+				'3 years ago' => array(-3,0,0,0, -2,0,0,0),
+		);
 
 	/**
-	* Files known to cause problems, and will be skipped in a plugin scan
-	* If you put appname => true, the whole app will be skipped.
-	*/
+	 * Files known to cause problems, and will be skipped in a plugin scan
+	 * If you put appname => true, the whole app will be skipped.
+	 */
 	protected static $blacklist_files = array(
 		'api' => true,
 		'etemplate' => true,
@@ -64,10 +64,10 @@ 
 	);
 
 	/**
-	* Class used to provide extra conversion functions
-	*
-	* Passed in as a param to conversion()
-	*/
+	 * Class used to provide extra conversion functions
+	 *
+	 * Passed in as a param to conversion()
+	 */
 	protected static $cclass = null;
 
 	/**

Spacing +185 added lines, -185 removed lines

@@ -32,19 +32,19 @@ 
 	 * Relative date ranges for filtering
 	 */
 	public static $relative_dates = array(      // Start: year,month,day,week, End: year,month,day,week
-                'Today'       => array(0,0,0,0,  0,0,1,0),
-                'Yesterday'   => array(0,0,-1,0, 0,0,0,0),
-                'This week'   => array(0,0,0,0,  0,0,0,1),
-                'Last week'   => array(0,0,0,-1, 0,0,0,0),
-                'This month'  => array(0,0,0,0,  0,1,0,0),
-                'Last month'  => array(0,-1,0,0, 0,0,0,0),
-                'Last 3 months' => array(0,-3,0,0, 0,0,0,0),
-                'This quarter'=> array(0,0,0,0,  0,0,0,0),      // Just a marker, needs special handling
-                'Last quarter'=> array(0,-4,0,0, 0,-4,0,0),     // Just a marker
-                'This year'   => array(0,0,0,0,  1,0,0,0),
-                'Last year'   => array(-1,0,0,0, 0,0,0,0),
-                '2 years ago' => array(-2,0,0,0, -1,0,0,0),
-                '3 years ago' => array(-3,0,0,0, -2,0,0,0),
+                'Today'       => array(0, 0, 0, 0, 0, 0, 1, 0),
+                'Yesterday'   => array(0, 0, -1, 0, 0, 0, 0, 0),
+                'This week'   => array(0, 0, 0, 0, 0, 0, 0, 1),
+                'Last week'   => array(0, 0, 0, -1, 0, 0, 0, 0),
+                'This month'  => array(0, 0, 0, 0, 0, 1, 0, 0),
+                'Last month'  => array(0, -1, 0, 0, 0, 0, 0, 0),
+                'Last 3 months' => array(0, -3, 0, 0, 0, 0, 0, 0),
+                'This quarter'=> array(0, 0, 0, 0, 0, 0, 0, 0), // Just a marker, needs special handling
+                'Last quarter'=> array(0, -4, 0, 0, 0, -4, 0, 0), // Just a marker
+                'This year'   => array(0, 0, 0, 0, 1, 0, 0, 0),
+                'Last year'   => array(-1, 0, 0, 0, 0, 0, 0, 0),
+                '2 years ago' => array(-2, 0, 0, 0, -1, 0, 0, 0),
+                '3 years ago' => array(-3, 0, 0, 0, -2, 0, 0, 0),
         );
 
 	/**
@@ -85,22 +85,22 @@ 
 	 * @param string $_format format of time string e.g.: d.m.Y H:i
 	 * @param int $_is_dst is day light saving time? 0 = no, 1 = yes, -1 = system default
 	 */
-	public static function custom_strtotime( $_string, $_format='', $_is_dst = -1) {
-		if ( empty( $_format ) ) return strtotime( $_string );
-		$fparams = explode( ',', chunk_split( $_format, 1, ',' ) );
+	public static function custom_strtotime($_string, $_format = '', $_is_dst = -1) {
+		if (empty($_format)) return strtotime($_string);
+		$fparams = explode(',', chunk_split($_format, 1, ','));
 		$spos = 0;
-		foreach ( $fparams as $fparam ) {
-
-			switch ( $fparam ) {
-				case 'd': (int)$day = substr( $_string, $spos, 2 ); $spos += 2; break;
-				case 'm': (int)$mon = substr( $_string, $spos, 2 ); $spos += 2; break;
-				case 'y': (int)$year = substr( $_string, $spos, 2 ); $spos += 2; break;
-				case 'Y': (int)$year = substr( $_string, $spos, 4 ); $spos += 4; break;
-				case 'H': (int)$hour = substr( $_string, $spos, 2 ); $spos += 2; break;
-				case 'h': (int)$hour = substr( $_string, $spos, 2 ); $spos += 2; break;
-				case 'i': (int)$min =  substr( $_string, $spos, 2 ); $spos += 2; break;
-				case 's': (int)$sec =  substr( $_string, $spos, 2 ); $spos += 2; break;
-				case 'O': (int)$offset = $year = substr( $_string, $spos, 5 ); $spos += 5; break;
+		foreach ($fparams as $fparam) {
+
+			switch ($fparam) {
+				case 'd': (int)$day = substr($_string, $spos, 2); $spos += 2; break;
+				case 'm': (int)$mon = substr($_string, $spos, 2); $spos += 2; break;
+				case 'y': (int)$year = substr($_string, $spos, 2); $spos += 2; break;
+				case 'Y': (int)$year = substr($_string, $spos, 4); $spos += 4; break;
+				case 'H': (int)$hour = substr($_string, $spos, 2); $spos += 2; break;
+				case 'h': (int)$hour = substr($_string, $spos, 2); $spos += 2; break;
+				case 'i': (int)$min = substr($_string, $spos, 2); $spos += 2; break;
+				case 's': (int)$sec = substr($_string, $spos, 2); $spos += 2; break;
+				case 'O': (int)$offset = $year = substr($_string, $spos, 5); $spos += 5; break;
 				case 'a': (int)$hour = $fparam == 'am' ? $hour : $hour + 12; break;
 				case 'A': (int)$hour = $fparam == 'AM' ? $hour : $hour + 12; break;
 				default: $spos++; // seperator
@@ -111,11 +111,11 @@ 
 		$timestamp = mktime($hour, $min, $sec, $mon, $day, $year, $_is_dst);
 
 		// offset given?
-		if ( isset( $offset ) && strlen( $offset == 5 ) ) {
+		if (isset($offset) && strlen($offset == 5)) {
 			$operator = $offset{0};
-			$ohour = 60 * 60 * (int)substr( $offset, 1, 2 );
-			$omin = 60 * (int)substr( $offset, 3, 2 );
-			if ( $operator == '+' ) $timestamp += $ohour + $omin;
+			$ohour = 60 * 60 * (int)substr($offset, 1, 2);
+			$omin = 60 * (int)substr($offset, 3, 2);
+			if ($operator == '+') $timestamp += $ohour + $omin;
 			else $timestamp -= $ohour + $omin;
 		}
 		return $timestamp;
@@ -126,11 +126,11 @@ 
 	 * @param mixed $_account_lid comma seperated list or array with lids
 	 * @return mixed comma seperated list or array with ids
 	 */
-	public static function account_name2id( &$_account_lids ) {
-		$account_lids = is_array( $_account_lids ) ? $_account_lids : explode( ',', $_account_lids );
+	public static function account_name2id(&$_account_lids) {
+		$account_lids = is_array($_account_lids) ? $_account_lids : explode(',', $_account_lids);
 		$skip = false;
-		foreach ( $account_lids as $key => $account_lid ) {
-			if($skip) {
+		foreach ($account_lids as $key => $account_lid) {
+			if ($skip) {
 				unset($account_lids[$key]);
 				$skip = false;
 				continue;
@@ -138,15 +138,15 @@ 
 			$account_lid = trim($account_lid);
 
 			// Handle any IDs that slip in
-			if(is_numeric($account_lid) && $GLOBALS['egw']->accounts->id2name($account_lid)) {
+			if (is_numeric($account_lid) && $GLOBALS['egw']->accounts->id2name($account_lid)) {
 				unset($account_lids[$key]);
 				$account_ids[] = (int)$account_lid;
 				continue;
 			}
 			// Check for [username]
-			if(strpos($account_lid,'[') !== false)
+			if (strpos($account_lid, '[') !== false)
 			{
-				if(preg_match('/\[(.+)\]/',$account_lid,$matches))
+				if (preg_match('/\[(.+)\]/', $account_lid, $matches))
 				{
 					$account_id = $GLOBALS['egw']->accounts->name2id($matches[1]);
 					unset($account_lids[$key]);
@@ -157,46 +157,46 @@ 
 
 			// Handle users listed as Lastname, Firstname instead of login ID
 			// Do this first, in case their first name matches a username
-			if ( $account_lids[$key+1][0] == ' ' && $account_id = $GLOBALS['egw']->accounts->name2id( trim($account_lids[$key+1]).' ' .$account_lid, 'account_fullname')) {
+			if ($account_lids[$key + 1][0] == ' ' && $account_id = $GLOBALS['egw']->accounts->name2id(trim($account_lids[$key + 1]).' '.$account_lid, 'account_fullname')) {
 				$account_ids[] = $account_id;
 				unset($account_lids[$key]);
 				$skip = true; // Skip the next one, it's the first name
-				continue ;
+				continue;
 			}
 
 			// Deal with groups listed as <name> Group, remove the Group
-			if(substr(trim($account_lid),-strlen(lang('Group'))) == lang('Group'))
+			if (substr(trim($account_lid), -strlen(lang('Group'))) == lang('Group'))
 			{
 				$account_lid = trim(substr(trim($account_lid), 0, -strlen(lang('Group'))));
 			}
 
-			if ( $account_id = $GLOBALS['egw']->accounts->name2id( $account_lid )) {
+			if ($account_id = $GLOBALS['egw']->accounts->name2id($account_lid)) {
 				$account_ids[] = $account_id;
 				unset($account_lids[$key]);
 				continue;
 			}
-			if ( $account_id = $GLOBALS['egw']->accounts->name2id( trim($account_lid), 'account_fullname' )) {
+			if ($account_id = $GLOBALS['egw']->accounts->name2id(trim($account_lid), 'account_fullname')) {
 				$account_ids[] = $account_id;
 				unset($account_lids[$key]);
 				continue;
 			}
 
 			// Handle groups listed as Group, <name>
-			if ( $account_lids[$key][0] == ' ' && $account_id = $GLOBALS['egw']->accounts->name2id( $account_lid)) {
+			if ($account_lids[$key][0] == ' ' && $account_id = $GLOBALS['egw']->accounts->name2id($account_lid)) {
 				$account_ids[] = $account_id;
-				unset($account_lids[$key-1]);
+				unset($account_lids[$key - 1]);
 				unset($account_lids[$key]);
 				continue;
 			}
 			// Group, <name> - remove the Group part
-			if($account_lid == lang('Group'))
+			if ($account_lid == lang('Group'))
 			{
 				unset($account_lids[$key]);
 				continue;
 			}
 		}
-		$_account_lids = (is_array($_account_lids) ? $account_lids : implode(',',array_unique($account_lids)));
-		return is_array( $_account_lids ) ? array_unique($account_ids) : implode( ',', array_unique((array)$account_ids ));
+		$_account_lids = (is_array($_account_lids) ? $account_lids : implode(',', array_unique($account_lids)));
+		return is_array($_account_lids) ? array_unique($account_ids) : implode(',', array_unique((array)$account_ids));
 
 	} // end of member function account_lid2id
 
@@ -206,14 +206,14 @@ 
 	 * @param mixed $_account_ids comma seperated list or array with ids
 	 * @return mixed comma seperated list or array with lids
 	 */
-	public static function account_id2name( $_account_id ) {
-		$account_ids = is_array( $_account_id ) ? $_account_id : explode( ',', $_account_id );
-		foreach ( $account_ids as $account_id ) {
-			if ( $account_lid = $GLOBALS['egw']->accounts->id2name( $account_id )) {
+	public static function account_id2name($_account_id) {
+		$account_ids = is_array($_account_id) ? $_account_id : explode(',', $_account_id);
+		foreach ($account_ids as $account_id) {
+			if ($account_lid = $GLOBALS['egw']->accounts->id2name($account_id)) {
 				$account_lids[] = $account_lid;
 			}
 		}
-		return is_array( $_account_id ) ? $account_lids : implode( ',', (array)$account_lids );
+		return is_array($_account_id) ? $account_lids : implode(',', (array)$account_lids);
 	} // end of member function account_id2lid
 
 	/**
@@ -222,12 +222,12 @@ 
 	 * @param mixed _cat_ids comma seperated list or array
 	 * @return mixed comma seperated list or array with cat_names
 	 */
-	public static function cat_id2name( $_cat_ids ) {
-		$cat_ids = is_array( $_cat_ids ) ? $_cat_ids : explode( ',', $_cat_ids );
-		foreach ( $cat_ids as $cat_id ) {
-			$cat_names[] = Api\Categories::id2name( (int)$cat_id );
+	public static function cat_id2name($_cat_ids) {
+		$cat_ids = is_array($_cat_ids) ? $_cat_ids : explode(',', $_cat_ids);
+		foreach ($cat_ids as $cat_id) {
+			$cat_names[] = Api\Categories::id2name((int)$cat_id);
 		}
-		return is_array( $_cat_ids ) ? $cat_names : implode(',',(array)$cat_names);
+		return is_array($_cat_ids) ? $cat_names : implode(',', (array)$cat_names);
 	} // end of member function category_id2name
 
 	/**
@@ -238,24 +238,24 @@ 
 	 * @param int $parent Optional parent ID to use for new categories
 	 * @return mixed comma seperated list or array with cat_ids
 	 */
-	public static function cat_name2id( $_cat_names, $parent = 0 ) {
-		$cats = new Api\Categories();	// uses current user and app (egw_info[flags][currentapp])
+	public static function cat_name2id($_cat_names, $parent = 0) {
+		$cats = new Api\Categories(); // uses current user and app (egw_info[flags][currentapp])
 
-		$cat_names = is_array( $_cat_names ) ? $_cat_names : explode( ',', $_cat_names );
-		foreach ( $cat_names as $cat_name ) {
+		$cat_names = is_array($_cat_names) ? $_cat_names : explode(',', $_cat_names);
+		foreach ($cat_names as $cat_name) {
 			$cat_name = trim($cat_name);
-			if ( $cat_name == '' ) continue;
-			if ( ( $cat_id = $cats->name2id( $cat_name ) ) == 0 && !self::$dry_run) {
-				$cat_id = $cats->add( array(
+			if ($cat_name == '') continue;
+			if (($cat_id = $cats->name2id($cat_name)) == 0 && !self::$dry_run) {
+				$cat_id = $cats->add(array(
 					'name' => $cat_name,
 					'parent' => $parent,
 					'access' => 'public',
-					'descr' => $cat_name. ' ('. lang('Automatically created by importexport'). ')'
+					'descr' => $cat_name.' ('.lang('Automatically created by importexport').')'
 				));
 			}
 			$cat_ids[] = $cat_id;
 		}
-		return is_array( $_cat_names ) ? $cat_ids : implode( ',', (array)$cat_ids );
+		return is_array($_cat_names) ? $cat_ids : implode(',', (array)$cat_ids);
 
 	} // end of member function category_name2id
 
@@ -294,76 +294,76 @@ 
 	 * @param object &$cclass calling class to process the '@ evals'
 	 * @return bool
 	 */
-	public static function conversion( &$_record,  $_conversion, &$_cclass = null ) {
-		if (empty( $_conversion ) ) return $_record;
+	public static function conversion(&$_record, $_conversion, &$_cclass = null) {
+		if (empty($_conversion)) return $_record;
 
-		self::$cclass =& $_cclass;
+		self::$cclass = & $_cclass;
 
 		$PSep = '||'; // Pattern-Separator, separats the pattern-replacement-pairs in conversion
 		$ASep = '|>'; // Assignment-Separator, separats pattern and replacesment
-		$CPre = '|['; $CPos = ']';  // |[_record-idx] is expanded to the corespondig value
+		$CPre = '|['; $CPos = ']'; // |[_record-idx] is expanded to the corespondig value
 		$TPre = '|T{'; $TPos = '}'; // |{_record-idx} is trimmed
-		$CntlPre = '|TC{';		    // Filter all cntl-chars \x01-\x1f and trim
-		$CntlnCLPre  = '|TCnCL{';   // Like |C{ but allowes CR and LF
-		$INE = '|INE{';             // Only insert if stuff in ^^ is not empty
+		$CntlPre = '|TC{'; // Filter all cntl-chars \x01-\x1f and trim
+		$CntlnCLPre = '|TCnCL{'; // Like |C{ but allowes CR and LF
+		$INE = '|INE{'; // Only insert if stuff in ^^ is not empty
 
-		foreach ( $_conversion as $idx => $conversion_string ) {
-			if ( empty( $conversion_string ) ) continue;
+		foreach ($_conversion as $idx => $conversion_string) {
+			if (empty($conversion_string)) continue;
 
 			// fetch patterns ($rvalues)
 			$rvalues = array();
-			$pat_reps = explode( $PSep, stripslashes( $conversion_string ) );
-			foreach( $pat_reps as $k => $pat_rep ) {
-				list( $pattern, $replace ) = explode( $ASep, $pat_rep, 2 );
-				if( $replace == '' ) {
+			$pat_reps = explode($PSep, stripslashes($conversion_string));
+			foreach ($pat_reps as $k => $pat_rep) {
+				list($pattern, $replace) = explode($ASep, $pat_rep, 2);
+				if ($replace == '') {
 					$replace = $pattern; $pattern = '^.*$';
 				}
-				$rvalues[$pattern] = $replace;	// replace two with only one, added by the form
+				$rvalues[$pattern] = $replace; // replace two with only one, added by the form
 			}
 
 			// conversion list may be longer than $_record aka (no_csv)
-			$val = array_key_exists( $idx, $_record ) ? $_record[$idx] : '';
+			$val = array_key_exists($idx, $_record) ? $_record[$idx] : '';
 
 			$c_functions = array('cat', 'account', 'strtotime', 'list');
-			if($_cclass) {
+			if ($_cclass) {
 				// Add in additional methods
 				$reflection = new ReflectionClass(get_class($_cclass));
 				$methods = $reflection->getMethods(ReflectionMethod::IS_STATIC);
-				foreach($methods as $method) {
+				foreach ($methods as $method) {
 					$c_functions[] = $method->name;
 				}
 			}
 			$c_functions = implode('|', $c_functions);
-			foreach ( $rvalues as $pattern => $replace ) {
+			foreach ($rvalues as $pattern => $replace) {
 				// Allow to include record indexes in pattern
 				$reg = '/\|\[([0-9]+)\]/';
-				while( preg_match( $reg, $pattern, $vars ) ) {
+				while (preg_match($reg, $pattern, $vars)) {
 					// expand all _record fields
 					$pattern = str_replace(
-						$CPre . $vars[1] . $CPos,
+						$CPre.$vars[1].$CPos,
 						$_record[array_search($vars[1], array_keys($_record))],
 						$pattern
 					);
 				}
-				if( preg_match('/'. (string)$pattern.'/', $val) ) {
+				if (preg_match('/'.(string)$pattern.'/', $val)) {
 
-					$val = preg_replace( '/'.(string)$pattern.'/', $replace, (string)$val );
+					$val = preg_replace('/'.(string)$pattern.'/', $replace, (string)$val);
 
 					$reg = '/\|\[([a-zA-Z_0-9]+)\]/';
-					while( preg_match( $reg, $val, $vars ) ) {
+					while (preg_match($reg, $val, $vars)) {
 						// expand all _record fields
 						$val = str_replace(
-							$CPre . $vars[1] . $CPos,
+							$CPre.$vars[1].$CPos,
 							$_record[array_search($vars[1], array_keys($_record))],
 							$val
 						);
 					}
-					$val = preg_replace_callback( "/($c_functions)\(([^)]*)\)/i", array( self, 'c2_dispatcher') , $val );
+					$val = preg_replace_callback("/($c_functions)\(([^)]*)\)/i", array(self, 'c2_dispatcher'), $val);
 					break;
 				}
 			}
 			// clean each field
-			$val = preg_replace_callback("/(\|T\{|\|TC\{|\|TCnCL\{|\|INE\{)(.*)\}/", array( self, 'strclean'), $val );
+			$val = preg_replace_callback("/(\|T\{|\|TC\{|\|TCnCL\{|\|INE\{)(.*)\}/", array(self, 'strclean'), $val);
 
 			$_record[$idx] = $val;
 		}
@@ -377,42 +377,42 @@ 
 	 *
 	 * @param array $_matches
 	 */
-	private static function c2_dispatcher( $_matches ) {
+	private static function c2_dispatcher($_matches) {
 		$action = &$_matches[1]; // cat or account ...
-		$data = &$_matches[2];   // datas for action
+		$data = &$_matches[2]; // datas for action
 
-		switch ( $action ) {
+		switch ($action) {
 			case 'strtotime' :
-				list( $string, $format ) = explode( ',', $data );
-				return self::custom_strtotime( trim( $string ), trim( $format ) );
+				list($string, $format) = explode(',', $data);
+				return self::custom_strtotime(trim($string), trim($format));
 			case 'list':
-				list( $split, $data, $index) = explode(',',$data);
+				list($split, $data, $index) = explode(',', $data);
 				$exploded = explode($split, $data);
 				// 1 based indexing for user ease
 				return $exploded[$index - 1];
 			default :
-				if(self::$cclass && method_exists(self::$cclass, $action)) {
+				if (self::$cclass && method_exists(self::$cclass, $action)) {
 					$class = get_class(self::$cclass);
 					return call_user_func("$class::$action", $data);
 				}
-				$method = (string)$action. ( is_int( $data ) ? '_id2name' : '_name2id' );
-				if(self::$cclass && method_exists(self::$cclass, $method)) {
+				$method = (string)$action.(is_int($data) ? '_id2name' : '_name2id');
+				if (self::$cclass && method_exists(self::$cclass, $method)) {
 					$class = get_class(self::$cclass);
 					return call_user_func("$class::$action", $data);
 				} else {
-					return self::$method( $data );
+					return self::$method($data);
 				}
 		}
 	}
 
-	private static function strclean( $_matches ) {
-		switch( $_matches[1] ) {
-			case '|T{' : return trim( $_matches[2] );
-			case '|TC{' : return trim( preg_replace( '/[\x01-\x1F]+/', '', $_matches[2] ) );
-			case '|TCnCL{' : return trim( preg_replace( '/[\x01-\x09\x11\x12\x14-\x1F]+/', '', $_matches[2] ) );
-			case '|INE{' : return preg_match( '/\^.+\^/', $_matches[2] ) ? $_matches[2] : '';
+	private static function strclean($_matches) {
+		switch ($_matches[1]) {
+			case '|T{' : return trim($_matches[2]);
+			case '|TC{' : return trim(preg_replace('/[\x01-\x1F]+/', '', $_matches[2]));
+			case '|TCnCL{' : return trim(preg_replace('/[\x01-\x09\x11\x12\x14-\x1F]+/', '', $_matches[2]));
+			case '|INE{' : return preg_match('/\^.+\^/', $_matches[2]) ? $_matches[2] : '';
 			default:
-				throw new Exception('Error in conversion string! "'. substr( $_matches[1], 0, -1 ). '" is not valid!');
+				throw new Exception('Error in conversion string! "'.substr($_matches[1], 0, -1).'" is not valid!');
 		}
 	}
 
@@ -423,23 +423,23 @@ 
 	 * @param string $_appname {<appname> | all}
 	 * @return array(<appname> => array( <type> => array(<plugin> => <title>)))
 	 */
-	public static function get_plugins( $_appname = 'all', $_type = 'all' ) {
+	public static function get_plugins($_appname = 'all', $_type = 'all') {
 		$plugins = Api\Cache::getTree(
 			__CLASS__,
 			'plugins',
-			array('importexport_helper_functions','_get_plugins'),
+			array('importexport_helper_functions', '_get_plugins'),
 			array(array_keys($GLOBALS['egw_info']['apps']), array('import', 'export')),
 			self::CACHE_EXPIRATION
 		);
 		$appnames = $_appname == 'all' ? array_keys($GLOBALS['egw_info']['apps']) : (array)$_appname;
-		$types = $_type == 'all' ? array('import','export') : (array)$_type;
+		$types = $_type == 'all' ? array('import', 'export') : (array)$_type;
 
 		// Testing: comment out Api\Cache call, use this
 		//$plugins = self::_get_plugins($appnames, $types);
-		foreach($plugins as $appname => $_types) {
-			if(!in_array($appname, $appnames)) unset($plugins[$appname]);
+		foreach ($plugins as $appname => $_types) {
+			if (!in_array($appname, $appnames)) unset($plugins[$appname]);
 		}
-		foreach($plugins as $appname => $types) {
+		foreach ($plugins as $appname => $types) {
 			$plugins[$appname] = array_intersect_key($plugins[$appname], $types);
 		}
 		return $plugins;
@@ -448,25 +448,25 @@ 
 	public static function _get_plugins(Array $appnames, Array $types) {
 		$plugins = array();
 		foreach ($appnames as $appname) {
-			if(array_key_exists($appname, self::$blacklist_files) && self::$blacklist_files[$appname] === true) continue;
+			if (array_key_exists($appname, self::$blacklist_files) && self::$blacklist_files[$appname] === true) continue;
 
-			$appdir = EGW_INCLUDE_ROOT. "/$appname/inc";
-			if(!is_dir($appdir)) continue;
+			$appdir = EGW_INCLUDE_ROOT."/$appname/inc";
+			if (!is_dir($appdir)) continue;
 			$d = dir($appdir);
 
 			// step through each file in appdir
 			while (false !== ($entry = $d->read())) {
 				// Blacklisted?
-				if(is_array(self::$blacklist_files[$appname]) && in_array($entry, self::$blacklist_files[$appname]))  continue;
+				if (is_array(self::$blacklist_files[$appname]) && in_array($entry, self::$blacklist_files[$appname]))  continue;
 				if (!preg_match('/^class\.([^.]+)\.inc\.php$/', $entry, $matches)) continue;
 				$classname = $matches[1];
-				$file = $appdir. '/'. $entry;
+				$file = $appdir.'/'.$entry;
 
 				foreach ($types as $type) {
-					if( !is_file($file) || strpos($entry, $type) === false || strpos($entry,'wizard') !== false) continue;
+					if (!is_file($file) || strpos($entry, $type) === false || strpos($entry, 'wizard') !== false) continue;
 					require_once($file);
 					$reflectionClass = new ReflectionClass($classname);
-					if($reflectionClass->IsInstantiable() &&
+					if ($reflectionClass->IsInstantiable() &&
 							$reflectionClass->implementsInterface('importexport_iface_'.$type.'_plugin')) {
 						try {
 							$plugin_object = new $classname;
@@ -482,7 +482,7 @@ 
 			$d->close();
 
 			$config = Api\Config::read('importexport');
-			if($config['update'] == 'auto') {
+			if ($config['update'] == 'auto') {
 				self::load_defaults($appname);
 			}
 		}
@@ -497,37 +497,37 @@ 
 	 * @return array $num => $appname
 	 */
 	public static function get_apps($_type, $ignore_acl = false) {
-		$apps = array_keys(self::get_plugins('all',$_type));
-		if($ignore_acl) return $apps;
+		$apps = array_keys(self::get_plugins('all', $_type));
+		if ($ignore_acl) return $apps;
 
-		foreach($apps as $key => $app) {
-			if(!self::has_definitions($app, $_type)) unset($apps[$key]);
+		foreach ($apps as $key => $app) {
+			if (!self::has_definitions($app, $_type)) unset($apps[$key]);
 		}
 		return $apps;
 	}
 
 	public static function load_defaults($appname) {
 		// Check for new definitions to import from $appname/setup/*.xml
-		$appdir = EGW_INCLUDE_ROOT. "/$appname/setup";
-		if(!is_dir($appdir)) return;
+		$appdir = EGW_INCLUDE_ROOT."/$appname/setup";
+		if (!is_dir($appdir)) return;
 		$d = dir($appdir);
 
 		// step through each file in app's setup
 		while (false !== ($entry = $d->read())) {
-			$file = $appdir. '/'. $entry;
-			list( $filename, $extension) = explode('.',$entry);
-			if ( $extension != 'xml' ) continue;
+			$file = $appdir.'/'.$entry;
+			list($filename, $extension) = explode('.', $entry);
+			if ($extension != 'xml') continue;
 			try {
 				// import will skip invalid files
-				importexport_definitions_bo::import( $file );
+				importexport_definitions_bo::import($file);
 			} catch (Exception $e) {
-				error_log(__CLASS__.__FUNCTION__. " import $appname definitions: " . $e->getMessage());
+				error_log(__CLASS__.__FUNCTION__." import $appname definitions: ".$e->getMessage());
 			}
 		}
 		$d->close();
 	}
 
-	public static function guess_filetype( $_file ) {
+	public static function guess_filetype($_file) {
 
 	}
 
@@ -538,24 +538,24 @@ 
 	 * @param string $_type {import | export | all}
 	 * @return boolean
 	 */
-	public static function has_definitions( $_appname = 'all', $_type = 'all' ) {
+	public static function has_definitions($_appname = 'all', $_type = 'all') {
 		$definitions = Api\Cache::getSession(
 			__CLASS__,
 			'has_definitions',
-			array('importexport_helper_functions','_has_definitions'),
+			array('importexport_helper_functions', '_has_definitions'),
 			array(array_keys($GLOBALS['egw_info']['apps']), array('import', 'export')),
 			self::CACHE_EXPIRATION
 		);
 		$appnames = $_appname == 'all' ? array_keys($GLOBALS['egw_info']['apps']) : (array)$_appname;
-		$types = $_type == 'all' ? array('import','export') : (array)$_type;
+		$types = $_type == 'all' ? array('import', 'export') : (array)$_type;
 
 		// Testing: Comment out cache call above, use this
 		//$definitions = self::_has_definitions($appnames, $types);
 
-		foreach($definitions as $appname => $_types) {
-			if(!in_array($appname, $appnames)) unset($definitions[$appname]);
+		foreach ($definitions as $appname => $_types) {
+			if (!in_array($appname, $appnames)) unset($definitions[$appname]);
 		}
-		foreach($definitions as $appname => $_types) {
+		foreach ($definitions as $appname => $_types) {
 			$definitions[$appname] = array_intersect_key($definitions[$appname], array_flip($types));
 		}
 		return count($definitions[$appname]) > 0;
@@ -565,11 +565,11 @@ 
 	public static function _has_definitions(Array $appnames, Array $types) {
 		$def = new importexport_definitions_bo(array('application'=>$appnames, 'type' => $types));
 		$list = array();
-		foreach((array)$def->get_definitions() as $id) {
+		foreach ((array)$def->get_definitions() as $id) {
 			// Need to instanciate it to check, but if the user doesn't have permission, it throws an exception
 			try {
 				$definition = new importexport_definition($id);
-				if($def->is_permitted($definition->get_record_array())) {
+				if ($def->is_permitted($definition->get_record_array())) {
 					$list[$definition->application][$definition->type][] = $id;
 				}
 			} catch (Exception $e) {
@@ -601,13 +601,13 @@ 
 			$plugin = is_object($plugin_name) ? $plugin_name : new $plugin_name();
 			$plugin_name = get_class($plugin);
 
-			if($record_classname == null) $record_classname = $plugin::get_egw_record_class();
-			if(!class_exists($record_classname)) throw new Exception('Bad class name ' . $record_classname);
+			if ($record_classname == null) $record_classname = $plugin::get_egw_record_class();
+			if (!class_exists($record_classname)) throw new Exception('Bad class name '.$record_classname);
 
-			if(!$wizard_plugin)
+			if (!$wizard_plugin)
 			{
-				$wizard_name = $app_name . '_wizard_' . str_replace($app_name . '_', '', $plugin_name);
-				if(!class_exists($wizard_name)) throw new Exception('Bad wizard name ' . $wizard_name);
+				$wizard_name = $app_name.'_wizard_'.str_replace($app_name.'_', '', $plugin_name);
+				if (!class_exists($wizard_name)) throw new Exception('Bad wizard name '.$wizard_name);
 				$wizard_plugin = new $wizard_name;
 			}
 		}
@@ -619,16 +619,16 @@ 
 
 		// Get field -> label map and initialize fields using wizard field order
 		$fields = $export_fields = array();
-		if(method_exists($wizard_plugin, 'get_export_fields'))
+		if (method_exists($wizard_plugin, 'get_export_fields'))
 		{
 			$fields = $export_fields = $wizard_plugin->get_export_fields();
 		}
 
-		foreach($record_classname::$types as $type => $type_fields)
+		foreach ($record_classname::$types as $type => $type_fields)
 		{
 			// Only these for now, until filter methods for others are figured out
-			if(!in_array($type, array('select','select-cat','select-account','date','date-time'))) continue;
-			foreach($type_fields as $field_name)
+			if (!in_array($type, array('select', 'select-cat', 'select-account', 'date', 'date-time'))) continue;
+			foreach ($type_fields as $field_name)
 			{
 				$fields[$field_name] = array(
 					'name' => $field_name,
@@ -639,21 +639,21 @@ 
 		}
 		// Add custom fields
 		$custom = Api\Storage\Customfields::get($app_name);
-		foreach($custom as $field_name => $settings)
+		foreach ($custom as $field_name => $settings)
 		{
 			$settings['name'] = '#'.$field_name;
 			$fields['#'.$field_name] = $settings;
 		}
 
-		foreach($fields as $field_name => &$settings) {
+		foreach ($fields as $field_name => &$settings) {
 			// Can't really filter on these (or at least no generic, sane way figured out yet)
-			if(!is_array($settings) || in_array($settings['type'], array('text','button', 'label','url','url-email','url-phone','htmlarea')))
+			if (!is_array($settings) || in_array($settings['type'], array('text', 'button', 'label', 'url', 'url-email', 'url-phone', 'htmlarea')))
 			{
 				unset($fields[$field_name]);
 				continue;
 			}
-			if($settings['type'] == 'radio') $settings['type'] = 'select';
-			switch($settings['type'])
+			if ($settings['type'] == 'radio') $settings['type'] = 'select';
+			switch ($settings['type'])
 			{
 				case 'checkbox':
 					// This isn't quite right - there's only 2 options and you can select both
@@ -676,7 +676,7 @@ 
 			}
 		}
 
-		if(method_exists($plugin, 'get_filter_fields'))
+		if (method_exists($plugin, 'get_filter_fields'))
 		{
 			$plugin->get_filter_fields($fields);
 		}
@@ -692,65 +692,65 @@ 
 	 */
 	public static function date_rel2abs($value)
 	{
-		if(is_array($value))
+		if (is_array($value))
 		{
 			$abs = array();
-			foreach($value as $key => $val)
+			foreach ($value as $key => $val)
 			{
 				$abs[$key] = self::date_rel2abs($val);
 			}
 			return $abs;
 		}
-		if($date = self::$relative_dates[$value])
+		if ($date = self::$relative_dates[$value])
 		{
-			$year  = (int) date('Y');
-			$month = (int) date('m');
-			$day   = (int) date('d');
-			$today = mktime(0,0,0,date('m'),date('d'),date('Y'));
+			$year  = (int)date('Y');
+			$month = (int)date('m');
+			$day   = (int)date('d');
+			$today = mktime(0, 0, 0, date('m'), date('d'), date('Y'));
 
-			list($syear,$smonth,$sday,$sweek,$eyear,$emonth,$eday,$eweek) = $date;
+			list($syear, $smonth, $sday, $sweek, $eyear, $emonth, $eday, $eweek) = $date;
 
-			if(stripos($value, 'quarter') !== false)
+			if (stripos($value, 'quarter') !== false)
 			{
 				// Handle quarters
-				$start = mktime(0,0,0,((int)floor(($smonth+$month) / 3.1)) * 3 + 1, 1, $year);
-				$end = mktime(0,0,0,((int)floor(($emonth+$month) / 3.1)+1) * 3 + 1, 1, $year);
+				$start = mktime(0, 0, 0, ((int)floor(($smonth + $month) / 3.1)) * 3 + 1, 1, $year);
+				$end = mktime(0, 0, 0, ((int)floor(($emonth + $month) / 3.1) + 1) * 3 + 1, 1, $year);
 			}
 			elseif ($syear || $eyear)
 			{
-				$start = mktime(0,0,0,1,1,$syear+$year);
-				$end   = mktime(0,0,0,1,1,$eyear+$year);
+				$start = mktime(0, 0, 0, 1, 1, $syear + $year);
+				$end   = mktime(0, 0, 0, 1, 1, $eyear + $year);
 			}
 			elseif ($smonth || $emonth)
 			{
-				$start = mktime(0,0,0,$smonth+$month,1,$year);
-				$end   = mktime(0,0,0,$emonth+$month,1,$year);
+				$start = mktime(0, 0, 0, $smonth + $month, 1, $year);
+				$end   = mktime(0, 0, 0, $emonth + $month, 1, $year);
 			}
 			elseif ($sday || $eday)
 			{
-				$start = mktime(0,0,0,$month,$sday+$day,$year);
-				$end   = mktime(0,0,0,$month,$eday+$day,$year);
+				$start = mktime(0, 0, 0, $month, $sday + $day, $year);
+				$end   = mktime(0, 0, 0, $month, $eday + $day, $year);
 			}
 			elseif ($sweek || $eweek)
 			{
-				$wday = (int) date('w'); // 0=sun, ..., 6=sat
-				switch($GLOBALS['egw_info']['user']['preferences']['calendar']['weekdaystarts'])
+				$wday = (int)date('w'); // 0=sun, ..., 6=sat
+				switch ($GLOBALS['egw_info']['user']['preferences']['calendar']['weekdaystarts'])
 				{
 					case 'Sunday':
-						$weekstart = $today - $wday * 24*60*60;
+						$weekstart = $today - $wday * 24 * 60 * 60;
 						break;
 					case 'Saturday':
-						$weekstart = $today - (6-$wday) * 24*60*60;
+						$weekstart = $today - (6 - $wday) * 24 * 60 * 60;
 						break;
 					case 'Moday':
 					default:
-						$weekstart = $today - ($wday ? $wday-1 : 6) * 24*60*60;
+						$weekstart = $today - ($wday ? $wday - 1 : 6) * 24 * 60 * 60;
 						break;
 				}
-				$start = $weekstart + $sweek*7*24*60*60;
-				$end   = $weekstart + $eweek*7*24*60*60;
+				$start = $weekstart + $sweek * 7 * 24 * 60 * 60;
+				$end   = $weekstart + $eweek * 7 * 24 * 60 * 60;
 			}
-			$end_param = $end - 24*60*60;
+			$end_param = $end - 24 * 60 * 60;
 
 			// Take 1 second off end to provide an inclusive range.for filtering
 			$end -= 1;

Braces +210 added lines, -83 removed lines

@@ -16,7 +16,8 @@ 
  * class importexport_helper_functions (only static methods)
  * use importexport_helper_functions::method
  */
-class importexport_helper_functions {
+class importexport_helper_functions
+{
 
 	/**
 	 * Plugins are scanned and cached for all instances using this source path for given time (in seconds)
@@ -85,13 +86,19 @@ 
 	 * @param string $_format format of time string e.g.: d.m.Y H:i
 	 * @param int $_is_dst is day light saving time? 0 = no, 1 = yes, -1 = system default
 	 */
-	public static function custom_strtotime( $_string, $_format='', $_is_dst = -1) {
-		if ( empty( $_format ) ) return strtotime( $_string );
+	public static function custom_strtotime( $_string, $_format='', $_is_dst = -1)
+	{
+		if ( empty( $_format ) )
+		{
+			return strtotime( $_string );
+		}
 		$fparams = explode( ',', chunk_split( $_format, 1, ',' ) );
 		$spos = 0;
-		foreach ( $fparams as $fparam ) {
+		foreach ( $fparams as $fparam )
+		{
 
-			switch ( $fparam ) {
+			switch ( $fparam )
+			{
 				case 'd': (int)$day = substr( $_string, $spos, 2 ); $spos += 2; break;
 				case 'm': (int)$mon = substr( $_string, $spos, 2 ); $spos += 2; break;
 				case 'y': (int)$year = substr( $_string, $spos, 2 ); $spos += 2; break;
@@ -111,12 +118,18 @@ 
 		$timestamp = mktime($hour, $min, $sec, $mon, $day, $year, $_is_dst);
 
 		// offset given?
-		if ( isset( $offset ) && strlen( $offset == 5 ) ) {
+		if ( isset( $offset ) && strlen( $offset == 5 ) )
+		{
 			$operator = $offset{0};
 			$ohour = 60 * 60 * (int)substr( $offset, 1, 2 );
 			$omin = 60 * (int)substr( $offset, 3, 2 );
-			if ( $operator == '+' ) $timestamp += $ohour + $omin;
-			else $timestamp -= $ohour + $omin;
+			if ( $operator == '+' )
+			{
+				$timestamp += $ohour + $omin;
+			}
+			else {
+				$timestamp -= $ohour + $omin;
+			}
 		}
 		return $timestamp;
 	}
@@ -126,11 +139,14 @@ 
 	 * @param mixed $_account_lid comma seperated list or array with lids
 	 * @return mixed comma seperated list or array with ids
 	 */
-	public static function account_name2id( &$_account_lids ) {
+	public static function account_name2id( &$_account_lids )
+	{
 		$account_lids = is_array( $_account_lids ) ? $_account_lids : explode( ',', $_account_lids );
 		$skip = false;
-		foreach ( $account_lids as $key => $account_lid ) {
-			if($skip) {
+		foreach ( $account_lids as $key => $account_lid )
+		{
+			if($skip)
+			{
 				unset($account_lids[$key]);
 				$skip = false;
 				continue;
@@ -138,7 +154,8 @@ 
 			$account_lid = trim($account_lid);
 
 			// Handle any IDs that slip in
-			if(is_numeric($account_lid) && $GLOBALS['egw']->accounts->id2name($account_lid)) {
+			if(is_numeric($account_lid) && $GLOBALS['egw']->accounts->id2name($account_lid))
+			{
 				unset($account_lids[$key]);
 				$account_ids[] = (int)$account_lid;
 				continue;
@@ -157,7 +174,8 @@ 
 
 			// Handle users listed as Lastname, Firstname instead of login ID
 			// Do this first, in case their first name matches a username
-			if ( $account_lids[$key+1][0] == ' ' && $account_id = $GLOBALS['egw']->accounts->name2id( trim($account_lids[$key+1]).' ' .$account_lid, 'account_fullname')) {
+			if ( $account_lids[$key+1][0] == ' ' && $account_id = $GLOBALS['egw']->accounts->name2id( trim($account_lids[$key+1]).' ' .$account_lid, 'account_fullname'))
+			{
 				$account_ids[] = $account_id;
 				unset($account_lids[$key]);
 				$skip = true; // Skip the next one, it's the first name
@@ -170,19 +188,22 @@ 
 				$account_lid = trim(substr(trim($account_lid), 0, -strlen(lang('Group'))));
 			}
 
-			if ( $account_id = $GLOBALS['egw']->accounts->name2id( $account_lid )) {
+			if ( $account_id = $GLOBALS['egw']->accounts->name2id( $account_lid ))
+			{
 				$account_ids[] = $account_id;
 				unset($account_lids[$key]);
 				continue;
 			}
-			if ( $account_id = $GLOBALS['egw']->accounts->name2id( trim($account_lid), 'account_fullname' )) {
+			if ( $account_id = $GLOBALS['egw']->accounts->name2id( trim($account_lid), 'account_fullname' ))
+			{
 				$account_ids[] = $account_id;
 				unset($account_lids[$key]);
 				continue;
 			}
 
 			// Handle groups listed as Group, <name>
-			if ( $account_lids[$key][0] == ' ' && $account_id = $GLOBALS['egw']->accounts->name2id( $account_lid)) {
+			if ( $account_lids[$key][0] == ' ' && $account_id = $GLOBALS['egw']->accounts->name2id( $account_lid))
+			{
 				$account_ids[] = $account_id;
 				unset($account_lids[$key-1]);
 				unset($account_lids[$key]);
@@ -206,10 +227,13 @@ 
 	 * @param mixed $_account_ids comma seperated list or array with ids
 	 * @return mixed comma seperated list or array with lids
 	 */
-	public static function account_id2name( $_account_id ) {
+	public static function account_id2name( $_account_id )
+	{
 		$account_ids = is_array( $_account_id ) ? $_account_id : explode( ',', $_account_id );
-		foreach ( $account_ids as $account_id ) {
-			if ( $account_lid = $GLOBALS['egw']->accounts->id2name( $account_id )) {
+		foreach ( $account_ids as $account_id )
+		{
+			if ( $account_lid = $GLOBALS['egw']->accounts->id2name( $account_id ))
+			{
 				$account_lids[] = $account_lid;
 			}
 		}
@@ -222,9 +246,11 @@ 
 	 * @param mixed _cat_ids comma seperated list or array
 	 * @return mixed comma seperated list or array with cat_names
 	 */
-	public static function cat_id2name( $_cat_ids ) {
+	public static function cat_id2name( $_cat_ids )
+	{
 		$cat_ids = is_array( $_cat_ids ) ? $_cat_ids : explode( ',', $_cat_ids );
-		foreach ( $cat_ids as $cat_id ) {
+		foreach ( $cat_ids as $cat_id )
+		{
 			$cat_names[] = Api\Categories::id2name( (int)$cat_id );
 		}
 		return is_array( $_cat_ids ) ? $cat_names : implode(',',(array)$cat_names);
@@ -238,14 +264,20 @@ 
 	 * @param int $parent Optional parent ID to use for new categories
 	 * @return mixed comma seperated list or array with cat_ids
 	 */
-	public static function cat_name2id( $_cat_names, $parent = 0 ) {
+	public static function cat_name2id( $_cat_names, $parent = 0 )
+	{
 		$cats = new Api\Categories();	// uses current user and app (egw_info[flags][currentapp])
 
 		$cat_names = is_array( $_cat_names ) ? $_cat_names : explode( ',', $_cat_names );
-		foreach ( $cat_names as $cat_name ) {
+		foreach ( $cat_names as $cat_name )
+		{
 			$cat_name = trim($cat_name);
-			if ( $cat_name == '' ) continue;
-			if ( ( $cat_id = $cats->name2id( $cat_name ) ) == 0 && !self::$dry_run) {
+			if ( $cat_name == '' )
+			{
+				continue;
+			}
+			if ( ( $cat_id = $cats->name2id( $cat_name ) ) == 0 && !self::$dry_run)
+			{
 				$cat_id = $cats->add( array(
 					'name' => $cat_name,
 					'parent' => $parent,
@@ -294,8 +326,12 @@ 
 	 * @param object &$cclass calling class to process the '@ evals'
 	 * @return bool
 	 */
-	public static function conversion( &$_record,  $_conversion, &$_cclass = null ) {
-		if (empty( $_conversion ) ) return $_record;
+	public static function conversion( &$_record,  $_conversion, &$_cclass = null )
+	{
+		if (empty( $_conversion ) )
+		{
+			return $_record;
+		}
 
 		self::$cclass =& $_cclass;
 
@@ -307,15 +343,21 @@ 
 		$CntlnCLPre  = '|TCnCL{';   // Like |C{ but allowes CR and LF
 		$INE = '|INE{';             // Only insert if stuff in ^^ is not empty
 
-		foreach ( $_conversion as $idx => $conversion_string ) {
-			if ( empty( $conversion_string ) ) continue;
+		foreach ( $_conversion as $idx => $conversion_string )
+		{
+			if ( empty( $conversion_string ) )
+			{
+				continue;
+			}
 
 			// fetch patterns ($rvalues)
 			$rvalues = array();
 			$pat_reps = explode( $PSep, stripslashes( $conversion_string ) );
-			foreach( $pat_reps as $k => $pat_rep ) {
+			foreach( $pat_reps as $k => $pat_rep )
+			{
 				list( $pattern, $replace ) = explode( $ASep, $pat_rep, 2 );
-				if( $replace == '' ) {
+				if( $replace == '' )
+				{
 					$replace = $pattern; $pattern = '^.*$';
 				}
 				$rvalues[$pattern] = $replace;	// replace two with only one, added by the form
@@ -325,19 +367,23 @@ 
 			$val = array_key_exists( $idx, $_record ) ? $_record[$idx] : '';
 
 			$c_functions = array('cat', 'account', 'strtotime', 'list');
-			if($_cclass) {
+			if($_cclass)
+			{
 				// Add in additional methods
 				$reflection = new ReflectionClass(get_class($_cclass));
 				$methods = $reflection->getMethods(ReflectionMethod::IS_STATIC);
-				foreach($methods as $method) {
+				foreach($methods as $method)
+				{
 					$c_functions[] = $method->name;
 				}
 			}
 			$c_functions = implode('|', $c_functions);
-			foreach ( $rvalues as $pattern => $replace ) {
+			foreach ( $rvalues as $pattern => $replace )
+			{
 				// Allow to include record indexes in pattern
 				$reg = '/\|\[([0-9]+)\]/';
-				while( preg_match( $reg, $pattern, $vars ) ) {
+				while( preg_match( $reg, $pattern, $vars ) )
+				{
 					// expand all _record fields
 					$pattern = str_replace(
 						$CPre . $vars[1] . $CPos,
@@ -345,12 +391,14 @@ 
 						$pattern
 					);
 				}
-				if( preg_match('/'. (string)$pattern.'/', $val) ) {
+				if( preg_match('/'. (string)$pattern.'/', $val) )
+				{
 
 					$val = preg_replace( '/'.(string)$pattern.'/', $replace, (string)$val );
 
 					$reg = '/\|\[([a-zA-Z_0-9]+)\]/';
-					while( preg_match( $reg, $val, $vars ) ) {
+					while( preg_match( $reg, $val, $vars ) )
+					{
 						// expand all _record fields
 						$val = str_replace(
 							$CPre . $vars[1] . $CPos,
@@ -377,11 +425,13 @@ 
 	 *
 	 * @param array $_matches
 	 */
-	private static function c2_dispatcher( $_matches ) {
+	private static function c2_dispatcher( $_matches )
+	{
 		$action = &$_matches[1]; // cat or account ...
 		$data = &$_matches[2];   // datas for action
 
-		switch ( $action ) {
+		switch ( $action )
+		{
 			case 'strtotime' :
 				list( $string, $format ) = explode( ',', $data );
 				return self::custom_strtotime( trim( $string ), trim( $format ) );
@@ -391,22 +441,28 @@ 
 				// 1 based indexing for user ease
 				return $exploded[$index - 1];
 			default :
-				if(self::$cclass && method_exists(self::$cclass, $action)) {
+				if(self::$cclass && method_exists(self::$cclass, $action))
+				{
 					$class = get_class(self::$cclass);
 					return call_user_func("$class::$action", $data);
 				}
 				$method = (string)$action. ( is_int( $data ) ? '_id2name' : '_name2id' );
-				if(self::$cclass && method_exists(self::$cclass, $method)) {
+				if(self::$cclass && method_exists(self::$cclass, $method))
+				{
 					$class = get_class(self::$cclass);
 					return call_user_func("$class::$action", $data);
-				} else {
+				}
+				else
+				{
 					return self::$method( $data );
 				}
 		}
 	}
 
-	private static function strclean( $_matches ) {
-		switch( $_matches[1] ) {
+	private static function strclean( $_matches )
+	{
+		switch( $_matches[1] )
+		{
 			case '|T{' : return trim( $_matches[2] );
 			case '|TC{' : return trim( preg_replace( '/[\x01-\x1F]+/', '', $_matches[2] ) );
 			case '|TCnCL{' : return trim( preg_replace( '/[\x01-\x09\x11\x12\x14-\x1F]+/', '', $_matches[2] ) );
@@ -423,7 +479,8 @@ 
 	 * @param string $_appname {<appname> | all}
 	 * @return array(<appname> => array( <type> => array(<plugin> => <title>)))
 	 */
-	public static function get_plugins( $_appname = 'all', $_type = 'all' ) {
+	public static function get_plugins( $_appname = 'all', $_type = 'all' )
+	{
 		$plugins = Api\Cache::getTree(
 			__CLASS__,
 			'plugins',
@@ -436,38 +493,63 @@ 
 
 		// Testing: comment out Api\Cache call, use this
 		//$plugins = self::_get_plugins($appnames, $types);
-		foreach($plugins as $appname => $_types) {
-			if(!in_array($appname, $appnames)) unset($plugins[$appname]);
+		foreach($plugins as $appname => $_types)
+		{
+			if(!in_array($appname, $appnames))
+			{
+				unset($plugins[$appname]);
+			}
 		}
-		foreach($plugins as $appname => $types) {
+		foreach($plugins as $appname => $types)
+		{
 			$plugins[$appname] = array_intersect_key($plugins[$appname], $types);
 		}
 		return $plugins;
 	}
 
-	public static function _get_plugins(Array $appnames, Array $types) {
+	public static function _get_plugins(Array $appnames, Array $types)
+	{
 		$plugins = array();
-		foreach ($appnames as $appname) {
-			if(array_key_exists($appname, self::$blacklist_files) && self::$blacklist_files[$appname] === true) continue;
+		foreach ($appnames as $appname)
+		{
+			if(array_key_exists($appname, self::$blacklist_files) && self::$blacklist_files[$appname] === true)
+			{
+				continue;
+			}
 
 			$appdir = EGW_INCLUDE_ROOT. "/$appname/inc";
-			if(!is_dir($appdir)) continue;
+			if(!is_dir($appdir))
+			{
+				continue;
+			}
 			$d = dir($appdir);
 
 			// step through each file in appdir
-			while (false !== ($entry = $d->read())) {
+			while (false !== ($entry = $d->read()))
+			{
 				// Blacklisted?
-				if(is_array(self::$blacklist_files[$appname]) && in_array($entry, self::$blacklist_files[$appname]))  continue;
-				if (!preg_match('/^class\.([^.]+)\.inc\.php$/', $entry, $matches)) continue;
+				if(is_array(self::$blacklist_files[$appname]) && in_array($entry, self::$blacklist_files[$appname]))
+				{
+					continue;
+				}
+				if (!preg_match('/^class\.([^.]+)\.inc\.php$/', $entry, $matches))
+				{
+					continue;
+				}
 				$classname = $matches[1];
 				$file = $appdir. '/'. $entry;
 
-				foreach ($types as $type) {
-					if( !is_file($file) || strpos($entry, $type) === false || strpos($entry,'wizard') !== false) continue;
+				foreach ($types as $type)
+				{
+					if( !is_file($file) || strpos($entry, $type) === false || strpos($entry,'wizard') !== false)
+					{
+						continue;
+					}
 					require_once($file);
 					$reflectionClass = new ReflectionClass($classname);
 					if($reflectionClass->IsInstantiable() &&
-							$reflectionClass->implementsInterface('importexport_iface_'.$type.'_plugin')) {
+							$reflectionClass->implementsInterface('importexport_iface_'.$type.'_plugin'))
+					{
 						try {
 							$plugin_object = new $classname;
 						}
@@ -482,7 +564,8 @@ 
 			$d->close();
 
 			$config = Api\Config::read('importexport');
-			if($config['update'] == 'auto') {
+			if($config['update'] == 'auto')
+			{
 				self::load_defaults($appname);
 			}
 		}
@@ -496,38 +579,56 @@ 
 	 * @param string $_type
 	 * @return array $num => $appname
 	 */
-	public static function get_apps($_type, $ignore_acl = false) {
+	public static function get_apps($_type, $ignore_acl = false)
+	{
 		$apps = array_keys(self::get_plugins('all',$_type));
-		if($ignore_acl) return $apps;
+		if($ignore_acl)
+		{
+			return $apps;
+		}
 
-		foreach($apps as $key => $app) {
-			if(!self::has_definitions($app, $_type)) unset($apps[$key]);
+		foreach($apps as $key => $app)
+		{
+			if(!self::has_definitions($app, $_type))
+			{
+				unset($apps[$key]);
+			}
 		}
 		return $apps;
 	}
 
-	public static function load_defaults($appname) {
+	public static function load_defaults($appname)
+	{
 		// Check for new definitions to import from $appname/setup/*.xml
 		$appdir = EGW_INCLUDE_ROOT. "/$appname/setup";
-		if(!is_dir($appdir)) return;
+		if(!is_dir($appdir))
+		{
+			return;
+		}
 		$d = dir($appdir);
 
 		// step through each file in app's setup
-		while (false !== ($entry = $d->read())) {
+		while (false !== ($entry = $d->read()))
+		{
 			$file = $appdir. '/'. $entry;
 			list( $filename, $extension) = explode('.',$entry);
-			if ( $extension != 'xml' ) continue;
+			if ( $extension != 'xml' )
+			{
+				continue;
+			}
 			try {
 				// import will skip invalid files
 				importexport_definitions_bo::import( $file );
-			} catch (Exception $e) {
+			}
+			catch (Exception $e) {
 				error_log(__CLASS__.__FUNCTION__. " import $appname definitions: " . $e->getMessage());
 			}
 		}
 		$d->close();
 	}
 
-	public static function guess_filetype( $_file ) {
+	public static function guess_filetype( $_file )
+	{
 
 	}
 
@@ -538,7 +639,8 @@ 
 	 * @param string $_type {import | export | all}
 	 * @return boolean
 	 */
-	public static function has_definitions( $_appname = 'all', $_type = 'all' ) {
+	public static function has_definitions( $_appname = 'all', $_type = 'all' )
+	{
 		$definitions = Api\Cache::getSession(
 			__CLASS__,
 			'has_definitions',
@@ -552,27 +654,36 @@ 
 		// Testing: Comment out cache call above, use this
 		//$definitions = self::_has_definitions($appnames, $types);
 
-		foreach($definitions as $appname => $_types) {
-			if(!in_array($appname, $appnames)) unset($definitions[$appname]);
+		foreach($definitions as $appname => $_types)
+		{
+			if(!in_array($appname, $appnames))
+			{
+				unset($definitions[$appname]);
+			}
 		}
-		foreach($definitions as $appname => $_types) {
+		foreach($definitions as $appname => $_types)
+		{
 			$definitions[$appname] = array_intersect_key($definitions[$appname], array_flip($types));
 		}
 		return count($definitions[$appname]) > 0;
 	}
 
 	// Api\Cache needs this public
-	public static function _has_definitions(Array $appnames, Array $types) {
+	public static function _has_definitions(Array $appnames, Array $types)
+	{
 		$def = new importexport_definitions_bo(array('application'=>$appnames, 'type' => $types));
 		$list = array();
-		foreach((array)$def->get_definitions() as $id) {
+		foreach((array)$def->get_definitions() as $id)
+		{
 			// Need to instanciate it to check, but if the user doesn't have permission, it throws an exception
 			try {
 				$definition = new importexport_definition($id);
-				if($def->is_permitted($definition->get_record_array())) {
+				if($def->is_permitted($definition->get_record_array()))
+				{
 					$list[$definition->application][$definition->type][] = $id;
 				}
-			} catch (Exception $e) {
+			}
+			catch (Exception $e) {
 				// That one doesn't work, keep going
 			}
 			$definition = null;
@@ -601,13 +712,22 @@ 
 			$plugin = is_object($plugin_name) ? $plugin_name : new $plugin_name();
 			$plugin_name = get_class($plugin);
 
-			if($record_classname == null) $record_classname = $plugin::get_egw_record_class();
-			if(!class_exists($record_classname)) throw new Exception('Bad class name ' . $record_classname);
+			if($record_classname == null)
+			{
+				$record_classname = $plugin::get_egw_record_class();
+			}
+			if(!class_exists($record_classname))
+			{
+				throw new Exception('Bad class name ' . $record_classname);
+			}
 
 			if(!$wizard_plugin)
 			{
 				$wizard_name = $app_name . '_wizard_' . str_replace($app_name . '_', '', $plugin_name);
-				if(!class_exists($wizard_name)) throw new Exception('Bad wizard name ' . $wizard_name);
+				if(!class_exists($wizard_name))
+				{
+					throw new Exception('Bad wizard name ' . $wizard_name);
+				}
 				$wizard_plugin = new $wizard_name;
 			}
 		}
@@ -627,7 +747,10 @@ 
 		foreach($record_classname::$types as $type => $type_fields)
 		{
 			// Only these for now, until filter methods for others are figured out
-			if(!in_array($type, array('select','select-cat','select-account','date','date-time'))) continue;
+			if(!in_array($type, array('select','select-cat','select-account','date','date-time')))
+			{
+				continue;
+			}
 			foreach($type_fields as $field_name)
 			{
 				$fields[$field_name] = array(
@@ -645,14 +768,18 @@ 
 			$fields['#'.$field_name] = $settings;
 		}
 
-		foreach($fields as $field_name => &$settings) {
+		foreach($fields as $field_name => &$settings)
+		{
 			// Can't really filter on these (or at least no generic, sane way figured out yet)
 			if(!is_array($settings) || in_array($settings['type'], array('text','button', 'label','url','url-email','url-phone','htmlarea')))
 			{
 				unset($fields[$field_name]);
 				continue;
 			}
-			if($settings['type'] == 'radio') $settings['type'] = 'select';
+			if($settings['type'] == 'radio')
+			{
+				$settings['type'] = 'select';
+			}
 			switch($settings['type'])
 			{
 				case 'checkbox':

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

@@ -445,7 +445,7 @@ 
 		return $plugins;
 	}
 
-	public static function _get_plugins(Array $appnames, Array $types) {
+	public static function _get_plugins(array $appnames, array $types) {
 		$plugins = array();
 		foreach ($appnames as $appname) {
 			if(array_key_exists($appname, self::$blacklist_files) && self::$blacklist_files[$appname] === true) continue;
@@ -562,7 +562,7 @@ 
 	}
 
 	// Api\Cache needs this public
-	public static function _has_definitions(Array $appnames, Array $types) {
+	public static function _has_definitions(array $appnames, array $types) {
 		$def = new importexport_definitions_bo(array('application'=>$appnames, 'type' => $types));
 		$list = array();
 		foreach((array)$def->get_definitions() as $id) {

importexport/inc/class.importexport_iface_egw_record.inc.php

Doc Comments +3 added lines

@@ -35,6 +35,7 @@ 
 	 * reads record from backend if identifier is given.
 	 *
 	 * @param string $_identifier
+	 * @return void
 	 */
 	public function __construct( $_identifier='' );
 	
@@ -50,6 +51,7 @@ 
 	 *
 	 * @param string $_attribute_name
 	 * @param data $data
+	 * @return void
 	 */
 	public function __set($_attribute_name, $data);
 	
@@ -126,6 +128,7 @@ 
 	/**
 	 * destructor
 	 *
+	 * @return void
 	 */
 	public function __destruct();
 

Spacing +5 added lines, -5 removed lines

@@ -36,7 +36,7 @@ 
 	 *
 	 * @param string $_identifier
 	 */
-	public function __construct( $_identifier='' );
+	public function __construct($_identifier = '');
 	
 	/**
 	 * magic method to set attributes of record
@@ -97,7 +97,7 @@ 
 	 *
 	 * @return string identifier
 	 */
-	public function save ( $_dst_identifier );
+	public function save($_dst_identifier);
 	
 	/**
 	 * copys current record to record identified by $_dst_identifier
@@ -105,7 +105,7 @@ 
 	 * @param string $_dst_identifier
 	 * @return string dst_identifier
 	 */
-	public function copy ( $_dst_identifier );
+	public function copy($_dst_identifier);
 	
 	/**
 	 * moves current record to record identified by $_dst_identifier
@@ -114,14 +114,14 @@ 
 	 * @param string $_dst_identifier
 	 * @return string dst_identifier
 	 */
-	public function move ( $_dst_identifier );
+	public function move($_dst_identifier);
 	
 	/**
 	 * delets current record from backend
 	 * @return void
 	 *
 	 */
-	public function delete ();
+	public function delete();
 	
 	/**
 	 * destructor

importexport/inc/class.importexport_iface_export_plugin.inc.php

Doc Comments +1 added lines, -1 removed lines

@@ -107,7 +107,7 @@
 	 * and automatic conversions.  In most cases they are named <appname>_egw_record,
 	 * but projectmanager is an exception to this.
 	 *
-	 * @return string;
+	 * @return string
 	 */
 	public static function get_egw_record_class();
 } // end of iface_export_plugin

Braces +2 added lines, -1 removed lines

@@ -34,7 +34,8 @@
  *
  * get_selector();  //returns array
  */
-interface importexport_iface_export_plugin {
+interface importexport_iface_export_plugin
+{
 	
 	/**
 	 * exports entries according to given definition object.

importexport/inc/class.importexport_iface_import_plugin.inc.php

Doc Comments +2 added lines, -2 removed lines

@@ -29,8 +29,8 @@
 	/**
 	 * imports entries according to given definition object.
 	 *
-	 * @param stram $_stram
-	 * @param definition $_definition
+	 * @param stram $_stream
+	 * @param importexport_definition $_definition
 	 * @return int number of successful imports
 	 */
 	public function import( $_stream, importexport_definition $_definition );

Indentation +21 added lines, -21 removed lines

@@ -88,33 +88,33 @@
 	public function get_selectors_etpl();
 
 	/**
-	* Returns errors that were encountered during importing
-	* Maximum of one error message per record, but you can concatenate them if you need to
-	*
-	* @return Array (
-	*	record_# => error message
-	*	)
-	*/
+	 * Returns errors that were encountered during importing
+	 * Maximum of one error message per record, but you can concatenate them if you need to
+	 *
+	 * @return Array (
+	 *	record_# => error message
+	 *	)
+	 */
 	public function get_errors();
 
 	/**
-	* Returns warnings that were encountered during importing
-	* Maximum of one warning message per record, but you can concatenate them if you need to
-	*
-	* @return Array (
-	*	record_# => warning message
-	*	)
-	*/
+	 * Returns warnings that were encountered during importing
+	 * Maximum of one warning message per record, but you can concatenate them if you need to
+	 *
+	 * @return Array (
+	 *	record_# => warning message
+	 *	)
+	 */
 	public function get_warnings();
 
 	/**
-	* Returns a list of actions taken, and the number of records for that action.
-	* Actions are things like 'insert', 'update', 'delete', and may be different for each plugin.
-	*
-	* @return Array (
-	*	action => record count
-	* )
-	*/
+	 * Returns a list of actions taken, and the number of records for that action.
+	 * Actions are things like 'insert', 'update', 'delete', and may be different for each plugin.
+	 *
+	 * @return Array (
+	 *	action => record count
+	 * )
+	 */
 	public function get_results();
 
 } // end of iface_export_plugin

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
 	 * @param definition $_definition
 	 * @return int number of successful imports
 	 */
-	public function import( $_stream, importexport_definition $_definition );
+	public function import($_stream, importexport_definition $_definition);
 
 	/**
 	 * Reads entries, and presents them back as they will be understood

Braces +2 added lines, -1 removed lines

@@ -24,7 +24,8 @@
  * So this interface just garanties the interaction with userinterfaces. It
  * has nothing to do with datatypes.
  */
-interface importexport_iface_import_plugin {
+interface importexport_iface_import_plugin
+{
 	
 	/**
 	 * imports entries according to given definition object.

importexport/inc/class.importexport_iface_import_record.inc.php

Doc Comments +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
 	 * cleanup
 	 *
 	 * @return 
-	 */
+	 void
 	public function __destruct( );
 
 	/**

Spacing +2 added lines, -2 removed lines

@@ -27,7 +27,7 @@ 
 	 * @param array $_options options for specific backends
 	 * @return bool
 	 */
-	public function __construct( $_stream, array $_options );
+	public function __construct($_stream, array $_options);
 
 	/**
 	 * cleanup
@@ -42,7 +42,7 @@ 
 	 * @param string _position may be: {first|last|next|previous|somenumber}
 	 * @return bool
 	 */
-	public function get_record( $_position = 'next' );
+	public function get_record($_position = 'next');
 
 	/**
 	 * Retruns total number of records for the open resource.

importexport/inc/class.importexport_import_csv.inc.php

Doc Comments +3 added lines, -3 removed lines

@@ -137,8 +137,8 @@ 
 	/**
 	 * updates $this->record
 	 *
-	 * @param mixed $_position
-	 * @return bool
+	 * @param string $_position
+	 * @return null|false
 	 */
 	private function get_raw_record( $_position = 'next' ) {
 		switch ($_position) {
@@ -244,7 +244,7 @@ 
 	/**
 	 * does conversions according to $this->conversion
 	 *
-	 * @return bool
+	 * @return boolean|null
 	 */
 	protected function do_conversions() {
 		if ( $record = importexport_helper_functions::conversion( $this->record, $this->conversion, $this->conversion_class )) {

Spacing +75 added lines, -75 removed lines

@@ -83,10 +83,10 @@ 
 	 * @param string $_resource resource containing data. May be each valid php-stream
 	 * @param array $_options options for the resource array with keys: charset and fieldsep
 	 */
-	public function __construct( $_resource,  array $_options ) {
+	public function __construct($_resource, array $_options) {
 		$this->resource = $_resource;
 		$this->csv_fieldsep = $_options['fieldsep'];
-		if($_options['charset'] == 'user') $_options['charset'] = $GLOBALS['egw_info']['user']['preferences']['common']['csv_charset'];
+		if ($_options['charset'] == 'user') $_options['charset'] = $GLOBALS['egw_info']['user']['preferences']['common']['csv_charset'];
 		$this->csv_charset = $_options['charset'];
 		return;
 	} // end of member function __construct
@@ -103,20 +103,20 @@ 
 	 * @param mixed _position may be: {current|first|last|next|previous|somenumber}
 	 * @return mixed array with data / false if no furtor records
 	 */
-	public function get_record( $_position = 'next' ) {
+	public function get_record($_position = 'next') {
 		
-		if ($this->get_raw_record( $_position ) === false) {
+		if ($this->get_raw_record($_position) === false) {
 			return false;
 		}
 		
 		// skip empty records
-		if( count( array_unique( $this->record ) ) < 2 ) return $this->get_record( $_position );
+		if (count(array_unique($this->record)) < 2) return $this->get_record($_position);
 		
-		if ( !empty( $this->conversion ) ) {
+		if (!empty($this->conversion)) {
 			$this->do_conversions();
 		}
 		
-		if ( !empty( $this->mapping ) ) {
+		if (!empty($this->mapping)) {
 			$this->do_fieldmapping();
 		}
 		
@@ -128,9 +128,9 @@ 
 	 *
 	 * @param int $_numToSkip
 	 */
-	public function skip_records( $_numToSkip ) {
-		while ( (int)$_numToSkip-- !== 0 ) {
-			fgetcsv( $this->resource, self::csv_max_linelength, $this->csv_fieldsep);
+	public function skip_records($_numToSkip) {
+		while ((int)$_numToSkip-- !== 0) {
+			fgetcsv($this->resource, self::csv_max_linelength, $this->csv_fieldsep);
 		}
 	}
 	
@@ -140,7 +140,7 @@ 
 	 * @param mixed $_position
 	 * @return bool
 	 */
-	private function get_raw_record( $_position = 'next' ) {
+	private function get_raw_record($_position = 'next') {
 		switch ($_position) {
 			case 'current' :
 				if ($this->current_position == 0) {
@@ -154,7 +154,7 @@ 
 				}
 				
 			case 'next' :
-				$csv_data = fgetcsv( $this->resource, self::csv_max_linelength, $this->csv_fieldsep);
+				$csv_data = fgetcsv($this->resource, self::csv_max_linelength, $this->csv_fieldsep);
 				if (!is_array($csv_data)) {
 					return false;
 				}
@@ -235,7 +235,7 @@ 
 		$this->record = array();
 		foreach ($this->mapping as $cvs_idx => $new_idx)
 		{
-			if( $new_idx == '' ) continue;
+			if ($new_idx == '') continue;
 			$this->record[$new_idx] = $record[$cvs_idx];
 		}
 		return true;
@@ -247,7 +247,7 @@ 
 	 * @return bool
 	 */
 	protected function do_conversions() {
-		if ( $record = importexport_helper_functions::conversion( $this->record, $this->conversion, $this->conversion_class )) {
+		if ($record = importexport_helper_functions::conversion($this->record, $this->conversion, $this->conversion_class)) {
 			$this->record = $record;
 			return;
 		}
@@ -264,25 +264,25 @@ 
 	 *
 	 * @return string warnings, if any
 	 */
-	public static function convert(Array &$record, Array $fields = array(), $appname = null, Array $selects = array(), $format=0) {
+	public static function convert(Array &$record, Array $fields = array(), $appname = null, Array $selects = array(), $format = 0) {
 		$warnings = array();
 
 		// Automatic conversions
-		if($appname) {
+		if ($appname) {
 
 			// Load translations
 			Api\Translation::add_app($appname);
 
-			if(!self::$cf_parse_cache[$appname]) {
+			if (!self::$cf_parse_cache[$appname]) {
 				$c_fields = importexport_export_csv::convert_parse_custom_fields($appname, $selects, $links, $methods);
 				self::$cf_parse_cache[$appname] = array($c_fields, $selects, $links, $methods);
 			}
 			list($c_fields, $c_selects, $links, $methods) = self::$cf_parse_cache[$appname];
 
 			// Add in any fields that are keys to another app
-			foreach((array)$fields['links'] as $link_field => $app)
+			foreach ((array)$fields['links'] as $link_field => $app)
 			{
-				if(is_numeric($link_field)) continue;
+				if (is_numeric($link_field)) continue;
 				$links[$link_field] = $app;
 				// Set it as a normal link field
 				$fields['links'][] = $link_field;
@@ -290,9 +290,9 @@ 
 			}
 
 			// Not quite a recursive merge, since only one level
-			foreach($fields as $type => &$list)
+			foreach ($fields as $type => &$list)
 			{
-				if($c_fields[$type]) {
+				if ($c_fields[$type]) {
 					$list = array_merge($c_fields[$type], $list);
 					unset($c_fields[$type]);
 				}
@@ -300,36 +300,36 @@ 
 			$fields += $c_fields;
 			$selects += $c_selects;
 		}
-		if($fields) {
-			foreach((array)$fields['select'] as $name) {
-				if($record[$name] != null && is_array($selects) && $selects[$name]) {
-					$key = array_search(strtolower($record[$name]), array_map('strtolower',$selects[$name]));
-					if($key !== false)
+		if ($fields) {
+			foreach ((array)$fields['select'] as $name) {
+				if ($record[$name] != null && is_array($selects) && $selects[$name]) {
+					$key = array_search(strtolower($record[$name]), array_map('strtolower', $selects[$name]));
+					if ($key !== false)
 					{
 						$record[$name] = $key;
 					}
 					else
 					{
-						$key = array_search(strtolower($record[$name]), array_map('strtolower',array_map('lang',$selects[$name])));
-						if($key !== false) $record[$name] = $key;
+						$key = array_search(strtolower($record[$name]), array_map('strtolower', array_map('lang', $selects[$name])));
+						if ($key !== false) $record[$name] = $key;
 					}
 				}
 			}
-			foreach((array)$fields['links'] as $name) {
-				if($record[$name] && $links[$name])
+			foreach ((array)$fields['links'] as $name) {
+				if ($record[$name] && $links[$name])
 				{
 					// Primary key to another app, not a link
 					// Text - search for a matching record
-					if(!is_numeric($record[$name]))
+					if (!is_numeric($record[$name]))
 					{
 						$results = Link::query($links[$name], $record[$name]);
-						if(count($results) >= 1)
+						if (count($results) >= 1)
 						{
 							// More than 1 result.  Check for exact match
 							$exact_count = 0;
-							foreach($results as $id => $title)
+							foreach ($results as $id => $title)
 							{
-								if($title == $record[$name])
+								if ($title == $record[$name])
 								{
 									$exact_count++;
 									$app_id = $id;
@@ -338,11 +338,11 @@ 
 								unset($results[$id]);
 							}
 							// Too many exact matches, or none good enough
-							if($exact_count > 1 || count($results) == 0)
+							if ($exact_count > 1 || count($results) == 0)
 							{
 								$warnings[] = lang('Unable to link to %1 "%2"',
 									lang($links[$name]), $record[$name]).
- 									' - ' .lang('too many matches');
+ 									' - '.lang('too many matches');
 								continue;
 							}
 							elseif ($exact_count == 1)
@@ -355,7 +355,7 @@ 
 						{
 							$warnings[] = lang('Unable to link to %1 "%2"',
 								lang($links[$name]), $record[$name]).
- 								' - ' . lang('no matches');
+ 								' - '.lang('no matches');
 							continue;
 						} else {
 							$record[$name] = key($results);
@@ -363,17 +363,17 @@ 
 					}
 				}
 			}
-			foreach((array)$fields['select-account'] as $name) {
+			foreach ((array)$fields['select-account'] as $name) {
 				// Compare against null to deal with empty arrays
 				if ($record[$name]) {
 					// Automatically handle text owner without explicit translation
 					$new_owner = importexport_helper_functions::account_name2id($record[$name]);
-					if(count($new_owner) != count(explode(',',$record[$name])))
+					if (count($new_owner) != count(explode(',', $record[$name])))
 					{
 						// Unable to parse value into account
-						$warnings[] = $name . ': ' .lang('%1 is not a known user or group', $record[$name]);
+						$warnings[] = $name.': '.lang('%1 is not a known user or group', $record[$name]);
 					}
-					if($new_owner != '') {
+					if ($new_owner != '') {
 						$record[$name] = $new_owner;
 					} else {
 						// Clear it to prevent trouble later on
@@ -381,24 +381,24 @@ 
 					}
 				}
 			}
-			foreach((array)$fields['select-bool'] as $name) {
-				if($record[$name] != null && $record[$name] != '') {
+			foreach ((array)$fields['select-bool'] as $name) {
+				if ($record[$name] != null && $record[$name] != '') {
 					$record[$name] = ($record[$name] == lang('Yes') || $record[$name] == '1' ? 1 : 0);
 				}
 			}
-			foreach((array)$fields['date-time'] as $name) {
+			foreach ((array)$fields['date-time'] as $name) {
 				if (isset($record[$name]) && !is_numeric($record[$name]) && strlen(trim($record[$name])) > 0)
 				{
 					// Need to handle format first
-					if($format == 1)
+					if ($format == 1)
 					{
 						$formatted = Api\DateTime::createFromFormat(
-							'!'.Api\DateTime::$user_dateformat . ' ' .Api\DateTime::$user_timeformat,
+							'!'.Api\DateTime::$user_dateformat.' '.Api\DateTime::$user_timeformat,
 							$record[$name],
 							Api\DateTime::$user_timezone
 						);
 
-						if(!$formatted && $errors = Api\DateTime::getLastErrors())
+						if (!$formatted && $errors = Api\DateTime::getLastErrors())
 						{
 							// Try again, without time
 							$formatted = Api\DateTime::createFromFormat(
@@ -407,70 +407,70 @@ 
 								Api\DateTime::$user_timezone
 							);
 							
-							if(!$formatted && $errors = Api\DateTime::getLastErrors())
+							if (!$formatted && $errors = Api\DateTime::getLastErrors())
 							{
 								// Try again, anything goes
 								try {
 									$formatted = new Api\DateTime($record[$name]);
 								} catch (Exception $e) {
-									$warnings[] = $name.': ' . $e->getMessage() . "\n" .
-										'Format: '.'!'.Api\DateTime::$user_dateformat . ' ' .Api\DateTime::$user_timeformat;
+									$warnings[] = $name.': '.$e->getMessage()."\n".
+										'Format: '.'!'.Api\DateTime::$user_dateformat.' '.Api\DateTime::$user_timeformat;
 									continue;
 								}
 								$errors = Api\DateTime::getLastErrors();
-								foreach($errors['errors'] as $char => $msg)
+								foreach ($errors['errors'] as $char => $msg)
 								{
 									$warnings[] = "$name: [$char] $msg\n".
-										'Format: '.'!'.Api\DateTime::$user_dateformat . ' ' .Api\DateTime::$user_timeformat;
+										'Format: '.'!'.Api\DateTime::$user_dateformat.' '.Api\DateTime::$user_timeformat;
 								}
 							}
 						}
-						if($formatted)
+						if ($formatted)
 						{
 							$record[$name] = $formatted->getTimestamp();
 							// Timestamp is apparently in server time, but apps will do the same conversion
-							$record[$name] = Api\DateTime::server2user($record[$name],'ts');
+							$record[$name] = Api\DateTime::server2user($record[$name], 'ts');
 						}
 					}
 					
-					if(is_array(self::$cf_parse_cache[$appname][0]['date-time']) &&
+					if (is_array(self::$cf_parse_cache[$appname][0]['date-time']) &&
 							in_array($name, self::$cf_parse_cache[$appname][0]['date-time'])) {
 						// Custom fields stored in a particular format (from customfields_widget)
 						$record[$name] = date('Y-m-d H:i:s', $record[$name]);
 					}
 				}
-				if(array_key_exists($name, $record) && strlen(trim($record[$name])) == 0)
+				if (array_key_exists($name, $record) && strlen(trim($record[$name])) == 0)
 				{
 					$record[$name] = null;
 				}
 			}
-			foreach((array)$fields['date'] as $name) {
+			foreach ((array)$fields['date'] as $name) {
 				if (isset($record[$name]) && !is_numeric($record[$name]) && strlen(trim($record[$name])) > 0)
 				{
 					// Need to handle format first
-					if($format == 1)
+					if ($format == 1)
 					{
 						$formatted = Api\DateTime::createFromFormat('!'.Api\DateTime::$user_dateformat, $record[$name]);
-						if($formatted && $errors = Api\DateTime::getLastErrors() && $errors['error_count'] == 0)
+						if ($formatted && $errors = Api\DateTime::getLastErrors() && $errors['error_count'] == 0)
 						{
 							$record[$name] = $formatted->getTimestamp();
 						}
 					}
-					$record[$name] = Api\DateTime::server2user($record[$name],'ts');
-					if(is_array(self::$cf_parse_cache[$appname][0]['date']) &&
+					$record[$name] = Api\DateTime::server2user($record[$name], 'ts');
+					if (is_array(self::$cf_parse_cache[$appname][0]['date']) &&
 							in_array($name, self::$cf_parse_cache[$appname][0]['date'])) {
 						// Custom fields stored in a particular format (from customfields_widget)
 						$record[$name] = date('Y-m-d', $record[$name]);
 					}
 				}
-				if(array_key_exists($name, $record) && strlen(trim($record[$name])) == 0)
+				if (array_key_exists($name, $record) && strlen(trim($record[$name])) == 0)
 				{
 					$record[$name] = null;
 				}
 			}
-			foreach((array)$fields['float'] as $name)
+			foreach ((array)$fields['float'] as $name)
 			{
-				if($record[$name] != null && $record[$name] != '') {
+				if ($record[$name] != null && $record[$name] != '') {
 					$dec_point = $GLOBALS['egw_info']['user']['preferences']['common']['number_format'][0];
 					if (empty($dec_point)) $dec_point = '.';
 					$record[$name] = floatval(str_replace($dec_point, '.', preg_replace('/[^\d'.preg_quote($dec_point).']/', '', $record[$name])));
@@ -478,23 +478,23 @@ 
 			}
 
 			// Some custom methods for conversion
-			foreach((array)$methods as $name => $method) {
-				if($record[$name]) $record[$name] = ExecMethod($method, $record[$name]);
+			foreach ((array)$methods as $name => $method) {
+				if ($record[$name]) $record[$name] = ExecMethod($method, $record[$name]);
 			}
 
 			// cat_name2id will use currentapp to create new categories
 			$current_app = $GLOBALS['egw_info']['flags']['currentapp'];
-			if($appname) {
+			if ($appname) {
 				$GLOBALS['egw_info']['flags']['currentapp'] = $appname;
 			}
-			$categories = new Api\Categories('',$appname);
-			foreach((array)$fields['select-cat'] as $name) {
-				if($record[$name]) {
+			$categories = new Api\Categories('', $appname);
+			foreach ((array)$fields['select-cat'] as $name) {
+				if ($record[$name]) {
 					// Only parse name if it needs it
-					if($format == 1)
+					if ($format == 1)
 					{
-						$existing_cat = $categories->exists('all',$record[$name]);
-						if($existing_cat)
+						$existing_cat = $categories->exists('all', $record[$name]);
+						if ($existing_cat)
 						{
 							$cat_id = $existing_cat;
 						}
@@ -503,14 +503,14 @@ 
 							$cat_id = importexport_helper_functions::cat_name2id($record[$name]);
 						}
 						// Don't clear it if it wasn't found
-						if($cat_id) $record[$name] = $cat_id;
+						if ($cat_id) $record[$name] = $cat_id;
 					}
 				}
 			}
 			$GLOBALS['egw_info']['flags']['currentapp'] = $current_app;
 		}
 
-		return implode("\n",$warnings);
+		return implode("\n", $warnings);
 	}
 } // end of import_csv
 ?>

Braces +143 added lines, -61 removed lines

@@ -22,7 +22,9 @@ 
  * @todo Throw away spechial chars and trim() entries ?
  * @todo Check for XSS like userinput! (see common_functions)
  */
-class importexport_import_csv implements importexport_iface_import_record { //, Iterator {
+class importexport_import_csv implements importexport_iface_import_record
+{
+//, Iterator {
 
 	const csv_max_linelength = 8000;
 	
@@ -83,10 +85,14 @@ 
 	 * @param string $_resource resource containing data. May be each valid php-stream
 	 * @param array $_options options for the resource array with keys: charset and fieldsep
 	 */
-	public function __construct( $_resource,  array $_options ) {
+	public function __construct( $_resource,  array $_options )
+	{
 		$this->resource = $_resource;
 		$this->csv_fieldsep = $_options['fieldsep'];
-		if($_options['charset'] == 'user') $_options['charset'] = $GLOBALS['egw_info']['user']['preferences']['common']['csv_charset'];
+		if($_options['charset'] == 'user')
+		{
+			$_options['charset'] = $GLOBALS['egw_info']['user']['preferences']['common']['csv_charset'];
+		}
 		$this->csv_charset = $_options['charset'];
 		return;
 	} // end of member function __construct
@@ -94,7 +100,8 @@ 
 	/**
 	 * cleanup
 	 */
-	public function __destruct( ) {
+	public function __destruct( )
+	{
 	} // end of member function __destruct
 
 	/**
@@ -103,20 +110,27 @@ 
 	 * @param mixed _position may be: {current|first|last|next|previous|somenumber}
 	 * @return mixed array with data / false if no furtor records
 	 */
-	public function get_record( $_position = 'next' ) {
+	public function get_record( $_position = 'next' )
+	{
 		
-		if ($this->get_raw_record( $_position ) === false) {
+		if ($this->get_raw_record( $_position ) === false)
+		{
 			return false;
 		}
 		
 		// skip empty records
-		if( count( array_unique( $this->record ) ) < 2 ) return $this->get_record( $_position );
+		if( count( array_unique( $this->record ) ) < 2 )
+		{
+			return $this->get_record( $_position );
+		}
 		
-		if ( !empty( $this->conversion ) ) {
+		if ( !empty( $this->conversion ) )
+		{
 			$this->do_conversions();
 		}
 		
-		if ( !empty( $this->mapping ) ) {
+		if ( !empty( $this->mapping ) )
+		{
 			$this->do_fieldmapping();
 		}
 		
@@ -128,8 +142,10 @@ 
 	 *
 	 * @param int $_numToSkip
 	 */
-	public function skip_records( $_numToSkip ) {
-		while ( (int)$_numToSkip-- !== 0 ) {
+	public function skip_records( $_numToSkip )
+	{
+		while ( (int)$_numToSkip-- !== 0 )
+		{
 			fgetcsv( $this->resource, self::csv_max_linelength, $this->csv_fieldsep);
 		}
 	}
@@ -140,22 +156,27 @@ 
 	 * @param mixed $_position
 	 * @return bool
 	 */
-	private function get_raw_record( $_position = 'next' ) {
-		switch ($_position) {
+	private function get_raw_record( $_position = 'next' )
+	{
+		switch ($_position)
+		{
 			case 'current' :
-				if ($this->current_position == 0) {
+				if ($this->current_position == 0)
+				{
 					return;
 				}
 				break;
 			case 'first' :
-				if (!$this->current_position == 0) {
+				if (!$this->current_position == 0)
+				{
 					$this->current_position = 0;
 					rewind($this->resource);
 				}
 				
 			case 'next' :
 				$csv_data = fgetcsv( $this->resource, self::csv_max_linelength, $this->csv_fieldsep);
-				if (!is_array($csv_data)) {
+				if (!is_array($csv_data))
+				{
 					return false;
 				}
 				$this->current_position++;
@@ -163,33 +184,41 @@ 
 				break;
 				
 			case 'previous' :
-				if ($this->current_position < 2) {
+				if ($this->current_position < 2)
+				{
 					throw new Exception('Error: There is no previous record!');
 				}
 				$final_position = --$this->current_position;
 				$this->current_position = 0;
 				rewind($this->resource);
-				while ($this->current_position !== $final_position) {
+				while ($this->current_position !== $final_position)
+				{
 					$this->get_raw_record();
 				}
 				break;
 				
 			case 'last' :
-				while ($this->get_raw_record()) {}
+				while ($this->get_raw_record())
+				{
+}
 				break;
 				
 			default: //somenumber
-				if (!is_int($_position)) {
+				if (!is_int($_position))
+				{
 					throw new Exception('Error: $position must be one of {current|first|last|next|previous} or an integer value');
 				}
-				if ($_position == $this->current_position) {
+				if ($_position == $this->current_position)
+				{
 					break;
 				}
-				elseif ($_position < $this->current_position) {
+				elseif ($_position < $this->current_position)
+				{
 					$this->current_position = 0;
 					rewind($this->resource);
 				}
-				while ($this->current_position !== $_position) {
+				while ($this->current_position !== $_position)
+				{
 					$this->get_raw_record();
 				}
 				break;				
@@ -202,12 +231,16 @@ 
 	 *
 	 * @return int
 	 */
-	public function get_num_of_records( ) {
-		if ($this->num_of_records > 0) {
+	public function get_num_of_records( )
+	{
+		if ($this->num_of_records > 0)
+		{
 			return $this->num_of_records;
 		}
 		$current_position = $this->current_position;
-		while ($this->get_raw_record()) {}
+		while ($this->get_raw_record())
+		{
+}
 		$this->num_of_records = $this->current_position;
 		$this->get_record($current_position);
 		return $this->num_of_records;
@@ -218,7 +251,8 @@ 
 	 *
 	 * @return int
 	 */
-	public function get_current_position( ) {
+	public function get_current_position( )
+	{
 		
 		return $this->current_position;
 		
@@ -230,12 +264,16 @@ 
 	 *
 	 * @return
 	 */
-	protected function do_fieldmapping( ) {
+	protected function do_fieldmapping( )
+	{
 		$record = $this->record;
 		$this->record = array();
 		foreach ($this->mapping as $cvs_idx => $new_idx)
 		{
-			if( $new_idx == '' ) continue;
+			if( $new_idx == '' )
+			{
+				continue;
+			}
 			$this->record[$new_idx] = $record[$cvs_idx];
 		}
 		return true;
@@ -246,8 +284,10 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function do_conversions() {
-		if ( $record = importexport_helper_functions::conversion( $this->record, $this->conversion, $this->conversion_class )) {
+	protected function do_conversions()
+	{
+		if ( $record = importexport_helper_functions::conversion( $this->record, $this->conversion, $this->conversion_class ))
+		{
 			$this->record = $record;
 			return;
 		}
@@ -264,16 +304,19 @@ 
 	 *
 	 * @return string warnings, if any
 	 */
-	public static function convert(Array &$record, Array $fields = array(), $appname = null, Array $selects = array(), $format=0) {
+	public static function convert(Array &$record, Array $fields = array(), $appname = null, Array $selects = array(), $format=0)
+	{
 		$warnings = array();
 
 		// Automatic conversions
-		if($appname) {
+		if($appname)
+		{
 
 			// Load translations
 			Api\Translation::add_app($appname);
 
-			if(!self::$cf_parse_cache[$appname]) {
+			if(!self::$cf_parse_cache[$appname])
+			{
 				$c_fields = importexport_export_csv::convert_parse_custom_fields($appname, $selects, $links, $methods);
 				self::$cf_parse_cache[$appname] = array($c_fields, $selects, $links, $methods);
 			}
@@ -282,7 +325,10 @@ 
 			// Add in any fields that are keys to another app
 			foreach((array)$fields['links'] as $link_field => $app)
 			{
-				if(is_numeric($link_field)) continue;
+				if(is_numeric($link_field))
+				{
+					continue;
+				}
 				$links[$link_field] = $app;
 				// Set it as a normal link field
 				$fields['links'][] = $link_field;
@@ -292,7 +338,8 @@ 
 			// Not quite a recursive merge, since only one level
 			foreach($fields as $type => &$list)
 			{
-				if($c_fields[$type]) {
+				if($c_fields[$type])
+				{
 					$list = array_merge($c_fields[$type], $list);
 					unset($c_fields[$type]);
 				}
@@ -300,9 +347,12 @@ 
 			$fields += $c_fields;
 			$selects += $c_selects;
 		}
-		if($fields) {
-			foreach((array)$fields['select'] as $name) {
-				if($record[$name] != null && is_array($selects) && $selects[$name]) {
+		if($fields)
+		{
+			foreach((array)$fields['select'] as $name)
+			{
+				if($record[$name] != null && is_array($selects) && $selects[$name])
+				{
 					$key = array_search(strtolower($record[$name]), array_map('strtolower',$selects[$name]));
 					if($key !== false)
 					{
@@ -311,11 +361,15 @@ 
 					else
 					{
 						$key = array_search(strtolower($record[$name]), array_map('strtolower',array_map('lang',$selects[$name])));
-						if($key !== false) $record[$name] = $key;
+						if($key !== false)
+						{
+							$record[$name] = $key;
+						}
 					}
 				}
 			}
-			foreach((array)$fields['links'] as $name) {
+			foreach((array)$fields['links'] as $name)
+			{
 				if($record[$name] && $links[$name])
 				{
 					// Primary key to another app, not a link
@@ -357,15 +411,19 @@ 
 								lang($links[$name]), $record[$name]).
  								' - ' . lang('no matches');
 							continue;
-						} else {
+						}
+						else
+						{
 							$record[$name] = key($results);
 						}
 					}
 				}
 			}
-			foreach((array)$fields['select-account'] as $name) {
+			foreach((array)$fields['select-account'] as $name)
+			{
 				// Compare against null to deal with empty arrays
-				if ($record[$name]) {
+				if ($record[$name])
+				{
 					// Automatically handle text owner without explicit translation
 					$new_owner = importexport_helper_functions::account_name2id($record[$name]);
 					if(count($new_owner) != count(explode(',',$record[$name])))
@@ -373,20 +431,26 @@ 
 						// Unable to parse value into account
 						$warnings[] = $name . ': ' .lang('%1 is not a known user or group', $record[$name]);
 					}
-					if($new_owner != '') {
+					if($new_owner != '')
+					{
 						$record[$name] = $new_owner;
-					} else {
+					}
+					else
+					{
 						// Clear it to prevent trouble later on
 						$record[$name] = '';
 					}
 				}
 			}
-			foreach((array)$fields['select-bool'] as $name) {
-				if($record[$name] != null && $record[$name] != '') {
+			foreach((array)$fields['select-bool'] as $name)
+			{
+				if($record[$name] != null && $record[$name] != '')
+				{
 					$record[$name] = ($record[$name] == lang('Yes') || $record[$name] == '1' ? 1 : 0);
 				}
 			}
-			foreach((array)$fields['date-time'] as $name) {
+			foreach((array)$fields['date-time'] as $name)
+			{
 				if (isset($record[$name]) && !is_numeric($record[$name]) && strlen(trim($record[$name])) > 0)
 				{
 					// Need to handle format first
@@ -412,7 +476,8 @@ 
 								// Try again, anything goes
 								try {
 									$formatted = new Api\DateTime($record[$name]);
-								} catch (Exception $e) {
+								}
+								catch (Exception $e) {
 									$warnings[] = $name.': ' . $e->getMessage() . "\n" .
 										'Format: '.'!'.Api\DateTime::$user_dateformat . ' ' .Api\DateTime::$user_timeformat;
 									continue;
@@ -434,7 +499,8 @@ 
 					}
 					
 					if(is_array(self::$cf_parse_cache[$appname][0]['date-time']) &&
-							in_array($name, self::$cf_parse_cache[$appname][0]['date-time'])) {
+							in_array($name, self::$cf_parse_cache[$appname][0]['date-time']))
+					{
 						// Custom fields stored in a particular format (from customfields_widget)
 						$record[$name] = date('Y-m-d H:i:s', $record[$name]);
 					}
@@ -444,7 +510,8 @@ 
 					$record[$name] = null;
 				}
 			}
-			foreach((array)$fields['date'] as $name) {
+			foreach((array)$fields['date'] as $name)
+			{
 				if (isset($record[$name]) && !is_numeric($record[$name]) && strlen(trim($record[$name])) > 0)
 				{
 					// Need to handle format first
@@ -458,7 +525,8 @@ 
 					}
 					$record[$name] = Api\DateTime::server2user($record[$name],'ts');
 					if(is_array(self::$cf_parse_cache[$appname][0]['date']) &&
-							in_array($name, self::$cf_parse_cache[$appname][0]['date'])) {
+							in_array($name, self::$cf_parse_cache[$appname][0]['date']))
+					{
 						// Custom fields stored in a particular format (from customfields_widget)
 						$record[$name] = date('Y-m-d', $record[$name]);
 					}
@@ -470,26 +538,37 @@ 
 			}
 			foreach((array)$fields['float'] as $name)
 			{
-				if($record[$name] != null && $record[$name] != '') {
+				if($record[$name] != null && $record[$name] != '')
+				{
 					$dec_point = $GLOBALS['egw_info']['user']['preferences']['common']['number_format'][0];
-					if (empty($dec_point)) $dec_point = '.';
+					if (empty($dec_point))
+					{
+						$dec_point = '.';
+					}
 					$record[$name] = floatval(str_replace($dec_point, '.', preg_replace('/[^\d'.preg_quote($dec_point).']/', '', $record[$name])));
 				}
 			}
 
 			// Some custom methods for conversion
-			foreach((array)$methods as $name => $method) {
-				if($record[$name]) $record[$name] = ExecMethod($method, $record[$name]);
+			foreach((array)$methods as $name => $method)
+			{
+				if($record[$name])
+				{
+					$record[$name] = ExecMethod($method, $record[$name]);
+				}
 			}
 
 			// cat_name2id will use currentapp to create new categories
 			$current_app = $GLOBALS['egw_info']['flags']['currentapp'];
-			if($appname) {
+			if($appname)
+			{
 				$GLOBALS['egw_info']['flags']['currentapp'] = $appname;
 			}
 			$categories = new Api\Categories('',$appname);
-			foreach((array)$fields['select-cat'] as $name) {
-				if($record[$name]) {
+			foreach((array)$fields['select-cat'] as $name)
+			{
+				if($record[$name])
+				{
 					// Only parse name if it needs it
 					if($format == 1)
 					{
@@ -503,7 +582,10 @@ 
 							$cat_id = importexport_helper_functions::cat_name2id($record[$name]);
 						}
 						// Don't clear it if it wasn't found
-						if($cat_id) $record[$name] = $cat_id;
+						if($cat_id)
+						{
+							$record[$name] = $cat_id;
+						}
 					}
 				}
 			}

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

@@ -264,7 +264,7 @@
 	 *
 	 * @return string warnings, if any
 	 */
-	public static function convert(Array &$record, Array $fields = array(), $appname = null, Array $selects = array(), $format=0) {
+	public static function convert(array &$record, array $fields = array(), $appname = null, array $selects = array(), $format=0) {
 		$warnings = array();
 
 		// Automatic conversions