Passed
Push — 1.10.x ( 91150c...5d85e2 )
by Angel Fernando Quiroz
212:03 queued 167:16
created
main/survey/survey.lib.php 1 patch
Doc Comments   +17 added lines, -15 removed lines patch added patch discarded remove patch
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
      * Deletes all survey invitations of a user
48 48
      * @param int $user_id
49 49
      *
50
-     * @return boolean
50
+     * @return false|null
51 51
      * @assert ('') === false
52 52
      */
53 53
     public static function delete_all_survey_invitations_by_user($user_id)
@@ -104,7 +104,7 @@  discard block
 block discarded – undo
104 104
      * Retrieves all the survey information
105 105
      *
106 106
      * @param integer $survey_id the id of the survey
107
-     * @param boolean $shared this parameter determines if
107
+     * @param integer $shared this parameter determines if
108 108
      * we have to get the information of a survey from the central (shared) database or from the
109 109
      * 		  course database
110 110
      * @param string course code optional
@@ -571,7 +571,7 @@  discard block
 block discarded – undo
571 571
      * This function deletes a survey (and also all the question in that survey
572 572
      *
573 573
      * @param int $survey_id id of the survey that has to be deleted
574
-     * @return true
574
+     * @return boolean
575 575
      *
576 576
      * @author Patrick Cool <[email protected]>, Ghent University
577 577
      * @version January 2007
@@ -756,7 +756,7 @@  discard block
 block discarded – undo
756 756
      *
757 757
      * @param int $survey_id id of the survey that has to be duplicated
758 758
      * @param int $courseId id of the course which survey has to be duplicated
759
-     * @return true
759
+     * @return boolean
760 760
      *
761 761
      * @author Eric Marguin <[email protected]>, Elixir Interactive
762 762
      * @version October 2007
@@ -796,7 +796,7 @@  discard block
 block discarded – undo
796 796
     /**
797 797
      * This function recalculates the number of people who have taken the survey (=filled at least one question)
798 798
      *
799
-     * @param int $survey_id the id of the survey somebody
799
+     * @param int $survey_data the id of the survey somebody
800 800
      * @return true
801 801
      *
802 802
      * @author Patrick Cool <[email protected]>, Ghent University
@@ -1367,7 +1367,7 @@  discard block
 block discarded – undo
1367 1367
      * This function deletes a shared survey question from the main database and all its options
1368 1368
      *
1369 1369
      * @param int $question_id the id of the question
1370
-     * @param int $shared
1370
+     * @param integer $survey_id
1371 1371
      *
1372 1372
      * @todo delete all the options of this question
1373 1373
      *
@@ -1494,8 +1494,8 @@  discard block
 block discarded – undo
1494 1494
      * This function deletes all the options of the questions of a given survey
1495 1495
      * This function is normally only called when a survey is deleted
1496 1496
      *
1497
-     * @param $survey_id the id of the survey that has to be deleted
1498
-     * @return true
1497
+     * @param integer $survey_id the id of the survey that has to be deleted
1498
+     * @return boolean
1499 1499
      *
1500 1500
      * @author Patrick Cool <[email protected]>, Ghent University
1501 1501
      * @version January 2007
@@ -1561,8 +1561,8 @@  discard block
 block discarded – undo
1561 1561
      * This function deletes all the answers anyone has given on this survey
1562 1562
      * This function is normally only called when a survey is deleted
1563 1563
      *
1564
-     * @param $survey_id the id of the survey that has to be deleted
1565
-     * @return true
1564
+     * @param integer $survey_id the id of the survey that has to be deleted
1565
+     * @return boolean
1566 1566
      *
1567 1567
      * @todo write the function
1568 1568
      *
@@ -1804,7 +1804,7 @@  discard block
 block discarded – undo
1804 1804
     /**
1805 1805
      * This function checks the parameters that are used in this page
1806 1806
      *
1807
-     * @return 	string 	The header, an error and the footer if any parameter fails, else it returns true
1807
+     * @return 	null|boolean 	The header, an error and the footer if any parameter fails, else it returns true
1808 1808
      * @author Patrick Cool <[email protected]>, Ghent University
1809 1809
      * @version February 2007
1810 1810
      */
@@ -2371,6 +2371,7 @@  discard block
 block discarded – undo
2371 2371
      * Display score data about a survey question
2372 2372
      * @param	array	Question info
2373 2373
      * @param	integer	The offset of results shown
2374
+     * @param integer $offset
2374 2375
      * @return	void 	(direct output)
2375 2376
      */
2376 2377
     public static function display_question_report_score($survey_data, $question, $offset)
@@ -3208,7 +3209,7 @@  discard block
 block discarded – undo
3208 3209
      * @param	array	User's answers
3209 3210
      * @param 	mixed	User ID or user details as string - Used as a string in the result string
3210 3211
      * @param	boolean	Whether to display user fields or not
3211
-     * @return	string	One line of the csv file
3212
+     * @return	string[]	One line of the csv file
3212 3213
      */
3213 3214
     public static function export_complete_report_row_xls(
3214 3215
         $survey_data,
@@ -3801,7 +3802,7 @@  discard block
 block discarded – undo
3801 3802
 
3802 3803
     /**
3803 3804
      * @param $params
3804
-     * @return bool|int
3805
+     * @return false|string
3805 3806
      */
3806 3807
     public static function save_invitation($params)
3807 3808
     {
@@ -3855,6 +3856,7 @@  discard block
 block discarded – undo
3855 3856
      *
3856 3857
      * @param int invitedUser - the userId (course user) or emailaddress of additional user
3857 3858
      * $param string $invitation_code - the unique invitation code for the URL
3859
+     * @param string|null $invitation_text
3858 3860
      * @return	void
3859 3861
      */
3860 3862
     public static function send_invitation_mail($invitedUser, $invitation_code, $invitation_title, $invitation_text)
@@ -4783,7 +4785,7 @@  discard block
 block discarded – undo
4783 4785
     /**
4784 4786
      * @author Isaac Flores Paz <[email protected]>
4785 4787
      * @param int $user_id - User ID
4786
-     * @param int $user_id_answer - User in survey answer table (user id or anonymus)
4788
+     * @param int $user_answer - User in survey answer table (user id or anonymus)
4787 4789
      * @return boolean
4788 4790
      */
4789 4791
     static function show_link_available($user_id, $survey_code, $user_answer)
@@ -4829,7 +4831,7 @@  discard block
 block discarded – undo
4829 4831
      * Display survey question chart
4830 4832
      * @param	array	Chart data
4831 4833
      * @param	boolean	Tells if the chart has a serie. False by default
4832
-     * @return	void 	(direct output)
4834
+     * @return	string 	(direct output)
4833 4835
      */
4834 4836
     public static function drawChart($chartData, $hasSerie = false, $chartContainerId = 'chartContainer')
4835 4837
     {
Please login to merge, or discard this patch.