EGroupware / egroupware

calendar/inc/class.calendar_groupdav.inc.php

Spacing +144 added lines, -144 removed lines

@@ -153,7 +153,7 @@ 
 	 * @param string $id =''
 	 * @return mixed boolean true on success, false on failure or string with http status (eg. '404 Not Found')
 	 */
-	function propfind($path,&$options,&$files,$user,$id='')
+	function propfind($path, &$options, &$files, $user, $id = '')
 	{
 		if ($this->debug)
 		{
@@ -179,10 +179,10 @@ 
 			'enum_recuring' => false,
 			'daywise' => false,
 			'date_format' => 'server',
-			'no_total' => true,	// we need no total number of rows (saves extra query)
-			'cfs' => array(),	// return custom-fields, as we use them to store X- attributes
+			'no_total' => true, // we need no total number of rows (saves extra query)
+			'cfs' => array(), // return custom-fields, as we use them to store X- attributes
 		);
-		foreach(array(
+		foreach (array(
 			'start' => $GLOBALS['egw_info']['user']['preferences']['groupdav']['calendar-past-limit'],
 			'end' => $GLOBALS['egw_info']['user']['preferences']['groupdav']['calendar-future-limit'],
 		) as $name => $value)
@@ -191,7 +191,7 @@ 
 			{
 				$value = $name == 'start' ? self::PAST_LIMIT : self::FUTURE_LIMIT;
 			}
-			$filter[$name] = $this->bo->now + 24*3600*($name == 'start' ? -1 : 1)*abs($value);
+			$filter[$name] = $this->bo->now + 24 * 3600 * ($name == 'start' ? -1 : 1) * abs($value);
 		}
 		if ($this->client_shared_uid_exceptions)	// do NOT return (non-virtual) exceptions
 		{
@@ -203,13 +203,13 @@ 
 			$filter['filter'] = 'owner';
 		}
 		// scheduling inbox, shows only not yet accepted or rejected events
-		elseif (substr($path,-7) == '/inbox/')
+		elseif (substr($path, -7) == '/inbox/')
 		{
 			$filter['filter'] = 'unknown';
-			$filter['start'] = $this->bo->now;	// only return future invitations
+			$filter['start'] = $this->bo->now; // only return future invitations
 		}
 		// ToDo: not sure what scheduling outbox is supposed to show, leave it empty for now
-		elseif (substr($path,-8) == '/outbox/')
+		elseif (substr($path, -8) == '/outbox/')
 		{
 			return true;
 		}
@@ -226,7 +226,7 @@ 
 			// when trying to request not supported components, eg. VTODO on a calendar collection
 			return true;
 		}
-		if ($id) $path = dirname($path).'/';	// caldav_name get's added anyway in the callback
+		if ($id) $path = dirname($path).'/'; // caldav_name get's added anyway in the callback
 
 		if ($this->debug > 1)
 		{
@@ -243,7 +243,7 @@ 
 
 			$this->sync_collection_token = null;
 
-			$filter['order'] = 'cal_modified ASC';	// return oldest modifications first
+			$filter['order'] = 'cal_modified ASC'; // return oldest modifications first
 			$filter['sync-collection'] = true;
 			// no end-date / limit into the future, as unchanged entries would never be transferted later on
 			unset($filter['end']);
@@ -251,7 +251,7 @@ 
 
 		if (isset($nresults))
 		{
-			unset($filter['no_total']);	// we need the total!
+			unset($filter['no_total']); // we need the total!
 			$files['files'] = $this->propfind_callback($path, $filter, array(0, (int)$nresults));
 
 			// hack to support limit with sync-collection report: events are returned in modified ASC order (oldest first)
@@ -260,13 +260,13 @@ 
 			if ($options['root']['name'] == 'sync-collection' && $this->bo->total > $nresults)
 			{
 				--$this->sync_collection_token;
-				$files['sync-token-params'][] = true;	// tel get_sync_collection_token that we have more entries
+				$files['sync-token-params'][] = true; // tel get_sync_collection_token that we have more entries
 			}
 		}
 		else
 		{
 			// return iterator, calling ourself to return result in chunks
-			$files['files'] = new Api\CalDAV\PropfindIterator($this,$path,$filter,$files['files']);
+			$files['files'] = new Api\CalDAV\PropfindIterator($this, $path, $filter, $files['files']);
 		}
 		if (isset($_GET['download']))
 		{
@@ -286,9 +286,9 @@ 
 		Api\Header\Content::type('calendar.ics', 'text/calendar');
 
 		$n = 0;
-		foreach($files as $file)
+		foreach ($files as $file)
 		{
-			if (!$n++) continue;	// first entry is collection itself
+			if (!$n++) continue; // first entry is collection itself
 
 			$icalendar = $file['props']['calendar-data']['val'];
 			if (($start = strpos($icalendar, 'BEGIN:VEVENT')) !== false &&
@@ -312,7 +312,7 @@ 
 						echo substr($icalendar, 0, $start);
 					}
 				}
-				echo substr($icalendar, $start, $end-$start);
+				echo substr($icalendar, $start, $end - $start);
 			}
 		}
 		if ($icalendar && $end)
@@ -330,12 +330,12 @@ 
 	 * @param array|boolean $start =false false=return all or array(start,num)
 	 * @return array with "files" array with values for keys path and props
 	 */
-	function &propfind_callback($path,array $filter,$start=false)
+	function &propfind_callback($path, array $filter, $start = false)
 	{
 		if ($this->debug) $starttime = microtime(true);
 
 		$calendar_data = $this->caldav->prop_requested('calendar-data', Api\CalDAV::CALDAV, true);
-		if (!is_array($calendar_data)) $calendar_data = false;	// not in allprop or autoindex
+		if (!is_array($calendar_data)) $calendar_data = false; // not in allprop or autoindex
 
 		$files = array();
 
@@ -347,11 +347,11 @@ 
 		$requested_multiget_ids = (array)$filter['query'][self::$path_attr];
 		$sync_collection = $filter['sync-collection'];
 
-		$events =& $this->bo->search($filter);
+		$events = & $this->bo->search($filter);
 
 		if ($events)
 		{
-			foreach($events as $event)
+			foreach ($events as $event)
 			{
 				// remove event from requested multiget ids, to be able to report not found urls
 				if ($requested_multiget_ids && ($k = array_search($event[self::$path_attr], $requested_multiget_ids)) !== false)
@@ -359,7 +359,7 @@ 
 					unset($requested_multiget_ids[$k]);
 				}
 				// sync-collection report: deleted entries need to be reported without properties, same for rejected or deleted invitations
-				if ($sync_collection && ($event['deleted'] && !$event['cal_reference'] || in_array($event['participants'][$filter['users']][0], array('R','X'))))
+				if ($sync_collection && ($event['deleted'] && !$event['cal_reference'] || in_array($event['participants'][$filter['users']][0], array('R', 'X'))))
 				{
 					$files[] = array('path' => $path.urldecode($this->get_path($event)));
 					continue;
@@ -390,7 +390,7 @@ 
 						!isset($calendar_data['children']['expand']) ? false :
 							($calendar_data['children']['expand']['attrs'] ? $calendar_data['children']['expand']['attrs'] : true));
 					$props['getcontentlength'] = bytes($content);
-					$props['calendar-data'] = Api\CalDAV::mkprop(Api\CalDAV::CALDAV,'calendar-data',$content);
+					$props['calendar-data'] = Api\CalDAV::mkprop(Api\CalDAV::CALDAV, 'calendar-data', $content);
 				}
 				/* Calendarserver reports new events with schedule-changes: action: create, which iCal request
 				 * adding it, unfortunately does not lead to showing the new event in the users inbox
@@ -409,7 +409,7 @@ 
 		// report not found multiget urls
 		if ($requested_multiget_ids)
 		{
-			foreach($requested_multiget_ids as $id)
+			foreach ($requested_multiget_ids as $id)
 			{
 				$files[] = array('path' => $path.$id.self::$path_extension);
 			}
@@ -445,7 +445,7 @@ 
 	private function _created_updated_by_prop($user, $time)
 	{
 		$props = array();
-		foreach(array(
+		foreach (array(
 			'first-name' => 'account_firstname',
 			'last-name' => 'account_lastname',
 			'href' => 'account_email',
@@ -488,23 +488,23 @@ 
 			$cal_end = $cal_filters['end']; unset($cal_filters['end']);
 			$num_filters = count($cal_filters);
 
-			foreach($options['filters'] as $filter)
+			foreach ($options['filters'] as $filter)
 			{
-				switch($filter['name'])
+				switch ($filter['name'])
 				{
 					case 'comp-filter':
 						if ($this->debug > 1) error_log(__METHOD__."($options[path],...) comp-filter='{$filter['attrs']['name']}'");
 
-						switch($filter['attrs']['name'])
+						switch ($filter['attrs']['name'])
 						{
 							case 'VTODO':
-								return false;	// return nothing for now, todo: check if we can pass it on to the infolog handler
+								return false; // return nothing for now, todo: check if we can pass it on to the infolog handler
 								// todos are handled by the infolog handler
 								//$infolog_handler = new groupdav_infolog();
 								//return $infolog_handler->propfind($options['path'],$options,$options['files'],$user,$method);
 							case 'VCALENDAR':
 							case 'VEVENT':
-								break;			// that's our default anyway
+								break; // that's our default anyway
 						}
 						break;
 					case 'prop-filter':
@@ -527,14 +527,14 @@ 
 						if ($this->debug) error_log(__METHOD__."($options[path],...) param-filter='{$filter['attrs']['name']}' not (yet) implemented!");
 						break;
 					case 'time-range':
-				 		if ($this->debug > 1) error_log(__FILE__ . __METHOD__."($options[path],...) time-range={$filter['attrs']['start']}-{$filter['attrs']['end']}");
+				 		if ($this->debug > 1) error_log(__FILE__.__METHOD__."($options[path],...) time-range={$filter['attrs']['start']}-{$filter['attrs']['end']}");
 				 		if (!empty($filter['attrs']['start']))
 				 		{
 					 		$cal_filters['start'] = $this->vCalendar->_parseDateTime($filter['attrs']['start']);
 				 		}
 				 		if (!empty($filter['attrs']['end']))
 				 		{
-					 		$cal_filters['end']   = $this->vCalendar->_parseDateTime($filter['attrs']['end']);
+					 		$cal_filters['end'] = $this->vCalendar->_parseDateTime($filter['attrs']['end']);
 				 		}
 						break;
 					default:
@@ -555,9 +555,9 @@ 
 		    <B:nresults>10</B:nresults>
 		  </B:limit>
 		*/
-		foreach((array)$options['other'] as $option)
+		foreach ((array)$options['other'] as $option)
 		{
-			switch($option['name'])
+			switch ($option['name'])
 			{
 				case 'nresults':
 					$nresults = (int)$option['data'];
@@ -566,7 +566,7 @@ 
 				case 'limit':
 					break;
 				case 'href':
-					break;	// from addressbook-multiget, handled below
+					break; // from addressbook-multiget, handled below
 				// rfc 6578 sync-report
 				case 'sync-token':
 					if (!empty($option['data']))
@@ -574,7 +574,7 @@ 
 						$parts = explode('/', $option['data']);
 						$sync_token = array_pop($parts);
 						$cal_filters['query'][] = 'cal_modified>'.(int)$sync_token;
-						$cal_filters['filter'] = 'everything';	// to return deleted entries too
+						$cal_filters['filter'] = 'everything'; // to return deleted entries too
 						// no standard time-range!
 						unset($cal_filters['start']);
 					}
@@ -603,25 +603,25 @@ 
 			if ($id)
 			{
 				$cal_filters['query'][self::$path_attr] = self::$path_extension ?
-					basename($id,self::$path_extension) : $id;
+					basename($id, self::$path_extension) : $id;
 			}
 			else	// fetch all given url's
 			{
-				foreach($options['other'] as $option)
+				foreach ($options['other'] as $option)
 				{
 					if ($option['name'] == 'href')
 					{
-						$parts = explode('/',$option['data']);
+						$parts = explode('/', $option['data']);
 						if (($id = urldecode(array_pop($parts))))
 						{
 							$cal_filters['query'][self::$path_attr][] = self::$path_extension ?
-								basename($id,self::$path_extension) : $id;
+								basename($id, self::$path_extension) : $id;
 						}
 					}
 				}
 			}
 
-			if ($this->debug > 1) error_log(__FILE__ . __METHOD__ ."($options[path],...,$id) calendar-multiget: ids=".implode(',',$ids).', cal_filters='.array2string($cal_filters));
+			if ($this->debug > 1) error_log(__FILE__.__METHOD__."($options[path],...,$id) calendar-multiget: ids=".implode(',', $ids).', cal_filters='.array2string($cal_filters));
 		}
 		return true;
 	}
@@ -634,9 +634,9 @@ 
 	 * @param int $user =null account_id
 	 * @return mixed boolean true on success, false on failure or string with http status (eg. '404 Not Found')
 	 */
-	function get(&$options,$id,$user=null)
+	function get(&$options, $id, $user = null)
 	{
-		if (!is_array($event = $this->_common_get_put_delete('GET',$options,$id)))
+		if (!is_array($event = $this->_common_get_put_delete('GET', $options, $id)))
 		{
 			return $event;
 		}
@@ -664,7 +664,7 @@ 
 	 * @param boolean|array $expand =false true or array with values for 'start', 'end' to expand recurrences
 	 * @return string
 	 */
-	private function iCal(array $event,$user=null, $method=null, $expand=false)
+	private function iCal(array $event, $user = null, $method = null, $expand = false)
 	{
 		static $handler = null;
 		if (is_null($handler)) $handler = $this->_get_handler();
@@ -688,13 +688,13 @@ 
 				if (isset($expand['start'])) $expand['start'] = $this->vCalendar->_parseDateTime($expand['start']);
 				if (isset($expand['end'])) $expand['end'] = $this->vCalendar->_parseDateTime($expand['end']);
 			}
-			$events =& self::get_series($event['uid'], $this->bo, $expand, $user);
+			$events = & self::get_series($event['uid'], $this->bo, $expand, $user);
 			// as alarm is now only on next recurrence, set alarm from original event on master
 			if ($event['alarm']) $events[0]['alarm'] = $event['alarm'];
 		}
-		elseif(!$this->client_shared_uid_exceptions && $event['reference'])
+		elseif (!$this->client_shared_uid_exceptions && $event['reference'])
 		{
-			$events[0]['uid'] .= '-'.$event['id'];	// force a different uid
+			$events[0]['uid'] .= '-'.$event['id']; // force a different uid
 		}
 		return $handler->exportVCal($events, '2.0', $method);
 	}
