Minor - format code

main/admin/skill_list.php

@@ -56,6 +56,7 @@ switch ($action) {
         exit;
         break;
     case 'disable':
+        /** @var \Chamilo\CoreBundle\Entity\Skill $skill */
         $skill = $entityManager->find('ChamiloCoreBundle:Skill', $skillId);
 
         if (is_null($skill)) {
@@ -77,12 +78,12 @@ switch ($action) {
             $entityManager->persist($skill);
 
             $skillObj = new Skill();
-            $childrens = $skillObj->getChildren($skill->getId());
+            $children = $skillObj->getChildren($skill->getId());
 
-            foreach ($childrens as $children) {
+            foreach ($children as $child) {
                 $skill = $entityManager->find(
                     'ChamiloCoreBundle:Skill',
-                    $children['id']
+                    $child['id']
                 );
 
                 if (empty($skill)) {
@@ -91,7 +92,6 @@ switch ($action) {
 
                 $skill->setStatus(0);
                 $skill->setUpdatedAt($updatedAt);
-
                 $entityManager->persist($skill);
             }
 
@@ -109,7 +109,7 @@ switch ($action) {
         exit;
         break;
     case 'list':
-        //no break
+        // no break
     default:
         $interbreadcrumb[] = array("url" => 'index.php', "name" => get_lang('PlatformAdmin'));
 
@@ -118,40 +118,44 @@ switch ($action) {
                 'add.png',
                 get_lang('CreateSkill'),
                 null,
-                ICON_SIZE_MEDIUM),
+                ICON_SIZE_MEDIUM
+            ),
             api_get_path(WEB_CODE_PATH).'admin/skill_create.php',
             ['title' => get_lang('CreateSkill')]
-            );
+        );
 
         $toolbar .= Display::url(
             Display::return_icon(
                 'wheel_skill.png',
                 get_lang('SkillsWheel'),
                 null,
-                ICON_SIZE_MEDIUM),
+                ICON_SIZE_MEDIUM
+            ),
             api_get_path(WEB_CODE_PATH).'admin/skills_wheel.php',
             ['title' => get_lang('SkillsWheel')]
-            );
+        );
 
         $toolbar .= Display::url(
             Display::return_icon(
                 'edit-skill.png',
                 get_lang('BadgesManagement'),
                 null,
-                ICON_SIZE_MEDIUM),
+                ICON_SIZE_MEDIUM
+            ),
             api_get_path(WEB_CODE_PATH).'admin/skill_badge_list.php',
             ['title' => get_lang('BadgesManagement')]
-            );
+        );
 
         $toolbar .= Display::url(
             Display::return_icon(
                 'import_csv.png',
                 get_lang('ImportSkillsListCSV'),
                 null,
-                ICON_SIZE_MEDIUM),
+                ICON_SIZE_MEDIUM
+            ),
             api_get_path(WEB_CODE_PATH).'admin/skills_import.php',
             ['title' => get_lang('ImportSkillsListCSV')]
-            );
+        );
 
         $extraField = new ExtraField('skill');
         $arrayVals = $extraField->get_handler_field_info_by_tags('tags');

main/lp/learnpath.class.php

@@ -4564,12 +4564,9 @@ class learnpath
     public function save_current()
     {
         $debug = $this->debug;
-        if ($debug > 0) {
-            error_log('learnpath::save_current()', 0);
-        }
         // TODO: Do a better check on the index pointing to the right item (it is supposed to be working
         // on $ordered_items[] but not sure it's always safe to use with $items[]).
-        if ($debug > 2) {
+        if ($debug) {
             error_log('New LP - save_current() saving item '.$this->current, 0);
             error_log(''.print_r($this->items, true), 0);
         }
@@ -4674,7 +4671,7 @@ class learnpath
                     WHERE
                         c_id = $course_id AND
                         lp_id = ".$this->get_id()." AND
-                        user_id = " . $this->get_user_id()." ".$session_condition;
+                        user_id = ".$this->get_user_id()." ".$session_condition;
 
             if ($this->debug > 2) {
                 error_log('New LP - Saving last item seen : '.$sql, 0);

main/lp/lp_controller.php

@@ -390,7 +390,7 @@ switch ($action) {
             if (isset($_POST['submit_button']) && !empty($post_title)) {
                 // If a title was submitted:
 
-                //Updating the lp.modified_on
+                // Updating the lp.modified_on
                 $_SESSION['oLP']->set_modified_on();
 
                 if (isset($_SESSION['post_time']) && $_SESSION['post_time'] == $_POST['post_time']) {