@@ -265,7 +265,7 @@ discard block |
||
265 | 265 | } |
266 | 266 | |
267 | 267 | /** |
268 | - * @return bool|string |
|
268 | + * @return string|false |
|
269 | 269 | */ |
270 | 270 | public function selectPicturePath() |
271 | 271 | { |
@@ -590,7 +590,7 @@ discard block |
||
590 | 590 | * @param string $Dimension - Resizing happens proportional according to given dimension: height|width|any |
591 | 591 | * @param integer $Max - Maximum size |
592 | 592 | * |
593 | - * @return boolean - true if success, false if failed |
|
593 | + * @return boolean|null - true if success, false if failed |
|
594 | 594 | * |
595 | 595 | * @author Toon Keppens |
596 | 596 | */ |
@@ -751,6 +751,7 @@ discard block |
||
751 | 751 | |
752 | 752 | /** |
753 | 753 | * Sets extra info |
754 | + * @param string $extra |
|
754 | 755 | */ |
755 | 756 | public function setExtra($extra) |
756 | 757 | { |
@@ -1261,7 +1262,7 @@ discard block |
||
1261 | 1262 | * |
1262 | 1263 | * @author Olivier Brouckaert |
1263 | 1264 | * @param array Course info of the destination course |
1264 | - * @return int ID of the new question |
|
1265 | + * @return false|string ID of the new question |
|
1265 | 1266 | */ |
1266 | 1267 | public function duplicate($course_info = null) |
1267 | 1268 | { |
@@ -1636,7 +1637,7 @@ discard block |
||
1636 | 1637 | * @param string $name |
1637 | 1638 | * @param int $course_id |
1638 | 1639 | * @param int $position |
1639 | - * @return bool|int |
|
1640 | + * @return false|string |
|
1640 | 1641 | */ |
1641 | 1642 | static function saveQuestionOption($question_id, $name, $course_id, $position = 0) |
1642 | 1643 | { |
@@ -1775,6 +1776,7 @@ discard block |
||
1775 | 1776 | * @param int Maximum result for the question |
1776 | 1777 | * @param int Type of question (see constants at beginning of question.class.php) |
1777 | 1778 | * @param int Question level/category |
1779 | + * @param string $quiz_id |
|
1778 | 1780 | */ |
1779 | 1781 | public function create_question( |
1780 | 1782 | $quiz_id, |
@@ -1857,6 +1859,7 @@ discard block |
||
1857 | 1859 | /** |
1858 | 1860 | * Get course medias |
1859 | 1861 | * @param int course id |
1862 | + * @param integer $course_id |
|
1860 | 1863 | */ |
1861 | 1864 | static function get_course_medias( |
1862 | 1865 | $course_id, |
@@ -1922,7 +1925,7 @@ discard block |
||
1922 | 1925 | } |
1923 | 1926 | |
1924 | 1927 | /** |
1925 | - * @return array |
|
1928 | + * @return integer[] |
|
1926 | 1929 | */ |
1927 | 1930 | public static function get_default_levels() |
1928 | 1931 | { |
@@ -417,8 +417,8 @@ |
||
417 | 417 | * @param int $question_id The question ID (to which the answer is attached) |
418 | 418 | * @param string $title The text of the answer |
419 | 419 | * @param string $comment The feedback for the answer |
420 | - * @param float|null $score The score you get when picking this answer |
|
421 | - * @param int|null $correct Whether this answer is considered *the* correct one (this is the unique answer type) |
|
420 | + * @param double $score The score you get when picking this answer |
|
421 | + * @param integer $correct Whether this answer is considered *the* correct one (this is the unique answer type) |
|
422 | 422 | */ |
423 | 423 | public function addAnswer( |
424 | 424 | $id, |
@@ -193,6 +193,9 @@ discard block |
||
193 | 193 | $this->visible = $visible; |
194 | 194 | } |
195 | 195 | |
196 | + /** |
|
197 | + * @param integer $id |
|
198 | + */ |
|
196 | 199 | public function set_session_id($id) |
197 | 200 | { |
198 | 201 | $this->session_id = $id; |
@@ -217,6 +220,12 @@ discard block |
||
217 | 220 | /** |
218 | 221 | * Retrieve links and return them as an array of extensions of AbstractLink. |
219 | 222 | * To keep consistency, do not call this method but LinkFactory::load instead. |
223 | + * @param integer $id |
|
224 | + * @param integer $type |
|
225 | + * @param integer $user_id |
|
226 | + * @param string $course_code |
|
227 | + * @param integer $category_id |
|
228 | + * @param integer $visible |
|
220 | 229 | */ |
221 | 230 | public static function load( |
222 | 231 | $id = null, |
@@ -288,7 +297,7 @@ discard block |
||
288 | 297 | } |
289 | 298 | |
290 | 299 | /** |
291 | - * @param $result |
|
300 | + * @param Doctrine\DBAL\Driver\Statement|null $result |
|
292 | 301 | * @return array |
293 | 302 | */ |
294 | 303 | private static function create_objects_from_sql_result($result) |
@@ -463,6 +472,7 @@ discard block |
||
463 | 472 | |
464 | 473 | /** |
465 | 474 | * Internal function used by get_target_categories() |
475 | + * @param integer $level |
|
466 | 476 | */ |
467 | 477 | private function add_target_subcategories($targets, $level, $catid) |
468 | 478 | { |
@@ -492,6 +502,7 @@ discard block |
||
492 | 502 | * Find links by name |
493 | 503 | * To keep consistency, do not call this method but LinkFactory::find_links instead. |
494 | 504 | * @todo can be written more efficiently using a new (but very complex) sql query |
505 | + * @param string $name_mask |
|
495 | 506 | */ |
496 | 507 | public function find_links ($name_mask,$selectcat) |
497 | 508 | { |
@@ -559,21 +570,21 @@ discard block |
||
559 | 570 | } |
560 | 571 | |
561 | 572 | /** |
562 | - * @param $name |
|
573 | + * @param string $name |
|
563 | 574 | */ |
564 | 575 | public function set_name($name) |
565 | 576 | { |
566 | 577 | } |
567 | 578 | |
568 | 579 | /** |
569 | - * @param $description |
|
580 | + * @param string $description |
|
570 | 581 | */ |
571 | 582 | public function set_description($description) |
572 | 583 | { |
573 | 584 | } |
574 | 585 | |
575 | 586 | /** |
576 | - * @param $max |
|
587 | + * @param integer $max |
|
577 | 588 | */ |
578 | 589 | public function set_max($max) |
579 | 590 | { |
@@ -77,7 +77,7 @@ discard block |
||
77 | 77 | } |
78 | 78 | |
79 | 79 | /** |
80 | - * @return float |
|
80 | + * @return integer|null |
|
81 | 81 | */ |
82 | 82 | public function get_certificate_min_score() |
83 | 83 | { |
@@ -97,7 +97,7 @@ discard block |
||
97 | 97 | } |
98 | 98 | |
99 | 99 | /** |
100 | - * @return mixed |
|
100 | + * @return integer |
|
101 | 101 | */ |
102 | 102 | public function get_parent_id() |
103 | 103 | { |
@@ -105,7 +105,7 @@ discard block |
||
105 | 105 | } |
106 | 106 | |
107 | 107 | /** |
108 | - * @return mixed |
|
108 | + * @return integer |
|
109 | 109 | */ |
110 | 110 | public function get_weight() |
111 | 111 | { |
@@ -121,7 +121,7 @@ discard block |
||
121 | 121 | } |
122 | 122 | |
123 | 123 | /** |
124 | - * @return mixed |
|
124 | + * @return boolean |
|
125 | 125 | */ |
126 | 126 | public function is_visible() |
127 | 127 | { |
@@ -244,7 +244,7 @@ discard block |
||
244 | 244 | } |
245 | 245 | |
246 | 246 | /** |
247 | - * @return null |
|
247 | + * @return null|integer |
|
248 | 248 | */ |
249 | 249 | public function get_grade_model_id() |
250 | 250 | { |
@@ -487,7 +487,7 @@ discard block |
||
487 | 487 | } |
488 | 488 | |
489 | 489 | /** |
490 | - * @param $result |
|
490 | + * @param Doctrine\DBAL\Driver\Statement|null $result |
|
491 | 491 | * |
492 | 492 | * @return array |
493 | 493 | */ |
@@ -863,7 +863,8 @@ discard block |
||
863 | 863 | |
864 | 864 | /** |
865 | 865 | * Calculate the score of this category |
866 | - * @param $stud_id student id (default: all students - then the average is returned) |
|
866 | + * @param integer $stud_id student id (default: all students - then the average is returned) |
|
867 | + * @param integer $session_id |
|
867 | 868 | * @return array (score sum, weight sum) |
868 | 869 | * or null if no scores available |
869 | 870 | */ |
@@ -1122,6 +1123,9 @@ discard block |
||
1122 | 1123 | * @param int student id |
1123 | 1124 | * @param string Course code |
1124 | 1125 | * @param int Session id |
1126 | + * @param integer $stud_id |
|
1127 | + * @param string $course_code |
|
1128 | + * @param integer $session_id |
|
1125 | 1129 | */ |
1126 | 1130 | public function get_root_categories_for_student($stud_id, $course_code = null, $session_id = null) |
1127 | 1131 | { |
@@ -1202,6 +1206,9 @@ discard block |
||
1202 | 1206 | * @param int user id (to return everything, use 'null' here) |
1203 | 1207 | * @param string course code (optional) |
1204 | 1208 | * @param int session id (optional) |
1209 | + * @param integer $user_id |
|
1210 | + * @param string $course_code |
|
1211 | + * @param integer $session_id |
|
1205 | 1212 | */ |
1206 | 1213 | public function get_root_categories_for_teacher($user_id, $course_code = null, $session_id = null) |
1207 | 1214 | { |
@@ -1300,6 +1307,7 @@ discard block |
||
1300 | 1307 | |
1301 | 1308 | /** |
1302 | 1309 | * Internal function used by get_target_categories() |
1310 | + * @param integer $level |
|
1303 | 1311 | */ |
1304 | 1312 | private function add_target_subcategories($targets, $level, $catid) |
1305 | 1313 | { |
@@ -1399,6 +1407,8 @@ discard block |
||
1399 | 1407 | |
1400 | 1408 | /** |
1401 | 1409 | * Internal function used by get_tree() |
1410 | + * @param integer $level |
|
1411 | + * @param null|integer $visible |
|
1402 | 1412 | */ |
1403 | 1413 | private function add_subtree ($targets, $level, $catid, $visible) |
1404 | 1414 | { |
@@ -1416,6 +1426,7 @@ discard block |
||
1416 | 1426 | |
1417 | 1427 | /** |
1418 | 1428 | * Generate an array of courses that a teacher hasn't created a category for. |
1429 | + * @param integer $user_id |
|
1419 | 1430 | * @return array 2-dimensional array - every element contains 2 subelements (code, title) |
1420 | 1431 | */ |
1421 | 1432 | public function get_not_created_course_categories ($user_id) |
@@ -1450,6 +1461,7 @@ discard block |
||
1450 | 1461 | |
1451 | 1462 | /** |
1452 | 1463 | * Generate an array of all courses that a teacher is admin of. |
1464 | + * @param integer $user_id |
|
1453 | 1465 | * @return array 2-dimensional array - every element contains 2 subelements (code, title) |
1454 | 1466 | */ |
1455 | 1467 | public function get_all_courses ($user_id) |
@@ -1530,7 +1542,7 @@ discard block |
||
1530 | 1542 | /** |
1531 | 1543 | * Retrieve all categories inside a course independent category |
1532 | 1544 | * that should be visible to a student. |
1533 | - * @param $cat_id parent category |
|
1545 | + * @param integer $cat_id parent category |
|
1534 | 1546 | * @param $stud_id student id |
1535 | 1547 | * @param $cats optional: if defined, the categories will be added to this array |
1536 | 1548 | */ |
@@ -1851,7 +1863,7 @@ discard block |
||
1851 | 1863 | * the platform administrator. |
1852 | 1864 | * @param int locked 1 or unlocked 0 |
1853 | 1865 | |
1854 | - * @return bool |
|
1866 | + * @return boolean|null |
|
1855 | 1867 | * */ |
1856 | 1868 | public function lock($locked) |
1857 | 1869 | { |
@@ -2282,7 +2294,7 @@ discard block |
||
2282 | 2294 | * @param float $score The achieved score |
2283 | 2295 | * @param int $userId The user id |
2284 | 2296 | * @param int $categoryId The gradebook category |
2285 | - * @return int The insert id |
|
2297 | + * @return false|string The insert id |
|
2286 | 2298 | */ |
2287 | 2299 | public static function registerCurrentScore($score, $userId, $categoryId) |
2288 | 2300 | { |
@@ -81,6 +81,9 @@ discard block |
||
81 | 81 | return $this->name; |
82 | 82 | } |
83 | 83 | |
84 | + /** |
|
85 | + * @return string |
|
86 | + */ |
|
84 | 87 | public function get_description() |
85 | 88 | { |
86 | 89 | return $this->description; |
@@ -208,7 +211,7 @@ discard block |
||
208 | 211 | * @param int $user_id user id (evaluation owner) |
209 | 212 | * @param string $course_code course code |
210 | 213 | * @param int $category_id parent category |
211 | - * @param $visible visible |
|
214 | + * @param integer $visible visible |
|
212 | 215 | */ |
213 | 216 | public static function load( |
214 | 217 | $id = null, |
@@ -270,7 +273,7 @@ discard block |
||
270 | 273 | |
271 | 274 | |
272 | 275 | /** |
273 | - * @param array $result |
|
276 | + * @param Doctrine\DBAL\Driver\Statement|null $result |
|
274 | 277 | * @return array |
275 | 278 | */ |
276 | 279 | private static function create_evaluation_objects_from_sql_result($result) |
@@ -658,6 +661,7 @@ discard block |
||
658 | 661 | |
659 | 662 | /** |
660 | 663 | * Internal function used by get_target_categories() |
664 | + * @param integer $level |
|
661 | 665 | */ |
662 | 666 | private function add_target_subcategories($targets, $level, $catid) |
663 | 667 | { |
@@ -372,7 +372,7 @@ |
||
372 | 372 | } |
373 | 373 | |
374 | 374 | /** |
375 | - * @return array|string |
|
375 | + * @return string |
|
376 | 376 | */ |
377 | 377 | public function get_type_name() |
378 | 378 | { |
@@ -121,7 +121,7 @@ |
||
121 | 121 | |
122 | 122 | /** |
123 | 123 | * Has anyone done this exercise yet ? |
124 | - * @return int |
|
124 | + * @return boolean |
|
125 | 125 | */ |
126 | 126 | public function has_results() |
127 | 127 | { |
@@ -95,7 +95,7 @@ |
||
95 | 95 | |
96 | 96 | /** |
97 | 97 | * Get the progress of this learnpath. Only the last attempt are taken into account. |
98 | - * @param $stud_id student id (default: all students who have results - then the average is returned) |
|
98 | + * @param integer $stud_id student id (default: all students who have results - then the average is returned) |
|
99 | 99 | * @return array (score, max) if student is given |
100 | 100 | * array (sum of scores, number of scores) otherwise |
101 | 101 | * or null if no scores available |
@@ -59,6 +59,9 @@ |
||
59 | 59 | $this->evaluation = $evaluation_id; |
60 | 60 | } |
61 | 61 | |
62 | + /** |
|
63 | + * @param string $creation_date |
|
64 | + */ |
|
62 | 65 | public function set_date($creation_date) |
63 | 66 | { |
64 | 67 | $this->created_at = $creation_date; |