diff --git a/main/inc/lib/formvalidator/FormValidator.class.php b/main/inc/lib/formvalidator/FormValidator.class.php index 2abbdf8ae7..37d8fb5143 100755 --- a/main/inc/lib/formvalidator/FormValidator.class.php +++ b/main/inc/lib/formvalidator/FormValidator.class.php @@ -233,8 +233,8 @@ EOT; } /** - * @param $name - * @param $label + * @param string $name + * @param string $label * @param array $options * @param array $attributes * @throws @@ -1063,7 +1063,7 @@ EOT; /** * This function has been created for avoiding changes directly within QuickForm class. * When we use it, the element is threated as 'required' to be dealt during validation. - * @param array $element The array of elements + * @param array $elements The array of elements * @param string $message The message displayed */ public function add_multiple_required_rule($elements, $message) @@ -1339,12 +1339,12 @@ EOT; } /** - * @param $name + * @param string $name * @param $label * @param bool $required * @param array $attributes * @param bool $allowNegative - * @param null $minValue + * @param integer $minValue * @param null $maxValue */ public function addFloat( diff --git a/main/inc/lib/hook/HookManagement.php b/main/inc/lib/hook/HookManagement.php index e07fcb0bc4..46d0da5d74 100644 --- a/main/inc/lib/hook/HookManagement.php +++ b/main/inc/lib/hook/HookManagement.php @@ -138,7 +138,7 @@ class HookManagement implements HookManagementInterface /** * Return a list an associative array where keys are the active hook observer class name - * @param $eventName + * @param string $eventName * * @return array */ diff --git a/main/inc/lib/link.lib.php b/main/inc/lib/link.lib.php index 2acaace1d0..29f278367d 100755 --- a/main/inc/lib/link.lib.php +++ b/main/inc/lib/link.lib.php @@ -470,6 +470,7 @@ class Link extends Model * * Get link info * @param int link id + * @param integer $id * @return array link info * **/ @@ -854,7 +855,7 @@ class Link extends Model } /** - * @param $categoryId + * @param integer $categoryId * @param $courseId * @param $sessionId * @param bool $withBaseContent @@ -941,8 +942,8 @@ class Link extends Model * @author Julio Montoya * * @param $catid - * @param $courseId - * @param $session_id + * @param integer $courseId + * @param integer $session_id * @return string */ public static function showLinksPerCategory($catid, $courseId, $session_id) @@ -1150,6 +1151,8 @@ class Link extends Model /** * Displays the edit, delete and move icons * @param int Category ID + * @param integer $currentCategory + * @param integer $countCategories * @return string * * @author Patrick Cool , Ghent University @@ -1290,6 +1293,7 @@ class Link extends Model /** * CSV file import functions * @author René Haentjens , Ghent University + * @param string $catname */ public static function get_cat($catname) { @@ -1324,6 +1328,11 @@ class Link extends Model /** * CSV file import functions * @author René Haentjens , Ghent University + * @param string $url + * @param string $title + * @param string $description + * @param string $on_homepage + * @param string $hidden */ public static function put_link($url, $cat, $title, $description, $on_homepage, $hidden) { diff --git a/main/inc/lib/login.lib.php b/main/inc/lib/login.lib.php index 24d38673c3..8f542be037 100755 --- a/main/inc/lib/login.lib.php +++ b/main/inc/lib/login.lib.php @@ -276,6 +276,7 @@ class Login * @global bool $is_platformAdmin * @global bool $is_allowedCreateCourse * @global object $_user + * @param boolean $reset */ public static function init_user($user_id, $reset) { @@ -366,7 +367,7 @@ class Login * @global type $is_allowed_in_course * * @param type $course_id - * @param type $reset + * @param boolean $reset */ static function init_course($course_id, $reset) { diff --git a/main/inc/lib/plugin.class.php b/main/inc/lib/plugin.class.php index 2561c0735c..f89bed0f7a 100755 --- a/main/inc/lib/plugin.class.php +++ b/main/inc/lib/plugin.class.php @@ -150,7 +150,7 @@ class Plugin /** * Returns the contents of the CSS defined by the plugin - * @return array + * @return string */ public function get_css() { @@ -168,7 +168,7 @@ class Plugin /** * Returns an HTML form (generated by FormValidator) of the plugin settings - * @return string FormValidator-generated form + * @return FormValidator FormValidator-generated form */ public function get_settings_form() { @@ -390,7 +390,7 @@ class Plugin * @param boolean $add_tool_link Whether to add a tool link or not * (some tools might just offer a configuration section and act on the backend) * - * @return boolean False on error, null otherwise + * @return boolean|null False on error, null otherwise */ public function install_course_fields($courseId, $add_tool_link = true) { @@ -501,7 +501,7 @@ class Plugin * tool on the course's homepage * @param int $courseId * - * @return void + * @return false|null */ public function uninstall_course_fields($courseId) { @@ -607,7 +607,7 @@ class Plugin * @param string $tabName * @param string $url * - * @return boolean + * @return false|string */ public function addTab($tabName, $url) { diff --git a/main/inc/lib/thematic.lib.php b/main/inc/lib/thematic.lib.php index f070b15592..39b44a71a5 100755 --- a/main/inc/lib/thematic.lib.php +++ b/main/inc/lib/thematic.lib.php @@ -242,6 +242,9 @@ class Thematic /** * get thematic list * @param int Thematic id (optional), get list by id + * @param integer $thematic_id + * @param string $course_code + * @param integer $session_id * @return array Thematic data */ public static function get_thematic_list( @@ -799,6 +802,7 @@ class Thematic /** * delete thematic advance * @param int Thematic advance id + * @param integer $thematic_advance_id * @return int Affected rows */ public function thematic_advance_destroy($thematic_advance_id) @@ -1116,6 +1120,7 @@ class Thematic /** * update done thematic advances from thematic details interface * @param int Thematic id + * @param integer $thematic_advance_id * @return int Affected rows */ public function update_done_thematic_advances($thematic_advance_id) @@ -1343,6 +1348,7 @@ class Thematic * Get average of advances by thematic * @param int Thematic id * @param string Course code (optional) + * @param string $course_code * @return float Average of thematic advances */ public function get_average_of_advances_by_thematic($thematic_id, $course_code = null) @@ -1407,6 +1413,7 @@ class Thematic * @param string Content * @param string Date and time * @param int Duration in hours + * @param integer $id * @return void */ public function set_thematic_advance_attributes( @@ -1437,7 +1444,7 @@ class Thematic /** * get thematic id - * @return void + * @return integer */ public function get_thematic_id() { diff --git a/main/user/subscribe_user.php b/main/user/subscribe_user.php index dbbe41ed95..8983220f0b 100755 --- a/main/user/subscribe_user.php +++ b/main/user/subscribe_user.php @@ -710,7 +710,6 @@ function reg_filter($user_id) * lock = the user can no longer use this account * @author Patrick Cool , Ghent University * @param int $active the current state of the account - * @param int $user_id The user id * @param string $url_params * @return string Some HTML-code with the lock/unlock button */ diff --git a/main/user/user.php b/main/user/user.php index 463bd2982d..a713fe1055 100755 --- a/main/user/user.php +++ b/main/user/user.php @@ -933,7 +933,6 @@ function get_user_data($from, $number_of_items, $column, $direction) * lock = the user can no longer use this account * @author Patrick Cool , Ghent University * @param int $active the current state of the account - * @param int $user_id The user id * @param string $urlParams * * @return string Some HTML-code with the lock/unlock button diff --git a/main/webservices/access_url.php b/main/webservices/access_url.php index 9e6a9d4037..80c757b15f 100644 --- a/main/webservices/access_url.php +++ b/main/webservices/access_url.php @@ -17,6 +17,9 @@ define('WS_ERROR_NOT_FOUND_RESULT', 2); define('WS_ERROR_INVALID_INPUT', 3); define('WS_ERROR_SETTING', 4); +/** + * @param integer $code + */ function return_error($code) { $fault = null; switch ($code) { diff --git a/src/Chamilo/CoreBundle/Entity/AccessUrlRelUserGroup.php b/src/Chamilo/CoreBundle/Entity/AccessUrlRelUserGroup.php index 3084e69981..0bc7f86cd9 100644 --- a/src/Chamilo/CoreBundle/Entity/AccessUrlRelUserGroup.php +++ b/src/Chamilo/CoreBundle/Entity/AccessUrlRelUserGroup.php @@ -40,7 +40,7 @@ class AccessUrlRelUserGroup * Set accessUrlId * * @param integer $accessUrlId - * @return AccessUrlRelUser + * @return AccessUrlRelUserGroup */ public function setAccessUrlId($accessUrlId) { diff --git a/src/Chamilo/CoreBundle/Entity/ChatVideo.php b/src/Chamilo/CoreBundle/Entity/ChatVideo.php index dc99b6af90..23a77cc052 100644 --- a/src/Chamilo/CoreBundle/Entity/ChatVideo.php +++ b/src/Chamilo/CoreBundle/Entity/ChatVideo.php @@ -61,7 +61,7 @@ class ChatVideo * Set fromUser * * @param integer $fromUser - * @return Chat + * @return ChatVideo */ public function setFromUser($fromUser) { @@ -84,7 +84,7 @@ class ChatVideo * Set toUser * * @param integer $toUser - * @return Chat + * @return ChatVideo */ public function setToUser($toUser) { @@ -107,7 +107,7 @@ class ChatVideo * Set room_name * * @param string $roomName - * @return Chat + * @return ChatVideo */ public function setRoomName($roomName) { @@ -130,7 +130,7 @@ class ChatVideo * Set datetime * * @param \DateTime $datetime - * @return Chat + * @return ChatVideo */ public function setDatetime($datetime) { diff --git a/src/Chamilo/CoreBundle/Entity/CourseRelUser.php b/src/Chamilo/CoreBundle/Entity/CourseRelUser.php index ad01696b59..9859d7a9d5 100644 --- a/src/Chamilo/CoreBundle/Entity/CourseRelUser.php +++ b/src/Chamilo/CoreBundle/Entity/CourseRelUser.php @@ -139,7 +139,7 @@ class CourseRelUser } /** - * @param $user + * @param User $user * @return $this */ public function setUser($user) diff --git a/src/Chamilo/CoreBundle/Entity/Language.php b/src/Chamilo/CoreBundle/Entity/Language.php index d7f18e384b..efbb4471dd 100644 --- a/src/Chamilo/CoreBundle/Entity/Language.php +++ b/src/Chamilo/CoreBundle/Entity/Language.php @@ -231,7 +231,7 @@ class Language /** * Get id * - * @return boolean + * @return integer */ public function getId() { diff --git a/src/Chamilo/CoreBundle/Entity/Message.php b/src/Chamilo/CoreBundle/Entity/Message.php index a98b6f8fce..37be84c8c0 100644 --- a/src/Chamilo/CoreBundle/Entity/Message.php +++ b/src/Chamilo/CoreBundle/Entity/Message.php @@ -320,7 +320,7 @@ class Message * Set votes * * @param integer $votes - * @return integer + * @return Message */ public function setVotes($votes) { diff --git a/src/Chamilo/CoreBundle/Entity/SequenceResource.php b/src/Chamilo/CoreBundle/Entity/SequenceResource.php index 505142da98..7a88e4aca7 100644 --- a/src/Chamilo/CoreBundle/Entity/SequenceResource.php +++ b/src/Chamilo/CoreBundle/Entity/SequenceResource.php @@ -58,7 +58,7 @@ class SequenceResource } /** - * @return string + * @return integer */ public function getType() { @@ -123,7 +123,7 @@ class SequenceResource } /** - * @param mixed $sequence + * @param Sequence $sequence * @return $this */ public function setSequence(Sequence $sequence) diff --git a/src/Chamilo/CoreBundle/Entity/SkillRelUser.php b/src/Chamilo/CoreBundle/Entity/SkillRelUser.php index a82a58a448..acb5b5d140 100644 --- a/src/Chamilo/CoreBundle/Entity/SkillRelUser.php +++ b/src/Chamilo/CoreBundle/Entity/SkillRelUser.php @@ -121,7 +121,7 @@ class SkillRelUser /** * Get user - * @return \User + * @return User */ public function getUser() { @@ -390,7 +390,7 @@ class SkillRelUser /** * Calculate the average value from the feedback comments - * @return int + * @return string */ public function getAverage() { diff --git a/src/Chamilo/CoreBundle/Entity/SkillRelUserComment.php b/src/Chamilo/CoreBundle/Entity/SkillRelUserComment.php index 1554d7ece1..273f52a653 100644 --- a/src/Chamilo/CoreBundle/Entity/SkillRelUserComment.php +++ b/src/Chamilo/CoreBundle/Entity/SkillRelUserComment.php @@ -108,7 +108,7 @@ class SkillRelUserComment /** * Get feedbackDateTime - * @return type + * @return \DateTime */ public function getFeedbackDateTime() { diff --git a/src/Chamilo/CourseBundle/Entity/CForumThread.php b/src/Chamilo/CourseBundle/Entity/CForumThread.php index 6122a04a23..145150a653 100644 --- a/src/Chamilo/CourseBundle/Entity/CForumThread.php +++ b/src/Chamilo/CourseBundle/Entity/CForumThread.php @@ -181,7 +181,7 @@ class CForumThread /** * set threadPeerQualify - * @param $threadPeerQualify + * @param integer $threadPeerQualify * @return $this */ public function setThreadPeerQualify($threadPeerQualify) diff --git a/src/Chamilo/FaqBundle/Entity/QuestionTranslation.php b/src/Chamilo/FaqBundle/Entity/QuestionTranslation.php index 87f2c1b4b5..230938ac55 100644 --- a/src/Chamilo/FaqBundle/Entity/QuestionTranslation.php +++ b/src/Chamilo/FaqBundle/Entity/QuestionTranslation.php @@ -58,7 +58,7 @@ class QuestionTranslation /** * @param string $headline - * @return CategoryTranslation + * @return QuestionTranslation */ public function setHeadline($headline) { @@ -77,7 +77,7 @@ class QuestionTranslation /** * @param string $body - * @return CategoryTranslation + * @return QuestionTranslation */ public function setBody($body) { diff --git a/src/Chamilo/ThemeBundle/Model/MenuItemInterface.php b/src/Chamilo/ThemeBundle/Model/MenuItemInterface.php index ccb6aa67d8..04a7fe7c58 100644 --- a/src/Chamilo/ThemeBundle/Model/MenuItemInterface.php +++ b/src/Chamilo/ThemeBundle/Model/MenuItemInterface.php @@ -20,29 +20,29 @@ interface MenuItemInterface { public function getIdentifier(); /** - * @return mixed + * @return string */ public function getLabel(); /** - * @return mixed + * @return string */ public function getRoute(); /** - * @return mixed + * @return boolean */ public function isActive(); /** - * @param $isActive + * @param boolean $isActive * - * @return mixed + * @return MenuItemModel */ public function setIsActive($isActive); /** - * @return mixed + * @return boolean */ public function hasChildren(); @@ -54,14 +54,14 @@ interface MenuItemInterface { /** * @param MenuItemInterface $child * - * @return mixed + * @return MenuItemModel */ public function addChild(MenuItemInterface $child); /** * @param MenuItemInterface $child * - * @return mixed + * @return MenuItemModel */ public function removeChild(MenuItemInterface $child); @@ -76,24 +76,24 @@ interface MenuItemInterface { public function getBadge(); /** - * @return mixed + * @return string */ public function getBadgeColor(); /** - * @return mixed + * @return MenuItemInterface */ public function getParent(); /** - * @return mixed + * @return boolean */ public function hasParent(); /** * @param MenuItemInterface $parent * - * @return mixed + * @return MenuItemModel */ public function setParent(MenuItemInterface $parent = null);