Completed
Pull Request — 1.11.x (#1431)
by José
56:30 queued 20:26
created
main/inc/lib/AnnouncementManager.php 1 patch
Doc Comments   +10 added lines, -6 removed lines patch added patch discarded remove patch
@@ -18,7 +18,7 @@  discard block
 block discarded – undo
18 18
     }
19 19
 
20 20
     /**
21
-     * @return array
21
+     * @return string[]
22 22
      */
23 23
     public static function get_tags()
24 24
     {
@@ -40,7 +40,7 @@  discard block
 block discarded – undo
40 40
      * @param string    $course_code
41 41
      * @param int       $session_id
42 42
      *
43
-     * @return mixed
43
+     * @return string
44 44
      */
45 45
     public static function parse_content($userId, $content, $course_code, $session_id = 0)
46 46
     {
@@ -183,7 +183,7 @@  discard block
 block discarded – undo
183 183
      * @param int $courseId
184 184
      * @param int $userId
185 185
      *
186
-     * @return array
186
+     * @return integer
187 187
      */
188 188
     public static function getAnnouncementInfoById($announcementId, $courseId, $userId)
189 189
     {
@@ -384,7 +384,7 @@  discard block
 block discarded – undo
384 384
      * @param array	    uploaded file $_FILES
385 385
      * @param string    Comment describing the attachment
386 386
      * @param bool  $sendToUsersInSession
387
-     * @return int      false on failure, ID of the announcement on success
387
+     * @return false|string      false on failure, ID of the announcement on success
388 388
      */
389 389
     public static function add_announcement(
390 390
         $emailTitle,
@@ -495,7 +495,7 @@  discard block
 block discarded – undo
495 495
      * @param string $file_comment
496 496
      * @param bool $sendToUsersInSession
497 497
      *
498
-     * @return bool|int
498
+     * @return false|string
499 499
      */
500 500
     public static function add_group_announcement(
501 501
         $emailTitle,
@@ -739,7 +739,7 @@  discard block
 block discarded – undo
739 739
 
740 740
     /**
741 741
      * @param int $insert_id
742
-     * @return bool
742
+     * @return false|null
743 743
      */
744 744
     public static function update_mail_sent($insert_id)
745 745
     {
@@ -759,6 +759,7 @@  discard block
 block discarded – undo
759 759
      * Gets all announcements from a user by course
760 760
      * @param	string course db
761 761
      * @param	int user id
762
+     * @param integer $user_id
762 763
      * @return	array html with the content and count of announcements or false otherwise
763 764
      */
764 765
     public static function get_all_annoucement_by_user_course($course_code, $user_id)
@@ -1224,6 +1225,7 @@  discard block
 block discarded – undo
1224 1225
      * has been sent to
1225 1226
      * @param    string  The tool (announcement, agenda, ...)
1226 1227
      * @param    int     ID of the element of the corresponding type
1228
+     * @param string $tool
1227 1229
      * @return   array   Array of users and groups to whom the element has been sent
1228 1230
      */
1229 1231
     public static function sent_to($tool, $id)
@@ -1348,6 +1350,8 @@  discard block
 block discarded – undo
1348 1350
      * @param int attach id
1349 1351
      * @param array uploaded file $_FILES
1350 1352
      * @param string file comment
1353
+     * @param integer $id_attach
1354
+     * @param string $file_comment
1351 1355
      * @return int
1352 1356
      */
1353 1357
     public static function edit_announcement_attachment_file($id_attach, $file, $file_comment)
Please login to merge, or discard this patch.
main/inc/lib/webservices/Rest.php 1 patch
Doc Comments   -1 removed lines patch added patch discarded remove patch
@@ -318,7 +318,6 @@
 block discarded – undo
318 318
     }
319 319
 
320 320
     /**
321
-     * @param int $courseId
322 321
      * @return array
323 322
      * @throws Exception
324 323
      */
Please login to merge, or discard this patch.
src/Chamilo/CoreBundle/Composer/ScriptHandler.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -45,7 +45,7 @@  discard block
 block discarded – undo
45 45
     }
46 46
 
47 47
     /**
48
-     * @return array
48
+     * @return string[]
49 49
      */
50 50
     public static function getFoldersToDelete()
51 51
     {
@@ -95,7 +95,7 @@  discard block
 block discarded – undo
95 95
     }
96 96
 
97 97
     /**
98
-     * @return array
98
+     * @return string[]
99 99
      */
100 100
     public static function getFilesToDelete()
101 101
     {
Please login to merge, or discard this patch.
plugin/ims_lti/ImsLtiPlugin.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -25,7 +25,7 @@
 block discarded – undo
25 25
     /**
26 26
      * Get the class instance
27 27
      * @staticvar MsiLtiPlugin $result
28
-     * @return MsiLtiPlugin
28
+     * @return ImsLtiPlugin
29 29
      */
30 30
     public static function create()
31 31
     {
Please login to merge, or discard this patch.
plugin/ims_lti/ImsLtiTool.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -20,6 +20,9 @@
 block discarded – undo
20 20
         return $this->id;
21 21
     }
22 22
 
23
+    /**
24
+     * @return string
25
+     */
23 26
     public function getName()
24 27
     {
25 28
         return $this->name;
Please login to merge, or discard this patch.
plugin/ims_lti/OAuthSimple.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -334,6 +334,9 @@
 block discarded – undo
334 334
         return preg_replace('/, $/','',$result);
335 335
     }
336 336
 
337
+    /**
338
+     * @param string $paramString
339
+     */
337 340
     private function _parseParameterString ($paramString)
338 341
     {
339 342
         $elements = explode('&',$paramString);
Please login to merge, or discard this patch.
main/inc/lib/fileUpload.lib.php 1 patch
Doc Comments   +7 added lines, -6 removed lines patch added patch discarded remove patch
@@ -53,7 +53,7 @@  discard block
 block discarded – undo
53 53
  * Returns the name without extension, used for the title
54 54
  *
55 55
  * @param string $name
56
- * @return name without the extension
56
+ * @return string without the extension
57 57
  */
58 58
 function get_document_title($name)
59 59
 {
@@ -67,7 +67,7 @@  discard block
 block discarded – undo
67 67
  * This function checks if the upload succeeded
68 68
  *
69 69
  * @param array $uploaded_file ($_FILES)
70
- * @return true if upload succeeded
70
+ * @return boolean if upload succeeded
71 71
  */
72 72
 function process_uploaded_file($uploaded_file, $show_output = true)
73 73
 {
@@ -1025,7 +1025,7 @@  discard block
 block discarded – undo
1025 1025
  * @param array  $courseInfo
1026 1026
  * @param array  $userInfo
1027 1027
  * @param array  $uploaded_file - follows the $_FILES Structure
1028
- * @param string $upload_path   - destination of the upload.
1028
+ * @param string $uploadPath   - destination of the upload.
1029 1029
  *                               This path is to append to $base_work_dir
1030 1030
  * @param string $base_work_dir  - base working directory of the module
1031 1031
  * @param int    $maxFilledSpace  - amount of bytes to not exceed in the base
@@ -1212,7 +1212,7 @@  discard block
 block discarded – undo
1212 1212
  * @param int $session_id Session ID, if any
1213 1213
  * @param int $userId creator id
1214 1214
  *
1215
- * @return int id if inserted document
1215
+ * @return string|false id if inserted document
1216 1216
  */
1217 1217
 function add_document(
1218 1218
     $_course,
@@ -1346,7 +1346,7 @@  discard block
 block discarded – undo
1346 1346
  *
1347 1347
  * @author	Olivier Cauberghe <[email protected]>
1348 1348
  * @param	path+filename eg: /main/document/document.php
1349
- * @return	The directory depth
1349
+ * @return	integer directory depth
1350 1350
  */
1351 1351
 function get_levels($filename) {
1352 1352
     $levels = explode('/', $filename);
@@ -1363,6 +1363,8 @@  discard block
 block discarded – undo
1363 1363
  * @author	Olivier Cauberghe <[email protected]>
1364 1364
  * @param	path,filename
1365 1365
  * action:	Adds an entry to the document table with the default settings.
1366
+ * @param string $upload_path
1367
+ * @param string $filename
1366 1368
  */
1367 1369
 function set_default_settings($upload_path, $filename, $filetype = 'file')
1368 1370
 {
@@ -1818,7 +1820,6 @@  discard block
 block discarded – undo
1818 1820
  * @param int $groupId
1819 1821
  * @param bool $output
1820 1822
  * @param array $parent
1821
- * @param string $uploadPath
1822 1823
  *
1823 1824
  */
1824 1825
 function add_all_documents_in_folder_to_database(
Please login to merge, or discard this patch.
main/inc/lib/groupmanager.lib.php 1 patch
Doc Comments   +12 added lines, -9 removed lines patch added patch discarded remove patch
@@ -371,7 +371,7 @@  discard block
 block discarded – undo
371 371
      * @author Bart Mollet
372 372
      * @param  int  $groupId iid
373 373
      * @param string $course_code Default is current course
374
-     * @return integer              - number of groups deleted.
374
+     * @return boolean              - number of groups deleted.
375 375
      */
376 376
     public static function delete_groups($groupId, $course_code = null)
377 377
     {
@@ -563,7 +563,7 @@  discard block
 block discarded – undo
563 563
      * @param bool      Whether self registration is allowed or not
564 564
      * @param bool      Whether self unregistration is allowed or not
565 565
      * @param int       $categoryId
566
-     * @return bool     TRUE if properties are successfully changed, false otherwise
566
+     * @return Statement|null     TRUE if properties are successfully changed, false otherwise
567 567
      */
568 568
     public static function set_group_properties(
569 569
         $group_id,
@@ -716,7 +716,7 @@  discard block
 block discarded – undo
716 716
      * @param int $group_id The iid of the group
717 717
      * @param string $course_code The course in which the group is (default =
718 718
      * current course)
719
-     * @return array The category
719
+     * @return integer The category
720 720
      */
721 721
     public static function get_category_from_group($group_id, $course_code = null)
722 722
     {
@@ -782,9 +782,9 @@  discard block
 block discarded – undo
782 782
      * Create group category
783 783
      * @param string $title The title of the new category
784 784
      * @param string $description The description of the new category
785
-     * @param bool $self_registration_allowed
786
-     * @param bool $self_unregistration_allowed
787
-     * @param int $max_number_of_students
785
+     * @param integer $self_registration_allowed
786
+     * @param integer $self_unregistration_allowed
787
+     * @param int $maximum_number_of_students
788 788
      * @param int $groups_per_user
789 789
      */
790 790
     public static function create_category(
@@ -990,6 +990,8 @@  discard block
 block discarded – undo
990 990
      * @param int $limit
991 991
      * @param bool $getCount
992 992
      * @param int $courseId
993
+     * @param string $column
994
+     * @param string $direction
993 995
      * @return array list of user id
994 996
      */
995 997
     public static function get_users(
@@ -1583,7 +1585,7 @@  discard block
 block discarded – undo
1583 1585
     /**
1584 1586
      * Subscribe tutor(s) to a specified group in current course
1585 1587
      * @param mixed $user_ids Can be an array with user-id's or a single user-id
1586
-     * @param int $group_id iid
1588
+     * @param int $groupId iid
1587 1589
      * @param int $course_id
1588 1590
      *
1589 1591
      * @author Patrick Cool <[email protected]>, Ghent University
@@ -1614,7 +1616,7 @@  discard block
 block discarded – undo
1614 1616
      * Unsubscribe user(s) from a specified group in current course
1615 1617
      * @param mixed $user_ids Can be an array with user-id's or a single user-id
1616 1618
      * @param int $group_id iid
1617
-     * @return bool TRUE if successful
1619
+     * @return boolean|null TRUE if successful
1618 1620
      */
1619 1621
     public static function unsubscribe_users($user_ids, $group_id)
1620 1622
     {
@@ -1633,7 +1635,7 @@  discard block
 block discarded – undo
1633 1635
 
1634 1636
     /**
1635 1637
      * Unsubscribe all users from one or more groups
1636
-     * @param int $group_id iid
1638
+     * @param integer $groupId iid
1637 1639
      * @return bool TRUE if successful
1638 1640
      */
1639 1641
     public static function unsubscribe_all_users($groupId)
@@ -1781,6 +1783,7 @@  discard block
 block discarded – undo
1781 1783
      *
1782 1784
      * @param $user_array_in list of users (must be sorted).
1783 1785
      * @param string $compare_field, the field to be compared
1786
+     * @param string $compare_field
1784 1787
      */
1785 1788
     public static function filter_duplicates($user_array_in, $compare_field)
1786 1789
     {
Please login to merge, or discard this patch.
main/webservices/cm_webservice.php 1 patch
Doc Comments   +5 added lines, -1 removed lines patch added patch discarded remove patch
@@ -37,6 +37,8 @@  discard block
 block discarded – undo
37 37
      *
38 38
      * @param int Error code
39 39
      * @param string Error message
40
+     * @param integer $code
41
+     * @param string $message
40 42
      */
41 43
     public function __construct($code, $message)
42 44
     {
@@ -48,6 +50,7 @@  discard block
 block discarded – undo
48 50
      * Sets the error handler
49 51
      *
50 52
      * @param WSErrorHandler Error handler
53
+     * @param WSCMSoapErrorHandler $handler
51 54
      */
52 55
     public static function setErrorHandler($handler)
53 56
     {
@@ -86,6 +89,7 @@  discard block
 block discarded – undo
86 89
 	 * Handle method
87 90
 	 *
88 91
 	 * @param WSError Error
92
+	 * @return void
89 93
 	 */
90 94
 	public function handle($error);
91 95
 }
@@ -114,7 +118,7 @@  discard block
 block discarded – undo
114 118
 	 * Verifies the API key
115 119
 	 *
116 120
 	 * @param string Secret key
117
-	 * @return mixed WSError in case of failure, null in case of success
121
+	 * @return WSCMError|null WSError in case of failure, null in case of success
118 122
 	 */
119 123
 	protected function verifyKey($secret_key)
120 124
     {
Please login to merge, or discard this patch.