Completed
Pull Request — 1.11.x (#1223)
by José
199:23 queued 154:45
created
src/Chamilo/CourseBundle/Component/CourseCopy/CourseRecycler.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -38,7 +38,7 @@
 block discarded – undo
38 38
     }
39 39
 
40 40
     /**
41
-     * @return bool
41
+     * @return false|null
42 42
      */
43 43
     public function create_user_folder()
44 44
     {
Please login to merge, or discard this patch.
src/Chamilo/CourseBundle/Component/CourseCopy/CourseRestorer.php 1 patch
Doc Comments   +1 added lines patch added patch discarded remove patch
@@ -1212,6 +1212,7 @@
 block discarded – undo
1212 1212
 	 * Restore a forum-post
1213 1213
 	 * @TODO Restore tree-structure of posts. For example: attachments to posts.
1214 1214
 	 * @param false|string $topic_id
1215
+	 * @param false|string $forum_id
1215 1216
 	 */
1216 1217
     public function restore_post($id, $topic_id, $forum_id, $sessionId = 0)
1217 1218
     {
Please login to merge, or discard this patch.
Chamilo/CourseBundle/Component/CourseCopy/Resources/DummyCourseCreator.php 1 patch
Doc Comments   +1 added lines patch added patch discarded remove patch
@@ -25,6 +25,7 @@
 block discarded – undo
25 25
 
26 26
     /**
27 27
      * Useful finder - experimental akelos like only use in notification.lib.php send function
28
+     * @param string $type
28 29
      */
29 30
     public function find($type, $options = null)
30 31
     {
Please login to merge, or discard this patch.
plugin/buycourses/src/buy_course_plugin.class.php 1 patch
Doc Comments   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -36,7 +36,7 @@  discard block
 block discarded – undo
36 36
 
37 37
     /**
38 38
      *
39
-     * @return StaticPlugin
39
+     * @return BuyCoursesPlugin
40 40
      */
41 41
     static function create()
42 42
     {
@@ -252,7 +252,7 @@  discard block
 block discarded – undo
252 252
     /**
253 253
      * Save a transfer account information
254 254
      * @param array $params The transfer account
255
-     * @return int Rows affected. Otherwise return false
255
+     * @return false|string Rows affected. Otherwise return false
256 256
      */
257 257
     public function saveTransferAccount($params)
258 258
     {
@@ -664,7 +664,7 @@  discard block
 block discarded – undo
664 664
     /**
665 665
      * Get session info
666 666
      * @param array $sessionId The session ID
667
-     * @return array
667
+     * @return Session
668 668
      */
669 669
     public function getSessionInfo($sessionId)
670 670
     {
@@ -757,7 +757,7 @@  discard block
 block discarded – undo
757 757
      * Register a sale
758 758
      * @param int $itemId The product ID
759 759
      * @param int $paymentType The payment type
760
-     * @return boolean
760
+     * @return false|string
761 761
      */
762 762
     public function registerSale($itemId, $paymentType)
763 763
     {
@@ -939,7 +939,7 @@  discard block
 block discarded – undo
939 939
 
940 940
     /**
941 941
      * Get payment types
942
-     * @return array
942
+     * @return string[]
943 943
      */
944 944
     public function getPaymentTypes()
945 945
     {
@@ -977,7 +977,7 @@  discard block
 block discarded – undo
977 977
 
978 978
     /**
979 979
      * Get the statuses for sales
980
-     * @return array
980
+     * @return string[]
981 981
      */
982 982
     public function getSaleStatuses()
983 983
     {
@@ -990,7 +990,7 @@  discard block
 block discarded – undo
990 990
 
991 991
     /**
992 992
      * Get the statuses for Payouts
993
-     * @return array
993
+     * @return string[]
994 994
      */
995 995
     public function getPayoutStatuses()
996 996
     {
@@ -1003,7 +1003,7 @@  discard block
 block discarded – undo
1003 1003
 
1004 1004
     /**
1005 1005
      * Get the list of product types
1006
-     * @return array
1006
+     * @return string[]
1007 1007
      */
1008 1008
     public function getProductTypes()
1009 1009
     {
@@ -1384,7 +1384,7 @@  discard block
 block discarded – undo
1384 1384
     /**
1385 1385
      * Register a item
1386 1386
      * @param array $itemData The item data
1387
-     * @return int The item ID. Otherwise return false
1387
+     * @return false|string The item ID. Otherwise return false
1388 1388
      */
1389 1389
     public function registerItem(array $itemData)
1390 1390
     {
@@ -1542,7 +1542,7 @@  discard block
 block discarded – undo
1542 1542
     /**
1543 1543
      * Verify if the beneficiary have a paypal account
1544 1544
      * @param int $userId
1545
-     * @return true if the user have a paypal account, false if not
1545
+     * @return boolean if the user have a paypal account, false if not
1546 1546
      */
1547 1547
     public function verifyPaypalAccountByBeneficiary($userId)
1548 1548
     {
Please login to merge, or discard this patch.
main/newscorm/learnpath.class.php 1 patch
Doc Comments   +28 added lines, -25 removed lines patch added patch discarded remove patch
@@ -84,7 +84,7 @@  discard block
 block discarded – undo
84 84
      * @param	string	$course Course code
85 85
      * @param	integer	$lp_id
86 86
      * @param	integer	$user_id
87
-     * @return mixed True on success, false on error
87
+     * @return boolean True on success, false on error
88 88
      */
89 89
     public function __construct($course, $lp_id, $user_id)
90 90
     {
@@ -486,7 +486,7 @@  discard block
 block discarded – undo
486 486
      * @param int $max_time_allowed
487 487
      * @param int $userId
488 488
      *
489
-     * @return int
489
+     * @return false|string
490 490
      */
491 491
     public function add_item(
492 492
         $parent,
@@ -1037,7 +1037,7 @@  discard block
 block discarded – undo
1037 1037
      * @param	array $courseInfo
1038 1038
      * @param	integer	Learnpath ID
1039 1039
      * @param	string	Whether to delete data or keep it (default: 'keep', others: 'remove')
1040
-     * @return	boolean	True on success, false on failure (might change that to return number of elements deleted)
1040
+     * @return	false|null	True on success, false on failure (might change that to return number of elements deleted)
1041 1041
      */
1042 1042
     public function delete($courseInfo = null, $id = null, $delete = 'keep')
1043 1043
     {
@@ -1171,7 +1171,7 @@  discard block
 block discarded – undo
1171 1171
      * @param	integer	$id Elem ID (0 if first)
1172 1172
      * @param	integer	$remove Whether to remove the resource/data from the
1173 1173
      * system or leave it (default: 'keep', others 'remove')
1174
-     * @return	integer	Number of elements moved
1174
+     * @return	false|null	Number of elements moved
1175 1175
      * @todo implement resource removal
1176 1176
      */
1177 1177
     public function delete_item($id, $remove = 'keep')
@@ -1253,7 +1253,7 @@  discard block
 block discarded – undo
1253 1253
      * @param   array   $audio The array resulting of the $_FILES[mp3] element
1254 1254
      * @param   int     $max_time_allowed
1255 1255
      * @param   string  $url
1256
-     * @return  boolean True on success, false on error
1256
+     * @return  false|null True on success, false on error
1257 1257
      */
1258 1258
     public function edit_item(
1259 1259
         $id,
@@ -1773,7 +1773,7 @@  discard block
 block discarded – undo
1773 1773
 
1774 1774
     /**
1775 1775
      * Gets the first element URL.
1776
-     * @return	string	URL to load into the viewer
1776
+     * @return	false|null	URL to load into the viewer
1777 1777
      */
1778 1778
     public function first()
1779 1779
     {
@@ -2289,6 +2289,7 @@  discard block
 block discarded – undo
2289 2289
      * @param int $student_id
2290 2290
      * @param string Course code (optional)
2291 2291
      * @param int $sessionId
2292
+     * @param string $courseCode
2292 2293
      * @return	bool
2293 2294
      */
2294 2295
     public static function is_lp_visible_for_student(
@@ -2586,7 +2587,7 @@  discard block
 block discarded – undo
2586 2587
 
2587 2588
     /**
2588 2589
      * Gets the learnpath session id
2589
-     * @return	string	Learnpath theme
2590
+     * @return	integer	Learnpath theme
2590 2591
      */
2591 2592
     public function get_lp_session_id()
2592 2593
     {
@@ -2619,7 +2620,7 @@  discard block
 block discarded – undo
2619 2620
     /**
2620 2621
      * @param string $size
2621 2622
      * @param string $path_type
2622
-     * @return bool|string
2623
+     * @return string|false
2623 2624
      */
2624 2625
     public function get_preview_image_path($size = null, $path_type = 'web')
2625 2626
     {
@@ -2765,6 +2766,8 @@  discard block
 block discarded – undo
2765 2766
      * Returns the XML DOM document's node
2766 2767
      * @param	resource	Reference to a list of objects to search for the given ITEM_*
2767 2768
      * @param	string		The identifier to look for
2769
+     * @param DOMNodeList $children
2770
+     * @param string $id
2768 2771
      * @return	mixed		The reference to the element found with that identifier. False if not found
2769 2772
      */
2770 2773
     public function get_scorm_xml_node(& $children, $id)
@@ -3012,7 +3015,7 @@  discard block
 block discarded – undo
3012 3015
     /**
3013 3016
      * Generate and return the table of contents for this learnpath. The (flat) table returned can be
3014 3017
      * used by get_html_toc() to be ready to display
3015
-     * @return	array	TOC as a table with 4 elements per row: title, link, status and level
3018
+     * @return	boolean	TOC as a table with 4 elements per row: title, link, status and level
3016 3019
      */
3017 3020
     public function get_toc()
3018 3021
     {
@@ -3148,7 +3151,7 @@  discard block
 block discarded – undo
3148 3151
     }
3149 3152
 
3150 3153
     /**
3151
-     * @return array
3154
+     * @return string[]
3152 3155
      */
3153 3156
     public static function getChapterTypes()
3154 3157
     {
@@ -4057,7 +4060,7 @@  discard block
 block discarded – undo
4057 4060
      * Open a resource = initialise all local variables relative to this resource. Depending on the child
4058 4061
      * class, this might be redefined to allow several behaviours depending on the document type.
4059 4062
      * @param integer Resource ID
4060
-     * @return boolean True on success, false otherwise
4063
+     * @return boolean|null True on success, false otherwise
4061 4064
      */
4062 4065
     public function open($id)
4063 4066
     {
@@ -4186,7 +4189,7 @@  discard block
 block discarded – undo
4186 4189
      * Can be used as abstract
4187 4190
      * @param	integer	$lp_id Learnpath id
4188 4191
      * @param	string	$set_visibility New visibility (v/i - visible/invisible)
4189
-     * @return bool
4192
+     * @return false|null
4190 4193
      */
4191 4194
     public static function toggle_publish($lp_id, $set_visibility = 'v')
4192 4195
     {
@@ -4271,7 +4274,7 @@  discard block
 block discarded – undo
4271 4274
      * Make sure the results are saved with anoter method. This method should probably be
4272 4275
      * redefined in children classes.
4273 4276
      * To use a similar method  statically, use the create_new_attempt() method
4274
-     * @return string URL to load in the viewer
4277
+     * @return boolean URL to load in the viewer
4275 4278
      */
4276 4279
     public function restart()
4277 4280
     {
@@ -4349,6 +4352,7 @@  discard block
 block discarded – undo
4349 4352
      * Saves the given item
4350 4353
      * @param	integer	$item_id. Optional (will take from $_REQUEST if null)
4351 4354
      * @param	boolean	$from_outside Save from url params (true) or from current attributes (false). Optional. Defaults to true
4355
+     * @param integer $item_id
4352 4356
      * @return	boolean
4353 4357
      */
4354 4358
     public function save_item($item_id = null, $from_outside = true)
@@ -4846,7 +4850,7 @@  discard block
 block discarded – undo
4846 4850
 
4847 4851
     /**
4848 4852
      * Sets use_max_score
4849
-     * @param   string  $use_max_score Optional string giving the new location of this learnpath
4853
+     * @param   integer  $use_max_score Optional string giving the new location of this learnpath
4850 4854
      * @return  boolean True on success / False on error
4851 4855
      */
4852 4856
     public function set_use_max_score($use_max_score = 1)
@@ -5249,6 +5253,7 @@  discard block
 block discarded – undo
5249 5253
      * Register the attempt mode into db thanks to flags prevent_reinit and seriousgame_mode flags
5250 5254
      *
5251 5255
      * @param string 'seriousgame', 'single' or 'multiple'
5256
+     * @param string $mode
5252 5257
      * @return boolean
5253 5258
      * @author ndiechburg <[email protected]>
5254 5259
      **/
@@ -5291,7 +5296,7 @@  discard block
 block discarded – undo
5291 5296
     /**
5292 5297
      * Switch between multiple attempt, single attempt or serious_game mode (only for scorm)
5293 5298
      *
5294
-     * @return boolean
5299
+     * @return boolean|null
5295 5300
      * @author ndiechburg <[email protected]>
5296 5301
      **/
5297 5302
     public function switch_attempt_mode()
@@ -5482,7 +5487,6 @@  discard block
 block discarded – undo
5482 5487
     /**
5483 5488
      * Function that creates a html list of learning path items so that we can add audio files to them
5484 5489
      * @author Kevin Van Den Haute
5485
-     * @param int $lp_id
5486 5490
      * @return string
5487 5491
      */
5488 5492
     public function overview()
@@ -5902,7 +5906,7 @@  discard block
 block discarded – undo
5902 5906
     /**
5903 5907
      * This function builds the action menu
5904 5908
      * @param bool $returnContent
5905
-     * @return void
5909
+     * @return string|null
5906 5910
      */
5907 5911
     public function build_action_menu($returnContent = false)
5908 5912
     {
@@ -6447,7 +6451,7 @@  discard block
 block discarded – undo
6447 6451
     /**
6448 6452
      * Function that displays a list with al the resources that
6449 6453
      * could be added to the learning path
6450
-     * @return string
6454
+     * @return boolean
6451 6455
      */
6452 6456
     public function display_resources()
6453 6457
     {
@@ -10028,7 +10032,7 @@  discard block
 block discarded – undo
10028 10032
 
10029 10033
     /**
10030 10034
      * @param int $lp_id
10031
-     * @return bool
10035
+     * @return false|null
10032 10036
      */
10033 10037
     public function scorm_export_to_pdf($lp_id)
10034 10038
     {
@@ -10569,7 +10573,7 @@  discard block
 block discarded – undo
10569 10573
 
10570 10574
     /**
10571 10575
      * @param int $itemId
10572
-     * @return learnpathItem|false
10576
+     * @return string
10573 10577
      */
10574 10578
     public function getItem($itemId)
10575 10579
     {
@@ -10619,7 +10623,6 @@  discard block
 block discarded – undo
10619 10623
     /**
10620 10624
      * Set whether this is a learning path with the possibility to subscribe
10621 10625
      * users or not
10622
-     * @param int $subscribeUsers (0 = false, 1 = true)
10623 10626
      */
10624 10627
     public function setSubscribeUsers($value)
10625 10628
     {
@@ -10744,7 +10747,7 @@  discard block
 block discarded – undo
10744 10747
 
10745 10748
     /**
10746 10749
      * Get the item of exercise type (evaluation type)
10747
-     * @return array The final evaluation. Otherwise return false
10750
+     * @return integer The final evaluation. Otherwise return false
10748 10751
      */
10749 10752
     public function getFinalEvaluationItem()
10750 10753
     {
@@ -10968,7 +10971,7 @@  discard block
 block discarded – undo
10968 10971
     /**
10969 10972
      * Create a forum for this learning path
10970 10973
      * @param type $forumCategoryId
10971
-     * @return int The forum ID if was created. Otherwise return false
10974
+     * @return false|string The forum ID if was created. Otherwise return false
10972 10975
      */
10973 10976
     public function createForum($forumCategoryId)
10974 10977
     {
@@ -11017,7 +11020,7 @@  discard block
 block discarded – undo
11017 11020
     /**
11018 11021
      * Get the LP Final Item Template
11019 11022
      *
11020
-     * @return html
11023
+     * @return string
11021 11024
      */
11022 11025
     private function getFinalItemTemplate()
11023 11026
     {
@@ -11040,7 +11043,7 @@  discard block
 block discarded – undo
11040 11043
     /**
11041 11044
      * Get the LP Final Item form
11042 11045
      *
11043
-     * @return html
11046
+     * @return string
11044 11047
      */
11045 11048
     public function getFinalItemForm()
11046 11049
     {
Please login to merge, or discard this patch.
main/exercice/question.class.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -264,7 +264,7 @@
 block discarded – undo
264 264
     }
265 265
 
266 266
     /**
267
-     * @return string|false
267
+     * @return string
268 268
      */
269 269
     public function selectPicturePath()
270 270
     {
Please login to merge, or discard this patch.
src/Chamilo/CoreBundle/Entity/Language.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -230,7 +230,7 @@
 block discarded – undo
230 230
     /**
231 231
      * Get id
232 232
      *
233
-     * @return boolean
233
+     * @return integer
234 234
      */
235 235
     public function getId()
236 236
     {
Please login to merge, or discard this patch.
src/Chamilo/CourseBundle/Component/CourseCopy/CourseBuilder.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -91,7 +91,7 @@
 block discarded – undo
91 91
     }
92 92
 
93 93
     /**
94
-     * @param array $array
94
+     * @param string[] $array
95 95
      */
96 96
     public function set_tools_to_build($array)
97 97
     {
Please login to merge, or discard this patch.
main/forum/forumfunction.inc.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -3054,7 +3054,7 @@
 block discarded – undo
3054 3054
  * @param integer $thread_qualify
3055 3055
  * @param integer $qualify_time
3056 3056
  * @param integer $session_id
3057
- * @return array optional
3057
+ * @return string|null optional
3058 3058
  * @author Isaac Flores <[email protected]>, U.N.A.S University
3059 3059
  * @version October 2008, dokeos  1.8.6
3060 3060
  */
Please login to merge, or discard this patch.