@@ -32,7 +32,7 @@ discard block |
||
32 | 32 | /** |
33 | 33 | * Constructor. |
34 | 34 | * |
35 | - * @param $question Ims2Question object we want to export. |
|
35 | + * @param Ims2Question $question Ims2Question object we want to export. |
|
36 | 36 | */ |
37 | 37 | function ImsAssessmentItem($question) |
38 | 38 | { |
@@ -101,7 +101,7 @@ discard block |
||
101 | 101 | * This is a default behaviour, some classes may want to override this. |
102 | 102 | * |
103 | 103 | * @param $standalone: Boolean stating if it should be exported as a stand-alone question |
104 | - * @return A string, the XML flow for an Item. |
|
104 | + * @return string string, the XML flow for an Item. |
|
105 | 105 | */ |
106 | 106 | function export($standalone = false) |
107 | 107 | { |
@@ -154,7 +154,7 @@ discard block |
||
154 | 154 | |
155 | 155 | /** |
156 | 156 | * Constructor. |
157 | - * @param $exe The Exercise instance to export |
|
157 | + * @param Exercise $exe The Exercise instance to export |
|
158 | 158 | * @author Amand Tihon <[email protected]> |
159 | 159 | */ |
160 | 160 | function ImsSection($exe) |
@@ -239,7 +239,7 @@ discard block |
||
239 | 239 | * Export the exercise in IMS/QTI. |
240 | 240 | * |
241 | 241 | * @param bool $standalone Wether it should include XML tag and DTD line. |
242 | - * @return a string containing the XML flow |
|
242 | + * @return string string containing the XML flow |
|
243 | 243 | * @author Amand Tihon <[email protected]> |
244 | 244 | */ |
245 | 245 | function export($standalone) |
@@ -383,7 +383,7 @@ discard block |
||
383 | 383 | * This is a default behaviour, some classes may want to override this. |
384 | 384 | * |
385 | 385 | * @param $standalone: Boolean stating if it should be exported as a stand-alone question |
386 | - * @return A string, the XML flow for an Item. |
|
386 | + * @return string string, the XML flow for an Item. |
|
387 | 387 | * @author Amand Tihon <[email protected]> |
388 | 388 | */ |
389 | 389 | function export($standalone = False) |
@@ -417,7 +417,7 @@ discard block |
||
417 | 417 | * |
418 | 418 | * @param int $exerciseId The exercise to export |
419 | 419 | * @param boolean $standalone Wether it should include XML tag and DTD line. |
420 | - * @return The XML as a string, or an empty string if there's no exercise with given ID. |
|
420 | + * @return string XML as a string, or an empty string if there's no exercise with given ID. |
|
421 | 421 | */ |
422 | 422 | function export_exercise_to_qti($exerciseId, $standalone = true) |
423 | 423 | { |
@@ -474,6 +474,9 @@ discard block |
||
474 | 474 | return htmlspecialchars($text); |
475 | 475 | } |
476 | 476 | |
477 | +/** |
|
478 | + * @param string $text |
|
479 | + */ |
|
477 | 480 | function cleanAttribute($text) |
478 | 481 | { |
479 | 482 | return $text; |
@@ -27,6 +27,7 @@ discard block |
||
27 | 27 | * @param int The JavaScript ID for this question. |
28 | 28 | * Due to the nature of interactions, we must have a natural sequence for |
29 | 29 | * questions in the generated JavaScript. |
30 | + * @param integer $js_id |
|
30 | 31 | */ |
31 | 32 | public static function export_question($questionId, $standalone = true, $js_id) |
32 | 33 | { |
@@ -687,7 +688,7 @@ discard block |
||
687 | 688 | /** |
688 | 689 | * Constructor. |
689 | 690 | * |
690 | - * @param $question The Question object we want to export. |
|
691 | + * @param ScormQuestion $question The Question object we want to export. |
|
691 | 692 | */ |
692 | 693 | public function ScormAssessmentItem($question, $standalone = false) |
693 | 694 | { |
@@ -915,7 +916,7 @@ discard block |
||
915 | 916 | * |
916 | 917 | * @param int $exerciseId The exercise to exporte |
917 | 918 | * @param boolean $standalone Wether it should include XML tag and DTD line. |
918 | - * @return The XML as a string, or an empty string if there's no exercise with given ID. |
|
919 | + * @return string XML as a string, or an empty string if there's no exercise with given ID. |
|
919 | 920 | */ |
920 | 921 | public static function export_exercise_to_scorm($exerciseId, $standalone=true) { |
921 | 922 | $exercise = new Exercise(); |
@@ -930,7 +931,7 @@ discard block |
||
930 | 931 | |
931 | 932 | /** |
932 | 933 | * Constructor. |
933 | - * @param $exe The Exercise instance to export |
|
934 | + * @param Exercise $exe The Exercise instance to export |
|
934 | 935 | * @author Amand Tihon <[email protected]> |
935 | 936 | */ |
936 | 937 | function ScormSection($exe) { |
@@ -1091,7 +1092,7 @@ discard block |
||
1091 | 1092 | * This is a default behaviour, some classes may want to override this. |
1092 | 1093 | * |
1093 | 1094 | * @param $standalone: Boolean stating if it should be exported as a stand-alone question |
1094 | - * @return A string, the XML flow for an Item. |
|
1095 | + * @return string string, the XML flow for an Item. |
|
1095 | 1096 | */ |
1096 | 1097 | function export() |
1097 | 1098 | { |
@@ -415,7 +415,7 @@ discard block |
||
415 | 415 | * @param null $feedback_type |
416 | 416 | * @param null $counter |
417 | 417 | * @param null $score |
418 | - * @return null|string |
|
418 | + * @return string |
|
419 | 419 | */ |
420 | 420 | public function return_header($feedback_type = null, $counter = null, $score = null) |
421 | 421 | { |
@@ -429,15 +429,16 @@ discard block |
||
429 | 429 | } |
430 | 430 | |
431 | 431 | /** |
432 | - * @param $separatorStartRegexp |
|
433 | - * @param $separatorEndRegexp |
|
434 | - * @param $correctItemRegexp |
|
435 | - * @param $questionId |
|
432 | + * @param string $separatorStartRegexp |
|
433 | + * @param string $separatorEndRegexp |
|
434 | + * @param string $correctItemRegexp |
|
435 | + * @param integer $questionId |
|
436 | 436 | * @param $correctItem |
437 | 437 | * @param $attributes |
438 | - * @param $answer |
|
438 | + * @param string $answer |
|
439 | 439 | * @param $listAnswersInfo |
440 | - * @param $displayForStudent |
|
440 | + * @param boolean $displayForStudent |
|
441 | + * @param integer $inBlankNumber |
|
441 | 442 | * @return string |
442 | 443 | */ |
443 | 444 | public static function getFillTheBlankHtml( |
@@ -736,11 +737,11 @@ discard block |
||
736 | 737 | * 0 : student answer is correct |
737 | 738 | * >0 : for fill the blank question with choice menu, is the index of the student answer (right answer indice is 0) |
738 | 739 | * |
739 | - * @param $testId |
|
740 | - * @param $questionId |
|
740 | + * @param integer $testId |
|
741 | + * @param integer $questionId |
|
741 | 742 | * @param $studentsIdList |
742 | - * @param $startDate |
|
743 | - * @param $endDate |
|
743 | + * @param string $startDate |
|
744 | + * @param string $endDate |
|
744 | 745 | * @param bool $useLastAnswerredAttempt |
745 | 746 | * @return array |
746 | 747 | * ( |
@@ -917,7 +918,7 @@ discard block |
||
917 | 918 | * return $text protected for use in regexp |
918 | 919 | * @param string $text |
919 | 920 | * |
920 | - * @return mixed |
|
921 | + * @return string |
|
921 | 922 | */ |
922 | 923 | public static function getRegexpProtected($text) |
923 | 924 | { |
@@ -975,7 +976,7 @@ discard block |
||
975 | 976 | * return the start separator for answer |
976 | 977 | * @param string $number |
977 | 978 | * |
978 | - * @return mixed |
|
979 | + * @return string |
|
979 | 980 | */ |
980 | 981 | public static function getStartSeparator($number) |
981 | 982 | { |
@@ -988,7 +989,7 @@ discard block |
||
988 | 989 | * return the end separator for answer |
989 | 990 | * @param string $number |
990 | 991 | * |
991 | - * @return mixed |
|
992 | + * @return string |
|
992 | 993 | */ |
993 | 994 | public static function getEndSeparator($number) |
994 | 995 | { |
@@ -94,7 +94,7 @@ discard block |
||
94 | 94 | * Sets the comment in the database for a particular path. |
95 | 95 | * @param string File path |
96 | 96 | * @param string Comment to set |
97 | - * @return string Result of the database operation (Database::query will output some message directly on error anyway) |
|
97 | + * @return Doctrine\DBAL\Driver\Statement|null Result of the database operation (Database::query will output some message directly on error anyway) |
|
98 | 98 | */ |
99 | 99 | function SetComment($path, $comment) |
100 | 100 | { |
@@ -112,6 +112,7 @@ discard block |
||
112 | 112 | /** |
113 | 113 | * Reads the file contents into a string. |
114 | 114 | * @param string Urlencoded path |
115 | + * @param string $full_file_path |
|
115 | 116 | * @return string The file contents or false on security error |
116 | 117 | */ |
117 | 118 | function ReadFileCont($full_file_path) |
@@ -257,6 +258,7 @@ discard block |
||
257 | 258 | * Searches for a node in the given array. |
258 | 259 | * @param reference Reference to the array to search |
259 | 260 | * @param string Node we are looking for in the array |
261 | + * @param string $node |
|
260 | 262 | * @return mixed Node name or false if not found |
261 | 263 | */ |
262 | 264 | function myarraysearch(&$array, $node) |
@@ -316,6 +318,8 @@ discard block |
||
316 | 318 | * Fills the folder name up to a certain length with "0". |
317 | 319 | * @param string Original folder name |
318 | 320 | * @param integer Length to reach |
321 | + * @param integer $name |
|
322 | + * @param integer $nsize |
|
319 | 323 | * @return string Modified folder name |
320 | 324 | */ |
321 | 325 | function FillFolderName($name, $nsize) |
@@ -18,6 +18,7 @@ |
||
18 | 18 | * Gets the results of all students (or just one student if access is limited) |
19 | 19 | * @param string The document path (for HotPotatoes retrieval) |
20 | 20 | * @param integer User ID. Optional. If no user ID is provided, we take all the results. Defauts to null |
21 | + * @param string $document_path |
|
21 | 22 | */ |
22 | 23 | public function getExercisesReporting($document_path, $hotpotato_name) |
23 | 24 | { |
@@ -52,7 +52,7 @@ |
||
52 | 52 | /** |
53 | 53 | * @param FormValidator $form |
54 | 54 | * @param null $objExercise |
55 | - * @return bool |
|
55 | + * @return null|false |
|
56 | 56 | */ |
57 | 57 | public function processCreation($form, $objExercise = null) |
58 | 58 | { |
@@ -58,7 +58,7 @@ |
||
58 | 58 | * @param null $feedback_type |
59 | 59 | * @param null $counter |
60 | 60 | * @param null $score |
61 | - * @return null|string |
|
61 | + * @return string |
|
62 | 62 | */ |
63 | 63 | function return_header($feedback_type = null, $counter = null, $score = null) |
64 | 64 | { |
@@ -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, |