EGroupware / egroupware

api/src/Contacts/Merge.php

Spacing +34 added lines, -34 removed lines

@@ -16,7 +16,7 @@ 
 use EGroupware\Api;
 
 // explicit import not namespaced classes
-use calendar_boupdate;	// we detect if it is not available
+use calendar_boupdate; // we detect if it is not available
 
 /**
  * Contacts document merge
@@ -55,22 +55,22 @@ 
 	 * @param boolean $ignore_acl =false true: no acl check
 	 * @return array|boolean
 	 */
-	protected function get_replacements($id,&$content=null,$ignore_acl=false)
+	protected function get_replacements($id, &$content = null, $ignore_acl = false)
 	{
-		if (!($replacements = $this->contact_replacements($id,'',$ignore_acl, $content)))
+		if (!($replacements = $this->contact_replacements($id, '', $ignore_acl, $content)))
 		{
 			return false;
 		}
-		if($content && strpos($content, '$$#') !== false)
+		if ($content && strpos($content, '$$#') !== false)
 		{
-			$this->cf_link_to_expand($this->contacts->read($id, $ignore_acl), $content, $replacements,'addressbook');
+			$this->cf_link_to_expand($this->contacts->read($id, $ignore_acl), $content, $replacements, 'addressbook');
 		}
 
 		// Links
 		$replacements += $this->get_all_links('addressbook', $id, '', $content);
-		if (!(strpos($content,'$$calendar/') === false))
+		if (!(strpos($content, '$$calendar/') === false))
 		{
-			$replacements += $this->calendar_replacements($id,!(strpos($content,'$$calendar/-1/') === false));
+			$replacements += $this->calendar_replacements($id, !(strpos($content, '$$calendar/-1/') === false));
 		}
 		return $replacements;
 	}
@@ -82,7 +82,7 @@ 
 	 * @param boolean $last_event_too =false also include information about the last event
 	 * @return array
 	 */