@@ -710,20 +710,20 @@ 
 	 * @param int $user =null account_id of calendar to display, to remove master, if current user does not participate in
 	 * @return array
 	 */
-	private static function &get_series($uid,calendar_bo $bo=null, $expand=false, $user=null)
+	private static function &get_series($uid, calendar_bo $bo = null, $expand = false, $user = null)
 	{
 		if (is_null($bo)) $bo = new calendar_bopdate();
 
 		$params = array(
 			'query' => array('cal_uid' => $uid),
-			'filter' => 'owner',  // return all possible entries
+			'filter' => 'owner', // return all possible entries
 			'daywise' => false,
 			'date_format' => 'server',
-			'cfs' => array(),	// read cfs as we use them to store X- attributes
+			'cfs' => array(), // read cfs as we use them to store X- attributes
 		);
 		if (is_array($expand)) $params += $expand;
 
-		if (!($events =& $bo->search($params)))
+		if (!($events = & $bo->search($params)))
 		{
 			return array();
 		}
@@ -731,12 +731,12 @@ 
 		// find master, which is not always first event, eg. when first event is an exception
 		$master = null;
 		$exceptions = array();
-		foreach($events as $k => &$recurrence)
+		foreach ($events as $k => &$recurrence)
 		{
 			if ($recurrence['recur_type'])
 			{
 				$master = $recurrence;
-				$exceptions =& $master['recur_exception'];
+				$exceptions = & $master['recur_exception'];
 				unset($events[$k]);
 				break;
 			}
@@ -744,13 +744,13 @@ 
 		// if recurring event starts in future behind horizont, nothing will be returned by bo::search()
 		if (!isset($master)) $master = $bo->read($uid);
 
-		foreach($events as $k => &$recurrence)
+		foreach ($events as $k => &$recurrence)
 		{
 			//error_log(__FILE__.'['.__LINE__.'] '.__METHOD__."($uid)[$k]:" . array2string($recurrence));
 			if ($master && $recurrence['reference'] != $master['id'])
 			{
 				unset($events[$k]);
-				continue;	// same uid, but references a different event or is own master
+				continue; // same uid, but references a different event or is own master
 			}
 			if (!$master || $recurrence['id'] != $master['id'])	// real exception
 			{
@@ -771,11 +771,11 @@ 
 				// at least Lightning "understands" EXDATE as exception from what's included
 				// in the whole resource / VCALENDAR component
 				// not removing it causes Lightning to remove the exception itself
-				if (($e = array_search($recurrence['recurrence'],$exceptions)) !== false)
+				if (($e = array_search($recurrence['recurrence'], $exceptions)) !== false)
 				{
 					unset($exceptions[$e]);
 				}
-				continue;	// nothing to change
+				continue; // nothing to change
 			}
 			// add alarms from master to recurrences, as clients otherwise have no alarms on virtual exceptions
 			if ($master && $master['alarm'])
@@ -786,14 +786,14 @@ 
 			if (!$expand && $master && $master['participants'] == $recurrence['participants'])
 			{
 				//error_log('NO exception: '.array2string($recurrence));
-				unset($events[$k]);	// no exception --> remove it
+				unset($events[$k]); // no exception --> remove it
 				continue;
 			}
 			// this is a virtual exception now (no extra event/cal_id in DB)
 			//error_log('virtual exception: '.array2string($recurrence));
 			$recurrence['recurrence'] = $recurrence['start'];
 			if ($master) $recurrence['reference'] = $master['id'];
-			$recurrence['recur_type'] = MCAL_RECUR_NONE;	// is set, as this is a copy of the master
+			$recurrence['recur_type'] = MCAL_RECUR_NONE; // is set, as this is a copy of the master
 			// not for included exceptions (Lightning): $master['recur_exception'][] = $recurrence['start'];
 		}
 		// only add master if we are not expanding and current user participates in master (and not just some exceptions)
@@ -829,11 +829,11 @@ 
 	 * @param string $prefix =null user prefix from path (eg. /ralf from /ralf/addressbook)
 	 * @return mixed boolean true on success, false on failure or string with http status (eg. '404 Not Found')
 	 */
-	function put(&$options,$id,$user=null,$prefix=null)
+	function put(&$options, $id, $user = null, $prefix = null)
 	{
-		if ($this->debug) error_log(__METHOD__."($id, $user)".print_r($options,true));
+		if ($this->debug) error_log(__METHOD__."($id, $user)".print_r($options, true));
 
-		if (!$prefix) $user = null;	// /infolog/ does not imply setting the current user (for new entries it's done anyway)
+		if (!$prefix) $user = null; // /infolog/ does not imply setting the current user (for new entries it's done anyway)
 
 		// work around missing handling / racecondition in Lightning, if event already exists on server,
 		// but Lightning has not yet synced with the server: Lightning just retries the PUT, not GETing the event
@@ -850,12 +850,12 @@ 
 		{
 			$_SERVER['HTTP_IF_SCHEDULE_TAG_MATCH'] = $_SERVER['HTTP_IF_SCHEDULE'];
 		}
-		$return_no_access = true;	// as handled by importVCal anyway and allows it to set the status for participants
-		$oldEvent = $this->_common_get_put_delete('PUT',$options,$id,$return_no_access,
-			isset($_SERVER['HTTP_IF_SCHEDULE_TAG_MATCH']));	// dont fail with 412 Precondition Failed in that case
+		$return_no_access = true; // as handled by importVCal anyway and allows it to set the status for participants
+		$oldEvent = $this->_common_get_put_delete('PUT', $options, $id, $return_no_access,
+			isset($_SERVER['HTTP_IF_SCHEDULE_TAG_MATCH'])); // dont fail with 412 Precondition Failed in that case
 		if (!is_null($oldEvent) && !is_array($oldEvent))
 		{
-			if ($this->debug) error_log(__METHOD__.': '.print_r($oldEvent,true).function_backtrace());
+			if ($this->debug) error_log(__METHOD__.': '.print_r($oldEvent, true).function_backtrace());
 			return $oldEvent;
 		}
 
@@ -947,7 +947,7 @@ 
 				if (($events = $handler->icaltoegw($vCalendar)))
 				{
 					$modified = 0;
-					foreach($events as $n => $event)
+					foreach ($events as $n => $event)
 					{
 						// for recurrances of event series, we need to read correct recurrence (or if series master is no first event)
 						if ($event['recurrence'] || $n && !$event['recurrence'] || isset($series))
@@ -958,7 +958,7 @@ 
 								$series = self::get_series($event['uid'], $this->bo);
 								//foreach($series as $s => $sEvent) error_log("series[$s]: ".array2string($sEvent));
 							}
-							foreach($series as $oldEvent)
+							foreach ($series as $oldEvent)
 							{
 								if ($oldEvent['recurrence'] == $event['recurrence']) break;
 							}
@@ -1023,9 +1023,9 @@ 
 					isset($oldEvent['participants'][$GLOBALS['egw_info']['user']['account_id']]))
 				{
 					// just update etag in database
-					$GLOBALS['egw']->db->update($this->bo->so->cal_table,'cal_etag=cal_etag+1',array(
+					$GLOBALS['egw']->db->update($this->bo->so->cal_table, 'cal_etag=cal_etag+1', array(
 						'cal_id' => $eventId,
-					),__LINE__,__FILE__,'calendar');
+					), __LINE__, __FILE__, 'calendar');
 				}
 			}
 		}
@@ -1077,9 +1077,9 @@ 
 	 * @param int $user =null account_id of owner, default null
 	 * @return mixed boolean true on success, false on failure or string with http status (eg. '404 Not Found')
 	 */
-	function post(&$options,$id,$user=null)
+	function post(&$options, $id, $user = null)
 	{
-		if ($this->debug) error_log(__METHOD__."($id, $user)".print_r($options,true));
+		if ($this->debug) error_log(__METHOD__."($id, $user)".print_r($options, true));
 
 		$vCalendar = htmlspecialchars_decode($options['content']);
 		$charset = null;
@@ -1101,7 +1101,7 @@ 
 			}
 		}
 
