EGroupware / egroupware

infolog/inc/class.infolog_export_csv.inc.php

Braces +56 added lines, -16 removed lines

@@ -18,7 +18,8 @@ 
 {
 
 
-	public function __construct() {
+	public function __construct()
+	{
 		translation::add_app('infolog');
 		$this->bo = new infolog_bo();
 		$this->get_selects();
@@ -50,7 +51,8 @@ 
 			case 'all':
 				// do we need to query the cf's
 				$query['custom_fields'] = false;
-				foreach($options['mapping'] + (array)$_definition->filter as $field => $map) {
+				foreach($options['mapping'] + (array)$_definition->filter as $field => $map)
+				{
 					if($field[0] == '#')
 					{
 						$query['custom_fields'] = true;
@@ -76,11 +78,20 @@ 
 					// Handle ranges
 					foreach($query['col_filter'] as $field => $value)
 					{
-						if(!is_array($value) || (!$value['from'] && !$value['to'])) continue;
+						if(!is_array($value) || (!$value['from'] && !$value['to']))
+						{
+							continue;
+						}
 
 						// Ranges are inclusive, so should be provided that way (from 2 to 10 includes 2 and 10)
-						if($value['from']) $query['col_filter'][] = "$field >= " . (int)$value['from'];
-						if($value['to']) $query['col_filter'][] = "$field <= " . (int)$value['to'];
+						if($value['from'])
+						{
+							$query['col_filter'][] = "$field >= " . (int)$value['from'];
+						}
+						if($value['to'])
+						{
+							$query['col_filter'][] = "$field <= " . (int)$value['to'];
+						}
 						unset($query['col_filter'][$field]);
 					}
 				}
@@ -97,9 +108,13 @@ 
 					{
 						foreach($selection as &$row)
 						{
-							if($row[$field]) $cf_preload[$app][] = $row[$field];
+							if($row[$field])
+							{
+								$cf_preload[$app][] = $row[$field];
+							}
 						}
-						if($cf_preload[$app]){
+						if($cf_preload[$app])
+						{
 							 $selects[$field] = egw_link::titles($app, $cf_preload[$app]);
 							//error_log('Preload ' . $field . '['.$app . ']: ' . implode(',',$cf_preload[$app]));
 						}
@@ -125,10 +140,17 @@ 
 		if($ids && $options['mapping']['info_link_id'])
 		{
 			$links = egw_link::get_links_multiple('infolog', $ids, true, '!'.egw_link::VFS_APPNAME);
-			foreach($links as $id => $link) {
-				if(!is_array($selection[$id])) break;
+			foreach($links as $id => $link)
+			{
+				if(!is_array($selection[$id]))
+				{
+					break;
+				}
 				$selection[$id]['info_link_id'] = $link;
-				if($options['convert']) $selection[$id]['info_link_id'] = egw_link::title($link['app'], $link['id']);
+				if($options['convert'])
+				{
+					$selection[$id]['info_link_id'] = egw_link::title($link['app'], $link['id']);
+				}
 			}
 		}
 		// If exporting PM fields, pre-load them all at once
@@ -137,7 +159,10 @@ 
 			$projects = egw_link::get_links_multiple('infolog', $ids, true, 'projectmanager');
 			foreach($projects as $id => $links)
 			{
-				if(!is_array($selection[$id])) break;
+				if(!is_array($selection[$id]))
+				{
+					break;
+				}
 				$selection[$id]['pm_id'] = current($links);
 				$selection[$id]['project'] = egw_link::title('projectmanager', $selection[$id]['pm_id']);
 				$this->selects['pl_id'] = ExecMethod('projectmanager.projectmanager_pricelist_bo.pricelist',$selection[$id]['pm_id']);
@@ -149,7 +174,10 @@ 
 			if(!is_array($_identifier))
 			{
 				$record = new infolog_egw_record($_identifier);
-				if($link = $links[$record->info_id]) $record->info_link_id = $options['convert'] ? egw_link::title($link['app'], $link['id']) : $link;
+				if($link = $links[$record->info_id])
+				{
+					$record->info_link_id = $options['convert'] ? egw_link::title($link['app'], $link['id']) : $link;
+				}
 				if($project = $projects[$record->info_id])
 				{
 					$record->pm_id = current($project);
@@ -171,7 +199,10 @@ 
 				// Force 0 times to ''
 				foreach(array('info_planned_time', 'info_used_time', 'info_replanned_time') as $field)
 				{
-					if($record->$field == 0) $record->$field = '';
+					if($record->$field == 0)
+					{
+						$record->$field = '';
+					}
 				}
 			}
 			else
@@ -179,7 +210,10 @@ 
 				// Implode arrays, so they don't say 'Array'
 				foreach($record->get_record_array() as $key => $value)
 				{
-					if(is_array($value)) $record->$key = implode(',', $value);
+					if(is_array($value))
+					{
+						$record->$key = implode(',', $value);
+					}
 				}
 			}
 			$export_object->export_record($record);
@@ -268,10 +302,16 @@ 
 	{
 		foreach($filters as $field_name => &$settings)
 		{
-			if($this->selects[$field_name]) $settings['values'] = $this->selects[$field_name];
+			if($this->selects[$field_name])
+			{
+				$settings['values'] = $this->selects[$field_name];
+			}
 			
 			// Infolog can't handle ranges in custom fields due to the way searching is done.
-			if(strpos($field_name, '#') === 0 && strpos($settings['type'],'date') === 0) unset($filters[$field_name]);
+			if(strpos($field_name, '#') === 0 && strpos($settings['type'],'date') === 0)
+			{
+				unset($filters[$field_name]);
+			}
 		}
 	}