-	protected function calendar_replacements($id,$last_event_too=false)
+	protected function calendar_replacements($id, $last_event_too = false)
 	{
 		if (!class_exists('calendar_boupdate')) return array();
 
@@ -101,7 +101,7 @@ 
 		{
 			$events = array();
 		}
-		if ($last_event_too==true)
+		if ($last_event_too == true)
 		{
 			$last = $calendar->search(array(
 				'end' => $calendar->now_su,
@@ -111,41 +111,41 @@ 
 				'order' => 'cal_start DESC',
 				'enum_recurring' => true
 			));
-			$events['-1'] = $last ? array_shift($last) : array();	// returned events are indexed by cal_id!
+			$events['-1'] = $last ? array_shift($last) : array(); // returned events are indexed by cal_id!
 		}
 		$replacements = array();
-		$n = 1;  // Returned events are indexed by cal_id, need to index sequentially
-		foreach($events as $key => $event)
+		$n = 1; // Returned events are indexed by cal_id, need to index sequentially
+		foreach ($events as $key => $event)
 		{
 			// Use -1 for previous key
-			if($key < 0) $n = $key;
+			if ($key < 0) $n = $key;
 
-			foreach($calendar->event2array($event) as $name => $data)
+			foreach ($calendar->event2array($event) as $name => $data)
 			{
-				if (substr($name,-4) == 'date') $name = substr($name,0,-4);
-				$replacements['$$calendar/'.$n.'/'.$name.'$$'] = is_array($data['data']) ? implode(', ',$data['data']) : $data['data'];
+				if (substr($name, -4) == 'date') $name = substr($name, 0, -4);
+				$replacements['$$calendar/'.$n.'/'.$name.'$$'] = is_array($data['data']) ? implode(', ', $data['data']) : $data['data'];
 			}
-			foreach(array('start','end') as $what)
+			foreach (array('start', 'end') as $what)
 			{
-				foreach(array(
+				foreach (array(
 					'date' => $GLOBALS['egw_info']['user']['preferences']['common']['dateformat'],
 					'day'  => 'l',
 					'time' => $GLOBALS['egw_info']['user']['preferences']['common']['timeformat'] == 12 ? 'h:i a' : 'H:i',
 				) as $name => $format)
 				{
-					$value = $event[$what] ? date($format,$event[$what]) : '';
+					$value = $event[$what] ? date($format, $event[$what]) : '';
 					if ($format == 'l') $value = lang($value);
 					$replacements['$$calendar/'.$n.'/'.$what.$name.'$$'] = $value;
 				}
 			}
-			$duration = ($event['end'] - $event['start'])/60;
-			$replacements['$$calendar/'.$n.'/duration$$'] = floor($duration/60).lang('h').($duration%60 ? $duration%60 : '');
+			$duration = ($event['end'] - $event['start']) / 60;
+			$replacements['$$calendar/'.$n.'/duration$$'] = floor($duration / 60).lang('h').($duration % 60 ? $duration % 60 : '');
 
 			++$n;
 		}
 
 		// Need to set some keys if there is no previous event
-		if($last_event_too && count($events['-1']) == 0) {
+		if ($last_event_too && count($events['-1']) == 0) {
 			$replacements['$$calendar/-1/start$$'] = '';
 			$replacements['$$calendar/-1/end$$'] = '';
 			$replacements['$$calendar/-1/owner$$'] = '';
@@ -168,18 +168,18 @@ 
 		echo '<tr><td colspan="4"><h3>'.lang('Contact fields:')."</h3></td></tr>";
 
 		$n = 0;
-		foreach($this->contacts->contact_fields as $name => $label)
+		foreach ($this->contacts->contact_fields as $name => $label)
 		{
-			if (in_array($name,array('tid','label','geo'))) continue;	// dont show them, as they are not used in the UI atm.
+			if (in_array($name, array('tid', 'label', 'geo'))) continue; // dont show them, as they are not used in the UI atm.
 
-			if (in_array($name,array('email','org_name','tel_work','url')) && $n&1)		// main values, which should be in the first column
+			if (in_array($name, array('email', 'org_name', 'tel_work', 'url')) && $n&1)		// main values, which should be in the first column
 			{
 				echo "</tr>\n";
 				$n++;
 			}
 			if (!($n&1)) echo '<tr>';
 			echo '<td>{{'.$name.'}}</td><td>'.$label.'</td>';
-			if($name == 'cat_id')
+			if ($name == 'cat_id')
 			{
 				if ($n&1) echo "</tr>\n";
 				echo '<td>{{categories}}</td><td>'.lang('Category path').'</td>';
@@ -190,13 +190,13 @@ 
 		}
 
 		echo '<tr><td colspan="4"><h3>'.lang('Custom fields').":</h3></td></tr>";
-		foreach($this->contacts->customfields as $name => $field)
+		foreach ($this->contacts->customfields as $name => $field)
 		{
 			echo '<tr><td>{{#'.$name.'}}</td><td colspan="3">'.$field['label']."</td></tr>\n";
 		}
 
 		echo '<tr><td colspan="4"><h3>'.lang('General fields:')."</h3></td></tr>";
-		foreach(array(
+		foreach (array(
 			'link' => lang('HTML link to the current record'),
 			'links' => lang('Titles of any entries linked to the current record, excluding attached files'),
  			'attachments' => lang('List of files linked to the current record'),
@@ -223,7 +223,7 @@ 
 
 		Api\Translation::add_app('calendar');
 		echo '<tr><td colspan="4"><h3>'.lang('Calendar fields:')." # = 1, 2, ..., 20, -1</h3></td></tr>";
-		foreach(array(
+		foreach (array(
 			'title' => lang('Title'),
 			'description' => lang('Description'),
 			'participants' => lang('Participants'),
@@ -245,7 +245,7 @@ 
 			'owner'    => lang('Owner'),
 		) as $name => $label)
 		{
-			if (in_array($name,array('start','end')) && $n&1)		// main values, which should be in the first column
+			if (in_array($name, array('start', 'end')) && $n&1)		// main values, which should be in the first column
 			{
 				echo "</tr>\n";
 				$n++;
@@ -274,8 +274,8 @@ 
 	 * @param int|string $export_limit =null export-limit, default $GLOBALS['egw_info']['server']['export_limit']
 	 * @return array see nextmatch_widget::egw_actions
 	 */
-	public static function document_action($dirs, $group=0, $caption='Insert in document', $prefix='document_', $default_doc='',
-		$export_limit=null)
+	public static function document_action($dirs, $group = 0, $caption = 'Insert in document', $prefix = 'document_', $default_doc = '',
+		$export_limit = null)
 	{
 		$actions = parent::document_action($dirs, $group, $caption, $prefix, $default_doc, $export_limit);
 
@@ -292,14 +292,14 @@ 
 			// Just email already filtered out
 			$action['children'] = $action['children']['message/rfc822']['children'];
 		}
-		if(strpos($action['egw_open'], 'edit-mail') === 0)
+		if (strpos($action['egw_open'], 'edit-mail') === 0)
 		{
 			unset($action['confirm_multiple']);
 			$action['onExecute'] = 'javaScript:app.addressbook.merge_mail';
 		}
 		else if ($action['children'])
 		{
-			foreach($action['children'] as &$child)
+			foreach ($action['children'] as &$child)
 			{
 				static::customise_mail_actions($child);
 			}