EGroupware / egroupware

phpgwapi/inc/class.categories.inc.php

Doc Comments +20 added lines, -28 removed lines

@@ -178,18 +178,16 @@ 
 	/**
 	 * return an array populated with categories
 	 *
-	 * @param string $type='all' 'subs','mains','appandmains','appandsubs','noglobal','noglobalapp', defaults to 'all'
-	 * @param int $start=0 see $limit
+	 * @param int $start see $limit
 	 * @param boolean|int $limit if true limited query to maxmatches rows (starting with $start)
 	 * @param string $query='' query-pattern
-	 * @param string $sort='ASC' sort order, defaults to 'ASC'
 	 * @param string $order='' order by, default cat_main, cat_level, cat_name ASC
 	 * @param boolean|string $globals includes the global egroupware categories or not,
 	 * 	'all_no_acl' to return global and all non-private user categories independent of ACL
-	 * @param array|int $parent_id=null return only subcats of $parent_id(s)
+	 * @param array|int $parent_id return only subcats of $parent_id(s)
 	 * @param int $lastmod = -1 if > 0 return only cats modified since then
 	 * @param string $column='' if column-name given only that column is returned, not the full array with all cat-data
-	 * @param array $filter=null array with column-name (without cat_-prefix) => value pairs (! negates the value)
+	 * @param array $filter array with column-name (without cat_-prefix) => value pairs (! negates the value)
 	 * @return array of cat-arrays or $column values
 	 */
 	function return_array($type='all', $start=0, $limit=true, $query='', $sort='ASC',$order='',$globals=false, $parent_id=null, $lastmod=-1, $column='', $filter=null)
@@ -324,15 +322,13 @@ 
 	/**
 	 * return a sorted array populated with categories (main sorting criteria is hierachy!)
 	 *
-	 * @param int $start=0 see $limit
-	 * @param boolean|int $limit if true limited query to maxmatches rows (starting with $start)
+	 * @param int $start see $limit
+	 * @param boolean $limit if true limited query to maxmatches rows (starting with $start)
 	 * @param string $query='' query-pattern
-	 * @param string $sort='ASC' sort order, either defaults to 'ASC'
-	 * @param string $order='cat_name' order by
 	 * @param boolean|string $globals includes the global egroupware categories or not,
 	 * 	'all_no_acl' to return global and all non-private user categories independent of ACL
-	 * @param array|int $parent_id=0 return only subcats of $parent_id(s)
-	 * @param boolean $unserialize_data=true return $cat['data'] as array (not serialized array)
+	 * @param array|int $parent_id return only subcats of $parent_id(s)
+	 * @param boolean $unserialize_data return $cat['data'] as array (not serialized array)
 	 * @return array with cats
 	 */
 	function return_sorted_array($start=0,$limit=True,$query='',$sort='ASC',$order='cat_name',$globals=False, $parent_id=0,$unserialize_data=true,$filter=null)
@@ -429,7 +425,7 @@ 
 	 *
 	 * Owner and appname are set from the values used to instanciate the class!
 	 *
-	 * @param array $value cat-data
+	 * @param array $values cat-data
 	 * @return int new cat-id
 	 */
 	function add($values)
@@ -508,9 +504,9 @@ 
 	 *
 	 * @param int $needed necessary ACL right: EGW_ACL_{READ|EDIT|DELETE}
 	 * @param mixed $category category as array or the category_id
-	 * @param boolean $no_acl_check=false if true, grants are NOT checked, gives access to all non-private categories of all users
+	 * @param boolean $no_acl_check if true, grants are NOT checked, gives access to all non-private categories of all users
 	 * @param boolean $allow_global_read if true, global cats are allowed (independent of app) for reading
