EGroupware / egroupware

api/src/Storage/Customfields.php

Doc Comments +4 added lines, -1 removed lines

@@ -169,7 +169,7 @@ 
 	 * @param int|boolean $account =false Filter private for given account_id,
 	 *	false for current user or true for all the private fields be returned too, default current user
 	 * @param string $only_type2 =null if given only return fields of type2 == $only_type2
-	 * @return boolen true: if there is a custom field useing html, false if not
+	 * @return boolean true: if there is a custom field useing html, false if not
 	 */
 	public static function use_html($app, $account=false, $only_type2=null)
 	{
@@ -595,6 +595,9 @@ 
 		}
 	}
 
+	/**
+	 * @param string $entry_id
+	 */
 	protected static function handle_file($entry_id, $field, $value)
 	{
 		$path = Api\Etemplate\Widget\Vfs::get_vfs_path($field['app'].":$entry_id:".$field['label']);

calendar/inc/class.calendar_import_csv.inc.php

Doc Comments +4 added lines, -4 removed lines

@@ -74,7 +74,7 @@ 
 	 * @param calendar_egw_record record The egw_record object being imported
 	 * @param importexport_iface_import_record import_csv Import object contains current state
 	 *
-	 * @return boolean success
+	 * @return null|boolean success
 	 */
 	public function import_record(\calendar_egw_record &$record, &$import_csv)
 	{
@@ -182,8 +182,9 @@ 
 	/**
 	 * Parse participants field into calendar resources
 	 *
-	 * @param string $participants
 	 *
+	 * @param calendar_egw_record $record
+	 * @param string $warnings
 	 * @return array
 	 */
 	protected function parse_participants($record, &$warnings)
@@ -289,7 +290,6 @@ 
 	 * perform the required action
 	 *
 	 * @param int $_action one of $this->actions
-	 * @param array $_data record data for the action
 	 * @return bool success or not
 	 */
 	protected function action ( $_action, importexport_iface_egw_record &$record, $record_num = 0 )
@@ -423,7 +423,7 @@ 
 	/**
 	 * Alter a row for preview to show multiple participants instead of Array
 	 *
-	 * @param egw_record $row_entry
+	 * @param importexport_iface_egw_record $row_entry
 	 */
 	protected function row_preview(importexport_iface_egw_record &$row_entry)
 	{

api/src/Framework/Updates.php

Doc Comments +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
 	 * Get versions of available updates
 	 *
 	 * @param string $api =null major api version to return security for, default latest
-	 * @return array verions for keys "current" and "security"
+	 * @return string verions for keys "current" and "security"
 	 */
 	public static function available($api=null)
 	{

mail/tests/SaveToVfsTest.php

Doc Comments +4 added lines

@@ -103,6 +103,10 @@
 class VfsTestMail extends \mail_bo
 {
 	// Expose for testing
+
+	/**
+	 * @param string $filename
+	 */
 	public static function clean_subject_for_filename($filename)
 	{
 		return parent::clean_subject_for_filename($filename);

infolog/tests/ProjectTemplateTest.php

Doc Comments -1 removed lines

@@ -116,7 +116,6 @@
 	/**
 	 * Check that the project elements are present, and have the provided status.
 	 *
-	 * @param String $status
 	 */
 	protected function checkClonedElements($clone_id)
 	{

notifications/inc/class.notifications_popup.inc.php

Doc Comments -1 removed lines

@@ -117,7 +117,6 @@
 	 * saves notification into database so that the client can fetch it from there
 	 *
 	 * @param string $_message
-	 * @param array $_user_sessions
 	 * @param array $_data
 	 */
 	private function save($_message, $_data) {

calendar/inc/class.calendar_merge.inc.php

Doc Comments +2 added lines, -2 removed lines

@@ -179,7 +179,7 @@ 
 				{
 					$value = date($format, $key == 'end' ? $id['end'] : $id['start']);
 					if($key == 'month') $value = lang($value);
-					$values["$\$range/$key$$"] = $value;
+					$values["$\$range/$key$key$"] = $value;
 				}
 			}
 			$replacements += $values;
@@ -246,7 +246,7 @@ 
 			}
 			foreach($types as $t_id => $type)
 			{
-				$replacements['$$'.($prefix ? $prefix . '/' : '') . "calendar_participants/{$t_id}$$"] = implode(', ',$type);
+				$replacements['$$'.($prefix ? $prefix . '/' : '') . "calendar_participants/{$t_id}}$"] = implode(', ',$type);
 			}
 		}
 		// Participant email list (not declined)

mail/inc/class.mail_ui.inc.php

Doc Comments +11 added lines, -7 removed lines

@@ -1820,9 +1820,9 @@ 
 	 * function header2gridelements - to populate the grid elements with the collected Data
 	 *
 	 * @param array $_headers headerdata to process
-	 * @param array $cols cols to populate
+	 * @param string[] $cols cols to populate
 	 * @param array $_folderName to ensure the uniqueness of the uid over all folders
-	 * @param array $_folderType used to determine if we need to populate from/to
+	 * @param integer $_folderType used to determine if we need to populate from/to
 	 * @return array populated result array
 	 */
 	public function header2gridelements($_headers, $cols, $_folderName, $_folderType=0)
@@ -2282,7 +2282,7 @@ 
 
 	/**
 	 * Export stored smime certificate in database
-	 * @return boolean return false if not successful
+	 * @return false|null return false if not successful
 	 */
 	function smimeExportCert()
 	{
@@ -3256,6 +3256,9 @@ 
 		return $GLOBALS['egw']->framework->header();
 	}
 
+	/**
+	 * @param string $body
+	 */
 	function showBody(&$body, $print=true,$fullPageTags=true)
 	{
 		$BeginBody = '<div class="mailDisplayBody">
@@ -3747,7 +3750,7 @@ 
 	 * @param string $_folder (passed by reference) will set the folder used. must be set with a folder, but will hold modifications if
 	 *					folder is modified
 	 * @param string $importID ID for the imported message, used by attachments to identify them unambiguously
-	 * @return mixed $messageUID or exception
+	 * @return string $messageUID or exception
 	 */
 	function importMessageToFolder($_formData,&$_folder,$importID='')
 	{
@@ -3908,7 +3911,7 @@ 
 	 *
 	 * @param string _messageID UID
 	 *
-	 * @return xajax response
+	 * @return string|null response
 	 */
 	function loadEmailBody($_messageID=null,$_partID=null,$_htmloptions=null)
 	{
@@ -4455,7 +4458,7 @@ 
 	 * ajax_deleteFolder - its called via json, so the function must start with ajax (or the class-name must contain ajax)
 	 * @param string $_folderName folder to delete
 	 * @param boolean $_return = false wheter return the success value (true) or send response to client (false)
-	 * @return nothing
+	 * @return boolean|null
 	 */
 	function ajax_deleteFolder($_folderName, $_return = false)
 	{
@@ -5224,8 +5227,9 @@ 
 	 * @param string _move2ArchiveMarker marker to indicate if a move 2 archive was triggered
 	 * @param boolean _return if true the function will return the result instead of
 	 * responding to client
+	 * @param string $_folderName
 	 *
-	 * @return xajax response
+	 * @return string|null response
 	 */
 	function ajax_copyMessages($_folderName, $_messageList, $_copyOrMove='copy', $_move2ArchiveMarker='_', $_return = false)
 	{

mail/inc/class.mail_compose.inc.php

Doc Comments +9 added lines, -2 removed lines

@@ -1600,6 +1600,10 @@ 
 	// $_mode can be:
 	// single: for a reply to one address
 	// all: for a reply to all
+
+	/**
+	 * @param Mail\Imap $_icServer
+	 */
 	function getDraftData($_icServer, $_folder, $_uid, $_partID=NULL)
 	{
 		unset($_icServer);	// not used
@@ -1800,6 +1804,9 @@ 
 		return false;
 	}
 
+	/**
+	 * @param Mail\Imap $_icServer
+	 */
 	function getForwardData($_icServer, $_folder, $_uid, $_partID, $_mode=false)
 	{
 		if ($_mode)
@@ -2072,7 +2079,7 @@ 
 	/**
 	 * Gather the replyData and save it with the session, to be used then
 	 *
-	 * @param $_mode can be:
+	 * @param string $_mode can be:
 	 * 		single: for a reply to one address
 	 * 		all: for a reply to all
 	 * 		forward: inlineforwarding of a message with its attachments
@@ -3625,7 +3632,7 @@ 
 	 * The results are limited to 10 each of group lists and normal lists
 	 *
 	 * @param String $_searchString
-	 * @param Contacts $contacts_obj
+	 * @param Api\Contacts $contacts_obj
 	 * @return array
 	 */
 	protected static function get_lists($_searchString, &$contacts_obj)