EGroupware / egroupware

importexport/inc/class.importexport_export_ui.inc.php

Spacing +78 added lines, -78 removed lines

@@ -34,47 +34,47 @@ 
 
 	public function __construct() {
 		$this->js = $GLOBALS['egw']->js = is_object($GLOBALS['egw']->js) ? $GLOBALS['egw']->js : CreateObject('phpgwapi.javascript');
-		$this->js->validate_file('.','export_dialog','importexport');
-		$this->js->validate_file('.','importexport','importexport');
+		$this->js->validate_file('.', 'export_dialog', 'importexport');
+		$this->js->validate_file('.', 'importexport', 'importexport');
 		$this->user = $GLOBALS['egw_info']['user']['user_id'];
-		$this->export_plugins = importexport_helper_functions::get_plugins('all','export');
+		$this->export_plugins = importexport_helper_functions::get_plugins('all', 'export');
 		$GLOBALS['egw_info']['flags']['include_xajax'] = true;
 
 	}
 
-	public function export_dialog($_content=array()) {
+	public function export_dialog($_content = array()) {
 		$tabs = 'general_tab|selection_tab|options_tab';
 		$sel_options = array();
 		$readonlys = array();
 		$preserv = array();
 
-		$et = new etemplate_new(self::_appname. '.export_dialog');
+		$et = new etemplate_new(self::_appname.'.export_dialog');
 		$_appname = $_content['appname'] ? $_content['appname'] : $_GET['appname'];
 		$_definition = $_content['definition'] ? $_content['definition'] : $_GET['definition'];
 		$_plugin = $_content['plugin'] ? $_content['plugin'] : $_GET['plugin'];
 		// Select all from context menu, means use all search results, not just selected
-		if($_GET['select_all'] == 'true') $_GET['selection'] = 'search';
+		if ($_GET['select_all'] == 'true') $_GET['selection'] = 'search';
 		$_selection = $_content['selection'] ? $_content['selection'] : $_GET['selection'];
-		if($_GET['selection'] || $_content['selection_passed']) $content['selection_passed'] = $preserv['selection_passed'] = true;
+		if ($_GET['selection'] || $_content['selection_passed']) $content['selection_passed'] = $preserv['selection_passed'] = true;
 
 		// Check global setting
-		if(!bo_merge::is_export_limit_excepted()) {
+		if (!bo_merge::is_export_limit_excepted()) {
 			$export_limit = bo_merge::getExportLimit($_appname);
-			if($export_limit == 'no') {
+			if ($export_limit == 'no') {
 				die(lang('Admin disabled exporting'));
 			}
 		}
 			//error_log(__FILE__.__FUNCTION__. '::$_GET[\'appname\']='. $_appname. ',$_GET[\'definition\']='. $_definition. ',$_GET[\'plugin\']='.$_plugin. ',$_GET[\'selection\']='.$_selection);
 		// if appname is given and valid, list available definitions (if no definition is given)
-		$readonlys['appname'] = (!empty($_appname) && $GLOBALS['egw']->acl->check('run',1,$_appname));
+		$readonlys['appname'] = (!empty($_appname) && $GLOBALS['egw']->acl->check('run', 1, $_appname));
 		$content['appname'] = $_appname;
 		$preserv['appname'] = $_appname;
-		if(empty($_appname)) {
-			$et->setElementAttribute('select_definition','disabled',true);
+		if (empty($_appname)) {
+			$et->setElementAttribute('select_definition', 'disabled', true);
 		}
 
 		// Check for preferred definition
-		if(!$_definition && $_appname) {
+		if (!$_definition && $_appname) {
 			$_definition = $GLOBALS['egw_info']['user']['preferences'][$_appname]['nextmatch-export-definition'];
 		}
 		// fill definitions
@@ -96,13 +96,13 @@ 
 			}
 			unset($definition);
 		}
-		if(count($sel_options['definition']) == 2 && !$content['definition']) {
+		if (count($sel_options['definition']) == 2 && !$content['definition']) {
 			$content['definition'] = end($sel_options['definition']);
 		}
 		unset($definitions);
 		//$sel_options['definition']['expert'] = lang('Expert options');
 
-		if(isset($_definition) && array_key_exists($_definition,$sel_options['definition'])) {
+		if (isset($_definition) && array_key_exists($_definition, $sel_options['definition'])) {
 			$content['definition'] = $_definition;
 		}
 
@@ -110,8 +110,8 @@ 
 		$sel_options['plugin'] = $this->export_plugins[$_appname]['export'];
 
 		// show definitions or plugins in ui?
-		if($content['definition'] == 'expert') {
-			if(isset($_plugin) && array_key_exists($_plugin,$sel_options['plugin'])) {
+		if ($content['definition'] == 'expert') {
+			if (isset($_plugin) && array_key_exists($_plugin, $sel_options['plugin'])) {
 				$content['plugin'] = $_plugin;
 				$selected_plugin = $_plugin;
 			}
@@ -130,7 +130,7 @@ 
 			$readonlys['save_definition'] = true;
 
 			$definition = new importexport_definition($content['definition']);
-			if($definition) {
+			if ($definition) {
 				$content += (array)$definition->plugin_options;
 				$selected_plugin = $definition->plugin;
 				$content['description'] = $definition->description;
@@ -147,27 +147,27 @@ 
 			'|'	=>	'|',
 			''	=>	lang('Other')
 		);
-		if(!$sel_options['delimiter'][$content['delimiter']]) $sel_options['delimiter'][$content['delimiter']] = $content['delimiter'];
-		$sel_options['delimiter'][$content['delimiter']] = lang('Use default') . ' "' . $sel_options['delimiter'][$content['delimiter']] . '"';
+		if (!$sel_options['delimiter'][$content['delimiter']]) $sel_options['delimiter'][$content['delimiter']] = $content['delimiter'];
+		$sel_options['delimiter'][$content['delimiter']] = lang('Use default').' "'.$sel_options['delimiter'][$content['delimiter']].'"';
 
-		if(!$_content['delimiter']) $et->setElementAttribute('other_delimiter','disabled',true);
+		if (!$_content['delimiter']) $et->setElementAttribute('other_delimiter', 'disabled', true);
 
 		// Other delimiter (options)
-		if($_content['other_delimiter']) $_content['delimiter'] = $_content['other_delimiter'];
+		if ($_content['other_delimiter']) $_content['delimiter'] = $_content['other_delimiter'];
 
 		// handle selector
-		if($selected_plugin) {
+		if ($selected_plugin) {
 			$content['plugin'] = $selected_plugin;
 			$plugin_object = new $selected_plugin;
 
 			$content['description'] = $plugin_object->get_description();
 
 			// fill options tab
- 			if(method_exists($plugin_object, 'get_selectors_html')) {
+ 			if (method_exists($plugin_object, 'get_selectors_html')) {
 				$content['plugin_options_html'] = $plugin_object->get_options_html();
 			} else {
 				$options = $plugin_object->get_options_etpl($definition);
-				if(is_array($options)) {
+				if (is_array($options)) {
 					$content['plugin_options_template'] = $options['name'];
 					$content += (array)$options['content'];
 					$sel_options += (array)$options['sel_options'];
@@ -180,7 +180,7 @@ 
 		}
 
 		// fill selection tab
-		if($definition && is_array($definition->plugin_options) && $definition->plugin_options['selection'] && !$content['selection_passed']) {
+		if ($definition && is_array($definition->plugin_options) && $definition->plugin_options['selection'] && !$content['selection_passed']) {
 			$_selection = $definition->plugin_options['selection'];
 		}
 		
@@ -190,11 +190,11 @@ 
 			$preserv['selection'] = $_selection;
 		}
 		elseif ($plugin_object) {
- 			if(method_exists($plugin_object, 'get_selectors_html')) {
+ 			if (method_exists($plugin_object, 'get_selectors_html')) {
 				$content['plugin_selectors_html'] = $plugin_object->get_selectors_html();
 			} else {
 				$options = $plugin_object->get_selectors_etpl($definition);
-				if(is_array($options)) {
+				if (is_array($options)) {
 					$content += is_array($options['content']) ? $options['content'] : array('selection' => $options['content']);
 					$sel_options += (array)$options['sel_options'];
 					$readonlys['selection'] = (array)$options['readonlys'];
@@ -204,12 +204,12 @@ 
 					$content['plugin_selectors_template'] = $options;
 				}
 			}
-			if(!$content['plugin_selectors_html'] && !$content['plugin_selectors_template']) {
+			if (!$content['plugin_selectors_html'] && !$content['plugin_selectors_template']) {
 				$readonlys[$tabs]['selection_tab'] = true;
 			}
 			$content['filter'] = $definition->filter;
 			$content['filter']['fields'] = importexport_helper_functions::get_filter_fields($_appname, $selected_plugin);
-			if(!$content['filter']['fields'])
+			if (!$content['filter']['fields'])
 			{
 				$this->js->set_onload("\$j('input[value=\"filter\"]').parent().hide();");
 				$content['no_filter'] = true;
@@ -228,13 +228,13 @@ 
 		{
 			$content['selection'] = 'filter';
 		}
-		if(!$content['selection'])
+		if (!$content['selection'])
 		{
 			$content['selection'] = 'search';
 		}
 
 		// Disable / hide definition filter if not selected
-		if($content['selection'] != 'filter')
+		if ($content['selection'] != 'filter')
 		{
 			$this->js->set_onload("
 				\$j('div.filters').hide();
@@ -245,33 +245,33 @@ 
 		//error_log(__METHOD__.__LINE__.array2string($apps));
 		if (empty($apps)) throw new Exception('Error: no application profiles available for export');
 		if (!is_array($apps) && $apps) $apps = (array)$apps;
-		$sel_options['appname'] = array('' => lang('Select one')) + array_combine($apps,$apps);
-		if(!$_application && !$selected_plugin) {
+		$sel_options['appname'] = array('' => lang('Select one')) + array_combine($apps, $apps);
+		if (!$_application && !$selected_plugin) {
 			$content['plugin_selectors_html'] = $content['plugin_options_html'] =
 			lang('You need to select an app and format first!');
 			$readonlys[$tabs] = array('selection_tab' => true, 'options_tab' => true);
 		}
 
-		if($_content['preview'] || $_content['export'])
+		if ($_content['preview'] || $_content['export'])
 		{
 			//error_log(__LINE__.__FILE__.'$_content: '.print_r($_content,true));
 			$response = egw_json_response::get();
 
 			if ($_content['definition'] == 'expert') {
 				$definition = new importexport_definition();
-				$definition->definition_id	= $_content['definition_id'] ? $_content['definition_id'] : '';
+				$definition->definition_id = $_content['definition_id'] ? $_content['definition_id'] : '';
 				$definition->name		= $_content['name'] ? $_content['name'] : '';
-				$definition->application	= $_content['appname'];
-				$definition->plugin		= $_content['plugin'];
+				$definition->application = $_content['appname'];
+				$definition->plugin = $_content['plugin'];
 				$definition->type		= 'export';
-				$definition->allowed_users	= $_content['allowed_users'] ? $_content['allowed_users'] : $this->user;
-				$definition->owner		= $_content['owner'] ? $_content['owner'] : $this->user;
+				$definition->allowed_users = $_content['allowed_users'] ? $_content['allowed_users'] : $this->user;
+				$definition->owner = $_content['owner'] ? $_content['owner'] : $this->user;
 			}
 			else {
 				$definition = new importexport_definition($_content['definition']);
 			}
 
-			if(!is_array($definition->plugin_options)) {
+			if (!is_array($definition->plugin_options)) {
 				$definition->plugin_options = array(
 					'mapping'	=>	array()
 				);
@@ -281,30 +281,30 @@ 
 			// Note that because not all dates are DB dates, the plugin has to handle them
 			$filter = array();
 			$_content['filter'] = $_content['filter_html'] ? $_content['filter_html'] : $_content['filter_tpl'];
-			if(is_array($_content['filter']))
+			if (is_array($_content['filter']))
 			{
-				foreach($_content['filter'] as $key => $value)
+				foreach ($_content['filter'] as $key => $value)
 				{
 					// Handle multiple values
-					if(!is_array($value) && strpos($value,',') !== false) $value = explode(',',$value);
+					if (!is_array($value) && strpos($value, ',') !== false) $value = explode(',', $value);
 
 					$filter[$key] = $value;
 
 					// Skip empty values or empty ranges
-					if($value == "" || is_null($value) || (is_array($value) && count($value) == 0) || is_array($value) && array_key_exists('from',$value) && !$value['from'] && !$value['to'] )
+					if ($value == "" || is_null($value) || (is_array($value) && count($value) == 0) || is_array($value) && array_key_exists('from', $value) && !$value['from'] && !$value['to'])
 					{
 						unset($filter[$key]);
 					}
-					if(is_array($value) && array_key_exists('from', $value) && $value['from'])
+					if (is_array($value) && array_key_exists('from', $value) && $value['from'])
 					{
-						$filter[$key]['from'] = egw_time::to($value['from'],'ts');
+						$filter[$key]['from'] = egw_time::to($value['from'], 'ts');
 					}
 					// If user selects an end date, they most likely want entries including that date
-					if(is_array($value) && array_key_exists('to',$value) && $value['to'] )
+					if (is_array($value) && array_key_exists('to', $value) && $value['to'])
 					{
 						// Adjust time to 23:59:59
 						$filter[$key]['to'] = new egw_time($value['to']);
-						$filter[$key]['to']->setTime(23,59,59);
+						$filter[$key]['to']->setTime(23, 59, 59);
 						$filter[$key]['to'] = $filter[$key]['to']->format('ts');
 					}
 				}
@@ -319,19 +319,19 @@ 
 				$_content
 			);
 
-			if(!$definition->plugin_options['selection']) {
-				$response->alert( lang('No records selected'));
+			if (!$definition->plugin_options['selection']) {
+				$response->alert(lang('No records selected'));
 				return;
 			}
 
-			$tmpfname = tempnam($GLOBALS['egw_info']['server']['temp_dir'],'export');
+			$tmpfname = tempnam($GLOBALS['egw_info']['server']['temp_dir'], 'export');
 			$file = fopen($tmpfname, "w+");
-			if (! $charset = $definition->plugin_options['charset']) {
+			if (!$charset = $definition->plugin_options['charset']) {
 				$charset = $GLOBALS['egw']->translation->charset();
 			}
-			if($charset == 'user')
+			if ($charset == 'user')
 			{
-				switch($definition->plugin)
+				switch ($definition->plugin)
 				{
 					case 'addressbook_export_vcard':
 						$charset = $GLOBALS['egw_info']['user']['preferences']['addressbook']['vcard_charset'];
@@ -341,9 +341,9 @@ 
 				}
 			}
 			$plugin_object = new $definition->plugin;
-			$result = $plugin_object->export( $file, $definition );
+			$result = $plugin_object->export($file, $definition);
 
-			if(is_object($result) && method_exists($result, 'get_num_of_records'))
+			if (is_object($result) && method_exists($result, 'get_num_of_records'))
 			{
 				$record_count = $result->get_num_of_records();
 			}
@@ -351,7 +351,7 @@ 
 			// Store charset to use in header
 			egw_cache::setSession('importexport', $tmpfname, $charset, 100);
 
-			if($_content['export'] == 'pressed') {
+			if ($_content['export'] == 'pressed') {
 				fclose($file);
 				$filename = pathinfo($tmpfname, PATHINFO_FILENAME);
 				$link_query = array(
@@ -363,25 +363,25 @@ 
 				);
 
 				// Allow plugins to suggest a file name - return false if they have no suggestion
-				if(method_exists($plugin_object, 'get_filename') && $plugin_filename = $plugin_object->get_filename())
+				if (method_exists($plugin_object, 'get_filename') && $plugin_filename = $plugin_object->get_filename())
 				{
 					$link_query['filename'] = $plugin_filename;
 				}
-				$response->redirect( $GLOBALS['egw']->link('/index.php',$link_query),true);
+				$response->redirect($GLOBALS['egw']->link('/index.php', $link_query), true);
 				egw_framework::window_close();
 				return;
 			}
-			elseif($_content['preview'] == 'pressed') {
+			elseif ($_content['preview'] == 'pressed') {
 				fseek($file, 0);
 				$item_count = 1;
 				$preview = '';
-				$search = array('[\016]','[\017]',
-								'[\020]','[\021]','[\022]','[\023]','[\024]','[\025]','[\026]','[\027]',
-								'[\030]','[\031]','[\032]','[\033]','[\034]','[\035]','[\036]','[\037]');
+				$search = array('[\016]', '[\017]',
+								'[\020]', '[\021]', '[\022]', '[\023]', '[\024]', '[\025]', '[\026]', '[\027]',
+								'[\030]', '[\031]', '[\032]', '[\033]', '[\034]', '[\035]', '[\036]', '[\037]');
 				$replace = $preview = '';
 
-				while(!feof($file) && $item_count < 30) {
-					$preview .= preg_replace($search,$replace,fgets($file,1024));
+				while (!feof($file) && $item_count < 30) {
+					$preview .= preg_replace($search, $replace, fgets($file, 1024));
 					$item_count++;
 				}
 
@@ -389,12 +389,12 @@ 
 				unlink($tmpfname);
 
 				// Convert back to system charset for display
-				$preview = $GLOBALS['egw']->translation->convert( $preview,
+				$preview = $GLOBALS['egw']->translation->convert($preview,
 					$charset,
 					$GLOBALS['egw']->translation->charset()
 				);
 
-				$preview = "<div class='header'>".lang('Preview') . "<span class='count'>".(int)$record_count."</span></div>".$preview;
+				$preview = "<div class='header'>".lang('Preview')."<span class='count'>".(int)$record_count."</span></div>".$preview;
 				
 				$et->setElementAttribute('preview-box', 'value', nl2br($preview));
 				return;
@@ -406,17 +406,17 @@ 
 			$readonlys[$tabs]['selection'] = false;
 		}
 		//error_log(print_r($content,true));
-		return $et->exec(self::_appname. '.importexport_export_ui.export_dialog',$content,$sel_options,$readonlys,$preserv,2);
+		return $et->exec(self::_appname.'.importexport_export_ui.export_dialog', $content, $sel_options, $readonlys, $preserv, 2);
 	}
 
-	public function ajax_get_definitions($_appname, xajaxResponse &$response = null) {
-		if(is_null($response)) {
+	public function ajax_get_definitions($_appname, xajaxResponse&$response = null) {
+		if (is_null($response)) {
 			$response = new xajaxResponse();
 		} else {
 			$no_return = true;
 		}
 		if (!$_appname) {
-			$response->jquery('tr.select_definition','hide');
+			$response->jquery('tr.select_definition', 'hide');
 			return $no_return ? '' : $response->getXML();
 		}
 
@@ -438,7 +438,7 @@ 
 				unset($definition);
 		}
 		unset($definitions);
-		$response->addScript("selectbox_add_option('exec[definition]','" . lang('Expert options') . "', 'expert',".($selected_plugin == $title ? 'true' : 'false').");");
+		$response->addScript("selectbox_add_option('exec[definition]','".lang('Expert options')."', 'expert',".($selected_plugin == $title ? 'true' : 'false').");");
 	}
 
 	public function ajax_get_definition_description($_definition) {
@@ -453,7 +453,7 @@ 
 			}
 			unset ($_object);
 		}
-		$_response->assign('importexport-export_dialog_plugin_description','innerHTML',$description);
+		$_response->assign('importexport-export_dialog_plugin_description', 'innerHTML', $description);
 	}
 
 	public function ajax_get_plugin_description($_plugin) {
@@ -463,7 +463,7 @@ 
 		if (is_a($plugin_object, 'importexport_iface_export_plugin')) {
 			$description = $plugin_object->get_description();
 		}
-		$_response->addAssign('importexport-export_dialog_plugin_description','innerHTML',$description);
+		$_response->addAssign('importexport-export_dialog_plugin_description', 'innerHTML', $description);
 		
 		unset ($plugin_object);
 	}
@@ -476,7 +476,7 @@ 
 	 */
 	public function download($_tmpfname = '') {
 		$tmpfname = $_tmpfname ? $_tmpfname : $_GET['_filename'];
-		$tmpfname = $GLOBALS['egw_info']['server']['temp_dir'] .'/'. $tmpfname;
+		$tmpfname = $GLOBALS['egw_info']['server']['temp_dir'].'/'.$tmpfname;
 		if (!is_readable($tmpfname)) die();
 
 		$appname = $_GET['_appname'];
@@ -485,13 +485,13 @@ 
 		// Turn off all output buffering
 		while (@ob_end_clean());
 
-		$file = fopen($tmpfname,'rb');
+		$file = fopen($tmpfname, 'rb');
 
 		// Get charset
 		$charset = egw_cache::getSession('importexport', $tmpfname);
 
 		html::content_header($nicefname.'.'.$_GET['_suffix'],
-			($_GET['_type'] ? $_GET['_type'] : 'application/text') . ($charset ? '; charset='.$charset : ''),
+			($_GET['_type'] ? $_GET['_type'] : 'application/text').($charset ? '; charset='.$charset : ''),
 			filesize($tmpfname));
 		fpassthru($file);
 

importexport/inc/class.importexport_widget_filter.inc.php

Spacing +25 added lines, -25 removed lines

@@ -38,18 +38,18 @@ 
 	public function beforeSendToClient($cname, Array $expand = Array())
 	{
 		$form_name = self::form_name($cname, $this->id);
-		if($this->getElementAttribute($form_name, 'customfields'))
+		if ($this->getElementAttribute($form_name, 'customfields'))
 		{
 			// Already done?  Still need to process, or sel_options may be missing
 			unset(self::$request->modifications[$form_name]);
 		}
-		$value =& self::get_array(self::$request->content, $form_name, true);
+		$value = & self::get_array(self::$request->content, $form_name, true);
 		$fields = $value['fields'];
 		unset($value['fields']);
 		$relative_dates = $this->attrs['relative_dates'];
 
 		// Fallback, so there's something there...
-		if(!is_array($fields))
+		if (!is_array($fields))
 		{
 			error_log("$this has no fields");
 			self::$transformation = array(
@@ -66,14 +66,14 @@ 
 		$this->setElementAttribute($form_name, 'prefix', self::$prefix);
 
 		$n = 1;
-		foreach($fields as $lname => &$field)
+		foreach ($fields as $lname => &$field)
 		{
-			$type =& $field['type'];
+			$type = & $field['type'];
 
 			// No filters are required
 			$field['needed'] = false;
 
-			switch($type)
+			switch ($type)
 			{
 				case 'date':
 				case 'date-time':
@@ -90,7 +90,7 @@ 
 						'get_rows'      => 'etemplate.ajax_select_widget.array_rows',
 						'id_field'      => ajax_select_widget::ARRAY_KEY,
 					);
-					if($field['rows']) {
+					if ($field['rows']) {
 						$options['num_rows'] = $field['rows'];
 					}
 
@@ -109,15 +109,15 @@ 
 					break;
 				case 'select':
 				default:
-					if(strpos($field['type'],'select') === 0)
+					if (strpos($field['type'], 'select') === 0)
 					{
 						if (count($field['values']) == 1 && isset($field['values']['@']))
 						{
 							$field['values'] = egw_customfields::get_options_from_file($field['values']['@']);
 						}
-						foreach((array)$field['values'] as $key => $val)
+						foreach ((array)$field['values'] as $key => $val)
 						{
-							if (substr($val = lang($val),-1) != '*')
+							if (substr($val = lang($val), -1) != '*')
 							{
 								$field['values'][$key] = $val;
 							}
@@ -129,27 +129,27 @@ 
 						$this->setElementAttribute($form_name.'['.self::$prefix.$lname.']', 'tags', TRUE);
 						$this->setElementAttribute($form_name.'['.self::$prefix.$lname.']', 'multiple', TRUE);
 					}
-					else if( $GLOBALS['egw_info']['apps'][$field['type']])
+					else if ($GLOBALS['egw_info']['apps'][$field['type']])
 					{
 						// Links
 					}
 					else
 					{
-						error_log('Trying to filter with unsupported field type ' . $lname . ': ' . $field['type']);
+						error_log('Trying to filter with unsupported field type '.$lname.': '.$field['type']);
 					}
 			}
 
 			// Send select options
-			if($field['values'])
+			if ($field['values'])
 			{
 				self::$request->sel_options[self::$prefix.$lname] = $field['values'];
 			}
 			$widget = self::factory($type, '<'.$type.' type="'.$type.'" id="'.self::$prefix.$lname.'"/>', self::$prefix.$lname);
-			if(method_exists($widget, 'beforeSendToClient'))
+			if (method_exists($widget, 'beforeSendToClient'))
 			{
 				$widget->id = self::$prefix.$lname;
 				$widget->attrs['type'] = $type;
-				if($type == 'link-to')
+				if ($type == 'link-to')
 				{
 					$widget->attrs['only_app'] = $field['type'];
 				}
@@ -161,7 +161,7 @@ 
 		parent::beforeSendToClient($cname, $expand);
 
 		$this->setElementAttribute($form_name, 'customfields', $fields);
-		$this->setElementAttribute($form_name, 'fields',array_fill_keys(array_keys($fields), true));
+		$this->setElementAttribute($form_name, 'fields', array_fill_keys(array_keys($fields), true));
 		return false;
 	}
 
@@ -177,13 +177,13 @@ 
 	protected static function do_relative_date($lname, Array &$value, $options, $readonly)
 	{
 		// Maybe this could be moved to date widget
-		$input = boetemplate::empty_cell('select',$lname, array(
+		$input = boetemplate::empty_cell('select', $lname, array(
 			'readonly'	=> $readonly,
 			'no_lang'	=> true,
 			'options'	=> $options,
 			'sel_options'	=> array('' => lang('all'))
 		));
-		foreach(importexport_helper_functions::$relative_dates as $label => $values)
+		foreach (importexport_helper_functions::$relative_dates as $label => $values)
 		{
 			$input['sel_options'][$label] = lang($label);
 		}
@@ -202,10 +202,10 @@ 
 	 */
 	protected static function do_absolute_date($lname, Array &$value, $options, $readonly)
 	{
-		$input = boetemplate::empty_cell('hbox',$lname);
+		$input = boetemplate::empty_cell('hbox', $lname);
 
 		$type = 'date';
-		$from = boetemplate::empty_cell($type, $lname.'[from]',array(
+		$from = boetemplate::empty_cell($type, $lname.'[from]', array(
 			'readonly'      => $readonly,
 			'no_lang'       => True,
 			'size'          => $options
@@ -217,7 +217,7 @@ 
 			'size'          => $options
 		));
 		boetemplate::add_child($input, $from);
-		boetemplate::add_child($input,boetemplate::empty_cell('label','',array(
+		boetemplate::add_child($input, boetemplate::empty_cell('label', '', array(
 			'label' => lang('to'),
 			'no_lang' => true
 		)));
@@ -226,18 +226,18 @@ 
 	}
 
 
-	public function validate($cname, array $expand, array $content, &$validated=array())
+	public function validate($cname, array $expand, array $content, &$validated = array())
 	{
 		$form_name = self::form_name($cname, $this->id, $expand);
 		if (!$this->is_readonly($cname, $form_name))
 		{
 			$value_in = (array)self::get_array($content, $form_name);
-			$valid =& self::get_array($validated, $this->id ? $form_name : $field, true);
+			$valid = & self::get_array($validated, $this->id ? $form_name : $field, true);
 
-			foreach($value_in as $key => $value)
+			foreach ($value_in as $key => $value)
 			{
 				// Client side cf widget automatically prefixes #
-				$valid[substr($key,strlen(self::$prefix))] = $value;
+				$valid[substr($key, strlen(self::$prefix))] = $value;
 			}
 		}
 	}