chamilo / chamilo-lms

main/inc/lib/search/IndexableChunk.class.php

Doc Comments +2 added lines

@@ -57,6 +57,7 @@ 
      * Add a value to the indexed item
      * @param  string  Key
      * @param  string  Value
+     * @param string $key
      * @return  void
      */
     function addValue($key, $value) {
@@ -109,6 +110,7 @@ 
 
     /**
      * Let add tool id term
+     * @param string $tool_id
      */
     public function addToolId($tool_id)
     {

src/Chamilo/CoreBundle/Entity/Course.php

Doc Comments +3 added lines, -3 removed lines

@@ -392,7 +392,7 @@ 
     }
 
     /**
-     * @return ArrayCollection
+     * @return CTool[]
      */
     public function getTools()
     {
@@ -427,7 +427,7 @@ 
     }
 
     /**
-     * @param $urls
+     * @param ArrayCollection $urls
      */
     public function setUrls($urls)
     {
@@ -1244,7 +1244,7 @@ 
     }
 
     /**
-     * @return array
+     * @return string[]
      */
     public static function getStatusList()
     {

src/Chamilo/ThemeBundle/EventListener/SidebarSetupMenuDemoListener.php

Doc Comments +3 added lines

@@ -43,6 +43,9 @@
 
     }
 
+    /**
+     * @param MenuItemModel[] $items
+     */
     protected function activateByRoute($route, $items) {
 
         foreach($items as $item) { /** @var $item MenuItemModel */

src/Chamilo/ThemeBundle/Model/MenuItemModel.php

Doc Comments +5 added lines

@@ -60,6 +60,11 @@
      */
     protected $parent = null;
 
+    /**
+     * @param string $id
+     * @param string $label
+     * @param string $route
+     */
     function __construct(
         $id,
         $label,

src/Chamilo/ThemeBundle/Model/MessageModel.php

Doc Comments +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
      *
      * @param UserInterface $from
      * @param string        $subject
-     * @param null          $sentAt
+     * @param \DateTime          $sentAt
      * @param UserInterface $to
      */
     function __construct(UserInterface $from = null, $subject= '', $sentAt = null, UserInterface $to = null)

src/Chamilo/ThemeBundle/Model/NotificationModel.php

Doc Comments +3 added lines

@@ -18,6 +18,9 @@
 
     protected $icon;
 
+    /**
+     * @param string $message
+     */
     function __construct($message = null, $type = 'info', $icon = 'fa fa-warning')
     {
         $this->message = $message;

src/Chamilo/ThemeBundle/Model/TaskModel.php

Doc Comments +2 added lines, -2 removed lines

@@ -50,7 +50,7 @@ 
     protected $title;
 
     /**
-     * @param null   $title
+     * @param string   $title
      * @param int    $progress
      * @param string $color
      */
@@ -93,7 +93,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return integer
      */
     public function getProgress()
     {

src/Chamilo/ThemeBundle/Theme/ThemeManager.php

Doc Comments +4 added lines

@@ -52,6 +52,10 @@
 
     }
 
+    /**
+     * @param string $id
+     * @param string $src
+     */
     public function registerStyle($id, $src, $deps = array()) {
         if(!isset($this->stylesheets[$id])) {
             $this->stylesheets[$id] = array(

main/inc/lib/display.lib.php

Doc Comments +26 added lines, -17 removed lines

@@ -32,7 +32,7 @@ 
     }
 
     /**
-     * @return array
+     * @return string[]
      */
     public static function toolList()
     {
@@ -290,7 +290,6 @@ 
      * 					'page_nr' = The page to display
      * 					'hide_navigation' =  true to hide the navigation
      * @param array $query_vars Additional variables to add in the query-string
-     * @param array $form actions Additional variables to add in the query-string
      * @param mixed An array with bool values to know which columns show.
      * i.e: $visibility_options= array(true, false) we will only show the first column
      * 				Can be also only a bool value. TRUE: show all columns, FALSE: show nothing
@@ -331,7 +330,6 @@ 
      * 					'page_nr' = The page to display
      * 					'hide_navigation' =  true to hide the navigation
      * @param array $query_vars Additional variables to add in the query-string
-     * @param array $form actions Additional variables to add in the query-string
      * @param mixed An array with bool values to know which columns show. i.e:
      *  $visibility_options= array(true, false) we will only show the first column
      * 	Can be also only a bool value. TRUE: show all columns, FALSE: show nothing
@@ -388,10 +386,11 @@ 
      * 					'per_page' = number of items to show per page
      * 					'page_nr' = The page to display
      * @param array $query_vars Additional variables to add in the query-string
-     * @param array $column_show Array of binaries 1= show columns 0. hide a column
+     * @param integer[] $column_show Array of binaries 1= show columns 0. hide a column
      * @param array $column_order An array of integers that let us decide how the columns are going to be sort.
      * 						      i.e:  $column_order=array('1''4','3','4'); The 2nd column will be order like the 4th column
      * @param array $form_actions Set optional forms actions
+     * @param string $table_name
      *
      * @author Julio Montoya
      */
@@ -444,7 +443,7 @@ 
      * @param bool	$filter (true) or not (false)
      * @param bool $returnValue
      *
-     * @return void
+     * @return string|null
      */
     public static function display_normal_message($message, $filter = true, $returnValue = false)
     {
@@ -474,7 +473,7 @@ 
     /**
      * Displays an confirmation message. Use this if something has been done successfully
      * @param bool	Filter (true) or not (false)
-     * @return void
+     * @return string|null
      */
     public static function display_confirmation_message ($message, $filter = true, $returnValue = false)
     {
@@ -491,7 +490,7 @@ 
      * @param string $message - include any additional html
      *                          tags if you need them
      * @param bool	Filter (true) or not (false)
-     * @return void
+     * @return string|null
      */
     public static function display_error_message ($message, $filter = true, $returnValue = false)
     {
@@ -681,6 +680,7 @@ 
      * @param string   The alt text (probably a language variable)
      * @param array    additional attributes (for instance height, width, onclick, ...)
      * @param integer  The wanted width of the icon (to be looked for in the corresponding img/icons/ folder)
+     * @param string $image
      * @return void
     */
     public static function display_icon(
@@ -886,6 +886,7 @@ 
     /**
      * Displays an HTML input tag
      *
+     * @param string $type
      */
     public static function input($type, $name, $value, $attributes = array())
     {
@@ -902,8 +903,8 @@ 
     }
 
     /**
-     * @param $name
-     * @param $value
+     * @param string $name
+     * @param string $value
      * @param array $attributes
      * @return string
      */
@@ -918,6 +919,7 @@ 
     /**
      * Displays an HTML select tag
      *
+     * @param string $blank_item_text
      */
     public static function select(
         $name,
@@ -984,8 +986,8 @@ 
                         $( "#tabs" ).tabs();
                     });
                 </script>
-     * @param   array   $headers list of the tab titles
-     * @param   array   $items
+     * @param   string[]   $headers list of the tab titles
+     * @param   string[]   $items
      * @param   string  $id id of the container of the tab in the example "tabs"
      * @param   array   $attributes for the ul
      * @param array $ul_attributes
@@ -1039,7 +1041,7 @@ 
 
     /**
      * @param $headers
-     * @param null $selected
+     * @param integer $selected
      *
      * @return string
      */
@@ -1077,6 +1079,7 @@ 
      * As you can see both function use the same "my_grid_name" this is very important otherwise nothing will work
      *
      * @param   string  the div id, this value must be the same with the first parameter of Display::grid_js()
+     * @param string $div_id
      * @return  string  html
      *
      */
@@ -1632,6 +1635,8 @@ 
      * @param  string  url that will be added (for jquery see hot_courses.tpl)
 	 * @param	string	point info array see function CourseManager::get_course_ranking()
 	 * @param	bool	add a div wrapper
+	 * @param string $id
+	 * @param string $url
 	 * @todo	use     templates
      **/
     public static function return_rating_system($id, $url, $point_info = array(), $add_div_wrapper = true)
@@ -1727,6 +1732,9 @@ 
         return '<'.$size.'>'.Security::remove_XSS($title).'</'.$size.'>';
     }
 
+    /**
+     * @param string $title
+     */
     public static function page_subheader2($title, $second_title = null)
     {
         return self::page_header($title, $second_title, 'h4');
@@ -1957,6 +1965,7 @@ 
 
     /**
      * @todo use twig
+     * @param string $title
      */
     public static function group_button($title, $elements)
     {
@@ -2174,9 +2183,9 @@ 
 
     /**
      * @param int $id
-     * @param array $content
+     * @param string[] $content
      * @param int $col
-     * @param bool|true $right
+     * @param boolean $right
      * @return string
      */
     public static function toolbarAction($id, $content = array(), $col = 2, $right = true)
@@ -2258,10 +2267,10 @@ 
     /**
      * @param string $title
      * @param string $content
-     * @param null $id
+     * @param string $id
      * @param array $params
-     * @param null $idAccordion
-     * @param null $idCollapse
+     * @param string $idAccordion
+     * @param string $idCollapse
      * @param bool|true $open
      * @param bool|false $fullClickable
      * @return null|string