EGroupware / egroupware

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_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);

api/src/Framework/Template.php

Spacing +34 added lines, -34 removed lines

@@ -32,7 +32,7 @@ 
 	 *	- function-name like set_var to get eg. all assignments or
 	 *	- handle- / variable-names - if you are only interested in some variables ;-)
 	 */
-	var $debug = False;	// array('cat_list','cat_list_t');
+	var $debug = False; // array('cat_list','cat_list_t');
 
 	/**
 	 * $file[handle] = 'filename';
@@ -128,9 +128,9 @@ 
 	 */
 	function set_file($handle, $filename = '')
 	{
-		if ($this->debug && $this->check_debug('set_file',$handle,$filename))
+		if ($this->debug && $this->check_debug('set_file', $handle, $filename))
 		{
-			echo "<p>Template::set_file('".print_r($handle,true)."','$filename')</p>\n";
+			echo "<p>Template::set_file('".print_r($handle, true)."','$filename')</p>\n";
 		}
 		if (!is_array($handle))
 		{
@@ -143,7 +143,7 @@ 
 		}
 		else
 		{
-			foreach($handle as $h => $f)
+			foreach ($handle as $h => $f)
 			{
 				$this->file[$h] = $this->filename($f);
 			}
@@ -160,7 +160,7 @@ 
 	 */
 	function set_block($parent, $handle, $name = '')
 	{
-		if ($this->debug && $this->check_debug('set_block',$parent,$handle,$name))
+		if ($this->debug && $this->check_debug('set_block', $parent, $handle, $name))
 		{
 			echo "<p>Template::set_block('$parent','$handle','$name')</p>\n";
 		}
@@ -177,14 +177,14 @@ 
 		$qhandle = preg_quote($handle);
 		$reg = "/<!--\\s+BEGIN $qhandle\\s+-->(.*)\n\\s*<!--\\s+END $qhandle\\s+-->/s";
 		$match = null;
-		if (!preg_match($reg,$str,$match))
+		if (!preg_match($reg, $str, $match))
 		{
 			// unfortunaly some apps set non-existing blocks, therefor I have to disable this diagnostics again for now
 			$this->halt("set_block: unable to find block '$handle' in '$parent'=<pre>".htmlspecialchars($str)."</pre> this->root=$this->root");
 			// return False;
 		}
-		$this->set_var($handle,$match[1]);
-		$this->set_var($parent,preg_replace($reg, '{' . "$name}",$str));
+		$this->set_var($handle, $match[1]);
+		$this->set_var($parent, preg_replace($reg, '{'."$name}", $str));
 	}
 
 	/* public: set_var(array $values)
@@ -205,18 +205,18 @@ 
 				$varname => $value
 			);
 		}
-		foreach($varname as $k => $v)
+		foreach ($varname as $k => $v)
 		{
 			if (!empty($k))
 			{
-				if ($this->debug && $this->check_debug('set_var',$k))
+				if ($this->debug && $this->check_debug('set_var', $k))
 				{
 					echo "<p>Template::set_var('$k','$v')</p>\n";
 				}
 				$this->varkeys[$k] = $this->varname($k);
 				$this->varvals[$k] = $this->egroupware_hack ? str_replace(
-					array('phpGroupWare','www.phpgroupware.org'),
-					array('eGroupWare','www.eGroupWare.org'),$v
+					array('phpGroupWare', 'www.phpgroupware.org'),
+					array('eGroupWare', 'www.eGroupWare.org'), $v
 				) : $v;
 			}
 		}
@@ -230,7 +230,7 @@ 
 	 */
 	function subst($handle)
 	{
-		if ($this->debug && $this->check_debug('subst',$handle))
+		if ($this->debug && $this->check_debug('subst', $handle))
 		{
 			echo "<p>Template::subst('$handle')</p>\n";
 		}
@@ -241,7 +241,7 @@ 
 		}
 
 		$str = $this->get_var($handle);
-		foreach($this->varkeys as $k => $v)
+		foreach ($this->varkeys as $k => $v)
 		{
 			$str = str_replace($v, $this->varvals[$k], $str);
 		}
@@ -276,7 +276,7 @@ 
 			$str = $this->subst($handle);
 			if ($append)
 			{
-				$this->set_var($target, $this->get_var($target) . $str);
+				$this->set_var($target, $this->get_var($target).$str);
 			}
 			else
 			{
@@ -285,7 +285,7 @@ 
 		}
 		else
 		{
-			foreach($handle as $h)
+			foreach ($handle as $h)
 			{
 				$str = $this->subst($h);
 				$this->set_var($target, $str);
@@ -332,7 +332,7 @@ 
 	 */
 	function get_vars()
 	{
-		foreach(array_keys($this->varkeys) as $k)
+		foreach (array_keys($this->varkeys) as $k)
 		{
 			$result[$k] = $this->varvals[$k];
 		}
@@ -353,7 +353,7 @@ 
 		}
 		else
 		{
-			foreach(array_keys($varname) as $k)
+			foreach (array_keys($varname) as $k)
 			{
 				$result[$k] = $this->varvals[$k];
 			}
@@ -382,7 +382,7 @@ 
 		{
 			return false;
 		}
-		foreach($m as $v)
+		foreach ($m as $v)
 		{
 			if (!isset($this->varkeys[$v]))
 			{
@@ -448,15 +448,15 @@ 
 	 * @param string $root ='' default $this->root
 	 * @param int $time =1
 	 */
-	protected function filename($filename,$root='',$time=1)
+	protected function filename($filename, $root = '', $time = 1)
 	{
-		if($root == '')
+		if ($root == '')
 		{
 			$root = $this->root;
 		}
-		if(substr($filename, 0, 1) != '/')
+		if (substr($filename, 0, 1) != '/')
 		{
-			$new_filename = $root . '/' . $filename;
+			$new_filename = $root.'/'.$filename;
 		}
 		else
 		{
@@ -465,14 +465,14 @@ 
 
 		if (!file_exists($new_filename))
 		{
-			if($time==2)
+			if ($time == 2)
 			{
 				$this->halt("filename: file $new_filename does not exist.");
 			}
 			else
 			{
-				$new_root = dirname($root) . DIRECTORY_SEPARATOR . 'default';
-				$new_filename = $this->filename(str_replace($root.'/','',$new_filename),$new_root,2);
+				$new_root = dirname($root).DIRECTORY_SEPARATOR.'default';
+				$new_filename = $this->filename(str_replace($root.'/', '', $new_filename), $new_root, 2);
 			}
 		}
 		return $new_filename;
@@ -491,9 +491,9 @@ 
 	 */
 	function loadfile($handle)
 	{
-		if ($this->debug && $this->check_debug('loadfile',$handle))
+		if ($this->debug && $this->check_debug('loadfile', $handle))
 		{
-			echo "<p>Template::loadfile('$handle') file=<pre>\n".print_r($this->file,True)."</pre>\n";
+			echo "<p>Template::loadfile('$handle') file=<pre>\n".print_r($this->file, True)."</pre>\n";
 			echo "<p>backtrace: ".function_backtrace()."</p>\n";
 		}
 		if (isset($this->varkeys[$handle]) && !empty($this->varvals[$handle]))
@@ -502,9 +502,9 @@ 
 		}
 		if (!isset($this->file[$handle]))
 		{
-			if ($this->debug && $this->check_debug('loadfile',$handle))
+			if ($this->debug && $this->check_debug('loadfile', $handle))
 			{
-				echo "varkeys =<pre>".print_r($this->varkeys,True)."</pre>varvals =<pre>".print_r($this->varvals,True)."</pre>\n";
+				echo "varkeys =<pre>".print_r($this->varkeys, True)."</pre>varvals =<pre>".print_r($this->varvals, True)."</pre>\n";
 			}
 			$this->halt("loadfile: $handle is not a valid handle.");
 			return false;
@@ -549,10 +549,10 @@ 
 	{
 		if (!$this->debug) return False;
 
-		foreach(func_get_args() as $arg)
+		foreach (func_get_args() as $arg)
 		{
 			if (!is_array($this->debug) && $this->debug === $arg ||
-				(is_array($this->debug) && (@$this->debug[$arg] || in_array($arg,$this->debug,True))))
+				(is_array($this->debug) && (@$this->debug[$arg] || in_array($arg, $this->debug, True))))
 			{
 				return True;
 			}
@@ -601,8 +601,8 @@ 
 		{
 			$GLOBALS['egw_info']['server']['template_set'] = 'idots';
 		}
-		$tpldir         = EGW_SERVER_ROOT . '/' . $appname . '/templates/' . $GLOBALS['egw_info']['server']['template_set'];
-		$tpldir_default = EGW_SERVER_ROOT . '/' . $appname . '/templates/default';
+		$tpldir         = EGW_SERVER_ROOT.'/'.$appname.'/templates/'.$GLOBALS['egw_info']['server']['template_set'];
+		$tpldir_default = EGW_SERVER_ROOT.'/'.$appname.'/templates/default';
 
 		if (@is_dir($tpldir))
 		{

api/src/Framework/Tutorial.php

Spacing +7 added lines, -8 removed lines

@@ -32,7 +32,7 @@ 
 	 *
 	 * @param array $content
 	 */
-	function popup ($content=null)
+	function popup($content = null)
 	{
 		// check and if not done register tutorial_menu hook
 		if (!Api\Hooks::exists('sidebox_all', 'api') ||
@@ -48,7 +48,7 @@ 
 		if (!is_array($content))
 		{
 			// Get tutorial object id
-			$tuid_indx = explode('-',$_GET['tuid']);
+			$tuid_indx = explode('-', $_GET['tuid']);
 			$appName = $tuid_indx[0];
 			$lang = $tuid_indx[1];
 			$id = $tuid_indx[2];
@@ -57,7 +57,7 @@ 
 		{
 			$appName = $content['list']['apps'];
 			$lang = $GLOBALS['egw_info']['user']['preferences']['common']['lang'];
-			$id ="0";
+			$id = "0";
 		}
 		// read tutorials json file to fetch data
 		$tutorials = json_decode(self::getJsonData(), true);
@@ -74,8 +74,7 @@ 
 		if (in_array($appName, array_keys($GLOBALS['egw_info']['user']['apps'])) || $appName === "introduction")
 		{
 			// fallback to english video
-			$tutorial = $tutorials[$appName][$lang][$id]? $tutorials[$appName][$lang][$id]:
-				$tutorials[$appName]['en'][$id];
+			$tutorial = $tutorials[$appName][$lang][$id] ? $tutorials[$appName][$lang][$id] : $tutorials[$appName]['en'][$id];
 
 			$list = array(
 				'apps' => $appName,
@@ -86,7 +85,7 @@ 
 				$v ['onclick'] = 'app[egw.app_name()].tutorial_videoOnClick("'.$v['src'].'")';
 				array_push($list, $v);
 			}
-			$content = array (
+			$content = array(
 				'src' => $tutorial['src'],
 				'title' => $tutorial['title'],
 				'desc' => $tutorial['desc'],
@@ -137,13 +136,13 @@ 
 	public static function tutorial_menu()
 	{
 		if (Api\Header\UserAgent::mobile()) return;
-		$tutorials = json_decode(self::getJsonData(),true);
+		$tutorials = json_decode(self::getJsonData(), true);
 		$appname = $GLOBALS['egw_info']['flags']['currentapp'];
 		if (!is_array($tutorials[$appname])) return false;
 		if (!$GLOBALS['egw_info']['server']['egw_tutorial_disable']
 			|| $GLOBALS['egw_info']['server']['egw_tutorial_disable'] == 'intro')
 		{
-			$file = Array (
+			$file = Array(
 				array(
 					'text'    => '<div id="egw_tutorial_'.$appname.'_sidebox" class="egwTutorial"/>',
 					'no_lang' => true,