EGroupware / egroupware

calendar/inc/class.calendar_wizard_export_csv.inc.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
 		unset($this->export_fields['participants-r']);
 
 		$custom = Api\Storage\Customfields::get('calendar', true);
-		foreach($custom as $name => $data) {
+		foreach ($custom as $name => $data) {
 			$this->export_fields['#'.$name] = $data['label'];
 		}
 	}

calendar/inc/class.calendar_egw_record.inc.php

Spacing +12 added lines, -12 removed lines

@@ -24,8 +24,8 @@ 
 
 	public static $types = array(
 		'select-cat'    => array('category'),
-		'select-account'=> array('owner','creator', 'modifier'),
-		'date-time'     => array('modified', 'created','start','end','recur_date'),
+		'select-account'=> array('owner', 'creator', 'modifier'),
+		'date-time'     => array('modified', 'created', 'start', 'end', 'recur_date'),
 		'date'			=> array('recur_enddate'),
 		'select-bool'	=> array('public', 'non_blocking'),
 		'select'	=> array('priority'),
@@ -37,12 +37,12 @@ 
 	 *
 	 * @param string $_identifier
 	 */
-	public function __construct( $_identifier='' ){
+	public function __construct($_identifier = '') {
 		$this->identifier = $_identifier;
-		if(!is_object($this->bo)) {
+		if (!is_object($this->bo)) {
 			$this->bo = new calendar_bo();
 		}
-		if($this->identifier) {
+		if ($this->identifier) {
 			$this->record = $this->bo->read($this->identifier);
 		}
 	}
@@ -96,7 +96,7 @@ 
 	 * @todo add some checks
 	 * @return void
 	 */
-	public function set_record(array $_record){
+	public function set_record(array $_record) {
 		$this->record = $_record;
 	}
 
@@ -124,7 +124,7 @@ 
 	 *
 	 * @return string identifier
 	 */
-	public function save ( $_dst_identifier ) {
+	public function save($_dst_identifier) {
 		// Not yet implemeted
 		$this->identifier = $_dst_identifier;
 	}
@@ -135,8 +135,8 @@ 
 	 * @param string $_dst_identifier
 	 * @return string dst_identifier
 	 */
-	public function copy ( $_dst_identifier ) {
-		unset($_dst_identifier);	// not used
+	public function copy($_dst_identifier) {
+		unset($_dst_identifier); // not used
 	}
 
 	/**
@@ -146,15 +146,15 @@ 
 	 * @param string $_dst_identifier
 	 * @return string dst_identifier
 	 */
-	public function move ( $_dst_identifier ) {
-		unset($_dst_identifier);	// not used
+	public function move($_dst_identifier) {
+		unset($_dst_identifier); // not used
 	}
 
 	/**
 	 * delets current record from backend
 	 *
 	 */
-	public function delete () {
+	public function delete() {
 
 	}
 

calendar/inc/class.calendar_timegrid_etemplate_widget.inc.php

Spacing +9 added lines, -9 removed lines

@@ -29,24 +29,24 @@
 	 * @param string $cname
 	 * @param array $expand values for keys 'c', 'row', 'c_', 'row_', 'cont'
 	 */
-	public function beforeSendToClient($cname, array $expand=null)
+	public function beforeSendToClient($cname, array $expand = null)
 	{
 		$form_name = self::form_name($cname, $this->id, $expand);
-		$value =& self::get_array(self::$request->content, $form_name, true);
-		if(!is_array($value)) $value = array();
+		$value = & self::get_array(self::$request->content, $form_name, true);
+		if (!is_array($value)) $value = array();
 
-		foreach($value as &$events)
+		foreach ($value as &$events)
 		{
-			if(!is_array($events))
+			if (!is_array($events))
 			{
 				continue;
 			}
-			foreach($events as &$event)
+			foreach ($events as &$event)
 			{
-				if(!is_array($event)) continue;
-				foreach(array('start','end') as $date)
+				if (!is_array($event)) continue;
+				foreach (array('start', 'end') as $date)
 				{
-					$event[$date] = Api\DateTime::to($event[$date],'Y-m-d\TH:i:s\Z');
+					$event[$date] = Api\DateTime::to($event[$date], 'Y-m-d\TH:i:s\Z');
 				}
 			}
 		}

calendar/inc/class.calendar_timezones.inc.php

Spacing +47 added lines, -47 removed lines

@@ -71,7 +71,7 @@ 
 	 */
 	public static function DateTimeZone($tzid)
 	{
-		if (($id = self::tz2id($tzid,'alias')))
+		if (($id = self::tz2id($tzid, 'alias')))
 		{
 			$tzid = self::id2tz($id);
 		}
@@ -89,34 +89,34 @@ 
 	 * @param string $what ='id' what to return, default id, null for whole array
 	 * @return int tz_id or null if not found
 	 */
-	public static function tz2id($tzid,$what='id')
+	public static function tz2id($tzid, $what = 'id')
 	{
-		$id =& self::$tz2id[$tzid];
+		$id = & self::$tz2id[$tzid];
 
 		if (!isset($id))
 		{
-			if (($data = $GLOBALS['egw']->db->select(self::TABLE,'*',array(
+			if (($data = $GLOBALS['egw']->db->select(self::TABLE, '*', array(
 				'tz_tzid' => $tzid,
-			),__LINE__,__FILE__,false,'','calendar')->fetch()))
+			), __LINE__, __FILE__, false, '', 'calendar')->fetch()))
 			{
 				$id = $data['tz_id'];
-				self::$tz_cache[$id] = Api\Db::strip_array_keys($data,'tz_');
+				self::$tz_cache[$id] = Api\Db::strip_array_keys($data, 'tz_');
 			}
 		}
 		// check if we can find a 3-part America timezone eg. check 'America/Argentina/Buenos_Aires' for 'America/Buenos_Aires'
 		if (!isset($id) && stripos($tzid, 'America/') === 0 && count($parts = explode('/', $tzid)) == 2)
 		{
-			if (($data = $GLOBALS['egw']->db->select(self::TABLE,'*',array(
+			if (($data = $GLOBALS['egw']->db->select(self::TABLE, '*', array(
 				'tz_tzid LIKE '.$GLOBALS['egw']->db->quote($parts[0].'/%/'.$parts[1]),
-			),__LINE__,__FILE__,false,'','calendar')->fetch()))
+			), __LINE__, __FILE__, false, '', 'calendar')->fetch()))
 			{
 				$id = $data['tz_id'];
-				self::$tz_cache[$id] = Api\Db::strip_array_keys($data,'tz_');
+				self::$tz_cache[$id] = Api\Db::strip_array_keys($data, 'tz_');
 			}
 		}
 		if (isset($id) && $what != 'id')
 		{
-			return self::id2tz($id,$what);
+			return self::id2tz($id, $what);
 		}
 		return $id;
 	}
@@ -134,24 +134,24 @@ 
 	 * @param string $what ='tzid' what data to return or null for whole data array, with keys 'id', 'tzid', 'component', 'alias', 'latitude', 'longitude'
 	 * @return mixed false: if not found
 	 */
-	public static function id2tz($id,$what='tzid')
+	public static function id2tz($id, $what = 'tzid')
 	{
-		$data =& self::$tz_cache[$id];
+		$data = & self::$tz_cache[$id];
 
 		if (!isset($data))
 		{
-			if (($data = $GLOBALS['egw']->db->select(self::TABLE,'*',array(
+			if (($data = $GLOBALS['egw']->db->select(self::TABLE, '*', array(
 				'tz_id' => $id,
-			),__LINE__,__FILE__,false,'','calendar')->fetch()))
+			), __LINE__, __FILE__, false, '', 'calendar')->fetch()))
 			{
-				$data = Api\Db::strip_array_keys($data,'tz_');
+				$data = Api\Db::strip_array_keys($data, 'tz_');
 				self::$tz2id[$data['tzid']] = $id;
 			}
 		}
 		// if not tzid queried, resolve aliases automatically
 		if ($data && $data['alias'] && $what != 'tzid' && $what != 'alias')
 		{
-			$data = self::id2tz($data['alias'],null);
+			$data = self::id2tz($data['alias'], null);
 		}
 		return !$data ? $data : ($what ? $data[$what] : $data);
 	}
@@ -170,8 +170,8 @@ 
 	 */
 	public static function init_static()
 	{
-		self::$tz_cache =& Api\Cache::getSession(__CLASS__,'tz_cache');
-		self::$tz2id =& Api\Cache::getSession(__CLASS__,'tz2id');
+		self::$tz_cache = & Api\Cache::getSession(__CLASS__, 'tz_cache');
+		self::$tz2id = & Api\Cache::getSession(__CLASS__, 'tz2id');
 
 		// init cache with mapping UTC <--> -1, as UTC is no real timezone, but we need to be able to use it in calendar
 		if (!is_array(self::$tz2id))
@@ -188,10 +188,10 @@ 
 		{
 			$updated = false;
 			$msg = self::import_zones($updated);
-			if ($updated) error_log($msg);	// log that timezones have been updated
+			if ($updated) error_log($msg); // log that timezones have been updated
 
 			$alias_msg = self::import_tz_aliases($updated);
-			if ($updated) error_log($alias_msg);	// log that timezone aliases have been updated
+			if ($updated) error_log($alias_msg); // log that timezone aliases have been updated
 
 			self::$import_msg = $msg.'<br/>'.$alias_msg;
 
@@ -210,7 +210,7 @@ 
 	 * @throws Api\Exception\WrongUserinput for broken sqlite extension
 	 * @link https://hg.mozilla.org/comm-central/raw-file/tip/calendar/timezones/zones.json
 	 */
-	public static function import_zones(&$updated=null, $file='calendar/setup/zones.json')
+	public static function import_zones(&$updated = null, $file = 'calendar/setup/zones.json')
 	{
 		$path = EGW_SERVER_ROOT.'/'.$file;
 
@@ -232,9 +232,9 @@ 
 		{
 			$updated = false;
 			fclose($fp);
-			return lang('Nothing to update, version is already %1.',$config['tz_version']);	// we already have the right
+			return lang('Nothing to update, version is already %1.', $config['tz_version']); // we already have the right
 		}
-		$json .= fread($fp, 1024*1024);
+		$json .= fread($fp, 1024 * 1024);
 		fclose($fp);
 		if (!($zones = json_decode($json, true)) || !isset($zones['aliases']) || !isset($zones['zones']))
 		{
@@ -242,38 +242,38 @@ 
 		}
 		// import zones first and then aliases
 		$tz2id = array();
-		foreach(array('zones', 'aliases') as $type)
+		foreach (array('zones', 'aliases') as $type)
 		{
-			foreach($zones[$type] as $tzid => $data)
+			foreach ($zones[$type] as $tzid => $data)
 			{
 				if ($type == 'aliases')
 				{
 					$data = array('alias' => $tz2id[$data['aliasTo']]);
-					if (!$data['alias']) continue;	// there's no such tzid
+					if (!$data['alias']) continue; // there's no such tzid
 				}
 				// check if already in database
-				$tz2id[$tzid] = $GLOBALS['egw']->db->select('egw_cal_timezones','tz_id',array(
+				$tz2id[$tzid] = $GLOBALS['egw']->db->select('egw_cal_timezones', 'tz_id', array(
 						'tz_tzid' => $tzid,
-					),__LINE__,__FILE__,false,'','calendar')->fetchColumn();
+					), __LINE__, __FILE__, false, '', 'calendar')->fetchColumn();
 
-				$GLOBALS['egw']->db->insert('egw_cal_timezones',array(
+				$GLOBALS['egw']->db->insert('egw_cal_timezones', array(
 					'tz_alias' => $data['alias'],
 					'tz_latitude' => $data['latitude'],
 					'tz_longitude' => $data['longitude'],
 					'tz_component' => $data['ics'],
-				),array(
+				), array(
 					'tz_tzid' => $tzid,
-				),__LINE__,__FILE__,'calendar');
+				), __LINE__, __FILE__, 'calendar');
 
 				// only query last insert id, if not already in database (gives warning for PostgreSQL)
-				if (!$tz2id[$tzid]) $tz2id[$tzid] = $GLOBALS['egw']->db->get_last_insert_id('egw_cal_timezones','tz_id');
+				if (!$tz2id[$tzid]) $tz2id[$tzid] = $GLOBALS['egw']->db->get_last_insert_id('egw_cal_timezones', 'tz_id');
 			}
 		}
 		Api\Config::save_value('tz_version', $tz_version, 'phpgwapi');
 
 		//_debug_array($tz2id);
 		$updated = true;
-		return lang('Timezones updated to version %1 (%2 records updated).',$tz_version,count($tz2id));
+		return lang('Timezones updated to version %1 (%2 records updated).', $tz_version, count($tz2id));
 	}
 
 	/**
@@ -284,7 +284,7 @@ 
 	 * @return string message about update
 	 * @throws Api\Exception\WrongParameter if $file is not readable or wrong format/version
 	 */
-	public static function import_tz_aliases(&$updated=null,$file='calendar/setup/tz_aliases.inc.php')
+	public static function import_tz_aliases(&$updated = null, $file = 'calendar/setup/tz_aliases.inc.php')
 	{
 		$path = EGW_SERVER_ROOT.'/'.$file;
 
@@ -297,27 +297,27 @@ 
 		if ($tz_aliases_mtime === $config['tz_aliases_mtime'])
 		{
 			$updated = false;
-			return lang('Nothing to update, version is already %1.',$tz_aliases_mtime);
+			return lang('Nothing to update, version is already %1.', $tz_aliases_mtime);
 		}
 		$tz_aliases = array();
-		include($path);	// sets $tz_aliases
+		include($path); // sets $tz_aliases
 
 		$updates = 0;
-		foreach($tz_aliases as $alias => $tzid)
+		foreach ($tz_aliases as $alias => $tzid)
 		{
-			if ((!($alias_id=self::tz2id($alias, 'alias')) || self::id2tz($alias_id, 'tzid') !== $tzid) &&	// not in DB or different
+			if ((!($alias_id = self::tz2id($alias, 'alias')) || self::id2tz($alias_id, 'tzid') !== $tzid) && // not in DB or different
 				($tz_id = self::tz2id($tzid)))	// given tzid for alias exists in DB
 			{
-				$GLOBALS['egw']->db->insert('egw_cal_timezones',array(
+				$GLOBALS['egw']->db->insert('egw_cal_timezones', array(
 					'tz_alias' => $tz_id,
-				),array(
+				), array(
 					'tz_tzid' => $alias,
-				),__LINE__,__FILE__,'calendar');
+				), __LINE__, __FILE__, 'calendar');
 				++$updates;
 			}
 			//error_log(__METHOD__."() alias=$alias, tzid=$tzid --> self::tz2id('$alias', 'alias') = ".array2string($alias_id).",  self::tz2id('$tzid')=".array2string($tz_id));
 		}
-		Api\Config::save_value('tz_aliases_mtime',$tz_aliases_mtime,$app='phpgwapi');
+		Api\Config::save_value('tz_aliases_mtime', $tz_aliases_mtime, $app = 'phpgwapi');
 
 		//_debug_array($tz2id);
 		$updated = true;
@@ -358,8 +358,8 @@ 
 		}
 		// $vtimezone is a string with a single VTIMEZONE component, afaik Horde_Icalendar can not add it directly
 		// --> we have to parse it and let Horde_Icalendar add it again
-		$horde_vtimezone = Horde_Icalendar::newComponent('VTIMEZONE',$container=false);
-		$horde_vtimezone->parsevCalendar($vtimezone,'VTIMEZONE');
+		$horde_vtimezone = Horde_Icalendar::newComponent('VTIMEZONE', $container = false);
+		$horde_vtimezone->parsevCalendar($vtimezone, 'VTIMEZONE');
 		// DTSTART is in UTC time, Horde_Icalendar parses it in server timezone, which we need to set again for printing
 		$standard = $horde_vtimezone->findComponent('STANDARD');
 		if (is_a($standard, 'Horde_Icalendar'))
@@ -390,7 +390,7 @@ 
 	 * @param string $type ='vcalendar' 'tzid' or everything tz2id supports, default 'vcalendar' = full vcalendar component
 	 * @return string
 	 */
-	public static function user_timezone($user=null, $type='vcalendar')
+	public static function user_timezone($user = null, $type = 'vcalendar')
 	{
 		if (!$user || $user == $GLOBALS['egw_info']['user']['account_id'])
 		{
@@ -409,7 +409,7 @@ 
 			case 'vcalendar':
 				// checking type of $val, now we included the object definition (no need to always include it!)
 				$vcal = new Horde_Icalendar;
-				$vcal->setAttribute('PRODID','-//EGroupware//NONSGML EGroupware Calendar '.$GLOBALS['egw_info']['apps']['calendar']['version'].'//'.
+				$vcal->setAttribute('PRODID', '-//EGroupware//NONSGML EGroupware Calendar '.$GLOBALS['egw_info']['apps']['calendar']['version'].'//'.
 					strtoupper($GLOBALS['egw_info']['user']['preferences']['common']['lang']));
 				self::add_vtimezone($vcal, $tzid);
 				$tzid = $vcal->exportvCalendar('utf-8');
@@ -417,7 +417,7 @@ 
 			case 'tzid':
 				break;
 			default:
-				$tzid = self::tz2id($tzid,$type == 'vcalendar' ? 'component' : $type);
+				$tzid = self::tz2id($tzid, $type == 'vcalendar' ? 'component' : $type);
 				break;
 		}
 		return $tzid;

calendar/inc/class.calendar_favorite_portlet.inc.php

Spacing +41 added lines, -42 removed lines

@@ -32,19 +32,19 @@ 
 		$reload = !is_null($need_reload);
 
 		// Let parent handle the basic stuff
-		parent::__construct($context,$need_reload);
+		parent::__construct($context, $need_reload);
 
-		if($this->favorite['state']['view'] == 'listview')
+		if ($this->favorite['state']['view'] == 'listview')
 		{
 			$this->context['template'] = 'calendar.list.rows';
 			$this->context['sel_options'] = array();
 			$this->nm_settings += array(
 				'csv_export'      => True,
-				'filter_no_lang'  => True,	// I  set no_lang for filter (=dont translate the options)
-				'no_filter2'      => True,	// I  disable the 2. filter (params are the same as for filter)
-				'no_cat'          => True,	// I  disable the cat-selectbox
+				'filter_no_lang'  => True, // I  set no_lang for filter (=dont translate the options)
+				'no_filter2'      => True, // I  disable the 2. filter (params are the same as for filter)
+				'no_cat'          => True, // I  disable the cat-selectbox
 				'filter'          => 'after',
-				'row_id'          => 'row_id',	// set in get rows "$event[id]:$event[recur_date]"
+				'row_id'          => 'row_id', // set in get rows "$event[id]:$event[recur_date]"
 				'row_modified'    => 'modified',
 				'get_rows'	=> 'calendar_favorite_portlet::get_rows',
 				// Use a different template so it can be accessed from client side
@@ -58,17 +58,17 @@ 
 		$need_reload = $reload && $need_reload;
 	}
 
-	public function exec($id = null, Etemplate &$etemplate = null)
+	public function exec($id = null, Etemplate&$etemplate = null)
 	{
 
 		// Always load app's javascript, so most actions have a chance of working
-		Framework::includeJS('.','app',$this->context['appname']);
+		Framework::includeJS('.', 'app', $this->context['appname']);
 
 		// Always load app's css
 		Framework::includeCSS('calendar', 'app-'.$GLOBALS['egw_info']['user']['preferences']['common']['theme']) ||
-			Framework::includeCSS('calendar','app');
+			Framework::includeCSS('calendar', 'app');
 
-		if($this->favorite['state']['view'] == 'listview' || is_array($this->favorite) && !$this->favorite['state']['view'])
+		if ($this->favorite['state']['view'] == 'listview' || is_array($this->favorite) && !$this->favorite['state']['view'])
 		{
 			$ui = new calendar_uilist();
 		}
@@ -77,16 +77,16 @@ 
 			$ui = new calendar_uiviews();
 			if ($this->favorite)
 			{
-				if($this->favorite['state']['start']) $ui->search_params['start'] = $this->favorite['state']['start'];
-				if($this->favorite['state']['cat_id']) $ui->search_params['cat_id'] = $this->favorite['state']['cat_id'];
+				if ($this->favorite['state']['start']) $ui->search_params['start'] = $this->favorite['state']['start'];
+				if ($this->favorite['state']['cat_id']) $ui->search_params['cat_id'] = $this->favorite['state']['cat_id'];
 				// Owner can be 0 for current user
-				if(array_key_exists('owner',$this->favorite['state'])) $ui->search_params['users'] = $this->favorite['state']['owner'];
-				if($ui->search_params['users'] && !is_array($ui->search_params['users']))
+				if (array_key_exists('owner', $this->favorite['state'])) $ui->search_params['users'] = $this->favorite['state']['owner'];
+				if ($ui->search_params['users'] && !is_array($ui->search_params['users']))
 				{
-					$ui->search_params['users'] = explode(',',$ui->search_params['users']);
+					$ui->search_params['users'] = explode(',', $ui->search_params['users']);
 				}
-				if($this->favorite['state']['filter']) $ui->search_params['filter'] = $this->favorite['state']['filter'];
-				if($this->favorite['state']['sortby']) $ui->search_params['sortby'] = $this->favorite['state']['sortby'];
+				if ($this->favorite['state']['filter']) $ui->search_params['filter'] = $this->favorite['state']['filter'];
+				if ($this->favorite['state']['sortby']) $ui->search_params['sortby'] = $this->favorite['state']['sortby'];
 				$ui->search_params['weekend'] = $this->favorite['state']['weekend'];
 			}
 			$etemplate->read('home.legacy');
@@ -96,7 +96,7 @@ 
 
 		$content = array('legacy' => '');
 
-		switch($this->favorite['state']['view'])
+		switch ($this->favorite['state']['view'])
 		{
 			case 'listview':
 				$this->context['sel_options']['filter'] = &$ui->date_filters;
@@ -110,9 +110,9 @@ 
 				$content = array();
 				$etemplate->read('calendar.planner');
 				$etemplate->set_dom_id($id);
-				$this->actions =& $etemplate->getElementAttribute('planner', 'actions');
+				$this->actions = & $etemplate->getElementAttribute('planner', 'actions');
 				// Don't notify the calendar app of date changes
-				$etemplate->setElementAttribute('planner','onchange',false);
+				$etemplate->setElementAttribute('planner', 'onchange', false);
 				$ui->planner_view = $this->favorite['state']['planner_view'];
 				$ui->planner(array(), $etemplate);
 				return;
@@ -120,20 +120,19 @@ 
 			case 'weekN':
 				$etemplate->read('calendar.view');
 				$etemplate->set_dom_id($id);
-				$this->actions =& $etemplate->getElementAttribute('view', 'actions');
+				$this->actions = & $etemplate->getElementAttribute('view', 'actions');
 
 				$ui->month($this->favorite['state']['view'] == 'month' ?
-					0 :
-					(int)$ui->cal_prefs['multiple_weeks'],
+					0 : (int)$ui->cal_prefs['multiple_weeks'],
 					$etemplate
 				);
 				return;
 			case 'week':
 				$etemplate->read('calendar.view');
 				$etemplate->set_dom_id($id);
-				$this->actions =& $etemplate->getElementAttribute('view', 'actions');
+				$this->actions = & $etemplate->getElementAttribute('view', 'actions');
 				// Don't notify the calendar app of date changes
-				$etemplate->setElementAttribute('view[0]','onchange',false);
+				$etemplate->setElementAttribute('view[0]', 'onchange', false);
 				$ui->week(array(), $etemplate);
 				return;
 			case 'day':
@@ -143,7 +142,7 @@ 
 				$days = $this->favorite['state']['days'] ? $this->favorite['state']['days'] : (
 					$this->favorite['state']['view'] == 'day' ? 1 : 4
 				);
-				$this->actions =& $etemplate->getElementAttribute('view', 'actions');
+				$this->actions = & $etemplate->getElementAttribute('view', 'actions');
 				$ui->week($days, $etemplate);
 				return;
 		}
@@ -156,7 +155,7 @@ 
 		$old_app = $GLOBALS['egw_info']['flags']['currentapp'];
 		$GLOBALS['egw_info']['flags']['currentapp'] = 'calendar';
 
-		$etemplate->exec(get_called_class() .'::process',$content);
+		$etemplate->exec(get_called_class().'::process', $content);
 		$GLOBALS['egw_info']['flags']['currentapp'] = $old_app;
 	}
 
@@ -199,27 +198,27 @@ 
 			else
 			{
 				$success = $failed = $action_msg = $msg = null;
-				if ($ui->action($values['nm']['action'],$values['nm']['selected'],$values['nm']['select_all'],
-						$success,$failed,$action_msg,'calendar_list',$msg, $values['nm']['checkboxes']['no_notifications']))
+				if ($ui->action($values['nm']['action'], $values['nm']['selected'], $values['nm']['select_all'],
+						$success, $failed, $action_msg, 'calendar_list', $msg, $values['nm']['checkboxes']['no_notifications']))
 				{
-					$msg .= lang('%1 event(s) %2',$success,$action_msg);
-					Api\Json\Response::get()->apply('egw.message',array($msg,'success'));
-					foreach($values['nm']['selected'] as &$id)
+					$msg .= lang('%1 event(s) %2', $success, $action_msg);
+					Api\Json\Response::get()->apply('egw.message', array($msg, 'success'));
+					foreach ($values['nm']['selected'] as &$id)
 					{
 						$id = 'calendar::'.$id;
 					}
 					// Directly request an update - this will get addressbook tab too
-					Api\Json\Response::get()->apply('egw.dataRefreshUIDs',array($values['nm']['selected']));
+					Api\Json\Response::get()->apply('egw.dataRefreshUIDs', array($values['nm']['selected']));
 				}
-				elseif(is_null($msg))
+				elseif (is_null($msg))
 				{
-					$msg .= lang('%1 entries %2, %3 failed because of insufficent rights !!!',$success,$action_msg,$failed);
-					Api\Json\Response::get()->apply('egw.message',array($msg,'error'));
+					$msg .= lang('%1 entries %2, %3 failed because of insufficent rights !!!', $success, $action_msg, $failed);
+					Api\Json\Response::get()->apply('egw.message', array($msg, 'error'));
 				}
-				elseif($msg)
+				elseif ($msg)
 				{
-					$msg .= "\n".lang('%1 entries %2, %3 failed.',$success,$action_msg,$failed);
-					Api\Json\Response::get()->apply('egw.message',array($msg,'error'));
+					$msg .= "\n".lang('%1 entries %2, %3 failed.', $success, $action_msg, $failed);
+					Api\Json\Response::get()->apply('egw.message', array($msg, 'error'));
 				}
 				unset($values['nm']['action']);
 				unset($values['nm']['select_all']);
@@ -235,9 +234,9 @@ 
 	public function get_properties()
 	{
 		$properties = parent::get_properties();
-		foreach($properties as &$property)
+		foreach ($properties as &$property)
 		{
-			if($property['name'] == 'favorite')
+			if ($property['name'] == 'favorite')
 			{
 				unset($property['select_options']['blank']);
 				break;
@@ -247,7 +246,7 @@ 
 	}
 
 	public function get_actions() {
-		if($this->favorite['state']['view'] == 'listview' || !$this->actions)
+		if ($this->favorite['state']['view'] == 'listview' || !$this->actions)
 		{
 			return array();
 		}

calendar/inc/class.calendar_tracking.inc.php

Spacing +14 added lines, -14 removed lines

@@ -114,18 +114,18 @@ 
 	 */
 	public function __construct()
 	{
-		parent::__construct('calendar');	// adds custom fields
+		parent::__construct('calendar'); // adds custom fields
 	}
 
 	/**
 	 * Tracks the changes in one entry $data, by comparing it with the last version in $old
 	 * Overrides parent to reformat participants into a format parent can handle
 	 */
-	public function track(array $data,array $old=null,$user=null,$deleted=null,array $changed_fields=null)
+	public function track(array $data, array $old = null, $user = null, $deleted = null, array $changed_fields = null)
 	{
 		// Don't try to track dates on recurring events.
 		// It won't change for the base event, and any change to the time creates an exception
-		if($data['recur_type'])
+		if ($data['recur_type'])
 		{
 			unset($data['start']); unset($data['end']);
 			unset($old['start']); unset($old['end']);
@@ -136,7 +136,7 @@ 
 		* If this is one of a recurring event, append the recur_date to the participant field so we can
 		* filter by it later.
 		*/
-		if(is_array($data['participants']))
+		if (is_array($data['participants']))
 		{
 			$participants = $data['participants'];
 			$data['participants'] = array();
@@ -144,25 +144,25 @@ 
 		}
 		// if clients eg. CalDAV do NOT set participants, they are left untouched
 		// therefore we should not track them, as all updates then show up as all participants removed
-		elseif(!isset($data['participants']))
+		elseif (!isset($data['participants']))
 		{
 			unset($old['participants']);
 		}
-		if(is_array($old['participants']))
+		if (is_array($old['participants']))
 		{
 			$participants = $old['participants'];
 			$old['participants'] = array();
 			$old = array_merge($old, $this->alter_participants($participants));
 		}
-		parent::track($data,$old,$user,$deleted, $changed_fields);
+		parent::track($data, $old, $user, $deleted, $changed_fields);
 	}
 
 	/**
 	 * Overrides parent because calendar_boupdates handles the notifications
 	 */
-	public function do_notifications($data,$old,$deleted=null)
+	public function do_notifications($data, $old, $deleted = null)
 	{
-		unset($data, $old, $deleted);	// unused, but required by function signature
+		unset($data, $old, $deleted); // unused, but required by function signature
 		return true;
 	}
 
@@ -177,21 +177,21 @@ 
 	 * @param array $old = null
 	 * @return array of keys with different values in $data and $old
 	 */
-	public function changed_fields(array $data,array $old=null)
+	public function changed_fields(array $data, array $old = null)
 	{
-		if(is_array($data['participants']))
+		if (is_array($data['participants']))
 		{
 			$participants = $data['participants'];
 			$data['participants'] = array();
 			$data = array_merge($data, $this->alter_participants($participants));
 		}
-		if(is_array($old['participants']))
+		if (is_array($old['participants']))
 		{
 			$participants = $old['participants'];
 			$old['participants'] = array();
 			$old = array_merge($old, $this->alter_participants($participants));
 		}
-		return parent::changed_fields($data,$old);
+		return parent::changed_fields($data, $old);
 	}
 
 	/**
@@ -202,7 +202,7 @@ 
 	protected function alter_participants($participants)
 	{
 		$data = array();
-		foreach($participants as $uid => $status)
+		foreach ($participants as $uid => $status)
 		{
 			$quantity = $role = $user_type = $user_id = null;
 			calendar_so::split_status($status, $quantity, $role);

api/asyncservices.php

Spacing +10 added lines, -11 removed lines

@@ -15,7 +15,7 @@ 
 use EGroupware\Api\Asyncservice;
 
 if (!isset($_REQUEST['domain'])) $_REQUEST['domain'] = isset($_SERVER['argv'][1]) ? $_SERVER['argv'][1] : 'default';
-$path_to_egroupware = realpath(__DIR__.'/..');	//  need to be adapted if this script is moved somewhere else
+$path_to_egroupware = realpath(__DIR__.'/..'); //  need to be adapted if this script is moved somewhere else
 
 // remove the comment from one of the following lines to enable loging
 // define('ASYNC_LOG','C:\\async.log');		// Windows
@@ -25,8 +25,8 @@ 
 if (defined('ASYNC_LOG'))
 {
 	$msg = date('Y/m/d H:i:s ').$_REQUEST['domain'].": asyncservice started\n";
-	$f = fopen(ASYNC_LOG,'a+');
-	fwrite($f,$msg);
+	$f = fopen(ASYNC_LOG, 'a+');
+	fwrite($f, $msg);
 	fclose($f);
 }
 $GLOBALS['egw_info']['flags'] = array(
@@ -43,8 +43,8 @@ 
 	}
 	if (defined('ASYNC_LOG'))
 	{
-		$f = fopen(ASYNC_LOG,'a+');
-		fwrite($f,$msg);
+		$f = fopen(ASYNC_LOG, 'a+');
+		fwrite($f, $msg);
 		fclose($f);
 	}
 	die($msg);
@@ -63,8 +63,8 @@ 
 	}
 	if (defined('ASYNC_LOG'))
 	{
-		$f = fopen(ASYNC_LOG,'a+');
-		fwrite($f,$msg);
+		$f = fopen(ASYNC_LOG, 'a+');
+		fwrite($f, $msg);
 		fclose($f);
 	}
 	die($msg);
@@ -75,8 +75,7 @@ 
 $async = new Asyncservice();
 $num = $async->check_run(isset($_REQUEST['run_by']) ? $_REQUEST['run_by'] : 'crontab');
 
-$msg = date('Y/m/d H:i:s ').$_REQUEST['domain'].': '.($num === false ? 'An error occured: can not obtain semaphore!' :
-	($num ? "$num job(s) executed" : 'Nothing to execute'))."\n\n";
+$msg = date('Y/m/d H:i:s ').$_REQUEST['domain'].': '.($num === false ? 'An error occured: can not obtain semaphore!' : ($num ? "$num job(s) executed" : 'Nothing to execute'))."\n\n";
 
 if (isset($_SERVER['HTTP_HOST']))
 {
@@ -87,7 +86,7 @@ 
 
 if (defined('ASYNC_LOG'))
 {
-	$f = fopen(ASYNC_LOG,'a+');
-	fwrite($f,$msg);
+	$f = fopen(ASYNC_LOG, 'a+');
+	fwrite($f, $msg);
 	fclose($f);
 }

api/images.php

Spacing +6 added lines, -6 removed lines

@@ -28,17 +28,17 @@ 
 
 include '../header.inc.php';
 
-$content = json_encode(Api\Image::map(preg_match('/^[a-z0-9_-]+$/i',$_GET['template']) ? $_GET['template'] : null, $_GET['svg']),
-	JSON_FORCE_OBJECT |	// export empty php-arrays as empty objects, not empty arrays
-	JSON_UNESCAPED_SLASHES | // do not escape slashes, smaller and better readable
+$content = json_encode(Api\Image::map(preg_match('/^[a-z0-9_-]+$/i', $_GET['template']) ? $_GET['template'] : null, $_GET['svg']),
+	JSON_FORCE_OBJECT|// export empty php-arrays as empty objects, not empty arrays
+	JSON_UNESCAPED_SLASHES|// do not escape slashes, smaller and better readable
 	(!empty($_GET['debug']) ? JSON_PRETTY_PRINT : 0));
 
 // use an etag over the image mapp
 $etag = '"'.md5($content).'"';
 
 // headers to allow caching, egw_framework specifies etag on url to force reload, even with Expires header
-Api\Session::cache_control(86400);	// cache for one day
-Header('Content-Type: '.(empty($_GET['debug'])?'text/javascript':'application/json').'; charset=utf-8');
+Api\Session::cache_control(86400); // cache for one day
+Header('Content-Type: '.(empty($_GET['debug']) ? 'text/javascript' : 'application/json').'; charset=utf-8');
 Header('ETag: '.$etag);
 
 // if servers send a If-None-Match header, response with 304 Not Modified, if etag matches
@@ -51,7 +51,7 @@ 
 if (empty($_GET['debug'])) $content = 'egw.set_images('.$content.", egw && egw.window !== window);\n";
 
 // we run our own gzip compression, to set a correct Content-Length of the encoded content
-if (in_array('gzip', explode(',',$_SERVER['HTTP_ACCEPT_ENCODING'])) && function_exists('gzencode'))
+if (in_array('gzip', explode(',', $_SERVER['HTTP_ACCEPT_ENCODING'])) && function_exists('gzencode'))
 {
 	$content = gzencode($content);
 	header('Content-Encoding: gzip');

api/src/Framework/IncludeMgr.php

Spacing +10 added lines, -10 removed lines

@@ -80,7 +80,7 @@ 
 	private function parse_file($file)
 	{
 		// file is from url and can contain query-params, eg. /phpgwapi/inc/jscalendar-setup.php?dateformat=d.m.Y&lang=de
-		if (strpos($file,'?') !== false) list($file) = explode('?',$file);
+		if (strpos($file, '?') !== false) list($file) = explode('?', $file);
 
 		// Mark the file as parsed
 		$this->parsed_files[$file] = true;
@@ -284,17 +284,17 @@ 
 	 * @returns the correct path on the server if the file is found or false, if the
 	 *  file is not found or no further processing is needed.
 	 */
-	private function translate_params($package, $file=null, $app='api')
+	private function translate_params($package, $file = null, $app = 'api')
 	{
 		if ($package[0] == '/' && is_readable(EGW_SERVER_ROOT.parse_url($path = $package, PHP_URL_PATH)) ||
 			// fix old /phpgwapi/js/ path by replacing it with /api/js/
 			substr($package, 0, 13) == '/phpgwapi/js/' && is_readable(EGW_SERVER_ROOT.parse_url($path = str_replace('/phpgwapi/js/', '/api/js/', $package), PHP_URL_PATH)) ||
 			$package[0] == '/' && is_readable(EGW_SERVER_ROOT.($path = $package)) ||
-			$package == '.' && is_readable(EGW_SERVER_ROOT.($path="/$app/js/$file.js")) ||
-			is_readable(EGW_SERVER_ROOT.($path="/$app/js/$package/$file.js")) ||
+			$package == '.' && is_readable(EGW_SERVER_ROOT.($path = "/$app/js/$file.js")) ||
+			is_readable(EGW_SERVER_ROOT.($path = "/$app/js/$package/$file.js")) ||
 			// fix not found by using app='api'
-			$app != 'api' && is_readable(EGW_SERVER_ROOT.($path="/api/js/$package/$file.js")) ||
-			$app != 'phpgwapi' && is_readable(EGW_SERVER_ROOT.($path="/phpgwapi/js/$package/$file.js")))
+			$app != 'api' && is_readable(EGW_SERVER_ROOT.($path = "/api/js/$package/$file.js")) ||
+			$app != 'phpgwapi' && is_readable(EGW_SERVER_ROOT.($path = "/phpgwapi/js/$package/$file.js")))
 		{
 			// normalise /./ to /
 			$path = str_replace('/./', '/', $path);
@@ -305,7 +305,7 @@ 
 			// 	adding the ctime to all js files...
 			if (is_array($file))
 			{
-				foreach($file as $name => $val)
+				foreach ($file as $name => $val)
 				{
 					$args .= (empty($args) ? '?' : '&').$name.'='.urlencode($val);
 				}
@@ -375,7 +375,7 @@ 
 	 * @param array $files
 	 * @param boolean $clear_files =false if true clear list of files, before including given ones
 	 */
-	public function include_files(array $files, $clear_files=false)
+	public function include_files(array $files, $clear_files = false)
 	{
 		if ($clear_files) $this->included_files = array();
 
@@ -391,7 +391,7 @@ 
 	 * @param boolean $clear_files =false if true clear list of files after returning them
 	 * @return array
 	 */
-	public function get_included_files($clear_files=false)
+	public function get_included_files($clear_files = false)
 	{
 		$ret = array_keys($this->included_files);
 		if ($clear_files) $this->included_files = array();
@@ -424,7 +424,7 @@ 
 
 	$paths = !empty($_GET['path']) ? (array)$_GET['path'] : (array)'/stylite/js/filemanager/filemanager.js';
 
-	foreach($paths as $path)
+	foreach ($paths as $path)
 	{
 		echo "\t<h1>".htmlspecialchars($path)."</h1>\n";
 		$mgr->include_js_file($path);