@@ -20,9 +20,9 @@ |
||
20 | 20 | * @param int $user_id User ID |
21 | 21 | * @param int $view_id View ID |
22 | 22 | * @param int $item_id Item ID |
23 | - * @param float $score Current score |
|
24 | - * @param float $max Maximum score |
|
25 | - * @param float $min Minimum score |
|
23 | + * @param integer $score Current score |
|
24 | + * @param integer $max Maximum score |
|
25 | + * @param integer $min Minimum score |
|
26 | 26 | * @param string $status Lesson status |
27 | 27 | * @param int $time Session time |
28 | 28 | * @param string $suspend Suspend data |
@@ -171,7 +171,8 @@ |
||
171 | 171 | /** |
172 | 172 | * Get images files from remote host (with webservices) |
173 | 173 | * @param array current ppt file |
174 | - * @return array images files |
|
174 | + * @param string $file |
|
175 | + * @return string images files |
|
175 | 176 | */ |
176 | 177 | private function _get_remote_ppt2lp_files($file) |
177 | 178 | { |
@@ -38,7 +38,7 @@ discard block |
||
38 | 38 | /** |
39 | 39 | * Gets html pages and compose them into a learning path |
40 | 40 | * @param array The files that will compose the generated learning path. Unused so far. |
41 | - * @return boolean False if file does not exit. Nothing otherwise. |
|
41 | + * @return false|null False if file does not exit. Nothing otherwise. |
|
42 | 42 | */ |
43 | 43 | public function make_lp($files = array()) |
44 | 44 | { |
@@ -92,6 +92,7 @@ discard block |
||
92 | 92 | * Manages chapter splitting |
93 | 93 | * @param string Chapter header |
94 | 94 | * @param string Content |
95 | + * @param string $content |
|
95 | 96 | * @return void |
96 | 97 | */ |
97 | 98 | function dealPerChapter($header, $content) |
@@ -164,6 +165,7 @@ discard block |
||
164 | 165 | * Manages page splitting |
165 | 166 | * @param string Page header |
166 | 167 | * @param string Page body |
168 | + * @param string $body |
|
167 | 169 | * @return void |
168 | 170 | */ |
169 | 171 | function dealPerPage($header, $body) |
@@ -320,6 +320,10 @@ |
||
320 | 320 | * @param icon - if ="icon" then the small icon will appear |
321 | 321 | * if ="wrap" then wrapped settings are used (and no icon is displayed) |
322 | 322 | * if ="nolink" then only the name is returned with no href and no icon (note:only in this case, the result is not displayed, but returned) |
323 | + * @param string $completed |
|
324 | + * @param string $id_in_path |
|
325 | + * @param string $builder |
|
326 | + * @param string $icon |
|
323 | 327 | * @todo this function is too long, rewrite |
324 | 328 | */ |
325 | 329 | function display_addedresource_link_in_learnpath($type, $id, $completed, $id_in_path, $builder, $icon, $level = 0) { |
@@ -271,7 +271,7 @@ discard block |
||
271 | 271 | * @param int $userMaxScore |
272 | 272 | * @param int $sessionId |
273 | 273 | * |
274 | - * @return bool Returns -1 on error |
|
274 | + * @return boolean|null Returns -1 on error |
|
275 | 275 | */ |
276 | 276 | public function import_manifest($courseCode, $userMaxScore = 1, $sessionId = 0, $userId = 0) |
277 | 277 | { |
@@ -925,7 +925,7 @@ discard block |
||
925 | 925 | * @param string course Code |
926 | 926 | * @param string LP ID (in database) |
927 | 927 | * @param string Manifest file path (optional if lp_id defined) |
928 | - * @return integer New LP ID or false on failure |
|
928 | + * @return boolean New LP ID or false on failure |
|
929 | 929 | * TODO @TODO Implement imsmanifest_path parameter |
930 | 930 | */ |
931 | 931 | public function reimport_manifest($courseCode, $lp_id = null, $imsmanifest_path = '') |
@@ -118,7 +118,7 @@ discard block |
||
118 | 118 | |
119 | 119 | /** |
120 | 120 | * This function retrieves the existing permissions of a user, group or role. |
121 | -* @param $content are we retrieving the rights of a user, a group or a role (the database depends on it) |
|
121 | +* @param string $content are we retrieving the rights of a user, a group or a role (the database depends on it) |
|
122 | 122 | * @param $id the id of the user, group or role |
123 | 123 | * @author Patrick Cool <[email protected]>, Ghent University |
124 | 124 | * @version 1.0 |
@@ -526,7 +526,7 @@ discard block |
||
526 | 526 | |
527 | 527 | /** |
528 | 528 | * This function gets all the roles that are defined |
529 | -* @param $content are we finding the roles for a user or a group (the database depends on it) |
|
529 | +* @param string $content are we finding the roles for a user or a group (the database depends on it) |
|
530 | 530 | * @param $id the id of the user or group |
531 | 531 | * @param string Deprecated parameter allowing use of 'platform' scope - the corresponding tables don't exist anymore so the scope is always set to 'course' |
532 | 532 | * @return array that contains the name of the roles the user has |
@@ -338,6 +338,10 @@ |
||
338 | 338 | * @param icon - if ="icon" then the small icon will appear |
339 | 339 | * if ="wrap" then wrapped settings are used (and no icon is displayed) |
340 | 340 | * if ="nolink" then only the name is returned with no href and no icon (note:only in this case, the result is not displayed, but returned) |
341 | +* @param string $completed |
|
342 | +* @param string $id_in_path |
|
343 | +* @param string $builder |
|
344 | +* @param string $icon |
|
341 | 345 | * @todo this function is too long, rewrite |
342 | 346 | */ |
343 | 347 | function display_addedresource_link_in_learnpath($type, $id, $completed, $id_in_path, $builder, $icon, $level = 0) |
@@ -735,7 +735,6 @@ |
||
735 | 735 | * lock = the user can no longer use this account |
736 | 736 | * @author Patrick Cool <[email protected]>, Ghent University |
737 | 737 | * @param int $active the current state of the account |
738 | - * @param int $user_id The user id |
|
739 | 738 | * @param string $urlParams |
740 | 739 | * |
741 | 740 | * @return string Some HTML-code with the lock/unlock button |
@@ -84,7 +84,7 @@ |
||
84 | 84 | } |
85 | 85 | |
86 | 86 | /** |
87 | - * @param $directoryPath |
|
87 | + * @param string $directoryPath |
|
88 | 88 | * @return bool |
89 | 89 | */ |
90 | 90 | function deleteDirectory($directoryPath) |