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 */

Spacing +3 added lines, -3 removed lines

@@ -45,12 +45,12 @@
 
     protected function activateByRoute($route, $items) {
 
-        foreach($items as $item) { /** @var $item MenuItemModel */
-            if($item->hasChildren()) {
+        foreach ($items as $item) { /** @var $item MenuItemModel */
+            if ($item->hasChildren()) {
                 $this->activateByRoute($route, $item->getChildren());
             }
             else {
-                if($item->getRoute() == $route) {
+                if ($item->getRoute() == $route) {
                     $item->setIsActive(true);
                 }
             }

Braces +1 added lines, -2 removed lines

@@ -250,8 +250,7 @@
         case RESULT_DISABLE_SHOW_SCORE_ONLY:
             if ($objExercise->feedback_type != EXERCISE_FEEDBACK_TYPE_END) {
                $header_names = array(get_lang('Attempt'), get_lang('StartDate'), get_lang('IP'), get_lang('Score'));
-            }
-            else {
+            } else {
                 $header_names = array(get_lang('Attempt'), get_lang('StartDate'), get_lang('IP'), get_lang('Score'), get_lang('Details'));
             }
             break;

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)

Spacing +2 added lines, -2 removed lines

@@ -51,11 +51,11 @@
      * @param null          $sentAt
      * @param UserInterface $to
      */
-    function __construct(UserInterface $from = null, $subject= '', $sentAt = null, UserInterface $to = null)
+    function __construct(UserInterface $from = null, $subject = '', $sentAt = null, UserInterface $to = null)
     {
         $this->to      = $to;
         $this->subject = $subject;
-        $this->sentAt  = $sentAt ? : new \DateTime();
+        $this->sentAt  = $sentAt ?: new \DateTime();
         $this->from    = $from;
     }
 

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()
     {

Spacing +3 added lines, -3 removed lines

@@ -19,15 +19,15 @@
     /**
      *
      */
-    const COLOR_AQUA   = 'aqua';
+    const COLOR_AQUA = 'aqua';
     /**
      *
      */
-    const COLOR_GREEN  = 'green';
+    const COLOR_GREEN = 'green';
     /**
      *
      */
-    const COLOR_RED    = 'red';
+    const COLOR_RED = 'red';
     /**
      *
      */

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(

Spacing +6 added lines, -6 removed lines

@@ -36,7 +36,7 @@ 
     function __construct($container, $resolverClass = null)
     {
         $this->container     = $container;
-        $this->resolverClass = $resolverClass?: 'Chamilo\ThemeBundle\Util\DependencyResolver';
+        $this->resolverClass = $resolverClass ?: 'Chamilo\ThemeBundle\Util\DependencyResolver';
     }
 
     public function registerScript($id, $src, $deps = array(), $location = "bottom")
@@ -53,7 +53,7 @@ 
     }
 
     public function registerStyle($id, $src, $deps = array()) {
-        if(!isset($this->stylesheets[$id])) {
+        if (!isset($this->stylesheets[$id])) {
             $this->stylesheets[$id] = array(
                 'src'      => $src,
                 'deps'     => $deps,
@@ -64,14 +64,14 @@ 
     public function getScripts($location = 'bottom') {
 
         $unsorted = array(); $srcList = array(); $assetList = array();
-        foreach($this->javascripts as $id => $scriptDefinition) {
-            if($scriptDefinition['location'] == $location) {
+        foreach ($this->javascripts as $id => $scriptDefinition) {
+            if ($scriptDefinition['location'] == $location) {
                 $unsorted[$id] = $scriptDefinition;
             }
         }
 
         $queue = $this->getResolver()->register($unsorted)->resolveAll();
-        foreach($queue as $def){
+        foreach ($queue as $def) {
             $srcList[] = $def['src'];
         }
         return $srcList;
@@ -80,7 +80,7 @@ 
     public function getStyles() {
         $srcList = array();
         $queue = $this->getResolver()->register($this->stylesheets)->resolveAll();
-        foreach($queue as $def){
+        foreach ($queue as $def) {
             $srcList[] = $def['src'];
         }
         return $srcList;

src/Chamilo/ThemeBundle/Form/FormDemoModelType.php

Spacing +2 added lines, -2 removed lines

@@ -11,7 +11,7 @@ 
 use Symfony\Component\Form\FormBuilderInterface;
 use Symfony\Component\OptionsResolver\OptionsResolverInterface;
 
-class FormDemoModelType extends AbstractType{
+class FormDemoModelType extends AbstractType {
 
     public function buildForm(FormBuilderInterface $builder, array $options)
     {
@@ -33,7 +33,7 @@ 
                 ->add('someChoices', 'choice', array('choices' => $choices, 'expanded' => true, 'multiple' => true))
                 ->add('username')
                 ->add('email')
-                ->add('termsAccepted','checkbox')
+                ->add('termsAccepted', 'checkbox')
                 ->add('message', 'textarea')
                 ->add('price')
                 ->add('date', 'date', array('widget' => 'single_text'))