Passed
Push — 1.10.x ( 10729d...9f4f4a )
by Angel Fernando Quiroz
142:16 queued 95:50
created
main/gradebook/lib/results_data_generator.class.php 1 patch
Doc Comments   +1 added lines patch added patch discarded remove patch
@@ -132,6 +132,7 @@
 block discarded – undo
132 132
      * @param float Current absolute score (max score is taken from $this->evaluation->get_max()
133 133
      * @param bool  Whether we want the real score (2/4 (50 %)) or the transformation (A, B, C, etc)
134 134
      * @param bool  Whether we want to ignore the score color
135
+     * @param boolean $realscore
135 136
      * @result string The score as we want to show it
136 137
      */
137 138
     private function get_score_display ($score, $realscore, $ignore_score_color = false)
Please login to merge, or discard this patch.
main/dropbox/dropbox_functions.inc.php 1 patch
Doc Comments   +8 added lines, -7 removed lines patch added patch discarded remove patch
@@ -22,7 +22,7 @@  discard block
 block discarded – undo
22 22
 
23 23
 /**
24 24
 * This function is a wrapper function for the multiple actions feature.
25
-* @return   Mixed   If there is a problem, return a string message, otherwise nothing
25
+* @return   string|null   If there is a problem, return a string message, otherwise nothing
26 26
 * @author   Patrick Cool <[email protected]>, Ghent University
27 27
 * @version  march 2006
28 28
 */
@@ -209,9 +209,9 @@  discard block
 block discarded – undo
209 209
 *
210 210
 * @param $id the id of the file we are moving
211 211
 * @param $target the id of the folder we are moving to
212
-* @param $part are we moving a received file or a sent file?
212
+* @param string $part are we moving a received file or a sent file?
213 213
 *
214
-* @return language string
214
+* @return string string
215 215
 *
216 216
 * @author Patrick Cool <[email protected]>, Ghent University
217 217
 * @version march 2006
@@ -290,7 +290,7 @@  discard block
 block discarded – undo
290 290
 * @param $id the unique id of the file
291 291
 * @param $part are we dealing with a sent or with a received file?
292 292
 *
293
-* @return html code
293
+* @return string code
294 294
 *
295 295
 * @author Patrick Cool <[email protected]>, Ghent University
296 296
 * @version march 2006
@@ -786,6 +786,7 @@  discard block
 block discarded – undo
786 786
 }
787 787
 
788 788
 /**
789
+* @param string $user_id
789 790
 * @return boolean indicating if user with user_id=$user_id is a course member
790 791
 * @todo check if this function is still necessary. There might be a library function for this.
791 792
 */
@@ -1147,7 +1148,7 @@  discard block
 block discarded – undo
1147 1148
 /**
1148 1149
 * This function returns the html code to display the feedback messages on a given dropbox file
1149 1150
 * @param $feedback_array an array that contains all the feedback messages about the given document.
1150
-* @return html code
1151
+* @return string code
1151 1152
 * @todo add the form for adding new comment (if the other party has not deleted it yet).
1152 1153
 *
1153 1154
 * @author Patrick Cool <[email protected]>, Ghent University
@@ -1164,7 +1165,7 @@  discard block
 block discarded – undo
1164 1165
 
1165 1166
 /**
1166 1167
 * this function returns the code for the form for adding a new feedback message to a dropbox file.
1167
-* @return html code
1168
+* @return string code
1168 1169
 *
1169 1170
 * @author Patrick Cool <[email protected]>, Ghent University
1170 1171
 * @version march 2006
@@ -1225,7 +1226,7 @@  discard block
 block discarded – undo
1225 1226
 }
1226 1227
 
1227 1228
 /**
1228
-* @return a language string (depending on the success or failure.
1229
+* @return string language string (depending on the success or failure.
1229 1230
 *
1230 1231
 * @author Patrick Cool <[email protected]>, Ghent University
1231 1232
 * @version march 2006
Please login to merge, or discard this patch.
main/coursecopy/classes/CourseRestorer.class.php 1 patch
Doc Comments   +8 added lines, -2 removed lines patch added patch discarded remove patch
@@ -84,7 +84,7 @@  discard block
 block discarded – undo
84 84
 
85 85
     /**
86 86
      * CourseRestorer constructor.
87
-     * @param array $course
87
+     * @param Course $course
88 88
      */
89 89
     public function __construct($course)