-		if (substr($options['path'],-8) == '/outbox/')
+		if (substr($options['path'], -8) == '/outbox/')
 		{
 			if (preg_match('/^METHOD:REQUEST(\r\n|\r|\n)(.*)^BEGIN:VFREEBUSY/ism', $vCalendar))
 			{
@@ -1152,7 +1152,7 @@ 
 	 */
 	protected function outbox_freebusy_request($ical, $charset, $user, array &$options)
 	{
-		unset($options);	// not used, but required by function signature
+		unset($options); // not used, but required by function signature
 
 		$vcal = new Horde_Icalendar();
 		if (!$vcal->parsevCalendar($ical, 'VCALENDAR', $charset))
@@ -1165,7 +1165,7 @@ 
 
 		$handler = $this->_get_handler();
 		$handler->setSupportedFields('groupdav');
-		$handler->calendarOwner = $handler->user = 0;	// to NOT default owner/organizer to something
+		$handler->calendarOwner = $handler->user = 0; // to NOT default owner/organizer to something
 		if (!($component = $vcal->getComponent(0)) ||
 			!($event = $handler->vevent2egw($component, $version, $handler->supportedFields, $this->caldav->current_user_principal, 'Horde_Icalendar_Vfreebusy')))
 		{
@@ -1190,13 +1190,13 @@ 
 		$xml->startDocument('1.0', 'UTF-8');
 		$xml->startElementNs('C', 'schedule-response', Api\CalDAV::CALDAV);
 
-		foreach(array_keys($event['participants']) as $uid)
+		foreach (array_keys($event['participants']) as $uid)
 		{
 			$xml->startElementNs('C', 'response', null);
 
 			$xml->startElementNs('C', 'recipient', null);
-			$xml->writeElementNs('D', 'href', 'DAV:', $attendee=array_shift($attendees));
-			$xml->endElement();	// recipient
+			$xml->writeElementNs('D', 'href', 'DAV:', $attendee = array_shift($attendees));
+			$xml->endElement(); // recipient
 
 			$xml->writeElementNs('C', 'request-status', null, '2.0;Success');
 			$xml->writeElementNs('C', 'calendar-data', null,
@@ -1204,13 +1204,13 @@ 
 					'UID' => $event['uid'],
 					'ORGANIZER' => $organizer,
 					'ATTENDEE' => $attendee,
-				)+(empty($mask_uid) || !is_string($mask_uid) ? array() : array(
+				) + (empty($mask_uid) || !is_string($mask_uid) ? array() : array(
 					'X-CALENDARSERVER-MASK-UID' => $mask_uid,
 				))));
 
-			$xml->endElement();	// response
+			$xml->endElement(); // response
 		}
-		$xml->endElement();	// schedule-response
+		$xml->endElement(); // schedule-response
 		$xml->endDocument();
 		echo $xml->outputMemory();
 
@@ -1225,14 +1225,14 @@ 
 	 * @param int $user account_id
 	 * @return mixed boolean true on success, false on failure or string with http status (eg. '404 Not Found')
 	 */
-	function free_busy_report($path,$options,$user)
+	function free_busy_report($path, $options, $user)
 	{
-		unset($path);	// unused, but required by function signature
+		unset($path); // unused, but required by function signature
 		if (!$this->bo->check_perms(calendar_bo::ACL_FREEBUSY, 0, $user))
 		{
 			return '403 Forbidden';
 		}
-		foreach($options['other'] as $filter)
+		foreach ($options['other'] as $filter)
 		{
 			if ($filter['name'] == 'time-range')
 			{
@@ -1244,7 +1244,7 @@ 
 		header('Content-Type: text/calendar');
 		echo $handler->freebusy($user, $end, true, 'utf-8', $start, 'REPLY', array());
 
-		exit();	// otherwise we get a 207 multistatus, not 200 Ok
+		exit(); // otherwise we get a 207 multistatus, not 200 Ok
 	}
 
 	/**
@@ -1256,7 +1256,7 @@ 
 	 * @param int $user =null owner of the collection, default current user
 	 * @return array with privileges
 	 */
-	public function current_user_privileges($path, $user=null)
+	public function current_user_privileges($path, $user = null)
 	{
 		$privileges = parent::current_user_privileges($path, $user);
 		//error_log(__METHOD__."('$path', $user) parent gave ".array2string($privileges));
@@ -1299,7 +1299,7 @@ 
 
 		// get array with orginal recurrences indexed by recurrence-id
 		$org_recurrences = $exceptions = array();
-		foreach(self::get_series($events[0]['uid'],$bo) as $k => $event)
+		foreach (self::get_series($events[0]['uid'], $bo) as $k => $event)
 		{
 			if (!$k) $master = $event;
 			if ($event['recurrence'])
@@ -1309,13 +1309,13 @@ 
 		}
 
 		// assign cal_id's to already existing recurrences and evtl. re-add recur_exception to master
-		foreach($events as $k => &$recurrence)
+		foreach ($events as $k => &$recurrence)
 		{
 			if (!$recurrence['recurrence'])
 			{
 				// master
 				$recurrence['id'] = $master['id'];
-				$master =& $events[$k];
+				$master = & $events[$k];
 				continue;
 			}
 
@@ -1339,12 +1339,12 @@ 
 		$master['recur_exception'] = array_merge($exceptions, $master['recur_exception']);
 
 		// delete not longer existing recurrences
-		foreach($org_recurrences as $org_recurrence)
+		foreach ($org_recurrences as $org_recurrence)
 		{
 			if ($org_recurrence['id'] != $master['id'])	// non-virtual recurrence
 			{
 				//error_log(__METHOD__.'() deleting #'.$org_recurrence['id']);
-				$bo->delete($org_recurrence['id']);	// might fail because of permissions
+				$bo->delete($org_recurrence['id']); // might fail because of permissions
 			}
 			else	// virtual recurrence
 			{
@@ -1366,14 +1366,14 @@ 
 	 * @param int $user account_id of collection owner
 	 * @return mixed boolean true on success, false on failure or string with http status (eg. '404 Not Found')
 	 */
-	function delete(&$options,$id,$user)
+	function delete(&$options, $id, $user)
 	{
 		if (strpos($options['path'], '/inbox/') !== false)
 		{
-			return true;	// simply ignore DELETE in inbox for now
+			return true; // simply ignore DELETE in inbox for now
 		}
-		$return_no_access = true;	// to allow to check if current use is a participant and reject the event for him
-		if (!is_array($event = $this->_common_get_put_delete('DELETE',$options,$id,$return_no_access)) || !$return_no_access ||
+		$return_no_access = true; // to allow to check if current use is a participant and reject the event for him
+		if (!is_array($event = $this->_common_get_put_delete('DELETE', $options, $id, $return_no_access)) || !$return_no_access ||
 			// Work around problems with Outlook CalDAV Synchroniser (https://caldavsynchronizer.org/)
 			// - sends a DELETE to reject a meeting request --> deletes event for all participants, if user has delete rights on the calendar
 			// --> only set status for everyone else but the organizer
@@ -1384,11 +1384,11 @@ 
 				// check if user is a participant or one of the groups he is a member of --> reject the meeting request
 				$ret = '403 Forbidden';
 				$memberships = $GLOBALS['egw']->accounts->memberships($this->bo->user, true);
-				foreach(array_keys($event['participants']) as $uid)
+				foreach (array_keys($event['participants']) as $uid)
 				{
 					if ($this->bo->user == $uid || in_array($uid, $memberships))
 					{
-						$this->bo->set_status($event,$this->bo->user, 'R');
+						$this->bo->set_status($event, $this->bo->user, 'R');
 						$ret = true;
 						break;
 					}
@@ -1403,7 +1403,7 @@ 
 		{
 			$ret = $this->bo->delete($event['id']);
 		}
-		if ($this->debug) error_log(__METHOD__."(,$id) return_no_access=$return_no_access, event[participants]=".array2string(is_array($event)?$event['participants']:null).", user={$this->bo->user} --> return ".array2string($ret));
+		if ($this->debug) error_log(__METHOD__."(,$id) return_no_access=$return_no_access, event[participants]=".array2string(is_array($event) ? $event['participants'] : null).", user={$this->bo->user} --> return ".array2string($ret));
 		return $ret;
 	}
 
@@ -1418,12 +1418,12 @@ 
 	 */
 	function read($id)
 	{
-		if (strpos($column=self::$path_attr,'_') === false) $column = 'cal_'.$column;
+		if (strpos($column = self::$path_attr, '_') === false) $column = 'cal_'.$column;
 
 		$event = $this->bo->read(array($column => $id, 'cal_deleted IS NULL', 'cal_reference=0'), null, true, 'server');
-		if ($event) $event = array_shift($event);	// read with array as 1. param, returns an array of events!
+		if ($event) $event = array_shift($event); // read with array as 1. param, returns an array of events!
 
-		if (!($retval = $this->bo->check_perms(calendar_bo::ACL_FREEBUSY,$event, 0, 'server')) &&
+		if (!($retval = $this->bo->check_perms(calendar_bo::ACL_FREEBUSY, $event, 0, 'server')) &&
 			// above can be true, if current user is not in master but just a recurrence
 			(!$event['recur_type'] || !($events = self::get_series($event['uid'], $this->bo))))
 		{
@@ -1459,11 +1459,11 @@ 
 	 *
 	 * @return string
 	 */
-	public function getctag($path,$user)
+	public function getctag($path, $user)
 	{
-		$ctag = $this->bo->get_ctag($user,$path == '/calendar/' ? 'owner' : 'default'); // default = not rejected
+		$ctag = $this->bo->get_ctag($user, $path == '/calendar/' ? 'owner' : 'default'); // default = not rejected
 
-		if ($this->debug > 1) error_log(__FILE__.'['.__LINE__.'] '.__METHOD__. "($path)[$user] = $ctag");
+		if ($this->debug > 1) error_log(__FILE__.'['.__LINE__.'] '.__METHOD__."($path)[$user] = $ctag");
 
 		return $ctag;
 	}
@@ -1475,7 +1475,7 @@ 
 	 * @param string $schedule_tag =null on return schedule-tag
 	 * @return string|boolean string with etag or false
 	 */
-	function get_etag($entry, &$schedule_tag=null)
+	function get_etag($entry, &$schedule_tag = null)
 	{
 		$etag = $this->bo->get_etag($entry, $schedule_tag, $this->client_shared_uid_exceptions);
 
@@ -1493,7 +1493,7 @@ 
 	 * @param int|string $retval
 	 * @param boolean $path_attr_is_name =true true: path_attr is ca(l|rd)dav_name, false: id (GroupDAV needs Location header)
 	 */
-	function put_response_headers($entry, $path, $retval, $path_attr_is_name=true)
+	function put_response_headers($entry, $path, $retval, $path_attr_is_name = true)
 	{
 		$schedule_tag = null;
 		$etag = $this->get_etag($entry, $schedule_tag);
@@ -1512,14 +1512,14 @@ 
 	 * @param array|int $event event-array or id
 	 * @return boolean null if entry does not exist, false if no access, true if access permitted
 	 */
-	function check_access($acl,$event)
+	function check_access($acl, $event)
 	{
 		if ($acl == Acl::READ)
 		{
 			// we need at least calendar_bo::ACL_FREEBUSY to get some information
 			$acl = calendar_bo::ACL_FREEBUSY;
 		}
-		return $this->bo->check_perms($acl,$event,0,'server');
+		return $this->bo->check_perms($acl, $event, 0, 'server');
 	}
 
 	/**
@@ -1532,52 +1532,52 @@ 
 	 * @param string $path =null path of the collection
 	 * @return array
 	 */
-	public function extra_properties(array $props, $displayname, $base_uri=null, $user=null, $path=null)
+	public function extra_properties(array $props, $displayname, $base_uri = null, $user = null, $path = null)
 	{
-		unset($base_uri);	// unused, but required by function signature
+		unset($base_uri); // unused, but required by function signature
 		if (!isset($props['calendar-description']))
 		{
 			// default calendar description: can be overwritten via PROPPATCH, in which case it's already set
-			$props['calendar-description'] = Api\CalDAV::mkprop(Api\CalDAV::CALDAV,'calendar-description',$displayname);
+			$props['calendar-description'] = Api\CalDAV::mkprop(Api\CalDAV::CALDAV, 'calendar-description', $displayname);
 		}
 		$supported_components = array(
-			Api\CalDAV::mkprop(Api\CalDAV::CALDAV,'comp',array('name' => 'VEVENT')),
+			Api\CalDAV::mkprop(Api\CalDAV::CALDAV, 'comp', array('name' => 'VEVENT')),
 		);
 		// outbox supports VFREEBUSY too, it is required from OS X iCal to autocomplete locations
-		if (substr($path,-8) == '/outbox/')
+		if (substr($path, -8) == '/outbox/')
 		{
-			$supported_components[] = Api\CalDAV::mkprop(Api\CalDAV::CALDAV,'comp',array('name' => 'VFREEBUSY'));
+			$supported_components[] = Api\CalDAV::mkprop(Api\CalDAV::CALDAV, 'comp', array('name' => 'VFREEBUSY'));
 		}
 		$props['supported-calendar-component-set'] = Api\CalDAV::mkprop(Api\CalDAV::CALDAV,
-			'supported-calendar-component-set',$supported_components);
+			'supported-calendar-component-set', $supported_components);
 		// supported reports
 		$props['supported-report-set'] = array(
-			'calendar-query' => Api\CalDAV::mkprop('supported-report',array(
-				Api\CalDAV::mkprop('report',array(
-					Api\CalDAV::mkprop(Api\CalDAV::CALDAV,'calendar-query',''))))),
-			'calendar-multiget' => Api\CalDAV::mkprop('supported-report',array(
-				Api\CalDAV::mkprop('report',array(
-					Api\CalDAV::mkprop(Api\CalDAV::CALDAV,'calendar-multiget',''))))),
-			'free-busy-query' => Api\CalDAV::mkprop('supported-report',array(
-				Api\CalDAV::mkprop('report',array(
-					Api\CalDAV::mkprop(Api\CalDAV::CALDAV,'free-busy-query',''))))),
+			'calendar-query' => Api\CalDAV::mkprop('supported-report', array(
+				Api\CalDAV::mkprop('report', array(
+					Api\CalDAV::mkprop(Api\CalDAV::CALDAV, 'calendar-query', ''))))),
+			'calendar-multiget' => Api\CalDAV::mkprop('supported-report', array(
+				Api\CalDAV::mkprop('report', array(
+					Api\CalDAV::mkprop(Api\CalDAV::CALDAV, 'calendar-multiget', ''))))),
+			'free-busy-query' => Api\CalDAV::mkprop('supported-report', array(
+				Api\CalDAV::mkprop('report', array(
+					Api\CalDAV::mkprop(Api\CalDAV::CALDAV, 'free-busy-query', ''))))),
 		);
 		// rfc 6578 sync-collection report for everything but outbox
 		// only if "delete-prevention" is switched on (deleted entries get marked deleted but not actualy deleted
 		if (strpos($path, '/outbox/') === false && $GLOBALS['egw_info']['server']['calendar_delete_history'])
 		{
-			$props['supported-report-set']['sync-collection'] = Api\CalDAV::mkprop('supported-report',array(
-				Api\CalDAV::mkprop('report',array(
-					Api\CalDAV::mkprop('sync-collection','')))));
+			$props['supported-report-set']['sync-collection'] = Api\CalDAV::mkprop('supported-report', array(
+				Api\CalDAV::mkprop('report', array(
+					Api\CalDAV::mkprop('sync-collection', '')))));
 		}
-		$props['supported-calendar-data'] = Api\CalDAV::mkprop(Api\CalDAV::CALDAV,'supported-calendar-data',array(
-			Api\CalDAV::mkprop(Api\CalDAV::CALDAV,'calendar-data', array('content-type' => 'text/calendar', 'version'=> '2.0')),
-			Api\CalDAV::mkprop(Api\CalDAV::CALDAV,'calendar-data', array('content-type' => 'text/x-calendar', 'version'=> '1.0'))));
+		$props['supported-calendar-data'] = Api\CalDAV::mkprop(Api\CalDAV::CALDAV, 'supported-calendar-data', array(
+			Api\CalDAV::mkprop(Api\CalDAV::CALDAV, 'calendar-data', array('content-type' => 'text/calendar', 'version'=> '2.0')),
+			Api\CalDAV::mkprop(Api\CalDAV::CALDAV, 'calendar-data', array('content-type' => 'text/x-calendar', 'version'=> '1.0'))));
 
 		// get timezone of calendar
 		if ($this->caldav->prop_requested('calendar-timezone'))
 		{
-			$props['calendar-timezone'] = Api\CalDAV::mkprop(Api\CalDAV::CALDAV,'calendar-timezone',
+			$props['calendar-timezone'] = Api\CalDAV::mkprop(Api\CalDAV::CALDAV, 'calendar-timezone',
 				calendar_timezones::user_timezone($user));
 		}
 		return $props;
@@ -1591,9 +1591,9 @@ 
 	private function _get_handler()
 	{
 		$handler = new calendar_ical();
-		$handler->setSupportedFields('GroupDAV',$this->agent);
-		$handler->supportedFields['attachments'] = true;	// enabling attachments
-		if ($this->debug > 1) error_log("ical Handler called: " . $this->agent);
+		$handler->setSupportedFields('GroupDAV', $this->agent);
+		$handler->supportedFields['attachments'] = true; // enabling attachments
+		if ($this->debug > 1) error_log("ical Handler called: ".$this->agent);
 		return $handler;
 	}
 
@@ -1608,7 +1608,7 @@ 
 		$pref = $GLOBALS['egw_info']['user']['preferences']['groupdav']['calendar-home-set'];
 		$calendar_home_set = $pref ? explode(',', $pref) : array();
 		// replace symbolic id's with real nummeric id's
-		foreach(array(
+		foreach (array(
 			'G' => $GLOBALS['egw_info']['user']['account_primary_group'],
 		) as $sym => $id)
 		{
@@ -1617,11 +1617,11 @@ 
 				$calendar_home_set[$key] = $id;
 			}
 		}
-		foreach(ExecMethod('calendar.calendar_bo.list_cals') as $entry)
+		foreach (ExecMethod('calendar.calendar_bo.list_cals') as $entry)
 		{
 			$id = $entry['grantor'];
-			if ($id && $GLOBALS['egw_info']['user']['account_id'] != $id &&	// no current user
-				(in_array('A',$calendar_home_set) || in_array((string)$id,$calendar_home_set)) &&
+			if ($id && $GLOBALS['egw_info']['user']['account_id'] != $id && // no current user
+				(in_array('A', $calendar_home_set) || in_array((string)$id, $calendar_home_set)) &&
 				is_numeric($id) && ($owner = $this->accounts->id2name($id)))
 			{
 				$shared[$id] = 'calendar-'.$owner;
@@ -1630,11 +1630,11 @@ 
 		// shared locations and resources
 		if ($GLOBALS['egw_info']['user']['apps']['resources'])
 		{
-			foreach(array('locations','resources') as $res)
+			foreach (array('locations', 'resources') as $res)
 			{
 				if (($pref = $GLOBALS['egw_info']['user']['preferences']['groupdav']['calendar-home-set-'.$res]))
 				{
-					foreach(explode(',', $pref) as $res_id)
+					foreach (explode(',', $pref) as $res_id)
 					{
 						$is_location = $res == 'locations';
 						$shared['r'.$res_id] = str_replace('s/', '-', Api\CalDAV\Principals::resource2name($res_id, $is_location));
@@ -1664,7 +1664,7 @@ 
 			{
 				$calendars[$entry['grantor']] = $entry['name'];
 			}
-			if ($user > 0) unset($calendars[$user]);	// skip current user
+			if ($user > 0) unset($calendars[$user]); // skip current user
 		}
 
 		$settings = array();
@@ -1672,7 +1672,7 @@ 
 			'type'   => 'multiselect',
 			'label'  => 'Calendars to sync in addition to personal calendar',
 			'name'   => 'calendar-home-set',
-			'help'   => lang('Only supported by a few fully conformant clients (eg. from Apple). If you have to enter a URL, it will most likly not be suppored!').'<br/>'.lang('They will be sub-folders in users home (%1 attribute).','CalDAV "calendar-home-set"'),
+			'help'   => lang('Only supported by a few fully conformant clients (eg. from Apple). If you have to enter a URL, it will most likly not be suppored!').'<br/>'.lang('They will be sub-folders in users home (%1 attribute).', 'CalDAV "calendar-home-set"'),
 			'values' => $calendars,
 			'xmlrpc' => True,
 			'admin'  => False,
@@ -1698,7 +1698,7 @@ 
 		if ($GLOBALS['egw_info']['user']['apps']['resources'] && ($all_resources = Api\CalDAV\Principals::get_resources()))
 		{
 			$resources = $locations = array();
-			foreach($all_resources as $resource)
+			foreach ($all_resources as $resource)
 			{
 				if (Api\CalDAV\Principals::resource_is_location($resource))
 				{
@@ -1709,7 +1709,7 @@ 
 					$resources[$resource['res_id']] = $resource['name'];
 				}
 			}
-			foreach(array(
+			foreach (array(
 				'locations' => $locations,
 				'resources' => $resources,
 			) as $name => $options)
@@ -1722,7 +1722,7 @@ 
 						'label'  => lang('%1 to sync', lang($name == 'locations' ? 'Location calendars' : 'Resource calendars')),
 						'no_lang'=> true,
 						'name'   => 'calendar-home-set-'.$name,
-						'help'   => lang('Only supported by a few fully conformant clients (eg. from Apple). If you have to enter a URL, it will most likly not be suppored!').'<br/>'.lang('They will be sub-folders in users home (%1 attribute).','CalDAV "calendar-home-set"'),
+						'help'   => lang('Only supported by a few fully conformant clients (eg. from Apple). If you have to enter a URL, it will most likly not be suppored!').'<br/>'.lang('They will be sub-folders in users home (%1 attribute).', 'CalDAV "calendar-home-set"'),
 						'values' => $options,
 						'xmlrpc' => True,
 						'admin'  => False,