-	 * @return boolean true permission granted, false for permission denied, null for category does not exist
+	 * @return null|boolean true permission granted, false for permission denied, null for category does not exist
 	 */
 	public function check_perms($needed, $category, $no_acl_check=false, $allow_global_read=false)
 	{
@@ -570,8 +566,8 @@ 
 	 * delete a category
 	 *
 	 * @param int $cat_id category id
-	 * @param boolean $drop_subs=false if true delete sub-cats too
-	 * @param boolean $modify_subs=false if true make the subs owned by the parent of $cat_id
+	 * @param boolean $drop_subs if true delete sub-cats too
+	 * @param boolean $modify_subs if true make the subs owned by the parent of $cat_id
 	 */
 	function delete($cat_id, $drop_subs = False, $modify_subs = False)
 	{
@@ -769,7 +765,6 @@ 
 	 * - cat's of other user
 	 *
 	 * @param string $cat_name cat-name
-	 * @param boolean|string $strip=false if true, strips 'X-'  ($strip) from category names added by some SyncML clients
 	 * @return int cat-id or 0 if not found
 	 */
 	function name2id($cat_name, $strip=false)
@@ -821,8 +816,8 @@ 
 	 * Check if category is global (owner <= 0 || appname == 'phpgw')
 	 *
 	 * @param int|array $cat
-	 * @param boolean $application_global=false true check for application global categories only (appname == 'phpgw')
-	 * @return boolean
+	 * @param boolean $application_global true check for application global categories only (appname == 'phpgw')
+	 * @return null|boolean
 	 */
 	static function is_global($cat,$application_global=false)
 	{
@@ -842,8 +837,7 @@ 
 	 * We use a shared cache together with read
 	 * $item == 'data' is returned as array (not serialized array)!
 	 *
-	 * @param int $cat_id=0 cat-id
-	 * @param string $item='name' requested information, 'path' = / delimited path of category names (incl. parents)
+	 * @param int $cat_id cat-id
 	 * @return string information or '--' if not found or !$cat_id
 	 */
 	static function id2name($cat_id=0, $item='name')
@@ -883,8 +877,8 @@ 
 	 *
 	 * @param string $type subs or mains
 	 * @param string $cat_name='' category name
-	 * @param int $cat_id=0 category id
-	 * @param int $parent=0 category id of parent
+	 * @param int $cat_id category id
+	 * @param int $parent category id of parent
 	 * @return int/boolean cat_id or false if cat not exists
 	 */
 	function exists($type,$cat_name = '',$cat_id = 0,$parent = 0)
@@ -914,8 +908,6 @@ 
 	 * Change the owner of all cats owned by $owner to $to OR deletes them if !$to
 	 *
 	 * @param int $owner owner or the cats to delete or change
-	 * @param int $to=0 new owner or 0 to delete the cats
-	 * @param string $app='' if given only cats matching $app are modifed/deleted
 	 */
 	function change_owner($owner,$to=0,$app='')
 	{
@@ -1030,7 +1022,7 @@ 
 	 *
 	 * @param string appname
 	 *
-	 * @return Returns the URL, but you do not need to do anything with it.
+	 * @return string the URL, but you do not need to do anything with it.
 	 */
 	public static function css($appname)
 	{
@@ -1047,7 +1039,7 @@ 
 	 * Delete categories belonging to a given account, when account got deleted
 	 *
 	 * @param int $account_id
-	 * @param int $new_owner=null for users data can be transfered to new owner
+	 * @param int $new_owner for users data can be transfered to new owner
 	 * @return int number of deleted or modified categories
 	 */
 	public static function delete_account($account_id, $new_owner=null)
@@ -1138,7 +1130,7 @@ 
 	/**
 	 * return into a select box, list or other formats
 	 *
-	 * @param string/array $format string 'select' or 'list', or array with all params
+	 * @param string $format string 'select' or 'list', or array with all params
 	 * @param string $type='' subs or mains
 	 * @param int/array $selected - cat_id or array with cat_id values
 	 * @param boolean $globals True or False, includes the global egroupware categories or not

phpgwapi/inc/class.common.inc.php

Doc Comments +14 added lines, -20 removed lines

@@ -31,6 +31,7 @@ 
 	 *
 	 * @param $category=LC_ALL category to set, see setlocal function
 	 * @param $charset=null default system charset
+	 * @param integer $category
 	 * @return string the local (or best estimate) set
 	 */
 	static function setlocale($category=LC_ALL,$charset=null)
@@ -158,7 +159,7 @@ 
 	* get the local content id from a global UID
 	*
 	* @param sting $_globalUid the global UID
-	* @return int local egw content id
+	* @return false|string local egw content id
 	*/
 	static function get_egwId($_globalUid)
 	{
@@ -278,7 +279,7 @@ 
 	/**
 	 * return a random string of size $size
 	 *
-	 * @param $size int-size of random string to return
+	 * @param integer $size int-size of random string to return
 	 */
 	static function randomstring($size)
 	{
@@ -307,6 +308,7 @@ 
 	 * This is used for reporting errors in a nice format.
 	 *
 	 * @param $error - array of errors
+	 * @param string $errors
 	 */
 	static function error_list($errors,$text='Error')
 	{
@@ -573,7 +575,7 @@ 
 	 * get template dir of an application
 	 *
 	 * @param $appname appication name optional can be derived from $GLOBALS['egw_info']['flags']['currentapp'];
-	 * @return string|boolean dir or false if no dir is found
+	 * @return string|false dir or false if no dir is found
 	 */
 	static function get_tpl_dir($appname = '')
 	{
@@ -631,6 +633,7 @@ 
 	 * 	on a per image basis to the default dir
 	 *
 	 * @deprecated
+	 * @param string $dir
 	 */
 	static function is_image_dir($dir)
 	{
@@ -742,11 +745,8 @@ 
 	/**
 	 * Searches an image of a given type, if not found also without extension
 	 *
-	 * @param string $appname
 	 * @param string|array $image one or more image-name in order of precedence
 	 * @param string $extension='' extension to $image, makes sense only with an array
-	 * @param boolean $svg=false should svg images be returned or not:
-	 *	true: always return svg, false: never return svg (current default), null: browser dependent, see svg_usable()
 	 * @return string url of image or null if not found
 	 */
 	static function image_on($app,$image,$extension='_on',$svg=false)
@@ -757,11 +757,8 @@ 
 	/**
 	 * Searches a appname, template and maybe language and type-specific image
 	 *
-	 * @param string $appname
 	 * @param string|array $image one or more image-name in order of precedence
 	 * @param string $extension='' extension to $image, makes sense only with an array
-	 * @param boolean $svg=false should svg images be returned or not:
-	 *	true: always return svg, false: never return svg (current default), null: browser dependent, see svg_usable()
 	 * @return string url of image or null if not found
 	 */
 	static function image($app,$image,$extension='',$svg=false)
@@ -841,8 +838,8 @@ 
 	 *
 	 * VFS image directory is treated like an application named 'vfs'.
 	 *
-	 * @param string $template_set=null 'default', 'idots', 'jerryr', default is template-set from user prefs
-	 * @param boolean $svg=null prefer svg images, default for all browsers but IE<9
+	 * @param string $template_set 'default', 'idots', 'jerryr', default is template-set from user prefs
+	 * @param boolean $svg
 	 * @return array of application => image-name => full path
 	 */
 	static function image_map($template_set=null, $svg=null)
@@ -1119,9 +1116,8 @@ 
 	/**
 	 * return a formatted timestamp or current time
 	 *
-	 * @param int $t=0 timestamp, default current time
 	 * @param string $format='' timeformat, default '' = read from the user prefernces
-	 * @param boolean $adjust_to_usertime=true should datetime::tz_offset be added to $t or not, default true
+	 * @param boolean $adjust_to_usertime should datetime::tz_offset be added to $t or not, default true
 	 * @deprecated use egw_time::to($time, $format) egw_time::server2user($time, $format)
 	 * @return string the formated date/time
 	 */
@@ -1139,8 +1135,7 @@ 
 	 *
 	 * @param string $yearstr year
 	 * @param string $monthstr month
-	 * @param string $day day
-	 * @param boolean $add_seperator=false add the separator specifed in the prefs or not, default no
+	 * @param boolean $add_seperator add the separator specifed in the prefs or not, default no
 	 * @return string
 	 */
 	static function dateformatorder($yearstr,$monthstr,$daystr,$add_seperator = False)
@@ -1165,7 +1160,6 @@ 
 	 *
 	 * @param int $hour hour
 	 * @param int $min minutes
-	 * @param int/string $sec='' defaults to ''
 	 * @return string formated time
 	 */
 	static function formattime($hour,$min,$sec='')
@@ -1494,8 +1488,8 @@ 
 	 * Return a value for the next id an app/class may need to insert values into LDAP
 	 *
 	 * @param string $appname app-name
-	 * @param int $min=0 if != 0 minimum id
-	 * @param int $max=0 if != 0 maximum id allowed, if it would be exceeded we return false
+	 * @param int $min if != 0 minimum id
+	 * @param int $max if != 0 maximum id allowed, if it would be exceeded we return false
 	 * @return int/boolean the next id or false if $max given and exceeded
 	 */
 	static function next_id($appname,$min=0,$max=0)
@@ -1524,8 +1518,8 @@ 
 	 * Return a value for the last id entered, which an app may need to check values for LDAP
 	 *
 	 * @param string $appname app-name
-	 * @param int $min=0 if != 0 minimum id
-	 * @param int $max=0 if != 0 maximum id allowed, if it would be exceeded we return false
+	 * @param int $min if != 0 minimum id
+	 * @param int $max if != 0 maximum id allowed, if it would be exceeded we return false
 	 * @return int current id in the next_id table for a particular app/class or -1 for no app and false if $max is exceeded.
 	 */
 	static function last_id($appname,$min=0,$max=0)

phpgwapi/inc/class.config.inc.php

Doc Comments +4 added lines, -4 removed lines

@@ -118,7 +118,7 @@ 
 	 * @param string $name name of the config-value
 	 * @param mixed $value content, empty or null values are not saved, but deleted
 	 * @param string $app app-name (depreacted to use default of $this->appname set via the constructor!)
-	 * @param boolean $update_cache=true update instance cache and for phpgwapi invalidate session-cache
+	 * @param boolean $update_cache update instance cache and for phpgwapi invalidate session-cache
 	 * @throws egw_exception_wrong_parameter if no $app parameter given for static call
 	 * @return boolean|int true if no change, else number of affected rows
 	 */
@@ -195,7 +195,7 @@ 
 	/**
 	 * sets a single value in the repositry, you need to call save_repository after
 	 *
-	 * @param $variable_name string name of the config
+	 * @param string $variable_name string name of the config
 	 * @param $variable_data mixed the content
 	 */
 	function value($variable_name,$variable_data)
@@ -224,8 +224,8 @@ 
 	 * get customfield array of an application
 	 *
 	 * @param string $app
-	 * @param boolean $all_private_too=false should all the private fields be returned too, default no
-	 * @param string $only_type2=null if given only return fields of type2 == $only_type2
+	 * @param boolean $all_private_too should all the private fields be returned too, default no
+	 * @param string $only_type2 if given only return fields of type2 == $only_type2
 	 * @deprecated use egw_customfields::get()
 	 * @return array with customfields
 	 */

phpgwapi/inc/class.contacts.inc.php

Doc Comments +8 added lines, -10 removed lines

@@ -67,14 +67,14 @@ 
 	 * This method was named read in eGW 1.0 and 1.2
 	 *
 	 * @deprecated since 1.3 use search() instead
-	 * @param int $start=0 starting number of the range, if $limit != 0
-	 * @param int $limit=0 max. number of entries to return, 0=all
-	 * @param array $fields=null fields to return or null for all stock fields, fields are either in the keys or values
+	 * @param int $start starting number of the range, if $limit != 0
+	 * @param int $limit max. number of entries to return, 0=all
+	 * @param array $fields fields to return or null for all stock fields, fields are either in the keys or values
 	 * @param string $query='' search pattern or '' for none
 	 * @param string $filter='' filters with syntax like <name>=<value>,<name2>=<value2>,<name3>=!'' for not empty
 	 * @param string $sort='' sorting: ASC or DESC
 	 * @param string $order='' column to order, default ('') n_family,n_given,email ASC
-	 * @param int $lastmod=-1 return only values modified after given timestamp, default (-1) return all
+	 * @param int $lastmod return only values modified after given timestamp, default (-1) return all
 	 * @param string $cquery='' return only entries starting with given character, default ('') all
 	 * @return array of contacts
 	 */
@@ -182,9 +182,8 @@ 
 	 * @deprecated since 1.3 use save() instead
 	 * @param int $owner owner of the entry
 	 * @param array $fields contains access, cat_id and tif if their param is null
-	 * @param string $access=null 'private' or 'public'
-	 * @param int $cat_id=null
-	 * @param string $tid=null 'n'
+	 * @param string $access 'private' or 'public'
+	 * @param int $cat_id
 	 * @return array/boolean contact or false on failure
 	 */
 	function add($owner,$fields,$access=NULL,$cat_id=NULL,$tid=NULL)
@@ -214,9 +213,8 @@ 
 	 * @param int $id id of the entry
 	 * @param int $owner owner of the entry
 	 * @param array $fields contains access, cat_id and tif if their param is null
-	 * @param string $access=null 'private' or 'public'
-	 * @param int $cat_id=null
-	 * @param string $tid=null 'n'
+	 * @param string $access 'private' or 'public'
+	 * @param int $cat_id
 	 * @return array/boolean contact or false on failure
 	 */
 	function update($id,$owner,$fields,$access=NULL,$cat_id=NULL,$tid=NULL)

phpgwapi/inc/class.country.inc.php

Doc Comments +2 added lines, -5 removed lines

@@ -350,8 +350,6 @@ 
 	 * Selectbox for country-selection
 	 *
 	 * @deprecated use html::select with country_array
-	 * @param string $selected 2-letter iso country-code
-	 * @param string $name='country'
 	 * @return string
 	 */
 	function form_select($code,$name='country')
@@ -362,8 +360,7 @@ 
 	/**
 	 * Get country-name from the 2-letter iso code
 	 *
-	 * @param string $selected 2-letter iso country-code
-	 * @param boolean $translated=true use translated name or english
+	 * @param boolean $translated use translated name or english
 	 * @return string
 	 */
 	function get_full_name($code,$translated=true)
@@ -424,7 +421,7 @@ 
 	/**
 	 * Get list of country names
 	 *
-	 * @param boolean $translated=true use translated names or english
+	 * @param boolean $translated use translated names or english
 	 * @return array with 2-letter code => name pairs
 	 */
 	function countries($translated=true)

phpgwapi/inc/class.db_backup.inc.php

Doc Comments +3 added lines, -2 removed lines

@@ -496,7 +496,7 @@ 
 	 * Restore data from a (compressed) csv file
 	 *
 	 * @param resource $f file opened with fopen for reading
-	 * @param int|string $insert_n_rows =10 how many rows to insert in one sql statement, or string with column-name used as unique key for insert
+	 * @param integer $insert_n_rows =10 how many rows to insert in one sql statement, or string with column-name used as unique key for insert
 	 * @returns int number of rows read from csv file
 	 */
 	function db_restore($f, $insert_n_rows=10)
@@ -1044,7 +1044,7 @@ 
 	/**
 	 * Backup all schemas in the form of a setup/tables_current.inc.php file
 	 *
-	 * @param resource|boolean $f
+	 * @param resource $f
 	 */
 	function schema_backup($f=False)
 	{
@@ -1089,6 +1089,7 @@ 
 	 * Dump an array as php source
 	 *
 	 * copied from etemplate/inc/class.db_tools.inc.php
+	 * @param integer $depth
 	 */
 	private static function write_array($arr,$depth,$parent='')
 	{

phpgwapi/inc/class.egw_cache.inc.php

Doc Comments +21 added lines, -24 removed lines

@@ -103,7 +103,7 @@ 
 	 * @param string $app application storing data
 	 * @param string $location location name for data
 	 * @param mixed $data
-	 * @param int $expiration=0 expiration time in seconds, default 0 = never
+	 * @param int $expiration expiration time in seconds, default 0 = never
 	 * @return boolean true if data could be stored, false otherwise
 	 */
 	static public function setCache($level,$app,$location,$data,$expiration=0)
@@ -138,9 +138,9 @@ 
 	 * @param string $level use egw_cache::(TREE|INSTANCE|SESSION|REQUEST)
 	 * @param string $app application storing data
 	 * @param string|array $location location(s) name for data
-	 * @param callback $callback=null callback to get/create the value, if it's not cache
+	 * @param callback $callback callback to get/create the value, if it's not cache
 	 * @param array $callback_params=array() array with parameters for the callback
-	 * @param int $expiration=0 expiration time in seconds, default 0 = never
+	 * @param int $expiration expiration time in seconds, default 0 = never
 	 * @return mixed NULL if data not found in cache (and no callback specified) or
 	 * 	if $location is an array: location => data pairs for existing location-data, non-existing is not returned
 	 */
@@ -242,7 +242,7 @@ 
 	 * @param string $app application storing data
 	 * @param string $location location name for data
 	 * @param mixed $data
-	 * @param int $expiration=0 expiration time in seconds, default 0 = never
+	 * @param int $expiration expiration time in seconds, default 0 = never
 	 * @return boolean true if data could be stored, false otherwise
 	 */
 	static public function setTree($app,$location,$data,$expiration=0)
@@ -256,9 +256,9 @@ 
 	 *
 	 * @param string $app application storing data
 	 * @param string $location location name for data
-	 * @param callback $callback=null callback to get/create the value, if it's not cache
+	 * @param callback $callback callback to get/create the value, if it's not cache
 	 * @param array $callback_params=array() array with parameters for the callback
-	 * @param int $expiration=0 expiration time in seconds, default 0 = never
+	 * @param int $expiration expiration time in seconds, default 0 = never
 	 * @return mixed NULL if data not found in cache (and no callback specified)
 	 */
 	static public function getTree($app,$location,$callback=null,array $callback_params=array(),$expiration=0)
@@ -284,7 +284,7 @@ 
 	 * @param string $app application storing data
 	 * @param string $location location name for data
 	 * @param mixed $data
-	 * @param int $expiration=0 expiration time in seconds, default 0 = never
+	 * @param int $expiration expiration time in seconds, default 0 = never
 	 * @return boolean true if data could be stored, false otherwise
 	 */
 	static public function setInstance($app,$location,$data,$expiration=0)
@@ -297,9 +297,9 @@ 
 	 *
 	 * @param string $app application storing data
 	 * @param string $location location name for data
-	 * @param callback $callback=null callback to get/create the value, if it's not cache
+	 * @param string[] $callback callback to get/create the value, if it's not cache
 	 * @param array $callback_params=array() array with parameters for the callback
-	 * @param int $expiration=0 expiration time in seconds, default 0 = never
+	 * @param int $expiration expiration time in seconds, default 0 = never
 	 * @return mixed NULL if data not found in cache (and no callback specified)
 	 */
 	static public function getInstance($app,$location,$callback=null,array $callback_params=array(),$expiration=0)
@@ -325,7 +325,7 @@ 
 	 * @param string $app application storing data
 	 * @param string $location location name for data
 	 * @param mixed $data
-	 * @param int $expiration=0 expiration time in seconds, default 0 = never
+	 * @param int $expiration expiration time in seconds, default 0 = never
 	 * @return boolean true if data could be stored, false otherwise
 	 */
 	static public function setSession($app,$location,$data,$expiration=0)
@@ -348,9 +348,9 @@ 
 	 *
 	 * @param string $app application storing data
 	 * @param string $location location name for data
-	 * @param callback $callback=null callback to get/create the value, if it's not cache
+	 * @param callback $callback callback to get/create the value, if it's not cache
 	 * @param array $callback_params=array() array with parameters for the callback
-	 * @param int $expiration=0 expiration time in seconds, default 0 = never
+	 * @param int $expiration expiration time in seconds, default 0 = never
 	 * @return mixed NULL if data not found in cache (and no callback specified)
 	 */
 	static public function &getSession($app,$location,$callback=null,array $callback_params=array(),$expiration=0)
@@ -404,7 +404,7 @@ 
 	 * @param string $app application storing data
 	 * @param string $location location name for data
 	 * @param mixed $data
-	 * @param int $expiration=0 expiration time is NOT used for REQUEST!
+	 * @param int $expiration expiration time is NOT used for REQUEST!
 	 * @return boolean true if data could be stored, false otherwise
 	 */
 	static public function setRequest($app,$location,$data,$expiration=0)
@@ -420,9 +420,9 @@ 
 	 *
 	 * @param string $app application storing data
 	 * @param string $location location name for data
-	 * @param callback $callback=null callback to get/create the value, if it's not cache
+	 * @param callback $callback callback to get/create the value, if it's not cache
 	 * @param array $callback_params=array() array with parameters for the callback
-	 * @param int $expiration=0 expiration time is NOT used for REQUEST!
+	 * @param int $expiration expiration time is NOT used for REQUEST!
 	 * @return mixed NULL if data not found in cache (and no callback specified)
 	 */
 	static public function getRequest($app,$location,$callback=null,array $callback_params=array(),$expiration=0)
@@ -459,7 +459,7 @@ 
 	 * The returned provider already has an opened connection
 	 *
 	 * @param string $level egw_cache::(TREE|INSTANCE) or install_id
-	 * @param boolean $log_not_found=true false do not log if no provider found, used eg. to supress error via unsetCache during installation
+	 * @param boolean $log_not_found false do not log if no provider found, used eg. to supress error via unsetCache during installation
 	 * @return egw_cache_provider
 	 */
 	static protected function get_provider($level, $log_not_found=true)
@@ -525,7 +525,6 @@ 
 	 * Get a system configuration, even if in setup and it's not read
 	 *
 	 * @param string $name
-	 * @param boolean $throw=true throw an exception, if we can't retriev the value
 	 * @return string|boolean string with config or false if not found and !$throw
 	 */
 	static public function get_system_config($name,$throw=true)
@@ -565,8 +564,7 @@ 
 	/**
 	 * Flush (delete) whole (instance) cache or application/class specific part of it
 	 *
-	 * @param $string $level=self::INSTANCE
-	 * @param string $app=null
+	 * @param string $app
 	 */
 	static public function flush($level=self::INSTANCE, $app=null)
 	{
@@ -612,7 +610,7 @@ 
 	/**
 	 * Generate a new instance key and by doing so effectivly flushes whole instance cache
 	 *
-	 * @param string $install_id=null default use install_id of current instance
+	 * @param string $install_id default use install_id of current instance
 	 * @return string new key also stored in self::$instance_key
 	 */
 	static public function generate_instance_key($install_id=null)
@@ -633,8 +631,7 @@ 
 	 * Get keys array from $level, $app and $location
 	 *
 	 * @param string $level egw_cache::(TREE|INSTANCE) or instance_id
-	 * @param string $app=null
-	 * @param string $location=null
+	 * @param string $location
 	 * @return array
 	 */
 	static public function keys($level, $app=null, $location=null)
@@ -714,7 +711,7 @@ 
 	 *
 	 * @param array $keys eg. array($level,$app,$location)
 	 * @param mixed $data
-	 * @param int $expiration=0
+	 * @param int $expiration
 	 * @return boolean true on success, false on error
 	 */
 	function set(array $keys,$data,$expiration=0);
@@ -769,7 +766,7 @@ 
 	/**
 	 * Run several checks on a caching provider
 	 *
-	 * @param boolean $verbose=false true: echo failed checks
+	 * @param boolean $verbose true: echo failed checks
 	 * @return int number of failed checks
 	 */
 	function check($verbose=false)

phpgwapi/inc/class.egw_cache_files.inc.php

Doc Comments +1 added lines, -2 removed lines

@@ -59,7 +59,7 @@ 
 	 *
 	 * @param array $keys eg. array($level,$app,$location)
 	 * @param mixed $data
-	 * @param int $expiration=0
+	 * @param int $expiration
 	 * @return boolean true on success, false on error
 	 */
 	function set(array $keys,$data,$expiration=0)
@@ -162,7 +162,6 @@ 
 	 * Create a path from $keys and $basepath
 	 *
 	 * @param array $keys
-	 * @param boolean $mkdir=false should we create the directory
 	 * @return string
 	 */
 	function filename(array $keys,$mkdir=false)

phpgwapi/inc/class.egw_cache_memcache.inc.php

Doc Comments +1 added lines, -1 removed lines

@@ -70,7 +70,7 @@
 	 *
 	 * @param array $keys eg. array($level,$app,$location)
 	 * @param mixed $data
-	 * @param int $expiration=0
+	 * @param int $expiration
 	 * @return boolean true on success, false on error
 	 */
 	function set(array $keys,$data,$expiration=0)