90 90
     {
@@ -133,7 +133,7 @@  discard block
 block discarded – undo
133 133
 	 * @param int	    $session_id
134 134
 	 * @param bool	    $update_course_settings Course settings are going to be restore?
135 135
      * @param bool      $respect_base_content
136
-     * @return bool
136
+     * @return false|null
137 137
 	 */
138 138
     public function restore(
139 139
         $destination_course_code = '',
@@ -1154,6 +1154,7 @@  discard block
 block discarded – undo
1154 1154
 
1155 1155
 	/**
1156 1156
 	 * Restore a forum-topic
1157
+	 * @param false|string $forum_id
1157 1158
 	 */
1158 1159
     public function restore_topic($thread_id, $forum_id, $sessionId = 0)
1159 1160
     {
@@ -1210,6 +1211,7 @@  discard block
 block discarded – undo
1210 1211
 	/**
1211 1212
 	 * Restore a forum-post
1212 1213
 	 * @TODO Restore tree-structure of posts. For example: attachments to posts.
1214
+	 * @param false|string $topic_id
1213 1215
 	 */
1214 1216
     public function restore_post($id, $topic_id, $forum_id, $sessionId = 0)
1215 1217
     {
@@ -2307,6 +2309,7 @@  discard block
 block discarded – undo
2307 2309
 
2308 2310
 	/**
2309 2311
 	 * Check availability of a survey code
2312
+	 * @param string $survey_code
2310 2313
 	 */
2311 2314
     public function is_survey_code_available($survey_code)
2312 2315
     {
@@ -2321,6 +2324,7 @@  discard block
 block discarded – undo
2321 2324
 
2322 2325
 	/**
2323 2326
 	 * Restore survey-questions
2327
+	 * @param string $survey_id
2324 2328
 	 */
2325 2329
     public function restore_survey_question($id, $survey_id)
2326 2330
     {
@@ -2777,6 +2781,8 @@  discard block
 block discarded – undo
2777 2781
     * @param string The path origin
2778 2782
     * @param string The path destination
2779 2783
     * @param boolean Option Overwrite
2784
+    * @param string $source
2785
+    * @param string $dest
2780 2786
     * @return void()
2781 2787
     * @deprecated
2782 2788
     */
Please login to merge, or discard this patch.
main/admin/user_list.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -288,7 +288,7 @@
 block discarded – undo
288 288
  * This function defines globals.
289 289
  * @param  int $userId
290 290
  *
291
- * @return bool    False on failure, redirection on success
291
+ * @return false|null    False on failure, redirection on success
292 292
  * @author Evie Embrechts
293 293
  * @author Yannick Warnier <[email protected]>
294 294
 */
Please login to merge, or discard this patch.
main/inc/lib/career.lib.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -84,7 +84,7 @@
 block discarded – undo
84 84
 
85 85
     /**
86 86
      * Available driver list.
87
-     * @return array
87
+     * @return string[]
88 88
      */
89 89
     private function getDefaultDriverList()
90 90
     {
Please login to merge, or discard this patch.
main/auth/shibboleth/app/view/shibboleth_display.class.php 1 patch
Doc Comments   +6 added lines patch added patch discarded remove patch
@@ -27,6 +27,9 @@  discard block
 block discarded – undo
27 27
         return $result;
28 28
     }
29 29
 
30
+    /**
31
+     * @param string $message
32
+     */
30 33
     public function error_page($message)
31 34
     {
32 35
         $page_title = get_lang('ShibbolethLogin');
@@ -37,6 +40,9 @@  discard block
 block discarded – undo
37 40
         die;
38 41
     }
39 42
     
43
+    /**
44
+     * @param string $message
45
+     */
40 46
     public function message_page($message, $title = '')
41 47
     {
42 48
         $title = $title ? $title : get_lang('ShibbolethLogin');
Please login to merge, or discard this patch.
main/inc/lib/message.lib.php 1 patch
Doc Comments   +7 added lines, -5 removed lines patch added patch discarded remove patch
@@ -639,6 +639,8 @@  discard block
 block discarded – undo
639 639
      * @param  int	message id
640 640
      * @param  int	message user id (receiver user id or sender user id)
641 641
      * @param  int	group id (optional)
642
+     * @param integer $message_id
643
+     * @param integer $message_uid
642 644
      * @return void
643 645
      */
644 646
     public static function delete_message_attachment_file($message_id, $message_uid, $group_id = 0)
@@ -683,7 +685,7 @@  discard block
 block discarded – undo
683 685
      * update messages by user id and message id
684 686
      * @param  int		$user_id
685 687
      * @param  int		$message_id
686
-     * @return resource
688
+     * @return false|null
687 689
      */
688 690
     public static function update_message($user_id, $message_id)
689 691
     {
@@ -703,7 +705,7 @@  discard block
 block discarded – undo
703 705
      * @param int $user_id
704 706
      * @param int $message_id
705 707
      * @param string $type
706
-     * @return bool
708
+     * @return false|null
707 709
      */
708 710
     public static function update_message_status($user_id, $message_id,$type)
709 711
     {
@@ -740,6 +742,7 @@  discard block
 block discarded – undo
740 742
     /**
741 743
      * get messages by group id
742 744
      * @param  int		group id
745
+     * @param integer $group_id
743 746
      * @return array
744 747
      */
745 748
     public static function get_messages_by_group($group_id)
@@ -1477,7 +1480,7 @@  discard block
 block discarded – undo
1477 1480
      * Sort date by desc from a multi-dimensional array
1478 1481
      * @param array $array1  first array to compare
1479 1482
      * @param array $array2  second array to compare
1480
-     * @return bool
1483
+     * @return integer
1481 1484
      */
1482 1485
     public function order_desc_date($array1, $array2)
1483 1486
     {
@@ -1555,7 +1558,6 @@  discard block
 block discarded – undo
1555 1558
     /**
1556 1559
      * @param $id
1557 1560
      * @param array $params
1558
-     * @param string $display
1559 1561
      * @return string
1560 1562
      */
1561 1563
     public static function generate_invitation_form($id, $params = array())
@@ -1644,7 +1646,7 @@  discard block
 block discarded – undo
1644 1646
 
1645 1647
     /**
1646 1648
      * @param string $keyword
1647
-     * @return null|string
1649
+     * @return string
1648 1650
      */
1649 1651
     public static function outbox_display($keyword = '')
1650 1652
     {
Please login to merge, or discard this patch.
main/inc/lib/exercise_show_functions.lib.php 1 patch
Doc Comments   +37 added lines patch added patch discarded remove patch
@@ -24,6 +24,10 @@  discard block
 block discarded – undo
24 24
 	 * @param int       Question ID
25 25
      * @param int $resultsDisabled
26 26
      * @param string $originalStudentAnswer
27
+     * @param integer $feedbackType
28
+     * @param string $answer
29
+     * @param integer $id
30
+     * @param integer $questionId
27 31
 	 * @return void
28 32
 	 */
29 33
 	public static function display_fill_in_blanks_answer($feedbackType, $answer, $id, $questionId, $resultsDisabled, $originalStudentAnswer = '')
@@ -62,6 +66,10 @@  discard block
 block discarded – undo
62 66
      * @param string    Answer text
63 67
      * @param int       Exercise ID
64 68
      * @param int       Question ID
69
+     * @param integer $feedback_type
70
+     * @param string $answer
71
+     * @param integer $id
72
+     * @param integer $questionId
65 73
      * @return void
66 74
      */
67 75
     static function display_calculated_answer($feedback_type, $answer, $id, $questionId)
@@ -95,6 +103,9 @@  discard block
 block discarded – undo
95 103
 	 * @param string    Answer text
96 104
 	 * @param int       Exercise ID
97 105
 	 * @param int       Question ID
106
+	 * @param integer $feedback_type
107
+	 * @param integer $exe_id
108
+	 * @param integer $questionId
98 109
 	 * @return void
99 110
 	 */
100 111
 	static function display_free_answer($feedback_type, $answer, $exe_id, $questionId, $questionScore = null)
@@ -117,6 +128,12 @@  discard block
 block discarded – undo
117 128
         }
118 129
 	}
119 130
 
131
+	/**
132
+	 * @param integer $feedback_type
133
+	 * @param integer $id
134
+	 * @param integer $questionId
135
+	 * @param Nanogong $nano
136
+	 */
120 137
 	static function display_oral_expression_answer($feedback_type, $answer, $id, $questionId, $nano = null)
121 138
     {
122 139
         if (isset($nano)) {
@@ -234,6 +251,12 @@  discard block
 block discarded – undo
234 251
 	 * @param integer Exercise ID
235 252
 	 * @param integer Question ID
236 253
 	 * @param boolean Whether to show the answer comment or not
254
+	 * @param integer $answerType
255
+	 * @param string $answer
256
+	 * @param string $answerComment
257
+	 * @param integer $answerCorrect
258
+	 * @param integer $id
259
+	 * @param integer $questionId
237 260
 	 * @return void
238 261
 	 */
239 262
 	static function display_unique_or_multiple_answer(
@@ -318,6 +341,13 @@  discard block
 block discarded – undo
318 341
      * @param integer Exercise ID
319 342
      * @param integer Question ID
320 343
      * @param boolean Whether to show the answer comment or not
344
+     * @param integer $feedback_type
345
+     * @param integer $answerType
346
+     * @param string $answer
347
+     * @param string $answerComment
348
+     * @param integer $answerCorrect
349
+     * @param integer $id
350
+     * @param integer $questionId
321 351
      * @return void
322 352
      */
323 353
     static function display_multiple_answer_true_false(
@@ -408,6 +438,13 @@  discard block
 block discarded – undo
408 438
      * @param integer Exercise ID
409 439
      * @param integer Question ID
410 440
      * @param boolean Whether to show the answer comment or not
441
+     * @param integer $feedback_type
442
+     * @param integer $answerType
443
+     * @param string $answer
444
+     * @param string $answerComment
445
+     * @param integer $answerCorrect
446
+     * @param integer $id
447
+     * @param integer $questionId
411 448
      * @return void
412 449
      */
413 450
     static function display_multiple_answer_combination_true_false(
Please login to merge, or discard this patch.
main/inc/lib/agenda.lib.php 1 patch
Doc Comments   +8 added lines, -3 removed lines patch added patch discarded remove patch
@@ -565,7 +565,7 @@  discard block
 block discarded – undo
565 565
      * @param string $color
566 566
      * @param bool $addAnnouncement
567 567
      *
568
-     * @return bool
568
+     * @return null|false
569 569
      */
570 570
     public function editEvent(
571 571
         $id,
@@ -1308,6 +1308,7 @@  discard block
 block discarded – undo
1308 1308
      *
1309 1309
      * @param int $eventId
1310 1310
      * @param int $courseId
1311
+     * @param integer $sessionId
1311 1312
      * @paraù int $sessionId
1312 1313
      *
1313 1314
      * @return array
@@ -2469,6 +2470,7 @@  discard block
 block discarded – undo
2469 2470
      * Adds x weeks to a UNIX timestamp
2470 2471
      * @param   int     The timestamp
2471 2472
      * @param   int     The number of weeks to add
2473
+     * @param integer $timestamp
2472 2474
      * @return  int     The new timestamp
2473 2475
      */
2474 2476
     function addWeek($timestamp, $num = 1)
@@ -2480,6 +2482,7 @@  discard block
 block discarded – undo
2480 2482
      * Adds x months to a UNIX timestamp
2481 2483
      * @param   int     The timestamp
2482 2484
      * @param   int     The number of years to add
2485
+     * @param integer $timestamp
2483 2486
      * @return  int     The new timestamp
2484 2487
      */
2485 2488
     function addMonth($timestamp, $num = 1)
@@ -2498,6 +2501,7 @@  discard block
 block discarded – undo
2498 2501
      * Adds x years to a UNIX timestamp
2499 2502
      * @param   int     The timestamp
2500 2503
      * @param   int     The number of years to add
2504
+     * @param integer $timestamp
2501 2505
      * @return  int     The new timestamp
2502 2506
      */
2503 2507
     function addYear($timestamp, $num = 1)
@@ -2697,7 +2701,7 @@  discard block
 block discarded – undo
2697 2701
     /**
2698 2702
      * @param array $courseInfo
2699 2703
      * @param $file
2700
-     * @return array|bool|string
2704
+     * @return false|string
2701 2705
      */
2702 2706
     public function importEventFile($courseInfo, $file)
2703 2707
     {
@@ -2810,7 +2814,7 @@  discard block
 block discarded – undo
2810 2814
 
2811 2815
     /**
2812 2816
      * Parse filter turns USER:12 to ['users' => [12])] or G:1 ['groups' => [1]]
2813
-     * @param $filter
2817
+     * @param integer $filter
2814 2818
      * @return array
2815 2819
      */
2816 2820
     public function parseAgendaFilter($filter)
@@ -3324,6 +3328,7 @@  discard block
 block discarded – undo
3324 3328
      * @param	int		user ID of the user
3325 3329
      * @param	string	Optional start date in datetime format (if no start date is given, uses today)
3326 3330
      * @param	string	Optional end date in datetime format (if no date is given, uses one year from now)
3331
+     * @param integer $user_id
3327 3332
      * @return	array	Array of events ordered by start date, in
3328 3333
      * [0]('datestart','dateend','title'),[1]('datestart','dateend','title','link','coursetitle') format,
3329 3334
      * where datestart and dateend are in yyyyMMddhhmmss format.
Please login to merge, or discard this patch.